Skip to content

feat: get metadata uri; pin AVM version 12#30

Merged
cusma merged 7 commits intomainfrom
feat/get-metadata-uri
Dec 28, 2025
Merged

feat: get metadata uri; pin AVM version 12#30
cusma merged 7 commits intomainfrom
feat/get-metadata-uri

Conversation

@cusma
Copy link
Copy Markdown
Collaborator

@cusma cusma commented Dec 28, 2025

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new ABI method arc89_get_metadata_partial_uri to retrieve the Asset Metadata ARC-90 partial URI without the compliance fragment. The implementation calls the existing arc90_box_query function with an empty box name to construct the URI.

Key Changes

  • Added new readonly ABI method to retrieve the ARC-90 partial URI
  • Refactored Bytes(b"") to Bytes() for cleaner empty bytes initialization
  • Updated TEAL compilation artifacts to include the new method routing

Reviewed changes

Copilot reviewed 4 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
smart_contracts/asa_metadata_registry/interface.py Added abstract method declaration for arc89_get_metadata_partial_uri
smart_contracts/asa_metadata_registry/contract.py Implemented the method by calling arc90_box_query with empty box name
smart_contracts/avm_common.py Simplified empty Bytes initialization from Bytes(b"") to Bytes()
tests/test_arc89_get_metadata_partial_uri.py Added test with workaround for broken .abi_return value
smart_contracts/artifacts/asa_metadata_registry/AsaMetadataRegistry.approval.teal Updated TEAL bytecode with new method routing and arc90_box_query subroutine
poetry.lock Reordered dependency markers and removed Python 3.8 wheel entries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cusma cusma changed the title feat: get metadata uri feat: get metadata uri; pin AVM version 12 Dec 28, 2025
@cusma cusma merged commit f604315 into main Dec 28, 2025
1 check passed
@cusma cusma deleted the feat/get-metadata-uri branch December 28, 2025 01:49
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.

2 participants