feat(op-interop-filter): add sysgo e2e tests [4/4]#11
Open
opsuperchain wants to merge 2 commits intofeat/interop-filter-sysgo-presetfrom
Open
feat(op-interop-filter): add sysgo e2e tests [4/4]#11opsuperchain wants to merge 2 commits intofeat/interop-filter-sysgo-presetfrom
opsuperchain wants to merge 2 commits intofeat/interop-filter-sysgo-presetfrom
Conversation
4863aa1 to
e5cdf5b
Compare
b785d7a to
77e4f3b
Compare
e5cdf5b to
bd3795e
Compare
77e4f3b to
65783b6
Compare
bd3795e to
99ed03d
Compare
This PR adds op-devstack/sysgo integration for op-interop-filter: - `op-service/apis/interop_filter.go`: InteropFilter API interfaces - `op-service/sources/interop_filter_client.go`: RPC client for interop filter - `op-devstack/stack/interop_filter.go`: InteropFilterID and InteropFilter interface - `op-devstack/shim/interop_filter.go`: Shim implementation - `op-devstack/sysgo/interop_filter.go`: WithInteropFilter function - `op-devstack/dsl/interop_filter.go`: DSL wrapper - `op-devstack/presets/minimal_with_interop_filter.go`: Preset for tests - Updated system interface to include InteropFilter methods - Added FirstInteropFilter matcher 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This PR adds sysgo integration tests for op-interop-filter: - Test interop filter startup and hydration - Test failsafe initial state - Test filter ready state after backfill - Matcher infrastructure test Note: The full e2e tests require the sysgo deployer infrastructure, which currently has some ABI mismatch issues. Tests will pass once those issues are resolved in the base branch. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
65783b6 to
fc2a3ff
Compare
karlfloersch
pushed a commit
that referenced
this pull request
Mar 5, 2026
…thereum-optimism#19272) * contracts: implement audit code fixes and add tests Add onlyDelegateCall enforcement to upgradeSuperchain, upgrade, and migrate functions (#17). Include msg.sender in deploy salt to prevent cross-caller CREATE2 collisions (#17). Add duplicate instruction key detection in upgrade validation (#9). Validate startingRespectedGameType against enabled game configs (#10). Add code-existence check in loadBytes (#18). Add setUp guard to VerifyOPCM.runSingle (#4). Remove unused _findChar function (#5). Pass real AddressManager in migrator proxy deploy args (#11). Add tests covering all audit fix behaviors. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * contracts: regenerate semver-lock.json for OPContractsManagerV2 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * contracts: bump OPContractsManagerV2 version to 7.0.10 Semver-diff requires a patch version bump when bytecode changes. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is PR 4 of 4 for the
op-interop-filterservice.This PR adds sysgo integration tests:
FirstInteropFiltermatcher worksStack
This PR is part of a stacked PR series:
Test plan
go build ./op-devstack/sysgo/...go test ./op-devstack/sysgo/... -run TestInteropFilter🤖 Generated with Claude Code