nicolas-codemate/skills/polish-code

Polish-Code — Framework-Aware Code Simplification

Delegates code simplification to the most relevant {framework}-simplifier subagent, mirroring the pattern used by /resolve step 07-simplify.

Input

$ARGUMENTS

Parse Arguments

• --agent generic|symfony|laravel — force specific agent (skip auto-detect)
• --scope modified|all — scope (default: modified)
• --file <path> — target a specific file (repeatable)
• --dry-run — analyze only, do not modify files

Step 1 — Detect Agent

If --agent is not provided, auto-detect from project files:

if [ -f "composer.json" ] && grep -q "symfony/framework-bundle" composer.json; then
    AGENT="symfony-simplifier"
elif [ -f "composer.json" ] && grep -q "laravel/framework" composer.json; then
    AGENT="laravel-simplifier"
else
    AGENT="code-simplifier"
fi

Display: Agent detected: {AGENT}

Step 2 — Resolve Target Files

If --file provided

Use the listed files as-is.

If --scope modified (default)

Determine base branch in this priority order:

1. .claude-work/{ticket-id}/status.json → options.base_branch
2. .claude/ticket-config.json → branches.default_base
3. Fallback: main

BASE_BRANCH=$(cat .claude-work/*/status.json 2>/dev/null | jq -r '.options.base_branch // empty' | head -1)
[ -z "$BASE_BRANCH" ] && BASE_BRANCH=$(cat .claude/ticket-config.json 2>/dev/null | jq -r '.branches.default_base // "main"')

# Uncommitted first, then branch diff
CHANGED=$(git diff --name-only)
[ -z "$CHANGED" ] && CHANGED=$(git diff --name-only "${BASE_BRANCH}...HEAD")
echo "$CHANGED"

Filter by extension based on agent:

• symfony-simplifier / laravel-simplifier → *.php
• code-simplifier → *.js *.ts *.jsx *.tsx *.py *.go *.rs *.rb

If --scope all

AskUserQuestion:
  question: "Simplifier tout le codebase peut prendre du temps. Continuer ?"
  header: "Scope"
  options:
    - label: "Oui, tout simplifier"
      description: "Analyser tous les fichiers correspondant à l'agent"
    - label: "Non, seulement les modifiés"
      description: "Limiter aux fichiers modifiés (recommandé)"

Step 3 — Delegate to the Subagent

Invoke the subagent directly (same pattern as /resolve step 07):

Agent:
  subagent_type: {AGENT}
  description: "Polish {N} file(s)"
  prompt: |
    Review and simplify the following files:
    {list of absolute paths, one per line}

    Project root: {cwd}
    Mode: {apply|dry-run}

    {if dry-run}
    DO NOT modify files. Only report proposed changes.
    {else}
    Apply improvements directly to each file using Edit.
    {endif}

    Focus on: clarity, naming, redundancy removal, early returns,
    pattern compliance, type annotations. Preserve all behavior.

    RETURN FORMAT (mandatory):
    ```
    POLISH_REPORT:
      agent: {AGENT}
      files_analyzed: {count}
      files_modified: {count}
      changes:
        - file: path/to/file.ext
          line: {line range}
          category: [Readability | Redundancy | Naming | Abstraction | Consistency]
          description: "{what was changed and why}"
      no_changes_needed:
        - path/to/file.ext (reason: already clean)
      summary: "{1-2 sentence overall assessment}"
    ```

Step 4 — Display the Report

Output the subagent's POLISH_REPORT verbatim. Do NOT summarize — the user needs to see every change to decide whether to keep them.

Step 5 — Next Action

If --dry-run

No changes applied. Re-run without --dry-run to apply, or target specific files:
  /polish-code --file {path}

If files were modified

git diff --stat

Display:

Modifications appliquées. Review with `git diff`, then commit when satisfied:

  git diff                                    # Full diff
  git add -A && git commit -m "refactor: polish code ({AGENT})"
  git restore -- <file>                       # Blocked by hook — ask for override if needed

Do NOT auto-commit and do NOT propose a revert (destructive git is hook-blocked).

Examples

# Auto-detect agent, simplify modified files
/polish-code

# Force Symfony agent
/polish-code --agent symfony

# Simplify specific file
/polish-code --file src/Service/UserService.php

# Preview without applying
/polish-code --dry-run

# Simplify entire codebase (with confirmation)
/polish-code --scope all

Language

User-facing messages in French. Code examples and technical output in English.

NOW

Begin with $ARGUMENTS:

1. Detect or use specified agent
2. Resolve target files
3. Delegate to the subagent with a structured prompt
4. Display the verbatim POLISH_REPORT
5. Point the user to git diff and commit instructions