fix: bump op-fetcher dep to fix daily-codegen CI#1200
Open
stevennevins wants to merge 2 commits intomainfrom
Open
fix: bump op-fetcher dep to fix daily-codegen CI#1200stevennevins wants to merge 2 commits intomainfrom
stevennevins wants to merge 2 commits intomainfrom
Conversation
The daily-codegen CircleCI workflow has been failing since ~Feb 4 because the op-fetcher's FetchChainInfo.s.sol script reverts against current on-chain contracts. Bumps optimism from a June 2025 pseudo-version to v1.13.7 with matching op-geth.
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
![wiz-inc-a178a98b5d[bot]](https://avatars.githubusercontent.com/u/58791460?s=60&v=4){kind=small}
| github.com/clbanning/mxj/v2 v2.7.0 // indirect | ||
| github.com/consensys/bavard v0.1.27 // indirect | ||
| github.com/consensys/gnark-crypto v0.16.0 // indirect | ||
| github.com/consensys/gnark-crypto v0.18.0 // indirect |
There was a problem hiding this comment.
The following vulnerability impacts github.com/consensys/gnark-crypto versions <0.18.1: GHSA-fj2x-735w-74vq.
It can be remediated by updating to version 0.18.1 or higher.
To ignore this finding as an exception, reply to this conversation with #wiz_ignore reason
If you'd like to ignore this finding in all future scans, add an exception in the .wiz file (learn more) or create an Ignore Rule (learn more).
To get more details on how to remediate this issue using AI, reply to this conversation with #wiz remediate
Suggested change
| github.com/consensys/gnark-crypto v0.18.0 // indirect | |
| github.com/consensys/gnark-crypto v0.18.1 // indirect |
ops/go.mod requires Go 1.24.0 after the op-fetcher dep bump. golangci-lint 1.61.0 was built with Go 1.23 and refuses to lint Go 1.24 code.
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
1 participant
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
daily-codegenCircleCI scheduled workflow has been failing since ~Feb 4 (~5 weeks)op-fetcher'sFetchChainInfo.s.solscript reverts against current on-chain contracts (OP Sepolia + OP Mainnet)ethereum-optimism/optimismfrom a June 2025 pseudo-version tov1.13.7with matchingop-geth(v1.101602.3-rc.1)CI failure logs
First failure (Feb 4):
Latest failure (Mar 13):
Test plan
go build ./...compilesgo test ./...— no new test failures (3 pre-existing local-only failures confirmed)daily_codegen_dispatchvia CircleCI API to verify the fix end-to-end