一键导入
review-fix
Code review with automatic fix. Detects bugs, stale references, duplication, provider abstraction gaps, and sort instability, then fixes and verifies.
用 Codex 或 Claude 帮你安装 复制这段 Prompt,粘贴到 Codex、Claude 或其他助手里,让它检查 Skill 页面并帮你完成安装。
菜单
Code review with automatic fix. Detects bugs, stale references, duplication, provider abstraction gaps, and sort instability, then fixes and verifies.
用 Codex 或 Claude 帮你安装 复制这段 Prompt,粘贴到 Codex、Claude 或其他助手里,让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Duplicate code detection using similarity-rs. Identifies refactoring candidates based on semantic similarity.
Duplicate code detection using similarity-rs. Identifies refactoring candidates based on semantic similarity.
Run performance benchmarks with criterion. Compare against baselines and profile hotspots.
Implementation verification - runs format, lint, test, and build checks on the workspace.
Production readiness audit - checks unwrap elimination, error handling, clippy, docs, channel usage, and test count.
Release deployment process for ccswarm. Version update, quality gates, build, tag, publish.
| name | review-fix |
| description | Code review with automatic fix. Detects bugs, stale references, duplication, provider abstraction gaps, and sort instability, then fixes and verifies. |
| user-invocable | true |
| context | fork |
| agent | review-fix-agent |
Review uncommitted changes, fix actionable issues, and verify the result.
git status --short and git diff --stat HEAD to determine what changed.Run build and lint to establish a clean starting point:
cargo clippy --workspace -- -D warnings 2>&1 | tail -20
cargo test --workspace --no-run 2>&1 | tail -5
If the baseline already fails, report the failures and stop. Do not mask pre-existing breakage.
Inspect the diff and new untracked files. Check for:
| Category | What to look for |
|---|---|
| Stale references | Deleted modules or commands still referenced in routing, docs, or templates |
| Code duplication | Identical structs/functions across multiple files |
| Provider abstraction gaps | Hard-coded CLI names instead of AgentProvider |
| Dead async | Async operations discarded and then repeated |
| Sort instability | Utc::now() used as a fallback sort key |
| Hardcoded paths | Filesystem paths that should be relative to repo_path |
| Unused params | CLI arguments accepted but silently ignored |
| Security | Path traversal, credential leaks |
Classify each finding:
For each fix-level finding:
cargo clippy --workspace -- -D warnings
cargo test --workspace --no-run
If clippy or test compilation fails, revert the offending edit and downgrade it to a warning.
Output a structured summary:
## Review + Fix Report
### Fixed
- <file>:<line> - <description>
### Warnings
- <file>:<line> - <description>
### Notes
- <description>
### Verification
- clippy: PASS / FAIL
- test compile: PASS / FAIL
- files modified: N