* contracts-governance: clean up the hardhat config

Deletes some unused types and cleans up the hh config file

* contracts-governance: more cleanup
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Cleans up natspec for Drippie and its dependencies.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Minor change, moves the TeleportrWithdrawer contract to the L1 folder to
keep in line with the placing of other Teleportr contracts. We currently
only use this contract on the L1 side. If we ever want an L2 to L1
teleportr, we might want to put all of these things in universal.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* contracts: fix typechain packaging

* deps: update typechain

* dtl: fix typechain import
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* contracts-periphery: add typechain

* ctp: fix depcheck
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* style(ctb): clean up hashing and encoding libs

Cleans up Hashing and Encoding libraries. Updates natspec and cleans up
function names.

* contracts-bedrock: cover nonce decoding

* contracts-bedrock: regenerate storage-layout
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>

Copies TeleportrDeposit and TeleportrDisburser to the contracts-periphery
package in preparation for an upcoming audit.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Makes it easier to debug when problems go wrong
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* op-node: testnet fixes - sanity check L2 block after block derivation pipeline reset, and fix log level of attributes queue

* Update op-node/rollup/derive/attributes_queue.go
Co-authored-by: Joshua Gutow <jgutow@optimism.io>
Co-authored-by: Joshua Gutow <jgutow@optimism.io>

Ports tests from `contracts`. Will help with coverage results.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Cleans up CrossDomainUtils, CrossDomainHashing, and WithdrawalVerifier
into two new libraries, Hashing and Encoding. Will be followed up by a
second PR to clean up function names and comments.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* op-bindings: test that op-bindings predeploy addresses match geth

* go: update bedrock geth dependency

* op-e2e: drop old Enabled flag
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Cleans up the PredeployAddresses contract. I went ahead and clearly
marked and separated the legacy addresses and the non-legacy addresses.
I also renamed the L2StandardTokenFactory address to correspond to the
new name (OptimismMintableERC20Factory) and deleted several predeploy
addresses which no longer exist in the state of L2.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

This will allow the `DepositTx` class to be more easily
reusable across projects.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* contracts-bedrock: less compiler warnings

Cleans up compiler warnings that were introduced in https://github.com/ethereum-optimism/optimism/pull/2938.
A bunch of unused variables are deleted, reverts are being asserted on
so no need to assign variables.

* contracts-bedrock: regenerate snapshot
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Removes "not implemented" errors in virtual functions. By removing these
errors, developers will get compile-time errors instead of runtime
errors which are easier to catch.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Enums should have CapitalCase names and UPPER_SNAKE_CASE values.
Standardizes around that. Also removes an unused enum in MerkleTrie.sol.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Errors weren't being bubbled up from subshells to the parent script.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* contracts-bedrock: add semver to L2 contracts

* contracts-periphery: add mkdirp for build

* deps: update and standardize some js deps

Without this commit, the build would fail due
to conflicting dependency versions

* deps: fix depcheck

* contracts-periphery: move semver into contracts-bedrock

* contracts-bedrock: use internal Semver contract

* contracts: add semver tag to natspec

* op-bindings: regenerate

* contracts-bedrock: snapshot

* contracts-bedrock: add semver to L1 contracts

* op-bindings: regenerate

* feat(ctb): clean up initialization logic

Cleans and standardizes the initialization logic across all contracts.
Importantly, removes reinitialization.

* contracts-bedrock: clean up hh genesis task
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>

Cleans up a few solc warnings in Proxy.sol.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Unnecessary library, only being used in a single place now and it can
just be an internal constant. I see no strong reason for this library to
continue to exist.

* ci: add devnet to ci

Prevent bugs that break the devnet by
doing a deposit in CI on the devnet

* contracts-bedrock: reorder imports
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* fix(ctb): rename token factory to erc20 factory

With the intent of mirroring the naming of the ERC721 bridge, I'm
renaming the OptimismMintableTokenFactory to
OptimismMintableERC20Factory.

* contracts-bedrock: fix test

* contracts-bedrock: snapshot
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>

Cleans up comments and errors for legacy contracts. It's fine to update
the bytecode for these legacy contracts since we're really only using
them for their interfaces.

style(ctb): Remove underscores from event args