CtrlK
BlogDocsLog inGet started
Tessl Logo

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

Quality

93%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

High

Do not use without reviewing

SKILL.md
Quality
Evals
Security

Quality

Content

92%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A dense, highly actionable review skill with executable commands, a mandatory scoring rubric, and a complete report template, backed by well-organized one-level-deep references. The only minor inefficiency is some restatable emphasis and maxims that could be tightened.

DimensionReasoningScore

Conciseness

Mostly lean and assumes Claude's competence, with executable commands and templates instead of concept explanations; the 'Review Discipline' maxims and some repeated MANDATORY emphasis could be trimmed without losing clarity.

4 / 5

Actionability

Fully executable guidance throughout — copy-paste gh/git commands, exact report template, scoring rubric with deduction anchors, and a precise auto-fix diff format covering the common review cases.

5 / 5

Workflow Clarity

A 10-step sequenced workflow with explicit validation checkpoints (Codex availability check, PR comment collection, mandatory scoring step, merge/dedup) and a confirmation gate before posting PR comments — a destructive/outward-facing action — giving a clear feedback loop.

5 / 5

Progressive Disclosure

SKILL.md serves as an overview pointing to four real, one-level-deep reference files, each clearly signaled and summarized in a 'Reference Files' section, with a triage table mapping changed-file patterns to the right reference.

5 / 5

Total

19

/

20

Passed

Description

95%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong, well-scoped description with explicit 'what' and 'when' guidance plus bilingual natural trigger phrases. The only soft spot is that capabilities are framed as review categories rather than enumerated discrete actions.

DimensionReasoningScore

Specificity

Names the domain (OneKey monorepo PR review) and lists several concrete capability areas — security, code quality, OneKey-specific patterns — but describes review categories rather than discrete actions, leaving minor coverage gaps.

4 / 5

Completeness

Explicitly answers both 'what' (PR code review covering security/quality/patterns) and 'when' ("Use when reviewing PRs, code changes, or diffs" plus concrete trigger phrases and an always-use clause).

5 / 5

Trigger Term Quality

Comprehensive natural trigger phrases in both English and Chinese ("review PR", "code review", "check this diff", "审查 PR", "代码审查", "review #123") covering synonyms users would actually say.

5 / 5

Distinctiveness Conflict Risk

Scoped specifically to the OneKey monorepo with repo-specific patterns (Fabric crashes, MIUI, BigNumber, import hierarchy), giving it a clear niche with minimal conflict risk.

5 / 5

Total

19

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

Total

15

/

16

Passed

Repository
OneKeyHQ/app-monorepo
Reviewed

Table of Contents

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.