From 4d24f79492aee461ce0154508e8b57d04da4bc71 Mon Sep 17 00:00:00 2001 From: immortan Date: Sun, 4 Jan 2026 10:37:21 -0800 Subject: [PATCH] refactor: remove unused isFirstRig param from initAgentBeads (gt-fugmy) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The isFirstRig parameter was no longer used - it was being assigned to a blank identifier. Since initAgentBeads is an internal function with only 2 callers in this repo, remove the parameter entirely. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- internal/rig/manager.go | 7 ++----- internal/rig/manager_test.go | 2 +- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/internal/rig/manager.go b/internal/rig/manager.go index 83f30550..9038993e 100644 --- a/internal/rig/manager.go +++ b/internal/rig/manager.go @@ -423,8 +423,7 @@ Use crew for your own workspace. Polecats are for batch work dispatch. // Create rig-level agent beads (witness, refinery) in rig beads. // Town-level agents (mayor, deacon) are created by gt install in town beads. - isFirstRig := len(m.config.Rigs) == 0 // Kept for backward compatibility - if err := m.initAgentBeads(rigPath, opts.Name, opts.BeadsPrefix, isFirstRig); err != nil { + if err := m.initAgentBeads(rigPath, opts.Name, opts.BeadsPrefix); err != nil { // Non-fatal: log warning but continue fmt.Printf(" Warning: Could not create agent beads: %v\n", err) } @@ -536,7 +535,7 @@ func (m *Manager) initBeads(rigPath, prefix string) error { // Format: -- (e.g., gt-gastown-witness) // // Agent beads track lifecycle state for ZFC compliance (gt-h3hak, gt-pinkq). -func (m *Manager) initAgentBeads(_, rigName, _ string, isFirstRig bool) error { // rigPath and prefix unused until Phase 2 +func (m *Manager) initAgentBeads(_, rigName, _ string) error { // rigPath and prefix unused until Phase 2 // TEMPORARY (gt-4r1ph): Currently all agent beads go in town beads. // After Phase 2, only Mayor/Deacon will be here; Witness/Refinery go to rig beads. townBeadsDir := filepath.Join(m.townRoot, ".beads") @@ -568,8 +567,6 @@ func (m *Manager) initAgentBeads(_, rigName, _ string, isFirstRig bool) error { } // Note: Mayor and Deacon are now created by gt install in town beads. - // isFirstRig parameter is kept for backward compatibility but no longer used. - _ = isFirstRig for _, agent := range agents { // Check if already exists diff --git a/internal/rig/manager_test.go b/internal/rig/manager_test.go index ca378922..c380441f 100644 --- a/internal/rig/manager_test.go +++ b/internal/rig/manager_test.go @@ -381,7 +381,7 @@ esac t.Setenv("BEADS_DIR", "") // Clear any existing BEADS_DIR manager := &Manager{townRoot: townRoot} - if err := manager.initAgentBeads(rigPath, "demo", "gt", false); err != nil { + if err := manager.initAgentBeads(rigPath, "demo", "gt"); err != nil { t.Fatalf("initAgentBeads: %v", err) }