fix: resolve all lint errors from golangci-lint v2

- errcheck: explicitly ignore Close() errors with _ = c.Close()
- revive: add doc comments on all exported types and functions
- staticcheck: remove trailing \n from error format strings

Author: DeliciousBuding

cmd/bridge/main.go

@@ -32,7 +32,7 @@ func main() {
 	if debugPath := os.Getenv("BRIDGE_DEBUG_LOG"); debugPath != "" {
 		if f, err := os.OpenFile(debugPath, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0644); err == nil {
 			logger = log.New(io.MultiWriter(os.Stderr, f), "[codex-bridge] ", log.LstdFlags)
-			defer f.Close()
+			defer func() { _ = f.Close() }()
 		}
 	}
 
@@ -71,7 +71,7 @@ func runMCP(pipeName string, logger *log.Logger) {
 		fmt.Fprintf(os.Stderr, "Failed to connect: %v\n", err)
 		os.Exit(1)
 	}
-	defer c.Close()
+	defer func() { _ = c.Close() }()
 
 	logger.Println("Connected to Codex browser pipe, starting MCP server...")
 
@@ -91,7 +91,7 @@ func runCLI(pipeName string, logger *log.Logger) {
 		fmt.Fprintf(os.Stderr, "Failed to connect: %v\n", err)
 		os.Exit(1)
 	}
-	defer c.Close()
+	defer func() { _ = c.Close() }()
 
 	fmt.Println("Connected to Codex browser pipe")
 	fmt.Println("Commands: tabs, create, close <id>, user-tabs, claim <id>, nav <id> <url>,")

internal/client/client.go

@@ -66,7 +66,7 @@ func Connect(pipeName string, logger *log.Logger) (*Client, error) {
 			// Health check: send a quick getInfo to verify the pipe is usable
 			result, err := c.SendRequest("getInfo", nil)
 			if err != nil {
-				c.Close()
+				_ = c.Close()
 				lastErr = err
 				if logger != nil {
 					logger.Printf("pipe %s health check failed: %v, trying next...", p.UUID, err)
@@ -78,15 +78,15 @@ func Connect(pipeName string, logger *log.Logger) (*Client, error) {
 			}
 			return c, nil
 		}
-		return nil, fmt.Errorf("all %d pipes failed. Last error: %w\n"+
+		return nil, fmt.Errorf("all %d pipes failed. Last error: %w"+
 			"Try: restart Codex Desktop, then re-open the Codex Chrome Extension.",
 			len(pipes), lastErr)
 	}
 
 	path := discovery.PipePath(pipeName)
 	conn, err := dialNamedPipe(path)
 	if err != nil {
-		return nil, fmt.Errorf("dial pipe %s: %w\n"+
+		return nil, fmt.Errorf("dial pipe %s: %w"+
 			"This usually means the pipe is stale (Codex Desktop restarted) or the extension lost its host.\n"+
 			"Try: restart Codex Desktop, then re-open the Codex Chrome Extension.", path, err)
 	}

internal/discovery/discovery.go

@@ -8,6 +8,7 @@ import (
 
 const codexPipePrefix = "codex-browser-use"
 
+// PipeInfo holds the name and UUID of a discovered Codex named pipe.
 type PipeInfo struct {
 	Name string
 	UUID string
@@ -44,6 +45,7 @@ func extractUUID(name string) string {
 	return rest
 }
 
+// PipePath returns the full \\.\pipe\ path for a named pipe.
 func PipePath(name string) string {
 	return `\\.\pipe\` + name
 }

internal/mcp/server.go

@@ -11,6 +11,7 @@ import (
 	"github.com/DeliciousBuding/codex-browser-bridge/internal/client"
 )
 
+// MCPServer is an MCP stdio server that exposes browser automation tools.
 type MCPServer struct {
 	client  *client.Client
 	tools   []Tool
@@ -20,6 +21,7 @@ type MCPServer struct {
 	version string
 }
 
+// Tool defines an MCP tool exposed by the server.
 type Tool struct {
 	Name        string                                        `json:"name"`
 	Description string                                        `json:"description"`
@@ -43,10 +45,12 @@ func imageContent(b64, mime string) Content {
 	return Content{Type: "image", Data: b64, MimeType: mime}
 }
 
+// NewMCPServer creates an MCP server using os.Stdin and os.Stdout for transport.
 func NewMCPServer(c *client.Client) *MCPServer {
 	return NewMCPServerWithIO(c, os.Stdin, os.Stdout)
 }
 
+// NewMCPServerWithIO creates an MCP server with custom I/O streams.
 func NewMCPServerWithIO(c *client.Client, in io.Reader, out io.Writer) *MCPServer {
 	s := &MCPServer{
 		client:  c,

internal/protocol/protocol.go

@@ -9,19 +9,22 @@ import (
 
 // JSON-RPC 2.0 message types (Codex uses JSON-RPC without the "jsonrpc":"2.0" field on responses)
 
+// Request is a JSON-RPC 2.0 request message.
 type Request struct {
 	JSONRPC string      `json:"jsonrpc"`
 	ID      int         `json:"id"`
 	Method  string      `json:"method"`
 	Params  interface{} `json:"params,omitempty"`
 }
 
+// Response is a JSON-RPC 2.0 response message.
 type Response struct {
 	ID     int             `json:"id"`
 	Result json.RawMessage `json:"result,omitempty"`
 	Error  *ErrorObject    `json:"error,omitempty"`
 }
 
+// ErrorObject carries a JSON-RPC error.
 type ErrorObject struct {
 	Code    int         `json:"code"`
 	Message string      `json:"message"`