From 25c04a915cb0b1f2ef51d19c066e6a4cb4ab0d23 Mon Sep 17 00:00:00 2001
From: mayor <steve.yegge@gmail.com>
Date: Fri, 2 Jan 2026 12:40:38 -0800
Subject: [PATCH] fix: Only count registered rigs in status line
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The status line was counting rigs based on tmux session names without
validating they were actually registered in mayor/rigs.json. This could
cause incorrect rig counts when session names matched the pattern but
were not real rigs.

Now loads registered rigs from rigs.json and only counts:
- Rigs that are actually registered
- Polecats belonging to registered rigs

Fixes #17

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 internal/cmd/statusline.go | 38 +++++++++++++++++++++++++++++++-------
 1 file changed, 31 insertions(+), 7 deletions(-)

diff --git a/internal/cmd/statusline.go b/internal/cmd/statusline.go
index c1d90b3f..a441f2b3 100644
--- a/internal/cmd/statusline.go
+++ b/internal/cmd/statusline.go
@@ -8,6 +8,7 @@ import (
 
 	"github.com/spf13/cobra"
 	"github.com/steveyegge/gastown/internal/beads"
+	"github.com/steveyegge/gastown/internal/config"
 	"github.com/steveyegge/gastown/internal/mail"
 	"github.com/steveyegge/gastown/internal/tmux"
 	"github.com/steveyegge/gastown/internal/workspace"
@@ -164,9 +165,20 @@ func runMayorStatusLine(t *tmux.Tmux) error {
 		townRoot, _ = workspace.Find(paneDir)
 	}
 
+	// Load registered rigs to validate against
+	registeredRigs := make(map[string]bool)
+	if townRoot != "" {
+		rigsConfigPath := filepath.Join(townRoot, "mayor", "rigs.json")
+		if rigsConfig, err := config.LoadRigsConfig(rigsConfigPath); err == nil {
+			for rigName := range rigsConfig.Rigs {
+				registeredRigs[rigName] = true
+			}
+		}
+	}
+
 	// Count polecats and rigs
 	// Polecats: only actual polecats (not witnesses, refineries, deacon, crew)
-	// Rigs: any rig with active sessions (witness, refinery, crew, or polecat)
+	// Rigs: only registered rigs with active sessions
 	polecatCount := 0
 	rigs := make(map[string]bool)
 	for _, s := range sessions {
@@ -174,12 +186,12 @@ func runMayorStatusLine(t *tmux.Tmux) error {
 		if agent == nil {
 			continue
 		}
-		// Count rigs from any rig-level agent (has non-empty Rig field)
-		if agent.Rig != "" {
+		// Count rigs from any rig-level agent, but only if registered
+		if agent.Rig != "" && registeredRigs[agent.Rig] {
 			rigs[agent.Rig] = true
 		}
-		// Count only polecats for polecat count
-		if agent.Type == AgentPolecat {
+		// Count only polecats for polecat count (in registered rigs)
+		if agent.Type == AgentPolecat && registeredRigs[agent.Rig] {
 			polecatCount++
 		}
 	}
@@ -229,6 +241,17 @@ func runDeaconStatusLine(t *tmux.Tmux) error {
 		townRoot, _ = workspace.Find(paneDir)
 	}
 
+	// Load registered rigs to validate against
+	registeredRigs := make(map[string]bool)
+	if townRoot != "" {
+		rigsConfigPath := filepath.Join(townRoot, "mayor", "rigs.json")
+		if rigsConfig, err := config.LoadRigsConfig(rigsConfigPath); err == nil {
+			for rigName := range rigsConfig.Rigs {
+				registeredRigs[rigName] = true
+			}
+		}
+	}
+
 	rigs := make(map[string]bool)
 	polecatCount := 0
 	for _, s := range sessions {
@@ -236,10 +259,11 @@ func runDeaconStatusLine(t *tmux.Tmux) error {
 		if agent == nil {
 			continue
 		}
-		if agent.Rig != "" {
+		// Only count registered rigs
+		if agent.Rig != "" && registeredRigs[agent.Rig] {
 			rigs[agent.Rig] = true
 		}
-		if agent.Type == AgentPolecat {
+		if agent.Type == AgentPolecat && registeredRigs[agent.Rig] {
 			polecatCount++
 		}
 	}