How to review PRs for security — credentials, injection, workflow permissions, supply chain, git operation safety.
Collected from public GitHub. Every version is saved exactly as it was and never changes. Install it at an exact version with the mdr tool, and every file is checked before it reaches your agent. The main file is checked against the SHA-256 fingerprint recorded here, and the others against the fingerprints GitHub keeps for its source commit. The automatic checks below grade the latest version, and the same checks always give the same file the same grade.
What the file says
## Context
Every PR that touches authentication, credentials, environment variables, file system operations, child process execution, user input handling, GitHub API calls, or workflow files must be reviewed for security. This skill provides a systematic checklist for catching security issues before they reach production.
Use this skill when a PR includes any of:
- Changes to `.github/workflows/` files
- New or modified environment variable usage
- File system read/write operations driven by user input
- Child process execution (`child_process.exec`, `child_process.spawn`)
- GitHub API calls (Octokit, `gh` CLI wrappers)
- New npm dependencies
- Credential or token handling
- Template rendering with dynamic values
## Patterns
### 1. Secrets and Credentials
**No secrets in source code — ever.**
**Review checklist:**
- ❌ No API keys, tokens, passwords, or connection strings in committed files
- ❌ No hardcoded `GITHUB_TOKEN`, `NPM_TOKEN`, or similar values
- ❌ No `.env` files committed (verify `.gitignore` covers them)
- ✅ Secrets must be referenced via `process.env.X` or `${{ secrets.X }}` in workflows
…
The line at the top of this page pins the label git:20260403.003f05f, which is what most people want. This one pins the sha256 fingerprint of the contents instead, so it survives a label being moved. Either way the exact version is written to mdr.lock, and mdr install downloads and checks those contents again, so it installs them exactly or stops with an error.
A 17 of 17 checks passed. Fixed automatic checks with no AI involved, so the same checks give the same answer every run.
pass: Frontmatter block present
pass: Frontmatter declares a name
pass: Frontmatter declares a description
pass: Size between 200 bytes and 200 KB (11732 bytes)
pass: No zero-width or bidi control characters
pass: No instruction hidden inside an HTML comment
pass: No link to an exfiltration or paste host
pass: No credential-shaped string
pass: No instruction to send local credentials anywhere
pass: No text hidden with inline styles
pass: No prompt-injection phrasing
pass: No curl or wget piped into a shell
pass: No recursive delete of root, home or parent
pass: No instruction to read or print local credentials
pass: No base64 blob over 200 characters
pass: No link to a raw IP address
pass: No script tag
Source
GitHub
bradygaster/squad · 3,298 stars · license MIT · pushed 2026-10-22 · branch dev
API
GET https://markdownregistry.com/api/v1/artifacts/art_5uhean47ykmmiyba
GET https://markdownregistry.com/api/v1/resolve?ref=bradygaster/squad/security-review
GET https://markdownregistry.com/api/v1/blob/1bc789a46d65e1c1269d2617eebb73a4cd9dec8a3598a2141b061b8f502fac06
Your AI assistant does the legwork. You hear about the deals worth your word. Hand yours the standing instructions at modelranch.com and it joins the network that reads files like this one.
bradygaster/squad · .copilot/skills/action-skill-contracts/SKILL.md · Author deterministic skills for operations that change state or invoke external tools
bradygaster/squad · .copilot/skills/agent-collaboration/SKILL.md · Standard collaboration patterns for all squad agents — worktree awareness, decisions, cross-agent communication
bradygaster/squad · .copilot/skills/architectural-proposals/SKILL.md · How to write comprehensive architectural proposals that drive alignment before code is written
affaan-m/ecc · .agents/skills/security-review/SKILL.md · Use this skill when adding authentication, handling user input, working with secrets, creating API endpoints, or…
affaan-m/ecc · .kiro/skills/security-review/SKILL.md · Use this skill when adding authentication, handling user input, working with secrets, creating API endpoints, or…
affaan-m/ecc · docs/es/skills/security-review/SKILL.md · Usar este skill al agregar autenticación, manejar entradas de usuario, trabajar con secretos, crear endpoints de API o…
microsoft/power-platform-skills · plugins/power-pages/skills/security-review/SKILL.md · Runs a guided, end-to-end security review of a Power Pages site and consolidates every finding into one HTML report…
kklimuk/docx-cli · .claude/skills/security-review/SKILL.md · Review code for security vulnerabilities. Use when the user says 'security review', 'security audit', 'check for…
