Update op-geth dependency to upstream geth v1.13.8 and migrate to slog

[sebastianst]

Description

Updated op-geth dependency to ethereum-optimism/op-geth#215 based on upstream geth v1.13.8

Also migrates logging infra to slog, after upstream migrated to slog in ethereum/go-ethereum#28187.

I opened an upstream PR to add the Handler getter to the Logger interface (slog's Logger interface itself has this getter). It's already in the op-geth version that this PR is based on.

Metadata

• Fixes https://github.com/ethereum-optimism/client-pod/issues/337

[codecov]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (fdcd8ff) 16.57% compared to head (6c61cc8) 24.62%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #8917      +/-   ##
===========================================
+ Coverage    16.57%   24.62%   +8.04%     
===========================================
  Files          119      104      -15     
  Lines         5098     3732    -1366     
  Branches      1130      536     -594     
===========================================
+ Hits           845      919      +74     
+ Misses        4178     2767    -1411     
+ Partials        75       46      -29     

Flag	Coverage Δ
cannon-go-tests	62.77% <0.00%> (?)
chain-mon-tests	?
contracts-bedrock-tests	0.65% <ø> (ø)
contracts-ts-tests	?
sdk-next-tests	?
sdk-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
cannon/cmd/run.go	0.00% <0.00%> (ø)
cannon/cmd/log.go	0.00% <0.00%> (ø)

... and 45 files with indirect coverage changes

[ajsutton]

Looks like the logging changes bit us.

[semgrep-app]

Semgrep found 10 sol-style-return-arg-fmt findings:

• packages/contracts-bedrock/scripts/libraries/LibSort.sol: L506, L387, L370, L353, L348, L331, L314, L279, L273, L267

Named return arguments to functions must be appended with an underscore (_)

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

Semgrep found 39 sol-style-notice-over-dev-natspec findings:

• packages/contracts-bedrock/scripts/libraries/LibSort.sol: L622, L581, L535, L497, L484, L476, L468, L457, L449, L439, L433, L427, L421, L415, L409, L403, L397, L391, L386, L369, L352, L347, L330, L313, L308, L303, L283, L277, L271, L265, L260, L255, L233, L221, L214, L64, L51, L44, L16

Prefer @notice over @dev in natspec comments

_{Ignore this finding from sol-style-notice-over-dev-natspec.}

[sebastianst]

Left a couple of pointers for reviewers.

[semgrep-app]

Semgrep found 7 ban-common-hex2bytes findings:

• op-service/testutils/random.go: L109
• op-challenger/game/fault/trace/alphabet/prestate.go: L13
• op-chain-ops/deployer/testdata.go: L20, L18, L16, L14, L12

Found banned use of common.Hex2Bytes. Use common.FromHex instead.

_{Ignore this finding from ban-common-hex2bytes.}

[trianglesphere]

Some small comments, but nothing big.

[protolambda]

Changes LGTM. A few imports seem to need lint fixes, specifically for "golang.org/x/exp/slog" in a few places