Skip to content

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

Merged
jxom merged 2 commits intowevm:mainfrom
hashcashier:kailua
Apr 21, 2026
Merged

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

Conversation

@hashcashier
Copy link
Copy Markdown
Contributor

@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
Contributor Author

@jxom bumping this up on your radar

@d4mr
Copy link
Copy Markdown

d4mr commented Apr 20, 2026

@jxom wanted to give this another bump

@jxom jxom merged commit 30ac0cf into wevm:main Apr 21, 2026
2 of 3 checks passed
@github-actions github-actions Bot mentioned this pull request Apr 21, 2026
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.

3 participants