The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).

Claude finished @julienrbrt's task in 1m 39s —— View job

Code Review

• Gather context and analyze PR changes
• Review dependency updates
• Check for consistency across modules
• Provide final review summary

Summary

This PR promotes github.com/evstack/ev-node from v1.0.0-rc.5 to the stable v1.0.0 release across all sub-modules in the monorepo, along with several transitive dependency bumps in tools/da-debug.

Observations

Positive:

• The ev-node stable version bump is applied consistently across all 9 modules that reference it (apps/evm, apps/grpc, apps/testapp, execution/evm, execution/evm/test, execution/grpc, test/docker-e2e, test/e2e, tools/da-debug).
• golang.org/x/crypto v0.48.0 update is a good security hygiene bump.
• github.com/libp2p/go-libp2p v0.47.0 minor version bump is included transitively.

Notable — Commented-out replace directive (apps/testapp/go.mod:5):

//replace github.com/evstack/ev-node => ../../.

The replace directive was commented out rather than removed. While this was likely done in the previous PR (#3158 "remove replaces from ev-node"), it's worth confirming whether this is intentional for developer convenience (easy toggle for local dev) or should be cleaned up. A commented-out directive in a committed go.mod can cause confusion — contributors may not realize they need to uncomment it for local development, or may accidentally uncomment it and push.

RC versions still present in some modules:

• apps/evm/go.mod: github.com/evstack/ev-node/execution/evm v1.0.0-rc.4
• apps/grpc/go.mod: github.com/evstack/ev-node/execution/grpc v1.0.0-rc.1

These sub-modules (execution/evm and execution/grpc) are still on RC versions. This appears intentional since those modules haven't cut a stable release yet, but worth tracking.

go.sum completeness: The apps/testapp/go.sum has 2 new lines but no removals, while other modules properly show both additions and replacements. This looks correct as testapp is pulling in new hashes for the v1.0.0 release.

Verdict

This is a clean, mechanical dependency promotion PR. No functional code changes, and the version updates look correct. The commented replace directive is the only minor nit worth addressing if this is not an intentional convention.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.22%. Comparing base (2c9ca1b) to head (9d2f431).
⚠️ Report is 2 commits behind head on main.

@@            Coverage Diff             @@
##             main    #3159      +/-   ##
==========================================
+ Coverage   60.20%   60.22%   +0.01%     
==========================================
  Files         115      115              
  Lines       11896    11896              
==========================================
+ Hits         7162     7164       +2     
+ Misses       3923     3922       -1     
+ Partials      811      810       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.