Repository navigation
fix(weave): byte-scoped whole-plus-one repair; proposed answers for approval - #76
Conversation
…r-bit substitutes Preserve message -> byte -> eight circle placements and require G0 in every public pair. Remove the PR 73/74 per-bit encoders and public feedback recurrence, correct the evidence scope and hidden-key perturbation premise, and surface Q11-Q14 plus Q9 for approval without enabling their proposed answers. Keep other Weave layers unchanged.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Exact-revision repair verificationReviewed repair head The hosted Weave run All seven current head checks are completed/success, including stack consistency, the three CodeQL language analyses, and the aggregate CodeQL check (now reports no new alerts, replacing its earlier transient missing-configuration warning). Codex review activity completed on this head at 2026-10-05T08:51:00Z; fresh reads show no inline review threads and no review submissions. This is not represented as an APPROVED review. The inspected diff is restricted to eight existing Weave files. It removes the per-bit encoders and public-feedback substitute, restores byte-scoped placements and G0-plus-one public views, corrects the invalid evidence interpretation, and leaves all new question answers explicitly pending. No cipher, native geometry integration, private-key security, release, or deployment is claimed. Merge admission remains conditional on the same head/base pair and mergeability at the final read. Merging this repair does not approve Q11-Q14 or Q9. |
Authorized repair
Repairs the user's bit/byte and whole-circle corrections against base
92ccda0620c06bc46654939c061ceb2d36476cb9. Head:04760570b86be8bf905ae384aa7600ed0ee6c3a1.Honest capability boundary
This repair implements byte/placement records and the public pair shape. It does not implement native UCNS geometry, UCHC origin integration, dynamic positional encryption, KeyGen, cryptographic degeneration, or private decryption. Construction source recovery retains plaintext and is not decryption. The existing full-profile missing-operation refusal remains intact. No new substitute cipher is supplied.
File plan
research/weave/stages/eight_circle.pyresearch/weave/stages/message_origin.pyresearch/weave/tests/test_eight_circle.py,test_message_origin.pyresearch/weave/test.pyresearch/weave/EIGHT_CIRCLE.md,QUESTIONS.md,SPECIFICATION.mdLocal evidence
22 focused methods passed, zero failures/errors/skips. Includes all 256 byte values, 40,320 assignments and seven whole/part views. Focused local import used an isolated empty package initializer because the runtime cannot resolve github.com; full unchanged Stack/Weave integration is delegated to the existing hosted workflow, not claimed locally. No runtime/workflow changes conceal this environment boundary.
Expected full suite: 52 unchanged transport/assembly methods plus 22 repair methods = 74. Hosted checks and live review state must be read before merge. Changes are restricted to eight existing files under
research/weave/; no libs, producer pins, authority transfer, new verifier, or deployment.Usage
cd research/weave && python test.py --receipt /tmp/weave-check.jsonRead
research/weave/QUESTIONS.mdfor the next approval set. The repair may be merged independently of approving its explicitly pending research proposals.hmmm: the actual forward/private-recovery positional construction remains unimplemented; approving a description is not evidence that it works.