finalize

name: finalize description: "Run the post-implementation quality assurance workflow including tests, code polishing, review, and commit. Use when the user asks to "finalize implementation", "finalize changes", "wrap up implementation", "finish up", "ready to commit", or "run QA workflow"."

Finalize Implementation

Post-implementation QA workflow: tests, code polishing, commit, and self-improvement.

Task Tracking

At the start, use update_plan to track each phase:

Run $polish-code skill
Run $simplify-docs skill
Run $update-changelog skill
Run $self-improve skill
Ship It

Phase 1: Run `$polish-code` Skill

Run the $polish-code skill for the current changes.

Phase 2: Run `$simplify-docs` Skill

Run the $simplify-docs skill on the staged changes (git diff --cached). Stage any edits it makes before continuing.

Phase 3: Run `$update-changelog` Skill

Run the $update-changelog skill.

Phase 4: Run `$self-improve` Skill

Run the $self-improve skill for the current session. Always run this phase even if the session seemed routine.

Phase 5: Ship It

Step 1: Analyze Split

Examine the staged changes and evaluate whether they form a single reviewable unit or several independently reviewable units. This step decides only whether to split; the chosen ship skill owns all repository-state detection and the commit, push, branch, and PR intent.

Run git diff --cached --stat and git diff --cached to understand the scope. Categorize changes along three dimensions:

Concern type: refactoring, bug fix, new feature, cleanup, dependency update
Layer/domain: backend, frontend, database migrations, i18n, tests, configuration
Logical unit: files that form a coherent, independently reviewable change

A split is warranted when the staged changes contain multiple reviewable units. Each unit should be independently understandable, testable, and revertable. When deciding group boundaries, consider whether a reviewer could evaluate each group without needing context from the others.

Step 2: Present Analysis and Choose Path

Output the split analysis as text.

If changes form a single cohesive unit, note this and run the $ship skill.

If changes span multiple reviewable units, propose an ordered list of groups. For each group, specify:

Name and one-line description
File list (flag files with mixed-concern hunks)
Dependencies: which earlier groups, if any, this group builds on

Use request_user_input to let the user choose whether to ship the changes together or split them up.

Ship together — ship all staged changes as one unit; run the $ship skill
Split up — ship each group as its own unit; run the $split-and-ship skill

Then update or check the active plan and proceed to any remaining task.

Rules

Diff size, number of files changed, passing tests, perceived user urgency, or context window concerns are not reasons to skip a phase. Each phase does work beyond what those signals cover. "The session was long" or "a prior phase was thorough" are never valid reasons to skip a later phase.
Never stage or commit files containing secrets (.env, credentials, API keys). Warn if detected.
Do not present diffs to the user — the user reviews diffs in an external git client. Use git diff internally as needed.
If a non-test step fails (polish, review), stop and report the failure. Do not skip ahead.