Commit

/commit — Smart Commit & Ship

> Smart commit: auto-selects checks by change type, generates a conventional commit message. > --ship mode: one-shot pipeline — merge base → full verify → PR creation. > Zone→Check mapping: see _shared/CHECK-ROUTING.md (shared with /verify). > Verification (TypeScript/Bun projects): bunx tsc --noEmit + bun test + bun build . Adapt to your project's actual toolchain.

Trigger

/commit or /commit or /commit --ship or /commit --amend

Workflow

Step 0: Scope Drift Check (before checks)

1. Read the active-plan field in your current-state file (e.g. _NEXT.md)
2. git diff --stat to view changed files
3. Changed files outside active-plan scope → warn (non-blocking): ⚠️ Scope drift: {file} not in [{plan}] scope
4. No active plan or no current-state file → silently skip

Step 1: Analyze Changes

1 merged Bash call:

echo "=== STATUS ===" && git status -s && echo "=== STAGED ===" && git diff --cached --name-only && echo "=== UNSTAGED ===" && git diff --name-only

Categorize ALL changed files into zones:

| Zone | Pattern | Required Checks | |------|---------|-----------------| | 📦 src | src/**/*.ts | bunx tsc --noEmit + bun test | | 🗃️ schema/migration | schema source + migration files | bunx tsc --noEmit + bun test (full) + manual migration safety review | | 🧪 test | test/**/*.test.ts | bun test | | 📄 docs | docs / config-only | (none) | | ⚙️ config | package.json, tsconfig.json, build config | bunx tsc --noEmit | | 🔀 mixed | src + anything else | Union of all applicable checks |

Step 2: Run Checks (parallel)

Based on zone detection, issue all Bash calls in parallel in the same message:

| Zone contains | Required check | Parallel? | |-----------|-------------|-------| | src / config / schema | bunx tsc --noEmit | ✓ | | src (changed .ts) | bun test | ✓ | | schema / migration | bun test (full, schema ripples widely) | ✓ | | docs / config only | none — skip | — |

Short-circuit rule (reduces redundant bun test): if the most recent /verify within this conversation PASSed within 5 minutes + no new source-file changes since + it covers this zone → skip Step 2's bun test (tsc still runs). If any condition fails → run full. Safe default: run full.

If ANY check fails → 🛑 STOP. Enter Violation attribution:

Step 2.1: Violation attribution (when a check fails)

1. Collect the file paths from the failure output (e.g. [FAIL] test/queue.test.ts → double-claim returns empty)
2. Compare against git diff --name-only (this set of changes)
3. Classify:

🔴 Introduced by this change (must fix before commit): [FAIL] src/queue/dispatch.ts → race
🟡 Pre-existing (not from this session, still blocking): [FAIL] test/foo.test.ts — pre-existing

1. Both classes must be fixed (no downgrade). Pre-existing → additionally log to your error journal tagged [pre-existing], surfaced at the next session start.

Step 3: Stage Files

• User-specified files → stage those; otherwise stage all modified/untracked
• NEVER stage: .env*, node_modules/, generated index/cache dirs, trace dirs, worktree dirs
• WARN + confirm before staging: harness settings files, build/ORM config

Step 4: Generate Commit Message

Format: type(scope): description (Chinese)

Type: new file→feat / modify existing→fix/refactor/chore / schema·migration→feat(db)·fix(db) / docs→docs / harness config→chore(harness) Scope: derive from the top-level module path (e.g. src/routes/**→routes, src/queue/**→queue, schema/migration→db, docs/**→docs, harness config→harness); multiple scopes → the most important one

Append co-author:

Co-Authored-By: Claude Opus 4.8 (1M context) 

Step 5: Commit

git commit -F- 
EOF

--amend (only when the user explicitly passes it): git commit --amend -F- as above.

Step 6: Post-Commit

• git log -1 --oneline to confirm

Examples

User: /commit
→ Detects: migration + schema source
→ Runs: bunx tsc --noEmit + bun test (full)
→ Message: "feat(db): inbox 表 + HMAC 幂等 schema"

User: /commit 修复重复派发
→ Detects: src/queue/dispatch.ts + test/queue.test.ts
→ Runs: bunx tsc --noEmit + bun test test/queue.test.ts
→ Message: "fix(queue): ready→running CAS 原子抢占防双派发"

User: /commit
→ Detects: docs/ only
→ Skips checks → Message: "docs(plan): ..." → commits directly

Safety Rules

1. NEVER commit .env, credentials, secrets, generated index/cache dirs
2. NEVER amend unless user says --amend
3. NEVER force push — only local commit (push requires explicit user request)
4. If checks fail, report + STOP — do not retry automatically

---

Ship mode (--ship)

> Full release pipeline. One-shot from commit to PR.

Ship Step 0: Pre-flight

1. Current branch is main → AskUserQuestion to confirm (solo projects often commit straight to main; branch only when a real PR flow is wanted)
2. git status — uncommitted changes present → run Step 0-6 first
3. Readiness Dashboard: Branch | Base: main | Commits: N | Files: M | tsc ✅/❌ | tests ✅/❌

Ship Step 1: Merge Base

git fetch origin main && git merge origin/main --no-edit

Conflicts → report conflicting files → AskUserQuestion (resolve/abort)

Ship Step 2: Full Verify (parallel)

| # | Command | Timeout | |---|------|------| | 1 | bunx tsc --noEmit | 60s | | 2 | bun test 2>&1 \| tail -20 | 180s | | 3 | bun build | 60s |

Failure Ownership Triage: git diff main...HEAD --name-only → introduced by this branch=must fix STOP; already exists in base=tag [pre-existing], log to error journal, non-blocking.

Ship Step 3: Test Coverage Audit

bun test --coverage → extract coverage. Compare against git diff main...HEAD changed files: new/changed without a corresponding test → warn (non-blocking, report only).

Ship Step 4: Plan Completion Audit

Read your current-state file's active-plan commitments vs git diff main...HEAD actual delivery: DONE / PARTIAL / NOT DONE / EXTRA. NOT DONE → AskUserQuestion (continue/next time/cancel ship).

Ship Step 5: Push & PR

git push -u origin HEAD
gh pr create --title "type(scope): 中文描述" --body "$(cat 
EOF
)"

Output the PR URL.

Ship Output

## Ship Report — {date}
| Step | Status | Details |
| Merge Base | ✅ | No conflicts |
| Verify | ✅ | tsc 0, all pass, build clean |
| Coverage | ⚠️ | {N} new files without tests |
| Plan Audit | ✅ | {n}/{m} DONE |
| PR | ✅ | #N created |
→ PR: {url}

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

• Author: AbyssCN
• Source: AbyssCN/oh-my-dag
• License: MIT

Install and usage instructions live in the source repository linked above.