attckdigital/smith/smith-finish

Smith — Session Finish

Skill Purpose

End-of-session workflow that ensures all work is committed, pushed, merged to main, specs are updated, and the workspace is clean. Prevents losing work when closing a development session.

When to Use

• User is done working and wants to wrap up
• User wants to ensure nothing is lost before ending a session
• Triggered by /smith-finish

How to Execute

Run all steps autonomously. Only stop to ask the user if a decision is genuinely ambiguous (e.g., unrelated uncommitted changes that may need separate branches).

Step 1: Inventory — Assess Current State

Gather all of the following in parallel:

# Resolve the configured base branch (falls back to main when unconfigured)
BASE_BRANCH=$(.specify/scripts/bash/get-base-branch.sh)
 
# What branch are we on?
git rev-parse --abbrev-ref HEAD
 
# Any uncommitted changes?
git status
 
# Any unpushed commits? (only if on a branch other than the base branch)
git log origin/$(git rev-parse --abbrev-ref HEAD)..HEAD --oneline 2>/dev/null
 
# What files changed vs the configured base branch?
git diff "$BASE_BRANCH" --name-only 2>/dev/null
 
# Any open PRs for this branch?
gh pr list --head $(git rev-parse --abbrev-ref HEAD) --state open --json number,title,url 2>/dev/null

Present a brief status summary to the user:

Session Status:
- Branch: <branch-name>
- Uncommitted changes: <count> files
- Unpushed commits: <count>
- Open PRs: <count>
- Files changed vs main: <count>

Decision tree based on state:

Step 1.5: Activate Workflow Tracking

Before any file is written (spec updates in Step 4, CHANGELOG/STATUS updates), create an active-workflow marker so the workflow-gate hook (PreToolUse) allows subsequent writes. smith-finish is often invoked standalone outside another workflow, so it must create its own marker.

Skip this step if the State table mapped to "Nothing to do" or "Skip to Step 6 (verify clean state)" — no edits will happen.

BRANCH=$(git rev-parse --abbrev-ref HEAD)
SAFE_BRANCH=$(echo "$BRANCH" | sed 's/[^a-zA-Z0-9._-]/-/g')
mkdir -p .smith/vault/active-workflows
cat > .smith/vault/active-workflows/finish-${SAFE_BRANCH}.yaml << EOF
workflow: smith-finish
feature: session-finish
branch: ${BRANCH}
started: $(date -u +"%Y-%m-%dT%H:%M:%S")
EOF

The marker is cleared in Step 6.5 below, regardless of whether the run reached merge or stopped at any earlier step.

Step 2: Commit

# Show what will be committed
git diff --stat
git diff --cached --stat

• Stage changed files explicitly by name (never git add -A or git add .)
• Exclude .env, credentials, secrets, and large binary files
• If there are untracked files, check if they should be included (new source files: yes, temp files: no)

Write a conventional commit message:

git commit -m "$(cat <<'EOF'
<type>: <concise description>
EOF
)"

If there are changes across unrelated areas that clearly belong to different features, ask the user:

"I see changes to both [area A] and [area B]. Should I bundle these into one commit, or split them?"

Step 3: Push

git push -u origin <branch-name>

Step 4: Spec Updates

Before creating/merging the PR, update documentation:

1. Identify changed files relative to the configured base branch:

``bash BASE_BRANCH=$(.specify/scripts/bash/get-base-branch.sh) git diff "$BASE_BRANCH" --name-only ``

1. Map files to system specs using these rules:

• services/command-center/ -> specs/system-15-command-center/spec.md
• services/email-pipeline/ -> specs/system-03-email-archive-contact-graph/spec.md
• services/sentiment-engine/ -> specs/sentiment-engine/spec.md
• services/communication-triage/ -> specs/system-05-communication-triage/spec.md
• services/content-strategy/ -> specs/system-12-content-social-engine/spec.md
• services/meeting-intelligence/ -> specs/system-09-meeting-intelligence/spec.md
• services/trend-intelligence/ -> specs/system-13-trend-intelligence/spec.md
• services/social-listening/ -> specs/system-10-social-listening/spec.md
• docker-compose.yml -> specs/system-01-core-infrastructure/spec.md
• db/ -> relevant system spec based on table names

1. For each affected spec.md: Read it, then append a dated implementation history entry describing what changed and why. Keep it concise and factual.

1. Update CHANGELOG.md with a dated entry including:

• The original prompt/intent
• Files created and modified
• Before/after details sufficient to revert

1. Update STATUS.md if system progress percentages changed.

1. Commit spec updates on the same branch:

``bash git add <spec files> CHANGELOG.md STATUS.md git commit -m "docs: update specs and changelog for <feature/fix description>" git push ``

Skip conditions:

• If only spec/doc files changed (no code), skip spec updates
• If changes are trivial (typo fixes, comment changes), skip spec updates

Step 5: PR & Merge

If no open PR exists, create one:

BASE_BRANCH=$(.specify/scripts/bash/get-base-branch.sh)
gh pr create --base "$BASE_BRANCH" --title "<type>: <short title>" --body "$(cat <<'EOF'
## Summary
<bullet points of changes>
 
## Test plan
<what was tested or needs testing>
 
Generated with [Claude Code](https://claude.com/claude-code)
EOF
)"

Merge the PR:

gh pr merge <pr-number> --squash --delete-branch

Return to the base branch:

BASE_BRANCH=$(.specify/scripts/bash/get-base-branch.sh)
git checkout "$BASE_BRANCH"
git pull origin "$BASE_BRANCH"

Step 6: Verify Clean State

Run these checks and report results:

BASE_BRANCH=$(.specify/scripts/bash/get-base-branch.sh)
 
# On the base branch?
git rev-parse --abbrev-ref HEAD
 
# Clean working tree?
git status --porcelain
 
# Any stale local branches?
git branch --merged "$BASE_BRANCH" | grep -v "$BASE_BRANCH"
 
# Up to date with remote?
git log "origin/$BASE_BRANCH..HEAD" --oneline

If Docker services were affected (code changes to any service directory):

docker compose up -d --build <affected-services>

Wait for healthy status, then report.

Step 6.5: Clear Workflow Tracking

If Step 1.5 created an active-workflow marker, remove it now via the shipped helper. Safe to run unconditionally — the helper is a no-op when the marker doesn't exist:

.specify/scripts/bash/clear-active-workflow.sh "finish-${SAFE_BRANCH}"

After this, the workflow-gate hook returns to denying ad-hoc edits — smith-finish no longer holds an active workflow.

Step 7: Final Report

Output a summary:

Session Complete:
- Committed: <commit hash> — <message>
- PR: #<number> (merged)
- Specs updated: <list of spec files>
- Services rebuilt: <list> (or "none needed")
- Branch: main (clean)

If there were any issues (failed merges, unhealthy services, skipped steps), flag them clearly:

Requires attention:
- <issue description>

Edge Cases

Multiple feature branches with uncommitted work

If the user has been switching branches and has stashed changes:

git stash list

Alert the user if stashes exist — they may contain work from this session.

Merge conflicts

If gh pr merge fails due to conflicts:

1. Report the conflict to the user
2. Do NOT force merge or auto-resolve
3. Suggest: "There are merge conflicts. Want me to resolve them, or would you prefer to handle it?"

Nothing to do

If the workspace is already clean (on main, no changes, no open PRs):

Workspace is clean. Nothing to commit, push, or merge.