Skip to content

spec: flatten skip-prone substeps in /ship and child skills#7

Draft
nick-inkeep wants to merge 1 commit intomainfrom
spec/flatten-skip-prone-substeps
Draft

spec: flatten skip-prone substeps in /ship and child skills#7
nick-inkeep wants to merge 1 commit intomainfrom
spec/flatten-skip-prone-substeps

Conversation

@nick-inkeep
Copy link
Contributor

Summary

  • Spec for restructuring /ship's phase hierarchy and child skill internals to reduce sub-step skipping by AI agents
  • Audit found ~60 high-skip-risk inline substeps across ship and 7 child skills
  • Proposes a three-layer defense: explicit phases (prevent), tasks (detect), artifacts (enforce)
  • 4 files changed: ship, /spec, /implement, /qa
  • Ship goes from 9 → 11 phases; 3 substeps move into child skills; /spec always runs

Changes proposed in the spec

File Change
ship Promote 2 substeps to explicit phases (codebase understanding, ownership review). Always invoke /spec. Update task list 9→11.
/spec Accept --scope param (feature/enhancement/bugfix). Absorb implementation-readiness validation into Step 8.
/implement Add Phase 4: Verify Output (mechanical verification before handing back).
/qa Add Step 6b: Re-run quality gates after any code changes made during testing.

Key decisions

  • D1: Only 2 substeps become ship phases (3 others belong in child skills)
  • D2: Spec validation → /spec, testing exit gate → /qa, implementation verification → /implement
  • D3: Always invoke /spec (with --scope calibration instead of bypassing for lightweight specs)
  • D5: Persist codebase understanding to tmp/ship/codebase-context.md (survives compaction)

Research

Test plan

  • Review spec for completeness and accuracy
  • Validate that proposed changes are backward compatible
  • Confirm implementation phases are correctly sequenced

🤖 Generated with Claude Code

Adds a spec for restructuring /ship's phase hierarchy and child skill
internals to reduce sub-step skipping by AI agents. Three-layer approach:
explicit phases (prevent skipping), task creation (detect skipping),
and artifact gates (enforce completion).

Key changes proposed:
- Ship: 9 → 11 phases (promote codebase understanding + ownership review)
- Ship: always invoke /spec (with --scope calibration)
- /spec: accept --scope param, absorb implementation-readiness validation
- /implement: add Phase 4 (Verify Output)
- /qa: add Step 6b (verify fixes don't regress)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant