@schlessera/brain-ui-react 0.35.0 → 0.36.0

package/src/components/quick-actions/add-form.tsx

@@ -0,0 +1,188 @@
+import { Button, Callout } from "@schlessera/brain-ui-kit";
+import { CheckCircle2 } from "lucide-react";
+import type { ReactNode, RefObject } from "react";
+import { useId } from "react";
+import { Kbd } from "../layout/kbd.js";
+
+/**
+ * The "Add to brain" form, rendered from props (S6). `AddPanel` is the
+ * container: it owns the draft, the `brain add` / `brain index` requests and
+ * the `operation.epoch` guard that keeps a stale save from landing in a
+ * fresh form; this owns the two screens — the form, and the receipt after a
+ * save — and nothing else.
+ *
+ * `state` is the container's machine: `editing` and `error` show the form
+ * (with the error under it), `saving` is the form with the Add button inert,
+ * `saved` is the receipt. On the receipt, `indexed` is three-valued because
+ * an older server does not report it: `false` offers a retry, `true` says
+ * so, `undefined` says the server did not say.
+ *
+ * Fields stay native (the kit has no text input); the buttons are the kit's.
+ * The content field's ref is the container's, because focus after open and
+ * after "Add another" is timed against the panel's slide, which the
+ * container owns.
+ */
+export type AddState = "editing" | "saving" | "saved" | "error";
+
+export interface AddDraft {
+  content: string;
+  title: string;
+  type: string;
+  tags: string;
+}
+
+export interface AddFormProps {
+  state: AddState;
+  draft: AddDraft;
+  /** Types already in the brain, offered as completions — a hint list, not a closed set. */
+  knownTypes: string[];
+  error: string;
+  savedPath: string;
+  indexed: boolean | undefined;
+  indexing: boolean;
+  contentRef: RefObject<HTMLTextAreaElement | null>;
+  onDraft: (patch: Partial<AddDraft>) => void;
+  onSave: () => void;
+  onRetryIndex: () => void;
+  onAddAnother: () => void;
+  onClose: () => void;
+}
+
+export function AddForm(p: AddFormProps) {
+  const typesId = useId();
+  const saving = p.state === "saving";
+  const canSave = p.draft.content.trim().length > 0 && !saving;
+
+  function onKeyDown(e: React.KeyboardEvent) {
+    // Cmd/Ctrl+Enter saves from anywhere in the form. The container refuses
+    // a save that is not from `editing` or `error`, so the receipt is safe.
+    if ((e.metaKey || e.ctrlKey) && e.key === "Enter") {
+      e.preventDefault();
+      p.onSave();
+    }
+  }
+
+  if (p.state === "saved") {
+    return (
+      <div className="flex h-full flex-col" onKeyDown={onKeyDown}>
+        <div className="flex flex-1 flex-col items-center justify-center gap-4 px-8 text-center">
+          <CheckCircle2 className="h-8 w-8 text-primary" />
+          <p className="text-sm text-foreground">Added to your brain.</p>
+          <p className="text-xs text-muted-foreground">
+            {p.indexed === false
+              ? "Saved, but not indexed. Your content is safe; search may not find it yet."
+              : p.indexed === true
+                ? "Saved and indexed. Run a sync when you want to push it."
+                : "Saved. This server did not report whether indexing completed."}
+          </p>
+          {p.savedPath && <p className="break-all text-xs text-muted-foreground">{p.savedPath}</p>}
+          {p.indexed === false && (
+            <div className="flex flex-col items-center gap-2" role="status">
+              {p.error && <p className="text-xs text-destructive">{p.error}</p>}
+              <Button
+                label={p.indexing ? "Indexing…" : "Retry indexing"}
+                icon="retry"
+                tone="primary"
+                size="sm"
+                block={false}
+                disabled={p.indexing}
+                onClick={p.onRetryIndex}
+              />
+            </div>
+          )}
+          <div className="flex gap-2">
+            <Button label="Add another" icon="add" tone="ghost" size="sm" block={false} onClick={p.onAddAnother} />
+            <Button label="Done" icon="confirm" tone="primary" size="sm" block={false} onClick={p.onClose} />
+          </div>
+        </div>
+      </div>
+    );
+  }
+
+  return (
+    <div className="flex h-full flex-col" onKeyDown={onKeyDown}>
+      <div className="flex-1 space-y-4 overflow-y-auto p-5">
+        <Field label="Note">
+          <textarea
+            ref={p.contentRef}
+            value={p.draft.content}
+            onChange={(e) => p.onDraft({ content: e.target.value })}
+            rows={8}
+            placeholder="What do you want to remember?"
+            className="w-full resize-y rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
+          />
+        </Field>
+
+        <Field label="Title" hint="optional, derived from the note if empty">
+          <input
+            value={p.draft.title}
+            onChange={(e) => p.onDraft({ title: e.target.value })}
+            className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors focus:border-primary/40"
+          />
+        </Field>
+
+        <div className="grid grid-cols-2 gap-3">
+          <Field label="Type" hint="optional">
+            <input
+              value={p.draft.type}
+              onChange={(e) => p.onDraft({ type: e.target.value })}
+              list={typesId}
+              placeholder="note"
+              className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
+            />
+            <datalist id={typesId}>
+              {p.knownTypes.map((t) => (
+                <option key={t} value={t} />
+              ))}
+            </datalist>
+          </Field>
+
+          <Field label="Tags" hint="comma-separated">
+            <input
+              value={p.draft.tags}
+              onChange={(e) => p.onDraft({ tags: e.target.value })}
+              placeholder="idea, reading"
+              className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
+            />
+          </Field>
+        </div>
+
+        {p.state === "error" && (
+          <div role="alert">
+            <Callout tone="red" variant="banner" icon="failed" mono text={p.error} />
+          </div>
+        )}
+      </div>
+
+      <div className="flex items-center justify-between border-t border-border px-5 py-3">
+        <span className="text-[11px] text-muted-foreground">
+          <Kbd>Ctrl</Kbd>/<Kbd>Cmd</Kbd> + <Kbd>↵</Kbd> to save
+        </span>
+        <div className="flex gap-2">
+          <Button label="Cancel" tone="quiet" size="sm" block={false} onClick={p.onClose} />
+          <Button
+            label={saving ? "Saving…" : "Add"}
+            icon="add"
+            tone="primary"
+            size="sm"
+            block={false}
+            disabled={!canSave}
+            onClick={p.onSave}
+          />
+        </div>
+      </div>
+    </div>
+  );
+}
+
+function Field({ label, hint, children }: { label: string; hint?: string; children: ReactNode }) {
+  return (
+    <label className="block space-y-1.5">
+      <span className="text-xs font-medium text-foreground">
+        {label}
+        {hint && <span className="font-normal text-muted-foreground"> — {hint}</span>}
+      </span>
+      {children}
+    </label>
+  );
+}

package/src/components/quick-actions/add-modal.tsx

@@ -1,10 +1,7 @@
 import { useState, useEffect, useRef } from "react";
-import { Loader2, CheckCircle2, XCircle } from "lucide-react";
 import { SlidePanel } from "../layout/slide-panel.js";
-import { Kbd } from "../layout/kbd.js";
-import { api } from "../../lib/api-client.js";
-
-type State = "editing" | "saving" | "saved" | "error";
+import { useBrainApi } from "../../root-context.js";
+import { AddForm, type AddState as State } from "./add-form.js";
 
 /** Split the tags field on commas; empties dropped so "a,,b," is clean. */
 function parseTags(raw: string): string[] {
@@ -14,6 +11,13 @@ function parseTags(raw: string): string[] {
     .filter(Boolean);
 }
 
+/**
+ * The container (S6): the draft, `brain add`, `brain index` and the
+ * `operation.epoch` guard live here; `AddForm` draws from props. The epoch
+ * is bumped on open, on close, on "Add another" and on every request, so a
+ * response from a form the user has already left cannot land in the one
+ * they are looking at.
+ */
 export function AddPanel({
   open,
   onClose,
@@ -21,6 +25,7 @@ export function AddPanel({
   open: boolean;
   onClose: () => void;
 }) {
+  const api = useBrainApi();
   const [content, setContent] = useState("");
   const [title, setTitle] = useState("");
   const [type, setType] = useState("");
@@ -40,6 +45,7 @@ export function AddPanel({
   useEffect(() => {
     if (!open) return;
     operation.epoch++;
+    setKnownTypes([]);
     setSavedPath("");
     setIndexed(undefined);
     setIndexing(false);
@@ -67,7 +73,7 @@ export function AddPanel({
       cancelled = true;
       clearTimeout(t);
     };
-  }, [open, operation]);
+  }, [open, operation, api]);
 
   async function handleSave() {
     const body = content.trim();
@@ -122,155 +128,28 @@ export function AddPanel({
     setTimeout(() => contentRef.current?.focus(), 0);
   }
 
-  function handleKeyDown(e: React.KeyboardEvent) {
-    // Cmd/Ctrl+Enter saves from anywhere in the form.
-    if ((e.metaKey || e.ctrlKey) && e.key === "Enter") {
-      e.preventDefault();
-      void handleSave();
-    }
-  }
-
-  const saving = state === "saving";
-  const canSave = content.trim().length > 0 && !saving;
-
   return (
     <SlidePanel open={open} onClose={onClose} title="Add to brain" wide>
-      <div className="flex h-full flex-col" onKeyDown={handleKeyDown}>
-        {state === "saved" ? (
-          <div className="flex flex-1 flex-col items-center justify-center gap-4 px-8 text-center">
-            <CheckCircle2 className="h-8 w-8 text-primary" />
-            <p className="text-sm text-foreground">Added to your brain.</p>
-            <p className="text-xs text-muted-foreground">
-              {indexed === false ? "Saved, but not indexed. Your content is safe; search may not find it yet."
-                : indexed === true ? "Saved and indexed. Run a sync when you want to push it."
-                : "Saved. This server did not report whether indexing completed."}
-            </p>
-            {savedPath && <p className="break-all text-xs text-muted-foreground">{savedPath}</p>}
-            {indexed === false && (
-              <div className="space-y-2" role="status">
-                {error && <p className="text-xs text-destructive">{error}</p>}
-                <button onClick={retryIndex} disabled={indexing} className="rounded-lg bg-primary px-4 py-1.5 text-xs text-primary-foreground disabled:opacity-40">
-                  {indexing ? "Indexing..." : "Retry indexing"}
-                </button>
-              </div>
-            )}
-            <div className="flex gap-2">
-              <button
-                onClick={handleAddAnother}
-                className="rounded-lg bg-surface-raised px-4 py-1.5 text-xs font-medium text-foreground transition-colors hover:bg-surface-overlay"
-              >
-                Add another
-              </button>
-              <button
-                onClick={onClose}
-                className="rounded-lg bg-primary px-4 py-1.5 text-xs font-medium text-primary-foreground transition-opacity hover:opacity-90"
-              >
-                Done
-              </button>
-            </div>
-          </div>
-        ) : (
-          <>
-            <div className="flex-1 space-y-4 overflow-y-auto p-5">
-              <Field label="Note">
-                <textarea
-                  ref={contentRef}
-                  value={content}
-                  onChange={(e) => setContent(e.target.value)}
-                  rows={8}
-                  placeholder="What do you want to remember?"
-                  className="w-full resize-y rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
-                />
-              </Field>
-
-              <Field label="Title" hint="optional, derived from the note if empty">
-                <input
-                  value={title}
-                  onChange={(e) => setTitle(e.target.value)}
-                  className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors focus:border-primary/40"
-                />
-              </Field>
-
-              <div className="grid grid-cols-2 gap-3">
-                <Field label="Type" hint="optional">
-                  <input
-                    value={type}
-                    onChange={(e) => setType(e.target.value)}
-                    list="brain-add-types"
-                    placeholder="note"
-                    className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
-                  />
-                  <datalist id="brain-add-types">
-                    {knownTypes.map((t) => (
-                      <option key={t} value={t} />
-                    ))}
-                  </datalist>
-                </Field>
-
-                <Field label="Tags" hint="comma-separated">
-                  <input
-                    value={tags}
-                    onChange={(e) => setTags(e.target.value)}
-                    placeholder="idea, reading"
-                    className="w-full rounded-lg border border-border bg-surface-raised px-3 py-2 text-sm text-foreground outline-none transition-colors placeholder:text-muted-foreground focus:border-primary/40"
-                  />
-                </Field>
-              </div>
-
-              {state === "error" && (
-                <div className="flex items-start gap-2 rounded-lg border border-destructive/30 bg-destructive/10 px-3 py-2">
-                  <XCircle className="mt-0.5 h-3.5 w-3.5 shrink-0 text-destructive" />
-                  <span className="text-xs text-destructive">{error}</span>
-                </div>
-              )}
-            </div>
-
-            <div className="flex items-center justify-between border-t border-border px-5 py-3">
-              <span className="text-[11px] text-muted-foreground">
-                <Kbd>Ctrl</Kbd>/<Kbd>Cmd</Kbd> + <Kbd>↵</Kbd> to save
-              </span>
-              <div className="flex gap-2">
-                <button
-                  onClick={onClose}
-                  className="rounded-lg bg-surface-raised px-4 py-1.5 text-xs font-medium text-foreground transition-colors hover:bg-surface-overlay"
-                >
-                  Cancel
-                </button>
-                <button
-                  onClick={handleSave}
-                  disabled={!canSave}
-                  className="flex items-center gap-1.5 rounded-lg bg-primary px-4 py-1.5 text-xs font-medium text-primary-foreground transition-opacity hover:opacity-90 disabled:opacity-40"
-                >
-                  {saving && <Loader2 className="h-3.5 w-3.5 animate-spin" />}
-                  {saving ? "Saving..." : "Add"}
-                </button>
-              </div>
-            </div>
-          </>
-        )}
-      </div>
+      <AddForm
+        state={state}
+        draft={{ content, title, type, tags }}
+        knownTypes={knownTypes}
+        error={error}
+        savedPath={savedPath}
+        indexed={indexed}
+        indexing={indexing}
+        contentRef={contentRef}
+        onDraft={(patch) => {
+          if (patch.content !== undefined) setContent(patch.content);
+          if (patch.title !== undefined) setTitle(patch.title);
+          if (patch.type !== undefined) setType(patch.type);
+          if (patch.tags !== undefined) setTags(patch.tags);
+        }}
+        onSave={() => void handleSave()}
+        onRetryIndex={() => void retryIndex()}
+        onAddAnother={handleAddAnother}
+        onClose={onClose}
+      />
     </SlidePanel>
   );
 }
-
-function Field({
-  label,
-  hint,
-  children,
-}: {
-  label: string;
-  hint?: string;
-  children: React.ReactNode;
-}) {
-  return (
-    <label className="block space-y-1.5">
-      <span className="text-xs font-medium text-foreground">
-        {label}
-        {hint && (
-          <span className="font-normal text-muted-foreground"> — {hint}</span>
-        )}
-      </span>
-      {children}
-    </label>
-  );
-}

package/src/components/quick-actions/search-modal.tsx

@@ -4,7 +4,8 @@ import { SlidePanel } from "../layout/slide-panel.js";
 import { Kbd } from "../layout/kbd.js";
 import { useFileStore } from "../../stores/file-store.js";
 import { useUIStore } from "../../stores/ui-store.js";
-import { api, type BrainSearchHit } from "../../lib/api-client.js";
+import type { BrainSearchHit } from "../../lib/api-client.js";
+import { useBrainApi } from "../../root-context.js";
 import { parseSnippet } from "../../lib/search-snippet.js";
 import { cn } from "../../lib/utils.js";
 
@@ -22,6 +23,7 @@ export function SearchPanel({
   open: boolean;
   onClose: () => void;
 }) {
+  const api = useBrainApi();
   const [query, setQuery] = useState("");
   const [state, setState] = useState<State>("idle");
   const [resultQuery, setResultQuery] = useState("");
@@ -90,7 +92,7 @@ export function SearchPanel({
     } finally {
       if (controllerRef.current === controller) controllerRef.current = null;
     }
-  }, [cancelSearch]);
+  }, [cancelSearch, api]);
 
   // Fresh panel every time it opens: stale results from a previous query would
   // otherwise flash before the first search lands.
@@ -110,7 +112,7 @@ export function SearchPanel({
     // mobile keyboard on some browsers.
     const t = setTimeout(() => inputRef.current?.focus(), 120);
     return () => { clearTimeout(t); cancelSearch(); };
-  }, [open, cancelSearch]);
+  }, [open, cancelSearch, api]);
 
   // Debounced search on typing.
   useEffect(() => {

package/src/components/quick-actions/streaming-modal.tsx

@@ -1,10 +1,14 @@
+import { useBrainUiRoot } from "../../root-context.js";
 import { useState, useEffect, useRef } from "react";
-import { Loader2, CheckCircle2, XCircle } from "lucide-react";
 import { SlidePanel } from "../layout/slide-panel.js";
 import { linkifyPaths } from "../chat/brain-markdown.js";
+import { StreamingOutput, type StreamState } from "./streaming-output.js";
 
-type StreamState = "idle" | "running" | "success" | "error" | "cancelled";
-
+/**
+ * The container (S6) for a streamed backend job: the request, the SSE reader
+ * loop, the abort controller and the "still my stream" checks live here;
+ * `StreamingOutput` draws from props.
+ */
 export function StreamingPanel({
   open,
   onClose,
@@ -15,9 +19,11 @@ export function StreamingPanel({
   open: boolean;
   onClose: () => void;
   title: string;
+  /** Backend-relative path, resolved against the current root. */
   endpoint: string;
   method?: string;
 }) {
+  const root = useBrainUiRoot();
   const [state, setState] = useState<StreamState>("idle");
   const [lines, setLines] = useState<string[]>([]);
   const scrollRef = useRef<HTMLDivElement>(null);
@@ -29,30 +35,41 @@ export function StreamingPanel({
     setState("running");
     setLines([]);
 
+    let disposed = false;
+    let reader: ReadableStreamDefaultReader<Uint8Array> | undefined;
     const controller = new AbortController();
+    const cancelReader = () => { void reader?.cancel().catch(() => {}); };
+    controller.signal.addEventListener("abort", cancelReader);
     controllerRef.current = controller;
 
     (async () => {
       try {
-        const res = await fetch(endpoint, {
+        const res = await root.request(root.backendUrl(endpoint), {
           method,
           signal: controller.signal,
         });
 
+        if (disposed || controller.signal.aborted) {
+          await res.body?.cancel();
+          return;
+        }
+
         if (!res.ok || !res.body) {
-          setState("error");
           const body = await res.json().catch(() => null);
+          if (disposed || controller.signal.aborted) return;
+          setState("error");
           const message = typeof body?.error === "string" ? body.error : `HTTP ${res.status}: ${res.statusText}`;
           if (!controller.signal.aborted) setLines((l) => [...l, message]);
           return;
         }
 
-        const reader = res.body.getReader();
+        reader = res.body.getReader();
         const decoder = new TextDecoder();
         let buffer = "";
 
         while (true) {
           const { done, value } = await reader.read();
+          if (disposed || controller.signal.aborted) return;
           if (done) break;
 
           buffer += decoder.decode(value, { stream: true });
@@ -77,6 +94,7 @@ export function StreamingPanel({
           }
         }
       } catch (err: any) {
+        if (disposed || controller.signal.aborted) return;
         if (err.name === "AbortError") {
           setLines((l) => [...l, "Cancelled."]);
           setState("cancelled");
@@ -85,15 +103,18 @@ export function StreamingPanel({
           setLines((l) => [...l, `Error: ${err.message}`]);
         }
       } finally {
-        controllerRef.current = null;
+        controller.signal.removeEventListener("abort", cancelReader);
+        reader?.releaseLock();
+        if (controllerRef.current === controller) controllerRef.current = null;
       }
     })();
 
     return () => {
+      disposed = true;
       controller.abort();
-      controllerRef.current = null;
+      if (controllerRef.current === controller) controllerRef.current = null;
     };
-  }, [open, endpoint, method]);
+  }, [open, endpoint, method, root]);
 
   // Auto-scroll
   useEffect(() => {
@@ -103,68 +124,18 @@ export function StreamingPanel({
   const isRunning = state === "running";
 
   return (
-    <SlidePanel
-      open={open}
-      onClose={isRunning ? () => {} : onClose}
-      title={title}
-    >
-      <div className="flex h-full flex-col">
-        {/* Status bar */}
-        <div className="flex items-center gap-2 border-b border-border px-5 py-2">
-          {isRunning && (
-            <Loader2 className="h-3.5 w-3.5 animate-spin text-primary" />
-          )}
-          {state === "success" && (
-            <CheckCircle2 className="h-3.5 w-3.5 text-green-500" />
-          )}
-          {(state === "error" || state === "cancelled") && (
-            <XCircle className="h-3.5 w-3.5 text-destructive" />
-          )}
-          <span className="text-xs text-muted-foreground">
-            {isRunning
-              ? "Running..."
-              : state === "success"
-                ? "Complete"
-                : state === "error"
-                  ? "Failed"
-                  : state === "cancelled"
-                    ? "Cancelled"
-                    : "Ready"}
-          </span>
-        </div>
-
-        {/* Log output */}
-        <div className="flex-1 overflow-y-auto p-4 font-[family-name:var(--font-mono)] text-xs leading-relaxed text-muted-foreground">
-          {lines.map((line, i) => (
-            <div key={i} className="whitespace-pre-wrap">
-              {linkifyPaths(line)}
-            </div>
-          ))}
-          {isRunning && lines.length === 0 && (
-            <div className="text-muted-foreground/50">Starting...</div>
-          )}
-          <div ref={scrollRef} />
-        </div>
-
-        {/* Footer */}
-        <div className="border-t border-border px-5 py-3 flex justify-end">
-          {isRunning ? (
-            <button
-              onClick={() => controllerRef.current?.abort()}
-              className="rounded-lg border border-destructive/30 px-4 py-1.5 text-xs font-medium text-destructive transition-colors hover:bg-destructive/10"
-            >
-              Cancel
-            </button>
-          ) : (
-            <button
-              onClick={onClose}
-              className="rounded-lg bg-surface-raised px-4 py-1.5 text-xs font-medium text-foreground transition-colors hover:bg-surface-overlay"
-            >
-              Close
-            </button>
-          )}
-        </div>
-      </div>
+    <SlidePanel open={open} onClose={isRunning ? () => {} : onClose} title={title}>
+      <StreamingOutput
+        state={state}
+        lines={lines.map((line) => linkifyPaths(line))}
+        anchor={<div ref={scrollRef} />}
+        onCancel={() => {
+          controllerRef.current?.abort();
+          setState("cancelled");
+          setLines((lines) => [...lines, "Cancelled."]);
+        }}
+        onClose={onClose}
+      />
     </SlidePanel>
   );
 }