docs(rfc-008): mark P4 DONE (P4d S7+S8 merged)#423
Merged
Conversation
S7 (#418) and S8 (#419) are merged, completing P4d and therefore P4 (P4a #397 + P4c #398 + P4d S1-S8 all merged). Flips the stale 'S7-S8 open' / 'IN PROGRESS' status in the RFC body ledger (:1216), the RFC-008/README.md phase matrix (:21), and P4-enforce-config.md status line + the S7/S8 slice-table rows. Rule 10 doc-sync; no code change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
lantiscooperdev
left a comment
lantiscooperdev
left a comment
Collaborator
There was a problem hiding this comment.
PR-level review (bot account): RFC-008 P4d doc-sync
Doc-only status reconciliation, verified against the merged state on main:
- S7 (#418) and S8 (#419) are both merged; the prior "S7-S8 open" / "IN PROGRESS" P4 status was stale. The three edited files now read DONE consistently (body ledger
:1216, README matrix:21,P4-enforce-config.mdstatus line + the S7/S8 slice rows). - The S8 slice row honestly notes the required-check toggle is tracked in #420 (the gate is advisory until branch protection is updated), which matches reality.
- No code, no deploy. The S7 CI guard pattern (
P4b.*(remains|reduced|cutover)) is not tripped by the new "superseded" wording.
No issues found. Verdict: ACCEPT (comment-level); maintainer to approve and merge.
lantiscooperdev
approved these changes
Jun 22, 2026
lantiscooperdev
pushed a commit
that referenced
this pull request
Jun 27, 2026
P5 (OpenCode enforcement plugin) merged via #424 (`f5dbaef`) + doc fix #425 (`a7f66c9`). Flip the stale `queued` P5 status in the three places that track it, mirroring #423's P4-DONE sync: - RFC-008/README.md phase matrix: the shared P5-P7 row now reads P5 DONE; P6 / P7 queued (Codex, Pi Agent still pending). - RFC body phase-index stub (:1220): same split. - P5-P7-tool-plugins.md status line: P5 DONE, P6/P7 queued. P6/P7 stay queued (shared row/file). RFC-008 remains `accepted` at the registry level (not all phases shipped), so docs/rfcs/_index.json + README.md are unchanged; em-rfc-validate passes (8/8/8 consistent). Rule 10 doc-sync; no code change. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
2 participants
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.
RFC-008 P4d doc-sync: mark P4 DONE
S7 (#418) and S8 (#419) are merged, completing P4d and therefore P4. This flips the stale status the prior docs slice (S7) could not self-mark at authoring time.
Changes (status flips only, no code)
RFC-008-decouple-enforcement-from-substrate.md(body ledger): P4 row from "IN PROGRESS / S7-S8 open" to "DONE (P4a feat(rfc-008): P4a — per-project enforce-config wired into the 3 pre_tool_use gates #397, P4c RFC-008 P4c: layer-wide active:false enforcement kill switch #398, P4d S1-S8 + ESC)".RFC-008/README.md(phase matrix): P4 row from "IN PROGRESS / S7-S8 open" to "DONE".RFC-008/P4-enforce-config.md: status line to DONE; slice table S7 to "merged (docs(rfc-008): P4d S7 phase index + Principle 12 record P4d (closes #403) #418, closed docs(rfc-008): refresh phase index for P4d (S1–S4 done) at P4d completion #403)", S8 to "merged (feat(rfc-008): P4d S8 P12 invariant CI gate (REQ-10/REQ-11) #419); required-check toggle tracked RFC-008 P4d S8 OD-3: mark p12-invariant-gate a required status check (branch protection) #420".Verification
grep -rnE "S7-S8|S8 .*open|IN PROGRESS|P4b.*(remains|reduced|cutover)"over the RFC-008 docs returns only an unrelated historical "v11 review in progress" note; no stale P4 status remains and the S7 CI guard pattern is not tripped.Rule 10 doc-sync. No code change, no deploy.