- 26 Oct, 2024 8 commits
-
-
Delweng authored
Signed-off-by:
jsvisa <delweng@gmail.com>
-
smartcontracts authored
Updates CI to check that the Kontrol summary dummy files are not modified by any PR unless the change is deliberate. Committed files are dummy files and aren't meant to change. Alternative here would be to generate the dummy files with a script that gets triggered before forge builds stuff. Main concern with using an autogen script is that the script would need to be added in front of build commands all over the place. Committing the files and checking that they aren't changed is slightly more janky but people rarely re-generate these files anyway so the projected impact is minimal. If this becomes a devx hassle we can consider autogen.
-
DenseDenise authored
* optimize the example cannon command * Update cannon/README.md Co-authored-by:
smartcontracts <kelvinfichter@gmail.com> * Update cannon/README.md Co-authored-by:
smartcontracts <kelvinfichter@gmail.com> --------- Co-authored-by:
smartcontracts <kelvinfichter@gmail.com>
-
Michael Amadi authored
* use deploy utils over new* in tests * replace new * with use of deployUtils * fix failing test * use deployutils for weth98 test file * fix semgrep * fixes * fixes * fixes...
-
Matthew Slipper authored
* op-deployer: Add Docker builds * fix job syntax * specify dockerfile * remove accidental whitespace changes
-
smartcontracts authored
Replaces the golang package for natspec/semver matching with a semgrep rule. Code reduction is good.
-
dependabot[bot] authored
Bumps [github.com/minio/minio-go/v7](https://github.com/minio/minio-go) from 7.0.78 to 7.0.79. - [Release notes](https://github.com/minio/minio-go/releases) - [Commits](https://github.com/minio/minio-go/compare/v7.0.78...v7.0.79) --- updated-dependencies: - dependency-name: github.com/minio/minio-go/v7 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by:
dependabot[bot] <support@github.com> Co-authored-by:
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
-
Michael Amadi authored
* fix failing interfaces * fix crossl2inbox, revert some changes * fix semgrep error
-
- 25 Oct, 2024 11 commits
-
-
smartcontracts authored
Removes the FeeVaultWithdrawal.s.sol script that was previously used to trigger withdrawals manually. All of this is automated now and you can just use Etherscan if you really want to do things manually so the value of the script is minimal.
-
Michael Amadi authored
* add interfaces for safe contracts, use named imports for DeputyGuardianModule * fix failing test
-
protolambda authored
* op-supervisor: cross-safe-updates PR squashed op-supervisor: experimental cross-safety, with hazard detection tweak: Add some errors/error returns in backend/cross. wip: Chain index <> ID mapping. fix: Check parent instead of re-checking hazardBlock. Remove Hazard Work Write missing DB Bindings OpenBlock, LocallyDerivedFrom, CrossDerivedFrom Configurable WorkFn for Workers op-supervisor: move chain-index <> chain ID translation into dependency set, fix some interfaces op-supervisor: update cross-safety worker routine op-supervisor: update more error handling op-supervisor: move errors to types package op-supervisor: check CanExecuteAt and CanInitiateAt op-supervisor: determine cross-safe candidate and L1 scope, and more fixes todo L1 scope increment op-supervisor: cross-safe L1 scope bump op-supervisor: dependency set getter op-supervisor: L1 scope increment fix op-supervisor: fix cross-safe updates typing op-node: signal L1 traversal of derivation to supervisor op-supervisor: fromda fixes and tests op-supervisor: fix OpenBlock, fix/add missing interface methods, hook up cross-safe worker routines OpenBlock to return map[uint32]ExecutingMessage Add Frontier Unit Tests fix WithParent panic op-node: register L1 traversal with op-supervisor op-node,op-supervisor: add logging, temp work around for interop local-safe updates Add safe_start_test, unsafe_start_test Add safe_update_test and unsafe_update_test add worker_test op-supervisor: fix cross-safe L1 scope bumping op-supervisor: fix logs DB test Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt> * op-node: fix interop deriver test * op-e2e: fix interop action test * op-supervisor: improve map init * op-node: link interop TODO comment to issue, in engine events emitter * op-supervisor: cleanup Worker instances of tests --------- Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt>
-
zhiqiangxu authored
-
Michael Amadi authored
* update test folder so that smegrep require and revert checks pass for it * fix test
-
smartcontracts authored
* feat(ct): add semgrep rule to use encodeCall encodeCall is almost always better than encodeWithSelector because it maintains type safety. Prefer encodeCall unless encodeWithSelector is actually necessary. * maint(ct): update contracts to use encodeCall Updates a number of contracts to use encodeCall instead of encodeWithSelector where possible.
-
Adrian Sutton authored
-
Inphi authored
* cannon: Simplify load/stores with helper functions * use subword utils in MIPS.sol * lint MIPS.sol * add natspec to MIPSInstructions.sol * use updateSubWord in MIPSInstructions.sol * bump MIPS contract semver * fix nits
-
Inphi authored
* cannon: Remove memory.SetUint32 Remove uint32 word stores from the `mipsevm.memory` interface. `SetUint32` is inflexible due to its word-alignment constraints. This prevents tests for 32 and 64-bit VMs from using the same program counter values when writing instructions to memory. Instead, tests should use the new `testutil.StoreInstruction` utility function to write instructions to any naturally aligned memory location. * use arch.Word csats in go-ffi
-
Matthew Slipper authored
Update WaitForBlock to maintain two timeouts: a no-change timeout, which fires if the chain's head does not change within a specified window, and an absolute timeout, which fires if the chain's head does not meet or exceed the specified block. These changes should ideally reduce the number of test flakes we're seeing. Everything takes longer when test executors are under load; by maintaining these two timeouts we can provide longer-running tests with more buffer while retaining the ability to fail fast if the chain gets stuck. As part of this PR I also refactored the wait method to use polling rather than WebSockets. I've found WebSockets to be unreliable in tests.
-
clabby authored
-
- 24 Oct, 2024 20 commits
-
-
protolambda authored
* op-supervisor: DB improvements for cross-safe updates Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt> * op-supervisor: dependency-set improvements Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt> --------- Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt>
-
protolambda authored
Co-authored-by:
axelKingsley <axel.kingsley@gmail.com> Co-authored-by:
Tyler Smith <mail@tcry.pt>
-
mbaxter authored
* op-program: Compile op-program for Cannon64 * op-program: Update comment
-
Matthew Slipper authored
op-deployer generates rollup configs, and those rollup configs are currently broken for older versions of op-node that don't support reading the EIP1559Params field in the SystemConfig. This PR adds a meta field within the SystemConfig that, when enabled, marshals the SystemConfig without the EIP1559Params field. This solution is backwards-compatible and minimally invasive, requiring no changes to the consensus-critical code that consumes the SystemConfig elsewhere. Closes https://github.com/ethereum-optimism/optimism/issues/12615.
-
Disco authored
* feat: support permit2 on superchainweth * chore: run pre-pr --------- Co-authored-by:
agusduha <agusnduha@gmail.com> Co-authored-by:
gotzenx <78360669+gotzenx@users.noreply.github.com>
-
smartcontracts authored
Updates semgrep config to apply expectRevert to tests. Fixes a few instances where this wasn't being followed.
-
Adrian Sutton authored
-
Matthew Slipper authored
This test fails a lot when executors are under load. This PR changes the test to find the first block with more than one batcher transaction in it, which should be sufficient to assert that the batcher is submitting multiple transactions at once.
-
smartcontracts authored
Bumps the number of heavy fuzz runs to 20k. 10k recently missed a flake so bumping to 20k to try to avoid that in the future. Not perfect but it will reduce the probability a bit. Developers can still manually set the number of fuzz runs for a particular test with annotations if 20k is too high.
-
clabby authored
* inclusion proof * fix test * add inclusion proof * goimports
-
rickck11 authored
* optimize the doc * small fix * reomve outdated content
-
Adrian Sutton authored
-
smartcontracts authored
Applies the semgrep rule for return-arg-fmt to Solidity tests.
-
Gutflo authored
-
Inphi authored
* cannon: Finish emulating rest of 64-bit instructions This fixes the 64-bit stubs for various instructions (except lld/scd). * review comments; fix dmult * add todo * test div by zero * add a couple more dmultu tests * remove dead code * cannon: Fix remaining mips64 emulation bugs * fix 64-bit Makefile build script; review comments * fix build script
-
Inphi authored
-
Paul Dowman authored
-
AgusDuha authored
* fix: rename crosschain events (#109) * fix: rename crosschain events * fix: semver --------- Co-authored-by:
Disco <131301107+0xDiscotech@users.noreply.github.com> Co-authored-by:
0xng <ng@defi.sucks> Co-authored-by:
0xParticle <particle@defi.sucks> Co-authored-by:
gotzenx <78360669+gotzenx@users.noreply.github.com> * fix: pre pr * fix: pre pr --------- Co-authored-by:
Disco <131301107+0xDiscotech@users.noreply.github.com> Co-authored-by:
0xng <ng@defi.sucks> Co-authored-by:
0xParticle <particle@defi.sucks> Co-authored-by:
gotzenx <78360669+gotzenx@users.noreply.github.com>
-
AgusDuha authored
* fix: remove superchain erc20 modifier (#111) * fix: remove superchain erc20 modifier --------- Co-authored-by:
Disco <131301107+0xDiscotech@users.noreply.github.com> Co-authored-by:
0xng <ng@defi.sucks> Co-authored-by:
0xParticle <particle@defi.sucks> Co-authored-by:
gotzenx <78360669+gotzenx@users.noreply.github.com> * fix: pre pr --------- Co-authored-by:
Disco <131301107+0xDiscotech@users.noreply.github.com> Co-authored-by:
0xng <ng@defi.sucks> Co-authored-by:
0xParticle <particle@defi.sucks> Co-authored-by:
gotzenx <78360669+gotzenx@users.noreply.github.com>
-
John Chase authored
* add WakeupTraversal_WithExitedThreads * In the TestEVM_WakeupTraversal_WithExitedThreads test, explicitly set the current active thread to exited state (activeThread.Exited = true) to test the wakeup traversal behavior when the active thread has exited && Modified test cases by resetting the active thread's index from exitedThreadIdx to avoid duplicate settings and confusion. * setting Futex fields with varying values.
-
- 23 Oct, 2024 1 commit
-
-
Adrian Sutton authored
-