feat(ctb): Forge invariant tests

fix(ctb): remove unnecessary ERC20 factory code

fix(ctb): missing FeeVault comments

feat(ctb): remove unnecessary ERC721 factory code

gas snapshot

feat(ctb): receivedMessages => failedMessages

Removes a legacy getter that was mistakenly added to the ERC20 factory.
We never had a getter for the bridge variable.

Adds missing constructor comments in the FeeVault contracts.

First removes the legacy bridge and remoteChainId getter functions.
Since the ERC721 factory is being deployed at a new address, it's not
necessary to maintain ABI compatibility. We can safely remove these
functions. Also removes constructor checks that aren't really necessary
since we can always upgrade if needed and the odds of getting something
like that wrong don't seem significant enough to warrant extra code.

fix(ctp): Downgrade `contracts-bedrock` dep to fix CI

Version Packages

Trigger release

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

fix(ctb): correct GPO abi and layout (#4582)

Fixes some small errors in the GPO's API and storage layout that did not
conform to the standards for how we are ugprading the L2 system.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Trigger Release

Changes the name of the receivedMessages variable to failedMessages.
Original name was confusing as it suggests that all messages will be
recorded in that mapping, even though only failed messages are recorded
in that mapping. Has created issues on several occasions.

op-node: Fix too eager advance in batch queue

op-node: Add stubbed Goerli config

op-chain-ops: better deploy config validation