chore(transport): retire the legacy TCP package

[yordis]

• Prevents the retired transport from remaining buildable or distributable by accident.

[cursor]

PR Summary

Medium Risk
Removes networking/SSL transport code that could still be referenced externally; in-repo references appear gone, but downstream consumers of the TCP library would break.

Overview
Retires the legacy EventStore.Transport.Tcp project so it can no longer be built or shipped from this repo. The change deletes the full TCP stack (plain and SSL connections, listeners, length-prefix framing, formatters, monitoring) and drops the project from EventStore.sln.

Central package management no longer pins EventStore.Client (v21.2.0), aligning dependencies with the remaining gRPC client packages already listed in Directory.Packages.props.

^{Reviewed by Cursor Bugbot for commit 8d3a13b. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 7e10efa5-c667-4e67-aeb0-95a4c84c17a8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 61a5de91-1e9a-4618-b463-3d562fa0cf4d

📥 Commits

Reviewing files that changed from the base of the PR and between ab718f7 and 2b3ea9b.

📒 Files selected for processing (23)

• src/Directory.Packages.props
• src/EventStore.Transport.Tcp/EventStore.Transport.Tcp.csproj
• src/EventStore.Transport.Tcp/Formatting/FormatterBase.cs
• src/EventStore.Transport.Tcp/Formatting/IMessageFormatter.cs
• src/EventStore.Transport.Tcp/Formatting/RawMessageFormatter.cs
• src/EventStore.Transport.Tcp/Framing/IMessageFramer.cs
• src/EventStore.Transport.Tcp/Framing/LengthPrefixMessageFramer.cs
• src/EventStore.Transport.Tcp/Framing/LengthPrefixMessageFramerWithBufferPool.cs
• src/EventStore.Transport.Tcp/Framing/PackageFramingException.cs
• src/EventStore.Transport.Tcp/Helper.cs
• src/EventStore.Transport.Tcp/IMonitoredTcpConnection.cs
• src/EventStore.Transport.Tcp/ITcpConnection.cs
• src/EventStore.Transport.Tcp/SocketArgsPool.cs
• src/EventStore.Transport.Tcp/TcpClientConnector.cs
• src/EventStore.Transport.Tcp/TcpConfiguration.cs
• src/EventStore.Transport.Tcp/TcpConnection.cs
• src/EventStore.Transport.Tcp/TcpConnectionBase.cs
• src/EventStore.Transport.Tcp/TcpConnectionMonitor.cs
• src/EventStore.Transport.Tcp/TcpConnectionSsl.cs
• src/EventStore.Transport.Tcp/TcpServerListener.cs
• src/EventStore.Transport.Tcp/TcpStats.cs
• src/EventStore.Transport.Tcp/TcpTypedConnection.cs
• src/EventStore.sln

💤 Files with no reviewable changes (23)

• src/EventStore.Transport.Tcp/Formatting/RawMessageFormatter.cs
• src/EventStore.Transport.Tcp/Framing/LengthPrefixMessageFramerWithBufferPool.cs
• src/EventStore.Transport.Tcp/EventStore.Transport.Tcp.csproj
• src/EventStore.Transport.Tcp/TcpTypedConnection.cs
• src/EventStore.Transport.Tcp/TcpConnection.cs
• src/EventStore.Transport.Tcp/TcpConnectionSsl.cs
• src/EventStore.Transport.Tcp/TcpServerListener.cs
• src/EventStore.Transport.Tcp/TcpConfiguration.cs
• src/EventStore.sln
• src/EventStore.Transport.Tcp/IMonitoredTcpConnection.cs
• src/EventStore.Transport.Tcp/Framing/IMessageFramer.cs
• src/EventStore.Transport.Tcp/Framing/PackageFramingException.cs
• src/EventStore.Transport.Tcp/TcpConnectionBase.cs
• src/EventStore.Transport.Tcp/ITcpConnection.cs
• src/EventStore.Transport.Tcp/TcpClientConnector.cs
• src/EventStore.Transport.Tcp/SocketArgsPool.cs
• src/EventStore.Transport.Tcp/TcpStats.cs
• src/EventStore.Transport.Tcp/Formatting/FormatterBase.cs
• src/EventStore.Transport.Tcp/Framing/LengthPrefixMessageFramer.cs
• src/Directory.Packages.props
• src/EventStore.Transport.Tcp/Formatting/IMessageFormatter.cs
• src/EventStore.Transport.Tcp/TcpConnectionMonitor.cs
• src/EventStore.Transport.Tcp/Helper.cs

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

The pull request removes the EventStore.Transport.Tcp project and all its formatting, framing, connection, listener, SSL, pooling, configuration, monitoring, and statistics code. It also removes the project from the solution and removes one central package version entry.

Changes

TCP transport removal

Layer / File(s)	Summary
Project and solution removal src/Directory.Packages.props, src/EventStore.Transport.Tcp/EventStore.Transport.Tcp.csproj, src/EventStore.sln	The EventStore.Transport.Tcp project and its solution configuration entries are removed. The central EventStore.Client version entry is also removed.
Formatting and framing removal src/EventStore.Transport.Tcp/Formatting/*, src/EventStore.Transport.Tcp/Framing/*	Formatter interfaces and implementations, length-prefix framers, and PackageFramingException are deleted.
Connection and monitoring removal src/EventStore.Transport.Tcp/ITcpConnection.cs, src/EventStore.Transport.Tcp/IMonitoredTcpConnection.cs, src/EventStore.Transport.Tcp/Tcp*.cs, src/EventStore.Transport.Tcp/SocketArgsPool.cs, src/EventStore.Transport.Tcp/Helper.cs	TCP and SSL connections, client and server socket handling, pooling, configuration, monitoring, statistics, helper methods, and typed connection wrapping are deleted.

Priority: ⬇️ Low

Estimated code review effort: 4 (Complex) | ~45 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 2b3ea

The removed transport project has no remaining tracked project, restore, packaging, or compilation references. No confirmed merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Title check	✅ Passed	The title clearly summarizes the main change: retirement of the legacy TCP transport package.
Description check	✅ Passed	The description accurately states that the changes prevent the retired transport from remaining buildable or distributable.

✨ 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 yordis/chore-remove-legacy-tcp-package

---

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

_{A rabbit reads each line,
The patch grows clear beneath the moon,
Small changes hop in place,
Tests guard the garden path,
Reviews bloom before the dawn.}

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