1k-code-review-pr
Comprehensive PR code review for OneKey monorepo. Use when reviewing PRs, code changes, or diffs — covers security (secrets/PII leakage, supply-chain, AuthN/AuthZ), code quality (hooks, race conditions, null safety, concurrent requests), and OneKey-specific patterns (Fabric crashes, MIUI, BigNumber). Triggers on "review PR", "review this PR", "code review", "check this diff", "审查 PR", "代码审查", "review
75
93%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
View guide
High
Do not use without reviewing
Security
1 high severity finding. You should review these findings carefully before considering using this skill.
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 file contents, include full diffs/old-code snippets in report auto-fix blocks, send ${FULL_DIFF} to a Codex subagent, and post suggestion comments — all of which would cause any secret literals in the changed files or diffs to be echoed verbatim in outputs.
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
In the required workflow, the agent uses the GitHub PR CLI/API to fetch and ingest all existing PR comments (including human-authored free text) at runtime via `gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews`, `.../pulls/{pr_number}/comments`, and `.../issues/{pr_number}/comments`, then analyzes the bodies with the LLM.
- Repository
- OneKeyHQ/app-monorepo
- Commit
3cff1b4
- 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.