docs: record post-M7 sequencing decisions and authorize wave 1#17
Merged
Conversation
Adds a dated Post-spine decisions (2026-07-04) subsection under M7 in MILESTONES.md, recording the owner's sitting: M7 exit evidence posture (EVRUN-partial mandatory, EVRUN-full as named debt), the internal SDK packaging boundary (N2), TUI/dashboard as extension-point-only, the hybrid smoke target, the wave-1 merge policy, wave-1 authorization (including the M8 proposal), and the repo-plan-m7.md open-question dispositions cited against jig ADRs 0022, 0023, 0024, and 0025. Adds a proposed M8: Planning-discipline uplift milestone entry for owner ratification in this PR's review. Updates ROADMAP.md's jig Next step paragraph to reflect the decided tail direction: transport evidence and packaging boundary authorized, TUI stays extension-point-only, freeze still gated.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 49d1906c87
ℹ️ 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".
Adds item 8 to the Post-spine decisions (2026-07-04) list: the contract owner for jig's two org seams is jig's design authority (the org/product owner deciding through jig's ADR flow), receiving routed seam-shape questions and alone executing any v0 freeze at T14. Closes the G-OWN gate from the approved plan.
Review follow-up on PR #17: the Repo planning handoff list is live forward guidance, so its open conditionals are updated to reflect the 2026-07-04 post-spine decisions. The TUI bullet now states the extension-point-only decision (decision 3); the transport bullet records the decided posture (decision 1: adjunct gating the freeze, not exit); the v0-freeze bullet points at decisions 1 and 2. The closeout checkpoint and decisions list are untouched.
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.
What
Docs-only PR that makes
MILESTONES.md(plus one smallROADMAP.mdtouch) the durablerecord of the owner's 2026-07-04 post-M7 sequencing decisions, and proposes a new M8
milestone for owner ratification in this review.
This PR is human-reviewed and human-merged. It is explicitly excluded from the wave-1
autonomous-squash-merge authorization it itself records (decision 5) — do not enable
auto-merge, do not merge without explicit human sign-off.
Where it lands
MILESTONES.md— a dated "Post-spine decisions (2026-07-04)" subsection appended underM7 (after the existing "Closeout checkpoint", before "Repo planning handoff"); existing
M7 text is untouched, this only appends. Plus a new
### M8: Planning-discipline upliftentry using the file's own Milestone Record Format,
State: proposed.ROADMAP.md— only the jig layer's "Next step" paragraph, updated in place (5 lines).profile/and templates untouched.The five decisions recorded (plus the wave-1 authorization)
for M7 exit. EVRUN-partial (scripted injected agent leg, explicit Limitations section) is
sufficient for exit; EVRUN-full (Codex-driven) is a named debt retired by the post-M7
Codex-transport track (N1). M7's
Statestayscurrent, flips todoneonly whenEVRUN-partial merges. Concrete transport evidence gates the v0 freeze, not org-level exit.
Lifecycle-edge split: no codex-plugin-cc-style lifecycle edges (broker, interrupt, cleanup,
Windows process-tree) gate M7 exit — all of them gate the transport ADR and, through it,
the freeze.
private: true; no stability promise now; posture can flip to public later. The third-partyprovider-ecosystem question stays explicitly open. jig's SDK-boundary design cycle (N3) is
authorized to start once this is recorded in jig's product docs.
reopen only on recorded operator pain.
(no credentials) + a dedicated private sandbox repo (
jig-smoke-target, repo-scopedfine-grained PAT, env-only) for full landing evidence (push + PR + merge + block
surfacing). An independent secret-scan is required on any evidence record before commit.
threads resolve and
pnpm checkis green — except this PR.Wave-1 authorization (recorded as decision 6 in the new subsection): jig's
absorption-ledger absorbed-vs-open update; jig's Codex-transport evidence-capture plan plus
a dated/hashed
evidence/convention; jig's product amendment recording decision 2; adesign-flow item for self-report-only/mock-adequacy conformance vocabulary; the
EVRUN-partial evidence task (after the smoke target exists); and M8 below, as a proposal
pending this review's ratification. M6 (Learning Loop Seed) stays
proposed; pull timingremains an open org choice.
M8 proposal (for ratification in this review)
### M8: Planning-discipline uplift— primary ownerdesign-to-plan, participatingtechnical-design+jig. Ports the prev-gen authoring-standard's durable planning-qualityideas, de-productized rather than ported wholesale.
State: proposed; exit criteria andartifacts intentionally left undefined pending planning, per the file's Planning Rules
(exit criteria are set when a milestone is planned, not invented ahead of it). Carve-out:
jig's
PlanValidatorplan-ingest preflights may proceed jig-locally under M4's existingplan-validation-feedback ownership without waiting for M8.
This entry is a proposal, not a fait accompli — flag in review if the owner does not want
to ratify M8 as scoped, and I will amend or drop it.
Repo-plan open-question dispositions (jig's
docs/delivery/m7-real-providers/repo-plan-m7.md)Each verified read-only against the jig checkout's actual merged ADRs before writing:
describe()vs async proof → settled by ADR 0022.jig/docs/design/decisions/0022-phase-6-real-driver-integration.md,Decision 3: prove-then-describe — the confinement proof runs async at compose time outside
describe();describe()stays a synchronous pure getter over an already-computedattestation. No P5-pinned seam surface flexes. Matches the brief's expected disposition
exactly. Confirm holds at the transport-ADR session.
action-union / provenance record impact → settled by ADR 0023 + ADR 0024.ADR 0023Decision 2 repairs
LandingRequest.actionfrom the single mis-encoded literal'push|open-pr|merge'to the real union'push' | 'open-pr' | 'merge'— a local port-typefix, freezes nothing, dry-run goldens stay byte-identical.
ADR 0024Decision 3 widens
CandidateWorkItem.provenancefrom the single literal'jig-validated'to an origin-bearing shape, legible in the record as an additive field. Both ADRs confirm
no observability-records v0 field change was needed — matches the brief's expected
disposition, citing both ADRs the brief named.
ADR 0025Decision 1 + Contract Impact Gate Q5: a separate, non-golden integrity sidecar
(
runs/<id>/integrity.json) with an environment-keyed HMAC — no digest/HMAC field added tothe observability-records v0 contract. Matches the brief's expected "sidecar, no contract
field" disposition; the ADR explicitly names open question 3 and defers final freeze
posture to the v0-freeze checkpoint (T14).
All three dispositions matched the brief's stated expectations; no ADR contradicted the
expected reading.
Contract-owner role named (item 8, added on review — G-OWN gate)
The post-spine decisions list also names the contract-owner role: the contract owner for
the two org seams jig owns (execution-plan, observability-records) — the role that receives
routed seam-shape questions (for example, the transport session-observability question and
any port-surface flex) and that alone executes any v0 contract freeze at the T14
checkpoint — is jig's design authority: the org/product owner deciding through jig's
ADR flow (
docs/design/decisions/), per M1's seam ownership. Delivery tracks andsub-agents do not decide seam shapes (the repo-plan "must not decide" boundary).
Gate
pnpm checkgreen in the worktree:Merge
Do not merge. Do not enable auto-merge. This PR is human-reviewed and human-merged per
decision 5 above — the one wave-1 PR excluded from the autonomous-squash-merge
authorization.