Merge pull request 'fix: use full Lock in State.Save() to prevent data race (closes #17 )' (#20 ) from fix/state-save-data-race into main

Reviewed-on: #20
fix: use full Lock in State.Save() to prevent data race (closes #17 )
2026-02-21 11:22:46 +01:00 · 2026-02-21 00:51:58 -08:00
1 changed files with 2 additions and 2 deletions
--- a/internal/state/state.go
+++ b/internal/state/state.go
@@ -156,8 +156,8 @@ func (s *State) Load() error {

 // Save writes the current state to disk atomically.
 func (s *State) Save() error {
-	s.mu.RLock()
-	defer s.mu.RUnlock()
+	s.mu.Lock()
+	defer s.mu.Unlock()

 	s.snapshot.LastUpdated = time.Now().UTC()
Author	SHA1	Message	Date
Jeffrey Paul	8cfff5dcc8	Merge pull request 'fix: use full Lock in State.Save() to prevent data race (closes #17 )' (#20 ) from fix/state-save-data-race into main Some checks failed Check / check (push) Failing after 5m43s Details Reviewed-on: #20	2026-02-21 11:22:46 +01:00
clawbot	b162ca743b	fix: use full Lock in State.Save() to prevent data race (closes #17 ) Some checks failed Check / check (pull_request) Failing after 5m31s Details State.Save() was using RLock but mutating s.snapshot.LastUpdated, which is a write operation. This created a data race since other goroutines could also hold a read lock and observe a partially written timestamp. Changed to full Lock to ensure exclusive access during the mutation.	2026-02-21 00:51:58 -08:00