Skip to content

fix(mcp-server): resolve NFT owners and cover EVM resources (PLT-347) - #335

Merged
alexander-sei merged 9 commits into
mainfrom
alexanderh/plt-347-update-sei-js-tests-to-cover-minimum-required-coverage-for
Aug 20, 2026
Merged

fix(mcp-server): resolve NFT owners and cover EVM resources (PLT-347)#335
alexander-sei merged 9 commits into
mainfrom
alexanderh/plt-347-update-sei-js-tests-to-cover-minimum-required-coverage-for

test(mcp-server): cover non-numeric NFT token IDs (PLT-347)

e488c02
Select commit
Loading
Failed to load commit list.
Seidroid / AI Review succeeded Aug 20, 2026 in 0s

Claude + Codex + Cursor Review: success

Solid, well-scoped fix: the NFT detail resource previously read a {address} param its URI template never defines, so owner was always "Unknown" — routing both the resource and the get_nft_info tool through a shared getERC721Owner corrects that, and the added tests (all 19 EVM resource handlers, plus a registration/URI-template guard) are a real coverage win. No blockers; remaining notes are about the half-applied "distinguish failure from absence" contract on isNFTOwner, raw provider error text now reaching the model, and minor docs/duplication cleanups.