Uh oh!
There was an error while loading. Please reload this page.
Reconcile os-dev contract and pm-dispatch channel references with measured seat reality - #12440
Merged
Merged
Conversation
os-dev.md half of the channel-reality fold: label-step gated-seat fallback (MCP read-union-write-readback) and objectui phantom-label mechanism statement; zero-quota payload read tier + mcp_calls report field; slot parking on the verify-lock queue; mechanical stall self-check; pre-installed Chromium fact; commit-before-ablation companion clause; tail/PR-body read-back and placeholder-word spelling for GitHub-bound text. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JANH3y7qe3MD8aLaLXci8N
Repo-scoped probe replaces the rate-limit dual-duty claim; rate-limit becomes the one-call credential discriminator; 2026-08-25 seat matrix (subagent seats gated on reads AND writes, gh absent, MCP + git push fine); payload tier added to the default read order and degradation ladder; augment-5 queue procedure (queue-ref decisive read, update-branch conflict surfacing, 405 retry positive) supersedes the churn heuristic; blocked download recorded as not-proof-of-absence. Adds paid by same-file cuts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JANH3y7qe3MD8aLaLXci8N
The channel-boundary header now states the marks were measured in a session-gate-open seat and that gated seats (os-dev subagents and some top-level seats, 2026-08-25) get 403 on the whole repo-scoped class, reads and writes; probe-first with the rate-limit credential discriminator; the additive-label row names the measured MCP read-union-write-readback fallback; semantic-search fallback qualified for gated seats. Adds paid by same-file cuts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JANH3y7qe3MD8aLaLXci8N
PM review amend: the matrix row overclaimed (subagent seats uniformly gated) with a same-day counterexample from the seat that wrote it. Now records both readings — 2026-08-25 five-plus dev seats all gated, and the 2026-08-26 open-gate os-dev seat (additive POST 200, rate-limit 15000/hr) — concluding gate state varies per container and the repo-scoped probe is the only safe read. Net add paid by tightening the silent-downgrade row. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JANH3y7qe3MD8aLaLXci8N
os-steve
marked this pull request as ready for review
August 26, 2026 05:00
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Fixes#12123
Fixes#12111
Fixes#12229
Fixes#12204
Fixes#12226
Fixes#12133
Fixes#12188
Fixes#12341
Family dispatch (8 folded members, one PR): every member reconciles written channel/contract text with measured platform reality on the same three-file surface —
.claude/agents/os-dev.md,.claude/skills/pm-dispatch/references/platform-readings.md,.claude/skills/pm-dispatch/references/rest-channel.md. GOVERNED surface: opened as DRAFT, stays draft, no auto-merge — maintainer reviews and merges by hand.Ratchet readings (gate:
check:pm-skill-ratchet, run at head2377fb33)db8c2881).claude/agents/os-dev.mdreferences/platform-readings.mdreferences/rest-channel.mdAll adds paid by same-file cuts; every line within the 120-byte budget. Gate verdict lines: "os-dev.md is 469 lines (ceiling 470; headroom 1)", "platform-readings.md is 314 lines (ceiling 314; headroom 0)", "rest-channel.md is 87 lines (ceiling 87; headroom 0)".
Per-member coverage
mcp_callsadded as a required report field.playwright install403s — Chromium is pre-installed #12229: pre-installed Chromium fact in os-dev.md toolchain traps (never run playwright install; a CDN 403 is not evidence of absence; launch via the explicit executable path); the generalizable half in platform-readings: a blocked download is not proof of absence — check for the artifact first.git checkout HEAD -- PATHwipes the implementation under test, and #11648's fix made this more likely, not less #12204: commit-before-ablation companion clause in the reverse-verification section: a restore leg targets HEAD; if HEAD predates the fix, a perfect restore deletes the fix while every existing post-restore check passes cleanly. The HEAD-explicit restore idiom itself stands untouched.scripts/__tests__/ci-cd-pipeline-doc.test.tspins exactly this), so it exempts nothing; empty-frontmatter changeset is the declaration there; read-back verifies the write landed but is blind to phantom gates.--helpand an exit-99 message, so no agent will use it — the fix lands in skills-owned files #12341:OS_VERIFY_LOCK_SLOT=NAMEparking taught in the queueing rule itself (park keeps the original arrival stamp; without it every departure re-queues at the back), meeting the card's executable criterion — the instruction reaches an agent from its own definition, not from help text or an exit-99 message. Premise re-verified on this tree: 9 occurrences inscripts/pm/os-verify-lock.sh, control probe 0.Merge-queue readings: the arm-verification sequence in platform-readings now carries the superseding procedure — queue-ref positive is the decisive read (also when PR reads are stale for ~15 minutes after a human merge); on absence run update-branch first to surface a hidden conflict (a silent conflict never enters the queue and does not eject the armed PR); only then re-arm, where a 405 "already in the merge queue" on retry is the decisive positive. The superseded empty-fields no-op signature is explicitly recorded as refuted ("空回显≠未挂上").
Cut ledger (adds paid by same-file cuts; every dropped detail survives in git history at
db8c2881)Verification
check:pm-skill-ratchetgreen at head2377fb33(readings above; exit captured before any pipe).node scripts/pm/dispatch-gates.mjs, no paths passed; derivation header named this repo) re-run as one AND-chain under the verify lock at head2377fb33:check:agent-model-declared,check:agent-test-spelling,check:doc-authoring,check:doc-formula-expressions(after building its two prerequisite packages),check:nul-bytes,check:pm-governed-merges,check:pm-skill-id-lint,check:pm-skill-ratchet,check:skill-frame-sync— lock verdict line: "VERDICT command-exit 0 · held the lock 18s".check-governed-queue-guardis NOT MEASURED locally by design (it reads the CI event payload and refuses to pass without one); it runs in CI, and this PR's landing path (draft, no queue, no auto-merge) is the state it guards.Generated by Claude Code