review
- 作者仓库星标 1,985
- 许可证 MIT
- 作者仓库 suneditor
Review the specified file(s) or current git diff for bugs, logic errors, and dead code.
Target: $ARGUMENTS (file path, or blank for current diff)
Steps:
- If arguments provided, read those files. Otherwise run
git diffto get current changes. - Exclude
test/dev/files from review — they are local dev-only and not review targets. - Analyze for:
- Logic bugs (off-by-one, null access, race conditions)
- Dead code (unreachable branches, unused variables)
- Missing edge cases
- Report findings with file path and line numbers.
- Do NOT fix anything — report only. Let the user decide what to fix.
Review guidelines — avoid false positives:
- Null/guard checks: If a null check already exists upstream in the call chain (caller side), do NOT flag the callee for lacking its own null guard. Only flag if there is a realistic unguarded path.
- Intentional patterns: If code looks intentional and consistent with surrounding patterns, do NOT report it as a bug. If unsure, note it as a confirmation question (e.g. "의도된 동작인지 확인: ...") rather than a bug.
- Severity filtering: Only report issues that could actually cause incorrect behavior or crashes. Do not report style preferences, hypothetical future risks, or "would be nice" improvements.
- staged/unstaged diff: Do not flag git staging state differences as findings.
- 流狐分类
- 工程开发
- 作者声明 Agent
- 未找到明确声明；不据此推断已兼容或已测试
- 静态检查
- 94 / 100 · 启发式扫描，不代表运行安全
- 作者 / 版本 / 许可
- @JiHong88 · MIT
- 流狐 Token 估算
- 低消耗
- 流狐接入估算
- 即装即用
- 是否需要外部 API Key
- 未发现要求
- 检测到的系统要求
- 未声明
- 底层运行要求
- 未声明
- 检测到的文件与系统行为
-
- 只读
- 检测到的网络行为
- 仅限本地
- 安装命令数
- 无（仅作为资料）
档案由构建时根据 SKILL.md 与安装命令自动衍生，可能与作者实际意图存在差异。
需要注意： 未限定 allowed-tools，默认拥有全部工具权限。
作者没有在当前 SKILL.md 中定义固定输出样例。 原文未拆分步骤
Review the specified file(s) or current git diff for bugs, logic errors, and dead code. Target: `$ARGUMENTS` (file path, or blank for current diff) Steps: 1. If arguments provided, read those files. Otherwise run `git diff` to get current changes. …
Review the specified file(s) or current git diff for bugs, logic errors, and dead code.
Target: `$ARGUMENTS` (file path, or blank for current diff)
Steps:
1. If arguments provided, read those files. Otherwise run `git diff` to get current changes.
2. Exclude `test/dev/` files from review — they are local dev-only and not review targets.
3. Analyze for:
- Logic bugs (off-by-one, null access, race conditions)
- Dead code (unreachable branches, unused variables)
- Missing edge cases
4. Report findings with file path and line numbers.
5. Do NOT fix anything — report only. Let the user decide what to fix.
Review guidelines — avoid false positives:
- **Null/guard checks**: If a null check already exists upstream in the call chain (caller side), do NOT flag the callee for lacking its own null guard. Only flag if there is a realistic unguarded path.
- **Intentional patterns**: If code looks intentional and consistent with surrounding patterns, do NOT report it as a bug. If unsure, note it as a confirmation question (e.g. "의도된 동작인지 확인: ...") rather than a bug.
- **Severity filtering**: Only report issues that could actually cause incorrect behavior or crashes. Do not report style preferences, hypothetical future risks, or "would be nice" improvements.
- **staged/unstaged diff**: Do not flag git staging state differences as findings. 证据边界与执行链路
作者原文负责流程事实；流狐只索引当前章节、要点、文件与命令。
章节 -> 原文未使用二、三级标题
要点 -> Null/guard checks · Intentional patterns · Severity filtering · staged/unstaged diff
文件/命令 -> $ARGUMENTS · git diff · test/dev/ · test/dev · Null/guard · staged/unstaged
内容 SHA-256 -> c59ac48ba51e
原文结构
适用与边界
原文中的明确线索
$ARGUMENTS、git diff、test/dev/、test/dev、Null/guard、staged/unstaged