Ensure LSP server exits on stdin close

[jakebailey]

Fixes #935

We want the server to exit when stdin closes, but the current code was returning nil, leaving the errgroup running, leaving the process behind.

[Copilot]

Pull Request Overview

This PR fixes an issue where the LSP server would continue running in the background when stdin was closed by ensuring the server exits properly.

• Updated the Run method in server.go to conditionally ignore io.EOF errors from the errgroup.
• Simplified error handling in lsp.go by removing redundant io.EOF checks and cleaning up unused imports.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
internal/lsp/server.go	Modified Run to check for io.EOF and return nil for this error, ensuring proper shutdown.
cmd/tsgo/lsp.go	Removed redundant error checks and imports, relying on the logic defined in Run.

Comments suppressed due to low confidence (1)

cmd/tsgo/lsp.go:50

• [nitpick] The removal of the explicit io.EOF check and unused imports simplifies error handling; ensure that this change remains consistent with the intended shutdown behavior documented in the Run method.

if err := s.Run(); err != nil {