@galda/cli 0.10.47 → 0.10.49

package/app/index.html

@@ -1329,13 +1329,17 @@
     color:var(--ink2);font:inherit;font-size:12.5px;white-space:nowrap;overflow:hidden;cursor:pointer;text-align:left;width:100%;position:relative}
   #fsRoot .fsproj:hover{background:var(--hover-s)}
   #fsRoot .fsproj.on{background:var(--hover);color:var(--ink)}
-  #fsRoot .fsproj i{width:4px;height:4px;border-radius:50%;background:var(--ink3);flex:0 0 4px}
+  #fsRoot .fsproj i{width:4px;height:4px;border-radius:50%;background:var(--ink3);flex:0 0 4px;position:relative;top:1.5px} /* top:1.5px = optical nudge so the dot/running-ring centre sits on the name's x-height, not the line-box centre (Masa 2026-07-24, rail=D). Same shift for idle dot + running ring keeps the column aligned. */
   #fsRoot .fsproj.on i{background:var(--green)}
   /* running project = solid saturated green dot with a breathing glow — never dims to
      grey, independent of selection (HANDOFF-v45 §11, flagship .proj.run port) */
-  #fsRoot .fsproj.run i{background:var(--green);box-shadow:0 0 0 0 color-mix(in srgb,var(--green) 55%,transparent);animation:fsrundot 2s ease-in-out infinite}
+  /* running project = a spinning broken green ring (matches the Doing marker · flagship #316).
+     Was a pulsing dot (fsrundot). Centre WITHOUT transform (spin's rotate would override a centring
+     translate → drift bottom-right) via top/left:50% + negative-half margin; concentric with the 4px dot. */
+  #fsRoot .fsproj.run i{background:none;box-shadow:none;position:relative}
+  #fsRoot .fsproj.run i::after{content:"";position:absolute;top:50%;left:50%;width:7px;height:7px;margin:-3.5px 0 0 -3.5px;box-sizing:border-box;border-radius:50%;border:1.3px solid #22c55e;border-top-color:transparent;animation:spin 1.8s linear infinite}
   @keyframes fsrundot{0%,100%{box-shadow:0 0 0 0 color-mix(in srgb,var(--green) 50%,transparent)}50%{box-shadow:0 0 5px 1.5px color-mix(in srgb,var(--green) 35%,transparent)}}
-  @media (prefers-reduced-motion:reduce){#fsRoot .fsproj.run i{animation:none}}
+  @media (prefers-reduced-motion:reduce){#fsRoot .fsproj.run i::after{animation:none}}
   #fsRoot .fsproj .pname{min-width:0;overflow:hidden;text-overflow:ellipsis}
   #fsRoot .fsproj .num{margin-left:auto;font-size:10px;color:var(--ink3);flex:0 0 auto}
   /* per-project counts (HANDOFF-v45 §10): To Do = rounded-square icon, Review = Scan
@@ -2276,10 +2280,13 @@
   #fsRoot .trow .tt{color:color-mix(in srgb,var(--ink) 84%,var(--panel))} /* §13.2 soften To Do row name (not pure black); left chat feed keeps --ink */
   #fsRoot .trow:hover{background:var(--hover-s)}
   #fsRoot .trow.now{font-weight:500}
-  #fsRoot .trow .st{width:13px;height:13px;border-radius:50%;border:1.5px solid var(--hair2);flex:0 0 13px}
-  #fsRoot .trow .st.run{border-color:var(--green);background:radial-gradient(circle,rgba(var(--green-rgb),.4),transparent 70%)}
-  #fsRoot .trow .st.run.bad{border-color:var(--danger);background:radial-gradient(circle,rgba(var(--danger-rgb),.42),transparent 70%)}
-  #fsRoot .trow .st.run.int{border-color:var(--amber);background:radial-gradient(circle,rgba(var(--amber-rgb),.4),transparent 70%)}
+  #fsRoot .trow .st{width:13px;height:13px;border-radius:50%;border:1.5px solid var(--hair2);flex:0 0 13px;position:relative;top:1px} /* top:1px = optical nudge so the whole status column (To Do/Doing/Failed/Done) centres on the row text, not the line-box (Masa 2026-07-24, board=C) */
+  /* Doing marker = a broken green ring that spins (pattern B · flagship #316). Running only —
+     .bad (failed) / .int (paused) stay static: no dead motion (DESIGN-RULES §5.5). Reuses @keyframes spin. */
+  #fsRoot .trow .st.run{border-color:var(--green);border-top-color:transparent;background:none;box-shadow:0 0 6px color-mix(in srgb,var(--green) 40%,transparent);animation:spin 1.8s linear infinite}
+  #fsRoot .trow .st.run.bad{border-color:var(--danger);background:radial-gradient(circle,rgba(var(--danger-rgb),.42),transparent 70%);animation:none;box-shadow:none}
+  #fsRoot .trow .st.run.int{border-color:var(--amber);background:radial-gradient(circle,rgba(var(--amber-rgb),.4),transparent 70%);animation:none;box-shadow:none}
+  @media (prefers-reduced-motion:reduce){#fsRoot .trow .st.run{animation:none}}
   /* Doing-failure state word (HANDOFF-v45 §11): colour TEXT only, no badge/pill surface —
      Failed = red, Interrupted = amber. */
   #fsRoot .trow .stword{font-size:10.5px;font-weight:600;color:var(--danger);flex:0 0 auto}
@@ -2380,7 +2387,7 @@
   /* GOTCHA (HANDOFF-v45 §11): the rule above wins over a plain .fsproj.run i by
      specificity and greys the running dot out on the Gradient wallpaper rail — this
      exception restores solid green so "running" never dims to grey. */
-  body[data-theme="gradient"] #fsRoot .fsproj.run i{background:#22c55e}
+  body[data-theme="gradient"] #fsRoot .fsproj.run i{background:none}
   body[data-theme="gradient"] #fsRoot .fsproj .num,body[data-theme="gradient"] #fsRoot .fsproj .pcount{color:rgba(20,23,31,.6)}
   body[data-theme="gradient"] #fsRoot .fsproj:not(.on):hover{background:rgba(255,255,255,.34)}
   body[data-theme="gradient"] #fsRoot .fsproj.on{background:rgba(255,255,255,.5);color:var(--ink)}
@@ -2517,7 +2524,7 @@
   body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj{color:rgba(255,255,255,.85);text-shadow:0 1px 3px rgba(0,0,0,.35)}
   body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj i{background:rgba(255,255,255,.6)}
   /* same GOTCHA exception as the light Gradient rail, dark variant (HANDOFF-v45 §11) */
-  body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj.run i{background:#22c55e}
+  body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj.run i{background:none}
   body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj .num,body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj .pcount{color:rgba(255,255,255,.6)}
   body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj.on{background:rgba(255,255,255,.15);color:#fff}
   body[data-theme="gradient"][data-arcdark="1"] #fsRoot .fsproj:not(.on):hover{background:rgba(255,255,255,.1)}
@@ -3323,7 +3330,7 @@ const savedTheme = (() => {
   if (v !== s) localStorage.setItem('theme', v);
   return v;
 })();
-const state = { repos: null, repoHome: '', /* GET /api/repos — the folders a project can point at (Context Bar) */ projects: [], goals: [], tasks: [], act: {}, actAt: {}, todos: {}, chatTurns: {}, active: 'default', editing: null, attach: [], connectCommand: '', models: ['sonnet'], agentModels: null, agentEfforts: null, agents: ['claude-code', 'codex'], expanded: new Set(), openThreads: new Set(), queueOrder: {}, externalActivity: [], selectedGoal: null, workflowColumns: {}, wfDraft: null, reviewDefinitions: {}, rdDraft: null, connectedApp: localStorage.getItem('sel:agent') || 'claude-code', reviewOpenGoal: null, reviewApprovals: {}, goalDetailOpen: null, qreplyOpen: new Set(), logCollapsed: localStorage.getItem('logCollapsed') !== '0', logEntries: [], layout: localStorage.getItem('layout') || 'flagship', theme: savedTheme, boardskin: localStorage.getItem('boardskin') || 'auto', summaryPattern: Number(localStorage.getItem('summaryPattern')) || 3, peraSpot: 0, skill: null, skillsCache: null, projectRules: {}, rulesOpen: false,
+const state = { repos: null, repoHome: '', /* GET /api/repos — the folders a project can point at (Context Bar) */ projects: [], goals: [], tasks: [], act: {}, actAt: {}, todos: {}, chatTurns: {}, active: localStorage.getItem('sel:project') || 'default', editing: null, attach: [], connectCommand: '', models: ['sonnet'], agentModels: null, agentEfforts: null, agents: ['claude-code', 'codex'], expanded: new Set(), openThreads: new Set(), queueOrder: {}, externalActivity: [], selectedGoal: null, workflowColumns: {}, wfDraft: null, reviewDefinitions: {}, rdDraft: null, connectedApp: localStorage.getItem('sel:agent') || 'claude-code', reviewOpenGoal: null, reviewApprovals: {}, goalDetailOpen: null, qreplyOpen: new Set(), logCollapsed: localStorage.getItem('logCollapsed') !== '0', logEntries: [], layout: localStorage.getItem('layout') || 'flagship', theme: savedTheme, boardskin: localStorage.getItem('boardskin') || 'auto', summaryPattern: Number(localStorage.getItem('summaryPattern')) || 3, peraSpot: 0, skill: null, skillsCache: null, projectRules: {}, rulesOpen: false,
   // per-message send overrides set via slash-commands (reset after each send,
   // except model which is sticky in localStorage). palette = open command list.
   msg: { mode: null, model: null, effort: null, skill: null, later: false, review: false },
@@ -6142,6 +6149,13 @@ function render(){
   if (typeof SA !== 'undefined' && SA.open) saSync(); // keep the open See all Ledger live
   const bv = $('botver'); if (bv && state.uiVersion) bv.textContent = 'build ' + String(state.uiVersion).slice(0, 8);
   paintReplyNote();   // a reply's answer/question survives the re-render it caused
+  // Persist the selected project so a reload (a real UI ship, or any reconnect)
+  // restores the user's view instead of snapping back to 'default' (Masa
+  // 2026-07-24 "勝手にデフォルトのプロジェクトに戻される"). Only store an id that
+  // currently exists as a project, so onboarding/transient sentinels
+  // ('__onboard') are never saved and the restore always lands on a real one;
+  // an id that later disappears self-heals via the guards in refresh()/'projects'.
+  if (state.projects.some((p) => p.id === state.active)) localStorage.setItem('sel:project', state.active);
 }
 
 // ============================================================================
@@ -6188,7 +6202,7 @@ function renderFsRail(){
   const activeProject = state.projects.find((p) => p.id === state.active);
   const activeLabel = $('fsProjBtnName');
   if (activeLabel) activeLabel.textContent = activeProject?.name ?? 'Projects';
-  el.innerHTML = state.projects.map((p) => {
+  const html = state.projects.map((p) => {
     const ptasks = state.tasks.filter((t) => t.projectId === p.id);
     const pgoals = state.goals.filter((g) => g.projectId === p.id);
     // running dot (HANDOFF-v45 §11): a REAL running task, independent of selection —
@@ -6202,6 +6216,11 @@ function renderFsRail(){
     return `<button class="fsproj${p.id === state.active ? ' on' : ''}${running ? ' run' : ''}" data-fsproj="${p.id}"><i></i><span class="pname">${esc(p.name)}</span>${pcount}</button>`;
   }).join('')
     + `<button class="fsprojadd" id="fsProjAdd" title="New project"><span class="pa-plus"></span><span class="pa-label">New project</span></button>`;
+  // Skip the DOM swap when the rail is unchanged: re-creating .fsproj.run would restart the
+  // spinning ring's CSS animation — a visible "jump" on every SSE-driven re-render (Masa 2026-07-24).
+  if (html === renderFsRail._html && el.childElementCount) return;
+  renderFsRail._html = html;
+  el.innerHTML = html;
   for (const b of el.querySelectorAll('[data-fsproj]')) b.onclick = (e) => {
     if (e.target.closest('.pname.editing') || b.querySelector('.pname.editing')) return;
     state.active = b.dataset.fsproj; document.body.classList.remove('projectsopen'); render();

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;

package/engine/server.mjs

@@ -728,13 +728,22 @@ if (process.env.MANAGER_SEED_RUNNING_FIXTURE === '1') {
 // res -> identity (see identityFor), captured at connection time so every
 // SSE push can be scoped per-connection without re-deriving auth per event.
 const sseClients = new Map();
-// UI hot-reload: open tabs compare this stamp (index.html mtime + boot
-// time) via /api/state and the 30s SSE ping, and reload themselves when a
-// new UI ships — no manual reload after a hot-copy or a restart.
-const BOOT_STAMP = Date.now();
+// UI hot-reload: open tabs compare this stamp (index.html mtime only) via
+// /api/state and the 30s SSE ping, and reload themselves when a new UI ships
+// — no manual reload after a hot-copy.
+//
+// It used to also fold in a per-process BOOT_STAMP (Date.now() at boot) so a
+// bare restart would force a reload too. That backfired on the always-on
+// resident: every relay-supervisor respawn / hot-copy restart, and every
+// reconnect that landed on a *different* engine process (each with its own
+// BOOT_STAMP), changed uiVersion and yanked a `location.reload()` on the 30s
+// ping — the "reloads every 30-60s and resets to the default project" report
+// (Masa 2026-07-24). mtime alone means a reload fires ONLY when app/index.html
+// actually changes, and every process serving the same file agrees on the
+// stamp, so restarts and multi-process relay churn no longer reload the tab.
 function uiVersion() {
-  try { return `${Math.round(statSync(join(ROOT, 'app', 'index.html')).mtimeMs)}-${BOOT_STAMP}`; }
-  catch { return String(BOOT_STAMP); }
+  try { return String(Math.round(statSync(join(ROOT, 'app', 'index.html')).mtimeMs)); }
+  catch { return '0'; }
 }
 
 // Broadcast to every connected client, unscoped — only for events that carry
@@ -4330,7 +4339,16 @@ const server = createServer(async (req, res) => {
       try {
         const { text } = JSON.parse(body);
         if (!text?.trim()) return json(res, 400, { error: 'text required' });
-        if (['partial', 'failed', 'interrupted', 'blocked'].includes(goal.status)) {
+        // Reply picks a parked goal back up into the work queue. 'review' MUST be
+        // here: a completed goal awaiting your approve/reject is 'review', and a
+        // reply to it means "not done — keep working" (Masa 2026-07-24: replied to
+        // an in-review card and it sat stuck in Review with no worker activity).
+        // Without this flip the reply task was queued but the goal never left the
+        // Review lane, so nothing looked like it was happening. Un-parking mirrors
+        // the already-working 'blocked'/'failed'/etc. "Needs you" replies: the goal
+        // flips to running and, since it still carries startedAt from its first run,
+        // shows in Doing as "Working on it" — then the worker resumes its session.
+        if (['review', 'partial', 'failed', 'interrupted', 'blocked'].includes(goal.status)) {
           goal.status = 'running';
           goal.blocked = null;
           saveGoal(goal);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@galda/cli",
-  "version": "0.10.47",
+  "version": "0.10.49",
   "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": {