Skip to content

feat: Support ARC65 errors #347

Merged
iglosiggio merged 11 commits intomainfrom
feat/logged-errors
Mar 17, 2026
Merged

feat: Support ARC65 errors #347
iglosiggio merged 11 commits intomainfrom
feat/logged-errors

Conversation

@Argimirodelpozo
Copy link
Contributor

Bringing support for logged errors and assertions (arc65).
The relevant PR enabling it in Puya and Puyapy is this one.

WIP: missing a little bit of love on the test side.

@iglosiggio iglosiggio force-pushed the feat/logged-errors branch 2 times, most recently from 59c8200 to d81eff9 Compare March 16, 2026 22:07
@iglosiggio iglosiggio marked this pull request as ready for review March 16, 2026 22:08
@iglosiggio iglosiggio changed the title Logged errors (ARC65) support feat: Support ARC65 errors Mar 17, 2026
@Argimirodelpozo
Copy link
Contributor Author

I had some qualms about the options object and forcing the {} into the API, but I'm convinced now that introducing that tiny bit of friction for messages (which we absolutely want people to keep short, and if you can do away with just a code then even better) and the prefix (which should default to ERR 99% of the time and will only be AER on very specific ARC error scenarios if ever) is perfectly fine.
Good to merge, thanks bobby and mega for review and contributions!

@iglosiggio
Copy link
Member

I had some qualms about the options object and forcing the {} into the API, but I'm convinced now that introducing that tiny bit of friction for messages (which we absolutely want people to keep short, and if you can do away with just a code then even better) and the prefix (which should default to ERR 99% of the time and will only be AER on very specific ARC error scenarios if ever) is perfectly fine. Good to merge, thanks bobby and mega for review and contributions!

Just added an overload where instead of options you just provide a string with the corresponding message. Updated the test accordingly.

@Argimirodelpozo Argimirodelpozo force-pushed the feat/logged-errors branch 2 times, most recently from 1fb1a53 to 6bf1fd0 Compare March 17, 2026 17:16
@iglosiggio iglosiggio merged commit d8b5fe6 into main Mar 17, 2026
8 checks passed
@iglosiggio iglosiggio deleted the feat/logged-errors branch March 17, 2026 18:15
@engineering-ci
Copy link
Contributor

🎉 This PR is included in version 1.2.0-beta.17 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants