Agent Skills
› pydantic/monty
› review-general
review-general
GitHub用于在合并前对代码分支进行全面审查,检查逻辑错误、沙箱逃逸、性能回归及文档一致性等问题。
Trigger Scenarios
用户请求对当前分支或PR进行合并前审查
需要检查代码中的Bug、安全漏洞或文档缺失
Install
npx skills add pydantic/monty --skill review-general -g -y
SKILL.md
Frontmatter
{
"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."
}
General branch review
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:
- Bugs — logic errors,
DropWithContextvalues not released on every exit path (the fix isdefer_drop!/DropGuard, not moredrop_withcalls), borrow/aliasing mistakes, unhandled error paths. - CPython divergence — different results, exception types or messages, missing
attributes. Check anything you're unsure of with
python-playground. - Sandbox escapes — sandboxed code reaching the host filesystem, environment, network or subprocesses.
- Resource-limit escapes — allocations not charged to the tracker (an unbounded or
amplifying
Stringbuild withoutStringBuilder), unbounded loops, recursion without a depth guard. - Performance — regressions the branch introduces, and improvements you spot.
- Verbose comments — docstrings and comments should be concise as per
CLAUDE.md. - Cleanups — duplication, misplaced logic, functions grown too complex.
./limitations/— a new divergence with no entry is a finding.- Docs parity — a user-visible change reflected in only one of
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 thedocs-parity-reviewersubagent.
Report
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.
Version History
-
a8ce0a5
Current 2026-08-28 20:47
新增对文档一致性的审查要求,包括检查README、docs/和limitations/目录的同步情况。
- 026b383 2026-08-20 09:39


