Getting it into your agent
One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.
npx skills add ryanzhao1011/workframe --skill code-reviewgit clone --depth 1 https://github.com/ryanzhao1011/workframeWrote this? Show the measurements
A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.
[](https://agentmods.dev/skills/ryanzhao1011/workframe/code-review)<a href="https://agentmods.dev/skills/ryanzhao1011/workframe/code-review"><img src="https://agentmods.dev/badge/skills/ryanzhao1011/workframe/code-review/github.svg" alt="Measured on agentmods" height="20"></a>Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.
<a href="https://agentmods.dev/skills/ryanzhao1011/workframe/code-review"><img src="https://agentmods.dev/badge/skills/ryanzhao1011/workframe/code-review.svg" alt="Reviewed on agentmods" width="80" height="20"></a>What it costs to keep this loaded
Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.
| Model | Per session | Once invoked |
|---|---|---|
| Fable 5.1 | $0.00264 | $0.04519 |
| Opus 5.5 | $0.00106 | $0.01808 |
| Sonnet 5.5 | $0.00053 | $0.00904 |
| Haiku 4.5 | $0.00026 | $0.00452 |
Grade A, and why
code-review scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 21d ago.
A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.
Nothing flagged
None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.
How it starts
The opening of the file, as written. The whole thing — 247 lines — stays where its author put it; the contents beside it link to each section on GitHub.
代码审查技能
产物去向
审查结论默认只在响应中呈现。用户要求存档时按 skill: document-norms §1 落
projects/modules/<basic>/<sub>/requirements/<req_slug>/<sub_req_slug>/reviews/；
发现的缺陷若需跟踪，走 projects/issues/ + board task 草稿（不直接写 board）。
适用场景
- @dev 交付代码变更后
- @qa 做回归验证的前置检查
- 安全/质量疑点时的深度审查
四步流程
第 1 步：变更范围识别
读取代码变更（diff 或指定文件），识别：
| 维度 | 内容 |
|---|---|
| 改动文件 | 新增/修改/删除的文件清单 |
| 改动范围 | 每个文件的具体改动行数和函数 |
| 影响模块 | 直接调用改动代码的其他模块 |
| 依赖变化 | 新增/移除的依赖关系 |
第 2 步：多维度审查
2.1 需求符合度（Requirement Conformance）
前提：先拿到规范资产（PRD 正文 / AC / spec / 词表 / 接口约定），逐条比对，不能只看 AC —— AC 是抽样，正文才是全量口径。
| 审查点 | 检查内容 |
|---|---|
| 规则落地 | 规范正文写明的每条规则，代码是否实现且口径一致 |
| 缺漏 | 规范写了、代码没做（含只做了一半、静默降级、开关/埋点等非功能要求） |
| 超纲 | 代码做了、规范没写（自行发挥的逻辑，需回溯确认） |
| 规范空白 | 规范本身没定，代码"忠实实现了不完整的规则" —— 不算实现错误，退回需求侧补口径 |
| 已符合项 | 明确列出实现正确的部分，供修其他问题时防误伤 |
「规范空白」与「缺漏」必须分开报：前者要 PM/需求方先拍板，研发改了也是白改；后者研发直接修。
2.2 正确性（Correctness）
| 审查点 | 检查内容 |
|---|---|
| 逻辑正确 | 代码实现是否符合需求/AC |
| 边界处理 | 空值、零值、极值、负数等边界条件 |
| 错误处理 | 异常捕获、错误返回、失败恢复 |
| 并发安全 | 竞态条件、死锁、共享状态 |
| 类型安全 | 类型断言、null 检查 |
2.3 安全性（Security）
| 审查点 | 检查内容 |
|---|---|
| 注入风险 | SQL 注入、命令注入、XSS |
| 权限校验 | 未登录访问、越权访问 |
| 敏感信息 | 密码/密钥泄露、日志打印敏感数据 |
| 输入校验 | 未校验的用户输入、文件上传 |
| 依赖安全 | 已知漏洞的依赖库 |
2.4 可维护性（Maintainability）
| 审查点 | 检查内容 |
|---|---|
| 命名清晰 | 变量、函数、类名能自解释 |
| 结构清晰 | 单一职责、合理抽象、避免过深嵌套 |
| 注释充分 | 复杂逻辑有解释，公开接口有文档 |
| 代码复用 | 避免重复代码，合理提取公共函数 |
| 测试覆盖 | 关键逻辑有测试 |
2.5 性能（Performance）
| 审查点 | 检查内容 |
|---|---|
| 算法复杂度 | O(n²) 或更差的循环 |
| 数据库查询 | N+1 查询、缺失索引、全表扫描 |
| 网络请求 | 串行请求应并行、缺失缓存 |
| 内存使用 | 大对象加载、内存泄漏 |
| IO 操作 | 同步阻塞 IO、文件句柄泄漏 |
审查自己参与过的改动时：一检（构建者正向）与二检（审计者反向）的完整判据见必载片 §验证纪律（必载，无需另找）。二检换视角不能替代独立审查——两者串联，本 skill 反模式末条说的就是这件事。
第 3 步：Finding 分级与归因
每个 Finding 打三个正交标签：风险级别（多急）+ 依据来源（凭什么）+ 处置方向（假绿还是假红，仅当被审对象自己在做判定时），再由这三者推出第四个字段：是否触发新一轮。
风险级别：
| 级别 | 含义 | 处理要求 |
|---|---|---|
| Critical | 必须修复 | 阻塞合并/发布，P0 问题 |
| Warning | 建议修复 | 下次迭代或同一 PR 修复 |
| Info | 建议改进 | 记录待办，不阻塞 |
What this file has done since we first saw it
Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.
- 21d ago Changed · +62 lines · +224 tokens per session e198d1f4fa68
- 1mo ago First seen · 185 lines · 40 tokens per session scan A c5cef397ceca
code-review is a skill published in the GitHub repository ryanzhao1011/workframe (7 stars, last pushed 23d ago), licensed MIT. It adds 264 tokens to every session and 4,519 once invoked, about $0.0011 per session on Opus 5.5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-31.
Other skills, from other repositories
migration-review
A review workflow for database migration files, which change the structure or data rules of a database. It checks the requested migration files and the code needed to assess their compatibility.
ocr
AI-powered multi-agent code review. Simulates a team of Principal Engineers reviewing code from different perspectives. Use when asked to review code, check a PR, analyze changes, or perform code review.
OCR Review-to-Approval Loop
Drive a PR to an approved code review by looping OCR's multi-agent review and address steps. Runs /ocr:review then /ocr:address repeatedly until the review verdict is APPROVE, then one final /ocr:address for leftover suggestions, posting every review and every address round to the GitHub PR as comments. Use when the…
pr-triage
4-phase PR backlog management with audit, deep code review, validated comments, and optional worktree setup. Use when triaging pull requests, catching up on pending code reviews, or managing a backlog of open PRs. Args: 'all' to review all, PR numbers to focus (e.g. '42 57'), 'en'/'fr' for language, no arg = audit…
audit-agents-skills
Audit Claude Code agents, skills, and commands for quality and production readiness. Use when evaluating skill quality, checking production readiness scores, or comparing agents against best-practice templates.
review-pr
Perform a comprehensive code review of a pull request.