fix(test): mutex-protect request recorder in outgoing impact fake

The test fake daemon recorded incoming requests via a goroutine
appending to a shared []map[string]any while the test main
goroutine read it through the snap() closure. Race detector caught
this on CI (TestAnalyzeOutgoingImpact_HappyPath). Wrap append and
read with sync.Mutex; snap() takes the lock around its copy.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: SimplyLiz

internal/query/impact_outgoing_test.go

@@ -9,6 +9,7 @@ import (
 	"os"
 	"path/filepath"
 	"strings"
+	"sync"
 	"testing"
 	"time"
 
@@ -35,13 +36,18 @@ func startOutgoingImpactDaemon(t *testing.T, payload map[string]any) func() []ma
 	prev := os.Getenv("LIP_SOCKET")
 	os.Setenv("LIP_SOCKET", sockPath)
 
-	var reqs []map[string]any
+	var (
+		reqsMu sync.Mutex
+		reqs   []map[string]any
+	)
 	reqC := make(chan map[string]any, 16)
 	done := make(chan struct{})
 	go func() {
 		defer close(done)
 		for r := range reqC {
+			reqsMu.Lock()
 			reqs = append(reqs, r)
+			reqsMu.Unlock()
 		}
 	}()
 
@@ -88,6 +94,8 @@ func startOutgoingImpactDaemon(t *testing.T, payload map[string]any) func() []ma
 		os.Setenv("LIP_SOCKET", prev)
 	})
 	return func() []map[string]any {
+		reqsMu.Lock()
+		defer reqsMu.Unlock()
 		out := make([]map[string]any, len(reqs))
 		copy(out, reqs)
 		return out