- 09 Aug, 2024 7 commits
-
-
Michael de Hoog authored
* chore: update to support geth v1.14.7 * Update geth * Fix "private key curve is not secp256k1" error * Temporarily quiet noisy logs * signature test * Add comments about S256 curve override fix * Update geth * go.mod: update to op-geth v1.101407.0-rc.1 * op-e2e: fix L1/L2 miner settings * Also update devnet L1 base image * ci: bump ci-builder in circleci --------- Co-authored-by:
Danyal Prout <me@dany.al> Co-authored-by:
protolambda <proto@protolambda.com> Co-authored-by:
Sebastian Stammler <seb@oplabs.co>
-
Maurelian authored
* Add RV reviews * Fix naming of trust audit to match convention
-
protolambda authored
* ci: update geth tool to v1.14.7 * ci: temporarily disable L1 geth version check, so L1 geth can be updated * ops-bedrock: explicit hash state.scheme on geth
-
Matt Joiner authored
* Fix devnet-up when op-program/bin is missing prestate-proof.json * Fix cannon-prestate to always run * Add prestate.json
-
Mark Tyneway authored
Migrate to just from pnpm. This is continued tech debt cleanup away from `pnpm` and typescript that is no longer used.
-
Mark Tyneway authored
* ci: better solc warnings check Use `forge build --force` to force a build so that the warning check can be observed. CI should fail on this commit * contracts-bedrock: fix warnings Fix the failed solc build See the failed workflow here: https://app.circleci.com/pipelines/github/ethereum-optimism/optimism/59631/workflows/b93bc600-e772-4ba9-98ed-a0b6f48683c7/jobs/2512271 * foundry: add ignored error code
-
Mark Tyneway authored
* contracts-bedrock: debug tests See why the tests are flaking and cannot be reproduced locally. Also modularize/cleanup CI along the way * typo: fix * debug: remove * just: update name * ci: better error help * ci: fix typo * contracts-bedrock: abstract * ci: run with foundry profile ci
-
- 08 Aug, 2024 6 commits
-
-
tre authored
-
protolambda authored
* devnet/e2e: run L1 Deneb and L2 Fjord by default * op-e2e: fix TestPostUnsafePayload * op-e2e: Fix TestGasPriceOracleFeeUpdates * op-e2e/actions: fix some tests The L2EngineAPI test needs more fixing * tests: fixes to support Ecotone/Fjord in more tests * op-e2e: one more fix --------- Co-authored-by:
Sebastian Stammler <seb@oplabs.co>
-
Adrian Sutton authored
This reverts commit 2491a373. make cannon-prestate doesn't declare all transitive dependencies properly so can fail to build when there are changes.
-
Mark Tyneway authored
* contracts-bedrock: improve `CrossL2Inbox` devex Improve the `CrossL2Inbox` devex by creating an alternative entrypoint. This design was not considered previously because there was a "top level call" restriction, aka the "only EOA" invariant. This was to allow for static analysis of transactions, keeping resource usage lower for validating transactions when building blocks or at the mempool layer. Since 3074/7702 render the enforcement of only eoa impossible, we decided to op/acc and lean into the approach of allowing subcalls to trigger `ExecutingMessage` events. This new interface allows another contract to be the entrypoint, the idea is that the user sends the `Identifier` and the serialized log (message) to whatever contract that they want and then pass it to `CrossL2Inbox.validateMessage` which then emits the event that consensus validates. This allows the calling smart contract to be aware of the schema for the log and deserialize it however they see fit. Since the serialized logs are done with the following algorithm: ```go msg := make([]byte, 0) for _, topic := range log.Topics { msg = append(msg, topic.Bytes()...) } msg = append(msg, log.Data...) ``` It is very easy to use `abi.decode` to decode a log, given that solidity was used to `emit` it. The topics are `bytes32` and then the data is abi encoded given the schema of the event itself. Unused parts like `topic[0]` (hash of the event name) can be dropped when decoding if they are not required. * ctb: fix typo * remove nonReentrant and add tests for validateMessage, rename ENTERED_SLOT preimage * add natspec for _checkIdentifier and update that of validateMessage * update version and semver-lock file * check all topics in crossl2inbox test, run pnpm snapshots * tests: fix --------- Co-authored-by:
Michael Amadi <amadimichaeld@gmail.com>
-
Adrian Sutton authored
Ensures that if it fails, the process exits with an error, rather than just exiting the runloop and continuing on in a zombie mode that doesn't do anything but doesn't exit.
-
Mark Tyneway authored
* contracts-bedrock: migrate to just Move away from `pnpm` for contracts scripts and towards `just`. `just` was made for this, since we are moving away from JS it makes no sense to keep `pnpm` around which is native to the JS ecosystem. * ci: install just * ops: cleanup, docs * ci: install just * justfile: update * ci: fix install * snapshots: update * contracts-bedrock: remove pnpm/node, cleanup dockerfile * slither ci: no longer need for workaround (TM) * ts-linting: remove * kontrol: regenerate snapshots * safecall: update test * test: update Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * docs: update Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * readme: update Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * docs: update Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * docs: update Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * docs: update Co-authored-by:
Maurelian <john@oplabs.co> * versions: use latest * ci: fix * CONTRIBUTING: update * ci: fix * lint: fix --------- Co-authored-by:
protolambda <proto@protolambda.com> Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> Co-authored-by:
Maurelian <john@oplabs.co>
-
- 07 Aug, 2024 6 commits
-
-
Matt Joiner authored
-
protolambda authored
-
protolambda authored
-
Inphi authored
-
Sebastian Stammler authored
-
clabby authored
* feat(challenger): `AsteriscKona` trace type Adds support for the new `asterisc-kona` game type in the `op-challenger` * feat: add `kona-host` `--l2-chain-id` flag * op-challenger: Separate oracle server executor (#11348) * fix(op-challenger): run trace cmd * fix(op-challenger): lints --------- Co-authored-by:
Adrian Sutton <adrian@oplabs.co> Co-authored-by:
refcell <abigger87@gmail.com>
-
- 06 Aug, 2024 8 commits
-
-
Adrian Sutton authored
-
Sam Stokes authored
* op-proposer: add retries to output proposal * op-proposer: proposeOutput returns err to help trigger retry * op-proposer: use retry.Do for FetchOutput, add unit tests * op-proposer: improve output fetching retry impl * op-proposer: move done signal check into inner loop --------- Co-authored-by:
Sebastian Stammler <seb@oplabs.co>
-
Adrian Sutton authored
* op-challenger: Add metric to report memory used in the cannon VM * op-challenger: Remove config override.
-
Adrian Sutton authored
-
Mark Tyneway authored
* ci: bump ci builder `0.50.0` Bumps the `ci-builder` image with an updated version of foundry. Foundry update: https://github.com/ethereum-optimism/optimism/pull/11325 Docker build: https://app.circleci.com/pipelines/github/ethereum-optimism/optimism/60263/workflows/154fd94a-4029-4e2f-8bce-4ea4c6d25897 * contracts-bedrock: fix tests after new foundry version * contracts-bedrock: comment why skip * test: better comments * gas-snapshot: regenerate * snapshots: update * snapshots: update * fix(ctb): Align expected ptr in `Bytes.slice` test An update to `forge-std` causes `bound` to set the free memory pointer at an unaligned offset. This commit updates the test such that the expected pointer is correctly aligned, without an assumption on the starting ptr's alignment. --------- Co-authored-by:
clabby <ben@clab.by>
-
dependabot[bot] authored
Bumps [golang.org/x/time](https://github.com/golang/time) from 0.5.0 to 0.6.0. - [Commits](https://github.com/golang/time/compare/v0.5.0...v0.6.0) --- updated-dependencies: - dependency-name: golang.org/x/time dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by:
dependabot[bot] <support@github.com> Co-authored-by:
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
-
dependabot[bot] authored
Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.7.0 to 0.8.0. - [Commits](https://github.com/golang/sync/compare/v0.7.0...v0.8.0) --- updated-dependencies: - dependency-name: golang.org/x/sync dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by:
dependabot[bot] <support@github.com> Co-authored-by:
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
-
Inphi authored
-
- 05 Aug, 2024 4 commits
-
-
Mark Tyneway authored
The `etherscan` package isn't maintained or used in the monorepo, so it should be deleted.
-
Matt Joiner authored
-
smartcontracts authored
Minor fix to fuzz testing for SuperchainWETH and related contracts. Recipient chain ID cannot be the same as the sending chain ID. Contracts throw an error when this is the case, causing fuzz tests to fail.
-
Inphi authored
* Reduce ChannelTimeout to 50 in Granite * Capture granite genesis files. * Update snapshots. * Update op-geth dependency and resolve TODOs. * update op-geth dependency to rc.1 * fix TestGetRollupConfig --------- Co-authored-by:
Adrian Sutton <adrian@oplabs.co>
-
- 03 Aug, 2024 8 commits
-
-
Mark Tyneway authored
* contracts-bedrock: remove typescript Moves to using Go from typescript, new code is autogenerated by claude. Can confirm that both work as expected. The `check-test-names` is not running in CI, it fails for both the Go and Typescript scripts. Maybe claude will be able to get the script to pass. * Update check-spacers.go Co-authored-by:
semgrep-app[bot] <63493438+semgrep-app[bot]@users.noreply.github.com> * Update packages/contracts-bedrock/scripts/checks/check-test-names.go Co-authored-by:
semgrep-app[bot] <63493438+semgrep-app[bot]@users.noreply.github.com> * contracts-bedrock: fix build * scripts: rename * scripts: rename * scripts: fixes --------- Co-authored-by:
protolambda <proto@protolambda.com> Co-authored-by:
semgrep-app[bot] <63493438+semgrep-app[bot]@users.noreply.github.com>
-
Matt Solomon authored
-
Inphi authored
-
Matt Solomon authored
-
Sebastian Stammler authored
-
Matt Solomon authored
* doc: update style guide * apply suggestion from code review Co-authored-by:
Maurelian <john@oplabs.co> * Update packages/contracts-bedrock/STYLE_GUIDE.md --------- Co-authored-by:
Maurelian <john@oplabs.co>
-
Miki Noir authored
* delete expired docs link * use the official doc
-
protolambda authored
* contracts-bedrock: Go version of generate-invariant-docs * contracts-bedrock: Go version of generate-snapshots
-
- 02 Aug, 2024 1 commit
-
-
Blaine Malone authored
-