Bump ugorji/go/codec and remove codecgen

[friedemannf]

Summary

• Bumping ugorji/go to: https://github.com/ugorji/go/releases/tag/codec%2Fv1.3.2
• Due to the removal of codecgen in https://github.com/ugorji/go/releases/tag/codec%2Fv1.3.0, removing the generated files and codecgen target in Makefile
• The encoder will only use json.Unmarshaler if Marshaler is implemented as well, therefore adding a (txt *TxType) MarshalJSON() implementation that will make the encoder respect the already-existing (txt *TxType) UnmarshalJSON(data []byte)

Benchmarks

chainlink-evm/pkg/types/block_json_benchmark_test.go

Line 73 in 6bd5cdd

func BenchmarkBlock_Small_JSONUnmarshal(b *testing.B) {

Before

BenchmarkBlock_Small_JSONUnmarshal
BenchmarkBlock_Small_JSONUnmarshal-12     	  351284	      3520 ns/op
BenchmarkBlock_Medium_JSONUnmarshal
BenchmarkBlock_Medium_JSONUnmarshal-12    	   22908	     52183 ns/op
BenchmarkBlock_Large_JSONUnmarshal
BenchmarkBlock_Large_JSONUnmarshal-12     	    3014	    412256 ns/op
BenchmarkBlock_XL_JSONUnmarshal
BenchmarkBlock_XL_JSONUnmarshal-12        	     358	   3746920 ns/op

After

BenchmarkBlock_Small_JSONUnmarshal
BenchmarkBlock_Small_JSONUnmarshal-12     	  267518	      4241 ns/op
BenchmarkBlock_Medium_JSONUnmarshal
BenchmarkBlock_Medium_JSONUnmarshal-12    	   19368	     61072 ns/op
BenchmarkBlock_Large_JSONUnmarshal
BenchmarkBlock_Large_JSONUnmarshal-12     	    2614	    471092 ns/op
BenchmarkBlock_XL_JSONUnmarshal
BenchmarkBlock_XL_JSONUnmarshal-12        	     314	   3814720 ns/op

[github-actions]

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-evm

⚠️ Breaking Changes (8)

pkg/types/blocks.(*BlockInternal) (3)

• CodecDecodeSelf — 🗑️ Removed

• CodecEncodeSelf — 🗑️ Removed

• IsCodecEmpty — 🗑️ Removed

pkg/types/blocks.(*TransactionInternal) (3)

• CodecDecodeSelf — 🗑️ Removed

• CodecEncodeSelf — 🗑️ Removed

• IsCodecEmpty — 🗑️ Removed

pkg/types/blocks.(*TxType) (1)

• CodecDecodeSelf — 🗑️ Removed

pkg/types/blocks.TxType (1)

• CodecEncodeSelf — 🗑️ Removed

✅ Compatible Changes (1)

pkg/types/blocks.(*TxType) (1)

• MarshalJSON — ➕ Added

---

📄 View full apidiff report

[Copilot]

🟡 Changes recommended

TxType.MarshalJSON can panic on a nil receiver, which is plausible when invoked via interface-based marshaling (e.g., through go/codec).

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the project’s JSON serialization path for EVM block/transaction intermediate types by upgrading github.com/ugorji/go/codec and removing the previously checked-in codecgen output, relying on non-generated behavior instead.

Changes:

• Bump github.com/ugorji/go/codec from v1.2.12 to v1.3.2.
• Remove codecgen usage: delete generated internal_types_codecgen.go, remove the //go:generate codecgen ... directive, and drop the codecgen Makefile target.
• Add TxType.MarshalJSON to ensure go/codec can use the custom JSON unmarshaller/serialization behavior for TxType.

File summaries

File	Description
pkg/types/blocks/transactions.go	Adds TxType.MarshalJSON alongside the existing custom UnmarshalJSON.
pkg/types/blocks/internal_types_codecgen.go	Removes generated codecgen implementation.
pkg/types/blocks/block.go	Removes //go:generate codecgen ... directive.
Makefile	Removes codecgen install target and from generate dependencies.
go.mod / go.sum	Updates github.com/ugorji/go/codec dependency version and sums.

Review details

• Files reviewed: 5/6 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.