signalk-pypilot-newui 3.1.3 → 3.2.0

package/CHANGELOG.md

@@ -1,5 +1,108 @@
 # Changelog
 
+## 3.2.0 — 2026-10-15 — AIS rose rebuilt on real positions, tide popup, chart kept warm
+
+Rolls up Rev470 → Rev482 on top of 3.1.4. Minor bump: new UI (tide
+button + popup, "Chart in the background" card, anchor-watch visor
+embedded in the rose) plus a set of AIS rose fixes found in port QA in
+Moaña. No breaking changes.
+
+### Fixed — AIS targets on the rose (root cause)
+
+- **The target position/SOG/COG cache never filled.** The REST calls
+  used `encodeURIComponent` on the vessel URN
+  (`urn%3Amrn%3Aimo%3Ammsi%3A…`) and Signal K answers 404 to that; the
+  raw URN answers 200. With an empty cache the hull fell back to
+  `cpaPositions.target` — the target's FUTURE position at CPA — so a
+  boat coming from 7 o'clock was drawn at 11 o'clock and "0 m" away.
+  The fallback is removed: no current position → no hull on the rose
+  (the infobox still shows the alarm). Cache refresh 30 s → 5 s.
+  (Rev474)
+- **Hull and projected track were drawn 180° off.** The sprite has its
+  bow at +x, so a direction needs `rotate(angle − 90)`; Rev457 used
+  `+ 90`. (Rev470)
+- **Moored boats.** A stopped target's COG is GPS noise: the hull is
+  oriented by `headingTrue` when available, by COG only when moving
+  (≥ 0.5 kn), otherwise it points at us; the dashed track is drawn only
+  for moving targets and always along COG. The "stationary" filter
+  (now 0.5 kn) is re-evaluated when the REST cache lands, not only on
+  the alarm delta. (Rev470, Rev471)
+- **Targets drawn on top of our own hull.** Radial position is now a
+  log scale (50 m → 2 NM) over the free ring between our hull outline
+  and the compass ring, with a distance label ("40 m", "0.4 NM") under
+  each target. (Rev472)
+- **Target name in the infobox** comes from the Signal K vessel `name`
+  first, then from the alarm text; "MMSI 123456789" is no longer shown
+  as a name. (Rev481)
+- **AIS card:** the Harbour / At anchor / Sailing selector showed raw
+  i18n keys; all texts rewritten in plain language (EN/ES). (Rev473)
+- **Unstyled flash on reload:** critical inline CSS in `<head>` and
+  `style.css?v=RevNNN` cache-busting from `deploy.ps1`. (Rev476)
+
+### Added
+
+- **Tide button** above the bottom-right corner tile of the rose (shown
+  only when `signalk-mareas-ihm` is installed). Opens a popup with the
+  station, current height, rising/falling, coefficient, next HW/LW and
+  a −6 h → +12 h tide curve. "Open full Tides" opens the
+  signalk-mareas-ihm anchor-watch visor inside the rose, same chrome as
+  Freeboard (maximize / close). Data from `/signalk-mareas-ihm/api/extremes`
+  and `environment.tide.*`, fetched only when the popup opens.
+  (Rev478 → Rev480, Rev482)
+- **Freeboard-SK kept loaded in the background** so a double-tap on the
+  rose opens it instantly with AIS names already in place. Own card
+  "Chart in the background" with a per-device switch and a "?" that
+  explains when to turn it off (lag / battery). (Rev475 → Rev477)
+
+## 3.1.4 — 2026-10-14 — iOS "Add to Home Screen" PWA basics (issue #5)
+
+Rev469. Patch bump: no code changes, just the HTML shell. Thanks to
+[@eric-fitzsimon](https://github.com/eric-fitzsimon) for the detailed
+report in [#5](https://github.com/Aitonos/signalk-pypilot-newui/issues/5):
+on iOS, adding the webapp to the home screen worked the first launch,
+then subsequent launches loaded the page but Tune gain sliders stayed
+empty and Chart did not populate.
+
+### Added — PWA meta tags in the shell
+
+```html
+<meta name="apple-mobile-web-app-capable" content="yes" />
+<meta name="apple-mobile-web-app-status-bar-style" content="black-translucent" />
+<meta name="apple-mobile-web-app-title" content="PyPilot" />
+<meta name="mobile-web-app-capable" content="yes" />
+<meta name="theme-color" content="#0a1220" />
+<link rel="apple-touch-icon" href="icon.svg" />
+```
+
+Previously the shell had none of these, so iOS treated the home-screen
+bookmark as a half-baked WebView bookmark with a storage context
+distinct from Safari and no explicit standalone contract.
+
+### Fixed — stale `app.js` cached indefinitely by iOS WebView
+
+Added `Cache-Control: no-cache, no-store, must-revalidate` + legacy
+`Pragma` / `Expires` HTTP-equiv metas to the shell. iOS Safari (both
+browser and standalone WebView) was caching the first successful load
+of `app.js` and ignoring the `?v=RevNNN` query bump that `deploy.ps1`
+writes on every deploy. Result: user's bookmark served a stale JS
+bundle for days, mismatching the backend API and leaving features
+unpopulated. These metas force the WebView to re-fetch the shell on
+every launch; the actual `app.js?v=RevNNN` request then matches the
+current backend.
+
+See `docs/_audit_ios_pwa_issue_5.md` for the full forensics.
+
+### Known limitations
+
+- `apple-touch-icon` points to `icon.svg`. iOS prefers PNG for the
+  home-screen icon; with SVG only, iOS may use a page-screenshot
+  fallback. A proper PNG set is TODO for a future release.
+- No Service Worker. Rev-A path only — if the Cache-Control metas do
+  not fully fix the issue on Eric's device, Rev-B/C/D will add a
+  `visibilitychange` reconnect handler, a standalone-mode banner with
+  "Reset app data" button, and eventually a proper SW. Not shipped
+  yet because they are not confirmed necessary.
+
 ## 3.1.3 — 2026-10-14 — Safety: offline-aware write gating + batched trace + AIS polish
 
 Rolls up Rev464 → Rev468 (5 revs) on top of 3.1.2. Patch bump: no

package/README.md

@@ -277,7 +277,10 @@ on its own), but installing them unlocks the features listed.
   — Double-tap the compass rose to open Freeboard-SK embedded inside
   the rose box (side panels stay visible and usable). Lets you drop
   from the autopilot view to the full chart with every AIS target in
-  one tap, without leaving the pypilot-newui session. **Pair with a
+  one tap, without leaving the pypilot-newui session. Freeboard is kept
+  loaded in the background so it opens instantly (per-device switch in
+  the "Chart in the background" card if a weak tablet prefers lazy
+  loading). **Pair with a
   Signal K chart provider (e.g. `@signalk/charts-plugin`) or the chart
   area will stay empty** — see
   [Chart provider is required](#chart-provider-is-required-for-the-embedded-plotter).
@@ -286,13 +289,24 @@ on its own), but installing them unlocks the features listed.
 - **[`signalk-collision-alerts`](https://www.npmjs.com/package/signalk-collision-alerts)**
   by @dirkwa — Provides the CPA/TCPA engine behind our AIS collision
   overlay. Pypilot-newui subscribes to
-  `notifications.navigation.closestApproach.*` and renders a red /
-  orange triangle on the rose + an infobox with ACK that routes
+  `notifications.navigation.closestApproach.*` and draws each target
+  on the rose at its real bearing and distance (log scale, distance
+  label), oriented by its heading / course with a projected track line
+  when it is moving, plus an infobox with ACK that routes
   through the Signal K v2 Notifications API, so an ACK from
   pypilot-newui is also seen by Freeboard-SK and every other SK
   client.
   See `docs/_feature_ais_overlay.md`.
 
+- **[`signalk-mareas-ihm`](https://www.npmjs.com/package/signalk-mareas-ihm)**
+  ("AnchorWatch Pro: Smart Anchoring, AIS & Tides" in the App Store) —
+  **Recommended.** Unlocks the tide button on the rose (popup with the
+  tide curve, next HW/LW and coefficient) and opens its anchor-watch
+  visor inside the rose. It also fills AIS vessel names into the Signal
+  K tree from its own persistent AIS database, so pypilot-newui,
+  Freeboard-SK and collision-alerts show names instead of raw MMSIs
+  right after a server restart.
+
 - **[`pypilot-autopilot-provider`](https://www.npmjs.com/package/pypilot-autopilot-provider)**
   by Panaaj — Shares the single pypilot socket cleanly with our
   autopilot provider (see next section) so KIP, Freeboard's Autopilot

package/dist/index.js

@@ -28,7 +28,7 @@ const trip_recorder_1 = require("./trip-recorder");
 const maneuver_trace_1 = require("./maneuver-trace");
 // Rev counter bumped on every build so the user can distinguish deploys
 // from the webapp header (feedback_revision_bump_each_build).
-const PLUGIN_REVISION = "Rev468";
+const PLUGIN_REVISION = "Rev482";
 // Rev59: read package.json once at load time so /status can report the
 // npm package version alongside the internal Rev counter.
 // eslint-disable-next-line @typescript-eslint/no-require-imports
@@ -193,7 +193,7 @@ module.exports = function (app) {
     let _tackStatePrev = null;
     let _tackDirection = null;
     // Rev283: persistent servo error log. Writes JSONL to dataDir on
-    // every off→on transition of a servo-* alarm rule and remembers the
+    // every off→on transition of a servo-* alarm rule and remembers the
     // 30 s pre-fault window from the historian.
     let servoErrorLog = null;
     // Rev286 (B2): profile advisor + last-published state for the SK
@@ -223,9 +223,9 @@ module.exports = function (app) {
     // Rev323 (Carlos, 2026-09-27): pseudo-modo visor-side. El visor lo
     // posta a `/maneuver-state` al arrancar / cerrar Aproado o Empopado.
     // Se usa para silenciar heading-deviation / cruise-drift / unable-
-    // to-steer durante la maniobra, INDEPENDIENTE de `ap.tack.state` —
+    // to-steer durante la maniobra, INDEPENDIENTE de `ap.tack.state` —
     // en modo wind pypilot NO obedece y ap.tack.state se queda en "none"
-    // aunque el visor esté pilotando la maniobra por otras vías.
+    // aunque el visor esté pilotando la maniobra por otras vías.
     let _maneuverPseudoActive = false;
     let lastNavState = null;
     let sessionSampleTimer = null;
@@ -382,24 +382,24 @@ module.exports = function (app) {
                 sensorsIgnored: {
                     type: "array",
                     title: "Ignored sensor SK paths",
-                    description: "List of Signal K paths the sailor has marked as 'not on this boat' from Setup → Sensor Quality. Ignored sensors stop turning the Setup chip red without hiding the row. Managed from the visor; edit here only if you know the exact path names.",
+                    description: "List of Signal K paths the sailor has marked as 'not on this boat' from Setup → Sensor Quality. Ignored sensors stop turning the Setup chip red without hiding the row. Managed from the visor; edit here only if you know the exact path names.",
                     items: { type: "string" },
                     default: [],
                 },
                 tripRecorderEnabled: {
                     type: "boolean",
-                    title: "Record logbook (bitácora) trips",
-                    description: "When ON, the plugin auto-opens a trip on every leg away from moored, records position + wind + speed samples, and closes on return with a summary + KPI. Files stay on the Pi 5. Managed from Setup → Logbook.",
+                    title: "Record logbook (bitácora) trips",
+                    description: "When ON, the plugin auto-opens a trip on every leg away from moored, records position + wind + speed samples, and closes on return with a summary + KPI. Files stay on the Pi 5. Managed from Setup → Logbook.",
                     default: true,
                 },
                 maneuverTraceEnabled: {
                     type: "boolean",
                     title: "Trace user maneuvers vs. pypilot response (sea-trial forensics)",
-                    description: "When ON, every maneuver-related action from the visor (aproado / empopado / tack / mode change / target PUT / engage) is logged along with pypilot state before and 300 ms after the command, so we can audit whether pypilot actually obeyed. Files land under <plugin data dir>/maneuver-trace/*.jsonl. Off by default — enable only during a sea trial.",
+                    description: "When ON, every maneuver-related action from the visor (aproado / empopado / tack / mode change / target PUT / engage) is logged along with pypilot state before and 300 ms after the command, so we can audit whether pypilot actually obeyed. Files land under <plugin data dir>/maneuver-trace/*.jsonl. Off by default — enable only during a sea trial.",
                     default: false,
                 },
                 // Rev299 (2026-09-24): per-rule alarm enable list. Managed from
-                // the visor's Setup → Alarms card. NO schema default here on
+                // the visor's Setup → Alarms card. NO schema default here on
                 // purpose: an existing install that upgraded from a pre-Rev299
                 // plugin must not have its (implicit) alarms silenced by a
                 // schema default of []. The visor is responsible for prompting
@@ -407,7 +407,7 @@ module.exports = function (app) {
                 alarmsEnabled: {
                     type: "array",
                     title: "Alarms enabled (rule ids)",
-                    description: "List of alarm rule ids that are active. When absent the plugin keeps its legacy behaviour (every rule on). Managed from the visor's Setup → Alarms card; edit here only if you know the rule ids (heading-deviation, servo-overcurrent, low-voltage, etc.).",
+                    description: "List of alarm rule ids that are active. When absent the plugin keeps its legacy behaviour (every rule on). Managed from the visor's Setup → Alarms card; edit here only if you know the rule ids (heading-deviation, servo-overcurrent, low-voltage, etc.).",
                     items: { type: "string" },
                 },
                 // Rev167 (Carlos): the Smart Pilot toggles (auto-profile, gust
@@ -438,8 +438,8 @@ module.exports = function (app) {
             });
             // Rev271 (audit R05): propagate the allowWrites gate to the
             // single write path in PypilotClient.set(). Every route to a
-            // pypilot write — provider interface, PUT handlers on action
-            // paths, momentary switches — goes through set(), so this
+            // pypilot write — provider interface, PUT handlers on action
+            // paths, momentary switches — goes through set(), so this
             // check covers them all.
             client.setAllowWrites(props.allowWrites !== false);
             client.on("connect", () => {
@@ -511,7 +511,7 @@ module.exports = function (app) {
                     if (changedField)
                         pushAutopilotUpdate(changedField);
                 }
-                // Rev299 (I2): watch ap.tack.state for the tacking → none
+                // Rev299 (I2): watch ap.tack.state for the tacking → none
                 // transition, which is the moment the sailor needs the
                 // catch-up nudge. `ap.tack.direction` is watched separately
                 // so we already know which side we tacked onto by the time
@@ -1275,7 +1275,7 @@ module.exports = function (app) {
             // reject embedded webapps served over Signal K. Routing tiles
             // through this backend lets us send the User-Agent that OSM's
             // policy requires (self-identifying app string) and centralises
-            // caching. Zero external dependencies — Node 18+ has fetch built
+            // caching. Zero external dependencies — Node 18+ has fetch built
             // in, browser handles per-tile caching via Cache-Control.
             router.get("/tiles/:z/:x/:y.png", async (req, res) => {
                 const z = parseInt(req.params.z, 10);
@@ -1300,7 +1300,7 @@ module.exports = function (app) {
                     }
                     const buf = Buffer.from(await upstream.arrayBuffer());
                     res.setHeader("Content-Type", "image/png");
-                    // Cache 24 h in the browser — OSM tiles are effectively
+                    // Cache 24 h in the browser — OSM tiles are effectively
                     // immutable within a day; a hard reload still fetches fresh.
                     res.setHeader("Cache-Control", "public, max-age=86400, immutable");
                     res.send(buf);
@@ -1627,7 +1627,7 @@ module.exports = function (app) {
                 //   PUT /raw { name: "ap.tack.direction", value: "port|starboard" }
                 //   PUT /raw { name: "ap.tack.state",     value: "begin" }
                 // Rev378-380 put the virtual-tack FSM behind the POST /ap/tack/:dir
-                // endpoint, but the frontend never calls it — the raw path bypassed
+                // endpoint, but the frontend never calls it — the raw path bypassed
                 // our provider.tack() entirely (seen in trace-20261001-161853 QA
                 // 2026-10-01 16:49: ap.tack.state=begin forwarded to pypilot
                 // unchanged, mode stayed "wind" throughout, no virtual-tack).
@@ -1635,14 +1635,14 @@ module.exports = function (app) {
                 // active, re-route it through the FSM and DO NOT forward the raw
                 // write to pypilot (otherwise pypilot runs its own buggy wind
                 // tack in parallel). The direction write that typically precedes
-                // "begin" by a few ms is harmless — pypilot stores it but our
+                // "begin" by a few ms is harmless — pypilot stores it but our
                 // FSM derives direction from its own argument.
                 // Rev382 (Carlos, 2026-10-01): memorise the direction the
                 // frontend sends in the ap.tack.direction raw write so the
                 // state=begin that follows ~100ms later uses the FRESH value.
                 // Rev381 QA on Tunatunes read ap.tack.direction from the
                 // pypilot values cache which still carried the previous tack's
-                // "starboard" when the user had just pressed "port" — the
+                // "starboard" when the user had just pressed "port" — the
                 // intercept launched the wrong direction (16:58:42 journal).
                 if (name === "ap.tack.direction" && (value === "port" || value === "starboard")) {
                     app._pypilotNewuiLastTackDirection = {
@@ -1663,7 +1663,7 @@ module.exports = function (app) {
                             ? dirCache
                             : (dirRaw === "port" || dirRaw === "starboard" ? dirRaw : "starboard");
                         // eslint-disable-next-line no-console
-                        console.log(`[/raw tack intercept] mode=${modeStr} dir=${dir} (cache=${dirCache ?? "n/a"} cache=${dirRaw ?? "n/a"}) → virtual-tack`);
+                        console.log(`[/raw tack intercept] mode=${modeStr} dir=${dir} (cache=${dirCache ?? "n/a"} cache=${dirRaw ?? "n/a"}) → virtual-tack`);
                         apProvider
                             .toProviderInterface()
                             .tack(dir, apProvider.deviceId)
@@ -1897,7 +1897,7 @@ module.exports = function (app) {
                 // Rev416 (Carlos sea trial 2026-10-04, obs #12): accept an
                 // optional `finalAngleRad` so the visor can trim the destination
                 // AWA (closer-hauled or bear-away) at tack-time. Values outside
-                // (-π, π] or non-finite are discarded by startVirtualTack.
+                // (-π, π] or non-finite are discarded by startVirtualTack.
                 const rawFinalAngle = req.body?.finalAngleRad;
                 const finalAngleRadOverride = typeof rawFinalAngle === "number" && Number.isFinite(rawFinalAngle)
                     ? rawFinalAngle : null;
@@ -2567,7 +2567,7 @@ module.exports = function (app) {
             });
             // Rev291 (F1): APB target preview. Reads the current SK course
             // data via app.getCourse and returns what the AP *would* steer to
-            // if the plugin drove NAV directly. Diagnostic only — pypilot's
+            // if the plugin drove NAV directly. Diagnostic only — pypilot's
             // own nav mode is still the one commanding the pilot.
             router.get("/nav/apb-preview", async (_req, res) => {
                 try {
@@ -2603,7 +2603,7 @@ module.exports = function (app) {
                 }
             });
             // Rev296 (Carlos, navigating - bug D): audit trail of profile
-            // switches. Answers "who cargó default?" — shows the last N
+            // switches. Answers "who cargó default?" — shows the last N
             // changes with source attribution (user / auto-profile /
             // gust-heavy / doctor / external / etc).
             router.get("/profile-change-log", (_req, res) => {
@@ -2997,9 +2997,9 @@ module.exports = function (app) {
             // flaky 4G link. This endpoint accepts {events: [{kind, payload,
             // visorRev}, ...]} and records them in order. Malformed events
             // are skipped silently (never block the batch). The visor side
-            // (public/app.js _mtLog + _mtFlush) encola localmente y envía
-            // cada 2 s en vez de uno a uno — ~20x menos HTTP overhead en
-            // navegación con muchos deltas.
+            // (public/app.js _mtLog + _mtFlush) encola localmente y envía
+            // cada 2 s en vez de uno a uno — ~20x menos HTTP overhead en
+            // navegación con muchos deltas.
             router.post("/maneuver-trace/events", (req, res) => {
                 if (!maneuverTrace)
                     return res.status(503).json({ error: "not initialised" });
@@ -3650,12 +3650,12 @@ module.exports = function (app) {
             rollFfTwaGateDeg: clampNumber(options.rollFfTwaGateDeg, 30, 179, 90),
             // Rev298 (H4): leeway adjustment. Practical range 9..12 for a
             // keelboat. 0 = disabled (nothing published). Upper cap 30 keeps
-            // a stray fat-finger config from producing a 40° leeway.
+            // a stray fat-finger config from producing a 40° leeway.
             leewayAdjustment: clampNumber(options.leewayAdjustment, 0, 30, 0),
             // Rev299 (H2): failsafe boat speed in knots. Clamped 0..15;
             // 0 = disabled. Practical range 3..8 kn for a cruising keelboat.
             failSafeBspKn: clampNumber(options.failSafeBspKn, 0, 15, 0),
-            // Rev299 (I2): tack catch-up. Peak 0..15° (0 disables); tau
+            // Rev299 (I2): tack catch-up. Peak 0..15° (0 disables); tau
             // 1..15 s. Defaults keep the module inert; sailor opts in
             // after sea trial validation.
             tackCatchupDeg: clampNumber(options.tackCatchupDeg, 0, 15, 0),
@@ -3665,7 +3665,7 @@ module.exports = function (app) {
             profileAdvisorRmsLowDeg: clampNumber(options.profileAdvisorRmsLowDeg, 0.1, 5, 1),
             profileAdvisorSustainSec: clampNumber(options.profileAdvisorSustainSec, 15, 600, 60),
             // Rev289 (B5): sanitise the persisted metadata against the current
-            // schema — a plugin downgrade could otherwise leave stray rows
+            // schema — a plugin downgrade could otherwise leave stray rows
             // with unknown conditions.
             profileMetadata: (0, profile_metadata_1.loadMetadata)(options.profileMetadata),
             // Rev290 (E2/E3): threshold overrides, clamped to sane ranges.
@@ -3674,8 +3674,8 @@ module.exports = function (app) {
             alarmServoMotorTempC: clampNumber(options.alarmServoMotorTempC, 40, 90, 70),
             alarmPypilotDiscSec: clampNumber(options.alarmPypilotDiscSec, 3, 300, 15),
             // Rev299 (I1): attitude envelope thresholds. Ranges kept wide
-            // enough for coastal cruisers (25°) and racers who purposely
-            // put the rail under (60°). Defaults are conservative.
+            // enough for coastal cruisers (25°) and racers who purposely
+            // put the rail under (60°). Defaults are conservative.
             alarmAttitudeHeelDeg: clampNumber(options.alarmAttitudeHeelDeg, 15, 70, 45),
             alarmAttitudePitchDeg: clampNumber(options.alarmAttitudePitchDeg, 10, 45, 25),
             // Rev299: per-rule enable list. Preserve undefined so legacy
@@ -3707,14 +3707,14 @@ module.exports = function (app) {
         if (!apProvider)
             return;
         // Rev387 (Carlos, 2026-10-01, GPT/Gemini consult): REVERTED Rev386
-        // masking. Both LLMs flagged it as a bus pollution bug — other SK
+        // masking. Both LLMs flagged it as a bus pollution bug — other SK
         // clients (KIP, derived-data, etc.) subscribe to steering.autopilot.*
         // and need the TRUTH about mode/target, not a convenient visor-side
         // lie. Masking belongs in the VISOR's presentation layer, consuming
         // the dedicated `steering.autopilot.virtualTack` path we now publish
         // alongside the canonical deltas. Rev388 frontend will do the
         // display-side override.
-        // Rev394: 'virtualTack' is a VT-only notification — skip the
+        // Rev394: 'virtualTack' is a VT-only notification — skip the
         // autopilotUpdate call and the canonical delta block below, only
         // emit the virtualTack path itself (which may now carry `null`
         // telling the visor to drop its shield).
@@ -3745,8 +3745,8 @@ module.exports = function (app) {
         // publishes ONLY state/engaged/availableActions. This kills the
         // race where a parallel /engage + /target pair produced two deltas
         // each carrying both fields; whichever landed second would clobber
-        // the sibling with a stale copy (the "DIA jumps to 115° then back
-        // to 75°" Carlos reported on Rev82). "all" is still used by the
+        // the sibling with a stale copy (the "DIA jumps to 115° then back
+        // to 75°" Carlos reported on Rev82). "all" is still used by the
         // 30-s keep-alive and startup push.
         const values = [];
         if (fields === "all" || fields === "engaged") {
@@ -3787,7 +3787,7 @@ module.exports = function (app) {
                 // Rev410 (fix T.1-A, Carlos 2026-10-04): prefer the authoritative
                 // `remainingDeg` written by _runVirtualTack (distance to the FINAL
                 // compass target, computed against the REAL heading after each
-                // step reached — accounts for Rev407 recompute). Fallback to the
+                // step reached — accounts for Rev407 recompute). Fallback to the
                 // old per-step stale calculation only when the FSM has not written
                 // one yet (very first snapshot, pre-publishState race).
                 const remainingDeg = (typeof vt.remainingDeg === "number")
@@ -3808,7 +3808,7 @@ module.exports = function (app) {
                         direction: vt.direction,
                         windMode: vt.windMode,
                         // Rev394: backend-computed kind so visor doesn't misdetect
-                        // from |AWA| vs 90°. "tack" = rotation crosses bow; "jibe"
+                        // from |AWA| vs 90°. "tack" = rotation crosses bow; "jibe"
                         // = crosses stern.
                         maneuverKind: g?.maneuverKind ?? null,
                         finalWindTargetRad: g?.angleNewRad ?? null,
@@ -3892,7 +3892,7 @@ module.exports = function (app) {
             return;
         // Rev328 (Carlos, 2026-09-28): master switch. When the sailor
         // disables the logbook from Setup we neither open new trips nor
-        // record samples into an open one — but we DO close any trip that
+        // record samples into an open one — but we DO close any trip that
         // was already recording so its summary lands cleanly on disk.
         if (props.tripRecorderEnabled === false) {
             if (tripRecorder.isRecording()) {
@@ -3908,7 +3908,7 @@ module.exports = function (app) {
         // Transitions.
         // Rev280 (audit T10): also open a trip when the plugin boots
         // with the vessel already off "moored". Previously the "start"
-        // path only fired on a moored → sailing transition, so restarting
+        // path only fired on a moored → sailing transition, so restarting
         // Signal K mid-outing missed the entire session and stored zero
         // trips. Treat lastNavState==null (boot) the same as coming from
         // "moored" for the purpose of opening a new trip.
@@ -4223,7 +4223,7 @@ module.exports = function (app) {
                 }
                 const currentProfile = client.getValues()["profile"];
                 if (currentProfile !== targetProfile) {
-                    _autoProfileLastReason = `TWS avg ${avgKn.toFixed(1)} kn → band ${targetBand} → profile ${targetProfile}`;
+                    _autoProfileLastReason = `TWS avg ${avgKn.toFixed(1)} kn → band ${targetBand} → profile ${targetProfile}`;
                     try {
                         profileChangeLog.markPlannedWrite(targetProfile, "auto-profile", _autoProfileLastReason);
                         client.set("profile", targetProfile);
@@ -4495,7 +4495,7 @@ module.exports = function (app) {
         };
     }
     // Rev322 (Carlos, 2026-09-27): snapshot pypilot + SK state for the
-    // maneuver trace log. Cheaper than collectSample() — no historian
+    // maneuver trace log. Cheaper than collectSample() — no historian
     // aggregations, no derivations. Read once, fill the flat object,
     // return. Called on every trace event pre + 300 ms later.
     function _captureManeuverContext() {
@@ -4547,9 +4547,9 @@ module.exports = function (app) {
     // with state / method / message so any downstream SK client honours
     // them out of the box.
     // Rev299: apply the per-rule enable list to the AlarmEngine.
-    // undefined = legacy → every rule keeps its defaultEnabled (=true
+    // undefined = legacy → every rule keeps its defaultEnabled (=true
     // for pre-Rev299 installs, so Tunatunes carries on unchanged).
-    // Array = user-selected → only listed ids are enabled, everything
+    // Array = user-selected → only listed ids are enabled, everything
     // else is silenced. Empty array = fresh install = ALL silent.
     // Rev299 (I2): watch pypilot values that describe the tack state
     // machine so we can trigger the catch-up module on completion.
@@ -4628,10 +4628,10 @@ module.exports = function (app) {
             // Rev311 (Carlos sea trial 2026-09-25): flag para las reglas
             // de heading-error / drift / unable-to-steer. Durante una
             // virada nativa de pypilot (ap.tack.state != "none") el error
-            // instantáneo es enorme por diseño; silenciamos esas alarmas.
-            // `_tackStatePrev` lo mantiene observeTackTransition() al día en
-            // el mismo callback de client.on("value"), así que siempre
-            // refleja el último valor conocido de pypilot.
+            // instantáneo es enorme por diseño; silenciamos esas alarmas.
+            // `_tackStatePrev` lo mantiene observeTackTransition() al día en
+            // el mismo callback de client.on("value"), así que siempre
+            // refleja el último valor conocido de pypilot.
             maneuverInProgress: !!(_tackStatePrev && _tackStatePrev !== "none"),
             // Rev323 (Carlos, 2026-09-27): flag pseudo-modo del visor.
             maneuverPseudoActive: _maneuverPseudoActive,
@@ -4834,7 +4834,7 @@ module.exports = function (app) {
     }
     // Rev286 (B2): drive the profile advisor from the sampler tick.
     // Publishes a SK notification only on state transitions to avoid
-    // flooding — the same idempotency the AlarmEngine uses.
+    // flooding — the same idempotency the AlarmEngine uses.
     function tickProfileAdvisor(nowMs) {
         if (!profileAdvisor || !kpis || !props.profileAdvisorEnabled)
             return;
@@ -4848,7 +4848,7 @@ module.exports = function (app) {
             publishProfileAdvisorNotification(ev);
         }
         else {
-            // No new event — but if we previously published an advisory and
+            // No new event — but if we previously published an advisory and
             // the cooldown has expired, revert the notification to normal so
             // downstream SK clients stop showing the banner.
             const st = profileAdvisor.status(nowMs);
@@ -4931,7 +4931,7 @@ module.exports = function (app) {
                 const meta = [
                     { path: "steering.autopilot.pypilot.tuning.rollFf.gain", value: { description: "Roll FF gain (unitless, output = -gain*rollHp). 0 = off." } },
                     { path: "steering.autopilot.pypilot.tuning.rollFf.rollHpRad", value: { units: "rad", description: "High-pass filtered roll (dynamic component only)." } },
-                    { path: "steering.autopilot.pypilot.tuning.rollFf.deltaRad", value: { units: "rad", description: "Feed-forward heading delta this tick. Computed even when disabled — but only non-zero when gain>0, AP engaged, and |TWA| above the gate." } },
+                    { path: "steering.autopilot.pypilot.tuning.rollFf.deltaRad", value: { units: "rad", description: "Feed-forward heading delta this tick. Computed even when disabled — but only non-zero when gain>0, AP engaged, and |TWA| above the gate." } },
                     { path: "steering.autopilot.pypilot.tuning.rollFf.appliedToAp", value: { description: "False until a later Rev flips the enable switch after sea trial." } },
                 ];
                 app.handleMessage(PLUGIN_ID, {
@@ -4949,7 +4949,7 @@ module.exports = function (app) {
             app.debug(`[roll-ff] publish failed: ${e?.message || e}`);
         }
     }
-    // Rev298 (H4) → Rev300: publish the leeway estimate on our own
+    // Rev298 (H4) → Rev300: publish the leeway estimate on our own
     // namespace instead of the canonical `performance.leeway`. Two
     // reasons: (1) signalk-derived-data already publishes on that
     // canonical path with a different formula and the sailor was
@@ -4966,7 +4966,7 @@ module.exports = function (app) {
                 const meta = [
                     { path: LEEWAY_PATH, value: {
                             units: "rad",
-                            description: "Estimated leeway (side-slip) angle from the plugin's own estimator. Formula: drift_deg = adj * heel_deg / bsp_kn^2 with adj set by the sailor (Setup → Smart Pilot → Sensor helpers). Positive = drifting to port. Published only when leewayAdjustment > 0. Kept off the canonical performance.leeway path so it does not overwrite signalk-derived-data.",
+                            description: "Estimated leeway (side-slip) angle from the plugin's own estimator. Formula: drift_deg = adj * heel_deg / bsp_kn^2 with adj set by the sailor (Setup → Smart Pilot → Sensor helpers). Positive = drifting to port. Published only when leewayAdjustment > 0. Kept off the canonical performance.leeway path so it does not overwrite signalk-derived-data.",
                         } },
                 ];
                 app.handleMessage(PLUGIN_ID, {
@@ -4990,7 +4990,7 @@ module.exports = function (app) {
     }
     // Rev299 (I2): publish the current tack catch-up delta as a
     // diagnostic path so downstream SK clients can watch the decay.
-    // NOT applied to the pilot yet — the sea trial gates that switch.
+    // NOT applied to the pilot yet — the sea trial gates that switch.
     let tackCatchupMetaSent = false;
     function publishTackCatchupPaths() {
         if (!tackCatchup)
@@ -5344,7 +5344,7 @@ module.exports = function (app) {
                 // refusal. The pre-check at the top of this handler covers
                 // `!client.connected`, but `client.set` can still refuse on
                 // zombie socket (no pong > 5s), coreOffline flag or
-                // writes-disabled config — all of which pre-Rev411 landed as
+                // writes-disabled config — all of which pre-Rev411 landed as
                 // "COMPLETED 200" to KIP/SK clients.
                 if (!client.set(name, value)) {
                     return { state: "COMPLETED", statusCode: 503, message: "pypilot write refused" };
@@ -6011,7 +6011,7 @@ module.exports = function (app) {
                 // resident for subscribers; re-emitting the CURRENT
                 // apProvider.data value periodically produced double-pushes
                 // that raced with the real change delta and got collapsed by
-                // minPeriod:500 on the visor's subscription — the sailor saw
+                // minPeriod:500 on the visor's subscription — the sailor saw
                 // engaged=false held for ~10 s after an external engage.
                 // The KIP-specific momentary + mode radio + engage switch
                 // values still ship (KIP UI needs them refreshed to stay in

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "signalk-pypilot-newui",
-  "version": "3.1.3",
+  "version": "3.2.0",
   "description": "Signal K friendly UI for the pypilot autopilot, with first-class Signal K paths and Simple Switches so any downstream Signal K client can drive or monitor the pilot. Touch-first mobile visor + full pypilot data (servo, gains, calibration, tack, warnings). Coexists with pypilot-autopilot-provider.",
   "main": "./dist/index.js",
   "scripts": {