Fix operatorFee bug

.circleci/github-event-handler.yml

@@ -0,0 +1,9 @@
+version: 1
+listen-for-events:
+  pull_request:
+    #https://docs.github.com/en/webhooks/webhook-events-and-payloads#issues
+    types: [labeled]
+    event-to-parameters-mappings:
+      - pull_request_number: .pull_request.number
+      - label_name: .label.name
+

.cursor/rules/solidity-styles.mdc

@@ -0,0 +1,71 @@
+---
+description:
+globs: *.sol
+alwaysApply: false
+---
+# Optimism Solidity Style Guide
+
+Applies to Solidity files.
+
+## Comments
+- Use triple-slash solidity natspec comment style
+- Always use `@notice` instead of `@dev`
+- Use a line-length of 100 characters
+- Custom tags:
+  - `@custom:proxied`: Add to a contract whenever it's meant to live behind a proxy
+  - `@custom:upgradeable`: Add to a contract whenever it's meant to be inherited by an upgradeable contract
+  - `@custom:semver`: Add to `version` variable which indicate the contracts semver
+  - `@custom:legacy`: Add to an event or function when it only exists for legacy support
+  - `@custom:network-specific`: Add to state variables which vary between OP Chains
+
+## Errors
+- When adding new errors, always use custom Solidity errors
+- Custom errors should take the format `ContractName_ErrorDescription`
+
+## Naming Conventions
+- Function parameters should be prefixed with an underscore
+- Function return arguments should be suffixed with an underscore
+- Event parameters should NOT be prefixed with an underscore
+- Immutable variables:
+  - Should be in `SCREAMING_SNAKE_CASE`
+  - Should be `internal`
+  - Should have a hand-written getter function
+- Spacers must be named `spacer_<slot>_<offset>_<length>` and be `private`
+
+## Upgradeability
+- Contracts should be built assuming upgradeability by default
+- Extend OpenZeppelin's `Initializable` or base contract
+- Use the `ReinitializableBase` contract
+- Include an `initialize` function with the `reinitializer(initVersion())` modifier
+- If including an `upgrade` function, use the `reinitializer(initVersion())` modifier
+- In the constructor:
+  - Call `_disableInitializers()`
+  - Set any immutables (though generally avoid immutables)
+
+## Versioning
+- All non-library/non-abstract contracts must inherit `ISemver` and expose `version()`
+- Production-ready contracts must have version `1.0.0` or greater
+- Version increments:
+  - `patch`: changes that don't modify bytecode (comments)
+  - `minor`: changes that modify bytecode OR expand ABI without breaking existing interface
+  - `major`: changes that break existing interface OR modify security model
+
+AI code review tools should NOT comment on the choice of version increment.
+
+## Dependencies
+- Prefer OpenZeppelin's Upgradeable contracts for basic functionality
+
+## State Changes
+- All state changing functions should emit a corresponding event
+
+## Testing
+- Tests should be written using Foundry
+- For testing reverts with low-level calls, use the `revertsAsExpected` pattern
+- Test function naming: `[method]_[functionName]_[reason]_[status]`
+  - `method`: `test`, `testFuzz`, or `testDiff`
+  - `status`: `succeeds`, `reverts`, `works`, `fails`, or `benchmark`
+- Test contract naming:
+  - `TargetContract_TestInit`: for basic setup contracts
+  - `TargetContract_FunctionName_Test`: for happy path tests
+  - `TargetContract_FunctionName_TestFail`: for sad path tests
+  - `TargetContract_Harness`: for harness contracts

.github/CODEOWNERS

@@ -2,7 +2,6 @@
 * @ethereum-optimism/go-reviewers
 
 # OP Stack general
-/bedrock-devnet @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 /op-alt-da      @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 /op-batcher     @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 /op-chain-ops   @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
@@ -11,6 +10,7 @@
 /op-proposer    @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 /op-wheel       @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 /ops-bedrock    @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
+/op-devstack    @ethereum-optimism/op-stack @ethereum-optimism/go-reviewers
 
 # Expert areas
 /op-node/rollup @ethereum-optimism/consensus @ethereum-optimism/go-reviewers
@@ -28,10 +28,13 @@
 /op-e2e/faultproofs    @ethereum-optimism/proofs @ethereum-optimism/go-reviewers
 
 # Ops
-/.circleci       @ethereum-optimism/monorepo-ops-reviewers
-/.github         @ethereum-optimism/monorepo-ops-reviewers
-/ops             @ethereum-optimism/monorepo-ops-reviewers
-/docker-bake.hcl @ethereum-optimism/monorepo-ops-reviewers
+/.cursor/rules/solidity-styles.mdc @ethereum-optimism/contract-reviewers
 
 # Contracts
-/packages/contracts-bedrock     @ethereum-optimism/contract-reviewers
+# We require a minimum of 2 reviewers for all changes to the contracts-bedrock
+# directory unless only markdown files are being changed.
+/packages/contracts-bedrock/**           @ethereum-optimism/contract-reviewers #[min:2]
+/packages/contracts-bedrock/**/*.md      @ethereum-optimism/contract-reviewers
+
+# Security docs
+/docs @ethereum-optimism/evm-safety

.github/cliff.toml

@@ -0,0 +1,82 @@
+# git-cliff ~ configuration file
+# https://git-cliff.org/docs/configuration
+
+[remote.github]
+owner = "ethereum-optimism"
+repo = "optimism"
+# token = "" # Set via the GITHUB_TOKEN environment variable.
+
+[changelog]
+# A Tera template to be rendered for each release in the changelog.
+# See https://keats.github.io/tera/docs/#introduction
+body = """
+## What's Changed
+
+{%- if version %} in {{ version }}{%- endif -%}
+{% for commit in commits %}
+  {% if commit.remote.pr_title -%}
+    {%- set commit_message = commit.remote.pr_title -%}
+  {%- else -%}
+    {%- set commit_message = commit.message -%}
+  {%- endif -%}
+  * {{ commit_message | split(pat="\n") | first | trim }}\
+    {% if commit.remote.username %} by @{{ commit.remote.username }}{%- endif -%}
+    {% if commit.remote.pr_number %} in \
+      [#{{ commit.remote.pr_number }}]({{ self::remote_url() }}/pull/{{ commit.remote.pr_number }}) \
+    {%- endif %}
+{%- endfor -%}
+
+{%- if github -%}
+{% if github.contributors | filter(attribute="is_first_time", value=true) | length != 0 %}
+  {% raw %}\n{% endraw -%}
+  ## New Contributors
+{%- endif %}\
+{% for contributor in github.contributors | filter(attribute="is_first_time", value=true) %}
+  * @{{ contributor.username }} made their first contribution
+    {%- if contributor.pr_number %} in \
+      [#{{ contributor.pr_number }}]({{ self::remote_url() }}/pull/{{ contributor.pr_number }}) \
+    {%- endif %}
+{%- endfor -%}
+{%- endif -%}
+
+{% if version %}
+    {% if previous.version %}
+      **Full Changelog**: {{ self::remote_url() }}/compare/{{ previous.version }}...{{ version }}
+    {% endif %}
+    🚢 Docker Image: https://us-docker.pkg.dev/oplabs-tools-artifacts/images/{{ version | replace(from="/", to=":") }}
+{% else -%}
+  {% raw %}\n{% endraw %}
+{% endif %}
+
+{%- macro remote_url() -%}
+  https://github.com/{{ remote.github.owner }}/{{ remote.github.repo }}
+{%- endmacro -%}
+"""
+# Remove leading and trailing whitespaces from the changelog's body.
+trim = true
+# A Tera template to be rendered as the changelog's footer.
+# See https://keats.github.io/tera/docs/#introduction
+footer = """
+<!-- generated by git-cliff -->
+"""
+# An array of regex based postprocessors to modify the changelog.
+# Replace the placeholder `<REPO>` with a URL.
+postprocessors = []
+
+[git]
+# Parse commits according to the conventional commits specification.
+# See https://www.conventionalcommits.org
+conventional_commits = false
+# Exclude commits that do not match the conventional commits specification.
+filter_unconventional = false
+# Split commits on newlines, treating each line as an individual commit.
+split_commits = false
+# An array of regex based parsers to modify commit messages prior to further processing.
+commit_preprocessors = [{ pattern = '\((\w+\s)?#([0-9]+)\)', replace = "" }]
+# Exclude commits that are not matched by any commit parser.
+filter_commits = false
+# Order releases topologically instead of chronologically.
+topo_order = false
+# Order of commits in each group/release within the changelog.
+# Allowed values: newest, oldest
+sort_commits = "newest"

.github/dependabot.yml

@@ -1,45 +1,5 @@
 version: 2
 updates:
-  - package-ecosystem: "docker"
-    directory: "/ops-bedrock"
-    schedule:
-      interval: "daily"
-      day: "tuesday"
-      time: "14:30"
-      timezone: "America/New_York"
-    open-pull-requests-limit: 10
-    commit-message:
-      prefix: "dependabot(docker): "
-    labels:
-      - "M-dependabot"
-
-  - package-ecosystem: "github-actions"
-    directory: "/"
-    schedule:
-      interval: "weekly"
-      day: "tuesday"
-      time: "14:30"
-      timezone: "America/New_York"
-    open-pull-requests-limit: 10
-    commit-message:
-      prefix: "dependabot(actions): "
-    labels:
-      - "M-dependabot"
-
-  - package-ecosystem: "npm"
-    directory: "/"
-    schedule:
-      interval: "weekly"
-      day: "tuesday"
-      time: "14:30"
-      timezone: "America/New_York"
-    open-pull-requests-limit: 10
-    versioning-strategy: "auto"
-    commit-message:
-      prefix: "dependabot(npm): "
-    labels:
-      - "M-dependabot"
-
   - package-ecosystem: "gomod"
     directory: "/"
     schedule:

.gitignore

@@ -4,6 +4,8 @@ results
 temp
 coverage.json
 **/lcov.info
+**/lcov-upgrade.info
+**/lcov-all.info
 
 yarn-error.log
 .yarn/*
@@ -18,7 +20,8 @@ cache
 !op-chain-ops/foundry/testdata/srcmaps/cache
 !op-chain-ops/foundry/testdata/srcmaps/artifacts
 
-packages/contracts-bedrock/deployments/devnetL1
+!op-deployer/pkg/deployer/artifacts
+
 packages/contracts-bedrock/deployments/anvil
 
 # vim
@@ -41,8 +44,10 @@ packages/contracts-bedrock/deployments/anvil
 
 coverage.out
 
-
 __pycache__
 
 # Ignore echidna artifacts
 crytic-export
+
+# ignore local asdf config
+.tool-versions

.gitmodules

@@ -13,7 +13,7 @@
 [submodule "packages/contracts-bedrock/lib/safe-contracts"]
 	path = packages/contracts-bedrock/lib/safe-contracts
 	url = https://github.com/safe-global/safe-contracts
-	branch = v1.4.0
+	branch = v1.4.1
 [submodule "packages/contracts-bedrock/lib/kontrol-cheatcodes"]
 	path = packages/contracts-bedrock/lib/kontrol-cheatcodes
 	url = https://github.com/runtimeverification/kontrol-cheatcodes
@@ -23,9 +23,6 @@
 [submodule "packages/contracts-bedrock/lib/lib-keccak"]
 	path = packages/contracts-bedrock/lib/lib-keccak
 	url = https://github.com/ethereum-optimism/lib-keccak
-[submodule "packages/contracts-bedrock/lib/automate"]
-	path = packages/contracts-bedrock/lib/automate
-	url = https://github.com/gelatodigital/automate
 [submodule "packages/contracts-bedrock/lib/openzeppelin-contracts-v5"]
 	path = packages/contracts-bedrock/lib/openzeppelin-contracts-v5
 	url = https://github.com/OpenZeppelin/openzeppelin-contracts
@@ -35,3 +32,10 @@
 [submodule "da-server"]
 	path = da-server
 	url = https://github.com/ethstorage/da-server
+[submodule "packages/contracts-bedrock/lib/superchain-registry"]
+	path = packages/contracts-bedrock/lib/superchain-registry
+	url = https://github.com/ethereum-optimism/superchain-registry
+[submodule "op-geth"]
+	path = op-geth
+	url = https://github.com/QuarkChain/op-geth
+	branch = op-es