Kailua FaultDisputeGame is not supported by op-stack module

[slavastartsev]

Check existing issues

• I checked there isn't already an issue for the bug I encountered.

Viem Version

2.31.7

Current Behavior

When proving withdrawal tx from OP stack L2 call publicClientL1.waitProve() fails because extraData field for KailuaGame provides different data layout comparing to bedrock's FaultDisputeGame contract.

Expected Behavior

Update implementation of waitProve function to support different IDisputeGame contract implementations. As a reference you can check which GameTypes optimism supports. Link to OP stack code

Steps To Reproduce

You can follow contract implementation logic to see that returned data has different layout and results in publicClientL1.waitProve() function to throw error.
Internally it will call getGames function. Function fetches latest games from disputeGameFactory contract

viem/src/op-stack/actions/getGames.ts

Lines 110 to 119 in 3aa8826

	(await readContract(client, {
	abi: disputeGameFactoryAbi,
	functionName: 'findLatestGames',
	address: disputeGameFactoryAddress,
	args: [
	gameType,
	BigInt(Math.max(0, Number(gameCount - 1n))),
	BigInt(Math.min(limit, Number(gameCount))),
	],
	})) as Game[]

and expects that every game returned will have extraData field containing 32 bytes which represent uint256 l2BlockNumber.

Link to findLatestGames function implementation. In a loop function fetches extraData from each contract implementing IDisputeGame.

FaultDisputeGame contract extraData function implementation -> returns 32 bytes storing single value l2BlockNumber

Kailua game contract extraData function implementation -> returns 24 bytes (0x08 l2BlockNumber, 0x08 parentGameIndex, 0x08 duplicationCounter)

hence abiDecodeParameters call fails

viem/src/op-stack/actions/getGames.ts

Lines 122 to 125 in 3aa8826

	const [blockNumber] = decodeAbiParameters(
	[{ type: 'uint256' }],
	game.extraData,
	)

Link to Minimal Reproducible Example

No response

Anything else?

Link to OP stack code

[hashcashier]

suggested fix: use the l2BlockNumber() method instead of manually parsing the extraData field.