fix(org-contacts): set org-contacts-files eagerly so launch doesn't error

At startup the agenda-finalize hook ran cj/org-contacts-anniversaries-safe, which calls org-contacts-anniversaries, which calls org-contacts-files. That function messages "[org-contacts] ERROR: Your custom variable `org-contacts-files' is nil." when the variable is nil, and at that point it was nil. The value was set via the use-package :custom, which only applies when org-contacts loads, and that load is deferred behind :after (org mu4e) — later than the first agenda finalize.

I set org-contacts-files eagerly at require time instead, so it's never nil by the time the hook fires. I also guarded the wrapper: org-contacts-files emits a message rather than signaling, so ignore-errors couldn't suppress it on its own. Now the call only runs when the variable is set.

Three tests cover the eager set, the guard skipping when files are nil, and the wrapper running when they're set. Full suite green.

1 files changed, 18 insertions, 4 deletions

diff --git a/modules/org-contacts-config.el b/modules/org-contacts-config.el
index 2f919c19..ff14629b 100644
--- a/modules/org-contacts-config.el
+++ b/modules/org-contacts-config.el
@@ -12,6 +12,16 @@
 
 (require 'user-constants)
 
+;; Set `org-contacts-files' eagerly at require time.  Setting it in the
+;; `use-package' form below would only apply when org-contacts loads, which is
+;; deferred behind `:after (org mu4e)' -- later than the first
+;; `org-agenda-finalize'.  The anniversaries hook below runs on that finalize
+;; and calls `org-contacts-files', which *messages* (not signals) an error when
+;; the variable is nil, so the hook's `ignore-errors' can't suppress it.  Set
+;; it here so the variable is never nil by the time the hook fires.
+(defvar org-contacts-files nil)
+(setq org-contacts-files (list contacts-file))
+
 ;; --------------------------- Org Agenda Integration --------------------------
 
 (with-eval-after-load 'org-agenda
@@ -29,8 +39,12 @@
     (let ((date (calendar-current-date))
           (entry "")
           (original-date (calendar-current-date)))
-      (ignore-errors
-        (org-contacts-anniversaries))))
+      ;; `org-contacts-anniversaries' calls `org-contacts-files', which
+      ;; messages an error when the variable is nil.  A message isn't a
+      ;; signal, so `ignore-errors' alone can't suppress it -- guard the call.
+      (when org-contacts-files
+        (ignore-errors
+          (org-contacts-anniversaries)))))
 
   ;; Use the safe wrapper instead
   (add-hook 'org-agenda-finalize-hook 'cj/org-contacts-anniversaries-safe))
@@ -209,8 +223,8 @@ module provides more sophisticated completion."
 
 (use-package org-contacts
   :after (org mu4e)
-  :custom
-  (org-contacts-files (list contacts-file))
+  ;; `org-contacts-files' is set eagerly near the top of this file, not here,
+  ;; so the agenda-finalize anniversaries hook never sees it nil at startup.
   :config
   (require 'mu4e)
   ;; Basic settings