security-review: missing reference files (infrastructure/*.md and languages/*.md) referenced in SKILL.md
- Dominant language
- Python
- Stars
- 993
- Forks
- 51
- Avg merge
- 7h 49m
- Merged PRs (30d)
- 2
Description
### Environment
Repo: getsentry/skills, skill: `skills/security-review`
Browsing at https://github.com/getsentry/skills/tree/main/skills/security-review
### Steps to Reproduce
1. Open `skills/security-review/SKILL.md`, section "Reference Files".
2. Compare the tables there against the actual contents of `skills/security-review/infrastructure/` and `skills/security-review/languages/`.
### Expected Result
Every file listed in SKILL.md's "Reference Files" tables should exist in the corresponding folder:
- `infrastructure/`: docker.md, kubernetes.md, terraform.md, ci-cd.md, cloud.md
- `languages/`: python.md, javascript.md, go.md, rust.md, java.md
### Actual Result
Several referenced files are missing:
- `infrastructure/` only contains `docker.md` — missing `kubernetes.md`, `terraform.md`, `ci-cd.md`, `cloud.md`
- `languages/` only contains `javascript.md` and `python.md` — missing `go.md`, `rust.md`, `java.md`
Since SKILL.md's "Load Language Guide" and "Load Infrastructure Guide" steps point agents at these paths, reviewing Go/Rust/Java code or Kubernetes/Terraform/CI-CD/cloud configs falls back to no guide being loaded.
(`references/` folder is complete and not affected.)
Contributor guide
Research direction
Start with skills/security-review/SKILL.md and compare its Reference Files tables with the infrastructure/ and languages/ directories. Review the existing docker.md, javascript.md, and python.md files to understand the guide format, then add the seven missing guides. Done means every listed path exists and the Load Language Guide and Load Infrastructure Guide steps no longer point to missing files.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, go, java, javascript, kubernetes, python, rust, terraform
- Domain
- devops, documentation
- Issue type
- Documentation
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100