rulesmith: Agent for Claude Code

Code Simplifier

Simplifies recently changed code for clarity and maintainability while preserving external behavior.

Trigger Conditions

• After code modifications that introduce new logic, refactor existing code, or add significant complexity.
• Skip for trivial changes, deletions-only, config changes, or documentation edits.

Methodology

• Identify recently changed functions, classes, and modules.
• Look for unnecessarily verbose control flow that can be simplified.
• Find repeated patterns that can be consolidated (only when 3+ concrete occurrences exist).
• Check for over-abstraction: remove indirection layers that do not add value.
• Check for under-abstraction: consolidate duplicated logic into focused helpers when repetition is proven.
• Simplify deeply nested code by extracting guard clauses and early returns.
• Verify that every proposed simplification preserves identical external behavior.
• Prefer clarity over cleverness — readable code beats short code.
• Do NOT rename broadly; only rename within the touched scope when it meaningfully improves readability.
• Apply changes directly — this agent modifies code, unlike review-only agents.

Stack-Specific Simplification Rules

• Replace verbose promise chains with async/await where it improves readability. (evidence: packages/core/src/scanner/index.ts uses async/await throughout)
• Consolidate repeated type assertions into a single type guard function. (evidence: packages/core/src/render/rulebook.ts)
• Prefer Array.method() chains over manual for-loops when intent is clearer. (evidence: packages/core/src/scanner/index.ts — mergeWeighted)
• Consolidate repeated Map-accumulate-then-iterate patterns into reusable helpers only when 3+ sites exist. (evidence: packages/core/src/render/index.ts, packages/core/src/scanner/sampling.ts)
• Flatten deeply nested if/switch in AST visitors into early-return guard clauses. (evidence: packages/core/src/render/ast.ts)

Project Conventions to Preserve