From b241a353f3e1b0b6c71f878de28d7e00332d6157 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Tue, 30 Dec 2025 22:29:10 -0800 Subject: [PATCH] Clean up stale issue references in comments (gt-z5bri) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - daemon.go: Update gt-2hzl4 comment to past tense (timeout fallback is implemented) - lifecycle.go: Remove "to be removed" promise for gt-psuw7 (code stays) - keepalive_test.go: Remove tombstoned gt-gaxo epic reference - doctor.go: Remove tombstoned gt-gaxo epic reference - daemon_test.go: Remove tombstoned gt-gaxo epic reference 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- internal/cmd/doctor.go | 1 - internal/daemon/daemon.go | 2 +- internal/daemon/daemon_test.go | 2 +- internal/daemon/lifecycle.go | 2 +- internal/keepalive/keepalive_test.go | 1 - 5 files changed, 3 insertions(+), 5 deletions(-) diff --git a/internal/cmd/doctor.go b/internal/cmd/doctor.go index 4f55d19e..d8212c04 100644 --- a/internal/cmd/doctor.go +++ b/internal/cmd/doctor.go @@ -102,7 +102,6 @@ func runDoctor(cmd *cobra.Command, args []string) error { d.Register(doctor.NewAgentBeadsCheck()) // NOTE: StaleAttachmentsCheck removed - staleness detection belongs in Deacon molecule - // See gt-gaxo epic for ZFC cleanup rationale // Config architecture checks d.Register(doctor.NewSettingsCheck()) diff --git a/internal/daemon/daemon.go b/internal/daemon/daemon.go index 9aca3b6e..b110a769 100644 --- a/internal/daemon/daemon.go +++ b/internal/daemon/daemon.go @@ -263,7 +263,7 @@ func (d *Daemon) ensureDeaconRunning() { if beadErr == nil { if beadState == "running" || beadState == "working" { // Agent reports it's running - trust it - // Note: gt-2hzl4 will add timeout fallback for stale state + // Timeout fallback for stale state is in lifecycle.go return } } diff --git a/internal/daemon/daemon_test.go b/internal/daemon/daemon_test.go index 58b64f95..73853bf3 100644 --- a/internal/daemon/daemon_test.go +++ b/internal/daemon/daemon_test.go @@ -207,7 +207,7 @@ func TestSaveLoadState_Roundtrip(t *testing.T) { } // NOTE: TestIsWitnessSession removed - isWitnessSession function was deleted -// as part of ZFC cleanup (gt-gaxo). Witness poking is now Deacon's responsibility. +// as part of ZFC cleanup. Witness poking is now Deacon's responsibility. func TestLifecycleAction_Constants(t *testing.T) { // Verify constants have expected string values diff --git a/internal/daemon/lifecycle.go b/internal/daemon/lifecycle.go index 5a93a282..930400ea 100644 --- a/internal/daemon/lifecycle.go +++ b/internal/daemon/lifecycle.go @@ -171,7 +171,7 @@ func (d *Daemon) executeLifecycleAction(request *LifecycleRequest) error { } } - // Check if session exists (legacy tmux detection - to be removed per gt-psuw7) + // Check if session exists (tmux detection still needed for lifecycle actions) running, err := d.tmux.HasSession(sessionName) if err != nil { return fmt.Errorf("checking session: %w", err) diff --git a/internal/keepalive/keepalive_test.go b/internal/keepalive/keepalive_test.go index c70b4ac6..5fc42077 100644 --- a/internal/keepalive/keepalive_test.go +++ b/internal/keepalive/keepalive_test.go @@ -61,7 +61,6 @@ func TestStateAge(t *testing.T) { // NOTE: IsFresh(), IsStale(), IsVeryStale() were removed as part of ZFC cleanup. // Staleness classification belongs in Deacon molecule, not Go code. - // See gt-gaxo epic for rationale. } func TestDirectoryCreation(t *testing.T) {