code-polish

Code Polish

Resolve scope once, make only high-confidence simplifications, fix evidenced defects by risk, and verify the final state.

Modes

• --simplify: simplify only.
• --review: review and fix only.
• Neither or both: simplify, then review the simplified result.
• --with-profile <name> / --skip-profile <name>: add or suppress review profiles; skip wins.

Fixed Scope

1. Require a Git repository.
2. Use explicit paths, patterns, ranges, natural-language targets, or a supplied resolved-scope block when present. Otherwise use only files modified in this session; if session history is unavailable, use all uncommitted tracked and untracked files.
3. Exclude lockfiles, generated outputs, vendored code, minified bundles, and large data snapshots from manual review unless explicitly requested. Validate relevant excluded outputs through their generator, schema, or invariants.
4. Resolve and retain one authoritative scope set and optional exclusions for execution. Do not broaden or recompute scope later. Stop if it is empty.

Simplify

Preserve public contracts, inputs, outputs, side effects, error behavior, performance-sensitive characteristics, telemetry, and operational guards. Apply only changes with a concrete comprehension or defect-risk benefit:

• flatten avoidable control-flow nesting;
• clarify misleading names or dense transforms;
• remove real duplication when the abstraction reduces total complexity;
• tighten local types and contracts without broad churn;
• remove only dead code caused by this session's edits.

Do not split by line count, perform architecture cleanup, convert sync/async APIs, add speculative configurability, or replace readable duplication with a one-use abstraction. A no-op is a valid result.

Review and Fix

Judge the diff against the user's request. Prioritize CRITICAL → HIGH → MEDIUM → LOW:

• CRITICAL: exploitable security, data loss, or critical outage path.
• HIGH: behavior, error-path, boundary, or performance defect affecting core behavior.
• MEDIUM: resource leak, complexity hotspot, test gap, over-scoped change, speculative complexity, or weak success criterion likely to cause defects.
• LOW: localized clarity or style issue with a real maintenance cost.