Skip to content

feat(standards): add NoteExecutionHint constants to MASM standards#2493

Merged
bobbinth merged 7 commits intonextfrom
mmagician-exec-hint-standards
Feb 25, 2026
Merged

feat(standards): add NoteExecutionHint constants to MASM standards#2493
bobbinth merged 7 commits intonextfrom
mmagician-exec-hint-standards

Conversation

@mmagician
Copy link
Copy Markdown
Collaborator

@mmagician mmagician commented Feb 23, 2026

Summary

  • Adds crates/miden-standards/asm/standards/note/execution_hint.masm exposing the four NoteExecutionHint variants as public MASM constants: NONE, ALWAYS, AFTER_BLOCK, ON_BLOCK_SLOT, matching the Rust NoteExecutionHint implementation
  • For AFTER_BLOCK and ON_BLOCK_SLOT the constants encode only the 8 tag bits; callers must compose the payload
  • The encoding of the payload could be added separately if/when the need arises.

🤖 Generated with Claude Code

mmagician and others added 2 commits February 23, 2026 10:59
Add `note/execution_hint.masm` under the standards library exposing the
four NoteExecutionHint variants as public constants:

- NONE = 0
- ALWAYS = 1
- AFTER_BLOCK = 2  (tag bits only; payload must be composed at runtime)
- ON_BLOCK_SLOT = 3  (tag bits only; payload must be composed at runtime)

These mirror the `NONE_TAG` / `ALWAYS_TAG` / `AFTER_BLOCK_TAG` /
`ON_BLOCK_SLOT_TAG` internal constants from the Rust
`NoteExecutionHint` implementation and allow MASM callers to reference
them via `use miden::standards::note::execution_hint::ALWAYS` (etc.)
rather than duplicating magic numbers locally.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mmagician
Copy link
Copy Markdown
Collaborator Author

@copilot add changelog entry

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 23, 2026

@mmagician I've opened a new pull request, #2496, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits February 23, 2026 14:31
Co-authored-by: mmagician <8402446+mmagician@users.noreply.github.com>
docs: add changelog entry for NoteExecutionHint MASM constants
@mmagician mmagician changed the title feat(standards): add NoteExecutionHint constants to MASM standards feat(standards): add NoteExecutionHint constants to MASM standards Feb 23, 2026
@mmagician mmagician self-assigned this Feb 23, 2026
@mmagician mmagician added agglayer PRs or issues related to AggLayer bridging integration pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority standards Related to standard note scripts or account components and removed agglayer PRs or issues related to AggLayer bridging integration labels Feb 23, 2026
Copy link
Copy Markdown
Contributor

@PhilippGackstatter PhilippGackstatter left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you!

@bobbinth bobbinth merged commit 477f1b2 into next Feb 25, 2026
17 checks passed
@bobbinth bobbinth deleted the mmagician-exec-hint-standards branch February 25, 2026 07:47
afa7789 pushed a commit to afa7789/miden-base that referenced this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority standards Related to standard note scripts or account components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants