create-pmos 0.47.0 → 0.48.0

package/workspace/public/pm/modes-live.js

@@ -23,23 +23,36 @@ const ROW = 'display:block;width:100%;padding:7px 14px;min-height:40px;text-alig
 const BTN = 'font-family:var(--mono);font-size:10.5px;padding:3px 9px;border:1px solid var(--border);color:var(--muted-foreground);min-height:26px';
 const NOTE = 'padding:8px 14px;font-size:11.5px;color:var(--muted-foreground);line-height:1.5';
 
-/* one cached read per endpoint; `reload(key)` drops it so the next render refetches */
+/* one cached read per endpoint; `reload(key)` drops it so the next render refetches.
+ *
+ * AN ANSWER OLDER THAN A RELOAD IS DROPPED. `reload` follows a write (a saved setting, a dismissed
+ * item). A read that was already in flight when the write went out carries the state from BEFORE it,
+ * and it used to land after the fresh read and overwrite it — then count as fresh for 15 s. Measured
+ * on Today: quiet hours saved, the note shown, and 1.5 s later the note gone while the server held
+ * the setting (pm-r22-test.js, about 1 run in 8). `gen` counts reloads per key; a response whose
+ * generation is out of date is not written. */
 const cache = new Map();
+const gen = new Map();
 function read(key, url) {
   const hit = cache.get(key);
   if (hit && (hit.state !== 'ok' || Date.now() - hit.at < 15000)) return hit;
   if (!hit || hit.state === 'ok') {
     const next = { state: hit ? hit.state : 'loading', data: hit ? hit.data : null, at: hit ? hit.at : 0 };
     cache.set(key, next);
+    const g = gen.get(key) || 0;
+    const current = () => (gen.get(key) || 0) === g;
     fetch(url).then(readJson).then(
-      (data) => { cache.set(key, { state: data && data.ok === false ? 'error' : 'ok', data, error: data && data.error, at: Date.now() }); setState({}); },
-      (e) => { cache.set(key, { state: 'error', error: (e && e.message) || String(e), at: Date.now() }); setState({}); },
+      (data) => { if (!current()) return; cache.set(key, { state: data && data.ok === false ? 'error' : 'ok', data, error: data && data.error, at: Date.now() }); setState({}); },
+      (e) => { if (!current()) return; cache.set(key, { state: 'error', error: (e && e.message) || String(e), at: Date.now() }); setState({}); },
     );
     return next;
   }
   return hit;
 }
-const reload = (key) => cache.delete(key);
+const reload = (key) => { gen.set(key, (gen.get(key) || 0) + 1); cache.delete(key); };
+/* refetch on the next render but keep showing the last answer meanwhile, so a list does not blank while it updates.
+ * It follows a write too, so it bumps the generation like `reload`: a read already in flight is older than the write. */
+const refresh = (key) => { gen.set(key, (gen.get(key) || 0) + 1); const h = cache.get(key); if (h && h.state === 'ok') h.at = 0; else cache.delete(key); };
 /* A route this server does not have answers 404 with the plain text `not found`. Parsing that as JSON
  * showed the PM "Unexpected token 'o', \"not found\" is not valid JSON" — true, and no help. The usual
  * cause is a server still running code older than the page, so say that and the one command. */
@@ -275,7 +288,8 @@ async function memDo(body) {
   const out = await post('/api/memory', { ws, ...body });
   memNote = out && out.ok ? (out.staged ? 'held for approval' : null) : ((out && out.error) || 'not saved');
   if (out && !out.ok && out.entries && out.entries.length) memNote += ' — current entries: ' + out.entries.map((e) => '“' + e.text.slice(0, 40) + '”').join(', ');
-  reload('memory'); setState({});
+  // the list caches its read per workspace, so the reload names that key; a bare 'memory' left the old list on screen for 15s
+  reload('memory:' + ws); reload('memory'); setState({});
 }
 const ask = (q, v) => (typeof window !== 'undefined' && window.prompt ? window.prompt(q, v || '') : null);
 let dataMod = null;
@@ -494,10 +508,147 @@ export function skillDetailLive(det) {
 
 /* ── knowledge ──────────────────────────────────────────────────────────────────────────────── */
 function flatten(node, out, depth) {
-  for (const d of node.docs || []) out.push({ path: d.path, name: d.name, depth });
+  for (const d of node.docs || []) out.push({ path: d.path, name: d.name, depth, category: d.category || null, basis: d.category_basis || null, why: d.category_why || null, kind: d.category_kind || null, disagrees: d.category_disagrees || [] });
   if (depth < 3) for (const sub of node.dirs || []) { out.push({ dir: sub.name || sub.path, depth }); flatten(sub, out, depth + 1); }
   return out;
 }
+/* jev-doc-categorizer Run 1 — each document's category, derived locally beside its own `type`, with the basis that decided it.
+ * `undetermined` is its own value: the file has no evidence, and it is shown as that, never as `other`. A press on a category
+ * narrows the list; a second press on it shows everything again. Nothing is written. */
+let knCat = '';
+const catTitle = (it) => it.category ? 'category ' + it.category + (it.kind ? ' (' + it.kind + ')' : '') + (it.basis ? ' · decided by ' + it.basis : ' · no evidence') + (it.why ? ' (' + it.why + ')' : '')
+  + (it.disagrees && it.disagrees.length ? ' · ' + it.disagrees.map((d) => d.basis + ' says ' + d.category).join(', ') : '') + ' · advisory' : '';
+const catChip = (it) => it.category ? el('span', { 'data-cat': it.category, 'data-cat-basis': it.basis || '', title: catTitle(it), style: 'margin-left:8px;font-size:9.5px;letter-spacing:.06em;color:var(--muted-foreground)' }, iv('· ' + it.category + (it.basis === 'jev' ? ' (jev)' : ''))) : null;
+function categoryBar(trees) {
+  const counts = {};
+  for (const t of trees) for (const it of flatten(t, [], 0)) if (it.path && it.category) counts[it.category] = (counts[it.category] || 0) + 1;
+  const names = Object.keys(counts).sort((a, b) => counts[b] - counts[a]);
+  if (!names.length) return null;
+  return el('div', { 'data-cat-bar': '1', style: 'display:flex;flex-wrap:wrap;gap:4px;padding:2px 14px 6px' },
+    names.map((n) => el('button', { 'data-cat-filter': n, 'aria-pressed': knCat === n ? 'true' : 'false', title: 'show only ' + n + ' documents', onClick: () => { knCat = knCat === n ? '' : n; setState({}); },
+      style: 'font-family:var(--mono);font-size:10px;padding:2px 7px;border:1px solid var(--border);' + (knCat === n ? 'background:var(--accent);' : '') }, iv(n + ' ' + counts[n]))));
+}
+/* D82 — Jev on documents the local rules leave undetermined. The switch is off until the PM turns it on; the first run shows
+ * its request count and sends only after the PM confirms that number; afterwards opening this list sends only new or changed
+ * files. Jev's categories are marked "(jev)" and are advisory. */
+let dPlan = null;            // the plan the PM is looking at: { ws, plan }
+let dNote = null;
+const dSay = (text) => { dNote = text; setState({}); };
+async function dSwitch(ws, on) {
+  const out = await post('/api/doccat/jev', { ws, categorize: on });
+  reload('library:' + ws); dPlan = null;
+  dSay(out && out.ok ? (on ? 'categorizing with Jev is on — review the first run’s count, then confirm it; nothing is sent before that' : 'categorizing with Jev is off') : ((out && out.error) || 'not saved'));
+}
+async function dResume(ws) {
+  const out = await post('/api/doccat/jev/resume', { ws });
+  reload('library:' + ws);
+  dSay(out && out.ok ? 'categorizing with Jev resumed' : ((out && out.error) || 'not resumed'));
+}
+async function dPlanFor(ws) {
+  const out = await post('/api/doccat/jev/run', { ws });
+  if (!out || out.ok === false) { dPlan = null; return dSay((out && out.error) || 'no plan'); }
+  dPlan = { ws, plan: out.plan }; dSay('review the count, then confirm to send');
+}
+async function dRun() {
+  if (!dPlan) return;
+  const { ws, plan } = dPlan;
+  dSay('sending ' + plan.requests + ' request(s) to ' + plan.host + ' …');
+  const out = await post('/api/doccat/jev/run', { ws, confirm: true, expect_requests: plan.requests, expect_fingerprint: plan.fingerprint });
+  dPlan = null; reload('library:' + ws);
+  dSay(!out || out.ok === false ? ((out && out.error) || 'the run failed') : 'asked ' + out.sent + ' · ' + out.decided + ' categorized · ' + out.unsure + ' unsure · ' + (out.unreadable || 0) + ' unreadable · ' + out.cached + ' answered from the cache · ' + out.failed + ' failed');
+}
+function doccatBlock(ws, j) {
+  if (!ws || !j) return [];
+  const kids = [];
+  kids.push(el('div', { 'data-doccat': 'explain', style: NOTE }, iv('Files with no local evidence are “undetermined”. Jev can be asked to categorize them from a short snippet: the title, the headings and the opening, never the whole file. Its answers are marked (jev), are advisory, and an unsure answer stays undetermined.')));
+  kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;flex-wrap:wrap;padding:2px 14px 6px' },
+    el('button', { style: BTN + (j.switch ? ';color:var(--foreground);border-color:var(--foreground)' : ''), hover: 'color:var(--foreground)', 'data-doccat': 'switch', onClick: () => dSwitch(ws, !j.switch) },
+      iv(j.switch ? 'categorize with Jev: ON — turn off' : 'categorize with Jev: OFF — turn on')),
+    el('button', { style: BTN, hover: 'color:var(--foreground)', disabled: j.open ? null : 'true', 'data-doccat': 'plan',
+      title: 'Shows how many requests the undetermined files would take, and sends only after you confirm that number.', onClick: () => dPlanFor(ws) }, 'ask Jev about undetermined files'),
+    dPlan && dPlan.ws === ws ? el('button', { style: BTN + ';color:var(--foreground);border-color:var(--foreground)', hover: 'background:var(--accent)', 'data-doccat': 'run', onClick: dRun }, 'confirm and send ' + dPlan.plan.requests + ' request(s)') : null));
+  kids.push(el('div', { 'data-doccat': 'gate', 'data-open': j.open ? 'yes' : 'no', style: NOTE + (j.open ? '' : ';color:var(--foreground)') },
+    iv(j.open ? (j.confirmed ? 'Active: new or changed undetermined files are sent when the workspace reads this repository’s documents (when it starts, and when this list opens), at most once a minute.' : 'Ready: nothing is sent until you confirm the first run’s count.') : 'Not active: ' + j.sentence + ' All three must hold before anything is sent.')));
+  if (j.paused) kids.push(el('div', { 'data-doccat': 'paused', style: 'display:flex;gap:8px;align-items:center;padding:2px 14px 6px;font-size:11.5px;color:var(--destructive)' },
+    el('span', null, iv(j.paused.reason)),
+    el('button', { style: BTN, hover: 'color:var(--foreground)', 'data-doccat': 'resume', onClick: () => dResume(ws) }, 'resume')));
+  const l = j.last;
+  if (l && l.waiting) kids.push(el('div', { 'data-doccat': 'waiting', style: NOTE + ';color:var(--foreground)' }, iv(l.waiting + ' new or changed file(s) are waiting: more than an automatic pass sends. Ask Jev and confirm the count to send them.')));
+  else if (l) kids.push(el('div', { 'data-doccat': 'last', style: NOTE }, iv('Last Jev pass ' + String(l.at).replace('T', ' ').slice(0, 16) + ': sent ' + l.sent + ' · categorized ' + l.decided + ' · unsure ' + l.unsure + ' · unreadable ' + (l.unreadable || 0) + ' · withheld ' + (l.withheld_count || 0) + ' · excluded ' + l.excluded + ' · failed ' + l.failed + '.')));
+  if (dPlan && dPlan.ws === ws) {
+    const p = dPlan.plan;
+    kids.push(el('div', { 'data-doccat': 'plan-text', style: NOTE }, iv(p.requests + ' request(s), about ' + p.characters.toLocaleString() + ' characters, to ' + p.host + ' · ' + p.candidates + ' undetermined file(s) · ' + p.cached + ' already answered · ' + p.excluded + ' excluded · ' + p.withheld_count + ' withheld by the credential scan.')));
+  }
+  if (dNote) kids.push(el('div', { 'data-doccat': 'note', style: NOTE }, iv(dNote)));
+  return kids;
+}
+/* jev-doc-categorizer Run 3 (C7) — the PM corrects a category. `fix` on a row opens a small control: a category, a ticket kind for
+ * a ticket, `save correction` and `clear correction`. A correction is kept per user, outside the repository, and never edits the
+ * file. `write it into the file` is a separate press: it shows the one-line diff first, and `apply` writes it only if the file is
+ * still the text the diff was made from (the server compares the sha1 and answers 409 when it is not). The server checks every
+ * value again; these two lists only fill the menus. */
+const CATS = ['prd', 'meeting-notes', 'report', 'ticket', 'knowledge', 'spec-artifact', 'other'];
+const KINDS = ['design', 'engineering', 'other'];
+let knFix = null;          // { path, category, ticket_kind } — the row whose control is open
+let knPlan = null;         // { path, category, ticket_kind, diff, sha, unchanged } — the write-back diff on screen
+let knNote = null;         // { path, tone, text }
+let knBusy = false;
+const FIX = 'font-family:var(--mono);font-size:9.5px;padding:1px 6px;margin-right:10px;border:1px solid var(--border);color:var(--muted-foreground);flex:none';
+const SEL = 'font-family:var(--mono);font-size:10.5px;padding:2px 4px;border:1px solid var(--border);background:var(--background);color:var(--foreground)';
+const knSay = (path, tone, text) => { knNote = { path, tone, text }; knBusy = false; setState({}); };
+function knOpen(it) {
+  if (knFix && knFix.path === it.path) { knFix = null; knPlan = null; knNote = null; setState({}); return; }
+  const cat = CATS.includes(it.category) ? it.category : 'other';
+  knFix = { path: it.path, category: cat, ticket_kind: cat === 'ticket' ? (it.kind || 'other') : null };
+  knPlan = null; knNote = null; setState({});
+}
+async function knSave(ws, clear) {
+  if (!knFix || knBusy) return;
+  knBusy = true; setState({});
+  const body = clear ? { ws, path: knFix.path, clear: true } : { ws, path: knFix.path, category: knFix.category, ticket_kind: knFix.category === 'ticket' ? knFix.ticket_kind : null };
+  const res = await post('/api/doccat/correct', body);
+  refresh('library:' + ws);
+  if (res && res.ok) knSay(knFix.path, 'ok', clear ? (res.cleared ? 'correction cleared — the rules decide this file again' : 'this file had no correction') : 'saved — your correction wins over every rule. The file is not changed.');
+  else knSay(knFix.path, 'bad', (res && res.error) || 'not saved');
+}
+async function knWriteback(ws, confirm) {
+  if (!knFix || knBusy) return;
+  knBusy = true; setState({});
+  const want = { category: knFix.category, ticket_kind: knFix.category === 'ticket' ? knFix.ticket_kind : null };
+  if (!confirm) {
+    const res = await post('/api/doccat/writeback', { ws, path: knFix.path, ...want });
+    if (res && res.ok) { knPlan = { path: knFix.path, ...want, diff: res.diff, sha: res.sha, next_sha: res.next_sha, unchanged: !!res.unchanged }; knSay(knFix.path, 'mut', res.unchanged ? 'the file already says this — nothing to write' : 'this is the change. Press apply to write it into the file.'); }
+    else { knPlan = null; knSay(knFix.path, 'bad', (res && res.error) || 'no diff'); }
+    return;
+  }
+  if (!knPlan) { knBusy = false; return; }
+  const res = await post('/api/doccat/writeback', { ws, path: knPlan.path, category: knPlan.category, ticket_kind: knPlan.ticket_kind, confirm: true, expect_sha: knPlan.sha, expect_next_sha: knPlan.next_sha });
+  knPlan = null;
+  refresh('library:' + ws);
+  if (res && res.ok) knSay(knFix.path, 'ok', res.written ? 'written into the file\'s frontmatter' : 'the file already says this — nothing was written');
+  else knSay(knFix.path, 'bad', (res && res.error) || 'not written');
+}
+function knPanel(ws, it) {
+  const f = knFix;
+  const pad = 'padding:2px 14px 8px ' + (20 + it.depth * 12) + 'px';
+  const btn = (attr, label, fn, disabled) => el('button', { [attr]: '1', disabled: knBusy || disabled, onClick: fn, style: BTN, hover: 'color:var(--foreground)' }, label);
+  const kids = [
+    el('select', { 'data-cat-select': '1', 'aria-label': 'category', style: SEL, onChange: (e) => { f.category = e.target.value; f.ticket_kind = f.category === 'ticket' ? (f.ticket_kind || 'other') : null; knPlan = null; setState({}); } },
+      CATS.map((c) => el('option', { value: c, selected: c === f.category }, c))),
+    f.category === 'ticket' ? el('select', { 'data-cat-kind': '1', 'aria-label': 'ticket kind', style: SEL, onChange: (e) => { f.ticket_kind = e.target.value; knPlan = null; setState({}); } },
+      KINDS.map((k) => el('option', { value: k, selected: k === f.ticket_kind }, k))) : null,
+    btn('data-cat-save', 'save correction', () => knSave(ws, false)),
+    it.basis === 'pm correction' ? btn('data-cat-clear', 'clear correction', () => knSave(ws, true)) : null,
+    btn('data-cat-writeback', 'write it into the file', () => knWriteback(ws, false)),
+  ];
+  const out = [el('div', { 'data-cat-panel': it.path, style: pad + ';display:flex;flex-wrap:wrap;gap:6px;align-items:center' }, kids)];
+  if (knPlan && knPlan.path === it.path && !knPlan.unchanged) {
+    out.push(el('pre', { 'data-cat-diff': '1', style: pad.replace('padding:2px', 'margin:0;padding:6px') + ';font-family:var(--mono);font-size:10.5px;white-space:pre-wrap;color:var(--foreground);background:var(--accent)' }, iv(knPlan.diff)));
+    out.push(el('div', { style: pad + ';display:flex;gap:6px' }, btn('data-cat-apply', 'apply', () => knWriteback(ws, true)), btn('data-cat-cancel', 'cancel', () => { knPlan = null; knNote = null; setState({}); })));
+  }
+  if (knNote && knNote.path === it.path) out.push(el('div', { 'data-cat-note': knNote.tone, style: pad + ';font-size:11px;color:' + TONE[knNote.tone] }, iv(knNote.text)));
+  return out;
+}
 export function knowledgeLive() {
   const ws = state.wsId || '';
   const r = read('library:' + ws, '/api/library?ws=' + encodeURIComponent(ws));
@@ -505,11 +656,18 @@ export function knowledgeLive() {
   if (st) return [st];
   const trees = (r.data && r.data.knowledge) || [];
   const kids = [el('div', { style: NOTE }, iv('This repository\'s knowledge: the OKF bundle, skills and build-skills. Open a document to read it.'))];
+  const bar = categoryBar(trees); if (bar) kids.push(bar);
+  kids.push(...doccatBlock(ws, r.data && r.data.doccat));
   for (const t of trees) {
     kids.push(el('div', { style: LABEL }, iv(t.path || t.name || 'knowledge')));
     for (const it of flatten(t, [], 0)) {
+      if (knCat && (it.dir || it.category !== knCat)) continue;
       if (it.dir) { kids.push(el('div', { style: 'padding:3px 14px 1px ' + (14 + it.depth * 12) + 'px;font-family:var(--mono);font-size:10px;color:var(--muted-foreground)' }, iv(it.dir + '/'))); continue; }
-      kids.push(el('button', { onClick: () => openDocIn(null, it.path), style: 'display:block;width:100%;text-align:left;padding:4px 14px 4px ' + (20 + it.depth * 12) + 'px;font-family:var(--mono);font-size:11.5px', hover: 'background:var(--accent)', 'data-r4': 'know' }, iv(it.name)));
+      const open = !!(knFix && knFix.path === it.path);
+      kids.push(el('div', { 'data-cat-row': it.path, style: 'display:flex;align-items:center' },
+        el('button', { onClick: () => openDocIn(null, it.path), style: 'display:block;flex:1 1 auto;min-width:0;text-align:left;padding:4px 14px 4px ' + (20 + it.depth * 12) + 'px;font-family:var(--mono);font-size:11.5px', hover: 'background:var(--accent)', 'data-r4': 'know' }, iv(it.name), catChip(it)),
+        el('button', { 'data-cat-fix': it.path, 'aria-expanded': open ? 'true' : 'false', title: 'correct the category of this document', onClick: () => knOpen(it), style: FIX + (open ? ';background:var(--accent);color:var(--foreground)' : ''), hover: 'color:var(--foreground)' }, 'fix')));
+      if (open) kids.push(...knPanel(ws, it));
     }
   }
   // R11 — the notes PMOS and its agents wrote here (requirement notes, report drafts and definitions, memory).
@@ -614,6 +772,12 @@ async function jSwitch(on) {
   reload('claims'); jPlan = null;
   jSay(out && out.ok ? (on ? 'checking with Jev automatically is on — the first check records what exists and sends nothing; after that only changed pairs are sent' : 'checking with Jev automatically is off') : ((out && out.error) || 'not saved'));
 }
+async function jStatus(on) {
+  const ws = vWs(); if (!ws) return;
+  const out = await post('/api/claims/jev/status', { ws, status_read: on });
+  reload('claims'); jPlan = null;
+  jSay(out && out.ok ? (on ? 'reading documents for status is on — the next check records the documents that exist and sends nothing for them; after that only changed documents are sent' : 'reading documents for status is off') : ((out && out.error) || 'not saved'));
+}
 async function jResume() {
   const ws = vWs(); if (!ws) return;
   const out = await post('/api/claims/jev/resume', { ws });
@@ -648,7 +812,11 @@ function jevBlock(ws, d) {
   kids.push(el('div', { 'data-r24': 'explain', style: NOTE }, iv('Where the local checks cannot decide, a pair of short texts (a requirement and the rubric’s dimensions, a story section and the requirement it cites, a sentence and the decision it cites, two sections of one PRD) can be put to Jev. Only pairs that changed are sent. Jev’s answers are suspicions: advisory, shown in words, and never counted as problems.')));
   kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;flex-wrap:wrap;padding:2px 14px 6px' },
     el('button', { style: BTN + (j.switch ? ';color:var(--foreground);border-color:var(--foreground)' : ''), hover: 'color:var(--foreground)', 'data-r24': 'switch', onClick: () => jSwitch(!j.switch) },
-      iv(j.switch ? 'check with Jev automatically: ON — turn off' : 'check with Jev automatically: OFF — turn on'))));
+      iv(j.switch ? 'check with Jev automatically: ON — turn off' : 'check with Jev automatically: OFF — turn on')),
+    el('button', { style: BTN + (j.status_read ? ';color:var(--foreground);border-color:var(--foreground)' : ''), hover: 'color:var(--foreground)', 'data-r24': 'status-switch',
+      title: 'Also send the documents tied to each initiative (its PRD, rubric and brief, and notes that name it), to ask whether a document is a stub and whether a note contradicts a requirement. The answers are suggestions in the initiative detail. Needs the switch on the left.',
+      onClick: () => jStatus(!j.status_read) },
+      iv(j.status_read ? 'read documents for status: ON — turn off' : 'read documents for status: OFF — turn on'))));
   kids.push(el('div', { 'data-r24': 'gate', 'data-open': j.open ? 'yes' : 'no', style: NOTE + (j.open ? '' : ';color:var(--foreground)') },
     iv(j.open ? 'Active: a key is saved, validation is on for this repository, automatic checking is on and this switch is on.' : 'Not active: ' + j.sentence + ' All four must hold before anything is sent.')));
   if (j.paused) kids.push(el('div', { 'data-r24': 'paused', style: 'display:flex;gap:8px;align-items:center;padding:2px 14px 6px;font-size:11.5px;color:var(--destructive)' },
@@ -810,6 +978,35 @@ export function validateList() {
 /* ── Today (proactive-engine D1): what is waiting, computed from the ledgers, with feedback ─────────────────────── */
 let tNote = null;
 let tMissed = false;
+/* The text inputs of the Today list are kept across redraws. The list is drawn again on every state change (the 4-second retry of
+ * an unread repository is one), and a new input on each draw threw away what the person was typing and the caret with it. Each
+ * input is made once; a redraw writes the stored value into it only while the person has not edited it since the last save. */
+const tInputs = new Map();
+function tInput(key, attrs, stored) {
+  let n = tInputs.get(key);
+  if (!n) {
+    n = el('input', { ...attrs, onInput: (e) => { e.target.dataset.dirty = '1'; } });
+    tInputs.set(key, n);
+  }
+  if (!n.dataset.dirty && n.value !== stored) n.value = stored;
+  // The redraw moves the input into a new row, and moving a node blurs it: give the focus and the caret back once the draw is done.
+  if (typeof document !== 'undefined' && document.activeElement === n) {
+    let a = null, b = null;
+    try { a = n.selectionStart; b = n.selectionEnd; } catch { /* a number input has no selection */ }
+    queueMicrotask(() => {
+      if (!n.isConnected || document.activeElement === n) return;
+      n.focus();
+      try { if (a !== null) n.setSelectionRange(a, b); } catch { /* as above */ }
+    });
+  }
+  return n;
+}
+/** Save a settings patch from kept inputs; on success their text is the stored value again. */
+async function tSaveFrom(inputs, patch) {
+  const out = await tSettings(patch);
+  if (out && out.ok !== false) for (const n of inputs) delete n.dataset.dirty;
+  setState({});
+}
 let tRetry = null;
 const tSay = (tone, text) => { tNote = { tone, text }; setState({}); };
 const OWNER_WORD = { pm: 'yours', agent: 'the agent\'s' };
@@ -834,6 +1031,7 @@ async function tSettings(patch) {
   const out = await post('/api/proactive/settings', patch);
   reload('today'); reload('brief'); reload('prepare'); reload('learn');
   tSay(!out || out.ok === false ? 'bad' : 'ok', !out || out.ok === false ? ((out && out.error) || 'not saved') : 'saved');
+  return out;
 }
 function tCard(c) {
   const b = (label, attr, fn) => el('button', { style: BTN, hover: 'color:var(--foreground)', 'data-r22': attr, onClick: fn }, label);
@@ -1011,7 +1209,7 @@ function learningSection() {
   kids.push(el('div', { style: NOTE, 'data-r26': 'last-run' }, iv(last ? 'Last weekly review: ' + String(last.at).replace('T', ' ').slice(0, 16) + ' UTC. It runs about every 7 days, only while the workspace server runs, and does not catch up.' : 'No weekly review has run yet. It runs about every 7 days once there is feedback, only while the workspace server runs, and does not catch up.')));
   kids.push(el('div', { style: 'display:flex;gap:4px;flex-wrap:wrap;padding:2px 14px 6px' },
     b('run the weekly review now', 'run', () => lPost('/api/proactive/learn/run', {}, (o) => 'review taken — ' + o.proposed.length + ' lesson(s) proposed'), 'refused while learning is paused'),
-    b('forget what it learned', 'reset', () => lPost('/api/proactive/learn/reset', { confirm: true }, 'reset — earlier feedback is ignored from now on, not deleted'), 'earlier feedback stops counting; nothing is deleted')));
+    b('forget what it learned', 'reset', () => { if (typeof window !== 'undefined' && window.confirm && !window.confirm('Forget what it learned? Earlier feedback stops counting. Nothing is deleted.')) return; lPost('/api/proactive/learn/reset', { confirm: true }, 'reset — earlier feedback is ignored from now on, not deleted'); }, 'earlier feedback stops counting; nothing is deleted')));
   return kids;
 }
 export function todayList() {
@@ -1024,9 +1222,9 @@ export function todayList() {
     el('button', { style: BTN, hover: 'color:var(--foreground)', title: 'read the brief and the list again', 'data-r22': 'refresh', onClick: () => { reload('today'); reload('brief'); reload('learn'); setState({}); } }, '↻ refresh'),
     el('button', { style: BTN, hover: 'color:var(--foreground)', 'data-r22': 'missed', onClick: () => { tMissed = !tMissed; setState({}); } }, 'something was missed')));
   if (tMissed) {
-    const ta = el('input', { type: 'text', placeholder: 'what should it have shown you?', 'data-r22': 'missed-input', style: 'flex:1;min-width:0;font-size:11.5px;padding:4px 8px;border:1px solid var(--border);background:var(--background);color:var(--foreground);min-height:28px' });
+    const ta = tInput('missed', { type: 'text', placeholder: 'what should it have shown you?', 'data-r22': 'missed-input', style: 'flex:1;min-width:0;font-size:11.5px;padding:4px 8px;border:1px solid var(--border);background:var(--background);color:var(--foreground);min-height:28px' }, '');
     kids.push(el('div', { style: 'display:flex;gap:4px;padding:2px 14px 6px' }, ta,
-      el('button', { style: BTN, hover: 'color:var(--foreground)', 'data-r22': 'missed-save', onClick: async () => { await tFeedback({ verdict: 'missed', note: ta.value }, 'recorded as missed'); if (tNote && tNote.tone === 'ok') tMissed = false; setState({}); } }, 'save')));
+      el('button', { style: BTN, hover: 'color:var(--foreground)', 'data-r22': 'missed-save', onClick: async () => { await tFeedback({ verdict: 'missed', note: ta.value }, 'recorded as missed'); if (tNote && tNote.tone === 'ok') { tMissed = false; delete ta.dataset.dirty; } setState({}); } }, 'save')));
   }
   if (tNote) kids.push(el('div', { 'data-r22': 'note', style: NOTE + ';color:' + (tNote.tone === 'bad' ? 'var(--destructive)' : tNote.tone === 'ok' ? 'var(--foreground)' : 'var(--muted-foreground)') }, iv(tNote.text)));
   const st = status(r, 'Today');
@@ -1053,16 +1251,16 @@ export function todayList() {
   kids.push(el('div', { style: LABEL }, iv('morning brief')));
   for (const k of Object.keys(s.briefSources || {})) kids.push(chk('read ' + k + (k === 'mail' ? ' (titles and senders only)' : ''), s.briefSources[k], 'set-src-' + k, (v) => ({ briefSources: { [k]: v } })));
   kids.push(chk('typed scoring (sends titles and metadata to the Typesafe service)', s.briefScoring, 'set-scoring', (v) => ({ briefScoring: v })));
-  const nm = el('input', { type: 'text', maxlength: '40', placeholder: 'your name for the greeting', value: s.name || '', 'data-r22': 'set-name', style: 'width:180px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' });
-  kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;padding:4px 14px;font-size:12px' }, nm, el('button', { style: BTN, 'data-r22': 'set-name-save', onClick: () => tSettings({ name: nm.value }) }, 'save')));
+  const nm = tInput('name', { type: 'text', maxlength: '40', placeholder: 'your name for the greeting', 'data-r22': 'set-name', style: 'width:180px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' }, s.name || '');
+  kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;padding:4px 14px;font-size:12px' }, nm, el('button', { style: BTN, 'data-r22': 'set-name-save', onClick: () => tSaveFrom([nm], { name: nm.value }) }, 'save')));
   kids.push(...learningSection());
   kids.push(chk('pause learning (no weights, no lessons, nothing narrowed)', s.pauseLearning, 'set-pause', (v) => ({ pauseLearning: v })));
-  const days = el('input', { type: 'number', min: '1', max: '90', value: String(s.stalledDays), 'data-r22': 'set-stalled-days', style: 'width:60px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' });
-  const qs = el('input', { type: 'text', placeholder: '22:00', value: s.quiet ? s.quiet.start : '', 'data-r22': 'set-quiet-start', style: 'width:56px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' });
-  const qe = el('input', { type: 'text', placeholder: '07:00', value: s.quiet ? s.quiet.end : '', 'data-r22': 'set-quiet-end', style: 'width:56px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' });
+  const days = tInput('stalled', { type: 'number', min: '1', max: '90', 'data-r22': 'set-stalled-days', style: 'width:60px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' }, String(s.stalledDays));
+  const qs = tInput('quiet-start', { type: 'text', placeholder: '22:00', 'data-r22': 'set-quiet-start', style: 'width:56px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' }, s.quiet ? s.quiet.start : '');
+  const qe = tInput('quiet-end', { type: 'text', placeholder: '07:00', 'data-r22': 'set-quiet-end', style: 'width:56px;font-size:11.5px;padding:2px 6px;border:1px solid var(--border);background:var(--background);color:var(--foreground)' }, s.quiet ? s.quiet.end : '');
   kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;flex-wrap:wrap;padding:4px 14px;font-size:12px' }, iv('stalled after'), days, iv('days'),
-    el('button', { style: BTN, 'data-r22': 'set-stalled-save', onClick: () => tSettings({ stalledDays: Number(days.value) }) }, 'save')));
+    el('button', { style: BTN, 'data-r22': 'set-stalled-save', onClick: () => tSaveFrom([days], { stalledDays: Number(days.value) }) }, 'save')));
   kids.push(el('div', { style: 'display:flex;gap:6px;align-items:center;flex-wrap:wrap;padding:4px 14px 8px;font-size:12px' }, iv('quiet hours'), qs, iv('to'), qe,
-    el('button', { style: BTN, 'data-r22': 'set-quiet-save', onClick: () => tSettings({ quiet: qs.value.trim() || qe.value.trim() ? { start: qs.value.trim(), end: qe.value.trim() } : null }) }, 'save')));
+    el('button', { style: BTN, 'data-r22': 'set-quiet-save', onClick: () => tSaveFrom([qs, qe], { quiet: qs.value.trim() || qe.value.trim() ? { start: qs.value.trim(), end: qe.value.trim() } : null }) }, 'save')));
   return [el('div', { style: 'padding:2px 0 8px', 'data-r22': 'today' }, kids)];
 }