Skip to content

fix(idl): Respect offset = ... in IDL generation for custom errors#4040

Merged
jacobcreech merged 1 commit intosolana-foundation:masterfrom
jamie-osec:idl-respect-error-offset
Nov 28, 2025
Merged

fix(idl): Respect offset = ... in IDL generation for custom errors#4040
jacobcreech merged 1 commit intosolana-foundation:masterfrom
jamie-osec:idl-respect-error-offset

Conversation

@jamie-osec
Copy link
Copy Markdown
Collaborator

Closes #4039

This issue led to a mismatch between generated IDLs and generated code if a custom offset was used with the error code.
Regex-based GH search indicates the impact of this was likely low.

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 31, 2025

@jamie-osec is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@jamie-osec jamie-osec force-pushed the idl-respect-error-offset branch 2 times, most recently from 3ef3599 to 19c5b6c Compare October 31, 2025 14:04
@jamie-osec jamie-osec force-pushed the idl-respect-error-offset branch from 19c5b6c to 8bfbbfd Compare October 31, 2025 14:06
@nutafrost nutafrost moved this to In Progress in Anchor 1.0 Oct 31, 2025
Copy link
Copy Markdown
Contributor

@Otter-0x4ka5h Otter-0x4ka5h left a comment

Choose a reason for hiding this comment

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

lgtm

@jamie-osec jamie-osec moved this from In Progress to Security Review Done in Anchor 1.0 Nov 3, 2025
@jamie-osec jamie-osec added breaking Minor Minor breaking change idl related to the IDL, either program or client side fix Bug fix PR labels Nov 3, 2025
@jacobcreech jacobcreech merged commit a129489 into solana-foundation:master Nov 28, 2025
54 of 55 checks passed
@github-project-automation github-project-automation bot moved this from Security Review Done to Done in Anchor 1.0 Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking fix Bug fix PR idl related to the IDL, either program or client side Merge Minor Minor breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

error_code macros should respect offset

5 participants