Skip to content

op-challenger: Support resolving optimistic zk games#18464

Merged
ajsutton merged 5 commits intodevelopfrom
aj/resolveable-game
Dec 3, 2025
Merged

op-challenger: Support resolving optimistic zk games#18464
ajsutton merged 5 commits intodevelopfrom
aj/resolveable-game

Conversation

@ajsutton
Copy link
Copy Markdown
Contributor

@ajsutton ajsutton commented Dec 2, 2025

Description

Adds support for resolving optimistic ZK games to challenger.

Tests

Added unit tests.

Additional context

Builds on #18442

Metadata

#18321

@ajsutton ajsutton requested review from a team, pauldowman and scharissis and removed request for a team December 2, 2025 01:32
@ajsutton ajsutton force-pushed the aj/support-zk-game-type branch from 378b340 to 3d62d52 Compare December 2, 2025 01:33
@ajsutton ajsutton force-pushed the aj/resolveable-game branch from eb18e85 to 72e2e9a Compare December 2, 2025 01:34
@ajsutton ajsutton force-pushed the aj/support-zk-game-type branch from 3d62d52 to 0447f09 Compare December 2, 2025 02:36
@ajsutton ajsutton force-pushed the aj/resolveable-game branch from 72e2e9a to cbf4b69 Compare December 2, 2025 02:36
Comment thread op-challenger/game/zk/actor.go Outdated
Copy link
Copy Markdown
Contributor

@Inphi Inphi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall. Just a couple comments.

Comment thread op-challenger/game/zk/actor.go Outdated
Comment thread op-challenger/game/zk/actor_test.go Outdated
@ajsutton ajsutton force-pushed the aj/resolveable-game branch from cbf4b69 to 3b6311d Compare December 2, 2025 21:05
Base automatically changed from aj/support-zk-game-type to develop December 2, 2025 23:19
@ajsutton ajsutton force-pushed the aj/resolveable-game branch from 6a28a1c to ed24172 Compare December 3, 2025 00:41
Copy link
Copy Markdown
Contributor

@Inphi Inphi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Consider simplifying the createResolveTx a bit.

Comment thread op-challenger/game/zk/actor.go
@ajsutton ajsutton added this pull request to the merge queue Dec 3, 2025
Merged via the queue into develop with commit 991a4a7 Dec 3, 2025
82 checks passed
@ajsutton ajsutton deleted the aj/resolveable-game branch December 3, 2025 04:16
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.

2 participants