diff --git a/docs/navigation-audit.md b/docs/navigation-audit.md index c34d8647..aa748a3f 100644 --- a/docs/navigation-audit.md +++ b/docs/navigation-audit.md @@ -1,11 +1,19 @@ # Frontend Navigation Audit — the "opaque click" anti-pattern -> **Status, 2026-07-31.** H1 (jobs), H2 (dashboards) and H4's groundwork are **done** — rows are real -> ``s and selection lives in `?selected=` / `useParams`. **H3 is void: the Projects feature was -> deleted**, so every reference to `Projects/`, `SELECTED_PROJECT` and `/projects/:id` below is historical -> and should be read as an example of the pattern, not as work to do. **H4 (chat detail reading -> `sessionId` from `useParams` rather than the `chat:selected-session` channel) is the one item still -> open.** The rules in this file are current and authoritative; the findings table is a snapshot. +> **Status, 2026-08-07. Phases 1–4 are closed except the two chat items.** Every finding H1–H5 and M1–M10 +> is done, void, or decided-and-recorded, and so is every Phase 4 line. What remains, and only this: +> **H4** (chat detail reading `sessionId` from `useParams` instead of the `chat:selected-session` channel) +> and the **"New Chat"** button that depends on it. Both are inside the chat nucleus, which has its own +> owner — they are not stalled here, they are somebody else's to land. +> +> Three findings turned out to be wrong on inspection and are marked as such rather than quietly dropped: +> **H3** is void (Projects was deleted, so every `Projects/`, `SELECTED_PROJECT` and `/projects/:id` +> reference below is historical), **M7**'s "every caller inherits the opaque click" was false (it had no +> callers, and the component is deleted), and the **Preview slug** item names an app that does not exist. +> Two components — `Combobox` and `BackButton` — were deleted rather than fixed, having never been used. +> +> The rules in this file are current and authoritative; the findings table is a snapshot. +> **Nothing below has had a runtime click-through.** Every "Needs runtime test" note is still true. **Date:** 2026-07-30 · **Origin:** written as exploration before any of the routing work was done. @@ -125,9 +133,17 @@ is **one design decision** that cascades across many files: costs no extra request. En route: the row's Focus and Close buttons were nested *inside* the row `