Skip to content

Bound the length refit derives from a proof - #928

Merged
mafintosh merged 2 commits into
mainfrom
fix/refit-length-bound
Nov 4, 2026
Merged

mafintosh merged 2 commits into
mainfrom
fix/refit-length-bound

Conversation

@marcus-pousette-hp

@marcus-pousette-hp marcus-pousette-hp commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Co-authored with AI

@marcus-pousette-hp
marcus-pousette-hp requested a review from a team November 4, 2026 14:14
@greptile-apps

greptile-apps Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

Bounds the length that refit derives from proof nodes.

  • Proof refitting rejects lengths above JavaScript's precise number limit.

No new actionable issues were found.

Reviews (2) · Last reviewed commit: "Move the max length into a constant" · Reviewed by Greptile

@mafintosh
mafintosh merged commit 5366a6a into main Nov 4, 2026
6 checks passed
@mafintosh
mafintosh deleted the fix/refit-length-bound branch November 4, 2026 17:25
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.

2 participants