菜单
Use when /claude-tweaks:review passes and you need to capture learnings, clean up specs/plans, update skills, and decide next steps. The lifecycle closure step.
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Use when you want to sweep every built-but-unsigned-off work record — or recap and sign off on ad hoc work from this same conversation that has no work record at all — and give each one a human verdict, approve or request changes. The durable acceptance gate distinct from tests passing (/test) and code-quality review (/review). Keywords - acceptance, sign-off, demo, verification brief, human verdict, demo:pending, session-recall.
Use when you need a quick reference for available commands, want to see workflow status, or need a recommendation for what to do next.
Use when the backlog needs hygiene — review stale backlog records, parked-trigger wakes, unsynced local records, and orphaned plans/worktrees
Use when you need to create, update, query, or resolve open items in a pipeline ledger file. Called by /claude-tweaks:build, /claude-tweaks:test, /claude-tweaks:review, /claude-tweaks:wrap-up, and /claude-tweaks:flow — or standalone for ledger inspection.
Use when implementing a work record, spec, or design doc end-to-end. Accepts a record reference (#N) or spec number (legacy alias) for full lifecycle tracking, or a design doc path to skip /claude-tweaks:specify and build directly from brainstorming output.
Use when you want to run an automated build → test → review → polish → wrap-up pipeline on a work record or spec without stopping between steps. Accepts record references (#N) and specs only — design docs must be decomposed via /claude-tweaks:specify first.
| name | claude-tweaks:wrap-up |
| description | Use when /claude-tweaks:review passes and you need to capture learnings, clean up specs/plans, update skills, and decide next steps. The lifecycle closure step. |
Interaction style: Present single decisions via the
AskUserQuestiontool (options with one marked Recommended) instead of a plain-text numbered list. For multi-item decisions, render a batch table with recommended actions pre-filled, then capture the apply-all/override decision via oneAskUserQuestioncall. Never make more than oneAskUserQuestioncall per logical decision — resolve each before showing the next. End skills with a## Next Actionsblock rendered viaAskUserQuestion(context-specific options, one recommended), not a navigation menu.
Post-review reflection, knowledge capture, and lifecycle cleanup. Part of the workflow lifecycle:
/claude-tweaks:init → /claude-tweaks:capture → /claude-tweaks:challenge → /superpowers:brainstorming → /claude-tweaks:specify → /claude-tweaks:build → /claude-tweaks:stories → /claude-tweaks:test → /claude-tweaks:review → [ /claude-tweaks:wrap-up ]
^^^^ YOU ARE HERE ^^^^
/claude-tweaks:review just passed and the work needs reflection and cleanup/claude-tweaks:help flags specs awaiting wrap-up/claude-tweaks:review verified the code is good. /claude-tweaks:wrap-up asks: what did we learn, and what needs cleaning up?
This skill handles reflection (capturing learnings), spec lifecycle (completion/cleanup), and knowledge routing (updating skills, CLAUDE.md, memory). It does NOT re-review code quality — that's /claude-tweaks:review's job.
Determine what type of work was completed:
$ARGUMENTS is provided:${RUN_DIR}/work/*-spec.md) — record mode| Type | Characteristics | Primary Focus |
|---|---|---|
| Record- or spec-based | A materialized header exists for this run, or — legacy alias — a spec file exists in specs/ | Full lifecycle: record/spec completion + plans + all assessments |
| Conversation-based | No record or spec, just work discussed | Assessments only (skip spec/plan cleanup steps) |
Note: Spec compliance (deliverables + acceptance criteria) was already verified in
/claude-tweaks:reviewStep 1. This step summarizes what was done — it does not re-verify.
Summarize the implementation against the record or spec:
/claude-tweaks:review) → record mode, github-issues: the record closes via merge (cleanup-procedures.md Section C's carrier commit); record mode, local-files: the record file is marked closed: true in place (cleanup-procedures.md item 5); legacy spec-file alias: the spec file will be deleted/claude-tweaks:review passed with minor gaps flagged) → identify what remainsReview conversation and recent commits to identify what was implemented and which key files changed.
When a pipeline run directory exists, read config.yml's ceremony-profile. Run /claude-tweaks:reflect in light mode when it is fast-lane; full mode otherwise (including standalone wrap-up, where no config.yml exists to read). Pass:
wrap-up--source wrap-up — only when no pipeline run directory exists (standalone wrap-up has no $PIPELINE_RUN_DIR to signal parent invocation on its own) — see /claude-tweaks:reflect's Component-Skill ContractFull mode handles all four reflection lenses (Surprises, Approach, Near-misses, Fresh start), the tradeoff review, insight routing, and ledger writes. Light mode (skills/reflect/light-mode.md) runs only the Near-misses and Fresh-start lenses and skips the tradeoff review — see docs/superpowers/specs/2026-07-15-fast-lane-pipeline-profile-design.md for the rationale. See /claude-tweaks:reflect for details on both.
If any insight is "Implement now", the reflect skill handles it before returning control. Proceed after all insights are resolved.
Skip entirely when config.yml's ceremony-profile is not fast-lane (including standalone wrap-up, where no config.yml exists). Otherwise, check both trigger conditions:
/claude-tweaks:review's summary (passed into this run) contain a finding at any severity?reflect/SKILL.md Step 3's routing table)?If either is true, downgrade config.yml's ceremony-profile to standard in place and log:
AUTO {time} — Ceremony profile downgraded fast-lane → standard: {trigger}. Remaining wrap-up steps run at standard depth.
Steps 6 and 7 below read the (possibly just-downgraded) value fresh at their own point of use — no other propagation needed. This never re-runs Step 3 itself, or any build-side step already completed under the original fast-lane value — see the design doc's Escape Hatch section for why this is deliberate, not a gap.
Identify unfinished spec sections that cannot be completed in the current work context, then route them per leftover-routing.md in this skill's directory — which owns the fix-exhaust qualification criteria, the auto-mode stage entry format, the interactive routing table (5 routing options), and the per-item routing semantics.
This step plans the cleanup — it does not execute. Actual deletions and archival run in Step 10 after the nothing-left-behind gate (Step 8.5) and the Review Console / batch decision (Step 8.6 or Step 9) approve them.
See cleanup-procedures.md in this skill's directory for the canonical cleanup list — Step 5 enumerates the same 8 items, in canonical order: execution plans, ledger, design caches, worktree, record/spec lifecycle, ephemeral dev server, issue claim release, pipeline run dir (always last — see the canonical list's ordering rule). Filter the list to rows whose Condition holds for this run (e.g., skip the worktree row when no worktree strategy was used). Carry the filtered list forward into Step 9's summary and Step 10's execution.
Batch collection. Step 6 collects potential documentation, CLAUDE.md/rules, and decision-record updates in a single pass across three sub-scans (Documentation, CLAUDE.md and Rules, Decision Records). No decisions are made here — everything is presented together in Step 9 for batch approval. Skill updates are handled separately in Step 7.
Parallel execution: Run all three sub-scans (documentation, CLAUDE.md/rules, decision records) as parallel tool calls — each checks independent sources and collects findings in the
[type] target — changeformat.
When config.yml's ceremony-profile is fast-lane (read fresh — see Step 3.5), skip all three sub-scans below entirely — report "No configuration updates needed (fast-lane: diff touches no registry-matched path, no new dependency, no schema/config file)" and proceed to Step 7 — when ALL of the following hold:
git diff --name-only against this work's base ref matches none of docs/REGISTRY.md's Auto-detect patterns. Reuse bin/lib/issues/blast-radius.js's classifyDiffFiles, passing the registry's patterns as the sensitivePaths argument — a result's isSensitive: true means a registry-pattern hit here, not a merge-sensitivity one; the function is generic path-glob matching regardless of which patterns list it's fed, and is already fully tested.git diff package.json (and any workspace-level equivalent) shows no added dependency.operational-checklist.md in skills/build/).If docs/REGISTRY.md doesn't exist, this pre-check cannot resolve the first condition — treat it as unmet (run the sub-scans normally) rather than skipping on incomplete information. This pre-check only applies under fast-lane; a standard-profile run (or standalone wrap-up, where no config.yml exists) always runs all three sub-scans as before.
Parallel execution: Read
docs/REGISTRY.mdand all doc files referenced in it as parallel Read calls.
Check if the work requires updates to project documentation, using the doc registry as a guide:
Registry-guided check — Read docs/REGISTRY.md. For each entry:
git diff --name-only)/build Step 6.5 already updated this doc (look for doc commits in git log)Non-registry docs — Also check setup guides, architecture references, API documentation, and ADRs as before (catches docs not yet in the registry, or projects without a registry)
Registry maintenance — Check if:
Docs-health check on touched docs, and missing-doc detection — read docs-health-integration.md in this skill's directory for the full procedure: D1 judges every doc this work edited or created against the shared docs-health criteria (genre-drift, depth-mismatch, findability, staleness), routing additive findings into this step's own [doc] collection and filing restructural findings as GitHub issues; D2 detects when this work introduced a new subsystem with zero doc coverage anywhere and proposes scaffolding a new doc from the genre-template library. Both fold their output into this step's [doc] collection and the Step 9 batch table alongside items 1-3 above.
→ Collect each needed update as: [doc] {file} — {what to add/change}
→ Collect registry updates as: [registry] {action} — {detail}
Registry updates are included in the Step 9 consolidated batch table alongside other config changes.
CLAUDE.md describes how to work in this codebase — patterns to follow, commands to run, conventions to respect, mistakes to avoid. Every update must describe something that exists and is actively used, not aspirational improvements.
Check if the work introduced project-wide conventions:
.claude/rules/Before adding to CLAUDE.md, check the size budget — keep it concise. Move detailed content to skills or rules. Route improvement ideas to a new backlog record (no stage label — the unified taxonomy's equivalent of the pre-migration "inbox" destination; per _shared/work-record.md), not CLAUDE.md — subject to the same per-item work-record-creation approval as any other new record (_shared/auto-mode-contract.md).
→ Collect each needed update as: [claude.md] {section} — {what to add/change} or [rule] {path scope} — {convention}
Capture the why behind significant decisions made during this work — distinct from decisions.md (the per-run auto-decision audit log) and the spec (which records what). Apply the 3-factor gate from _shared/decision-records.md (read it for the gate, the location convention, and the template).
[ADR-candidate]-tagged constraints in the brainstorming brief (flagged by /claude-tweaks:challenge)/build Common Step 4.5[ADR-candidate] by /claude-tweaks:deepen/review and reflection insights about approachdocs/decisions/NNNN-{slug}.md using the template in _shared/decision-records.md (find the highest existing NNNN and increment).→ Collect each as: [adr] docs/decisions/NNNN-{slug}.md — {decision title}
ADR proposals are routed through the Step 9 batch table / Review Console alongside other configuration updates — never written silently. Most wrap-ups produce zero ADRs; that is correct. ADRs are valuable because they are rare.
Analyze whether project skills need updating, and whether the work warrants a new skill — based on what was built. This step runs standalone (not batched with Step 6) because it requires reading and comparing full skill files — a heavier weight of analysis.
Unlike a pure consumer, Step 7 generates candidates from the work itself. Ledger entries (build/skill, review/skill, [skill: …]-tagged) and reflection insights are seeds that focus the analysis — but an independent, domain-scoped scan inspects the skills whose domain overlaps the changed files even when nothing was tagged, and gap detection looks for reusable patterns no skill covers. New-skill candidates are proposed when ≥2 of 3 criteria (reusability, complexity, project-specificity) are met — not all three.
For the full procedure — seed gathering (7.1), the independent scan + gap detection (7.2), the 6-dimension analysis (7.3), the ≥2-of-3 new-skill gate (7.4), quality gates (7.5), and auto/interactive stage-or-present (7.6) — read skill-curation.md in this skill's directory.
Skill curation declares "No skill updates needed" only when seeds, the independent scan, and gap detection all come up empty — never merely because no ledger entry was tagged. Staged updates and new-skill candidates surface at the Wrap-Up Review Console (Step 8.6), or the interactive batch table per skill-curation.md.
Determine:
/claude-tweaks:help for that case.Suggest running /claude-tweaks:help to see the full workflow status.
The record this run just closed is already known — record: {n} from the materialized header (the same field the close-via-merge carrier commit used). Check whether closing it unblocked anything, purely informational — this must never gate, block, or delay the wrap-up; on any error, log and continue.
work-backend: github-issues:
gh issue list --state open --json number,title,body --limit 200 > /tmp/wrapup-open-records.json
node -e "
const { parseDependencies } = require(process.env.CLAUDE_PLUGIN_ROOT + '/bin/lib/issues/record.js');
const records = require('/tmp/wrapup-open-records.json');
const closedNum = ${CLOSED_NUM};
const dependents = records
.map((r) => ({ number: r.number, title: r.title, blockedBy: parseDependencies(r.body) }))
.filter((r) => r.blockedBy.includes(closedNum));
require('fs').writeFileSync('/tmp/wrapup-dependents.json', JSON.stringify(dependents));
"
If dependents is non-empty, check whether every other blocker (excluding the record that just closed) is also already resolved — one batch call, not one query per blocker id:
gh issue list --state all --json number,state --limit 200 > /tmp/wrapup-all-states.json
node -e "
const dependents = require('/tmp/wrapup-dependents.json');
const allStates = require('/tmp/wrapup-all-states.json');
const stateOf = new Map(allStates.map((i) => [i.number, i.state]));
const closedNum = ${CLOSED_NUM};
const unblocked = dependents.filter((d) => d.blockedBy.every((b) => b === closedNum || stateOf.get(b) === 'CLOSED'));
require('fs').writeFileSync('/tmp/wrapup-unblocked.json', JSON.stringify(unblocked));
"
work-backend: local-files:
node -e "
const { queryRecords, readRecord } = require(process.env.CLAUDE_PLUGIN_ROOT + '/bin/lib/issues/local-store.js');
const glob = require('fs').readdirSync('specs').filter((f) => /^\d+-.*\.md\$/.test(f));
const closedNum = ${CLOSED_NUM};
const openRecords = queryRecords('specs', {}); // excludes closed by default — correct for finding open dependents
const dependents = openRecords
.map((r) => ({ id: r.id, title: r.title, blockedBy: r.facets.blockedBy || [] }))
.filter((r) => r.blockedBy.includes(closedNum));
const isBlockerResolved = (id) => {
const file = glob.find((f) => f.startsWith(id + '-'));
if (!file) return true; // already gone — treat as resolved
const r = readRecord('specs/' + file);
return r.facets.closed === true;
};
const unblocked = dependents.filter((d) => d.blockedBy.every((b) => b === closedNum || isBlockerResolved(b)));
require('fs').writeFileSync('/tmp/wrapup-unblocked.json', JSON.stringify(unblocked));
"
For every record in the resulting /tmp/wrapup-unblocked.json: log one line to decisions.md (AUTO {time} — Step 8: closing #{n} unblocked #{m} ("{title}"). Reversibility: n/a (informational).), and carry it forward as this run's "newly unblocked" signal — feeds the Next Actions table below and the Pipeline Summary's Key Outputs.
Run the resolve gate from /claude-tweaks:ledger (see ledger skill for the three-phase procedure: Phase 1 fix-exhaust silently → Phase 2 present remainder for per-item user decision → Phase 3 apply). If the ledger doesn't exist (standalone wrap-up, or work predating the ledger), skip this gate.
Hard requirements:
fix / defer / accept decisions. Status acknowledged (e.g., ops items the user has read — each one stages a work record proposal, resolved via the Review Console's own mandatory per-item approval) may be bulk-staged via a single explicit "I've read every item" choice, since the actual record creation still gets its own per-item gate downstream. Never bulk-resolve fix / defer / accept. Never assume "obvious" defers. Never offer a "Fix all (Recommended)" or "Defer all" shortcut — those bias the user toward whichever bulk action is easier to type.auto mode does NOT silence this gate.parked and backlog are valid stage destinations for a new work record, but every individual item requires an explicit per-item user choice — no record is ever staged autonomously.The fast path applies only when every ledger item already has terminal status (fixed, deferred, accepted, acknowledged, observation) at gate entry. If a single item has status open, the fast path does NOT apply — Phase 1 → Phase 2 → Phase 3 must run in full sequence without exception. Before reporting completion, check every acknowledged item for a staged proposal (a producer can create an item pre-set to acknowledged, bypassing Phase 3 entirely — e.g. build/worktree-setup.md's auto-mode divergence entry): stage one now, per ledger/resolve-gate.md Phase 3's Acknowledge disposition, for any that lack one. Then report: "All {N} ledger items resolved. No open items." and proceed to Step 9.
Phase 2 is on the "What auto does NOT silence" list in _shared/auto-mode-contract.md — it is never skipped, regardless of auto state, when any open item exists.
Ops items represent infrastructure changes the user needs to action post-merge — bulk-acknowledging them risks the user not reading them. Present each item, and require explicit confirmation rather than a shortcut that defaults to bulk action:
The following ops items need acknowledgment. These represent infrastructure changes you need to action post-merge — read each one before choosing:
| # | What | Where |
|---|------|-------|
| 1 | {description} | {source} |
Unattended-tier auto-acknowledge: if unattended-tier: on (see _shared/unattended-tier.md),
skip the AskUserQuestion below entirely — for every item, stage a record proposal and update
status to acknowledged per ledger/resolve-gate.md Phase 3's Acknowledge disposition, log
AUTO {time} — Ops acknowledgment: {N} items auto-acknowledged, staged for filing. Reversibility: high. to
decisions.md, and continue to Step 8.6. Otherwise, present the block below.
Call AskUserQuestion with question: "How do you want to handle these ops items?", header: "Ops items", multiSelect: false — neither option's label is marked as the default:
label: "Acknowledge all", description: "I've read every item"label: "Show details", description: "I have questions about specific items"After option 1, apply ledger/resolve-gate.md Phase 3's Acknowledge disposition to every item — stage a record proposal per item and update status to acknowledged. The actual record creation is a separate, mandatory per-item approval at the Review Console's Queue writes section (bulk-acknowledging here only stages the proposal; it does not silently create N records). After option 2, surface each item with full detail and apply the same per-item Acknowledge disposition on confirmation.
The Review Console is the second bookend of the pipeline (see _shared/auto-mode-contract.md). Runs in auto or hybrid mode when a pipeline run directory exists. Skipped in interactive mode and in standalone wrap-up. Reads decisions.md, staged/, and config.yml from the run directory, then presents one consolidated batch table with up to six sections (Auto-applied / Pending review / Low-confidence findings / Contested findings / Skill updates / Configuration updates) and three actions (Approve all / Override / Stop). The two coordination-derived sections (Low-confidence findings, Contested findings) render only when non-empty.
Multi-spec defer: when MULTISPEC_REVIEW_DEFER=1 is set by /flow multi-spec orchestration, skip the per-spec console — the consolidated end-of-run console at /flow handles all approvals across every spec in the run. Leave staged/ and decisions.md untouched, append a "deferred" log entry, and proceed to Step 9.
Empty-console fast path: if decisions.md has zero entries AND staged/ is empty AND no skill/config updates exist, skip the console entirely and proceed to Step 9.
For the run-directory resolution sequence, the multi-spec defer protocol, the full console template with all six section tables (including the conditionally-rendered Low-confidence and Contested findings sections), approval/override/stop semantics, and the sort-order requirement, read review-console.md in this skill's directory.
## Wrap-Up: {"Record #{n}" when a materialized header exists, else "Spec {number}"} — {title}
{Origin: {origin} — record mode only, the materialized header's origin field: by:code-health / by:harness-health / by:journey-health / by:docs-health / by:capture, or "human" when absent. Omit this line entirely for legacy spec-file-mode runs.}
### Reflection Insights
1. {insight} → {destination}
(or: No significant insights.)
### Implementation Status
- {section}: {status}
Overall: {X}% complete
### Cleanup Actions (planned in Step 5; executed in Step 10)
See `cleanup-procedures.md` for the canonical cleanup list. Render only rows whose Condition holds (e.g., no worktree, no design caches). Under `MULTISPEC_REVIEW_DEFER=1`, items marked deferred in `cleanup-procedures.md` are skipped here too.
- [ ] Leftover work: {recommendation}
### Configuration Updates (from Step 6)
| # | Type | Target | Change |
|---|------|--------|--------|
| 1 | {doc/claude.md/rule/adr/docs-health-issue} | {target} | {what to add/change} |
| 2 | ... | ... | ... |
(or: No configuration updates needed.)
### Manual Steps Required
| # | What | Where | Status |
|---|------|-------|--------|
| 1 | {description} | {source} | Filed as #{n} |
(or: No manual steps — nothing to do outside the codebase.)
> Complete these after merging. Each row is a real, trackable record (`ledger/resolve-gate.md`'s `Acknowledge` disposition) — not just a note in this transcript.
### Skill Updates
Resolved in Step 7 — {N} updates applied / 0 updates needed.
### Actions Performed
| Action | Detail | Ref |
|--------|--------|-----|
| Operational | Closed record #{n} via merge (`Fixes #{n}`) — no local file to delete | `{hash}` |
| Operational | Deleted spec `specs/{N}.md`, updated `specs/INDEX.md` (legacy spec-file-mode alias only) | `{hash}` |
| Operational | Deleted plans `docs/plans/{files}` | — |
| Operational | Deleted ledger | — |
| Operational | Deleted design wrapper caches (`*-audit.json`, `*-recommendations.json`, `*-declined.json`) | — |
| Operational | Removed worktree `{path}`, deleted branch `{branch}` | — |
| Ledger fix | {item} ({phase}) — {resolution} | `{hash}` |
Generate from: cleanup actions in Step 10, config/skill updates applied, ledger items resolved in Step 8.5, and, when present, the run dir's `events.jsonl` (hook-recorded commit breadcrumbs — hash reflects HEAD at hook time, not verified against commit success — and contract violations).
(Next Actions are rendered as a top-level section after Step 10 — see `## Next Actions` below. Do NOT render them here in the per-spec summary template.)
Conditional batch decision — only present when the Wrap-Up Review Console (Step 8.6) did NOT run:
auto or hybrid mode with a pipeline run directory) → cleanup + config items were already approved at the Review Console. Skip this batch table and proceed to Step 10 execution. Rendering a second batch table here duplicates the Review Console and violates the "one decision per message" + bookend ("at most two stops in auto") promises.Render the cleanup rows from cleanup-procedures.md's canonical list (filtered by Condition), followed by configuration update rows from Step 6:
| # | Type | Action | Details |
|---|------|--------|---------|
| 1 | cleanup | {row from cleanup-procedures.md canonical list} | {details} |
| ... | cleanup | ... | ... |
| N | config | {doc/claude.md/rule/adr/docs-health-issue} | {what to add/change} |
The table renders as markdown, as above. Immediately below it, call AskUserQuestion with:
question: "How do you want to apply these changes?", header: "Apply changes", multiSelect: falselabel: "Apply all (Recommended)", description: "Apply all cleanup and configuration items"label: "Override specific items", description: "Tell me which #s to change"If the user chooses to override, let them pick which items to skip or change.
After presenting the summary, output an explicit closure line — record mode (materialized header present):
Work archived. Record #{n} closes via this merge (or the wrap-up commit, in current-branch mode); its plans and ledger have been deleted. The code and learnings remain.
Legacy spec-file-mode alias (no materialized header):
Work archived. Spec {N}, its plans, and ledger have been deleted. The code and learnings remain.
This signals clearly that the lifecycle is complete — there's nothing left to do for this spec.
Next Actions are rendered as a top-level ## Next Actions section after Step 10's verification — see the section near the end of this file. They replace the old single-line handoff with a context-signal-driven table.
Execute the cleanup planned in Step 5 (canonical list in cleanup-procedures.md) plus the configuration / skill updates approved at the Review Console (Step 8.6) or batch decision (Step 9). The 8 cleanup items, in execution order, are defined in cleanup-procedures.md's canonical list — do not re-enumerate here. Filter rows by Condition.
MULTISPEC_REVIEW_DEFER branch: When $MULTISPEC_REVIEW_DEFER=1 is set, Step 10 SKIPS the state-changing cleanups marked deferred in cleanup-procedures.md (items 3 Design caches, 4 Git worktree, 6 Ephemeral dev server, 7 Issue claim release, 8 Pipeline run dir archival). Those defer to /flow's consolidated multi-spec Review Console at end-of-run, which has authority to apply or override them across all specs in the run. Step 10 still executes the idempotent cleanups (items 1 Execution plans, 2 Open items ledger, 5 Record/spec lifecycle) — those do not interact with parent-orchestrated cleanup.
After the cleanup, also apply:
[doc] {file} — Create: … row (wrap-up's own D2 gap-detection, docs-health-integration.md), scaffold the new file from the matching section of skills/_shared/diataxis-genre-templates.md and fill in real content from this work's session context, then register it in docs/REGISTRY.md if a registry existsdocs-health-integration.md's D1) approved at the Console or batch, re-run validate-findings without --dry-run and file each surviving payload via gh issue create, per that file's filing proceduredocs/decisions/NNNN-{slug}.md files (Step 6.3) using the template in _shared/decision-records.md, and add them to docs/REGISTRY.md if a registry exists/review only produced a recommendation), then apply demo:pending and post the Verification Brief; see verification-brief.md in this skill's directory for the full bootstrap, safety-net, sourcing, and posting procedureCommit with a message summarizing the wrap-up actions. When the run is current-branch mode
and a materialized header exists for this spec (${RUN_DIR}/work/*-spec.md — its record:
field is the issue number), include one Fixes #{issue} line per resolved issue in this commit
message — it is the closing-keyword carrier for current-branch runs (see
cleanup-procedures.md Section C); GitHub closes the issues when the commit reaches the
default branch. A legacy spec-file-mode run (no materialized header) carries no closing
keyword — there was never an issue to close.
Before emitting the closure line, confirm every approved action actually ran:
ls specs/{N}*.md returns nothing (or git status shows the status edit committed)git log -1 --stat specs/INDEX.md shows this run's commitls docs/superpowers/plans/*{spec-slug}* docs/plans/*-ledger.md returns nothing*-audit.json / *-recommendations.json / *-declined.json for this spec remain in docs/plans/.claude-tweaks/pipelines/{run-id}/ is gone; .claude-tweaks/pipelines/archive/{run-id}/ exists, with the work/ subdirectory (when present) still git-tracked at its new path (skipped when MULTISPEC_REVIEW_DEFER=1)git worktree list no longer shows the feature worktree pathgit log {default-branch} --grep="Fixes #{issue}" shows the carrier commit for each resolved issue once merged (or git log {feature-branch} --grep=... if the branch is still open under "keep as-is" or a pending PR)work-backend: github-issues: gh issue view {issue} --json labels -q '.labels[].name' includes demo:pending and the issue's last comment contains ## Verification Brief with a ### Confirmed section; work-backend: local-files: the record's body contains ## Verification Brief with a ### Confirmed section and its frontmatter has acceptance: pending. For a testable record, confirm the safety-net gate actually resolved (no high/critical visual-review finding left unfixed) before this line was reached.If any approved action did not land, do NOT emit the closure line. Surface the gap (BLOCKED — cleanup step {N} did not complete: {reason}) and stop.
/claude-tweaks:review should have been run before /claude-tweaks:wrap-up — this skill assumes code quality is verifiedgit reset or git checkout . to discard changes. If the conflict involves spec or INDEX.md files being deleted by wrap-up but modified on main, prefer the deletion (the spec is complete).When invoked by /flow ($PIPELINE_RUN_DIR is set), omit this block — the parent /flow renders its own Pipeline Summary + Next Actions after Step 9.
When invoked directly by a user (standalone wrap-up), resolve 2-4 options based on context signals; always include the "next unblocked spec" option when one exists so the user doesn't have to run /help to find it. The signal-to-option lookup table below stays as-is — the assistant's own logic for picking which options apply, never itself shown to the user or converted into an AskUserQuestion option:
| Signal | Option |
|---|---|
| Next spec exists (Step 8) | /claude-tweaks:flow {N} — full pipeline on spec {N}: "{title}" (Recommended) |
Newly unblocked records (Step 8's dependent check — /tmp/wrapup-unblocked.json, one option per entry) | /claude-tweaks:flow #{N} — record #{N} "{title}" now unblocked by this closure (bare {N} under work-backend: local-files) |
| Always | /claude-tweaks:help — full pipeline status |
Once the signals are resolved, call AskUserQuestion with question: "What's next?", header: "Next step", multiSelect: false, and:
label: "Full pipeline (Recommended)", description: "/claude-tweaks:flow {N} — full pipeline on spec {N}: \"{title}\""/tmp/wrapup-unblocked.json, up to the tool's option cap) — label: "Pipeline #{N}", description: "/claude-tweaks:flow #{N} — record #{N} \"{title}\" now unblocked by this closure"label: "Pipeline status", description: "/claude-tweaks:help — full pipeline status"When $PIPELINE_RUN_DIR is set, /wrap-up is running inside a /flow pipeline. In that case:
## Next Actions block at the end of Step 9 — the parent /flow renders its own pipeline summary.$MULTISPEC_REVIEW_DEFER — if set, skip the per-spec console and let /flow's consolidated console handle approvals.When $PIPELINE_RUN_DIR is unset, /wrap-up runs standalone — render Next Actions as usual.
| Pattern | Why It Fails |
|---|---|
| Running wrap-up before review | Wrap-up assumes code quality is verified — skipping review means capturing learnings from unvalidated work |
| Deleting specs that aren't 100% complete | Partial specs need leftover work routed, not deleted — use Step 4 first |
| Adding every insight to CLAUDE.md | CLAUDE.md has a size budget — route detailed content to skills, rules, or memory files |
| Skipping reflection for "simple" work | Simple work still surfaces surprises and near-misses worth capturing |
| Keeping design docs and plans after wrap-up | Consumed artifacts create stale references — the spec and code are the durable records |
| Silently dropping insights with no obvious destination | Every insight gets an explicit decision — even "don't capture" requires a stated reason from the user |
| Completing wrap-up with open ledger items | The nothing-left-behind gate exists to prevent dropped work — resolve every item before presenting the summary |
| Scanning the entire skill library on every wrap-up | Step 7's independent scan is bounded to the ~5 skills whose domain overlaps the changed files (~2 under a fast-lane ceremony profile; plus seeded skills) — a whole-library audit on every wrap-up wastes effort and produces noise. Domain-scoped scanning is expected; whole-library scanning is the anti-pattern |
| Skipping skill curation because nothing was ledger-tagged | Step 7 generates candidates from the work itself — the independent scan and gap detection run even with zero seeds. Declaring "no skill updates needed" just because no entry was tagged is the failure this step exists to fix |
| Proposing generic skill updates with no concrete anchor | Every skill update must trace to a ledger entry, a reflection insight, or a specific changed-file observation from the independent scan — updates with no anchor are indistinguishable from hallucinated ones |
| Mixing skill updates into the doc/CLAUDE.md batch table | Skill updates require full file reads and Update Mode patches — they get their own decision table in Step 7 |
| Writing an ADR for every decision | ADRs are valuable because they are rare — Step 6.3's 3-factor gate (hard-to-reverse AND surprising AND a real trade-off) keeps them so. Most wrap-ups produce zero ADRs, and that is correct |
Treating demo:pending as optional for "trivial" record-mode work | The Acceptance axis applies uniformly — /claude-tweaks:demo's batch view is where triviality gets a fast path, not wrap-up's labeling step |
| Skill | Relationship |
|---|---|
/claude-tweaks:review | Must pass before /claude-tweaks:wrap-up — handles verification, code review, and simplification. Skill-routed ledger entries from lens 3a (phase review/skill) and /reflect hindsight findings tagged [skill: …] (phase review/hindsight) feed into Step 7 skill analysis. |
/claude-tweaks:test | Indirect dependency — /test passes before /review, which passes before /wrap-up. Open QA ledger entries (test/qa phase) carried forward from /test surface in Step 8.5's resolve gate as items requiring per-item user decision. |
/claude-tweaks:review (visual modes) | Visual complement — findings from visual review may feed into wrap-up's reflection lenses |
/claude-tweaks:visual-review | verification-brief.md's Step 2.5 safety-net gate invokes /claude-tweaks:visual-review directly when a testable record reaches wrap-up without a full pass already having run (standalone /review in code mode, or /build run outside /flow) — reuses /review Step 6's own mode resolution, never a separate implementation. |
/claude-tweaks:reflect | Invoked BY /wrap-up (Step 3) in full mode, or light mode under a fast-lane ceremony profile. Full mode handles all four reflection lenses, tradeoff review, insight routing, and ledger writes with phase wrap-up; light mode narrows to two lenses and skips the tradeoff review. |
/claude-tweaks:capture | /claude-tweaks:wrap-up may file a new backlog record directly (no stage label) for genuinely new ideas discovered during implementation — the same recordPayload composition /capture itself uses, without going through this skill |
/claude-tweaks:demo | /claude-tweaks:wrap-up applies demo:pending and posts the Verification Brief (Step 10, verification-brief.md) — record mode only, gated on a clean visual-review pass (Step 2.5's safety net). /claude-tweaks:demo later resolves the label to demo:approved/demo:changes-requested and, on the latter, files a linked follow-up record. |
Work records (parked, via leftover routing) | /claude-tweaks:wrap-up routes leftover work here — a new record with a Trigger: body line and the parked label (origin spec/record, files, trigger); see leftover-routing.md in this skill's directory |
/claude-tweaks:help | /claude-tweaks:wrap-up suggests running /claude-tweaks:help to see what's unblocked |
/claude-tweaks:tidy | /claude-tweaks:wrap-up cleans artifacts for a single spec — /claude-tweaks:tidy does periodic bulk cleanup |
/claude-tweaks:build | Runs BEFORE /claude-tweaks:review — produces the code and journeys that wrap-up reflects on. build/skill ledger entries from Step 4.5 feed into Step 7 skill analysis (alongside [skill: …]-tagged entries from any other phase). |
/superpowers:finishing-a-development-branch | When build used worktree git strategy, wrap-up Step 10 verifies the feature branch was completed (merged, PR created, or discarded), then removes the worktree directory and deletes the merged branch |
/claude-tweaks:init | Step 7 references skill-template.md for Update Mode format and quality gates. /wrap-up Step 6 maintains the doc registry created by /init Phase 8.5. |
/claude-tweaks:ledger | Manages the open items ledger. /wrap-up appends reflection insights (Step 3), runs the resolve gate (Step 8.5), plans deletion in Step 5, and executes deletion in Step 10. |
/claude-tweaks:design | /wrap-up plans cleanup of the design wrapper's per-spec caches (*-audit.json, *-recommendations.json, *-declined.json in docs/plans/) in Step 5 and executes the deletion in Step 10 alongside the ledger. |
/claude-tweaks:flow | Invoked BY /flow as the pipeline's final step; flow waits for /wrap-up's Review Console (Step 8.6) before archiving the run directory. Multi-spec runs set MULTISPEC_REVIEW_DEFER=1 so per-spec wrap-ups defer to flow's consolidated console. |
/claude-tweaks:deepen | Interface trade-offs /deepen flags [ADR-candidate] are picked up by Step 6.3 and run through the 3-factor gate for possible ADR creation |
_shared/decision-records.md | Canonical 3-factor ADR gate, location convention, and template applied by Step 6.3 |
_shared/criteria-docs-diataxis.md, docs-health-integration.md | Step 6.1 item 4 applies this shared judgment inline to docs touched by the current work (same reuse pattern as _shared/harness-health-analysis.md in Step 7), and detects missing documentation from the diff. |
_shared/auto-mode-contract.md | Single source of truth for auto-mode behavior — read before adding any auto-mode handling |
_shared/issue-claims.md | Cleanup item 7 (Section E of cleanup-procedures.md) releases claims for specs with a materialized header, with the branch outcome as the release reason. |
/claude-tweaks:harness-health and _shared/harness-health-analysis.md | Step 7's 7.3-7.5 apply this shared procedure instead of an inline copy — sharing its judgment logic and its .claude-tweaks/harness-health/ cursor/cache state with /claude-tweaks:init and the standalone /claude-tweaks:harness-health routine. |