🔧 Update CI workflow to use 'go build .' and fix errcheck linting issues by properly handling ignored error returns

Author: eliezedeck

.github/workflows/ci.yml

@@ -64,7 +64,7 @@ jobs:
 
     - name: Build binary
       working-directory: ./sidekick
-      run: go build -v -o sidekick main.go processes.go notifications.go
+      run: go build -v -o sidekick .
 
     - name: Upload coverage to Codecov
       if: matrix.os == 'ubuntu-latest'

sidekick/main.go

@@ -198,7 +198,7 @@ func handleGracefulShutdown() {
 			err := terminateProcessGroup(tracker.Process.Process.Pid)
 			if err != nil {
 				// If platform-specific termination fails, use standard process.Kill()
-				tracker.Process.Process.Kill()
+				_ = tracker.Process.Process.Kill()
 			}
 		}
 		tracker.Mutex.RUnlock()
@@ -239,7 +239,7 @@ func handleGracefulShutdown() {
 			err := forceKillProcessGroup(tracker.Process.Process.Pid)
 			if err != nil {
 				// If platform-specific force kill fails, use standard process.Kill()
-				tracker.Process.Process.Kill()
+				_ = tracker.Process.Process.Kill()
 			}
 		}
 		tracker.Mutex.RUnlock()

sidekick/notifications.go

@@ -28,13 +28,13 @@ func handleSpeak(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToo
 
 	// 🔊 Play system sound asynchronously
 	go func() {
-		exec.Command("afplay", "/System/Library/Sounds/Glass.aiff", "-v", "5").Run()
+		_ = exec.Command("afplay", "/System/Library/Sounds/Glass.aiff", "-v", "5").Run()
 	}()
 
 	// 🗣️ Speak the text after a short delay
 	go func() {
 		time.Sleep(500 * time.Millisecond)
-		exec.Command("say", "-v", "Zoe (Premium)", text).Run()
+		_ = exec.Command("say", "-v", "Zoe (Premium)", text).Run()
 	}()
 
 	return mcp.NewToolResultText("Notification spoken!"), nil

sidekick/processes.go

@@ -576,7 +576,7 @@ func handleSpawnProcess(ctx context.Context, request mcp.CallToolRequest) (*mcp.
 			// Start background goroutine to wait and then execute
 			go func() {
 				time.Sleep(delay)
-				executeDelayedProcess(context.Background(), tracker, envVars)
+				_ = executeDelayedProcess(context.Background(), tracker, envVars)
 			}()
 
 			result = map[string]any{