Add notes to finished tracks, single Nav voice, fix speed coloring

- TrackDetailSheet: + Add note button opens a dialog with text, camera,
  and gallery. Entry is saved with timestampMs=track.endMs so it lands
  inside the track's time window and appears automatically in the event
  log. logEventAdapter.update() refreshes the list after save.

- TripReportGenerator: removed NarrativeStyle enum and 4-style branch.
  generateNarrative() now emits a concise PASSAGE header (date, time
  range, duration), stats line (nm / avg kt / max kt), optional conditions
  (seas, temp), and a timestamped deck log. Fragment and layout updated to
  remove the ChipGroup style picker.

- TrackColors: replaced Expression.get("color") — which silently fails
  for lineColor on LineLayer in MapLibre Android 11.x — with
  Expression.step(Expression.get("speed"), ...) that maps the numeric
  "speed" property directly to color literals. "color" string property
  removed from speedSegments() features.

https://claude.ai/code/session_01YUbuZNDAoLea4cf9UGQ9qn

1 files changed, 9 insertions, 5 deletions

diff --git a/.agent/worklog.md b/.agent/worklog.md
index cfae053..a31dbd0 100644
--- a/.agent/worklog.md
+++ b/.agent/worklog.md
@@ -2,15 +2,19 @@
 
 ## Backlog (fix later)
 
-- **Add notes/photos to a finished track** — UI to attach logbook entries to a past track by timestamp/geo proximity. Must flow into the trip report (currently only live-session entries are scoped).
+- **Tack detection is buggy, especially at low speed** — False positives at low SOG (GPS heading noise looks like tacks). Need minimum-SOG gate (e.g. skip detection when SOG < 1.5 kt) and review the stability window / MIN_DELTA thresholds against real tracks.
 
-- **Trip report voice** — The current narrative styles (Professional / Adventurous / Journal / Pirate) are generic and not right for Nav. Nav is an opinionated sailing app; the report should have a single voice that reflects that — concise, knowledgeable, seamanlike. Rip out the style picker and write one good template.
+## Current Task Goal
+Logbook + Trip Report Improvements — COMPLETE (2026-05-26)
 
-- **Tack detection is buggy, especially at low speed** — False positives at low SOG (GPS heading noise looks like tacks). Need minimum-SOG gate (e.g. skip detection when SOG < 1.5 kt) and review the stability window / MIN_DELTA thresholds against real tracks.
+Delivered:
+- **Add notes/photos to finished track** — `+ Add note` button in `TrackDetailSheet` opens a dialog with text entry, camera, and gallery. Notes are saved with `timestampMs = track.endMs` so they fall within the track window and appear in `buildLogEvents()` automatically. The map is re-built via `logEventAdapter.update()` after save.
+- **Trip report single Nav voice** — Removed `NarrativeStyle` enum and the 4-style ChipGroup. `generateNarrative(summary)` now emits a concise `PASSAGE` header with date/time range, stats, conditions, and a timestamped deck log. `fragment_trip_report.xml` no longer has the style picker.
+- **Speed coloring fixed** — Replaced the broken `Expression.get("color")` (reads a string feature property, silently fails for `lineColor` on `LineLayer` in MapLibre Android 11.x) with `Expression.step(Expression.get("speed"), ...)` which maps the numeric `"speed"` property directly to color literals. `speedSegments()` no longer writes the unused `"color"` string property.
 
-- **Speed coloring on the map doesn't work** — `Expression.get("color")` on a FeatureCollection of 2-point LineStrings fails silently in MapLibre Android 11.x. The focused-track layer in MapHandler still uses this broken approach. Needs investigation: either find a working data-driven expression syntax for this MapLibre version, or pre-segment the GeoJSON differently.
+---
 
-## Current Task Goal
+## Previous Task Goal
 Instrument Sheet Visual Redesign — IN PROGRESS (2026-04-06)
 
 **Plan:** `docs/superpowers/plans/2026-04-04-instrument-sheet-visual-redesign.md`