507 Review
交付审查：围绕明确的 commit、branch、diff 或路径检查 Standards、Spec 与 Code Quality，只报告问题、证据和路由。Use when user says review this change, code review, review 一下, 检查, 审查, 审查改动, 审查这次改动, 检查提交, 检查分支, 过一遍代码, 把关, 需求符合度, 交付审查. 不用于主动寻找全库架构机会。
Install
$ agentstack add skill-ssdiwu-507-skills-507-review ✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
Verified badge
Passed review? Show it. Paste this badge into your README, it links to the public security report.
Links to your public security report.[](https://agentstack.voostack.com/security/report/skill-ssdiwu-507-skills-507-review) Reliability & compatibility
Declared compatibility
Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.
We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.
How agent discovery & health will work →About
交付审查（review）
对一个明确的代码或文档变更范围检查：
- Standards（项目规范）：项目规则、既有决策与文档约束；
- Spec（需求符合度）：需求、验收标准、范围和行为；
- Code Quality（代码质量）：真实的 bug、安全、错误处理、性能和可维护性风险。
本 skill 只读审查：只给发现、证据和路由，不修改文件。
确定审查范围
按顺序选择：
- 用户明确给出的 commit（提交）、范围、branch（分支）、tag（标签）或路径；
- 当前 staged（已暂存）、unstaged（未暂存）和相关 untracked（未跟踪）变更；
- 当前分支相对默认主干的差异；
- 已提交且工作区干净时，仍审当前分支相对主干的差异；
- 没有可用基准时，临时审最近一个 commit 并明确说明；
- 仍无法判断时，只询问一次范围并给出推荐基准。
工作区干净不等于没有可审内容。开始前说明范围、基准和选择原因。
审查依据
优先读取相关 AGENTS.md、根及目标目录 README.md、doc/README.md、术语表、决策档案、贡献/风格规范、测试与构建配置，以及用户提供的 issue（问题单）、PRD（需求文档）、spec（规格）或验收标准。
没有明确需求来源时，将 Spec 标为“无明确 spec”，仍检查范围蔓延、行为风险和可验证性。只把已有证据当作规范，不把个人偏好包装成 Standards。
审查重点
Standards
- 是否违反项目规范、既有决策、不做边界或稳定术语；
- 职责、入口、运行方式、配置或权限变化是否同步文档；
- 用户可见变化是否同步项目要求的变更记录和版本信息。
Spec
- 需求是否完整实现，有无漏做、误做或范围蔓延；
- 行为、文档、测试和验收标准是否一致；
- 验收是否可复验，入口 README、项目地图和发布产物是否完整。
Code Quality
只报告真实或有明确证据的风险：边界/空值/竞态/状态错误，注入/权限/泄密，静默失败，不必要重算或泄漏，以及复杂度、误导命名、死代码和重复逻辑。不要重复 formatter（格式器）或 linter（静态检查器）已经明确捕获的表面问题。
输出格式
## Review Scope
- 范围：...
- 基准：...
- 原因：...
- 依据：...
## Summary
- Standards：通过 / 有问题 N 个
- Spec：通过 / 有问题 N 个 / 无明确 spec
- Code Quality：通过 / 有问题 N 个
- 最严重问题：...
## Findings
### 🔴 严重 / 🟡 警告 / 🔵 信息
- `file:line`：问题
- 依据：...
- 影响：...
- 建议：...
## Recommended Next Mode
- 默认下一步：...
- 路由理由：...
## Verification
- 已检查 / 已运行 / 未运行 / 建议验证：...没有发现阻塞问题时明确写“未发现阻塞问题”，并说明范围和验证限制。
后续路由
- bug、逻辑或安全问题 →
507-fix； - 测试缺口/失败 →
507-test；明确 test-first →507-tdd； - 行为不变内部简化 →
507-simplify；架构摩擦 →507-inspect； - 地图文档失真 →
507-map；完整项目初始化或规范巡检 →507-setup； - 漏做需求 →
507-prd或507-issue； - 已完成且用户明确要求 Git 提交 →
507-commit。
红线
- 不直接修改、修复、重构、补测试或提交；
- 不把无证据风险断言为事实；
- 不因工作区干净跳过已提交差异；
- 不大段复述 diff（差异），只列有行动价值的问题。
完成与接力
- 完成信号：明确范围已按 Standards、Spec 与 Code Quality 审完，给出带证据的 findings（发现）和清晰 verdict（结论）。
- 产物：只读审查报告与问题路由，不直接修改。
- 候选出口：按发现进入
507-fix、507-test、507-simplify、507-map、507-setup、507-prd或507-issue；没有阻塞且用户明确要求提交时进入507-commit；只需审查结论时直接结束。 - 回退条件：审查基线或目标规格不明确时先返回
507-explore或507-grill，不对猜测中的需求判通过/失败。
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: ssdiwu
- Source: ssdiwu/507-skills
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet, be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.