fix: alarm scheduling fuer manual-only feeds + Test-Coverage

- sw.js: Korrekte Alarm-Updates auch bei deaktiviertem Global-Intervall
- tests/sw-schedule.test.mjs: Regression-Coverage fuer Scheduling
- CHANGELOG/README: aktualisiert

Author: Lukas Geiger

CHANGELOG.md

@@ -5,9 +5,13 @@
 ### Added
 - Added automated light/dark theme coverage for popup and options CSS variables.
 - Added a read-only GitHub Actions CI workflow for the Node test suite.
+- Added regression coverage for service-worker alarm scheduling.
+
+### Fixed
+- Fixed alarm updates for manual-only feeds when global interval is disabled but other feeds define per-feed intervals.
 
 ### Verified
-- `npm test` now covers 25 dependency-free Node tests, including theme coverage.
+- `npm test` now covers 26 dependency-free Node tests, including theme and service-worker scheduling coverage.
 
 ## [1.1.2] — 2026-04-30
 

README.md

@@ -59,7 +59,7 @@ Store listing is planned after the remaining browser and screenshot checks.
 
 ## Development
 
-RSS-BOOK has no build step. The repository includes 25 dependency-free Node tests for parser behavior, OPML, storage, bookmark cleanup, feed discovery, folder export, store assets, and light/dark theme CSS coverage:
+RSS-BOOK has no build step. The repository includes 26 dependency-free Node tests for parser behavior, OPML, storage, bookmark cleanup, feed discovery, folder export, store assets, service-worker scheduling, and light/dark theme CSS coverage:
 
 ```bash
 npm test

sw.js

@@ -71,13 +71,11 @@ async function ensureAlarm() {
 
 async function runUpdateCycle(reason) {
   console.log(`[RSS-BOOK] Update cycle (${reason})`);
+  const { settings } = await getState();
   const feeds = await getEnabledFeeds();
 
   for (const feed of feeds) {
-    if (reason === "alarm" && feed.intervalMinutes > 0) {
-      const due = !feed.lastFetch || (Date.now() - feed.lastFetch) >= feed.intervalMinutes * 60_000;
-      if (!due) continue;
-    }
+    if (!shouldUpdateFeedForReason(feed, reason, settings)) continue;
     try {
       await updateOneFeed(feed.id);
     } catch (err) {
@@ -97,6 +95,18 @@ async function runUpdateCycle(reason) {
   }
 }
 
+export function shouldUpdateFeedForReason(feed, reason, settings = {}, now = Date.now()) {
+  if (reason !== "alarm") return true;
+
+  const feedInterval = Number(feed.intervalMinutes) || 0;
+  if (feedInterval > 0) {
+    return !feed.lastFetch || (now - feed.lastFetch) >= feedInterval * 60_000;
+  }
+
+  const globalInterval = Number(settings?.globalIntervalMinutes) || 0;
+  return globalInterval > 0;
+}
+
 async function updateOneFeed(feedId) {
   const { feeds } = await getState();
   const feed = feeds?.[feedId];

tests/sw-schedule.test.mjs

@@ -0,0 +1,87 @@
+import { afterEach, test } from "node:test";
+import assert from "node:assert/strict";
+
+afterEach(() => {
+  delete globalThis.chrome;
+});
+
+function installChromeMock() {
+  const addListener = () => {};
+
+  globalThis.chrome = {
+    runtime: {
+      onInstalled: { addListener },
+      onStartup: { addListener },
+      onMessage: { addListener }
+    },
+    alarms: {
+      onAlarm: { addListener },
+      async get() {
+        return null;
+      },
+      async clear() {},
+      async create() {}
+    },
+    storage: {
+      onChanged: { addListener },
+      local: {
+        async get() {
+          return { settings: {}, feeds: {} };
+        },
+        async set() {}
+      }
+    }
+  };
+}
+
+test("alarm scheduling skips manual-only feeds when global interval is disabled", async () => {
+  installChromeMock();
+  const { shouldUpdateFeedForReason } = await import(`../sw.js?sw-schedule=${Date.now()}`);
+  const now = Date.parse("2026-05-01T12:00:00Z");
+
+  assert.equal(
+    shouldUpdateFeedForReason(
+      { intervalMinutes: 0, lastFetch: 0 },
+      "alarm",
+      { globalIntervalMinutes: 0 },
+      now
+    ),
+    false
+  );
+  assert.equal(
+    shouldUpdateFeedForReason(
+      { intervalMinutes: 0, lastFetch: 0 },
+      "alarm",
+      { globalIntervalMinutes: 15 },
+      now
+    ),
+    true
+  );
+  assert.equal(
+    shouldUpdateFeedForReason(
+      { intervalMinutes: 30, lastFetch: now - 29 * 60_000 },
+      "alarm",
+      { globalIntervalMinutes: 0 },
+      now
+    ),
+    false
+  );
+  assert.equal(
+    shouldUpdateFeedForReason(
+      { intervalMinutes: 30, lastFetch: now - 31 * 60_000 },
+      "alarm",
+      { globalIntervalMinutes: 0 },
+      now
+    ),
+    true
+  );
+  assert.equal(
+    shouldUpdateFeedForReason(
+      { intervalMinutes: 0, lastFetch: 0 },
+      "manual",
+      { globalIntervalMinutes: 0 },
+      now
+    ),
+    true
+  );
+});