- 19 Aug, 2022 19 commits
-
-
Mark Tyneway authored
For some reason `strings.TrimPrefix` is removing an extra character sometimes. We want to remove the `.json` extension and using it will sometimes also remove the final character in the contract name. This is problematic because when you call `hh.GetDeployment`, with the name of the contract, it will not be able to find the deployment because the name of the deployment will not match the name passed in by the user. This instead slices off the `.json` from the string manually.
-
Mark Tyneway authored
- Add support for `&common.Address` - Take locks in initialization of `hardhat.Hardhat`
-
Mark Tyneway authored
contracts-bedrock: migrate deploy config to json
-
Mark Tyneway authored
Migrates the deploy config from typescript to json so that it can be read into a go program.
-
Mark Tyneway authored
Adds the ability to serialize strings in state. It only supports strings less than 32 bytes long which would require major refactoring and we do not currently need that functionality. Main test cases come from mainnet WETH contract Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
smartcontracts authored
Updates the SDK to load contract addresses from the deployments folder. Also updates the Contracts package to export addresses from the deployments folder. More robust to address changes. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
smartcontracts authored
Makes it possible to easily deploy via PK if not deploying via Ledger. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
smartcontracts authored
Refactors contract addresses and related constants into the chain-constants.ts file. Much cleaner separation of logic. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
state-surgery: implement bytes32 handling
-
Mark Tyneway authored
state-surgery: fix flakey tests
-
Mark Tyneway authored
contracts-bedrock: make a library call internal
-
Mark Tyneway authored
state-surgery: handle fs errors
-
Mark Tyneway authored
-
Mark Tyneway authored
Sometimes `require.Equal` flakes on the comparison of `big.Int`. https://github.com/ethereum-optimism/optimism/blob/b31d35b67755479645dd150e7cc8c6710f0b4a56/op-node/rollup/derive/fuzz_parsers_test.go#L42Co-authored-by:
Joshua Gutow <jgutow@optimism.io>
-
Mark Tyneway authored
Adds the ability to set bytes32 values in state. There is a single bytes32 value in our L2 predeploys. Adds test coverage that the value is set properly and encoded properly. Both `common.Hash` and hex strings are supported to be able to be encoded into a bytes32 suitable for storage.
-
smartcontracts authored
Fixes a bug in the SDK that would cause withdrawals to revert for certain custom bridges on Kovan. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
Handle errors when walking the filesystem so that they do not silently fail.
-
Mark Tyneway authored
So we don't have to handle library linking in the state surgery
-
Joshua Gutow authored
I forgot to initialize the forceReset channel which means that the RPC isn't able to signal to the state loop that it needs to reset. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
- 18 Aug, 2022 8 commits
-
-
smartcontracts authored
Removes the ICrossChainMessenger interface. We don't have a strong reason to maintain this interface and it introduces complexity to the process of developing the SDK. With the interface, we are required to make changes to multiple files when we simply want to change the CrossChainMessenger class. We are also not maintaing multiple implementations of the CrossChainMessenger that would make the interface useful. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
smartcontracts authored
Deploys Drippie to the Optimism Goerli testnet. Have not set any configurations up yet, still waiting on Gelato support. Not verified on Etherscan since we don't have an Etherscan for OP Goerli yet. Addresses are different than on other chains because of the update to Solidity 0.8.15. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
smartcontracts authored
Updates Drippie config for mainnet. A new drip was created after the previous drip was archived. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
* predeploys: fix address names Updates the address names and also create a mapping of addresses so that they can be iterated over. * op-bindings: export predeploys mapping * fix test Co-authored-by:
Matthew Slipper <me@matthewslipper.com>
-
Mark Tyneway authored
contracts-bedrock: update @foundry-rs/hardhat-forge
-
Mark Tyneway authored
Fixes the build issues in CI
-
Joshua Gutow authored
* op-node: Add ResetDerivationPipeline RPC This is an RPC call this is useful for testing the robustness of the derivation pipeline to resets. * op-node: Create new admin namespace Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
* state-surgery: add solc package Move types into solc package that are associated with solc. Add `CompilerInput` and `CompilerOutput` types. * state-surgery: clean up hardhat package Use some solc types * state-surgery: add state package Implement some of smock's utils in go * state-surgery: cleanup * state-surgery: refactor + add better test coverage * state-surgery: cleanup * state-surgery: add tests for merging storage slots * state-surgery: godoc * state-surgery: more tests * state-surgery: more cleanup
-
- 16 Aug, 2022 4 commits
-
-
Diederik Loerakker authored
-
Mark Tyneway authored
* sdk: fix eth withdraw bug Fixes the eth withdrawal bug when creating withdrawl transactions that include value. There is no need to send ETH on L1 when finalizing the withdrawal since the `OptimismPortal` is holding onto the ETH that will be unlocked and sent to the user. co-authored-by:
smartcontracts <kelvin@optimism.io> * sdk: cleanup code + fix hanging eth deposits co-authored-by:
smartcontracts <kelvin@optimism.io> * ci: run devnet on sdk changes * lint: fix Co-authored-by:
smartcontracts <kelvin@optimism.io> Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Diederik Loerakker authored
Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
Deploy the `AddressManager` and the `ProxyAdmin`. These contracts are required as part of the system on L1. The indexer requires the `AddressManager` to be deployed. Having the `ProxyAdmin` deployed will make the devnet setup more similar to how the system will be deployed in production. Note that these deployments are only good for fresh deployments. We will need to update the deploy scripts to run against existing networks in the future. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
- 15 Aug, 2022 4 commits
-
-
Mark Tyneway authored
* indexer: clean up duplicated code Remove `ParseL1Address` and `ParseL2Address` and instead use `ParseAddress`. Addresses on L1 and L2 are the same and we can use as much upstream code as possible instead of code from `l2geth`. * Update indexer/crypto.go Co-authored-by:
Javed Khan <javed@optimism.io> Co-authored-by:
Javed Khan <javed@optimism.io> Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
Deletes an unused file from the indexer. The debug file is created by hardhat and will point to the build info file and this isn't needed by the indexer. Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Mark Tyneway authored
Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Matthew Slipper authored
To improve the concurrency of the Bedrock actor tests, I hooked up an ETH faucet. Now, the actors can use the faucet to fund themselves before each run.
-
- 13 Aug, 2022 2 commits
-
-
Joshua Gutow authored
* op-node: Add comments to pipeline stages This explains each stage in a bit better detail. * Update op-node/rollup/derive/attributes_queue.go Co-authored-by:
Javed Khan <javed@optimism.io> Co-authored-by:
Javed Khan <javed@optimism.io>
-
Diederik Loerakker authored
-
- 12 Aug, 2022 2 commits
-
-
Mark Tyneway authored
* state-surgery: add immutables package Add a package for dealing with immutables so that the deployed bytecode can be set directly in state. Each contract from the `op-bindings` package is imported and deployed to a simulated backend. The `op-bindings` package is enforced to be up to date by CI. Each of the contracts was double checked that the arguments passed are not immutables, the existing immutable values are hardcoded into the contract itself. To handle the case where immutables are dynamic, we will need to read in config and pass that through. The methodology on how to do that is documented in a TODO message. The build step runs in CI meaning that changes to the contract constructor interfaces will result in this package failing to build. This code could be slightly modularized to be used in `op-e2e` for creating the initial L2 state. Will leave that for an additional follow up PR. * state-surgery: additional sanity check Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-
Joshua Gutow authored
-
- 11 Aug, 2022 1 commit
-
-
Mark Tyneway authored
* state-surgery: hardhat utils Add hardhat utils for reading hardhat artifacts from disk. This can read hardhat artifacts, buildinfo and hardhat deploy artifacts. This is necessary for implementing the state surgery in go, because these files must be consumed as part of the state surgery process. * ci: run state-surgery tests in ci Co-authored-by:
mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
-