version: "1.0.1" argument-hint: '' context: fork disable-model-invocation: false name: code-polish user-invocable: true description: This skill should be used when the user asks to "polish code", "simplify and review", "clean up and review code", "full code polish", "simplify then review", "refactor and review", "simplify and fix", "clean up and fix", or wants a combined simplification and review workflow on recently changed code.

Code Polish

Combined simplification and review pipeline. This skill orchestrates two sub-skills in sequence:

1. `code-simplify` — simplify for readability and maintainability
2. `code-review --fix` — review for correctness, security, and quality, auto-applying all fixes

Optimize for one scope resolution, one user-facing report, and no redundant simplify verification.

Scope Resolution

1. Verify repository context: git rev-parse --git-dir. If this fails, stop and tell the user to run from a git repository.
2. If user provides file paths/patterns, a commit/range, or a Resolved scope fenced block with one repo-relative path per line, scope is exactly those targets.
3. Otherwise, scope is only session-modified files. Do not include other uncommitted changes.
4. If there are no session-modified files, fall back to all uncommitted tracked + untracked files:

• tracked: git diff --name-only --diff-filter=ACMR
• untracked: git ls-files --others --exclude-standard
• combine both lists and de-duplicate.

1. Exclude generated/low-signal files unless requested: lockfiles, minified bundles, build outputs, vendored code.
2. If scope still resolves to zero files, report and stop.
3. Normalize the final scope into a Resolved scope fenced block with one repo-relative path per line. Reuse that exact block for both sub-skills instead of asking them to rediscover scope.

Workflow

1) Resolve scope once

• Apply the "Scope Resolution" rules above.
• Treat the resulting Resolved scope block as authoritative for all downstream work.
• Forward user intent, constraints, and risk preferences, but skip raw scope selectors already captured in the resolved block.

2) Run code-simplify

Invoke the code-simplify skill with:

• the authoritative Resolved scope block
• --no-verify
• --no-report
• any non-scope user intent that still matters

Tell code-simplify not to broaden or rediscover scope.

3) Run code-review --fix

Invoke the code-review skill with:

• the same authoritative Resolved scope block
• --fix
• any non-scope user intent that still matters

If the user explicitly asks for a speed-first pass over maintainability coverage, you may also append --skip-profile naming. Do not skip the naming profile by default.

4) Final verification

• Treat code-review's post-fix verification as the final verification summary when it already covers the final touched scope.
• If verification was skipped, partial, or no longer matches the final diff, run one narrow final verification pass across the final touched scope.
• Always report skipped checks explicitly.

5) Report

Combine the final state into one summary:

1. Scope: Files and functions touched.
2. Simplifications: Key changes from code-simplify, derived from the actual diff when needed because --no-report was used.
3. Review findings and fixes: Findings and applied fixes from code-review.
4. Verification: Commands run and outcomes.
5. Residual risks: Assumptions or items needing manual review.