1k-code-review-pr
Review OneKey PRs and diffs for security, correctness, concurrency, React/RN pitfalls, and repository-specific regressions. Use for code review or 审查 PR.
74
92%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
View guide
Critical
Do not install without reviewing
Security
2 findings: 1 critical severity, 1 high severity. Installing this skill is not recommended: please review these findings carefully if you do intend to do so.
E006: Malicious code pattern detected in skill scripts.
What this means
Detected high-risk code patterns in the skill content — including its prompts, tool definitions, and resources — such as data exfiltration, backdoors, remote code execution, credential theft, system compromise, supply chain attacks, and obfuscation techniques.
Why it was flagged
The SKILL.md workflow explicitly instructs dispatching the full PR diff to an external "codex:codex-rescue" subagent (including ${FULL_DIFF}), which can leak sensitive source/secrets to an external service — a high-risk data-exfiltration pattern.
W007: Insecure credential handling detected in skill instructions.
What this means
The skill handles credentials insecurely by requiring the agent to include secret values verbatim in its generated output. This exposes credentials in the agent’s context and conversation history, creating a risk of data exfiltration.
Why it was flagged
The skill instructs the agent to read and include full diffs, file contents, and auto-fix code snippets (and to embed ${FULL_DIFF} in a subagent prompt and post suggestion bodies), which will cause any secrets present in the repo or diff to be output verbatim, creating an exfiltration risk.
Low
Low-risk findings.
1 low severity finding. Worth noting, but not necessarily harmful.
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
Yes—at runtime the workflow uses GitHub CLI `gh api` to fetch PR reviews/comments (both bot and human) and then the agent analyzes that comment `body` text in the LLM context; this is outsider-authored free text from PR commenters.
- Repository
- OneKeyHQ/app-monorepo
- Commit
d71e6b7
- 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.