gh-code-review
Review GitHub pull requests using gh CLI and the GitHub API. Use when asked to review a PR, inspect PR changes, or choose approve/comment/request-changes.
72
88%
Does it follow best practices?
Impact
—
No eval scenarios have been run
Advisory
Suggest reviewing before use
Security
2 findings — 2 medium severity. This skill can be installed but you should review these findings before use.
W011: Third-party content exposure detected (indirect prompt injection risk)
What this means
The skill exposes the agent to untrusted, user-generated content from public third-party sources, creating a risk of indirect prompt injection. This includes browsing arbitrary URLs, reading social media posts or forum comments, and analyzing content from unknown websites.
Why it was flagged
Third-party content exposure detected (high risk: 0.90). The skill explicitly fetches PR metadata, per-file patches, and raw file contents from GitHub (see scripts/collect_pr.sh calling gh api "repos/{owner}/{repo}/pulls/$PR_NUMBER/files" and scripts/fetch_raw_files.sh using the Contents API) and SKILL.md requires the agent to read and base review decisions on those user-generated PR contents, so untrusted third-party content can materially influence the agent's actions.
W012: Unverifiable external dependency detected (runtime URL that controls agent)
What this means
The skill fetches instructions or code from an external URL at runtime, and the fetched content directly controls the agent’s prompts or executes code. This dynamic dependency allows the external source to modify the agent’s behavior without any changes to the skill itself.
Why it was flagged
Potentially malicious external URL detected (high risk: 0.90). The skill calls GitHub API endpoints at runtime (e.g., repos/{owner}/{repo}/pulls/$PR_NUMBER/files and repos/{owner}/{repo}/contents/$api_path?ref=$headRefOid) via `gh api` to fetch patches/raw files which are injected into the agent's review context, so external repo content can directly control prompts.
- Repository
- bkircher/skills
- Commit
310ca43
- Audited
- Security analysis
Is this your skill?
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.