Skip to content

fix: Replace extraData decoding with l2SequenceNumber() call in getGames#4428

Open
hashcashier wants to merge 2 commits intowevm:mainfrom
hashcashier:kailua
Open

fix: Replace extraData decoding with l2SequenceNumber() call in getGames#4428
hashcashier wants to merge 2 commits intowevm:mainfrom
hashcashier:kailua

Conversation

@hashcashier
Copy link
Copy Markdown

@hashcashier hashcashier commented Mar 26, 2026

Resolves #3794

Summary

  • Replace manual decodeAbiParameters on game.extraData with an l2SequenceNumber() contract call on each game proxy
  • Extract proxy addresses from the metadata (GameId) field and use multicall to batch the calls efficiently
  • Add disputeGameAbi with the l2SequenceNumber function signature

Test plan

  • Existing getGames tests pass
  • Existing getGame tests pass (downstream consumer)
  • TypeScript type check passes

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 26, 2026

🦋 Changeset detected

Latest commit: 1ec0929

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

@hashcashier is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@hashcashier
Copy link
Copy Markdown
Author

@jxom bumping this up on your radar

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