Fix
Address Codex review findings from the ledger. Shows open findings and guides resolution. Each finding must get an explicit resolution state.From its SKILL.md
npx -y skills add akshan-main/review-by-opp --skill fixAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 6 stars6 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
2.3 KB, as published. Nobody here has run it
You are addressing Codex review findings. You are the builder - fix what needs fixing.
What to do
-
Load the ledger:
- Read
reviews/current.json - If missing, tell user: "No active session. Run
/review-by-opp:startfirst."
- Read
-
Show open findings:
- List all findings with status
open, grouped by severity (critical first) - For each, show: ID, severity, category, title, file, line, description, suggested_fix
- List all findings with status
-
Address findings one by one: For each finding, you MUST do one of these - no exceptions:
a. Fix it → Make the code change, then update the finding:
- Set
statustofixed - Set
resolution_noteto a brief description of what you changed - The fix must be verifiable (actual file changes, not just claiming it's done)
b. Mark as won't_fix → If the finding is intentional or acceptable:
- Set
statustowont_fix - Set
resolution_noteexplaining why
c. Mark as not_reproducible → If you cannot reproduce the issue:
- Set
statustonot_reproducible - Set
resolution_notewith your investigation
d. Mark as needs_context → If you need more information:
- Set
statustoneeds_context - Set
resolution_notedescribing what context is needed
e. Mark as duplicate → If this duplicates another finding:
- Set
statustoduplicate - Set
duplicate_ofto the other finding's ID
- Set
-
Update the ledger:
- Write updated findings back to
reviews/current.json - Every finding must have a resolution state - you cannot skip any
- Write updated findings back to
-
Report progress:
- Show how many findings were resolved this pass
- Show remaining open findings
- If blocking findings remain: "Run
/review-by-opp:fixagain or/review-by-opp:reviewfor another Codex pass" - If no blocking findings: "Ready for another review round (
/review-by-opp:review) or finalize (/review-by-opp:finalize)"
CRITICAL: You MUST NOT claim "all fixed" unless every finding has an explicit resolution state. The ledger enforces this - hand-waving is not allowed.
$ARGUMENTS