Repository navigation
Paraphrase owner quotations in the new-WSL layer-consensus record and regenerate the manifest and handbook - #1159
Closed
seathatflowsinourveins wants to merge 1 commit into
Conversation
The GPT read of #1128 at bd3eecf (blocker 1, P1) found owner quotation spans in the generated handbook's agent-messaging record. They came from the layer-consensus record: assemble_manifest.py copies its rows and amendments into the definitive manifest, and build_new_wsl_handbook.py copies each manifest slot whole into the handbook, so the fix is in the canonical input and the outputs are regenerated from it. Layer-consensus record (consensus.json): six owner quotation spans in five fields become dated, attributed paraphrases of the form: the owner, <date> (paraphrased): ..., with the date taken from each record's own context: - authorization: verbatim -> paraphrase (2026-10-02) - amend_rows[4] agent-messaging amendment: owner_verbatim -> owner_paraphrase (2026-10-02); its decision, text, arms and experiment contract are unchanged - wave2 context-supply interim authority: verbatim -> paraphrase (2026-10-03) - wave5 agent-messaging owner default: two spans in its reason (2026-10-02 and 2026-10-03) and one in its posture (2026-10-03); the public record of the answer stays linked by URL Regenerated through the existing generators: - assemble_manifest.py (write; --check exit 0): definitive-manifest.json - render_tables.py --check: the 2026-10-01 record's tables stay current - build_new_wsl_handbook.py --write, then --check exit 0: docs/new-wsl-handbook.json and docs/new-wsl-handbook.md The handbook loses five owner spans: the four the read named (base lines 851, 933, 934) and one in the context-supply record's overturned interim (base line 2383), which the read did not list. Bindings: the handbook receipt's current outputs, a regenerations entry and a previous_outputs entry; manifests/evidence.json re-registers the eight changed hashed files; scripts/validate.py passes. Regression: tests/test_new_wsl_handbook.py OwnerQuotationTests checks by structure, with invented prose only: a field that holds the owner's words, or a sentence that attributes a double-quoted span to the owner or the user, fails; a dated paraphrase and separately defined protocol or contract text pass. Against the base tree it fails with 13 locators (consensus record 5, manifest 4, handbook 4); after the change all three tests pass. tests/test_new_wsl_definitive_defaults.py reads the paraphrase fields, checks the consensus decision record with the same attribution rule, and its wave-3 relay check no longer holds any of the owner's words (it checks the record's block quote instead). assemble_manifest.py: comment only, the interim authority now carries a dated paraphrase. Not changed: the owner batches' authority.verbatim fields (wave3, wave4, wave5) stay in consensus.json. assemble_manifest.py requires them (OWNER_AUTHORITY_FIELDS, line 389) and refuses a batch whose hashed relaying decision record does not contain them (check_owner_batch, lines 542-544); they never reach the manifest or the handbook. Changing that relay contract needs its own ruling. The regression keeps exactly those paths, and only while the assembler requires the field. This supersedes, for the canonical inputs, the integration resolution that chose exact quotation (evidence/artifacts/ final-architecture-round2-20261004/integration-resolutions.json:543, :630, :801). No history rewrite: earlier commits keep the old text. Co-Authored-By: Claude Opus 5.5 <[email protected]>
5 tasks done
Collaborator
Author
|
Superseded by #1161, which contains this commit and lands the scrub as one PR. |
seathatflowsinourveins
added a commit
that referenced
this pull request
Oct 16, 2026
…er's words; remove #1159's exemption Command center ruling 2026-10-15T12:38Z (P1). The no-owner-quote rule of 2026-10-14 supersedes, for every published file, the round-2 resolution to quote the owner exactly (evidence/artifacts/final-architecture-round2-20261004/integration-resolutions.json:543, :630, :801). - assemble_manifest.py: OWNER_AUTHORITY_FIELDS swaps verbatim for paraphrase and relay_record. check_owner_batch requires the paraphrase to begin "the owner, <date_utc> (paraphrased): " and the committed, hash-bound relaying decision record to state it; relay_record names the command center's private record by name and location only, and anything digest-like there is refused (a hash of private bytes would break digest custody, and a short owner sentence is open to guess-and-check). - consensus.json: the wave3, wave4 and wave5 authority.verbatim fields become dated paraphrases plus relay_record; the three batches re-bind to their decision records' new sha256. - The three relaying decision records (2026-10-04 token-full-stack-owner-default, 2604-e2e-fix-wave, final-architecture-round2) state the paraphrases; a second owner quotation in the round-2 record (a 2026-10-03 answer) is paraphrased too. - The definitive manifest and the handbook are regenerated through their builders; the handbook receipt gains a regeneration entry. - tests: #1159's temporary exemption (relay_contract_paths) is removed; a verbatim field in an owner decision is always a finding; test_the_relaying_decision_records_carry_the_paraphrase_not_the_words; the assembler's negative controls cover a missing paraphrase, the words coming back as a verbatim key, an undated paraphrase, a digest in relay_record and a paraphrase the record does not state. Fail-before (#1159 head a3d53be with these tests): the canonical-inputs check lists the three verbatim fields, the records test fails on all three batches, and the assembler test errors. After: the 8 modules that read these files, 615 tests OK (3 skips gated on tools); assemble_manifest --check, render_tables --check and build_new_wsl_handbook --check exit 0; validate rc 0. No history rewrite: earlier commits keep the old text. Co-Authored-By: Claude Opus 5.5 <[email protected]>
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.
Superseded by #1161, which contains this commit (a3d53be) and lands the whole owner-quote scrub alone (command center, 2026-10-15T13:01Z).