fix(server): resolve stdio server context cancellation bug (#331)

* fix(server): resolve stdio server context cancellation bug

Fix a critical bug in the stdio server's readNextLine function that was
causing "Error reading input" and "context cancelled" errors when Claude
Desktop attempted to connect to MCP servers.

The issue was in the select-default pattern that would immediately bypass
context cancellation checks and proceed with blocking ReadString operations
even when the context was cancelled. This prevented proper cleanup and
caused connection failures.

Changes:
- Remove problematic select-default pattern in readNextLine goroutine
- Simplify the read logic to directly call reader.ReadString('\n')
- Maintain proper done channel handling for result communication
- Ensure context cancellation is respected throughout the read operation

This resolves connection issues introduced in version 0.30.0 where the
stdio transport became unreliable for client connections.

Fixes: stdio server connection failures with Claude Desktop

* refactor: remove redudant signaling channels

---------

Co-authored-by: Marcos <[email protected]>

Author: sonirico

server/stdio.go

@@ -189,39 +189,23 @@ func (s *StdioServer) processInputStream(ctx context.Context, reader *bufio.Read
 // returns an empty string and the context's error. EOF is returned when the input
 // stream is closed.
 func (s *StdioServer) readNextLine(ctx context.Context, reader *bufio.Reader) (string, error) {
-	readChan := make(chan string, 1)
-	errChan := make(chan error, 1)
-	done := make(chan struct{})
-	defer close(done)
+	type result struct {
+		line string
+		err  error
+	}
+
+	resultCh := make(chan result, 1)
 
 	go func() {
-		select {
-		case <-done:
-			return
-		default:
-			line, err := reader.ReadString('\n')
-			if err != nil {
-				select {
-				case errChan <- err:
-				case <-done:
-				}
-				return
-			}
-			select {
-			case readChan <- line:
-			case <-done:
-			}
-			return
-		}
+		line, err := reader.ReadString('\n')
+		resultCh <- result{line: line, err: err}
 	}()
 
 	select {
 	case <-ctx.Done():
-		return "", ctx.Err()
-	case err := <-errChan:
-		return "", err
-	case line := <-readChan:
-		return line, nil
+		return "", nil
+	case res := <-resultCh:
+		return res.line, res.err
 	}
 }