Skip to content

manifest: add inception phase + imports[] field support#264

Open
leeovery wants to merge 1 commit into
mainfrom
idea/inception-pr-1-manifest-foundations
Open

manifest: add inception phase + imports[] field support#264
leeovery wants to merge 1 commit into
mainfrom
idea/inception-pr-1-manifest-foundations

Conversation

@leeovery
Copy link
Copy Markdown
Owner

@leeovery leeovery commented May 9, 2026

Summary

  • Phase 1 of the inception/discovery-map initiative — foundational only, no user-visible behaviour change. Adds inception to VALID_PHASES (placed first to reflect epic pipeline order) and restricts inception item statuses to in-progress only, enforcing the design's hard-delete model (items exist or they don't — no cancelled graveyard).
  • Adds support for a top-level imports[] array on work-unit manifests — the first non-phases array field at the work-unit root. Each entry is a {path, imported_at} record describing seed material copied into .workflows/{wu}/imports/ at work-unit creation. Documented as a convention for future top-level array fields.
  • Fixes cmdPull to use deep equality instead of reference-only indexOf, so object-shaped array entries (notably imports[] records) can be removed by value. Existing scalar pull paths unchanged — the deep-equal helper short-circuits on ===.

Test plan

  • Full manifest test suite passes (214/214 assertions, 41 new)
  • All 37 migration tests pass — no regression
  • Manual smoke test: init epic, init-phase inception, set summary/routing, hard-delete via delete {wu}.inception items.{topic}, reject cancelled status, push/pull imports[] round-trip
  • CI green on the branch

🤖 Generated with Claude Code

Phase 1 of the inception/discovery-map initiative. Foundational only —
no user-visible behaviour change.

- VALID_PHASES gains `inception` (placed first, reflecting its position
  as the entry phase for epics).
- VALID_PHASE_STATUSES restricts inception items to `in-progress`. No
  `cancelled` or `completed` — inception items follow a hard-delete
  model per the design ("items exist or they don't").
- `cmdPull` now matches array entries by deep equality so object-shaped
  records (notably the new `imports[]` entries) can be removed by value.
  Previous reference-equality match worked only for scalar arrays.
- SKILL.md documents the inception phase status and the `imports[]`
  top-level array convention with example commands.
- Tests cover inception init-phase, status validation (accept
  in-progress, reject cancelled/completed), free-form summary/routing,
  hard-delete via `delete {wu}.<phase> items.<topic>`, wildcard read,
  and imports[] push/pull/round-trip including non-array push rejection.

Co-Authored-By: Claude Opus 4.7 (1M context) <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