review
- Repo stars 1,985
- License MIT
- Author repo 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.
- Fluxly category
- Engineering
- Author-declared agents
- No explicit declaration found; this is not inferred or tested compatibility
- Static check
- 94 / 100 · heuristic scan, not runtime safety proof
- Author / version / license
- @JiHong88 · MIT
- Fluxly token estimate
- Lean
- Fluxly setup estimate
- Plug-and-play
- External API key
- No requirement detected
- Detected OS requirements
- Unspecified
- Runtime requirements
- Unspecified
- Detected file/system behavior
-
- Read-only
- Detected network behavior
- Local-only
- Install commands
- None (reference only)
Profile is derived at build time from SKILL.md and install vectors. Subject to drift from author intent.
Heads up: 未限定 allowed-tools,默认拥有全部工具权限。
The current SKILL.md does not define a fixed output example. 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. Author text anchors workflow facts; Fluxly only indexes current sections, terms, files, and commands.
sections -> no H2/H3 headings
terms -> Null/guard checks · Intentional patterns · Severity filtering · staged/unstaged diff
files/cmd -> $ARGUMENTS · git diff · test/dev/ · test/dev · Null/guard · staged/unstaged
body sha256 -> c59ac48ba51e
Decide Fit First
Design Intent
How To Use It
Boundaries And Review