Finalize

Finalize

Use this skill to finish a feature branch with three ordered passes:

1. Run simplify on the current branch and commit all fixes.
2. Run strict-quality-review on the current branch, address found issues, and commit all fixes.
3. Run /okf update on the current branch, update the OKF documentation bundle, and commit all fixes.

The parent agent owns orchestration, acceptance, commits, and final reporting. Each pass must run in its own subagent so the cleanup, review, and documentation updates happen from focused execution contexts.

Assumptions

• The scope is the current branch against the repository's default base branch unless the user names a different base.
• The user has authorized automatic acceptance of changes produced by the three passes when they are in scope, behavior-preserving, and validation passes.
• Stop and ask before accepting product behavior changes, broad architecture changes outside the branch's intent, dependency changes, migrations, generated-file churn, or changes that touch secrets or credentials.

Pre-flight

1. Confirm you are in a git repository.
2. Inspect:
   • git status --short
   • the current branch name
   • the likely base branch, usually origin/main, main, origin/master, or master
   • the branch diff against that base
3. If the working tree contains unrelated user changes that are clearly outside the feature branch, stop and ask how to handle them.
4. Identify the harness's general-purpose subagent mechanism, such as a delegate, worker, task, or general-purpose agent. Use the generic subagent available in the current environment rather than a harness-specific recipe.
5. Set <path-to-skills-directory> to the directory that contains the companion skill folders.
6. Verify these skill files exist:
   • <path-to-skills-directory>/simplify/SKILL.md
   • <path-to-skills-directory>/strict-quality-review/SKILL.md
   • <path-to-skills-directory>/okf/SKILL.md
7. If subagents or any dependency skill file is unavailable, stop and report the missing capability. Do not emulate this workflow entirely in the parent session.

Phase 1: simplify subagent

Spawn a subagent and give it the simplify skill file path. Do not inline, summarize, or reinterpret the skill content for the subagent.

Use this prompt for the phase 1 subagent:

Finalize phase 1 for the current feature branch.

First read the `simplify` skill file at this exact path:

<path-to-skills-directory>/simplify/SKILL.md

Then follow that skill's instructions on the current branch diff. Refine recently modified code for clarity, consistency, and maintainability while preserving behavior.

Scope:
- Current branch changes against the repository base branch.
- Do not expand product scope.
- Do not perform the strict quality review or OKF update phases.
- Do not commit. The parent agent will inspect and commit accepted changes.
- Do not launch other subagents.

Acceptance:
- Apply safe, in-scope simplifications directly.
- Do not ask the user to accept each safe cleanup change.
- Stop and report any change that would alter behavior or exceed the branch intent.

Validation:
- Run focused formatting, lint, typecheck, or tests that match the changed files when available.
- Report commands run, results, changed files, and any remaining risks.

When the subagent completes:

1. Inspect git status --short and git diff.
2. Reject or revert only changes that violate scope or validation. Keep safe simplify changes.
3. Run any missing focused validation that is needed to trust the diff.
4. Commit the phase 1 changes if any exist.

Commit guidance:

• Stage files explicitly. Do not use git add . or git add -A.
• Use a conventional commit message, usually refactor: simplify branch implementation or a more specific scoped variant.
• If there are no changes, do not create an empty commit. Record that phase 1 produced no commit.

Phase 2: strict quality review subagent

Spawn a separate subagent and give it the strict-quality-review skill file path. Do not inline, summarize, or reinterpret the skill content for the subagent. This subagent audits the post-simplify branch and applies fixes for accepted findings.

Use this prompt for the phase 2 subagent:

Finalize phase 2 for the current feature branch.

First read the `strict-quality-review` skill file at this exact path:

<path-to-skills-directory>/strict-quality-review/SKILL.md

Then follow that skill's instructions on the current branch diff after the `simplify` phase. Perform a strict maintainability review focused on abstraction quality, file growth, spaghetti-condition growth, type boundaries, canonical helpers, modularity, and code-judo simplifications.

Scope:
- Current branch changes against the repository base branch.
- Address high-confidence issues directly when the fix is in scope and behavior-preserving.
- Do not expand product scope or rewrite unrelated code.
- Do not commit. The parent agent will inspect and commit accepted changes.
- Follow the skill's adversarial review workflow, including spawning parallel adversarial review subagents when available.

Acceptance:
- Apply safe, in-scope fixes directly.
- Do not ask the user to accept each safe cleanup change.
- Stop and report blockers when a finding requires a product decision, broad redesign, risky migration, dependency change, or behavior change.

Validation:
- Run focused formatting, lint, typecheck, or tests that match the changed files when available.
- Report findings addressed, findings deferred with reasons, commands run, results, changed files, and any remaining risks.

When the subagent completes:

1. Inspect git status --short and git diff.
2. Accept safe, in-scope strict-review fixes by default.
3. Reject or revert only changes that violate scope, fail validation, or require user approval.
4. Run any missing focused validation that is needed to trust the diff.
5. Commit the phase 2 changes if any exist.

Commit guidance:

• Stage files explicitly. Do not use git add . or git add -A.
• Use a conventional commit message, usually refactor: address strict quality review or a more specific scoped variant.
• If there are no changes, do not create an empty commit. Record that phase 2 produced no commit.

Phase 3: OKF update subagent

Spawn a separate subagent and give it the okf skill file path. Do not inline, summarize, or reinterpret the skill content for the subagent. This subagent runs the /okf update workflow to update the repository's OKF documentation bundle after the cleanup and review passes.

Use this prompt for the phase 3 subagent:

Finalize phase 3 for the current feature branch.

First read the `okf` skill file at this exact path:

<path-to-skills-directory>/okf/SKILL.md

Then follow that skill's `/okf update` workflow for the current branch diff after the `simplify` and `strict-quality-review` phases. Update the repository's OKF bundle so it reflects the finalized branch changes.

Scope:
- Current branch changes against the repository base branch.
- Update only relevant OKF concepts, specs, ADRs, indexes, logs, and cross-links called for by the branch changes.
- Do not expand product scope or rewrite unrelated documentation.
- Do not commit. The parent agent will inspect and commit accepted changes.
- Do not launch other subagents.

Acceptance:
- Apply safe, in-scope documentation updates directly.
- Do not ask the user to accept each safe documentation update.
- Stop and report blockers when a documentation decision requires product direction, architecture direction, or broader project context not present in the branch.

Validation:
- Run focused formatting, docs build, lint, or tests that match the changed documentation when available.
- Report OKF concepts updated, documentation intentionally left unchanged with reasons, commands run, results, changed files, and any remaining risks.

When the subagent completes:

1. Inspect git status --short and git diff.
2. Accept safe, in-scope documentation updates by default.
3. Reject or revert only changes that violate scope, fail validation, or require user approval.
4. Run any missing focused validation that is needed to trust the diff.
5. Commit the phase 3 OKF changes if any exist.

Commit guidance:

• Stage files explicitly. Do not use git add . or git add -A.
• Use a conventional commit message, usually docs: update finalized branch documentation or a more specific scoped variant.
• If there are no changes, do not create an empty commit. Record that phase 3 produced no commit.

Final verification

After all three phases:

1. Run git status --short.
2. Run a final validation command appropriate for the repository when one is discoverable and reasonably bounded, such as lint, typecheck, or targeted tests. If full validation is expensive, say what you ran and what remains unverified.
3. Confirm the branch has no uncommitted finalize changes unless a blocker remains.
4. Summarize:
   • phase 1 commit hash or no changes
   • phase 2 commit hash or no changes
   • phase 3 commit hash or no changes
   • validation commands and results
   • unresolved blockers or deferred issues

Stop rules

Stop and ask the user before proceeding when:

• a subagent requests a product, architecture, dependency, migration, or behavior decision
• validation fails and the fix is unclear
• the worktree contains unrelated dirty changes that would be swept into finalize commits
• the branch has merge conflicts or cannot identify a reasonable base branch
• required subagent or skill support is unavailable

Do not

• Do not run multiple phases in the same subagent.
• Do not run the strict-quality-review pass before the simplify pass.
• Do not run the /okf update pass before the cleanup and review passes.
• Do not ask the user to approve every safe cleanup change.
• Do not commit with broad staging commands.
• Do not leave subagent-applied fixes or documentation updates uncommitted unless there is a blocker.
• Do not let child subagents launch their own subagent workflows.

Skills Dependencies

This skill depends on these companion skills:

• simplify
• strict-quality-review
• okf

Before starting phase work, set <path-to-skills-directory> to the directory that contains the companion skill folders. Verify these files exist and pass the matching path to each subagent:

• <path-to-skills-directory>/simplify/SKILL.md
• <path-to-skills-directory>/strict-quality-review/SKILL.md
• <path-to-skills-directory>/okf/SKILL.md

If any skill file is unavailable, stop, alert the user which dependency is missing, and ask for next steps.