Skip to content

feat(enrichment): add enrichment twin command suite#53

Merged
mdheller merged 1 commit into
mainfrom
feat/enrichment-command
Jun 12, 2026
Merged

feat(enrichment): add enrichment twin command suite#53
mdheller merged 1 commit into
mainfrom
feat/enrichment-command

Conversation

@mdheller

Copy link
Copy Markdown
Member

Summary

  • Adds prophet enrichment command with four subcommands: run, validate, status, lifecycle
  • run drives the full K3 lifecycle (INIT_SESSION → GATED_HOST_UPDATE) with dry-run support; default locus local; burst_cloud refused without explicit seed approval
  • validate runs genesis-seed-schema, path-content-conflict, placement-locus-policy, and spine gate checks
  • status shows architecture-phase DoD state including adr_0002_decisions_closed: true
  • lifecycle lists all 8 K3 stages with descriptions; marks GATED_HOST_UPDATE as approval_required: true
  • Adds three Makefile smoke checks to the validate target chain

Test plan

  • CGO_ENABLED=0 make validate passes all checks including new enrichment smoke targets
  • prophet enrichment run --seed seed:enrichment/photo-v1 --dry-run produces full lifecycle trace
  • prophet enrichment status shows adr_0002_decisions_closed: true, end_to_end_local_run: false
  • prophet enrichment lifecycle lists all 8 stages with GATED_HOST_UPDATE marked approval_required
  • prophet enrichment run --locus burst_cloud rejected with approval gate error

Wires `prophet enrichment` with four subcommands covering the full K3
mission lifecycle:
  - run:       drives INIT_SESSION → GATED_HOST_UPDATE; default locus
               local; burst_cloud refused without seed approval_profile
  - validate:  runs genesis-seed-schema, path-content-conflict, placement-
               locus-policy, and spine gate checks before any run
  - status:    shows architecture-phase DoD state including ADR-0002 closure
  - lifecycle: lists all 8 K3 stages with descriptions and approval gates

Adds three Makefile smoke targets to the validate chain. Build verified
with CGO_ENABLED=0; `prophet enrichment status` and dry-run produce correct
JSON.
@mdheller mdheller merged commit 33bb4cc into main Jun 12, 2026
11 checks passed
@mdheller mdheller deleted the feat/enrichment-command branch June 12, 2026 03:17

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d492481624

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +113 to +114
runEnrichmentGate("spine-gates",
"", []string{}), // delegated to prophet spine validate

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Wire spine validation instead of a placeholder

When prophet enrichment validate is run in any environment, this entry calls runEnrichmentGate with an empty repo and step list, which returns status: "not-yet-wired"; the aggregation below only treats "ok" as passing, so the new validate command can never report success even if the three concrete enrichment gates pass. Since this command is advertised as running the spine gate checks before a mission, invoke prophet spine validate (or don't include this placeholder in the pass/fail set) instead of hard-coding a failing gate.

Useful? React with 👍 / 👎.

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