feat: reduce KV writes - only save on status change or once daily

Agent-Logs-Url: https://github.com/SillyLittleTech/UpMagic/sessions/4b09e790-8b74-410c-bab6-5257f12e7b43

Co-authored-by: kiyarose <75678535+kiyarose@users.noreply.github.com>

Author: Copilot

worker/index.js

@@ -123,8 +123,10 @@ async function runHeartbeat(env) {
   if (!state.historyByService) state.historyByService = {};
 
   let statusChanged = false;
+  let needsSave = false;
   let notifications = [];
   const heartbeatAt = Date.now();
+  const todayUTC = new Date(heartbeatAt).toISOString().slice(0, 10); // "YYYY-MM-DD"
 
   for (const target of targets) {
     if (state.services[target.id]?.isManual) {
@@ -168,25 +170,46 @@ async function runHeartbeat(env) {
       status: newStatus,
       latency: latency
     });
+
+    // Internal log every heartbeat (no KV write)
+    console.log(`[heartbeat] ${target.name}: ${newStatus}${latency !== null ? ` (${latency}ms)` : ''}`);
   }
 
-  // Fetch maintained issues from ProjectHub
-  try {
-    const issuesRes = await fetch('https://api.github.com/repos/SillyLittleTech/projecthub/issues?labels=maintain&state=open', {
-      headers: {
-        'User-Agent': 'SLT-Status-Worker',
-        'Authorization': env.ISSUES_PAT ? `Bearer ${env.ISSUES_PAT}` : ''
+  // Fetch maintained issues from ProjectHub at most once per day
+  const lastIssuesFetchDate = state._lastIssuesFetchDay || '';
+  if (lastIssuesFetchDate !== todayUTC) {
+    try {
+      const issuesRes = await fetch('https://api.github.com/repos/SillyLittleTech/projecthub/issues?labels=maintain&state=open', {
+        headers: {
+          'User-Agent': 'SLT-Status-Worker',
+          'Authorization': env.ISSUES_PAT ? `Bearer ${env.ISSUES_PAT}` : ''
+        }
+      });
+      if (issuesRes.ok) {
+        const issues = await issuesRes.json();
+        state.maintenance = issues.map(i => ({ title: i.title, url: i.html_url, created_at: i.created_at }));
+        state._lastIssuesFetchDay = todayUTC;
+        console.log(`[daily] Fetched ${issues.length} maintenance issue(s) for ${todayUTC}`);
+        needsSave = true; // persist updated maintenance list
       }
-    });
-    if (issuesRes.ok) {
-      const issues = await issuesRes.json();
-      state.maintenance = issues.map(i => ({ title: i.title, url: i.html_url, created_at: i.created_at }));
+    } catch (e) {
+      console.error('Failed to fetch maintenance issues', e);
     }
-  } catch (e) {
-    console.error('Failed to fetch maintenance issues', e);
   }
 
-  await saveState(env, state);
+  // Only write to KV when something changed, or once per day as a daily summary
+  const lastDailySaveDate = state._lastDailySaveDay || '';
+  const isDailySave = lastDailySaveDate !== todayUTC;
+  if (statusChanged || needsSave || isDailySave) {
+    if (isDailySave) {
+      state._lastDailySaveDay = todayUTC;
+      const summary = Object.entries(state.services)
+        .map(([, svc]) => `${svc.name}: ${svc.status}`)
+        .join(', ');
+      console.log(`[daily summary] ${todayUTC} — ${summary}`);
+    }
+    await saveState(env, state);
+  }
 }
 
 async function saveState(env, state) {

wrangler.toml

@@ -20,4 +20,4 @@ id = "d6179b2f901645de81b9805c4e7ac591"
 preview_id = "59e7164e44154d7fa79b2a093b338530"
 
 [triggers]
-crons = ["* * * * *"] # Run every minute for heartbeat
+crons = ["*/5 * * * *"] # Run every 5 minutes for heartbeat