feat: competitive parity with Copilot CLI and Claude Code

.github/workflows/ci.yml

@@ -0,0 +1,80 @@
+name: CI
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+permissions:
+  contents: read
+  packages: read
+
+env:
+  GOPRIVATE: github.com/GoCodeAlone/*
+  GONOSUMCHECK: github.com/GoCodeAlone/*
+
+jobs:
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.26"
+          cache: true
+      - name: Configure Git for private modules
+        run: git config --global url."https://${{ secrets.GITHUB_TOKEN }}@github.com/".insteadOf "https://github.com/"
+      - name: Build
+        run: go build ./...
+
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    needs: build
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.26"
+          cache: true
+      - name: Configure Git for private modules
+        run: git config --global url."https://${{ secrets.GITHUB_TOKEN }}@github.com/".insteadOf "https://github.com/"
+      - name: Test with race detector
+        run: go test -race -coverprofile=coverage.out -covermode=atomic ./...
+      - name: Upload coverage
+        uses: codecov/codecov-action@v4
+        with:
+          files: coverage.out
+          fail_ci_if_error: false
+
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.26"
+          cache: true
+      - name: Configure Git for private modules
+        run: git config --global url."https://${{ secrets.GITHUB_TOKEN }}@github.com/".insteadOf "https://github.com/"
+      - name: Run golangci-lint
+        uses: golangci/golangci-lint-action@v8
+        with:
+          version: v2.11.3
+
+  vet:
+    name: Vet
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.26"
+          cache: true
+      - name: Configure Git for private modules
+        run: git config --global url."https://${{ secrets.GITHUB_TOKEN }}@github.com/".insteadOf "https://github.com/"
+      - name: Go vet
+        run: go vet ./...

.golangci.yml

@@ -0,0 +1,37 @@
+version: "2"
+
+linters:
+  default: standard
+  settings:
+    errcheck:
+      # Exclude common patterns where ignoring errors is intentional:
+      # deferred Close/Rollback calls, goroutine server Serve calls, and
+      # fire-and-forget writes in HTTP handlers or gRPC streams.
+      exclude-functions:
+        - (io.Closer).Close
+        - (*database/sql.Rows).Close
+        - (*database/sql.Tx).Rollback
+        - (*net.Listener).Close
+        - (*net/http.Server).Close
+        - os.Remove
+        - os.MkdirAll
+        - os.WriteFile
+        - (net.Listener).Close
+        - (*google.golang.org/grpc.Server).Serve
+        - (*google.golang.org/grpc.ClientConn).Close
+        - (net.Conn).Close
+        - (*encoding/json.Encoder).Encode
+        - fmt.Fprintf
+        - fmt.Fprintln
+        - fmt.Fprint
+        - (io.WriteCloser).Close
+
+  exclusions:
+    rules:
+      # Relax errcheck in test files — Close/Cleanup patterns in tests are fine.
+      - linters: [errcheck]
+        path: _test\.go
+      # defer c.Close() / defer f.Close() / defer db.Close() are idiomatic Go;
+      # the error from a deferred Close is not actionable.
+      - linters: [errcheck]
+        text: "Error return value of .*(Close|Rollback|Send|KillSession|KillAgent). is not checked"

cmd/ratchet/cmd_agent.go

@@ -19,7 +19,7 @@ func handleAgent(args []string) {
 		fmt.Fprintf(os.Stderr, "error: %v\n", err)
 		os.Exit(1)
 	}
-	defer c.Close()
+	defer func() { _ = c.Close() }()
 
 	switch args[0] {
 	case "list":

cmd/ratchet/cmd_chat.go

@@ -34,7 +34,7 @@ func handleOneShot(prompt string) {
 		fmt.Fprintf(os.Stderr, "error: %v\n", err)
 		os.Exit(1)
 	}
-	defer c.Close()
+	defer func() { _ = c.Close() }()
 
 	wd, _ := os.Getwd()
 	session, err := c.CreateSession(ctx, &pb.CreateSessionReq{

cmd/ratchet/cmd_provider.go

@@ -20,7 +20,7 @@ func handleProvider(args []string) {
 		fmt.Fprintf(os.Stderr, "error: %v\n", err)
 		os.Exit(1)
 	}
-	defer c.Close()
+	defer func() { _ = c.Close() }()
 
 	switch args[0] {
 	case "add":