菜单
Review the current branch against its merge base for bugs, CPython divergence, sandbox escapes, resource-limit escapes, performance regressions, verbose comments and missing ./limitations/ or docs/ updates. Use for a general pre-merge review of a branch or PR.
默认使用会先检查来源的 Prompt；你也可以切换为直接命令，或下载本地副本。
决定是否安装前，请先阅读 SKILL.md，以及 SkillsMP 当前展示的配套文件。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
直接命令不会经过审查 Prompt；运行前请先检查来源。
npx skills add https://github.com/pydantic/monty --skill review-general命令会保持在同一行。复制前请横向滚动并检查完整内容。
想先保存到本地？可下载 SkillsMP 当前能够提供的文件。
正在显示 SKILL.md
| name | review-general |
| description | Review the current branch against its merge base for bugs, CPython divergence, sandbox escapes, resource-limit escapes, performance regressions, verbose comments and missing ./limitations/ or docs/ updates. Use for a general pre-merge review of a branch or PR. |
git diff --stat origin/main...HEAD # scope first
git diff origin/main...HEAD
Read the changed files in full — a hunk is rarely enough to judge correctness. Look for:
DropWithContext values not released on every exit path (the
fix is defer_drop!/DropGuard, not more drop_with calls), borrow/aliasing
mistakes, unhandled error paths.python-playground.String build without StringBuilder), unbounded loops, recursion without
a depth guard.CLAUDE.md../limitations/ — a new divergence with no entry is a finding.README.md, docs/,
limitations/ and the crate READMEs is a finding. CLAUDE.md "Documentation surfaces
that must stay in sync" has the obligation table. For a full pass, delegate to the
docs-parity-reviewer subagent.Concise, most severe first. Per finding: file:line, what's wrong, the concrete failure
it causes. Don't pad with what the branch got right.
Report only, unless the user asks for fixes.
How to write prose that reads like human technical documentation rather than LLM output. Use whenever writing or editing docstrings, comments, limitations/ docs, READMEs, commit messages or PR descriptions, and when prose reads as smooth, salesy or generic.
Security review of the current branch against its merge base — sandbox escapes, memory errors, panics and resource-limit bypasses. Use when reviewing changes for security risk, or before merging anything touching heap.rs, path_security.rs, the wire protocol or the pool.
Read the review comments left by the known agent reviewers on the current PR, resolve and reply. Use when asked to deal with PR comments, review feedback, or bot review findings.
