refactor(execution/grpc): move execution service where it belongs

[julienrbrt]

Follow-up of #3297 implements #3297 (review)

[coderabbitai]

📝 Walkthrough

Walkthrough

Relocates protobuf type generation and imports for gRPC execution services from top-level types/pb/ to execution/grpc/types/pb/, including new Buf generation configuration, updated proto package options, and revised imports across execution gRPC module files.

Changes

Cohort / File(s)	Summary
Build & Generation Configuration .just/proto.just, buf.gen.grpc.yaml, buf.yaml, CHANGELOG.md	Adds new Buf generation template for gRPC with ConnectRPC support; removes FIELD_NO_DELETE breaking-rule exception for execution.proto; documents breaking changes related to execution service relocation and tx_batch payload changes.
Proto Definitions execution/grpc/proto/evnode/v1/execution.proto, execution/grpc/types/pb/evnode/v1/v1connect/execution.connect.go	Updates go_package option to point generated code to execution/grpc/types/pb/evnode/v1; updates scope annotation in ExecutorService comment.
gRPC Client & Server execution/grpc/client.go, execution/grpc/server.go, execution/grpc/handler.go	Switches protobuf type imports from types/pb/... to execution/grpc/types/pb/... for request/response handling and service registration.
Support Code & Dependencies execution/grpc/tx_batch.go, execution/grpc/go.mod	Updates pb import path for TxBatch type; removes local replace directive and explicit ev-node requirement from module file.
Tests execution/grpc/server_test.go, execution/grpc/tx_batch_test.go	Redirects test imports to use relocated protobuf types under execution/grpc/types/pb/evnode/v1.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• chore: prep rc.2 #3231 — Modifies module resolution by removing local go.mod replace directives, similar pattern to this PR's go.mod changes.
• feat: add grpc socket and flattn tx batches to allow for lower allocations #3297 — Directly supports gRPC proto/TxBatch and gRPC client/server refactoring that aligns with this PR's relocation changes.

Suggested reviewers

• tac0turtle

Poem

🐰 Hopping through modules, paths rearranged,
From types/pb to execution/grpc, freshly changed,
Protobuf types tucked where they belong,
Buf generation sings a harmonious song! 🎵

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description is minimal and only references a follow-up PR and prior review comment without providing substantive details about the change, its motivation, or impact.	Add an overview explaining what the refactor accomplishes, why the reorganization is beneficial, and which components are affected (e.g., protobuf generation config, gRPC service location).

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: moving the execution service to its proper location under execution/grpc, which aligns with the file reorganization evident throughout the changeset.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch julien/execution-grpc

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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

[claude]

Claude Code is working…

I'll analyze this and get back to you.

View job run

[github-actions]

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

Build	Format	Lint	Breaking	Updated (UTC)
✅ passed	⏩ skipped	✅ passed	❌ failed (11)	Apr 29, 2026, 1:43 PM

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.55%. Comparing base (4063698) to head (9a35a4d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3302      +/-   ##
==========================================
- Coverage   62.72%   60.55%   -2.17%     
==========================================
  Files         124      126       +2     
  Lines       13248    13721     +473     
==========================================
- Hits         8310     8309       -1     
- Misses       4034     4508     +474     
  Partials      904      904              

Flag	Coverage Δ
combined	60.55% <ø> (-2.17%)	⬇️

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

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[julienrbrt]

will fix the linting issue before merging.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CHANGELOG.md`:
- Around line 16-18: The changelog contains an empty markdown link token "[]()"
in the list item "Move execution service where it belongs in execution/grpc.
[]()"; remove the empty link or replace it with the correct PR reference (e.g.,
add the PR number and URL) so the line reads without a broken link; update the
same list item in CHANGELOG.md by editing the text "Move execution service where
it belongs in execution/grpc. []()" to either "Move execution service where it
belongs in execution/grpc." or "Move execution service where it belongs in
execution/grpc. [`#PR_NUMBER`](PR_URL)".

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7ef2bafb-da54-40bc-aaaf-56f49e01f144

📥 Commits

Reviewing files that changed from the base of the PR and between 4063698 and 9a35a4d.

⛔ Files ignored due to path filters (2)

• execution/grpc/go.sum is excluded by !**/*.sum
• execution/grpc/types/pb/evnode/v1/execution.pb.go is excluded by !**/*.pb.go

📒 Files selected for processing (13)

• .just/proto.just
• CHANGELOG.md
• buf.gen.grpc.yaml
• buf.yaml
• execution/grpc/client.go
• execution/grpc/go.mod
• execution/grpc/handler.go
• execution/grpc/proto/evnode/v1/execution.proto
• execution/grpc/server.go
• execution/grpc/server_test.go
• execution/grpc/tx_batch.go
• execution/grpc/tx_batch_test.go
• execution/grpc/types/pb/evnode/v1/v1connect/execution.connect.go

💤 Files with no reviewable changes (1)

• execution/grpc/go.mod

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Fix the empty markdown link.

[]() on Line 17 will trip markdownlint and renders as a broken changelog entry. Replace it with the actual PR reference or remove the link syntax.

Suggested fix

-  - Move execution service where it belongs in execution/grpc. []()
+  - Move execution service where it belongs in execution/grpc. [`#3302`](https://github.com/evstack/ev-node/pull/3302)

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	- **BREAKING:** (execution/grpc)
	- Move execution service where it belongs in execution/grpc. []()
	- Replace legacy gRPC execution `txs` payload fields with `tx_batch` so clients and servers use contiguous transaction buffers [#3297](https://github.com/evstack/ev-node/pull/3297)
	- **BREAKING:** (execution/grpc)
	- Move execution service where it belongs in execution/grpc. [`#3302`](https://github.com/evstack/ev-node/pull/3302)
	- Replace legacy gRPC execution `txs` payload fields with `tx_batch` so clients and servers use contiguous transaction buffers [`#3297`](https://github.com/evstack/ev-node/pull/3297)

🧰 Tools

🪛 markdownlint-cli2 (0.22.1)

[warning] 17-17: No empty links

(MD042, no-empty-links)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@CHANGELOG.md` around lines 16 - 18, The changelog contains an empty markdown
link token "[]()" in the list item "Move execution service where it belongs in
execution/grpc. []()"; remove the empty link or replace it with the correct PR
reference (e.g., add the PR number and URL) so the line reads without a broken
link; update the same list item in CHANGELOG.md by editing the text "Move
execution service where it belongs in execution/grpc. []()" to either "Move
execution service where it belongs in execution/grpc." or "Move execution
service where it belongs in execution/grpc. [`#PR_NUMBER`](PR_URL)".