refactor(web): decompose MemorySection into a features/memory vertical slice

apps/AGENTS.md

@@ -9,6 +9,12 @@ Follow the root `AGENTS.md` first. This file only records module-level boundarie
 - `apps/desktop`: Electron shell. Desktop does not guess the web port; it reads runtime status through sidecar IPC and opens the reported web URL.
 - `apps/packaged`: Thin packaged Electron runtime entry. It starts packaged daemon/web sidecars, registers the `od://` entry protocol, and delegates desktop host behavior to `apps/desktop`.
 
+## apps/web frontend refactors (vertical slices)
+
+Before decomposing a large `apps/web` component (a "god-component" like `SettingsDialog`, `MemorySection`, or `ChatComposer`), read `docs/adr/0002-frontend-vertical-slice-decomposition.md` first. It is the canonical design for this work and covers: the layer split (wire DTOs → `packages/contracts`, transport adapters → `apps/web/src/providers/`, in-slice `features/<slice>/` owning ports + rules + hooks + dumb components + barrel), the injected-port hook paradigm, and the **Testing & coverage strategy for a slice** subsection (how to reach ≥95% honestly — classify each "unreachable" branch and apply the matching fix instead of `/* v8 ignore */`).
+
+The `MemorySection` slice (`apps/web/src/features/memory/`) is the worked canary; mirror its structure. A detailed session working-log for that canary lives in the branch's `ADS-project-knowledge/reports/*-memory-slice-*handoff.md` when present.
+
 ## Daemon layout
 
 - `apps/daemon/src/` contains only daemon app source.

apps/web/package.json

@@ -62,6 +62,7 @@
     "@types/node": "20.19.39",
     "@types/react": "18.3.28",
     "@types/react-dom": "18.3.7",
+    "@vitest/coverage-v8": "4.1.6",
     "jsdom": "29.1.1",
     "postcss": "8.5.15",
     "tailwindcss": "4.3.0",

apps/web/src/components/MemoryModelInline.tsx

@@ -40,6 +40,7 @@ import type {
   MemoryExtractionProvider,
   MemoryListResponse,
 } from '@open-design/contracts';
+import { patchMemoryExtractionConfig } from '../providers/memory';
 import type { AgentModelOption, ApiProtocol, ExecMode } from '../types';
 import {
   SUGGESTED_MODELS_BY_PROTOCOL,
@@ -170,22 +171,6 @@ async function fetchMemoryExtraction(): Promise<MemoryExtractionMaskedConfig | n
   }
 }
 
-async function saveMemoryExtraction(
-  extraction: MemoryExtractionConfigShape | null,
-): Promise<MemoryExtractionMaskedConfig | null | undefined> {
-  const resp = await fetch('/api/memory/config', {
-    method: 'PATCH',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify({ extraction }),
-  });
-  if (!resp.ok) return undefined;
-  const json = (await resp.json()) as {
-    enabled: boolean;
-    extraction: MemoryExtractionMaskedConfig | null;
-  };
-  return json.extraction ?? null;
-}
-
 export function MemoryModelInline({
   mode,
   apiProtocol,
@@ -313,7 +298,7 @@ export function MemoryModelInline({
     ) => {
       setBusy(true);
       try {
-        const result = await saveMemoryExtraction(next);
+        const result = await patchMemoryExtractionConfig(next);
         if (result !== undefined) {
           setConfig(result);
           // Skip the "Saved!" flash on background re-syncs (provider

apps/web/src/components/MemoryProfilePanel.tsx

@@ -48,8 +48,10 @@ function parseProfileBody(body: string): Record<string, string> {
   for (const line of lines) {
     const m = /^\s*-\s*([^:]+):\s*(.*)$/.exec(line);
     if (!m) continue;
-    const rawLabel = (m[1] ?? '').trim().toLowerCase();
-    const value = (m[2] ?? '').trim();
+    // Both groups are guaranteed present once `.exec` matches (`[^:]+` requires
+    // ≥1 char; `.*` always yields a string), so the captures are non-null here.
+    const rawLabel = m[1]!.trim().toLowerCase();
+    const value = m[2]!.trim();
     const field = PROFILE_FIELDS.find((f) => f.label.toLowerCase() === rawLabel);
     if (field) out[field.label] = value;
   }