@meri-imperiumi/signalk-passage-briefing 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## [Unreleased]
 
+## [0.8.1] - 2026-10-08
+
+### Added
+- Branding on the webapp page (per the Signal K visuals spec): the plugin's app icon (`icon-256.png`) serves as favicon and the base canvas color ships as the `theme-color` meta, so browser chrome matches the console background in both day and night modes.
+
+### Fixed
+- The per-minute source-status tick no longer touches the logbook — fixes "Server unreachable: Fetch is aborted" with an empty route picker on production since 0.8.0. The checklist's logbook probe went through the resources API, whose `logentries` provider answers even a `limit: 1` listing by reading and schema-validating every day file — on a multi-year log that meant overlapping full-store scans every 60 s starving the server so `/api/status` and `/api/routes` blew the webapp's 8 s abort timeout. The logbook is a once-used learning resource (backfill), not an ongoing ingest source, so it leaves the data source statuses entirely; with no provider installed, backfill now fails loudly with the reason instead of silently reporting zero events.
+
 ## [0.8.0] - 2026-10-06
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meri-imperiumi/signalk-passage-briefing",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Offshore passage daily briefing webapp for Signal K",
   "main": "plugin/index.js",
   "scripts": {

package/plugin/index.js

@@ -25,7 +25,6 @@ const { mkdir, readFile, unlink, writeFile } = require("node:fs/promises");
 const { PassageStateMachine } = require("./state-machine.js");
 const { PassageDatabase } = require("./sqlite-db.js");
 const {
-  logbookAvailable,
   readLogbookEntries,
   readLogbookSailEvents,
 } = require("./logbook-source.js");
@@ -1997,14 +1996,6 @@ module.exports = (app) => {
         now,
       });
     }
-    sourceStatus.record({
-      id: "sk-logbook",
-      label: "Logbook (signalk-logbook)",
-      kind: "signalk",
-      url: "resources/logentries",
-      ...((await logbookAvailable(app)) ? {} : { absent: true }),
-      now,
-    });
   }
 
   /**

package/plugin/logbook-source.js

@@ -465,17 +465,20 @@ function normalizeLogEntry(entry) {
  * explicit window, so "all history" is an explicit epoch-to-eternity
  * range.
  *
- * A missing resources API (old server) or a failed listing (the
- * logbook plugin not installed — no provider for the type) reads as
- * an empty store, never a failure: this consumer degrades, the same
- * way a missing store directory used to.
+ * Loud on an unusable source: a missing resources API (old server)
+ * or a failed listing (the logbook plugin not installed — no provider
+ * for the type) throws with the reason, so the human-initiated runs
+ * that read the logbook (backfill, event diagnostics) can say why
+ * they found nothing instead of silently reporting zero events.
  *
  * @param {object} app - Signal K plugin `app`
  * @returns {Promise<Array<object>>} Normalized entries, oldest first
+ * @throws {Error} When the resources API or the `logentries` provider
+ *   is unavailable
  */
 async function readLogbookEntries(app) {
   if (typeof app?.resourcesApi?.listResources !== "function") {
-    return []; // No resources API on this server
+    throw new Error("Resources API not available on this server");
   }
   let resourceMap;
   try {
@@ -483,8 +486,10 @@ async function readLogbookEntries(app) {
       from: ALL_HISTORY_FROM,
       to: ALL_HISTORY_TO,
     });
-  } catch (_error) {
-    return []; // No provider for the type (logbook not installed)
+  } catch (error) {
+    throw new Error(
+      `No logentries resource provider on this server — install signalk-logbook to backfill from the log (${error.message})`,
+    );
   }
   if (!resourceMap || typeof resourceMap !== "object") {
     return [];
@@ -542,26 +547,6 @@ async function readLogbookEntriesRest({ baseUrl, token } = {}) {
     );
 }
 
-/**
- * Whether the `logentries` resource is being served on this server
- * (the signalk-logbook provider is registered). Probed with a
- * one-entry listing — cheap even on years of logs.
- *
- * @param {object} app - Signal K plugin `app`
- * @returns {Promise<boolean>}
- */
-async function logbookAvailable(app) {
-  if (typeof app?.resourcesApi?.listResources !== "function") {
-    return false;
-  }
-  try {
-    await app.resourcesApi.listResources(RESOURCE_TYPE, { limit: 1 });
-    return true;
-  } catch (_error) {
-    return false;
-  }
-}
-
 /**
  * Reads the logbook through the resources API and extracts
  * sail-change events, optionally bounded to a window (state tracking
@@ -609,7 +594,6 @@ module.exports = {
   normalizeLogEntry,
   readLogbookEntries,
   readLogbookEntriesRest,
-  logbookAvailable,
   readLogbookSailEvents,
   foldTwaDegrees,
 };

package/public/index.html

@@ -3,7 +3,9 @@
 <head>
   <meta charset="utf-8">
   <meta name="viewport" content="width=device-width, initial-scale=1">
+  <meta name="theme-color" content="#080a0c">
   <title>Passage Briefing</title>
+  <link rel="icon" href="icon-256.png">
   <link rel="stylesheet" href="css/visuals.css">
   <script type="module" src="app.js"></script>
 </head>

package/tests/logbook-source.test.js

@@ -350,8 +350,8 @@ describe("readLogbookEntries", () => {
     assert.ok(queries[0].from && queries[0].to);
   });
 
-  test("no resources API or no provider reads as an empty store", async () => {
-    assert.deepEqual(await readLogbookEntries({}), []);
+  test("no resources API or no provider fails loudly with the reason", async () => {
+    await assert.rejects(readLogbookEntries({}), /Resources API not available/);
     const failing = {
       resourcesApi: {
         async listResources() {
@@ -359,7 +359,10 @@ describe("readLogbookEntries", () => {
         },
       },
     };
-    assert.deepEqual(await readLogbookEntries(failing), []);
+    await assert.rejects(
+      readLogbookEntries(failing),
+      /logentries resource provider/,
+    );
   });
 });
 
@@ -453,7 +456,7 @@ describe("readLogbookSailEvents", () => {
     assert.equal(windowed[0].eventType, "REEF_INCREASE");
   });
 
-  test("missing provider is not fatal", async () => {
+  test("missing provider fails loudly with the reason", async () => {
     const failing = {
       resourcesApi: {
         async listResources() {
@@ -461,7 +464,13 @@ describe("readLogbookSailEvents", () => {
         },
       },
     };
-    assert.deepEqual(await readLogbookSailEvents({ app: failing }), []);
-    assert.deepEqual(await readLogbookSailEvents({ app: {} }), []);
+    await assert.rejects(
+      readLogbookSailEvents({ app: failing }),
+      /logentries resource provider/,
+    );
+    await assert.rejects(
+      readLogbookSailEvents({ app: {} }),
+      /Resources API not available/,
+    );
   });
 });

package/tests/plugin.test.js

@@ -2110,7 +2110,6 @@ test("source status: SK availability recorded and served at /sources", async ()
   assert.equal(entries["sk-polar"].lastStatus, "ok");
   assert.equal(entries["sk-ships-time"].lastStatus, "absent");
   assert.equal(entries["sk-routes-resources"].lastStatus, "fail");
-  assert.equal(entries["sk-logbook"].lastStatus, "absent");
 
   // Configured sources have entries even before their first cycle
   assert.equal(entries["weather-track"].lastStatus, null);