Sahil Saghir db744e7d1e feat(simplify-code): add risk-tiered application, Chesterton's Fence, slop + silent failure detection ... Five targeted enhancements to the upstream simplify-code skill: 1. Risk-tiered application (SAFE/CAREFUL/RISKY) — safe changes auto-applied, careful changes verified per-file, risky changes flagged for human review. Prevents auto-applying N+1 restructures and public API renames. 2. Chesterton's Fence — before flagging anything for removal, reviewers run 'git blame' to understand why it exists. Low-confidence findings are escalated rather than guessed. 3. AI slop detection — Quality reviewer now catches: extra comments restating obvious code, unnecessary defensive null-checks on validated inputs, 'as any' casts, and patterns inconsistent with the rest of the file. 4. Silent failure detection — Efficiency reviewer now catches: empty catch blocks, ignored error returns, except:pass, .catch(()=>{}) with no handling, and error propagation gaps. 5. Structured reviewer output with confidence+risk tags — reviewers report in 'file:line → problem → fix | confidence: H/M/L | risk: SAFE/CAREFUL/RISKY' format, enabling the orchestrator to tier the application. Plus 3 new pitfalls: over-trusting dead code tools, public contract awareness, and preserving intentional error handling. Total: +45/-8 lines. Keeps the 212-line compact spirit. Ref: #379		2026-06-19 07:35:36 -07:00
..
SKILL.md	feat(simplify-code): add risk-tiered application, Chesterton's Fence, slop + silent failure detection	2026-06-19 07:35:36 -07:00