pi-weave 0.1.11 → 0.1.13

package/src/web/client/shell/keys.ts

@@ -3,8 +3,8 @@
  *
  * `keys.model.ts` decides what a keystroke means; this attaches one listener
  * and performs the answer. Four lines of it are the listener and the rest is
- * dispatch, which is deliberate — the same split `viewport.ts` uses, and for
- * the same reason: a subscription written inline in a `useEffect` is one no
+ * dispatch, which is deliberate — a subscription written inline in a
+ * `useEffect` is one no
  * test can reach.
  *
  * ## One listener, on the document, at the bubble phase

package/src/web/client/shell/shell.model.ts

@@ -1,9 +1,9 @@
 /**
  * Everything the shell *decides*, as pure functions (weave-workspace §1.2).
  *
- * The header's status summary, the connection indicator's wording, the
- * per-column empty-state copy and the status bar's text all live here rather
- * than inside the components that render them. That is not a stylistic
+ * The header's status summary, the per-column empty-state copy and the status
+ * bar's text all live here rather than inside the components that render them.
+ * That is not a stylistic
  * preference: §10 forbids adding a DOM test environment, so a conditional
  * inside a `.tsx` is a conditional that can never be covered, and §14 lists
  * "coverage gate blocks the UI work" as a live risk whose stated mitigation
@@ -15,9 +15,11 @@
  */
 
 import type { GraphPayload, WireNodeKind, WireStalenessState } from "../../shared/wire";
-import type { ConnectionState } from "../state";
-import type { ColumnId, DividerId, ResolvedColumn } from "./layout.model";
-import { DIVIDERS } from "./layout.model";
+/** The three fixed workspace surfaces. */
+export type ColumnId = "tree" | "note" | "graph";
+
+/** Columns in keyboard and visual order. */
+export const COLUMNS: readonly ColumnId[] = ["tree", "note", "graph"];
 
 // --- the header summary --------------------------------------------------------
 
@@ -77,93 +79,22 @@ export function summaryParts(summary: HeaderSummary): readonly string[] {
   return [`vault:${summary.notes}`, `repo:${repoLabel(summary.repo)}`, `${summary.nodes} nodes`];
 }
 
-// --- the connection indicator ----------------------------------------------------
-
-/**
- * How a connection state is presented.
- *
- * `tone` is a class suffix rather than a colour, so the palette stays in the
- * stylesheet where §5.2's nonce'd CSS can own it and this module stays free
- * of presentation constants it cannot test the appearance of.
- */
-export interface ConnectionView {
-  readonly label: string;
-  readonly tone: "ok" | "warn" | "bad";
-  /** The `title=` tooltip. Says what the user should expect to happen next. */
-  readonly hint: string;
-}
-
-const CONNECTION_VIEWS: Readonly<Record<ConnectionState, ConnectionView>> = {
-  live: { label: "live", tone: "ok", hint: "watching the vault and the repository for changes" },
-  reconnecting: {
-    label: "reconnecting",
-    tone: "warn",
-    // Naming the recovery matters: §6 has the client refetch everything on
-    // reopen, so the user's screen will catch up on its own and they should
-    // not go looking for a reload button.
-    hint: "the event stream dropped — retrying, and everything refetches when it returns",
-  },
-  offline: {
-    label: "offline",
-    tone: "bad",
-    hint: "the workspace server is gone — the header's refresh control retries",
-  },
-};
-
-/** Present a connection state. Total over the three states of §1.3. */
-export function connectionView(state: ConnectionState): ConnectionView {
-  return CONNECTION_VIEWS[state];
-}
-
 // --- empty states ------------------------------------------------------------------
 
-/**
- * A column's title, and the placeholder shown while its content is someone
- * else's phase.
- *
- * P1's deliverable for the three columns was an *honest* empty state naming
- * the phase that would fill it — more useful than a spinner implying a load,
- * and far more useful than a blank pane that reads as a bug.
- *
- * **As of P3 no column uses one.** All three are built, so `Columns.tsx`
- * renders their real surfaces and each has its *own* empty states —
- * `treeEmptyMessage`, `noteEmptyMessage` and `graphEmptyMessage`, which
- * distinguish "loading" from "filtered to nothing" from "genuinely empty" in a
- * way one static sentence cannot.
- *
- * The table stays because {@link EmptyStateCopy.title} is still every column's
- * heading and its `aria-label`. {@link EmptyStateCopy.body} and `phase` are
- * now dead for all three, and are kept accurate rather than deleted for one
- * reason: `tests/web/client-shell.test.ts` asserts the `phase` values against
- * §11, so a column whose phase silently disagreed with the doc would be the
- * first sign that this table had stopped tracking reality. `EmptyState.tsx`
- * itself now has no caller, and is deleted rather than kept warm — an unused
- * component is a thing the next reader has to work out is unused.
- */
+/** The title used by a column heading and its `aria-label`. */
 export interface EmptyStateCopy {
   readonly title: string;
-  readonly body: string;
-  /** The phase that fills this column in, e.g. `"P2"`. */
-  readonly phase: string;
 }
 
 const EMPTY_STATES: Readonly<Record<ColumnId, EmptyStateCopy>> = {
   tree: {
     title: "Tree",
-    // Built in P2; body and phase are no longer rendered. See the header.
-    body: "The vault and repository outline, expandable, filterable, with provenance markers.",
-    phase: "P2",
   },
   note: {
     title: "Note",
-    body: "Select anything to read it here — rendered Markdown, front matter, tags and wikilinks.",
-    phase: "P2",
   },
   graph: {
     title: "Graph",
-    // Built in P3; body and phase are no longer rendered. See the header.
-    body: "The knowledge graph, laid out and navigable, with the selection's neighbourhood highlighted.",
-    phase: "P3",
   },
 };
 
@@ -172,18 +103,9 @@ export function emptyStateFor(column: ColumnId): EmptyStateCopy {
   return EMPTY_STATES[column];
 }
 
-/**
- * The context rail's title.
- *
- * Built in P2.5, so `body`/`phase` are no longer rendered either — the rail's
- * real empty states are `RAIL_EMPTY` in `context/context.model.ts`, which
- * separates "loading" from "nothing selected" from "this node is isolated".
- * `title` is still the rail's heading and `aria-label`.
- */
+/** The context rail's heading and `aria-label`. */
 export const CONTEXT_EMPTY: EmptyStateCopy = {
   title: "Context",
-  body: "Links, backlinks and mentions for whatever is selected.",
-  phase: "P2",
 };
 
 // --- the status bar -------------------------------------------------------------------
@@ -200,7 +122,6 @@ export const CONTEXT_EMPTY: EmptyStateCopy = {
 export interface StatusBarModel {
   readonly cwd: string;
   readonly selection: string;
-  readonly connection: ConnectionView;
   /** `null` before the first successful graph fetch. */
   readonly stamp: string | null;
 }
@@ -212,13 +133,11 @@ export const NO_VALUE = "—";
 export function statusBarModel(
   cwd: string,
   selectedId: string | null,
-  connection: ConnectionState,
   stamp: string | null,
 ): StatusBarModel {
   return {
     cwd: cwd === "" ? NO_VALUE : cwd,
     selection: selectedId ?? "nothing selected",
-    connection: connectionView(connection),
     stamp,
   };
 }
@@ -239,48 +158,6 @@ export function shortStamp(stamp: string | null): string {
   return match?.[1] ?? stamp;
 }
 
-// --- pairing columns with dividers ---------------------------------------------------
-
-/**
- * One rendered column, plus the divider that follows it (if any).
- *
- * `divider` is `null` for the last column: there are three columns and two
- * dividers, and the trailing edge of the grid is the window, not a handle.
- */
-export interface ColumnSlot {
-  readonly column: ResolvedColumn;
-  readonly divider: DividerId | null;
-}
-
-/**
- * Interleave resolved columns with the dividers between them.
- *
- * This exists as a model function rather than an `index + 1 < length` check
- * inside the JSX for two reasons. The first is coverage: that check is a
- * branch, and a branch in a `.tsx` is a branch no test can reach (§10). The
- * second is that the naive version is *wrong at a breakpoint* — at `"medium"`
- * only tree and note render, so the divider after `note` must not appear even
- * though `DIVIDERS` contains one. Deriving the pairing from the columns
- * actually being rendered, rather than from the static divider list, makes
- * that impossible to get wrong.
- */
-export function columnSlots(resolved: readonly ResolvedColumn[]): readonly ColumnSlot[] {
-  return resolved.map((column, index) => ({
-    column,
-    // A divider is *named by the column to its left* (`layout.model.ts`'s
-    // `dividerPair`), so the id is the column's own — no index arithmetic into
-    // `DIVIDERS`, which would need an unreachable `?? null` to satisfy
-    // `noUncheckedIndexedAccess` and would leave a branch no test can cover.
-    // It exists only where a next column follows to resize against.
-    divider: index < resolved.length - 1 && isDivider(column.id) ? column.id : null,
-  }));
-}
-
-/** Whether a column has a divider named after it — i.e. is not the last one. */
-function isDivider(column: ColumnId): column is DividerId {
-  return (DIVIDERS as readonly ColumnId[]).includes(column);
-}
-
 // --- the refresh button ------------------------------------------------------------
 
 /**

package/src/web/client/shell/theme.model.ts

@@ -45,7 +45,7 @@ export type ThemeChoice = "system" | "light" | "dark";
 /** The choices in cycle order. */
 export const THEME_CHOICES: readonly ThemeChoice[] = ["system", "light", "dark"];
 
-/** The `localStorage` key. Namespaced and versioned, like the layout's. */
+/** The `localStorage` key. Namespaced and versioned. */
 export const THEME_STORAGE_KEY = "pi-weave.theme.v1";
 
 /** The slice of `Storage` this module needs. */
@@ -144,4 +144,4 @@ export function themeButton(choice: ThemeChoice): ThemeButtonView {
     glyph: THEME_GLYPHS[choice],
     hint: `Colour theme: ${choice} — click for ${cycleTheme(choice)}`,
   };
-}
+}

package/src/web/client/shell/theme.ts

@@ -6,8 +6,8 @@
  *
  * `page.ts` emits a nonce'd `<style>` block, but it is deliberately tiny —
  * the custom-property palette and enough body rules that the first paint has
- * a background colour before `app.js` parses. It knows nothing about a grid,
- * a divider or a status bar, and it is owned by the server tier, which this
+ * a background colour before `app.js` parses. It knows nothing about the
+ * status bar, and it is owned by the server tier, which this
  * work may not edit. There is also no `/app.css` route. So the shell's own
  * rules have to travel in the bundle and be installed at runtime.
  *
@@ -19,8 +19,8 @@
  *  1. **A literal `style="…"` attribute in markup is blocked.** That is what
  *     `'unsafe-inline'` governs for styles, and it is absent.
  *  2. **CSSOM writes are not.** `el.style.setProperty(…)` has no CSP hook at
- *     all, which is why `cssvars.ts` applies column widths that way and why
- *     dynamic layout works here. Confirmed against Preact's own behaviour:
+ *     all, which is why dynamic note preview positions use CSSOM. Confirmed
+ *     against Preact's own behaviour:
  *     `preact/src/diff/props.js` handles a `style` prop via
  *     `dom.style.cssText` or `style.setProperty`, never `setAttribute`.
  *  3. **A script-created `<style>` element needs the nonce.** Inserting a
@@ -161,7 +161,7 @@ export const THEME_CSS = `
        ui       11.5  status bar, the note's meta row, small controls
        row      12    list rows, text inputs, inline code
        base     13    body copy, h3-h6, preview-card text
-       body     13.5  the note column's prose and its mono editor
+       body     13.5  the note column's prose
        subhead  14    palette input, note-body h2, header icon buttons
        title    15    note-body h1
        display  20    the note's page title
@@ -244,30 +244,21 @@ body{font-size:var(--weave-px-base)}
 }
 .weave-theme:hover{color:var(--weave-fg);background:var(--weave-line)}
 
-/* connection indicator -------------------------------------------------- */
-.weave-conn{display:inline-flex;align-items:center;gap:5px;font-size:var(--weave-px-ui);white-space:nowrap}
-.weave-conn-dot{font-size:var(--weave-px-prov);line-height:1}
-/* Reconnecting breathes instead of sitting there: the one state whose whole
-   message is "hold on, I am still working" is the one that should keep
-   saying it. The offline dot stays still — its message needs a decision,
-   not patience. The global reduced-motion kill switch restores the static
-   dot, the same way it does every animation in this sheet. */
-.weave-conn-ok .weave-conn-dot,.weave-conn-ok{color:var(--weave-ok)}
-.weave-conn-warn .weave-conn-dot,.weave-conn-warn{color:var(--weave-warn)}
-.weave-conn-warn .weave-conn-dot{animation:weave-conn-pulse 1.6s ease-in-out infinite}
-.weave-conn-bad .weave-conn-dot,.weave-conn-bad{color:var(--weave-bad)}
-@keyframes weave-conn-pulse{0%,100%{opacity:1}50%{opacity:.35}}
-
 /* the grid -------------------------------------------------------------- */
-/* Widths arrive as custom properties from cssvars.ts — the CSSOM path. The
-   fallbacks keep the layout sane for the first frame and if a write is ever
-   missed. */
-.weave-grid{display:grid;min-height:0;overflow:hidden;background:var(--weave-line)}
-.weave-grid[data-columns="3"]{
-  grid-template-columns:var(--weave-col-tree,22%) 1px var(--weave-col-note,46%) 1px var(--weave-col-graph,32%);
-}
-.weave-grid[data-columns="2"]{grid-template-columns:var(--weave-col-tree,32%) 1px var(--weave-col-note,68%)}
-.weave-grid[data-columns="1"]{grid-template-columns:1fr}
+/* One boring grid. CSS owns the breakpoints; no saved widths, divider state
+   or viewport listener can make the shell disagree with the browser. */
+.weave-grid{
+  display:grid;min-height:0;overflow:hidden;background:var(--weave-line);
+  grid-template-columns:minmax(180px,22fr) minmax(320px,46fr) minmax(260px,32fr);
+}
+@media (max-width:1099px){
+  .weave-grid{grid-template-columns:minmax(180px,32fr) minmax(320px,68fr)}
+  .weave-col-graph{display:none}
+}
+@media (max-width:799px){
+  .weave-grid{grid-template-columns:1fr}
+  .weave-col-tree{display:none}
+}
 
 .weave-col{
   display:flex;flex-direction:column;min-width:0;min-height:0;overflow:hidden;
@@ -286,18 +277,8 @@ body{font-size:var(--weave-px-base)}
    inside the same region instead of squeezing the graph. */
 .weave-col-graph{display:grid;grid-template-rows:auto minmax(0,1fr) minmax(96px,40%);background:var(--weave-bg)}
 
-/* dividers -------------------------------------------------------------- */
-.weave-divider{
-  background:var(--weave-line);cursor:col-resize;position:relative;
-  touch-action:none;
-}
-/* A 1 px target is unhittable, so the hit area is widened with a pseudo
-   element rather than by making the visible rule thicker. */
-.weave-divider::after{content:"";position:absolute;inset:0 -3px;z-index:1}
-.weave-divider:hover,.weave-divider:focus-visible{background:var(--weave-accent);outline:none}
-
 /* tree column ----------------------------------------------------------- */
-/* --weave-depth is written per row by \`depthVar\`, through the same CSSOM
+/* --weave-depth is written per row by \`depthVar\`, through CSSOM
    path as the column widths, and multiplied by a step this sheet owns — so
    the tree's density stays a CSS decision. */
 .weave-tree{display:flex;flex-direction:column;min-height:0;flex:1}
@@ -332,26 +313,17 @@ body{font-size:var(--weave-px-base)}
    hover from reading as a stray grey rectangle — the row answers the pointer
    in both channels at once. */
 .weave-row:hover .weave-label{color:var(--weave-fg)}
-/* The session fold (tree.model.ts's isMuted): machine-written memory that
-   accrues by the dozens with near-duplicate titles, dropped one notch so the
-   rows a human wrote keep the foreground. --weave-dim holds ≥ 4.5 on both
-   grounds, so this is quiet, not illegible — and the two states below always
-   restore full weight, which is why the model never emits both classes on
-   one row. */
-.weave-row-muted .weave-label{color:var(--weave-dim)}
-.weave-row-muted:hover .weave-label{color:var(--weave-fg)}
 /* The selection has one voice. On the tinted ground every quieter token still
    fails contrast — even --weave-dim lands under 4.5 — so a selected row's
    kind, provenance and meta children join its label in --weave-fg. The
    provenance glyph shape carries the distinction the colour swap drops;
    elsewhere the hues are unaffected. The palette's hit rows need the same
    remap: same selected ground, same failure. The label is restated last
-   because .weave-row-muted outranks the inheritance the row-on rule relies
-   on. */
+   so the selected row is always readable. */
 .weave-row-on .weave-twisty,.weave-row-on .weave-kind,.weave-row-on .weave-prov,
 .weave-row-on .weave-meta,.weave-row-on .weave-label{color:var(--weave-fg)}
 .weave-hit-on .weave-hit-badge,.weave-hit-on .weave-hit-detail{color:var(--weave-fg)}
-/* A newly-arrived node (a file or note added since the last update, §6):
+/* A newly-arrived node (a file or note added since the last update, ):
    one short highlight that fades while the label settles from bold back to
    normal. The class is computed from the frame diff in workspace.ts and
    expires with it, so collapsing and re-expanding later does not replay the
@@ -378,7 +350,7 @@ body{font-size:var(--weave-px-base)}
 
 /* icons ------------------------------------------------------------------
    The sprite's only CSS: one colour (the glyph inherits the row's, so a
-   muted row's icon recedes with it and a selected row's brightens with it)
+   quiet row's icon recedes with it and a selected row's brightens with it)
    and the twisty's rotation, which is the *same* 16px chevron the rail's
    group headings use — right-pointing closed, rotated 90° open. The
    rotation is not in the motion block's transition list, on purpose: that
@@ -394,8 +366,8 @@ body{font-size:var(--weave-px-base)}
    so it lies on --weave-page — one step off the desk and the canvas grounds.
    The 2px rule flush to its left edge is the spine: it takes the note's
    provenance colour (see the --weave-spine map), so the document's origin is
-   readable peripherally, before any glyph is. Head, body and editor share the
-   --weave-note-gutter so toggling the editor does not reflow the measure. */
+   readable peripherally, before any glyph is. Head and body share the
+   --weave-note-gutter so the page has one consistent measure. */
 .weave-note{display:flex;flex-direction:column;min-height:0;flex:1;overflow:auto;background:var(--weave-page)}
 .weave-note-human{--weave-spine:var(--weave-ok)}
 .weave-note-agent{--weave-spine:var(--weave-accent)}
@@ -407,9 +379,7 @@ body{font-size:var(--weave-px-base)}
    alternative, a title that scrolls away, is how a reader ends up annotating
    the wrong file. It carries the page ground rather than a fill or a shadow,
    so the reveal reads as the page continuing under it and not as a bar
-   arriving; the hairline is the only seam. The editor bar is the *next*
-   sibling in the column flow, so it can never slide underneath — no stacking
-   contest exists between them. The head's z-index is only what keeps it above
+   arriving; the hairline is the only seam. The head's z-index is only what keeps it above
    the prose (and the wikilinks inside it) as that prose scrolls beneath. */
 .weave-note-head{position:sticky;top:0;z-index:2;padding:14px var(--weave-note-gutter) 9px;background:var(--weave-page);border-bottom:1px solid var(--weave-line)}
 /* The page's largest voice, and the point of the review's headline finding:
@@ -423,13 +393,9 @@ body{font-size:var(--weave-px-base)}
    so the eye can read the whole of it without ever reading it. */
 .weave-note-meta{margin:0;display:flex;align-items:center;gap:8px;font-size:var(--weave-px-ui);color:var(--weave-dim);flex-wrap:wrap}
 .weave-note-time{color:var(--weave-faint)}
-/* \`Open in $EDITOR\`, demoted (P6.3). It used to be the bar's full-width
-   bordered button — the note's loudest control, sitting between its title and
-   its prose. Now it is an icon at the end of the meta line: the hint rides
-   the \`title\`/\`aria-label\` (\`editor.model.ts\`'s OPEN_HINT/OPEN_LABEL), and
-   the control is quiet until the pointer asks for it. */
+/* \`Open in $EDITOR\` is a quiet icon at the end of the meta line. */
 .weave-note-open{
-  margin-left:auto;display:inline-flex;align-items:center;justify-content:center;
+  display:inline-flex;align-items:center;justify-content:center;
   width:22px;height:22px;padding:0;font:inherit;color:var(--weave-faint);
   background:none;border:0;border-radius:var(--weave-radius);cursor:pointer;
 }
@@ -444,9 +410,7 @@ body{font-size:var(--weave-px-base)}
    withdrawn at the user's call: this is a workspace, and the note shares the
    width the desk gives it. It reads at --weave-px-body on 1.7 rather than the
    chrome's 1.6: the extra leading is what a page of running text needs that a
-   status bar does not. Code and the raw editor stay mono deliberately —
-   reading prose and reading raw are different postures, and the face switch
-   is the toggle. */
+   status bar does not. Code stays mono deliberately. */
 .weave-note-body{padding:12px var(--weave-note-gutter) 28px;font-size:var(--weave-px-body);line-height:1.7;color:var(--weave-fg)}
 .weave-note-body>*:first-child{margin-top:0}
 .weave-note-body h1,.weave-note-body h2,.weave-note-body h3,
@@ -534,53 +498,6 @@ body{font-size:var(--weave-px-base)}
    underlines a hover should not land with a thud. */
 @keyframes weave-preview-in{from{opacity:0;transform:translateY(2px)}}
 
-/* note editor (§11 P5) ---------------------------------------------------- */
-/* A \`<textarea>\`, not CodeMirror: §0 V10 measured CM6 at 118 KB gzip, more
-   than the entire rest of the client. It inherits the body's type so toggling
-   \`⌘E\` does not reflow the column into a different shape. */
-.weave-note-bar{
-  display:flex;align-items:center;gap:6px;flex-wrap:wrap;
-  padding:5px var(--weave-gutter);border-bottom:1px solid var(--weave-line);
-}
-/* \`weave-note-open\` is styled in the note column above (P6.3 moved it into
-   the head's meta row as an icon), and no longer shares the bar's bordered
-   button voice. */
-.weave-note-toggle,.weave-note-save,.weave-note-action{
-  font:inherit;font-size:var(--weave-px-ui);padding:2px 8px;cursor:pointer;
-  color:var(--weave-fg);background:var(--weave-raise);
-  border:1px solid var(--weave-line);border-radius:var(--weave-radius);
-}
-.weave-note-toggle:hover,.weave-note-save:hover,.weave-note-action:hover{
-  border-color:var(--weave-accent);
-}
-.weave-note-toggle[aria-pressed="true"]{border-color:var(--weave-accent);color:var(--weave-accent)}
-.weave-note-save:disabled{opacity:.45;cursor:default;border-color:var(--weave-line)}
-.weave-note-dirty{color:var(--weave-accent);font-size:var(--weave-px-subhead);line-height:1}
-.weave-note-status{font-size:var(--weave-px-ui);color:var(--weave-dim)}
-.weave-note-status-ok{color:var(--weave-ok)}
-.weave-note-status-warn{color:var(--weave-warn)}
-/* The prompt takes the full row rather than sitting inline: reload-or-overwrite
-   is a decision about the user's unsaved text, and a decision squeezed between
-   two buttons reads as a hint. */
-.weave-note-prompt{
-  flex-basis:100%;display:flex;align-items:center;gap:8px;flex-wrap:wrap;
-  margin-top:4px;padding:5px 8px;border-radius:var(--weave-radius);
-  background:var(--weave-raise);border:1px solid var(--weave-warn);
-}
-.weave-note-prompt-conflict,.weave-note-prompt-collision{border-color:var(--weave-warn)}
-.weave-note-prompt-external,.weave-note-prompt-discard{border-color:var(--weave-line)}
-.weave-note-prompt-text{margin:0;font-size:var(--weave-px-ui);color:var(--weave-fg)}
-.weave-note-prompt-actions{margin:0;display:flex;gap:6px;flex-wrap:wrap}
-.weave-note-editor{
-  flex:1;min-height:240px;resize:none;
-  padding:10px var(--weave-note-gutter) 24px;
-  /* Exactly the prose body's size and leading. The invariant is documented
-     and deliberate: ⌘E must not reflow, and reading raw at the same measure
-     the prose uses is what makes the toggle a lens rather than a jump. */
-  font:inherit;font-family:var(--weave-mono);font-size:var(--weave-px-body);line-height:1.7;
-  color:var(--weave-fg);background:var(--weave-page);border:0;outline-offset:-2px;
-}
-
 /* graph column ---------------------------------------------------------- */
 /* The canvas is a plain block sigma appends its own <canvas> layers into. It
    must have a definite size before sigma measures it, hence \`min-height\` and
@@ -691,14 +608,14 @@ body{font-size:var(--weave-px-base)}
 /* status bar ------------------------------------------------------------ */
 /* One separator vocabulary, shared with the header summary: every run of
    status items after the first opens with the same mid-dot, in the same
-   --weave-faint. cwd · selection | stamp · connection reads as one grammar,
+   --weave-faint. cwd · selection · stamp reads as one grammar,
    the same way the header's three-part summary does. */
 .weave-status{
   display:flex;align-items:center;gap:8px;height:22px;padding:0 var(--weave-gutter);
   font-family:var(--weave-mono);font-size:var(--weave-px-ui);color:var(--weave-dim);
   border-top:1px solid var(--weave-line);background:var(--weave-panel);
 }
-.weave-status-sel::before,.weave-conn::before{content:"·";margin-right:8px;color:var(--weave-faint)}
+.weave-status-sel::before{content:"·";margin-right:8px;color:var(--weave-faint)}
 .weave-status-cwd,.weave-status-sel{overflow:hidden;text-overflow:ellipsis;white-space:nowrap}
 .weave-status-cwd{max-width:38%;color:var(--weave-faint)}
 .weave-status-sel{color:var(--weave-fg)}
@@ -767,7 +684,7 @@ body{font-size:var(--weave-px-base)}
    --weave-new tint is exactly an accent at a strength text stays legible in. */
 ::selection{background:var(--weave-new)}
 /* Every scroller in the workspace (rows, note body, rail, palette results,
-   help sheet, editor) gets the same thin chrome: the default scrollbar is a
+   help sheet) gets the same thin chrome: the default scrollbar is a
    15 px system object the hairline aesthetic cannot afford, and Firefox and
    the Blink/WebKit pair cover the whole surface with these five rules. */
 *{scrollbar-width:thin;scrollbar-color:var(--weave-line-strong) transparent}
@@ -787,18 +704,13 @@ body{font-size:var(--weave-px-base)}
    properties only — colour, border, opacity, transform; height and padding
    snap. */
 .weave-row,.weave-ctx-link,.weave-chip,.weave-search,.weave-refresh,.weave-theme,
-.weave-divider,.weave-hit,.weave-note-toggle,.weave-note-save,.weave-note-open,
-.weave-note-action{
+.weave-hit,.weave-note-open{
   transition:background-color 120ms ease,border-color 120ms ease,color 120ms ease;
 }
 .weave-scrim{animation:weave-fade-in 140ms ease-out both}
 .weave-palette,.weave-help{animation:weave-overlay-in 160ms ease-out both}
-/* The conflict prompt mounts once per conflict, so its entrance cannot strobe
-   the way a context-rail flash would — that rail deliberately stays still. */
-.weave-note-prompt{animation:weave-prompt-in 180ms ease-out both}
 @keyframes weave-fade-in{from{opacity:0}}
 @keyframes weave-overlay-in{from{opacity:0;transform:translateY(4px)}}
-@keyframes weave-prompt-in{from{opacity:0;transform:translateY(-2px)}}
 @media (prefers-reduced-motion: reduce){*{transition:none!important;animation:none!important}}
 `;
 

package/src/web/client/state.ts

@@ -1,100 +1,20 @@
-/**
- * The context bus (weave-workspace §1.3).
- *
- * No event bus, no pub/sub — five Preact signals. Selecting in the tree,
- * clicking a graph node, following a wikilink and hitting a search result all
- * write {@link selectedId}; the note column, the graph highlight and the
- * context rail are computed from it.
- *
- * ## Tier rules (§2)
- *
- * This is `src/web/client/**`: it may import `src/web/shared` and browser
- * deps, and must **never** import `src/core` or `node:*`. Core is
- * Node-flavoured TypeScript that would drag `node:fs` into the bundle.
- *
- * ## What the P0 placeholders became
- *
- * This file used to declare three local interfaces behind `TODO(P1)` markers,
- * standing in for contracts that had not been written yet. Two of them are
- * now real and imported from `src/web/shared/wire.ts` — {@link graph} carries
- * a `GraphPayload` and {@link noteBody} a `ViewNote`, the exact values
- * `src/web/client/api.ts` returns. That removes a translation step that
- * existed only to satisfy a stale type: the fetchers already produce these
- * shapes, so anything narrower here was a downcast waiting to lose a field.
- *
- * The third resolved the other way. The TODO said to take `TreeState` from
- * the wire too, and that was simply wrong: which rows a user has expanded is
- * **not** a wire contract. It never crosses the network, the server has no
- * opinion about it, and putting it in `wire.ts` would have made the
- * client/server contract carry a purely local UI preference. So
- * {@link TreeState} stays here, declared where it is owned, with the marker
- * removed rather than carried forward.
- */
-
-import { signal } from "@preact/signals";
 import type { GraphPayload, NotePayload } from "../shared/wire";
 
-/** Connection state shown in the status bar. */
-export type ConnectionState = "live" | "reconnecting" | "offline";
-
 /**
  * Which tree rows are open.
  *
  * Client-owned, deliberately: see the module header. An interface rather than
  * a bare array so that P2's filter text and provenance-cycling state have an
- * obvious home that does not change this signal's type.
+ * obvious home that does not change this state shape.
  */
-export interface TreeState {
-  /** Ids of expanded tree rows. */
-  readonly expanded: readonly string[];
+export interface WorkspaceState {
+  graph: GraphPayload | null;
+  graphFailed: boolean;
+  note: NotePayload | null;
+  recentIds: ReadonlySet<string>;
+  selectedId: string | null;
 }
 
-/** The empty tree: nothing expanded. */
-export function initialTreeState(): TreeState {
-  return { expanded: [] };
+export function initialWorkspaceState(): WorkspaceState {
+  return { graph: null, graphFailed: false, note: null, recentIds: new Set(), selectedId: null };
 }
-
-/** The graph node id currently in context — everywhere. */
-export const selectedId = signal<string | null>(null);
-
-/** The whole graph, as delivered by the server. `null` until first load. */
-export const graph = signal<GraphPayload | null>(null);
-
-/**
- * Whether the *first* graph fetch failed.
- *
- * Distinct from {@link connection} on purpose: the socket can be live while a
- * fetch 500s, and "offline" is then the wrong sentence. With a graph already
- * on screen a failed refetch leaves the stale value standing (see
- * `workspace.ts`'s header — a stale graph beats a blank one), so this only
- * ever describes the boot, which is the one moment there is nothing to show.
- */
-export const graphFailed = signal(false);
-
-/**
- * Derived: fetched when {@link selectedId} names a note.
- *
- * A {@link NotePayload} rather than a bare `ViewNote` as of P5. The revision
- * travels **with** the body because the editor saves against it, and a
- * revision fetched separately would describe a state the draft was not typed
- * against — the exact window a conflict check exists to close. The note
- * column reads `.note` and is otherwise unchanged.
- */
-export const noteBody = signal<NotePayload | null>(null);
-
-/** Which tree rows are open. */
-export const treeState = signal<TreeState>(initialTreeState());
-
-/** SSE liveness (§6). */
-export const connection = signal<ConnectionState>("live");
-
-/**
- * Node ids that appeared in the most recent graph update (weave-workspace
- * §6 frame → tree feedback).
- *
- * Empty on the mount fetch — a first load must not animate the whole tree.
- * The controller replaces the set on every published payload and clears it
- * after {@link ../workspace!RECENT_TTL_MS}, so a row collapsed and re-expanded
- * minutes later does not replay its arrival animation.
- */
-export const recentIds = signal<ReadonlySet<string>>(new Set());