Fix lint issues in spotlight mode

- Remove ineffectual assignment in test (use _ = output)
- Remove unnecessary nil check before len() in status function

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

Author: bborn

internal/mcp/server.go

@@ -1014,7 +1014,7 @@ func (s *Server) spotlightStatus(worktreePath, mainRepoDir string) (string, erro
 	msg := "🔦 Spotlight mode: ACTIVE\n\n"
 	msg += fmt.Sprintf("Worktree: %s\n", worktreePath)
 	msg += fmt.Sprintf("Main repo: %s\n", mainRepoDir)
-	if stateData != nil && len(stateData) > 0 {
+	if len(stateData) > 0 {
 		for _, line := range strings.Split(string(stateData), "\n") {
 			if strings.HasPrefix(line, "started=") {
 				msg += fmt.Sprintf("Started: %s\n", strings.TrimPrefix(line, "started="))

internal/mcp/server_test.go

@@ -1220,6 +1220,7 @@ func TestSpotlightSync(t *testing.T) {
 
 	server, output := testServer(database, task.ID, string(reqBytes))
 	server.Run()
+	_ = output // first response not needed
 
 	// Now modify the worktree file
 	os.WriteFile(filepath.Join(worktreeDir, "file.txt"), []byte("modified again"), 0644)