fix: make navigate usecase close idempotent

commit: 83a2b1bb325259d19ce2c82d1c42d1feef0882c2
author: brice · 8mo ago · 2026-02-07 08:26 UTC
parent: c948582
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +4 -1

M internal/application/usecase/navigate.go +4 -1


@@ -56,6 +56,7 @@ type NavigateUseCase struct {
56	56		defaultZoom float64
57	57		recentMu sync.Mutex
58	58		recentVisits map[string]paneHistoryState // key: pane ID, value: per-pane history state
	59	+	closeOnce sync.Once
59	60
60	61		// Async history recording
61	62		historyQueue chan historyRecord

@@ -95,7 +96,9 @@ func NewNavigateUseCase(
95	96		// Close shuts down the background history worker and drains any pending records.
96	97		// This should be called when the application is shutting down.
97	98		func (uc *NavigateUseCase) Close() {
98		-	close(uc.done)
	99	+	uc.closeOnce.Do(func() {
	100	+	close(uc.done)
	101	+	})
99	102		uc.wg.Wait()
100	103		}
101	104

 	defaultZoom  float64
 	recentMu     sync.Mutex
 	recentVisits map[string]paneHistoryState // key: pane ID, value: per-pane history state
+	closeOnce    sync.Once
 	// Async history recording
 	historyQueue chan historyRecord
 // Close shuts down the background history worker and drains any pending records.
 // This should be called when the application is shutting down.
 func (uc *NavigateUseCase) Close() {
-	close(uc.done)
+	uc.closeOnce.Do(func() {
+		close(uc.done)
+	})
 	uc.wg.Wait()
 }