@galda/cli 0.10.46 → 0.10.48

package/app/index.html

@@ -2052,6 +2052,12 @@
      — no re-layout, no flash. */
   .redesign .rst.working{color:var(--green)}
   .redesign .rst.working .rdot{background:var(--green)}
+  /* A "Needs you" goal opened as this card reads its state, grounded in the flagship list's
+     state colours: a stopped/failed error is red (--danger); interrupted (resumable) is amber. */
+  .redesign .rst.err{color:var(--danger)}
+  .redesign .rst.err .rdot{background:var(--danger)}
+  .redesign .rst.int{color:var(--amber)}
+  .redesign .rst.int .rdot{background:var(--amber)}
   .sa-item.working{box-shadow:0 1px 2px rgba(20,23,31,.05),0 0 0 1px color-mix(in oklab,var(--green) 22%,var(--hair))}
   .replynote .wkline .wkdot{display:inline-block;width:7px;height:7px;border-radius:50%;background:var(--green);box-shadow:0 0 0 3px color-mix(in oklab,var(--green) 22%,transparent);margin-right:9px;vertical-align:middle;position:relative;top:-1px}
   .redesign .rtitle{font-family:var(--ui);font-size:18px;font-weight:680;letter-spacing:-.018em;line-height:1.3;margin:8px 0 0 !important;color:var(--ink)}
@@ -4664,7 +4670,7 @@ function renderTasks(){
       // inline expand.
       if (el.dataset.attn) {
         const g = goalOf(state.tasks.find((t) => t.id === id));
-        if (g) { goToGoalInChat(g.id); openGoalDetail(g.id); }
+        if (g) { goToGoalInChat(g.id); openGoalOrCard(g.id); }
         return;
       }
       state.expanded.has(id) ? state.expanded.delete(id) : state.expanded.add(id);
@@ -4686,7 +4692,7 @@ function renderTasks(){
         : goal;
       goToGoalInChat(opened.id);
       if (opened.status === 'review') openReviewChecklist(opened.id);
-      else openGoalDetail(opened.id);
+      else openGoalOrCard(opened.id);
     });
   }
   for (const b of $('tasklist').querySelectorAll('.reqopen')) {
@@ -5878,6 +5884,13 @@ function openGoalDetail(goalId){
   renderGoalDetail();
   $('goalDetailOverlay').classList.add('show');
 }
+// "Needs you" (attention) goals open as the SAME review card (Masa 2026-07-24, Option A);
+// any other non-review goal keeps the goal-detail overlay.
+function openGoalOrCard(goalId){
+  const g = state.goals.find((x) => x.id === goalId);
+  if (g && ['blocked', 'partial', 'failed', 'interrupted'].includes(g.status) && typeof saOpen === 'function') { saOpen(goalId, true); return; }
+  openGoalDetail(goalId);
+}
 function closeGoalDetail(){
   $('goalDetailOverlay').classList.remove('show');
   const goalId = state.goalDetailOpen;
@@ -6068,6 +6081,9 @@ async function sendDismiss(goalId, text){
   } catch { showErr('Dismiss failed.'); }
   $('gdApprove').disabled = false; $('gdDismiss').disabled = false; $('gdSend').disabled = false;
 }
+// Residual goal-detail overlay path (non-review, non-attention goals — a done/running goal
+// opened for a note). "Needs you" goals no longer come here: they open the review card
+// (openGoalOrCard → saOpen solo), where the reply is the in-place "Working on it" thread.
 async function sendGoalInstruction(goalId, text){
   $('gdSend').disabled = true; $('gdDiscard').disabled = true;
   try {
@@ -6424,6 +6440,10 @@ function fsToggleLog(taskId){
   toggleGoalDetail(task.goalId);
 }
 function toggleGoalDetail(goalId){
+  // "Needs you" (attention) goals open in the SAME review card, not the bespoke overlay
+  // (Masa 2026-07-24, Option A). Everything else keeps the goal-detail overlay.
+  const g = state.goals.find((x) => x.id === goalId);
+  if (g && ['blocked', 'partial', 'failed', 'interrupted'].includes(g.status)) { saOpen(goalId, true); return; }
   const overlay = $('goalDetailOverlay');
   if (state.goalDetailOpen === goalId && overlay.classList.contains('show')) closeGoalDetail();
   else openGoalDetail(goalId);
@@ -6794,6 +6814,13 @@ function renderFsTodo(){
   // 一時停止したゴール(blocked kind:'paused')は Doing に「Paused」1行で留める(Masa 2026-07-19)。
   // Needs you 側(failed/blocked)へは出さず、その走行/中断タスク行も抑止して二重行を作らない。
   const pausedGoalIds = new Set(pgoals.filter((g) => g.status === 'blocked' && g.blocked?.kind === 'paused').map((g) => g.id));
+  // A goal picked back up by a "Needs you" reply is `running` but may have no running WORK task
+  // (only a reply task, which Doing excludes) — it used to vanish from the lane (Masa 2026-07-24).
+  // Show it in Doing as a goal row, and stop its old stopped task from lingering under "Needs you".
+  const runningTaskGoalIds = new Set(running.filter((t) => !t.reply).map((t) => t.goalId));
+  const resumedGoalIds = new Set(pgoals.filter((g) => g.status === 'running' && g.startedAt).map((g) => g.id));
+  const doingGoals = pgoals.filter((g) => g.status === 'running' && g.startedAt
+    && !runningTaskGoalIds.has(g.id) && !pausedGoalIds.has(g.id));
   const tt = (s, n) => esc(String(s ?? '').replace(/\s+/g, ' ').slice(0, n ?? 70));
   // LATER = deliberately shelved goals (composer "Later" send → server status 'pending';
   // PRD §8.2 top gap — they were invisible in flagship). Staged disclosure: the cap
@@ -6815,7 +6842,10 @@ function renderFsTodo(){
     running.filter((t) => !pausedGoalIds.has(t.goalId)).map((t) => `<div class="drow"><div class="trow now"><span class="st run"></span>${noSpan(t.goalId)}<span class="tt">${tt(t.title)}</span><span class="pct num">${progressFor(t)}%</span>${fsActs(t.goalId)}</div>
       <div class="tbar"><i style="width:${progressFor(t)}%"></i></div></div>`).join('')
     // 一時停止したゴールは Doing に「Paused」1行(アンバー)で留める＋Resume。走行/中断行は上と needsRows で抑止済み。
-    + pgoals.filter((g) => pausedGoalIds.has(g.id)).map((g) => `<div class="drow"><div class="trow now"><span class="st run int"></span>${noSpan(g.id)}<span class="tt">${tt(g.reviewSummary?.check || g.plan?.[0] || g.text, 60)}</span><span class="stword int">Paused</span><button class="ab txt" data-fspause="1" title="Resume — 再開">Resume</button></div></div>`).join('');
+    + pgoals.filter((g) => pausedGoalIds.has(g.id)).map((g) => `<div class="drow"><div class="trow now"><span class="st run int"></span>${noSpan(g.id)}<span class="tt">${tt(g.reviewSummary?.check || g.plan?.[0] || g.text, 60)}</span><span class="stword int">Paused</span><button class="ab txt" data-fspause="1" title="Resume — 再開">Resume</button></div></div>`).join('')
+    // A goal picked back up by a reply (running, no running work task yet): show it here so it
+    // lands in Doing instead of vanishing. No % bar — nothing measurable is running yet.
+    + doingGoals.map((g) => `<div class="drow"><div class="trow now"><span class="st run"></span>${noSpan(g.id)}<span class="tt">${tt(g.reviewSummary?.check || g.plan?.[0] || g.text, 60)}</span><span class="stword" style="color:var(--green)">Working on it</span>${fsActs(g.id)}</div></div>`).join('');
   // NEEDS YOU: stopped/errored work that used to fold into Doing, now under its own caption so
   // it no longer reads as "still in flight" (Masa 2026-07-19). Three kinds share it, each
   // keeping the affordances it already had — nothing about the failed-task and blocked-goal
@@ -6832,7 +6862,7 @@ function renderFsTodo(){
     // both still call the same /retry endpoint, which already resumes the session when one
     // exists (server-side retry = "resume if possible, else re-run"; there is no separate
     // resume endpoint to call).
-    failed.filter((t) => !pausedGoalIds.has(t.goalId)).map((t) => { const isInt = t.status === 'interrupted';
+    failed.filter((t) => !pausedGoalIds.has(t.goalId) && !resumedGoalIds.has(t.goalId)).map((t) => { const isInt = t.status === 'interrupted';
       const word = isInt ? 'Interrupted' : 'Failed', icon = isInt ? '⏸' : '✗', label = isInt ? 'Resume' : 'Retry';
       // don't restate the state word as the "reason" too (e.g. "Interrupted / Interrupted")
       // when the task has no distinct result text — the coloured word already said it, so
@@ -6873,8 +6903,8 @@ function renderFsTodo(){
   // goals) + Next up (queued) + Later (pending goals). "· next-up order" stays keyed off
   // queued.length specifically, since that suffix describes the Next-up sublist's own sort,
   // not the lane total.
-  const needsCount = failed.length + blocked.length + needsInput.length;
-  const todoTotal = running.length + needsCount + queued.length + later.length;
+  const needsCount = failed.filter((t) => !resumedGoalIds.has(t.goalId)).length + blocked.length + needsInput.length;
+  const todoTotal = running.length + doingGoals.length + needsCount + queued.length + later.length;
   // Free-tier pending counter (H22): on the free plan the header shows the wall
   // — pending goals out of the 5-slot cap — reddening at the cap and gone once
   // paid. Reads the SAME numbers the gate enforces (state.billing, never a local
@@ -8481,7 +8511,7 @@ function closeSummaryPera(){ $('summaryOverlay').classList.remove('show'); }
 // 4 archived / 5 reverted — session-local; the server round-trip is what
 // settles it (see saSync's "returned to review" reset).
 // ============================================================================
-const SA = { open: false, items: [], diffs: {}, actLog: {}, msgs: {}, convOpen: new Set(), shotBox: {}, detOpen: new Set(), focusGid: null, focusScrolled: false, focusTimer: null, cur: null };
+const SA = { open: false, items: [], diffs: {}, actLog: {}, msgs: {}, convOpen: new Set(), shotBox: {}, detOpen: new Set(), focusGid: null, focusScrolled: false, focusTimer: null, cur: null, solo: null };
 window.SA = SA; // top-level const isn't a window property; external checks probe window.SA (same convention as window.state above)
 const SA_SVG = {
   ok: '<svg viewBox="0 0 24 24"><path d="M4 12l5 5L20 6"/></svg>',
@@ -8491,7 +8521,12 @@ const SA_SVG = {
 function saRows(){
   const gs = (state.goals || []).filter((g) => g.projectId === state.active);
   const ts = (state.tasks || []).filter((t) => t.projectId === state.active);
-  return buildReviewDigest({ goals: gs, tasks: ts }).map((r) => {
+  return buildReviewDigest({ goals: gs, tasks: ts }).map((r) => saRowFromDigest(r, gs, ts));
+}
+// One digest-row → one See-all card item. Extracted from saRows (2026-07-24) so a single
+// "Needs you" goal can be opened as the SAME review card (saLedgerRows solo mode) — the card's
+// whole shape (Request/Result/Proof/Activity, empty sections auto-hidden) is reused verbatim.
+function saRowFromDigest(r, gs, ts){
     const g = gs.find((x) => x.id === r.goalIds[0]);
     const pt = ts.find((t) => r.goalIds.includes(t.goalId) && t.proof && (t.proof.png || t.proof.gif));
     // "What we did" (H23 §2, replaces the old Plan/Review-thread folds): one bullet
@@ -8616,7 +8651,44 @@ function saRows(){
         history: goalTokenHistoryLocal(gs, ts, g?.id),
       }),
     };
-  });
+}
+// A "Needs you" (attention) goal, shaped as a digest row so saRowFromDigest builds it the same
+// way as a review card. No PR-merge/plan (those are review-only); everything else falls out of
+// the goal + its tasks exactly as buildReviewDigest does for a review goal.
+function saSoloDigestRow(g, ts){
+  const goalTasks = (ts ?? []).filter((t) => t.goalId === g.id);
+  const proofTask = goalTasks.find((t) => t.proof);
+  const checkLine = (g.reviewSummary?.check || g.plan?.[0] || g.text || '').replace(/\s+/g, ' ').trim().slice(0, 66);
+  return {
+    goalIds: [g.id], reviewNumber: goalReviewNumber({ goal: g, tasks: ts }), checkLine,
+    pr: g.pr ?? null, testResult: g.testResult ?? null,
+    proof: proofTask?.proof ?? null, proofTaskId: proofTask?.id ?? null, plan: false, planText: null,
+  };
+}
+// The See-all Ledger's cards. Normally the review digest (the "See all" batch stays review-only);
+// in solo mode (a single "Needs you" goal tapped) it is just that one goal's card, reusing the
+// exact review-card render (Masa 2026-07-24, Option A).
+function saLedgerRows(){
+  if (SA.solo != null) {
+    const gs = (state.goals || []).filter((g) => g.projectId === state.active);
+    const ts = (state.tasks || []).filter((t) => t.projectId === state.active);
+    const g = gs.find((x) => x.id === SA.solo);
+    if (!g) return [];
+    const row = saRowFromDigest(saSoloDigestRow(g, ts), gs, ts);
+    // A stuck goal: surface WHY it stopped and what to do in the card's Result — the same
+    // plain-language, language-following summary the old Attention panel showed
+    // (summarizeAttention), mapped into the review card instead of a bespoke "What happened /
+    // Your call" layout (Masa 2026-07-24). The Request section already shows the ask, so drop
+    // the summary's "必要だったこと:/Needed:" lead to avoid repeating it.
+    if (['blocked', 'partial', 'failed', 'interrupted'].includes(g.status)) {
+      const s = summarizeAttention(g, ts.filter((t) => t.goalId === g.id));
+      const why = String(s.happened || '').replace(/^\s*(必要だったこと:|Needed:)[^。.]*[。.]\s*/, '').trim();
+      const merged = [why, s.decision].filter(Boolean).join('\n\n');
+      if (merged) row.report = merged;
+    }
+    return [row];
+  }
+  return saRows();
 }
 // (P1, Masa 2026-07-21) saMetaRowHtml() was deleted here — the pre-redesign meta row,
 // uncalled since the review-card redesign. Its "Run locally" chip only ever opened a
@@ -8854,7 +8926,11 @@ function saWorkflowHtml(it){
   // merge→deploy, varying per person. There is no place to record such a pipeline yet,
   // so inventing one here would be this layer guessing someone's process. Left as a
   // decision, not faked.)
-  const steps = [['Implemented', 'ok'], ['Tested', ran ? 'ok' : 'todo'], ['In review', 'now'], ['Merge', 'todo']];
+  // A stopped / being-picked-back-up goal sits at Implemented — it has NOT reached review, so
+  // "In review" must not read as the current step (Masa 2026-07-24). Only a real review goal
+  // marks Implemented done (✓) and In review as now (→).
+  const atImplemented = it.st === 3 || ['blocked', 'partial', 'failed', 'interrupted', 'running'].includes(it.goal?.status);
+  const steps = [['Implemented', atImplemented ? 'now' : 'ok'], ['Tested', ran ? 'ok' : 'todo'], ['In review', atImplemented ? 'todo' : 'now'], ['Merge', 'todo']];
   return '<div class="rwf">' + steps.map(([s, st]) => `<div class="wf-s ${st}"><span class="mk">${st === 'ok' ? '✓' : st === 'now' ? '→' : '□'}</span>${s}</div>`).join('') + '</div>';
 }
 function saItemHtml(it, i){
@@ -9092,8 +9168,18 @@ function saItemHtml(it, i){
   // BELOW the box (the same .replynote surface the answer flow uses) — so sending only APPENDS
   // a line and never re-lays-out the card. It leaves Review only when the card is closed.
   const working = it.st === 3;
+  // A "Needs you" (attention) goal opened as this card reads its real state, not "In review"
+  // (Masa 2026-07-24). After a reply it becomes "Working on it" like any other card.
+  const gstatus = it.goal?.status;
+  const attn = !working && ['blocked', 'partial', 'failed', 'interrupted'].includes(gstatus);
+  // Match the flagship list's state colours (Masa's Interrupt=amber / Failed=red rule): an
+  // interrupted goal is amber+resumable, everything else stopped is a red error.
+  const attnInt = gstatus === 'interrupted';
+  const attnLabel = gstatus === 'interrupted' ? 'Interrupted' : gstatus === 'blocked' ? 'Blocked' : 'Failed';
   const rst = working
     ? '<span class="rst working"><span class="rdot"></span>Working on it</span>'
+    : attn
+    ? `<span class="rst ${attnInt ? 'int' : 'err'}"><span class="rdot"></span>${attnLabel}</span>`
     : '<span class="rst"><span class="rdot"></span>In review</span>';
   return `<div class="sa-item lg redesign${it.goalId === SA.cur ? ' cur' : ''}${working ? ' working' : ''}" data-gid="${it.goalId}">
     <div class="rhead"><span class="rtag">${it.no != null ? `#${it.no}` : ''}</span>${rst}</div>
@@ -9112,7 +9198,7 @@ function saItemHtml(it, i){
     ${conversationSec}
     ${activitySec}
     ${S('Workflow', saWorkflowHtml(it))}
-    <div class="sa-act"><button class="sa-app" data-saact="app:${i}">${SA_SVG.ok}Approve</button><button class="sa-dis" data-saact="dis:${i}">Dismiss</button><div class="sa-chatwrap"><input class="sa-chat" data-i="${i}" data-goal="${it.goalId}" placeholder="Reply, ask, or send back to the AI…"><button class="sa-send" data-saact="send:${i}" title="Send"><svg viewBox="0 0 24 24"><path d="M12 19V5M5 12l7-7 7 7"/></svg></button></div><button class="sa-ic" title="Archive" data-saact="arch:${i}">${SA_SVG.arch}</button><button class="sa-ic" title="Revert" data-saact="rev:${i}">${SA_SVG.trash}</button></div>
+    <div class="sa-act">${attn ? '' : `<button class="sa-app" data-saact="app:${i}">${SA_SVG.ok}Approve</button><button class="sa-dis" data-saact="dis:${i}">Dismiss</button>`}<div class="sa-chatwrap"><input class="sa-chat" data-i="${i}" data-goal="${it.goalId}" placeholder="Reply, ask, or send back to the AI…"><button class="sa-send" data-saact="send:${i}" title="Send"><svg viewBox="0 0 24 24"><path d="M12 19V5M5 12l7-7 7 7"/></svg></button></div><button class="sa-ic" title="Archive" data-saact="arch:${i}">${SA_SVG.arch}</button><button class="sa-ic" title="Revert" data-saact="rev:${i}">${SA_SVG.trash}</button></div>
   </div>`;
 }
 // Scroll the See-all body so a card's top edge (.rhead) sits at the top of the visible
@@ -9180,10 +9266,15 @@ function saRender(){
   const S = $('seeall'); if (!S) return;
   const pending = SA.items.filter((x) => x.st === 0);
   const green = pending.filter((x) => !x.testResult || x.testResult.ok !== false).length;
-  const c = `${pending.length} ready · ${green === pending.length ? 'all green' : `${pending.length - green} failing`}`;
+  // Solo mode = a single "Needs you" goal opened as this card. It is not a review batch, so the
+  // header drops "Review / N ready / all green" and the Approve/Dismiss hints (Masa 2026-07-24).
+  const solo = SA.solo != null;
+  const hdTitle = solo ? 'Needs you' : 'Review';
+  const c = solo ? '' : `${pending.length} ready · ${green === pending.length ? 'all green' : `${pending.length - green} failing`}`;
   const curIdx = pending.findIndex((x) => x.goalId === SA.cur);
-  const posLabel = pending.length && curIdx >= 0 ? `${curIdx + 1} / ${pending.length}` : '';
-  S.innerHTML = `<div class="sa-panel"><div class="sa-hd"><span class="t">Review</span><span class="c">${c}</span><span class="sa-count">${posLabel}</span><span class="keys">↑↓ MOVE · A APPROVE · D DISMISS · ESC CLOSE</span><button class="sa-x" data-saact="close:0">✕</button></div>
+  const posLabel = solo ? '' : (pending.length && curIdx >= 0 ? `${curIdx + 1} / ${pending.length}` : '');
+  const hdKeys = solo ? '↑↓ MOVE · ESC CLOSE' : '↑↓ MOVE · A APPROVE · D DISMISS · ESC CLOSE';
+  S.innerHTML = `<div class="sa-panel"><div class="sa-hd"><span class="t">${hdTitle}</span><span class="c">${c}</span><span class="sa-count">${posLabel}</span><span class="keys">${hdKeys}</span><button class="sa-x" data-saact="close:0">✕</button></div>
     <div class="sa-body">${SA.items.length ? SA.items.map((it, i) => saItemHtml(it, i)).join('') : '<div class="sa-empty">Nothing waiting for review.</div>'}</div></div>`;
   // Native <details> would collapse on every rebuild (SSE tick, approve, etc.) —
   // track what's open per goal+section in SA.detOpen (persists across re-renders)
@@ -9237,7 +9328,7 @@ function saSync(){
   if (!SA.open) return;
   const ae = document.activeElement;
   if (ae && $('seeall').contains(ae) && ae.tagName === 'INPUT') return;
-  const rows = saRows();
+  const rows = saLedgerRows();
   for (const row of rows) {
     const ex = SA.items.find((x) => x.goalId === row.goalId);
     // Preserve st===3 ("Working on it", 決定1-4): a reply held the card open in place; if a
@@ -9258,9 +9349,10 @@ async function saFetchDiffs(){
   }));
   if (SA.open) saSync();
 }
-function saOpen(focusGoalId){
+function saOpen(focusGoalId, solo = false){
   SA.open = true;
-  SA.items = saRows().map((r) => ({ ...r, st: 0 }));
+  SA.solo = solo ? focusGoalId : null;   // solo = a single "Needs you" goal opened as one card
+  SA.items = saLedgerRows().map((r) => ({ ...r, st: 0 }));
   SA.detOpen = new Set();
   // H14: focus a specific goal's card (single-review tap). Map the tapped goal
   // to the card that CONTAINS it (goals can group in the digest), then let
@@ -9289,7 +9381,7 @@ function saOpen(focusGoalId){
   saFetchDiffs();
   saEnhanceShotBoxes();
 }
-function saClose(){ SA.open = false; SA.focusGid = null; if (SA.focusTimer) { clearTimeout(SA.focusTimer); SA.focusTimer = null; } $('seeall').classList.remove('open'); }
+function saClose(){ SA.open = false; SA.solo = null; SA.focusGid = null; if (SA.focusTimer) { clearTimeout(SA.focusTimer); SA.focusTimer = null; } $('seeall').classList.remove('open'); }
 // Continuous review (Masa 2026-07-08): move a persistent "current" card with
 // ↑↓ and auto-advance to the next pending after each verdict — same screen, same
 // Ledger, so approving flows without hunting for the next one.
@@ -9369,7 +9461,10 @@ async function saSendReply(it, i, text){
   // Append the line to the card's own thread immediately — this is what the person sees, and
   // its `seq` fixes its order regardless of when any round trip returns.
   it.rthread = [...(it.rthread || []), { who: 'you', text, seq }];
-  const followUp = it.st === 3 || it.inflight;   // card already in conversation → another instruction
+  // A "Needs you" (attention) reply is a fix instruction, not a review verdict: it always goes
+  // straight to /reply (no answer/unsure routing) and picks the goal back up (Masa 2026-07-24).
+  const isAttn = ['blocked', 'partial', 'failed', 'interrupted'].includes(it.goal?.status);
+  const followUp = it.st === 3 || it.inflight || isAttn;   // card in conversation / attention → another instruction
 
   if (followUp) {
     it.st = 3; it.rstatus = 'working'; SA.cur = goalId;
@@ -9377,6 +9472,14 @@ async function saSendReply(it, i, text){
     try {
       const r = await fetch(withKey(`/api/goals/${goalId}/reply`), { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify({ text }) });
       if (!r.ok) throw new Error('reply failed');
+      if (isAttn) {
+        // A Needs-you reply picks the goal back up → it belongs in Doing, not lingering in
+        // "Needs you" or vanishing (Masa 2026-07-24). Flip it optimistically so the lane behind
+        // the card updates on close; the SSE 'goal' event reconciles the same value.
+        const g = state.goals.find((x) => x.id === goalId);
+        if (g) { g.status = 'running'; g.blocked = null; if (!g.startedAt) g.startedAt = new Date().toISOString(); }
+        render();   // saSync skips the focused card, so this only refreshes the lane behind it
+      }
     } catch {
       it.rthread = it.rthread.filter((m) => m.seq !== seq);   // roll back just this line
       if (!it.rthread.some((m) => m.who === 'you')) { it.st = 0; it.rstatus = null; }
@@ -9547,6 +9650,9 @@ document.addEventListener('keydown', (e) => {
     return saMove(e.key === 'ArrowDown' || e.key === 'j' ? 1 : -1);
   }
   if (!['a', 'A', 'd', 'D'].includes(e.key)) return;
+  // A "Needs you" card (solo) has no Approve/Dismiss — the buttons are hidden, so the
+  // keyboard shortcut must not fire an approve/reject on a stuck goal either (Masa 2026-07-24).
+  if (SA.solo != null) return;
   // Act on the CURRENT card (fall back to the top pending), then auto-advance
   // focus to the next pending — so A A A blitzes down the queue with tempo.
   const it = SA.items.find((x) => x.goalId === SA.cur && x.st === 0) || SA.items.find((x) => x.st === 0);

package/app/theme.css

@@ -21,7 +21,7 @@
   --tint:20,23,31;
   --hair:rgba(var(--tint),.10); --hair2:rgba(var(--tint),.16);
   --glass:rgb(255 255 255 / .72);
-  --green-rgb:10,154,106; --blue-rgb:56,116,214; --amber-rgb:176,120,20; --danger-rgb:200,44,44;
+  --green-rgb:10,154,106; --blue-rgb:56,116,214; --amber-rgb:170,128,40; --danger-rgb:200,44,44;
   --purple-rgb:150,90,220; --glow-rgb:120,150,230;
   --green:rgb(var(--green-rgb)); --amber:rgb(var(--amber-rgb)); --blue:rgb(var(--blue-rgb)); --danger:rgb(var(--danger-rgb));
   --iri:linear-gradient(112deg,#b794f6 0%,#7cc4ff 38%,#5ce0c0 72%,#ffd27a 100%);
@@ -44,7 +44,7 @@ body[data-theme="banff"]{
   --tint:255,255,255;
   --hair:rgba(var(--tint),.09); --hair2:rgba(var(--tint),.16);
   --glass:rgba(12,13,17,.6);
-  --green-rgb:79,215,156; --blue-rgb:124,196,255; --amber-rgb:224,180,92; --danger-rgb:240,115,111;
+  --green-rgb:79,215,156; --blue-rgb:124,196,255; --amber-rgb:228,198,120; --danger-rgb:240,115,111;
   --purple-rgb:183,148,246; --glow-rgb:146,180,255;
   --green:rgb(var(--green-rgb)); --amber:rgb(var(--amber-rgb)); --blue:rgb(var(--blue-rgb)); --danger:rgb(var(--danger-rgb));
   --on-accent:#0a0a0b; --invink:#0a0a0b;
@@ -62,7 +62,7 @@ body[data-theme="dark"],body[data-theme="midnight"],body[data-theme="cinema"]{
   --tint:255,255,255;
   --hair:rgba(var(--tint),.07); --hair2:rgba(var(--tint),.12);
   --glass:rgba(22,22,27,.55);
-  --green-rgb:52,211,153; --blue-rgb:124,196,255; --amber-rgb:224,180,92; --danger-rgb:255,120,120;
+  --green-rgb:52,211,153; --blue-rgb:124,196,255; --amber-rgb:228,198,120; --danger-rgb:255,120,120;
   --purple-rgb:183,148,246; --glow-rgb:146,180,255;
   --on-accent:#0a0a0b; --invink:#0a0a0b;
   --send-bg:var(--iri); --send-ink:#0a0a0b;
@@ -92,7 +92,7 @@ body[data-theme="glass"]{
   --tint:255,255,255;
   --hair:rgba(255,255,255,.16); --hair2:rgba(255,255,255,.24);
   --glass:rgba(12,13,17,.6);
-  --green-rgb:102,230,198; --blue-rgb:143,208,255; --amber-rgb:230,192,106; --danger-rgb:255,138,134;
+  --green-rgb:102,230,198; --blue-rgb:143,208,255; --amber-rgb:232,206,134; --danger-rgb:255,138,134;
   --purple-rgb:183,148,246; --glow-rgb:146,180,255;
   --green:rgb(var(--green-rgb)); --amber:rgb(var(--amber-rgb)); --blue:rgb(var(--blue-rgb)); --danger:rgb(var(--danger-rgb));
   --on-accent:#0a0a0b; --invink:#0a0a0b;
@@ -117,7 +117,7 @@ body[data-theme="gradient"]{
   --tint:20,23,31;
   --hair:rgba(var(--tint),.10); --hair2:rgba(var(--tint),.16);
   --glass:rgb(255 255 255 / .72);
-  --green-rgb:79,215,156; --blue-rgb:37,99,235; --amber-rgb:184,134,11; --danger-rgb:240,115,111;
+  --green-rgb:79,215,156; --blue-rgb:37,99,235; --amber-rgb:178,140,40; --danger-rgb:240,115,111;
   --purple-rgb:150,90,220; --glow-rgb:120,150,230;
   --green:rgb(var(--green-rgb)); --amber:rgb(var(--amber-rgb)); --blue:rgb(var(--blue-rgb)); --danger:rgb(var(--danger-rgb));
   --on-accent:#0a0a0b; --invink:#ffffff;

package/engine/lib.mjs

@@ -1086,7 +1086,15 @@ export function parseReviewSummary(raw, fallback = {}) {
   };
 }
 
-const DOC_ONLY_RE = /(^|\/)(README|START-HERE|CHANGELOG|LICENSE)(\.[\w.-]+)?$|(^|\/)docs\/.*\.(md|mdx|txt)$|\.mdx?$|\.txt$/i;
+// A change is "doc-only" (skip the project test suite) when every file is
+// documentation. EVERYTHING under docs/ counts — not just .md: docs/ holds design
+// mockups (docs/design/**/*.html), handoffs, and images that the engine's tests
+// never import, so running the 58-file suite for them is pure waste. Before this,
+// only docs/**.md|mdx|txt was excused, so a CDO design-lab goal editing a single
+// docs/design/*.html re-ran the whole engine suite in its worktree and sat in
+// `running` ("Working on it") for 20+ minutes on a busy machine (dogfood 2026-07-24,
+// goal-632). Plus top-level README/CHANGELOG/LICENSE and any *.md/*.txt anywhere.
+const DOC_ONLY_RE = /(^|\/)(README|START-HERE|CHANGELOG|LICENSE)(\.[\w.-]+)?$|(^|\/)docs\/|\.mdx?$|\.txt$/i;
 export function hasTestRelevantChanges(changedFiles) {
   const files = (changedFiles ?? []).map((f) => String(f ?? '').trim()).filter(Boolean);
   if (!files.length) return true;
@@ -4126,6 +4134,30 @@ export function licenseTokenIdentity(token) {
   } catch { return ''; }
 }
 
+// Decode a license token's payload (unverified — signature is checked elsewhere,
+// offline via verifyLicenseToken with the baked-in public key). Returns null when
+// it can't be decoded. Used to read exp/iat/isPaying for the refresh scheduler.
+export function licenseTokenPayload(token) {
+  if (!token || typeof token !== 'string') return null;
+  const parts = token.split('.');
+  if (parts.length !== 3) return null;
+  try { return JSON.parse(Buffer.from(parts[1], 'base64url').toString('utf8')); }
+  catch { return null; }
+}
+
+// License tokens are short-lived (7 days) BY DESIGN — the engine is meant to fetch
+// a fresh one while online. Before this scheduler that fetch never happened, so a
+// paying user was silently logged out every 7 days (relay rejects the expired
+// token → the app shows "run npx @galda/cli" forever; dogfood 2026-07-24). Refresh
+// once the token is within `refreshWithinMs` of expiry (default 2 days) — INCLUDING
+// already-expired, so an agent that was offline past expiry still recovers on its
+// own without a manual sign-in (the Worker bounds how stale it may be). A token far
+// from expiry, or one that can't be decoded (not ours / malformed), is left alone.
+export function shouldRefreshLicense({ exp, now = Date.now(), refreshWithinMs = 2 * 24 * 3600 * 1000 } = {}) {
+  if (typeof exp !== 'number' || !Number.isFinite(exp)) return false;
+  return (exp * 1000 - now) < refreshWithinMs;
+}
+
 // Pure reducer for the relay-client's reconnect decision. Returns the next state
 // and whether to dial the relay now. State: { stoodDown, standingIdentity }.
 // Events:

package/engine/server.mjs

@@ -21,7 +21,7 @@ import { homedir } from 'node:os';
 import { fileURLToPath } from 'node:url';
 import { pathToFileURL } from 'node:url';
 import { needsProjectFolder, folderLabel, chooseFolderScript, parseChosenFolder, buildFolderQuestion, resolveFolderAnswer, classifyFolderTarget, buildFolderInitConfirm, isFolderInitConfirmed, needsReviewPreference, buildReviewPreferenceQuestion, resolveReviewPreferenceAnswer, routeQuestionAnswer, notUnderstoodNote, parseRelocalizedQuestion, classifyAuthProbe, authBannerText, makeSigninChallenge, parseClaimResponse } from './lib.mjs';
-import { parseStreamEvents, parseCodexEvents, buildCodexArgs, parsePlan, refinePlanTasks, canEditGoal, canDeleteGoal, shouldAskClarification, workerExitReason, isForcedStop, taskStatusAfterVerify, workerResultText, taskCountsAsComplete, resolveWantsPR, resolveGoalSource, buildGoalSummary, buildGoalProofMd, buildGoalPrBody, buildReviewSummaryPrompt, parseReviewSummary, buildReviewRuleSection, nextGoalStatus, isPrMerged, isPrApproved, reviewToDoneStatus, advanceReviewGoal, revertReviewGoal, approveGoal, dismissGoal, rejectGoal, reopenGoal, permissionModeFor, isPlanReview, nextAfterPlanApprove, GOAL_MODES, parseSkillFrontmatter, collectSkills, retestGoal, cancelRetestGoal, computeRetestOutcome, revertGoal, planRevertActions, archiveGoal, unarchiveGoal, undismissGoal, parseNumstat, truncateDiffText, sumUsage, resolveEntryUrl, parseGitLog, diffNewCommits, replayQueueLog, reconcileOrphanGoals, reorderQueue, sortQueueByPriority, TASK_PRIORITIES, validateWorkflowColumns, DEFAULT_WORKFLOW_COLUMNS, validateReviewDefinition, DEFAULT_REVIEW_DEFINITION, buildEphemeralSeedLog, buildEphemeralSeedProjects, trimTaskActivityForState, buildAskContext, WORKER_TOOLS, workerPrompt, verifyCfAccessJwt, resolveIdentity, goalVisibleTo, clampParallelLimit, canStartMore, nextRunnableTasks, goalsConflict, detectConflicts, pickFoldTarget, hasTestRelevantChanges, parseFailingTestNames, classifyTestGate, isInconclusiveTestRun, countInfraFlakes, classifyRunFailures, classifyVerifyGate, buildExecutionPlan, buildContextHandoffSummary, buildFailurePostmortem, appendFailureMemory, latestFailurePolicy, classifyChangeRisk, checkRunBudget, usageBudgetTokens, isRateLimited, nextResumeDelay, checkFreeTierLimit, resolveEntitlement, resolveCachedEntitlement, FREE_TIER_LIMITS, QUEUED_GOAL_STATUSES, verifyLicenseToken, shouldEmitSetupCompleted, pickAnalyticsUid, shouldEmitFreeExhausted, detectRequestLanguage, isNothingVerifiable, classifyComposerIntentHeuristic, detectPauseIntent, buildIntentPrompt, parseIntentResponse, buildBoardSnapshot, validateBoardSnapshot, boardIsEmpty, decideBoardPull, resolveConnectedAgents, pickAvailableAgent, pickUtilityAgent, utilityModel, liveTakeoverDecision, shouldOpenBrowserOnBoot, goalTaskTitle, buildGoalTask, RUN_DECL_FILE, parseRunDeclaration, SHOT_DIR, collectShotNames, parseGoalAddress, REPLY_OUTCOMES, REPLY_INTENTS, buildReplyIntentPrompt, parseReplyIntent, buildGoalMessage, serializeGoalMessage, parseGoalMessages } from './lib.mjs';
+import { parseStreamEvents, parseCodexEvents, buildCodexArgs, parsePlan, refinePlanTasks, canEditGoal, canDeleteGoal, shouldAskClarification, workerExitReason, isForcedStop, taskStatusAfterVerify, workerResultText, taskCountsAsComplete, resolveWantsPR, resolveGoalSource, buildGoalSummary, buildGoalProofMd, buildGoalPrBody, buildReviewSummaryPrompt, parseReviewSummary, buildReviewRuleSection, nextGoalStatus, isPrMerged, isPrApproved, reviewToDoneStatus, advanceReviewGoal, revertReviewGoal, approveGoal, dismissGoal, rejectGoal, reopenGoal, permissionModeFor, isPlanReview, nextAfterPlanApprove, GOAL_MODES, parseSkillFrontmatter, collectSkills, retestGoal, cancelRetestGoal, computeRetestOutcome, revertGoal, planRevertActions, archiveGoal, unarchiveGoal, undismissGoal, parseNumstat, truncateDiffText, sumUsage, resolveEntryUrl, parseGitLog, diffNewCommits, replayQueueLog, reconcileOrphanGoals, reorderQueue, sortQueueByPriority, TASK_PRIORITIES, validateWorkflowColumns, DEFAULT_WORKFLOW_COLUMNS, validateReviewDefinition, DEFAULT_REVIEW_DEFINITION, buildEphemeralSeedLog, buildEphemeralSeedProjects, trimTaskActivityForState, buildAskContext, WORKER_TOOLS, workerPrompt, verifyCfAccessJwt, resolveIdentity, goalVisibleTo, clampParallelLimit, canStartMore, nextRunnableTasks, goalsConflict, detectConflicts, pickFoldTarget, hasTestRelevantChanges, parseFailingTestNames, classifyTestGate, isInconclusiveTestRun, countInfraFlakes, classifyRunFailures, classifyVerifyGate, buildExecutionPlan, buildContextHandoffSummary, buildFailurePostmortem, appendFailureMemory, latestFailurePolicy, classifyChangeRisk, checkRunBudget, usageBudgetTokens, isRateLimited, nextResumeDelay, checkFreeTierLimit, resolveEntitlement, resolveCachedEntitlement, FREE_TIER_LIMITS, QUEUED_GOAL_STATUSES, verifyLicenseToken, licenseTokenPayload, shouldRefreshLicense, shouldEmitSetupCompleted, pickAnalyticsUid, shouldEmitFreeExhausted, detectRequestLanguage, isNothingVerifiable, classifyComposerIntentHeuristic, detectPauseIntent, buildIntentPrompt, parseIntentResponse, buildBoardSnapshot, validateBoardSnapshot, boardIsEmpty, decideBoardPull, resolveConnectedAgents, pickAvailableAgent, pickUtilityAgent, utilityModel, liveTakeoverDecision, shouldOpenBrowserOnBoot, goalTaskTitle, buildGoalTask, RUN_DECL_FILE, parseRunDeclaration, SHOT_DIR, collectShotNames, parseGoalAddress, REPLY_OUTCOMES, REPLY_INTENTS, buildReplyIntentPrompt, parseReplyIntent, buildGoalMessage, serializeGoalMessage, parseGoalMessages } from './lib.mjs';
 import { createSerialQueue } from './lib.mjs';
 import { collectProjectRules } from './lib.mjs';
 import { openPR } from './pr.mjs';
@@ -293,6 +293,46 @@ async function activateLicense(token) {
   return { ok: true, email: result.payload.email };
 }
 
+// Keep the short-lived (7-day) license token fresh WHILE ONLINE so a paying user
+// is never silently logged out when it expires. The token can only otherwise be
+// re-minted by a full Google sign-in, so before this a $12/mo customer was
+// dropped to "run npx @galda/cli" every 7 days (relay rejects the expired token;
+// dogfood 2026-07-24). When the on-disk token is within ~2 days of expiry (or
+// already expired but still refreshable), ask billing-api for a fresh one and
+// persist it through activateLicense — which verifies the new token OFFLINE with
+// the baked key before writing, so a bad/forged response can't wedge us. The
+// relay-client watches license.token and reconnects the instant it changes, so
+// the refresh re-authenticates the relay with NO user action. Best-effort: any
+// failure (offline / 401 too-stale → needs sign-in / misconfig) leaves the
+// current token untouched and retries next tick.
+async function refreshLicenseIfNeeded() {
+  try {
+    if (!BILLING_API_URL || !existsSync(licenseFile)) return;
+    const token = readFileSync(licenseFile, 'utf8').trim();
+    const payload = licenseTokenPayload(token);
+    if (!payload || !shouldRefreshLicense({ exp: payload.exp })) return;
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(), 10000);
+    let res;
+    try {
+      res = await fetch(`${BILLING_API_URL}/license/refresh`, {
+        method: 'POST', headers: { 'content-type': 'application/json' },
+        body: JSON.stringify({ token }), signal: controller.signal,
+      });
+    } finally { clearTimeout(timer); }
+    if (!res.ok) return; // 401 (too stale → a real sign-in is needed), 501 misconfig, 5xx — retry later
+    const body = await res.json().catch(() => null);
+    if (!body?.token || body.token === token) return; // nothing new
+    const r = await activateLicense(body.token);
+    if (r.ok) console.log(`[manager] license renewed for ${r.email} before expiry (no sign-in needed)`);
+  } catch { /* best-effort — keep the current token, retry next tick */ }
+}
+// Check on boot and twice a day. A 7-day token with a 2-day refresh window is
+// renewed ~5 days in, long before the relay would reject it, as long as the
+// engine is running at any point in that window.
+refreshLicenseIfNeeded();
+setInterval(refreshLicenseIfNeeded, 12 * 60 * 60 * 1000).unref?.();
+
 // The HTML shown in the popup after Google sign-in. It notifies the app window
 // (postMessage; the app validates origin+shape and refreshes) and closes
 // itself. Everything is inline + self-contained — this page can be opened

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@galda/cli",
-  "version": "0.10.46",
+  "version": "0.10.48",
   "type": "module",
   "description": "Galda - hand off work to Claude Code or Codex, get proof back. Runs on your existing subscription, no extra API cost.",
   "scripts": {