Skip to content

chore: Bump NOTE_MAX_SIZE to 256 KiB#2651

Merged
mmagician merged 2 commits intorelease/v0.14.0-betafrom
pgackst-bump-note-size-limit
Mar 20, 2026
Merged

chore: Bump NOTE_MAX_SIZE to 256 KiB#2651
mmagician merged 2 commits intorelease/v0.14.0-betafrom
pgackst-bump-note-size-limit

Conversation

@PhilippGackstatter
Copy link
Copy Markdown
Contributor

Bumps the NOTE_MAX_SIZE to be equal to ACCOUNT_UPDATE_MAX_SIZE to allow compiler artifacts to be tested in current networks.

Will leave a note in #1465 to bring it down again in the future.

closes #2648

Copy link
Copy Markdown
Collaborator

@greenhat greenhat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you for the quick fix!

Copy link
Copy Markdown
Collaborator

@mmagician mmagician left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

back and forth on the limit :)

@mmagician mmagician merged commit 059c97c into release/v0.14.0-beta Mar 20, 2026
15 checks passed
@mmagician mmagician deleted the pgackst-bump-note-size-limit branch March 20, 2026 09:32
PhilippGackstatter added a commit that referenced this pull request Mar 20, 2026
* chore: bump NOTE_MAX_SIZE to 256 KiB

* chore: add changelog
@bobbinth
Copy link
Copy Markdown
Contributor

Let's create an issue for figuring out what to do on this front long term. We may want to bring down the limit back to 32KB - or maybe keep the 256KB limit, but make the fees associated with the note script size scale superlinearly.

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.

OutputNoteError::NoteSizeLimitExceeded error with the Rust P2ID note script

4 participants