Skip to content

Post-merge custody repair: bind two-scale manuscripts to accepted reviews - #171

Closed
d6g8k5htny-coder wants to merge 7 commits into
mainfrom
chatgpt/two-scale-custody-postmerge-repair-20260929
Closed

d6g8k5htny-coder wants to merge 7 commits into
mainfrom
chatgpt/two-scale-custody-postmerge-repair-20260929

Conversation

@d6g8k5htny-coder

Copy link
Copy Markdown
Owner

Engineering/source-custody repair only. Scientific effect NONE.

PR #166 merged with the reviewed mathematical manuscripts unchanged, but the landed packet verifier still permitted a coordinated manuscript+MANIFEST refresh (or manuscript removal+MANIFEST refresh) while REVIEW_RECORD stayed untouched. That defect was independently reproduced in the integration lane before merge.

This successor fixes the landed packet without changing either mathematical manuscript, SOURCES.json, geometry.py, RESULTS.json, any graph/status/claim, or any review verdict:

  • adds ACCEPTED_BINDINGS.json binding the exact accepted mathematical head, both manuscript byte/SHA256/blob identities, and the actual scoped A/B/C review IDs;
  • makes verify.py enforce those accepted manuscript identities independently of MANIFEST and require the exact review-id mapping;
  • requires essential accepted/replay leaves to remain in the packet inventory;
  • adds real regressions showing that a coordinated manuscript+manifest rebinding, review-id rewrite, or manuscript removal is rejected;
  • regenerates MANIFEST for the repair;
  • adds a scoped push: main path trigger so future landed packet changes re-run this custody workflow.

Accepted mathematics remains exactly:

  • TWO_SCALE_LAW.md blob a32fd5f7d941bbe1fe943df045b1e0fbec8d691c
  • RADIAL_TAIL.md blob 0f14417ef7038b9c6f50e4b01e393a58b7b5e3a4

Bound reviews remain A 5360227991, B 5360178611, C 5360216551. One Claude provider/session, zero organizational-independence claim.

This does not authenticate GitHub review objects over the network, prove the mathematics, or prevent an intentionally reviewed future change to the verifier contract itself; it closes the demonstrated packet-self-rebinding hole and makes such a change explicit/reviewable rather than silently satisfiable by regenerating MANIFEST.

Please review the verifier contract and regression cases only; do not reopen the mathematical proofs.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-30T01:57:48.380042Z 7504ba0 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copy link
Copy Markdown
Owner Author

@codex review — bounded engineering/source-custody review only. Please reproduce the three new negative cases (coordinated manuscript+manifest refresh, review-id rewrite, manuscript removal), verify ACCEPTED_BINDINGS is independent of MANIFEST authority for the two reviewed manuscripts, and check the scoped main-push trigger. Do not review or reinterpret the mathematical manuscripts; their blobs must remain a32fd5f7 and 0f14417e.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 7504ba081b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Merge current main fa1cf7b into the custody-repair branch while preserving all repair blobs from 7504ba0. Scientific effect NONE.

Copy link
Copy Markdown
Owner Author

Source-preserving current-main refresh complete: head bf00963 merges only current main fa1cf7b into reviewed repair head 7504ba0. The five PR-delta blobs are unchanged, so Codex comment5902558878 still binds the exact engineering repair bytes it reviewed. Fresh current-head CI is running; merge remains held until both packet and downstream aggregates succeed and the thread/head/main readback stays clean.

Copy link
Copy Markdown
Owner Author

Coordination update: PR #172 landed concurrently from the earlier C25 integration owner and implements the same custody repair more completely (frozen machine-readable REVIEW_RECORD.json, exact proof+SOURCES bindings, required-payload membership, nine binding regressions, and main-push replay). I independently fetched the three live native #166 reviews and verified #172's recorded body identities exactly: A5360227991 = 7840 B / c63efc292d7cf209e2770419cc159276897d0ba955c32ee1d2af88c8aa7f0ed8; B5360178611 = 7631 B / 3042d958b769b0a9ada1359b6f8d9461c090060fe5057cff795c99b6c6c2894d; C5360216551 = 4277 B / 9a2c89c6e0c122fc75cc96a2b6392b47ab33bc47feb072b9f2a56315c527df84. I am closing this duplicate rather than fragmenting the repair. No scientific or review status changes from this branch; #172 is the integration target.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant