feat: Move checkOutput from DeployOPChainOutput to DeployOPChain script

[maurelian]

Description

Moves checkOutput() and assertValidDeploy() and all other similar assertX() functions from DeployOPChainOutput to DeployOPChain, as the I/O contracts should be kept as simple as possible.

The amount of code which is moved makes the diff confusing to read. The actual changes are limited to:

1. moving the code as described above
2. Adding a DeployOutput _doo argument to each assert function
3. Prepending calls to get contract addresses with _doo (ie. opChainProxyAdmin() becomes _doo.opChainProxyAdmin().

[semgrep-app]

Semgrep found 6 sol-style-input-arg-fmt findings:

• packages/contracts-bedrock/src/L2/interfaces/ISuperchainWETH.sol
  • L20 - Triage
  • L25 - Triage
  • L26 - Triage
  • L29 - Triage
  • L30 - Triage
  • L32 - Triage

Inputs to functions must be prepended with an underscore (_)

_{Ignore this finding from sol-style-input-arg-fmt.}

Semgrep found 1 golang_fmt_errorf_no_params finding:

• op-supervisor/supervisor/backend/db/db_test.go
  • L280 - Triage

No fmt.Errorf invocations without fmt arguments allowed

_{Ignore this finding from golang_fmt_errorf_no_params.}

Semgrep found 1 math-random-used finding:

• op-supervisor/supervisor/backend/source/head_monitor_test.go
  • L7 - Triage

Do not use math/rand. Use crypto/rand instead.

_{Ignore this finding from math-random-used.}

Semgrep found 1 marshal-json-pointer-receiver finding:

• op-supervisor/supervisor/backend/db/heads/types.go
  • L99 - Triage

MarshalJSON with a pointer receiver has surprising results: golang/go#22967

_{Ignore this finding from marshal-json-pointer-receiver.}

Semgrep found 1 err-nil-check finding:

• op-supervisor/supervisor/backend/db/db.go
  • L108-112 - Triage

superfluous nil err check before return

_{Ignore this finding from err-nil-check.}

[blmalone]

LGTM