Fix TcpAcceptor not setting connection state after accept

Both sync and async accept methods were accepting directly into
conn->socket() without calling conn->accept(), leaving the
connection state as Disconnected. This caused read() to immediately
return NotConnected for every accepted connection.

Fix: accept into a separate socket, then call conn->accept(socket)
which properly sets state to Connected.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Dendi Suhubdy

src/net/acceptor.cpp

@@ -95,12 +95,14 @@ std::expected<std::shared_ptr<TcpConnection>, AcceptorError> TcpAcceptor::accept
     }
 
     try {
-        auto conn = std::make_shared<TcpConnection>(io_context_);
         boost::system::error_code ec;
-        acceptor_.accept(conn->socket(), ec);
+        tcp::socket socket(io_context_);
+        acceptor_.accept(socket, ec);
         if (ec) {
             return std::unexpected(AcceptorError::AcceptFailed);
         }
+        auto conn = std::make_shared<TcpConnection>(io_context_);
+        conn->accept(std::move(socket));
         return conn;
     } catch (...) {
         return std::unexpected(AcceptorError::AcceptFailed);
@@ -113,11 +115,13 @@ void TcpAcceptor::async_accept(AcceptHandler handler) {
         return;
     }
 
-    auto conn = std::make_shared<TcpConnection>(io_context_);
-    acceptor_.async_accept(conn->socket(), [conn, handler](boost::system::error_code ec) {
+    auto socket = std::make_shared<tcp::socket>(io_context_);
+    acceptor_.async_accept(*socket, [this, socket, handler](boost::system::error_code ec) {
         if (ec) {
             handler(std::unexpected(AcceptorError::AcceptFailed));
         } else {
+            auto conn = std::make_shared<TcpConnection>(io_context_);
+            conn->accept(std::move(*socket));
             handler(conn);
         }
     });