Add the operating-mode transition proposal (docs/transition.md) - #253
Conversation
Construction mode says every output stays inactive until an operator-merged operating-mode transition, but nothing in the repo told the operator what that transition actually is. This adds docs/transition.md as the operator-facing proposal: it is the checklist and the exact proposed edits, not the transition. It records what construction mode built (roadmap step -> unit path -> merged or pending PR), what "implementation-complete" does and does not mean (nothing has run against a real target; the malicious-instructions and reviewer-severity eval families are declared with no seeds; cost/latency and live default-adapter regression evidence are absent), the exact config/construction-mode.json fields the operator's own PR would change and why, the post_transition_ruleset restore (which today is a verification, because the live ruleset already matches it), what steps 7-12 unlock in order, what the operator must supply, what stays human forever, and ten open questions. Claude may not edit config/construction-mode.json, AGENTS.md, REVIEW.md, or ROADMAP.md, and this commit does not: the document only proposes. Also indexes the new file: a one-line pointer next to README's construction-mode paragraph, an appended entry at the end of ci/required-files.txt under "# Operating-mode transition proposal", and a one-line mention in RESTORE.md beside docs/components.md. Proof: - bash scripts/check-rename.sh -> clean - every path in ci/required-files.txt resolves to a regular file (missing=0) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Deploying ystack with
|
| Latest commit: |
ffb09e1
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://44e28cfe.fabrica-6yx.pages.dev |
| Branch Preview URL: | https://ystack-docs-operating-mode-t.fabrica-6yx.pages.dev |
Codex reviewer (cross-vendor, read-only)Reviewed-head: 13e7879 Posted verbatim by The new transition runbook overstates current completion and gives an incomplete transition scope that would leave the repository documentation stale. These are actionable documentation defects in a file meant to drive a safety-critical operating-mode change. Full review comments:
|
…sition PR The proposal called the twelve rollout units complete while its own table listed open PRs and units still being built; it now describes the table as current state, sets "every row merged" as the precondition for writing the transition PR, and drops the completion claim. The "nothing else" instruction for the transition PR left README and the docs saying construction mode was active after the record retired it; the PR now also updates those passages, as the repo's README/docs-in-sync rule requires. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Codex reviewer (cross-vendor, read-only)Reviewed-head: f8f9cbe Posted verbatim by The new transition checklist contains conflicting instructions about both the transition PR scope and whether pending roadmap units may remain unfinished. Those contradictions affect the operating-mode transition procedure and should be corrected before merging. Full review comments:
|
Section 3 said the transition PR touches nothing an agent could have authored, then required README and docs updates; the scope now names the record, the ruleset verification, and exactly those documentation passages. Open questions 6 and 7 offered to proceed with pending units or merge them under restored gates, contradicting the "every row merged" precondition; they now state that the pending PRs (#249–#255) land first and ask only whether any should be dropped from the roadmap. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Codex reviewer (cross-vendor, read-only)Reviewed-head: c0487b3 Posted verbatim by The new operator-facing transition document contains an internal contradiction about the status of required roadmap PRs, which can mislead the transition gate. No code execution issue was found. Review comment:
|
Codex reviewer (cross-vendor, read-only)Reviewed-head: ffb09e1 Posted verbatim by The changes add a transition proposal document, link it from README/RESTORE, and include it in the restore-critical manifest. I did not find any actionable correctness issues in the changed files. |
Construction mode ends only through an operator-merged operating-mode transition, and
config/construction-mode.jsonis a path construction may not edit. This document is the proposal the operator acts on: what construction built (step → unit → status table), what "implementation-complete" does and does not mean, the exact fields of the mode record the transition PR changes and their proposed values, the ruleset verification perpost_transition_ruleset, what becomes possible after the transition mapped to roadmap steps 7–12 in order, what the operator must provide (never pasted into chat or the repo), what stays human forever, and ten one-line open questions for the operator.One finding the document records: the mode record's
post_transition_rulesetis field-for-field identical to the live main ruleset today, so construction never loosened the branch rules; the relaxation lives in the AGENTS/REVIEW overlays and the publisher gate keyed offstatus: active, and flippingstatusis the single load-bearing change.Documentation only:
docs/transition.md(260 lines), a pointer in README next to the construction-mode paragraph, a line in RESTORE.md beside the components doc, and the path appended toci/required-files.txt.Proof run on 13e7879:
bash scripts/check-rename.shclean; every path inci/required-files.txtresolves;bash scripts/test/portable-core-schema.test.sh→ failures: 0.Construction mode: inactive output, no human gate. Nothing in this PR performs the transition.
🤖 Generated with Claude Code