Skip to content

feat(stellar-wallet-snap)!: remove asset handler entry points - #262

Open
ulissesferreira wants to merge 1 commit into
mainfrom
WPN-2014-remove-asset-handlers
Open

feat(stellar-wallet-snap)!: remove asset handler entry points#262
ulissesferreira wants to merge 1 commit into
mainfrom
WPN-2014-remove-asset-handlers

Conversation

@ulissesferreira

Copy link
Copy Markdown
Contributor

Closes WPN-2014

Summary

  • Removes the onAssetsLookup, onAssetsConversion, onAssetHistoricalPrice, and onAssetsMarketData entry points from stellar-wallet-snap
  • Removes the now-unused AssetsHandler module (src/handlers/asset/)
  • Removes the now-unused endowment:assets permission from the manifest

The AssetMetadataService, PriceService, and src/api/asset structs are kept since they are used by client request handlers, cronjobs, and sync services.

Breaking change

The asset handler entry points are no longer exported by the snap.

@ulissesferreira
ulissesferreira requested a review from a team as a code owner September 2, 2026 17:14
Remove the onAssetsLookup, onAssetsConversion, onAssetHistoricalPrice,
and onAssetsMarketData entry points, along with the now-unused
AssetsHandler module and the endowment:assets permission.

Closes WPN-2014
@ulissesferreira
ulissesferreira force-pushed the WPN-2014-remove-asset-handlers branch from 3ae3fd4 to 734f1ce Compare September 2, 2026 17:31
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.

1 participant