@bobfrankston/rmfmail 1.1.94 → 1.1.96

package/.commitmsg

@@ -1,8 +1,15 @@
-Tabs: keep the strip visible with one tab so "+" is discoverable
+Alarm/snooze: log decisions to the daemon log so snooze is verifiable
 
-The view-tab strip was hidden until a 2nd tab existed — but with it
-hidden there was no hint that tabs exist or that Ctrl+T creates one
-(Bob: "there needs to be some indicator that ^t creates a tab"). The
-strip now stays visible with a single tab, showing that tab plus the
-"+" button; its tooltip names the Ctrl+T shortcut. Hidden only before
-the very first tab is seeded.
+On review the snooze keying is actually correct — snoozeItem stores under
+item.uuid, which IS the full per-occurrence key (uuid:startMs@offset for
+calendar, uuid:dueMs for tasks) that collectDueAlarms reads back as
+snoozed[key]. They match.
+
+What was missing was visibility: the [alarm] decisions used plain
+console.log, which from the WebView never reaches the daemon log file —
+so "is snooze working?" couldn't be answered after the fact. Added an
+alog() helper that mirrors to the daemon log via logClientEvent, a
+"snooze saved" line (key + minutes + untilMs), and routed the existing
+fire-decision / dismiss / unknown-result logs through it. Next snooze is
+now traceable: the "snooze saved" line and the following "fire decision"
+line confirm the snooze took and for how long.

package/client/app.bundle.js

@@ -4829,6 +4829,13 @@ var alarms_exports = {};
 __export(alarms_exports, {
   startAlarmPoller: () => startAlarmPoller
 });
+function alog(tag, data) {
+  console.log(`[alarm] ${tag}`, data ?? "");
+  try {
+    logClientEvent(`[alarm] ${tag}`, data);
+  } catch {
+  }
+}
 function loadDismissed() {
   try {
     return JSON.parse(localStorage.getItem(DISMISSED_KEY) || "{}");
@@ -4903,7 +4910,7 @@ async function collectDueAlarms(now) {
         }
       }
       if (pick) {
-        console.log(`[alarm] fire decision key=${pick.key} title="${ev.title || ""}" startMs=${startMs}`);
+        alog("fire decision", { key: pick.key, title: ev.title || "", startMs, effective: pick.effective });
         items.push({
           uuid: pick.key,
           kind: "calendar",
@@ -5176,7 +5183,7 @@ async function firePopupForItem(item) {
         const m = loadDismissed();
         m[item.uuid] = true;
         saveDismissed(m);
-        console.log(`[alarm] dismiss saved key=${item.uuid}`);
+        alog("dismiss saved", { key: item.uuid, via: r.button });
       }
       break;
     case "Delete":
@@ -5189,7 +5196,7 @@ async function firePopupForItem(item) {
       deleteCalendarEvent(item.uuid).catch((e) => console.error(`  [alarm] delete failed for ${item.uuid}: ${e?.message || e}`));
       break;
     default:
-      console.warn(`[alarm] unknown popup result: button=${JSON.stringify(r.button)}, form=${JSON.stringify(r.form)}`);
+      alog("unknown popup result", { button: r.button, form: r.form });
       {
         const m = loadDismissed();
         m[item.uuid] = true;
@@ -5204,6 +5211,7 @@ function snoozeItem(item, minutes) {
   map[item.uuid] = now + minutes * 6e4;
   saveSnoozed(map);
   firedThisSession.delete(item.uuid);
+  alog("snooze saved", { key: item.uuid, minutes, untilMs: map[item.uuid] });
 }
 async function pollAlarms() {
   if (document.hidden)
@@ -7166,7 +7174,7 @@ function showComposeOverlay(title = "Compose") {
   if (isSmall) {
     wrapper.style.cssText = "position:fixed;inset:0;z-index:1000;display:flex;flex-direction:column;background:#fff;";
   } else {
-    wrapper.style.cssText = "position:fixed;bottom:0;right:16px;width:min(900px,55vw);height:min(700px,70vh);z-index:1000;border-radius:8px 8px 0 0;box-shadow:0 -4px 24px rgba(0,0,0,0.3);display:flex;flex-direction:column;overflow:hidden;";
+    wrapper.style.cssText = "position:fixed;top:48px;left:calc((100vw - min(900px,55vw)) / 2);width:min(900px,55vw);height:min(700px,70vh);z-index:1000;border-radius:8px;box-shadow:0 4px 24px rgba(0,0,0,0.3);display:flex;flex-direction:column;overflow:hidden;";
   }
   const titleBar = document.createElement("div");
   titleBar.style.cssText = "display:flex;align-items:center;justify-content:space-between;padding:4px 8px;background:#e8ecf0;border-radius:8px 8px 0 0;cursor:move;user-select:none;flex-shrink:0;";