docs: archive client host slices A/B and create Slice C intent baseline - #148
Merged
Conversation
Scope:\n- create the main openspec/specs/client-host-orchestration/spec.md by folding in the merged Slice A and Slice B deltas\n- archive the completed client-host-orchestration-doc-baseline and client-headless-event-envelope-v1 changes plus their feature aggregation docs\n- update the issue #135 master TODO ledger to mark Slice A/B done and claim Slice C\n- create the Slice C feature aggregation doc and OpenSpec artifacts for client-external-control-plane-v1\n\nRationale:\nPR #141 and PR #145 are already merged, so the repository should no longer keep Slice A and Slice B as active OpenSpec changes. Before starting external control-plane work, the host-orchestration capability needs a canonical main spec and a clean active-change set. This commit closes the previous slices, preserves their audit trail under archive paths, and establishes the docs-first kickoff baseline for Slice C around --control-stdin, canonical action bridging, and structured control responses.
Scope:\n- update docs/features/client-external-control-plane-v1.md with the newly created docs-only intent PR link\n- keep the feature aggregation doc aligned with the current review/merge-gate state\n\nRationale:\nThe Slice C kickoff branch is now published and PR #148 exists. The feature aggregation doc is the source of truth for review and merge-gate evidence, so it needs the PR link recorded immediately after creation to keep governance evidence consistent.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ed2fc194c7
ℹ️ 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".
Scope:\n- restore the Slice B normative guardrails that were dropped when folding the archived deltas into openspec/specs/client-host-orchestration/spec.md\n- add back the requirements that chat rejects headless-only flags and incompatible headless/legacy flag mixes fail with deterministic parameter errors\n- record the review thread link in the Slice C feature evidence doc\n\nRationale:\nThe canonical host-orchestration main spec must preserve the regression boundary established by Slice B. Without these two guardrails, a future CLI change could silently reintroduce fallback behavior while still appearing spec-compliant. This commit restores the missing normative constraints and keeps the review evidence linked from the active Slice C aggregation doc.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is a docs-only intent PR for the next Issue #135 slice.
It does three things in one governance-safe step:
client-host-orchestrationby folding in the merged Slice A/B deltasclient-external-control-plane-v1) with claim ownership, feature evidence, and OpenSpec artifactsWhy
PR #141 and PR #145 are already merged, so the repository should no longer keep Slice A and Slice B as active OpenSpec changes.
Before implementation can proceed on Slice C, the repo needs:
openspec/specs/client-host-orchestration/spec.mdWithout this cleanup, the host-orchestration initiative remains split across two merged-but-active deltas and has no main spec for future slices to modify.
Scope
Included in this PR:
client-host-orchestration-doc-baselineclient-headless-event-envelope-v1docs/features/archive/openspec/specs/client-host-orchestration/spec.mdclient-external-control-plane-v1Not included in this PR:
--control-stdinimplementationTODO / Change Mapping
CCLI-001,CCLI-002CCLI-003,CCLI-004CCLI-005,CCLI-006client-external-control-plane-v1Validation
Executed:
openspec listopenspec validate client-external-control-plane-v1 --type change --strict --json --no-interactive./scripts/ci/check_governance_evidence_truth.shResults:
client-external-control-plane-v1is recognized as a new active change (0/7 tasks)0 issuesMerge Gate Intent
This PR is intended to satisfy the docs-only kickoff gate for Slice C.
After this PR is merged to
main, the next implementation step is to land the external control plane MVP for:approvals:list/poll/grant/deny/revokemcp:list/reload/show-toolskills:liststatus:getvia
--control-stdin.