chore(transport): retire the legacy TCP package

[yordis]

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

[cursor]

PR Summary

Medium Risk
Large deletion of networking transport code; risk is mainly if any external or undocumented consumers still referenced the project—in-repo references appear gone.

Overview
Removes the legacy EventStore.Transport.Tcp project from the solution and deletes its implementation (TCP/SSL connections, length-prefix framing, formatters, listeners, and monitoring).

Build hygiene: Drops the project from EventStore.sln (including all platform build mappings) and removes the central EventStore.Client package version from Directory.Packages.props, so the retired transport cannot be built or pulled in accidentally.

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

[coderabbitai]

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.}