From 7ab517cbe333cc86075e50c8265d5ee5137a8fdd Mon Sep 17 00:00:00 2001
From: Will Estes <westes575@gmail.com>
Date: Sat, 29 Aug 2026 07:33:30 -0400
Subject: [PATCH] Fall back to a saint's collectCommon/collectName in
 commemorations too
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

sanctoralCommemorationPart() only ever checked a commemorated saint's
own propers, so a saint like St. Sabina — no propers of her own, but
a real collectCommon + collectName template fallback, verified text
and all — rendered as an outright "missing" commemoration. The day's
primary-collect resolver (getDayCollect) already had this same
collectCommon/collectName fallback tier; this brings the
commemoration path in line with it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VFGHb4XMe6Wya4pGpPhEEi
---
 src/hours/resolve-common.ts | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/hours/resolve-common.ts b/src/hours/resolve-common.ts
index 84edaec..23b2c98 100644
--- a/src/hours/resolve-common.ts
+++ b/src/hours/resolve-common.ts
@@ -542,7 +542,16 @@ function sanctoralCommemorationPart(commemoration: Extract<Commemoration, { kind
     if (combined.status.la !== 'missing' || combined.status.en !== 'missing') {
       return { kind: 'preces', text: combined, label };
     }
-    return { kind: 'preces', text: resolveCommon(`${saint.propers}-collect`), label };
+    const properCollect = resolveCommon(`${saint.propers}-collect`);
+    if (properCollect.status.la !== 'missing' || properCollect.status.en !== 'missing') {
+      return { kind: 'preces', text: properCollect, label };
+    }
+  }
+  if (saint?.collectCommon && saint.collectName) {
+    const template = resolveCommon(saint.collectCommon);
+    if (template.status.la !== 'missing' || template.status.en !== 'missing') {
+      return { kind: 'preces', text: substituteName(template, saint.collectName), label };
+    }
   }
   return { kind: 'preces', text: { text: {}, status: { la: 'missing', en: 'missing' } }, label };
 }