Skip to content

Integrate OOReporter and SignedProposer audit fixes - #87

Closed
md0x wants to merge 61 commits into
masterfrom
pablo/oz-final-integration
Closed

md0x wants to merge 61 commits into
masterfrom
pablo/oz-final-integration

Conversation

@md0x

@md0x md0x commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Final audit review now follows the original branches: #76 at 71d3df10c659542dc5a92028c62f0586efb6b80d and #78 at 113e6729f9130ac95795e7c4c7f2a667893aacae. All seven fix PRs (#80–#86) are merged into their original bases. Combining those two final branches exactly reproduces this PR’s repository tree. This draft is retained as an unchanged reference; no merge of this PR is needed for that review.

For the reduced final review, see #88: two existing follow-up commits across six files, with the exact same final head as this PR. Its description also links the #84/#80 documentation and interface adjustments already included in the comparison base. This PR remains the complete integration against master.

Integrate #76, #77 and #78 with all six audit fixes (#81–#86) and the callback reserve change (#80) into one branch for final review. The original PR heads are preserved in the merge history. Keep this PR in draft pending final audit review.

Review commit: 7f95d18ae86d4a874953cc951be2606eb1a026da. Tracks FRO-146.

Integration decisions:

Validation with Foundry 1.3.6:

  • 221 non-fork tests pass, including reporter settlement gas exhaustion, upgrade state, real Permit2 signatures and all audit regressions.
  • 18 Polygon fork tests pass, including production upgrade preservation. Eight optional VNet/Amoy tests skip because their RPCs are not configured.
  • Production Managed OO storage-layout validation passes against the pinned reference build.
  • forge build --sizes, forge fmt --check and diff checks pass.

Reinis-FRP and others added 30 commits June 22, 2026 19:24
Signed-off-by: Matt Rice <matthewcrice32@gmail.com>
Co-authored-by: Reinis Martinsons <reinis@umaproject.org>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Chris Maree <christopher.maree@gmail.com>
Co-authored-by: Pablo Maldonado <pablo@umaproject.org>
@md0x

md0x commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator Author

Closing this reference PR because its changes are already integrated into master through the original audited branches. The repository tree at this PR head (7f95d18) matches master immediately before PR #79 (886c23b). No additional merge is needed.

@md0x md0x closed this Oct 15, 2026
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.

4 participants