fix: resolve 14 review issues across the service control plane

Makefile

@@ -120,10 +120,10 @@ test-desktop: $(NODE_MODULES) ## Run the desktop unit tests
 
 # services/tests builds the component binaries it drives into a temporary
 # directory, so the cross-process suite does not need build-services first.
-test-services: ## Run go test in every services module
+test-services: ## Run go test (with -race) in every services module
 	@for module in $(GO_MODULES); do \
 		printf '\ngo test: %s\n' "$$module"; \
-		(cd "$$module" && go test ./...) || exit 1; \
+		(cd "$$module" && go test -race ./...) || exit 1; \
 	done
 
 build-binaries: $(NODE_MODULES) ## Compile the Go service binaries into desktop/cli-bin

desktop/dead-code-omissions.json

@@ -2,11 +2,10 @@
     "$comment": "Known false positives for the dead-code analyzer. Add entries after reviewing the report.",
     "$reasons": {
         "@tailwindcss/postcss": "Loaded by the PostCSS/Vite config, never imported from source.",
-        "tailwindcss": "Loaded by the PostCSS/Vite config, never imported from source.",
-        "@electron-toolkit/preload": "Imported for the ElectronAPI type by src/preload/index.d.ts. knip.json excludes ambient .d.ts files from analysis, so the only reference is invisible to it — uninstalling breaks the preload type declarations."
+        "tailwindcss": "Loaded by the PostCSS/Vite config, never imported from source."
     },
     "files": [],
     "exports": [],
     "types": [],
-    "dependencies": ["@tailwindcss/postcss", "tailwindcss", "@electron-toolkit/preload"]
+    "dependencies": ["@tailwindcss/postcss", "tailwindcss"]
 }