Skip to content

feat: migrate metadata from doc-comments to IDL annotations#1301

Merged
m62624 merged 5 commits into
masterfrom
feat/port-annotations
Apr 15, 2026
Merged

feat: migrate metadata from doc-comments to IDL annotations#1301
m62624 merged 5 commits into
masterfrom
feat/port-annotations

Conversation

@m62624
Copy link
Copy Markdown
Contributor

@m62624 m62624 commented Apr 15, 2026

The current implementation uses doc-comment string matching (/// #[payable]) to pass system metadata to generators. This task migrates these to structured annotations (@payable, @indexed) using the type-registry system to separate human-readable documentation from system-level logic.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request transitions the handling of metadata such as indexed, payable, and returns_value from doc comments to formal attributes (#[type_info(...)]) and IDL annotations (@...). The changes span macro expansion logic, IDL generation, and parsing to ensure consistent use of structured attributes over string-based doc tags. Feedback was provided regarding a function name in rs/macros/core/src/event/ethexe.rs that is now misleading and likely requires a corresponding update to its unit tests.

Comment thread rs/macros/core/src/event/ethexe.rs Outdated
@m62624 m62624 force-pushed the feat/port-annotations branch from baae75b to 6723d87 Compare April 15, 2026 06:13
@m62624 m62624 marked this pull request as ready for review April 15, 2026 06:14
@m62624 m62624 requested a review from vobradovich April 15, 2026 06:41
@m62624 m62624 merged commit cbe7090 into master Apr 15, 2026
4 checks passed
@m62624 m62624 deleted the feat/port-annotations branch April 15, 2026 10:23
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