signalk-pypilot-newui 3.4.0 → 3.5.0

package/CHANGELOG.md

@@ -1,5 +1,69 @@
 # Changelog
 
+## 3.5.0 — 2026-10-19 — Fixes from two sea trials, logbook while sailing
+
+Rolls up Rev509 → Rev516 on top of 3.4.0. Minor bump: live logbook,
+heading in the compass tack summary, trace pruning. No breaking changes.
+
+Rev515 comes from the sail of 2026-10-19: it reverts two changes that
+made things worse (Rev510 compass-until-6°, Rev512 head-to-wind in
+compass) and fixes what the sail showed. Audit:
+`docs/_audit_sail_2026-10-19.md`.
+
+### Changed
+- **Maneuver traces keep the newest 60 files** (pruned when a new trace
+  opens). 201 files / 888 MB had piled up on the Tunatunes.
+- **Stuck pilot recovery only on a real false end stop**: it now needs
+  pypilot's directional overcurrent fault. 0 A is not evidence (the
+  TinyPilot reads 0 A while driving, `DRIVER_TIMEOUT`), it never acts
+  during pypilot's own tack or for 20 s after, and a target change of
+  more than 10° restarts its clock. On 2026-10-19 it cycled the AP three
+  times by mistake, twice in the middle of a tack from the remote.
+- **Gybe confirmation only right after a cancel**: the long-way gybe
+  question (> 180°) now appears only within 20 s of cancelling a tack -
+  the 2026-10-18 accident - not on deliberate gybes. Yes / No buttons
+  are translated.
+
+### Added
+- **Logbook while sailing**: the trip being recorded appears in the
+  logbook with an "IN PROGRESS" badge, live statistics (distance, time,
+  speeds, wind, tacks, energy...) and the track on the chart up to now.
+- **Tack summary in compass mode** shows the mean heading before / after
+  instead of the apparent wind angle.
+- **Confirmation before a gybe the long way** (see above).
+
+### Fixed
+- **Cancelling a virtual tack left a "ghost" target**: pypilot processes
+  the mode change in its own loop and overwrote our "stay here" target
+  with one recomputed from the compass error (2026-10-19: 56° -> 3.9°,
+  bow into the wind). The cancel now waits for the mode change, writes
+  the target, checks it and rewrites it if pypilot moved it.
+- **Remote tack HUD turning the wrong way**: pypilot overwrites
+  `ap.tack.direction` with the wind side ~0.5 s after the remote starts a
+  tack; the HUD now keeps the direction it had at the start. It also stays
+  open until the boat is within 10° of the new target (max 15 s), not only
+  until pypilot's 50 % threshold.
+- **Tack countdown**: shows the distance from the wind arrow to the target
+  diamond along the turn (what the rose shows), no "ajustando" text.
+- **Two screens showing different modes after a tack**: the mode now
+  comes from the backend (plain mode label, no "TACKING ->" prefix).
+- **Previous manoeuvre's summary popping up during the next one**.
+- **Settings tab**: dragging a slider no longer darkens the whole tab
+  (only the other rows), and the darkening clears if the touch is
+  interrupted. A value the sailor writes stays on screen until the
+  backend confirms it (or reports a different one).
+- **`ap.tack.direction` noise in standby**: published as `none` once
+  while the AP is off (pypilot flips it ~40 times a minute there).
+- **Gust ghost arrow almost never visible**: the fade started in the
+  same frame the ghost was created, so the browser skipped it.
+
+### Reverted
+- Rev510 "virtual tack holds compass until 6° of the final heading": the
+  apparent wind at the final heading is not the mirror, so wind mode then
+  swung back 40-50° (S-turn); handover is at 20° again.
+- Rev512 "head to wind holds in compass": head to wind is always in
+  apparent wind, as before.
+
 ## 3.4.0 — 2026-10-18 — The sailor always wins, and the pilot gets unstuck
 
 Rolls up Rev506 → Rev508 on top of 3.3.0, from the strong-wind sea

package/README.md

@@ -149,10 +149,11 @@ up automatically.
 17. **Auto-disengage on lost authority** — RMS heading error > 30°
     with servo duty > 90% sustained 10 s. **Stuck pilot recovery**
     (opt-in): with no rudder sensor, pypilot treats an overcurrent as
-    the end of rudder travel and stops driving that way; when the error
-    keeps growing and the servo is not pushing, the plugin turns the AP
-    off and on (same mode and target) to clear it, at most 3 times per
-    minute.
+    the end of rudder travel and stops driving that way; when pypilot
+    reports that directional overcurrent fault and the error keeps
+    growing, the plugin turns the AP off and on (same mode and target)
+    to clear it, at most 3 times per minute. Never during pypilot's own
+    tack.
 18. **Pre-departure Autopilot Check** — a single READY / CAVEATS /
     DO-NOT-ENGAGE verdict aggregating every sensor, servo, voltage
     and link signal.
@@ -200,6 +201,8 @@ up automatically.
     split, min voltage, AP-engaged %), OpenSeaMap trace coloured by
     SOG with markers on peak gust, peak SOG, min voltage and every
     detected tack. Share as portrait PNG report card or as plain text.
+    The trip in progress can be opened while sailing (live statistics
+    and the track so far).
 27. **Chart tab** — 9 live Trip Stats cards, history with
     30 s / 2 min / 10 min windows and hover-freeze, plus a Servo
     Health card.

package/dist/autopilot-provider.js

@@ -26,6 +26,7 @@ const TARGET_PENDING_MS = 2500;
 // value the user had just replaced. 0.1° covers float32 → double
 // rounding on the wire without accidentally consuming a stale echo.
 const TARGET_ECHO_TOL_RAD = 0.1 * DEG_TO_RAD;
+const LONG_JIBE_CONFIRM_AFTER_CANCEL_MS = 20000;
 // Rev272 (audit R07): bounded-cost normalisation. The previous while
 // loop could spin forever on pathological inputs (e.g. |a-b| >= 1e15
 // where 2π is below the mantissa's precision, so subtracting a full
@@ -113,6 +114,8 @@ class AutopilotProvider {
         /** Rev506: recovery cycles (steer-recovery) toggle the AP off/on on
          *  purpose; that must NOT annul the tack the cycle is trying to save. */
         this._recoveryCycleUntil = 0;
+        /** Rev515: last cancel of a running virtual tack (any source). */
+        this._lastVtCancelTs = 0;
         if (opts?.allowDodge)
             this.allowDodge = true;
         this.onDataChanged = opts?.onDataChanged;
@@ -766,6 +769,24 @@ class AutopilotProvider {
             && Number.isFinite(opts.finalAngleRadOverride)
             && Math.abs(opts.finalAngleRadOverride) <= Math.PI
             ? opts.finalAngleRadOverride : null;
+        // Rev511: same geometry the FSM will compute; refuse a long-way gybe
+        // unless the sailor confirmed it. Rev515 (Carlos 2026-10-19: 7 asks
+        // in a sail of deliberate gybes): only right after a cancel, which is
+        // the 2026-10-18 18:11:44 accident (second tap 3 s after cancelling).
+        if (!opts.confirmLongJibe && Date.now() - this._lastVtCancelTs < LONG_JIBE_CONFIRM_AFTER_CANCEL_MS) {
+            const g = (0, virtual_tack_1.computeTackGeometry)({
+                angleStartRad: awaRad,
+                hStartRad: headingRad,
+                direction: opts.direction,
+                angleNewRadOverride: finalAngleOverride ?? undefined,
+            });
+            if ((0, virtual_tack_1.needsLongJibeConfirm)(g)) {
+                const err = new Error(`virtual-tack: gybe the long way (${(g.deltaHRad * RAD_TO_DEG).toFixed(0)} deg) needs confirmation`);
+                err.code = "confirm_long_jibe";
+                err.deltaDeg = g.deltaHRad * RAD_TO_DEG;
+                throw err;
+            }
+        }
         this._runVirtualTack(opts.direction, windMode, headingRad, awaRad, finalAngleOverride).catch((e) => {
             // eslint-disable-next-line no-console
             console.log(`[startVirtualTack] driver threw: ${e?.message || e}`);
@@ -1302,13 +1323,32 @@ class AutopilotProvider {
                 }
                 catch { /* noop */ }
                 if (awaRad !== null) {
-                    this._vtInternalWrite = true;
-                    try {
-                        await this.setTarget(awaRad);
-                    }
-                    catch { /* best-effort */ }
-                    finally {
-                        this._vtInternalWrite = false;
+                    // Rev515 (sail 2026-10-19 19:08:24, 19:15:44, 19:17:10): pypilot
+                    // processes the mode change in its own loop and "keeps the
+                    // heading" by recomputing heading_command from the compass error
+                    // - AFTER our target had landed, so it overwrote it (56 -> 3.9,
+                    // -57 -> 4.8: bow into the wind, "target fantasma"). Wait for
+                    // the mode echo plus a few pypilot loops, write, check, rewrite.
+                    await this._waitValue("ap.mode", windMode, 1500);
+                    await sleep(400);
+                    for (let attempt = 0; attempt < 2; attempt++) {
+                        this._vtInternalWrite = true;
+                        try {
+                            await this.setTarget(awaRad);
+                        }
+                        catch { /* best-effort */ }
+                        finally {
+                            this._vtInternalWrite = false;
+                        }
+                        await sleep(1200);
+                        const cmdDeg = Number((this.client.getValues?.() || {})["ap.heading_command"]);
+                        if (!Number.isFinite(cmdDeg))
+                            break;
+                        const diff = Math.abs(((cmdDeg - awaRad * RAD_TO_DEG) % 360 + 540) % 360 - 180);
+                        if (diff <= 2)
+                            break;
+                        // eslint-disable-next-line no-console
+                        console.log(`[virtual-tack ${vtId}] cancel: pypilot moved the target to ${cmdDeg.toFixed(1)}deg, rewriting ${(awaRad * RAD_TO_DEG).toFixed(1)}deg`);
                     }
                 }
             }
@@ -1361,6 +1401,7 @@ class AutopilotProvider {
             return;
         // eslint-disable-next-line no-console
         console.log(`[virtual-tack] external cancel at phase=${this._virtualTack.phase} (${reason})`);
+        this._lastVtCancelTs = Date.now();
         this._virtualTack.phase = "cancelling";
         this._virtualTack.outcomeReason = reason;
         this.notifyChanged("all");
@@ -1382,6 +1423,18 @@ class AutopilotProvider {
         this.notifyChanged("virtualTack");
         return true;
     }
+    /** Rev515: wait until pypilot reports key === expected (no cancel check;
+     *  used by the cancel path itself). */
+    async _waitValue(key, expected, timeoutMs) {
+        const t0 = Date.now();
+        while (Date.now() - t0 < timeoutMs) {
+            const v = this.client.getValues?.() || {};
+            if (v[key] === expected)
+                return true;
+            await new Promise((r) => setTimeout(r, 50));
+        }
+        return false;
+    }
     /** Rev506: true while a virtual tack is running (not terminal). */
     isVirtualTackActive() {
         const vt = this._virtualTack;

package/dist/index.js

@@ -32,7 +32,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 = "Rev508";
+const PLUGIN_REVISION = "Rev516";
 // 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
@@ -523,7 +523,12 @@ module.exports = function (app) {
                 publishCatalogDerived(catalog);
             });
             client.on("value", (name, value) => {
-                publishValue(name, value);
+                // Rev513: ap.tack.direction is meaningless in standby and pypilot
+                // flips it none <-> port/starboard ~40 times a minute there.
+                if (name === "ap.tack.direction")
+                    _publishTackDirection(value);
+                else
+                    publishValue(name, value);
                 if (apProvider) {
                     // Rev352: receiveValue now returns the field that changed
                     // (engaged | target | all | null) so we push field-scoped
@@ -535,6 +540,14 @@ module.exports = function (app) {
                     if (changedField)
                         pushAutopilotUpdate(changedField);
                 }
+                if (name === "ap.enabled") {
+                    // Rev513: re-sync the direction on engage / disengage (after the
+                    // provider has taken the new engaged state).
+                    try {
+                        _publishTackDirection(client?.getValues?.()["ap.tack.direction"]);
+                    }
+                    catch { /* silent */ }
+                }
                 // 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
@@ -2044,10 +2057,16 @@ module.exports = function (app) {
                         direction: dir,
                         requestId,
                         finalAngleRadOverride,
+                        confirmLongJibe: req.body?.confirmLongJibe === true,
                     });
                     res.json(r);
                 }
                 catch (e) {
+                    // Rev511: 428 (not 409, which the visor retries) so the visor
+                    // asks the sailor before a long-way gybe.
+                    if (e?.code === "confirm_long_jibe") {
+                        return res.status(428).json({ error: e.message, code: e.code, deltaDeg: e.deltaDeg });
+                    }
                     res.status(409).json({ error: e?.message || String(e) });
                 }
             });
@@ -3270,9 +3289,12 @@ module.exports = function (app) {
                     return res.status(503).json({ error: "recorder not initialised" });
                 // Return list + attach summary for each so the visor can render
                 // the card grid with one call.
+                const liveId = tripRecorder.currentTripId();
                 const rows = tripRecorder.list().map((r) => ({
                     ...r,
                     summary: tripRecorder.summary(r.id),
+                    // Rev516: the trip being recorded right now.
+                    live: r.id === liveId,
                 }));
                 res.json({ trips: rows });
             });
@@ -3309,6 +3331,9 @@ module.exports = function (app) {
                 const id = String(req.params.id || "").replace(/[^A-Za-z0-9\-]/g, "");
                 // eslint-disable-next-line @typescript-eslint/no-require-imports
                 const fs = require("fs");
+                // Rev516: trip in progress -> write the pending samples first.
+                if (id === tripRecorder.currentTripId())
+                    tripRecorder.flushNow();
                 const file = tripRecorder.jsonlPath(id);
                 if (!fs.existsSync(file))
                     return res.status(404).json({ error: "not found" });
@@ -3981,6 +4006,11 @@ module.exports = function (app) {
                         stepIndex: vt.stepIndex,
                         totalSteps: g?.intermediatesRad?.length ?? 0,
                         remainingDeg,
+                        // Rev510: whole compass rotation of this tack, sailor's +-1/+-10
+                        // included, so every visor can show "X deg of Y".
+                        totalDeg: (g && typeof g.deltaHRad === "number")
+                            ? Math.abs(g.deltaHRad * RAD2DEG + (typeof vt.adjustRad === "number" ? vt.adjustRad * RAD2DEG : 0))
+                            : null,
                         outcomeReason: vt.outcomeReason ?? null,
                     },
                 });
@@ -4269,6 +4299,7 @@ module.exports = function (app) {
             servoCurrentA: s.servoCurrent,
             servoFlags: typeof pv["servo.flags"] === "string" ? pv["servo.flags"] : null,
             nativeTack: !!(_tackStatePrev && _tackStatePrev !== "none"),
+            cmdDeg: typeof pv["ap.heading_command"] === "number" ? pv["ap.heading_command"] : null,
             recoveryEnabled: !!props.steerRecoveryEnabled && !!props.allowWrites && !!apProvider,
         });
         if (action !== "cycle" || !apProvider || _steerRecoveryInFlight)
@@ -5422,6 +5453,24 @@ module.exports = function (app) {
         _applyWatches(c, catalog);
         _startWatchSweep(c);
     }
+    // Rev513 (audit 2026-10-19): pypilot tacking.py sets ap.tack.direction
+    // to 'none' on every loop while the AP is off, and its wind log sets
+    // it back to port/starboard every 0.25 s when the wind holds on one
+    // side. 2026-10-18: 7551 of 7587 changes were in standby (~40/min);
+    // overnight at the berth 99.9 % of the maneuver-trace lines. In standby
+    // we publish 'none' once; engaged, the real value as before.
+    let _tackDirPublished = undefined;
+    function _publishTackDirection(raw) {
+        const pv = (client && client.connected) ? client.getValues() : {};
+        const engaged = apProvider
+            ? !!apProvider.data.engaged
+            : (pv["ap.enabled"] === true || pv["ap.enabled"] === 1);
+        const v = engaged ? raw : "none";
+        if (v === undefined || v === _tackDirPublished)
+            return;
+        _tackDirPublished = v;
+        publishValue("ap.tack.direction", v);
+    }
     function publishValue(name, value) {
         if (publisher_1.RESERVED_PYPILOT_KEYS.has(name))
             return;

package/dist/maneuver-trace.js

@@ -101,6 +101,38 @@ class ManeuverTraceLog {
         const pad = (n) => String(n).padStart(2, "0");
         const name = `trace-${d.getFullYear()}${pad(d.getMonth() + 1)}${pad(d.getDate())}-${pad(d.getHours())}${pad(d.getMinutes())}${pad(d.getSeconds())}.jsonl`;
         this.currentFile = path.join(this.dir, name);
+        try {
+            this.pruneToCount();
+        }
+        catch { /* silent */ }
+    }
+    /** Rev514 (Carlos 2026-10-19, "limita a 60"): keep only the newest
+     *  `maxCount` trace files (201 files / 888 MB had piled up since
+     *  2026-09-28). Oldest first by name (trace-YYYYMMDD-HHMMSS), never the
+     *  file in use. Same rule as SessionRecorder.pruneToCount (Rev505). */
+    pruneToCount(maxCount = 60) {
+        let deleted = 0;
+        let names = [];
+        try {
+            names = fs.readdirSync(this.dir).filter((f) => f.startsWith("trace-") && f.endsWith(".jsonl"));
+        }
+        catch {
+            return { deleted };
+        }
+        names.sort();
+        const current = this.currentFile ? path.basename(this.currentFile) : null;
+        const closed = names.filter((n) => n !== current);
+        const keepClosed = Math.max(0, maxCount - (current ? 1 : 0));
+        for (const name of closed.slice(0, Math.max(0, closed.length - keepClosed))) {
+            try {
+                fs.unlinkSync(path.join(this.dir, name));
+                deleted += 1;
+            }
+            catch { /* skip */ }
+        }
+        if (deleted > 0)
+            this.log("info", `[maneuver-trace] pruned ${deleted} old traces (keeping the newest ${maxCount})`);
+        return { deleted };
     }
     record(input, pre, getPost) {
         const entry = {

package/dist/steer-watch.js

@@ -11,14 +11,20 @@
 //
 // Pure state machine: fed once per second, no I/O, no timers.
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.SteerWatch = exports.DROPPED_MIN_SOG_MS = exports.DROPPED_ALARM_MS = exports.CYCLE_MIN_GAP_MS = exports.CYCLE_WINDOW_MS = exports.MAX_CYCLES = exports.IDLE_CURRENT_A = exports.ENGAGE_GRACE_MS = exports.CONVERGE_MIN_DROP_DEG = exports.CONVERGE_WINDOW_MS = exports.STUCK_SUSTAIN_MS = exports.STUCK_ERR_DEG = void 0;
+exports.SteerWatch = exports.DROPPED_MIN_SOG_MS = exports.DROPPED_ALARM_MS = exports.CYCLE_MIN_GAP_MS = exports.CYCLE_WINDOW_MS = exports.MAX_CYCLES = exports.TARGET_CHANGE_RESET_DEG = exports.NATIVE_TACK_HOLDOFF_MS = exports.ENGAGE_GRACE_MS = exports.CONVERGE_MIN_DROP_DEG = exports.CONVERGE_WINDOW_MS = exports.STUCK_SUSTAIN_MS = exports.STUCK_ERR_DEG = void 0;
 exports.hasDirectionalFault = hasDirectionalFault;
 exports.STUCK_ERR_DEG = 25;
 exports.STUCK_SUSTAIN_MS = 4000;
 exports.CONVERGE_WINDOW_MS = 4000;
 exports.CONVERGE_MIN_DROP_DEG = 3;
 exports.ENGAGE_GRACE_MS = 5000;
-exports.IDLE_CURRENT_A = 0.3;
+// Rev515 (sail 2026-10-19): no current-based fallback. With the
+// TinyPilot the current reads 0 A while the motor IS driving (servo.flags
+// DRIVER_TIMEOUT), which fired three false cycles (18:53:32 and 18:54:14
+// mid remote tack, 19:23:45 after a big course change). Only pypilot's
+// directional overcurrent fault - the thing a cycle actually clears - counts.
+exports.NATIVE_TACK_HOLDOFF_MS = 20000;
+exports.TARGET_CHANGE_RESET_DEG = 10;
 exports.MAX_CYCLES = 3;
 exports.CYCLE_WINDOW_MS = 60000;
 exports.CYCLE_MIN_GAP_MS = 10000;
@@ -39,6 +45,8 @@ class SteerWatch {
         this.gaveUp = false;
         this.droppedAtMs = null;
         this.dirFault = false;
+        this.lastNativeTackMs = null;
+        this.lastCmdDeg = null;
     }
     tick(i) {
         const now = i.nowMs;
@@ -55,6 +63,26 @@ class SteerWatch {
         if (this.engagedSinceMs == null)
             this.engagedSinceMs = now;
         this.droppedAtMs = null;
+        if (i.nativeTack)
+            this.lastNativeTackMs = now;
+        const inTackHoldoff = this.lastNativeTackMs != null && now - this.lastNativeTackMs < exports.NATIVE_TACK_HOLDOFF_MS;
+        if (typeof i.cmdDeg === "number" && isFinite(i.cmdDeg)) {
+            if (this.lastCmdDeg != null) {
+                const jump = Math.abs(((i.cmdDeg - this.lastCmdDeg) % 360 + 540) % 360 - 180);
+                if (jump > exports.TARGET_CHANGE_RESET_DEG) {
+                    // A new target is a new error baseline (sailor's course change).
+                    this.errHist = [];
+                    this.condSinceMs = null;
+                }
+            }
+            this.lastCmdDeg = i.cmdDeg;
+        }
+        if (inTackHoldoff) {
+            this.errHist = [];
+            this.condSinceMs = null;
+            this.stuck = false;
+            return null;
+        }
         if (typeof i.errDeg !== "number" || !isFinite(i.errDeg)) {
             this.errHist = [];
             this.condSinceMs = null;
@@ -68,8 +96,7 @@ class SteerWatch {
         }
         const graceOk = now - this.engagedSinceMs >= exports.ENGAGE_GRACE_MS;
         const bigErr = absErr > exports.STUCK_ERR_DEG;
-        const notPushing = this.dirFault
-            || (typeof i.servoCurrentA === "number" && i.servoCurrentA < exports.IDLE_CURRENT_A);
+        const notPushing = this.dirFault;
         const notConverging = this.notConverging(now, absErr);
         const cond = graceOk && bigErr && notPushing && notConverging;
         if (!cond) {

package/dist/trip-recorder.js

@@ -323,8 +323,22 @@ class TripRecorder {
             clearInterval(this.flushTimer);
             this.flushTimer = null;
         }
-        // Compute summary.
-        const summary = {
+        const summary = this._buildSummary(endTs);
+        try {
+            if (this.currentSummaryFile)
+                fs.writeFileSync(this.currentSummaryFile, JSON.stringify(summary, null, 2));
+        }
+        catch (e) {
+            this.log("error", `trip summary write failed: ${e?.message || e}`);
+        }
+        this.log("info", `[trip] closed ${this.currentId} (${this.sampleCount} samples, ${summary.distanceNm.toFixed(1)} nm, ${this.tackCount} tacks)`);
+        this._resetActive();
+        return summary;
+    }
+    // Rev516: shared by stop() and liveSummary().
+    _buildSummary(endTs) {
+        const durMs = endTs - this.startTs;
+        return {
             id: this.currentId,
             startTs: this.startTs,
             endTs,
@@ -360,17 +374,18 @@ class TripRecorder {
             revision: this.header?.revision ?? "?",
             sampleCount: this.sampleCount,
         };
-        try {
-            if (this.currentSummaryFile)
-                fs.writeFileSync(this.currentSummaryFile, JSON.stringify(summary, null, 2));
-        }
-        catch (e) {
-            this.log("error", `trip summary write failed: ${e?.message || e}`);
-        }
-        this.log("info", `[trip] closed ${this.currentId} (${this.sampleCount} samples, ${summary.distanceNm.toFixed(1)} nm, ${this.tackCount} tacks)`);
-        this._resetActive();
-        return summary;
     }
+    /** Rev516 (Carlos 2026-10-19: "que lo que se está grabando de bitácora
+     *  se pueda consultar mientras está navegando"): summary of the trip in
+     *  progress from the same running stats stop() uses. */
+    liveSummary() {
+        if (!this.currentFile || this.startTs == null || !this.currentId)
+            return null;
+        return { ...this._buildSummary(Date.now()), live: true };
+    }
+    /** Rev516: write the pending samples so a download of the trip in
+     *  progress carries the track up to now (buffer flushes every 30 s). */
+    flushNow() { this.flush(); }
     flush() {
         if (!this.currentFile || this.buffer.length === 0)
             return;
@@ -504,6 +519,8 @@ class TripRecorder {
         }
     }
     summary(id) {
+        if (this.currentId && id === this.currentId)
+            return this.liveSummary();
         const sumPath = path.join(this.dir, `trip-${id}.summary.json`);
         try {
             return JSON.parse(fs.readFileSync(sumPath, "utf8"));

package/dist/virtual-tack.js

@@ -30,6 +30,7 @@ exports.signedAngleDelta = signedAngleDelta;
 exports.computeTackGeometry = computeTackGeometry;
 exports.fractionateRotation = fractionateRotation;
 exports.makeInitialState = makeInitialState;
+exports.needsLongJibeConfirm = needsLongJibeConfirm;
 exports.adjustedFinalCompassRad = adjustedFinalCompassRad;
 exports.adjustedFinalWindRad = adjustedFinalWindRad;
 exports.accumulateTackAdjust = accumulateTackAdjust;
@@ -252,6 +253,15 @@ function makeInitialState() {
 }
 /** Rev506: largest total nudge accepted during one manoeuvre. */
 exports.MAX_TACK_ADJUST_RAD = 60 * exports.DEG;
+/** Rev511 (Carlos sea trial 2026-10-18 18:11:44): a second tap on the
+ *  same button after a cancelled tack, with the wind now on the other
+ *  bow, asked for a 278 deg gybe from close-hauled in 25 kn. A gybe the
+ *  long way (rotation > 180 deg, only possible from |AWA| < 90) needs an
+ *  explicit confirmation. Normal gybes from a run/broad reach and long
+ *  tacks from downwind are unaffected. */
+function needsLongJibeConfirm(g) {
+    return g.maneuverKind === "jibe" && Math.abs(g.deltaHRad) > Math.PI;
+}
 /** Rev506: final compass heading of the rotation phase after the
  *  sailor's nudges. Bow to starboard = heading increases. */
 function adjustedFinalCompassRad(g, adjustRad) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "signalk-pypilot-newui",
-  "version": "3.4.0",
+  "version": "3.5.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",
   "files": [

package/public/app.js

@@ -462,6 +462,10 @@
       "mode.label.compass":   "COMPASS",
       "mode.label.gps":       "GPS",
       "mode.label.wind":      "APPARENT WIND",
+      "dlg.yes":              "Yes",
+      "dlg.no":               "No",
+      "vt.longJibe.title":    "Gybe the long way?",
+      "vt.longJibe.body":     "The wind is on the other bow: this button turns through the stern ({deg}°). To tack, use the other button.",
       "mode.label.true-wind": "TRUE WIND",
       "mode.label.nav":       "WAYPOINT",
       "mode.label.aproado":   "HEAD TO WIND",
@@ -739,6 +743,8 @@
       "trips.caption.voltMin":  "Min volt",
       "trips.caption.sags":     "sags",
       "trips.status.recording": "LOGBOOK ENTRY IN PROGRESS for",
+      "trips.live":             "IN PROGRESS",
+      "tackStats.hdg":          "HDG",
       "trips.status.navState":  "Nav state",
       "trips.status.noneOpen":  "no logbook entry in progress.",
       "trips.status.disabled":  "Logbook recording is OFF.",
@@ -1395,6 +1401,10 @@
       "mode.label.compass":   "COMPÁS",
       "mode.label.gps":       "GPS",
       "mode.label.wind":      "APARENTE",
+      "dlg.yes":              "Sí",
+      "dlg.no":               "No",
+      "vt.longJibe.title":    "¿Trasluchar por popa?",
+      "vt.longJibe.body":     "El viento está en la otra amura: este botón da la vuelta por popa ({deg}°). Para virar, usa el otro botón.",
       "mode.label.true-wind": "REAL",
       "mode.label.nav":       "RUMBO A WAYPOINT",
       "mode.label.aproado":   "APROADO",
@@ -1678,6 +1688,8 @@
       "trips.caption.voltMin":  "Volt min",
       "trips.caption.sags":     "caidas",
       "trips.status.recording": "BITÁCORA EN CURSO desde hace",
+      "trips.live":             "EN CURSO",
+      "tackStats.hdg":          "RUMBO",
       "trips.status.navState":  "Estado nav",
       "trips.status.noneOpen":  "sin bitácora en curso.",
       "trips.status.disabled":  "Grabación de bitácora OFF.",
@@ -1998,6 +2010,8 @@
       "mode.label.compass":   "KOMPASS",
       "mode.label.gps":       "GPS",
       "mode.label.wind":      "SCHEINBARER WIND",
+      "dlg.yes":              "Ja",
+      "dlg.no":               "Nein",
       "mode.label.true-wind": "WAHRER WIND",
       "mode.label.nav":       "WEGPUNKT",
       "mode.label.aproado":   "AUFSCHIESSEN",
@@ -2197,6 +2211,8 @@
       "mode.label.compass":   "COMPAS",
       "mode.label.gps":       "GPS",
       "mode.label.wind":      "VENT APPARENT",
+      "dlg.yes":              "Oui",
+      "dlg.no":               "Non",
       "mode.label.true-wind": "VENT REEL",
       "mode.label.nav":       "WAYPOINT",
       "mode.label.aproado":   "FACE AU VENT",
@@ -2956,6 +2972,7 @@
             try { renderTargetArrow(); } catch {}
           }
           try { setSelect("#mode-select", state.virtualTack.windMode || state.mode); } catch {}
+          try { _renderModeSelectBtn(); } catch {}
           try { _lastTackDirSeen = state.virtualTack.direction; } catch {}
           try { if (typeof _openExternalTackOverlay === "function") _openExternalTackOverlay(state.virtualTack); } catch (e) { console.warn("[vt] open overlay", e); }
         }
@@ -2984,10 +3001,18 @@
             } else if (typeof state._targetBeforeVt === "number") {
               state.target = state._targetBeforeVt;
             }
-            if (typeof state._modeBeforeVt === "string") {
-              state.mode = state._modeBeforeVt;
-              try { setSelect("#mode-select", state._modeBeforeVt); } catch {}
+            // Rev510: the FSM restores its windMode on completed, cancelled
+            // and failed. _modeBeforeVt could hold "compass" when the
+            // FSM's own compass delta beat the first VT snapshot to this
+            // device - that left one screen on COMPASS and another on WIND.
+            const modeAfter = (typeof state.virtualTack?.windMode === "string" && state.virtualTack.windMode)
+              ? state.virtualTack.windMode
+              : state._modeBeforeVt;
+            if (typeof modeAfter === "string") {
+              state.mode = modeAfter;
+              try { setSelect("#mode-select", modeAfter); } catch {}
             }
+            try { _renderModeSelectBtn(); } catch {}
             // Rev410 (fix U.3, Carlos 2026-10-04): explicit failure toast
             // fed by the backend's `outcomeReason`, replacing the fragile
             // time-since-delta heuristic. Fires on `failed` only — a
@@ -5611,20 +5636,56 @@
     // hudOwner semantics: overlay open + _externalManeuver=false
     // means the local path owns it; we must not intrude.
     if (overlayOpen && !_externalManeuver) return;
+    // Rev515: a new tack while the previous one is still finishing.
+    if (isActive && !wasActive && _externalArrivalTimer) {
+      _closeExternalTackOverlay();
+    }
     if (isActive && !wasActive && !_externalManeuver) {
       _openExternalTackOverlay();
       return;
     }
     if (!isActive && _externalManeuver) {
+      // Rev515 (Carlos 2026-10-19 18:55: "se desactiva el HUD antes de
+      // terminar la virada"): pypilot ends its tack at ap.tack.threshold
+      // (50 %) with ~40 deg still to turn. For pypilot's own tacks keep
+      // the HUD until the boat is within 10 deg of the new target, max
+      // 15 s. Virtual tacks close from their own snapshot as before.
+      if (!state.virtualTack) { _closeExternalWhenArrived(); return; }
       _closeExternalTackOverlay();
     }
   }
+  let _externalArrivalTimer = null;
+  function _closeExternalWhenArrived() {
+    if (_externalArrivalTimer) return;
+    const t0 = Date.now();
+    _externalArrivalTimer = setInterval(() => {
+      const modeStr = String(state.mode || "").toLowerCase();
+      const cur = modeStr.includes("wind")
+        ? (modeStr.includes("true") ? state.windAngleTrue : state.windAngle)
+        : state.heading;
+      let arrived = false;
+      if (typeof cur === "number" && typeof state.target === "number") {
+        let e = (cur - state.target) * RAD2DEG;
+        while (e > 180) e -= 360;
+        while (e < -180) e += 360;
+        arrived = Math.abs(e) < 10;
+      }
+      if (arrived || Date.now() - t0 > 15000 || !_externalManeuver) {
+        clearInterval(_externalArrivalTimer);
+        _externalArrivalTimer = null;
+        if (_externalManeuver) _closeExternalTackOverlay();
+      }
+    }, 500);
+  }
   function _openExternalTackOverlay(vt = null) {
     // Rev358 (Carlos, 2026-09-30, QA Rev355): a new external maneuver
     // arrived — close any prior stats HUD so it does not stay behind
     // the mirror overlay and get overwritten later when this new tack
     // completes.
     try { if (_tackStats.hideAt) _tackStatsHide(); } catch { /* silent */ }
+    // Rev515 (Carlos 2026-10-19 19:19): the previous manoeuvre's post
+    // window (5 s + 10 s) re-showed its summary in the middle of this one.
+    _tackStats.postScheduledFor = 0;
     // Rev395 (Carlos, 2026-10-02, Round-3): prefer the backend snapshot
     // over local heuristics for direction, mode and maneuverKind. The
     // backend knows WHICH side it is driving and WHICH classification
@@ -5733,19 +5794,34 @@
           // `remainingDeg` is no longer the useful metric. Show a short
           // "settling" label so Carlos reads the correct phase.
           const vt = state.virtualTack;
-          const inSettling = vt && (vt.phase === "handover" || vt.phase === "settling");
-          const remDeg = (vt && vt.active && typeof vt.remainingDeg === "number")
-            ? vt.remainingDeg : null;
-          if (inSettling) {
-            el.textContent = t("tack.settling") || "settling";
-          } else if (remDeg !== null) {
+          // Rev515: wind arrow -> diamond (what the rose shows), also while
+          // settling; no "ajustando" text (Carlos: read as a new mode).
+          const inSettling = false;
+          const remDeg = (vt && vt.active)
+            ? (_vtWindRemainingDeg(vt) ?? (typeof vt.remainingDeg === "number" ? vt.remainingDeg : null))
+            : null;
+          if (remDeg !== null) {
             el.textContent = `${Math.round(remDeg)}°`;
           } else {
             el.textContent = "— —";
           }
+          // Rev510: "de Y°" under the number, like the device that started it.
+          const degLine = document.getElementById("tack-countdown-deg");
+          if (degLine) {
+            if (vt && vt.active && typeof vt.totalDeg === "number" && vt.totalDeg > 0 && !inSettling) {
+              degLine.textContent = `de ${Math.round(vt.totalDeg)}°`;
+              degLine.style.display = "";
+            } else {
+              degLine.style.display = "none";
+            }
+          }
         }
         // Re-detect kind if direction/AWA showed up after open.
-        if (_lastTackDirSeen && _tackCountdownDir !== _lastTackDirSeen) {
+        // Rev515 (Carlos 2026-10-19 18:53-18:55, HUD turning the wrong way):
+        // only when the direction was unknown at open. pypilot's wind log
+        // (tacking.py poll) overwrites ap.tack.direction with the wind side
+        // ~0.5 s after the remote's begin; the value at begin is the turn.
+        if (!_tackCountdownDir && _lastTackDirSeen && _lastTackDirSeen !== "none") {
           _tackCountdownDir = _lastTackDirSeen === "port" ? "port" : "starboard";
           try { _refreshOverlayKind(); } catch { /* silent */ }
         }
@@ -5772,6 +5848,7 @@
     }
   }
   function _closeExternalTackOverlay() {
+    if (_externalArrivalTimer) { clearInterval(_externalArrivalTimer); _externalArrivalTimer = null; }
     if (_externalManeuverTimer) { clearInterval(_externalManeuverTimer); _externalManeuverTimer = null; }
     if (_externalManeuverWatchdog) { clearTimeout(_externalManeuverWatchdog); _externalManeuverWatchdog = null; }
     _externalManeuver = false;
@@ -5802,6 +5879,9 @@
     // does not stay behind the new countdown and confuse the sailor
     // when the new maneuver completes and overwrites its content.
     try { if (_tackStats.hideAt) _tackStatsHide(); } catch { /* silent */ }
+    // Rev515 (Carlos 2026-10-19 19:19): the previous manoeuvre's post
+    // window (5 s + 10 s) re-showed its summary in the middle of this one.
+    _tackStats.postScheduledFor = 0;
     _tackCountdownKind = MANEUVER_STYLES[kind] ? kind : "tack";
     _tackDiamondBlink = true;
     renderTargetArrow();
@@ -6114,6 +6194,8 @@
   let _externalManeuverTimer = null;    // pure UI refresh, no logic
   let _externalManeuverWatchdog = null; // 60 s safety close
   function _tackCountdownDegTotal() {
+    const vt = _vtActiveSnapshot();
+    if (vt && typeof vt.totalDeg === "number" && vt.totalDeg > 0) return Math.round(vt.totalDeg);
     if (_tackAccumulatedRotationDeg > 0) return Math.round(_tackAccumulatedRotationDeg);
     const raw = state.pypilotValues?.["ap.tack.angle"];
     if (typeof raw === "number" && raw > 10 && raw < 180) return Math.round(raw);
@@ -6169,7 +6251,41 @@
     const expectSign = isWind ? -hullSign : hullSign;
     _tackRotationDoneDeg = Math.max(0, _tackRotationDoneDeg + deltaDeg * expectSign);
   }
+  // Rev510 (Carlos sea trial 2026-10-18: "te pone los grados, pero no los
+  // que le quedan por llegar"): while a virtual tack runs, the backend's
+  // remainingDeg (real heading vs the FINAL compass target, +-10 included)
+  // is the truth. The local AWA integration drifted with gusts, the boat
+  // slowing down and the sailor's adjustments.
+  function _vtActiveSnapshot() {
+    const vt = state.virtualTack;
+    return (vt && vt.active) ? vt : null;
+  }
+  // Rev515 (Carlos 2026-10-19: "el contador miente … no se corresponde
+  // flecha viento con target y dato"): count what the rose shows - the
+  // wind arrow's distance to the target diamond, along the turn. Near the
+  // target (settling, overshoot) the plain error; no "ajustando" text.
+  function _vtWindRemainingDeg(vt) {
+    if (!vt || typeof vt.finalWindTargetRad !== "number") return null;
+    const useTrue = String(vt.windMode || "").toLowerCase().includes("true");
+    const cur = useTrue ? state.windAngleTrue : state.windAngle;
+    if (typeof cur !== "number") return null;
+    let e = (cur - vt.finalWindTargetRad) * RAD2DEG;
+    while (e > 180) e -= 360;
+    while (e < -180) e += 360;
+    if (Math.abs(e) < 30) return Math.round(Math.abs(e));
+    // Hull to starboard -> wind angle decreases (and vice versa).
+    const s = vt.direction === "starboard" ? 1 : -1;
+    let d = ((cur - vt.finalWindTargetRad) * RAD2DEG * s) % 360;
+    if (d < 0) d += 360;
+    return Math.round(d);
+  }
   function _tackCountdownDegRemaining() {
+    const vt = _vtActiveSnapshot();
+    if (vt) {
+      const w = _vtWindRemainingDeg(vt);
+      if (w != null) return w;
+      if (typeof vt.remainingDeg === "number") return Math.max(0, Math.round(vt.remainingDeg));
+    }
     const total = _tackAccumulatedRotationDeg > 0 ? _tackAccumulatedRotationDeg : null;
     if (total == null) return null;
     return Math.max(0, Math.round(total - _tackRotationDoneDeg));
@@ -6500,6 +6616,8 @@
       awaSigned: typeof state.windAngle === "number" ? state.windAngle : null,
       twaSigned: typeof state.windAngleTrue === "number" ? state.windAngleTrue : null,
       sog: typeof state.sog === "number" ? state.sog * 1.94384 : null,
+      // Rev516: heading for the compass-mode summary.
+      hdg: typeof state.heading === "number" ? state.heading : null,
     });
     // Rev172 (Carlos): the gust-ghost detector piggybacks on this 1 Hz
     // ring buffer - baseline AWS and rolling AWA both come from the
@@ -6531,8 +6649,10 @@
     // Rev225 (Carlos): also keep the SIGN of AWA so the HUD can show
     // "45deg B" or "45deg E" instead of the bare absolute value.
     let awaSignedSum = 0, awaSignedN = 0;
+    let hx = 0, hy = 0, hN = 0;
     for (const s of _tackStats.samples) {
       if (s.ts < fromTs || s.ts > toTs) continue;
+      if (typeof s.hdg === "number") { hx += Math.cos(s.hdg); hy += Math.sin(s.hdg); hN += 1; }
       if (typeof s.tws === "number")     { tws += s.tws;     twsN += 1; }
       if (typeof s.awaAbs === "number")  { awa += s.awaAbs;  awaN += 1; }
       if (typeof s.awaSigned === "number") {
@@ -6546,6 +6666,7 @@
       awa: awaN > 0 ? awa / awaN : null,
       awaSide: awaSignedN > 0 ? (awaSignedSum >= 0 ? "E" : "B") : null,
       sog: sogN > 0 ? sog / sogN : null,
+      hdg: hN > 0 ? ((Math.atan2(hy, hx) * RAD2DEG) + 360) % 360 : null,
     };
   }
   function _tackStatsOnTackCompleted() {
@@ -6877,7 +6998,13 @@
     // transform we could unknowingly inherit later.
     svg.appendChild(ghost);
     _gustGhostActive = { g: ghost, expireAt: Date.now() + GUST_GHOST_FADE_MS + 500 };
-    requestAnimationFrame(() => { ghost.style.opacity = "0"; });
+    // Rev509 (Carlos 2026-10-18, "las rachas no salen casi nunca"): one
+    // rAF is not enough - its callback runs before the browser has ever
+    // styled the new node, so there is no 0.5 starting point and the
+    // transition is skipped: the ghost went invisible at once (headless
+    // Chrome: opacity 0 after 1 s with one rAF, 0.47 with two). The second
+    // rAF starts the fade after the first frame painted it.
+    requestAnimationFrame(() => requestAnimationFrame(() => { ghost.style.opacity = "0"; }));
     setTimeout(() => {
       try { ghost.remove(); } catch { /* silent */ }
       _gustGhostActive = null;
@@ -6996,8 +7123,23 @@
       if (o == null || o.awa == null) return (isPost && pending) ? "…" : "--";
       return `${Math.round(o.awa)}° ${o.awaSide || ""}`.trim();
     };
-    set("tack-stats-awa-before", awaFmt(pre, false));
-    set("tack-stats-awa-after",  awaFmt(post, true));
+    // Rev516 (Carlos 2026-10-19: "cuando estamos en modo compás no debe
+    // hablar del viento aparente sino del rumbo"): compass manoeuvre ->
+    // mean heading before / after.
+    // Frozen when the summary first shows, so a mode change before the
+    // post window lands does not flip the row.
+    if (post == null || _tackStats.compassStats == null) {
+      const modeNow = String(_maneuverModeAtStart || state.mode || "").toLowerCase();
+      _tackStats.compassStats = !modeNow.includes("wind");
+    }
+    const compassStats = _tackStats.compassStats;
+    set("tack-stats-awa-lbl", compassStats ? (t("tackStats.hdg") || "HDG") : "AWA");
+    const hdgFmt = (o, isPost) => {
+      if (o == null || o.hdg == null) return (isPost && pending) ? "…" : "--";
+      return `${String(Math.round(o.hdg) % 360).padStart(3, "0")}°`;
+    };
+    set("tack-stats-awa-before", compassStats ? hdgFmt(pre, false) : awaFmt(pre, false));
+    set("tack-stats-awa-after",  compassStats ? hdgFmt(post, true) : awaFmt(post, true));
     set("tack-stats-sog-before", (pre?.sog == null ? "--" : pre.sog.toFixed(1) + " kn"));
     set("tack-stats-sog-after",  fmt(post?.sog, " kn", 1));
     // Rev224: execution time (from arming-to-executing switch to
@@ -7319,10 +7461,16 @@
       document.body.classList.add("adjusting");
       row.classList.add("active");
     });
-    rng.addEventListener("pointerup", () => {
+    // Rev515: Android turns a drag into a scroll (pointercancel, no
+    // pointerup) and the tab stayed dimmed; clear on every end event.
+    const endAdjust = () => {
       document.body.classList.remove("adjusting");
       row.classList.remove("active");
-    });
+    };
+    rng.addEventListener("pointerup", endAdjust);
+    rng.addEventListener("pointercancel", endAdjust);
+    rng.addEventListener("lostpointercapture", endAdjust);
+    rng.addEventListener("blur", endAdjust);
     rng.addEventListener("input", () => { val.innerHTML = `${Number(rng.value).toFixed(decimals)}${unitHtml}`; });
     rng.addEventListener("change", () => { try { opts.onChange(Number(rng.value)); } catch { /* silent */ } });
     row.querySelectorAll(".nudge-btn").forEach((btn) => {
@@ -7527,7 +7675,13 @@
       if (row.classList.contains("active")) return;
       const rng = row.querySelector("input[type=range]");
       const val = row.querySelector(".value");
-      if (rng && Number(rng.value) !== v) rng.value = String(v);
+      // Half a slider step: the range input snaps 0.7 to 0.6945.
+      const halfStep = rng ? (Number(rng.step) || 0) / 2 : 0;
+      if (rng && Math.abs(Number(rng.value) - v) > Math.max(halfStep, 1e-6 * Math.max(1, Math.abs(v)))) {
+        // Rev515: log every value the refresh changes on screen.
+        try { _mtLog("other", { stage: "visor", via: "tune-refresh-change", name, from: Number(rng.value), to: v }); } catch { /* silent */ }
+        rng.value = String(v);
+      }
       if (val) {
         const unitStr = _gainUnits(name);
         const unitHtml = unitStr
@@ -7777,10 +7931,24 @@
   // Now uses skFetch (adds JWT if we have one), retries with backoff, and
   // opens the login modal after 3 consecutive 401s.
   let _refreshValues401Streak = 0;
-  // Rev507: name -> ts of the sailor's last successful RangeSetting
-  // write (pluginRaw). See refreshPypilotValues.
-  const _pvLocalWriteTs = new Map();
-  const PV_LOCAL_WRITE_HOLD_MS = 5000;
+  // Rev507: name -> the sailor's last successful RangeSetting write
+  // (pluginRaw). Rev515 (Carlos 2026-10-19: "se siguen borrando los
+  // cambios"): a fixed 5 s hold painted the old value back whenever the
+  // backend had not received pypilot's echo yet. Now the written value
+  // stays until the backend reports it (confirmed) or a THIRD value
+  // (someone else changed it), at most PV_LOCAL_WRITE_MAX_MS.
+  const _pvLocalWriteTs = new Map();   // name -> { value, prev, ts }
+  const PV_LOCAL_WRITE_MAX_MS = 5 * 60_000;
+  const _pvSame = (a, b) => typeof a === "number" && typeof b === "number" && Math.abs(a - b) <= 1e-6 * Math.max(1, Math.abs(a));
+  // true = keep the sailor's value, false = take the fetched one.
+  function _pvKeepLocal(k, fetchedV, nowTs) {
+    const w = _pvLocalWriteTs.get(k);
+    if (!w) return false;
+    if (nowTs - w.ts > PV_LOCAL_WRITE_MAX_MS || _pvSame(fetchedV, w.value)) { _pvLocalWriteTs.delete(k); return false; }
+    if (w.prev === undefined || _pvSame(fetchedV, w.prev)) return true;   // stale echo
+    _pvLocalWriteTs.delete(k);                                             // changed elsewhere
+    return false;
+  }
   async function refreshPypilotValues(opts) {
     const replace = !!(opts && opts.replace);
     try {
@@ -7803,17 +7971,13 @@
       // {replace:true} for that case so the sliders reflect the new
       // profile instead of the previous one.
       const fetched = await r.json();
-      // Rev507: a value the sailor wrote < PV_LOCAL_WRITE_HOLD_MS ago
-      // wins over the snapshot (pypilot may not have echoed it yet).
+      // Rev507/515: the sailor's own write wins until confirmed.
       const nowTs = Date.now();
-      const freshLocal = (k) => {
-        const ts = _pvLocalWriteTs.get(k);
-        return ts != null && (nowTs - ts) < PV_LOCAL_WRITE_HOLD_MS;
-      };
+      const freshLocal = (k) => _pvKeepLocal(k, (fetched || {})[k], nowTs);
       if (replace) {
         const prev = state.pypilotValues || {};
         state.pypilotValues = fetched || {};
-        for (const k of _pvLocalWriteTs.keys()) {
+        for (const k of Array.from(_pvLocalWriteTs.keys())) {
           if (freshLocal(k) && prev[k] != null) state.pypilotValues[k] = prev[k];
         }
       } else {
@@ -8091,6 +8255,17 @@
   function _renderModeSelectBtn() {
     const btn = document.getElementById("mode-select-btn");
     if (!btn) return;
+    // Rev510 (Carlos sea trial 2026-10-18, 18:07-18:12: "la tablet decia
+    // un modo y el telefono decia otro"): during a virtual tack pypilot is
+    // in compass on purpose. Every visor shows the wind mode of the tack,
+    // from the backend snapshot (same on every screen). Rev515 (Carlos
+    // 2026-10-19): no "TACKING ->" prefix, it read as a new mode.
+    const vt = state.virtualTack;
+    if (vt && vt.active) {
+      const txt = _modeLabel(vt.windMode || state.mode || "wind");
+      if (btn.textContent !== txt) btn.textContent = txt;
+      return;
+    }
     const m = state.mode || "";
     btn.textContent = m ? _modeLabel(m) : "—";
   }
@@ -9892,8 +10067,11 @@
     if (res.ok && state.catalog?.[name]?.type === "RangeSetting"
         && typeof value === "number" && isFinite(value)) {
       if (!state.pypilotValues) state.pypilotValues = {};
+      const prevV = state.pypilotValues[name];
       state.pypilotValues[name] = value;
-      _pvLocalWriteTs.set(name, Date.now());
+      _pvLocalWriteTs.set(name, { value, prev: prevV, ts: Date.now() });
+      // Rev515: evidence for "se borran los cambios" next time.
+      try { _mtLog("other", { stage: "visor", via: "tune-write", name, value, prev: prevV }); } catch { /* silent */ }
     }
     if (!res.ok) {
       const bodyTxt = await res.text().catch(() => "");
@@ -10630,6 +10808,7 @@
           // (500ms each → ~2s total) swallowing 409s silently, then
           // only flash fail on persistent failure. Other HTTP errors
           // (400/500/network) still fail immediately.
+          let confirmLongJibe = false;
           const vtStart = async () => {
             // Rev400 (Carlos QA Rev399): subo retries 4→10 (~5s total).
             // Rev399 QA mostró casos donde el VT previo tardaba >2s en
@@ -10644,6 +10823,7 @@
               // Clamped to ±170° (deg) to prevent abuse; backend
               // additionally rejects anything outside (-π, π].
               const body = { direction: dir, requestId };
+              if (confirmLongJibe) body.confirmLongJibe = true;
               try {
                 const rawOv = localStorage.getItem("pypilotnewui.tackFinalAngleDeg");
                 const deg = rawOv == null ? NaN : Number(rawOv);
@@ -10661,7 +10841,20 @@
               await new Promise((res) => setTimeout(res, RETRY_DELAY_MS));
             }
           };
-          const r = await vtStart();
+          let r = await vtStart();
+          // Rev511 (Carlos sea trial 2026-10-18 18:11:44): the backend
+          // asks before a gybe the long way (> 180 deg from close-hauled).
+          if (r.status === 428) {
+            const j = await r.json().catch(() => ({}));
+            const deg = Math.round(Math.abs(Number(j.deltaDeg) || 0));
+            const ok = await _pmConfirm(
+              t("vt.longJibe.title") || "¿Trasluchar por popa?",
+              (t("vt.longJibe.body") || "El viento está en la otra amura: este botón da la vuelta por popa ({deg}°). Para virar, usa el otro botón.").replace("{deg}", String(deg)),
+            );
+            if (!ok) return;
+            confirmLongJibe = true;
+            r = await vtStart();
+          }
           if (!r.ok) {
             const err = await r.text().catch(() => String(r.status));
             console.warn(`[vt] start failed: ${err}`);
@@ -12470,10 +12663,15 @@
     // Rev330 (Carlos, 2026-09-28): tipografía más grande + colores del
     // sistema (--fg, --fg-dim, --accent). Nombre en accent cuando
     // existe, fecha siempre en fg-dim.
+    // Rev516 (Carlos 2026-10-19: consult the logbook while sailing): the
+    // trip being recorded shows live stats and this badge.
+    const liveBadge = trip.live
+      ? `<span style="display:inline-block;margin-left:8px;padding:1px 7px;border-radius:4px;font-size:10px;font-weight:700;letter-spacing:0.5px;color:#ff9500;background:rgba(255,149,0,0.15);border:1px solid #ff9500;vertical-align:middle">● ${t("trips.live")}</span>`
+      : "";
     const titleHtml = nameEsc
-      ? `<div style="font-size:16px;font-weight:600;color:var(--accent,#4dd4ff);line-height:1.2">${nameEsc}</div>
+      ? `<div style="font-size:16px;font-weight:600;color:var(--accent,#4dd4ff);line-height:1.2">${nameEsc}${liveBadge}</div>
          <div style="font-size:13px;color:var(--fg-dim);margin-top:2px">${dateStr}</div>`
-      : `<div style="font-size:15px;font-weight:600;color:var(--fg);line-height:1.2">${dateStr}</div>`;
+      : `<div style="font-size:15px;font-weight:600;color:var(--fg);line-height:1.2">${dateStr}${liveBadge}</div>`;
     const kpiHtml = s
       ? `<div style="font-size:13px;color:var(--fg);margin-top:6px"><b>${distNm} nm</b> · ${durMin} min · SOG ${avg} ${sog} kn</div>
          <div style="font-size:12px;color:var(--fg-dim);margin-top:2px">TWS ${avg} ${tws} kn · ${tacks} ${bordos} · peak SOG ${s.sogMaxKn?.toFixed(1) ?? "?"} kn · peak TWS ${s.twsMaxKn?.toFixed(1) ?? "?"} kn</div>`
@@ -13473,6 +13671,11 @@
     // set, falling back to the timestamp. Inline pencil button reopens
     // the rename prompt.
     _tripsRenderTitle(id, s.name || null, dateStr);
+    if (s.live) {
+      // Rev516: trip still being recorded - stats and track up to now.
+      const titleEl = document.getElementById("trip-detail-title");
+      if (titleEl) titleEl.insertAdjacentHTML("beforeend", `<span style="margin-left:8px;padding:1px 7px;border-radius:4px;font-size:10px;font-weight:700;color:#ff9500;background:rgba(255,149,0,0.15);border:1px solid #ff9500">● ${t("trips.live")}</span>`);
+    }
     // Rev332 (Carlos, 2026-09-28): "quita los colorines" — todas las
     // KPI cards con el mismo color neutro var(--fg). El tint arg se
     // conserva por si un día queremos volver a resaltar algo, pero por

package/public/index.html

@@ -27,7 +27,7 @@
        style.css never shows the rose unstyled on a white page. The rose
        is hidden until style.css (loaded below) makes it visible again. -->
   <style>html,body{background:#0a1220;color:#e6edf3}#wind-rose{visibility:hidden}</style>
-  <link rel="stylesheet" href="style.css?v=Rev508" />
+  <link rel="stylesheet" href="style.css?v=Rev516" />
 </head>
 <body class="theme-dark">
   <!-- Rev490 (GPU audit 2026-10-15): flag the Raspberry Pi's own screen
@@ -286,7 +286,7 @@
                     fill="#ffffff" font-size="12" font-weight="800"
                     font-family="ui-monospace, monospace">--</text>
               <line x1="-72" y1="-14" x2="72" y2="-14" stroke="#4a5867" stroke-width="0.6"/>
-              <text x="-72" y="0" fill="#8b98a5" font-size="10" font-weight="700"
+              <text x="-72" y="0" id="tack-stats-awa-lbl" fill="#8b98a5" font-size="10" font-weight="700"
                     font-family="ui-monospace, monospace">AWA</text>
               <text x="-28" y="0" id="tack-stats-awa-before" text-anchor="middle"
                     fill="#ffffff" font-size="12" font-weight="800"
@@ -2651,6 +2651,6 @@
     </div>
   </div>
 
-  <script src="app.js?v=Rev508"></script>
+  <script src="app.js?v=Rev516"></script>
 </body>
 </html>

package/public/style.css

@@ -1773,8 +1773,10 @@ hr.tune-separator {
 
 /* Translucent effect while adjusting - see-through so user can watch heading
    converge to target in the Control tab underneath. */
-body.adjusting main #tab-tune { opacity: 0.32; transition: opacity 0.1s ease; }
-body.adjusting main #tab-tune .gain-row.active { opacity: 1; }
+/* Rev515 (Carlos 2026-10-19: "se pone negro, no puedo ver lo que hago"):
+   dim the OTHER rows only. Dimming the whole tab capped the active row
+   at 32 % too (a child cannot be more opaque than its parent). */
+body.adjusting main #tab-tune .gain-row:not(.active) { opacity: 0.32; transition: opacity 0.1s ease; }
 
 .section-title {
   color: var(--fg-dim);