From 3aa0c71c32c72038e64c82e2d6fae8a7ccf69647 Mon Sep 17 00:00:00 2001 From: Harry Culpan Date: Tue, 30 Jun 2026 18:33:32 -0400 Subject: [PATCH] Log and return PersistSession error in check command A failed file write was silently ignored, causing attendance updates to be lost with no indication of failure to cron or the log. Co-Authored-By: Claude Sonnet 4.6 --- cmd/cli/cmd/check.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cmd/cli/cmd/check.go b/cmd/cli/cmd/check.go index e9ceb1f..b359e2c 100644 --- a/cmd/cli/cmd/check.go +++ b/cmd/cli/cmd/check.go @@ -121,7 +121,10 @@ will attend the next session.`, } } - data.PersistSession(session) + if err := data.PersistSession(session); err != nil { + log.Default().Printf("ERROR persisting session: %s", err) + return err + } } return nil