Agent Skillspydantic/monty › review-general

review-general

GitHub

用于在合并前对分支或PR进行通用代码审查,检查Bug、CPython差异、沙箱逃逸、资源限制、性能回归及注释规范,并生成简洁的缺陷报告。

.agents/skills/review-general/SKILL.md pydantic/monty

Trigger Scenarios

需要对新分支或PR进行合并前审查 用户请求检查代码中的逻辑错误、安全漏洞或性能问题

Install

npx skills add pydantic/monty --skill review-general -g -y
More Options

Non-standard path

npx skills add https://github.com/pydantic/monty/tree/main/.agents/skills/review-general -g -y

Use without installing

npx skills use pydantic/monty@review-general

指定 Agent (Claude Code)

npx skills add pydantic/monty --skill review-general -a claude-code -g -y

安装 repo 全部 skill

npx skills add pydantic/monty --all -g -y

预览 repo 内 skill

npx skills add pydantic/monty --list

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\/ 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, 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.
  • 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 String build without StringBuilder), 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.

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

  • 026b383 Current 2026-08-20 09:39

Same Skill Collection

.agents/skills/fastmod/SKILL.md
.agents/skills/fix-pr-comments/SKILL.md
.agents/skills/python-playground/SKILL.md
.agents/skills/review-security/SKILL.md
.agents/skills/review-usability/SKILL.md
.agents/skills/review-verbosity/SKILL.md
.agents/skills/writing-style/SKILL.md

Metadata

Files
0
Version
026b383
Hash
93dbb0a2
Indexed
2026-08-20 09:39

- 위키
Copyright © 2011-2026 iteam. Current version is 2.155.2. UTC+08:00, 2026-08-25 09:28
浙ICP备14020137号-1 $방문자$