@galda/cli 0.10.79 → 0.10.80

package/app/index.html

@@ -6828,7 +6828,10 @@ function renderFsBoard(){
     // errored is status-based now (blocked/failed/partial/interrupted live in the Needs-you column,
     // not Doing). Doing holds only what's actively moving, so bkt==='doing' ⇒ healthy in flight.
     const errored = ['blocked', 'failed', 'partial', 'interrupted'].includes(g.status);
-    const intr = g.status === 'interrupted';
+    // `partial` can mean an earlier step completed and the latest one was
+    // interrupted. Read the concrete task so this card doesn't turn amber,
+    // resumable work into a red generic failure.
+    const intr = g.status === 'interrupted' || list.some((t) => t.goalId === g.id && t.status === 'interrupted');
     const isPaused = g.status === 'blocked' && g.blocked?.kind === 'paused'; // 一時停止=エラーでなくアンバー扱い
     const amber = intr || isPaused;
     const healthyRun = bkt === 'doing' && !errored; // paused(=errored) goals now sit in 'doing' too but carry no progress bar
@@ -7161,7 +7164,10 @@ function errorButtonsHtml(meta, { taskId = null, goalId = null, interrupted = fa
   const out = [];
   for (const id of meta.actionIds || []) {
     if (id === 'retry' && taskId != null) out.push(`<button class="errbtn" data-fsretry="${taskId}">${interrupted ? C.resume : C.retry}</button>`);
-    else if (id === 'run-with-codex' && canCodex) out.push(`<button class="errbtn" data-fscodex="${goalId}">${C.other}</button>`);
+    // This keeps the existing agent-switch endpoint, but says what it really does
+    // for an interrupted run: continue the saved goal with another agent rather
+    // than imply a brand-new, contextless task will be created.
+    else if (id === 'run-with-codex' && canCodex) out.push(`<button class="errbtn" data-fscodex="${goalId}">${interrupted ? (failureLang() === 'ja' ? 'Codexで再開' : 'Resume with Codex') : C.other}</button>`);
     else if (id === 'connect') out.push(`<button class="errbtn" data-fsconnectworker="1">${C.connect}</button>`);
     else if (id === 'check-connection') out.push(`<button class="errbtn" data-fscheckworker="1">${C.check}</button>`);
     else if (id === 'retry-tests' && goalId != null) out.push(`<button class="errbtn" data-fsreverify="${goalId}">${C.tests}</button>`);
@@ -7297,10 +7303,6 @@ function renderFsTodo(){
   const needsCount = failed.filter((t) => !resumedGoalIds.has(t.goalId)).length + blocked.length + needsInput.length;
   const doingCount = running.length + finalizingGoals.length + pausedGoalIds.size;
   const nextCount = parkedReplyGoals.length + queued.length;
-  const hasPipelineWork = doingCount > 0 || nextCount > 0 || pgoals.some((g) => ['planning', 'stacked'].includes(g.status));
-  const laterIdle = later.length && !hasPipelineWork
-    ? `<div class="lateridle"><span class="msg">${document.documentElement.lang === 'ja' ? `実行待ちはありません。Laterの${later.length}件は自動開始されません。` : `Nothing is queued. ${later.length} Later item${later.length === 1 ? '' : 's'} will not start automatically.`}</span><button data-fsstartnextlater="${later[0].id}">${document.documentElement.lang === 'ja' ? '次を開始' : 'Start next'}</button></div>`
-    : '';
   // 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
@@ -7322,7 +7324,6 @@ function renderFsTodo(){
       <div class="qadd" data-fsnew="1" role="button" tabindex="0" title="New goal"><span class="paplus"></span><span>New</span></div>
       <span class="tlcap">Later</span>
       <div class="qadd lateradd" data-fslaternew="1" role="button" tabindex="0" title="New Later goal"><span class="paplus"></span><span>New</span></div>
-      ${laterIdle}
       ${laterRows}
     </div>
     ${doneAll.length ? `<div class="doneline${fsUI.doneOpen ? ' open' : ''}" id="fsDoneLine">✓ Done<span class="num">${doneAll.length}</span><span class="fold">▾</span></div>
@@ -7387,13 +7388,6 @@ function renderFsTodo(){
     let r; try { r = await fetch(withKey(`/api/goals/${b.dataset.fspstart}/activate`), { method: 'POST' }); }
     catch { showErr('Start failed.'); return; }
     if (!r.ok) showErr('Start failed.'); else await refresh(); };
-  const startNextLater = el.querySelector('[data-fsstartnextlater]');
-  if (startNextLater) startNextLater.onclick = async (e) => { e.stopPropagation();
-    startNextLater.disabled = true;
-    let r; try { r = await fetch(withKey(`/api/goals/${startNextLater.dataset.fsstartnextlater}/activate`), { method: 'POST' }); }
-    catch { showErr(document.documentElement.lang === 'ja' ? '開始できませんでした' : 'Could not start the next item.'); startNextLater.disabled = false; return; }
-    if (!r.ok) { showErr(document.documentElement.lang === 'ja' ? '開始できませんでした' : 'Could not start the next item.'); startNextLater.disabled = false; }
-    else await refresh(); };
   for (const b of el.querySelectorAll('[data-fspdel]')) b.onclick = async (e) => { e.stopPropagation();
     const preview = b.closest('.trow')?.querySelector('.tt')?.textContent || '';
     if (!(await confirmDelete({ title: 'Delete this shelved goal?', preview }))) return;
@@ -8933,7 +8927,14 @@ async function send(){
         // review goal → the AI decides continue/rescope/split/answer (same path as the
         // review card's own reply box), with the note anchored under the composer.
         const body = await replyToGoal(bind.goal.id, fullText, '#input');
-        if (body) { upsert(state.goals, body.goal); if (body.task) upsert(state.tasks, body.task); render(); }
+        if (body) {
+          upsert(state.goals, body.goal); if (body.task) upsert(state.tasks, body.task);
+          // A Review reply is a one-shot handoff. Keeping the composer bound
+          // afterwards adds a redundant “Replying to #N” banner and makes the
+          // next, unrelated instruction easy to send to the old goal by mistake.
+          state.selectedGoal = null;
+          render();
+        }
       } else {
         // running/partial/failed/interrupted/blocked → a plain follow-up on the same goal.
         const r = await fetch(withKey(`/api/goals/${bind.goal.id}/reply`), { method: 'POST',
@@ -9178,6 +9179,11 @@ function latestGoalRunForOutcome(goalId, tasks, outcome){
 function saRowFromDigest(r, gs, ts){
     const g = gs.find((x) => x.id === r.goalIds[0]);
     const outcome = latestGoalOutcomeTask(g?.id, ts);
+    // A timeout commonly settles its parent goal as `partial`: some earlier work
+    // may be real, while the final task is retryable. Preserve the task-level truth
+    // here so the card never calls that state a generic failure.
+    const interruptedTask = [...ts].filter((t) => r.goalIds.includes(t.goalId) && t.status === 'interrupted')
+      .sort((a, b) => taskSortTime(b) - taskSortTime(a))[0] || null;
     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
     // per finished requirement task's own report; falls back to the goal's plan
@@ -9207,6 +9213,7 @@ function saRowFromDigest(r, gs, ts){
       .map((x) => ({ id: x.id, title: saReviewTitle(x) }));
     return {
       ids: r.goalIds, goalId: r.goalIds[0], goal: g, folded,
+      interruptedTask,
       no: r.reviewNumber,
       title: saReviewTitle(g),
       check: String(g?.reviewSummary?.check || r.checkLine || '').replace(/\s+/g, ' ').trim(),
@@ -9733,10 +9740,27 @@ function saItemHtml(it, i){
     ? `<div class="ropen rdeclerr"><p class="rbody v">${failureLang() === 'ja' ? 'AIが開けるものを用意しようとしましたが、記録が読み取れませんでした。返信して直させてください。' : 'The AI tried to declare something to open here, but the record was unreadable. Reply to have it rewritten.'}</p></div>`
     : '';
   const openBody = (it.run?.cmd || it.run?.url)
-    ? `<div class="ropen" id="runlocal${i}"><button type="button" class="olink" data-saact="preview:${i}" id="prevlink${i}">${esc(it.run.url || 'Start it')}<span class="ar">↗</span></button><p class="rbody v">${esc(it.run.note || it.run.cmd || 'Starts when you press it.')}</p></div>`
+    ? `<div class="ropen" id="runlocal${i}"><button type="button" class="olink" data-saact="preview:${i}" id="prevlink${i}">${esc(it.run.url || 'Start it')}<span class="ar">↗</span></button><p class="rbody v">${esc(it.run.note || it.run.cmd || 'Starts when you press it.')}${it.interruptedTask ? ' Available now; it does not wait for this run to finish.' : ''}</p></div>`
     : it.pr
       ? `<div class="ropen"><a class="olink" href="${esc(it.pr)}" target="_blank" rel="noopener">${esc(it.pr)}<span class="ar">↗</span></a><p class="rbody v">Review and merge this pull request to finish the item.</p></div>`
       : runDeclErrorBody;
+  // An interrupted worker is not a failed implementation. Keep the recovery
+  // record factual and compact: the last recorded step, files already changed,
+  // whether verification remains, and the exact resume action. These are all
+  // state fields — no Manager-generated diagnosis or task re-planning.
+  const interruptedBody = it.interruptedTask ? (() => {
+    const t = it.interruptedTask;
+    const last = (state.act[t.id] ?? t.activity ?? []).filter(Boolean).at(-1);
+    const files = [...new Set((t.changedFiles ?? []).concat(filePaths))];
+    const tests = it.testResult?.ran
+      ? (it.testResult.ok ? 'Verification completed before the interruption.' : 'Verification remains incomplete or failed.')
+      : 'Verification has not completed yet.';
+    const reason = String(t.result || 'The worker reached its run limit before finishing.').replace(/\s+/g, ' ').slice(0, 240);
+    return `<div class="rinfra"><p class="rbody v"><b>Interrupted — resume available</b><br>${esc(reason)}</p>`
+      + (last ? `<p class="rbody v">Last recorded step: ${esc(String(last).slice(0, 220))}</p>` : '')
+      + (files.length ? `<p class="rbody v">Changed so far: ${esc(files.join(' · '))}</p>` : '')
+      + `<p class="rbody v">${esc(tests)}</p><div class="acts2"><button class="errbtn" data-saact="resume:${i}">Resume</button>${errorButtonsHtml(taskErrorMeta(t, it.goal), { taskId: null, goalId: it.goalId, interrupted: true, agent: t.agent || it.goal?.agent })}</div></div>`;
+  })() : '';
   const prRepairBody = it.goal?.prError
     ? `<div class="prfail"><b>${esc(it.goal.prRepair?.title || 'PR blocked')}</b> <span class="dim">${esc(it.goal.prError)}</span>${it.goal.prRepair?.steps?.length ? `<ul>${it.goal.prRepair.steps.map((s) => `<li>${esc(s)}</li>`).join('')}</ul>` : ''}${it.goal.prRepair?.dirtyFiles?.length ? `<div class="dim">dirty: ${it.goal.prRepair.dirtyFiles.map(esc).join(' · ')}</div>` : ''}<div class="acts2"><button class="errbtn" data-saact="retrypr:${i}">${failureLang() === 'ja' ? 'PR再試行' : 'Retry PR'}</button><button class="errbtn" data-saact="glog:${i}">${failureLang() === 'ja' ? 'ログ確認' : 'View log'}</button></div></div>`
     : '';
@@ -9899,8 +9923,8 @@ function saItemHtml(it, i){
   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 attnInt = gstatus === 'interrupted' || Boolean(it.interruptedTask);
+  const attnLabel = attnInt ? 'Interrupted — resume available' : gstatus === 'blocked' ? 'Blocked' : 'Failed';
   const rst = working
     ? '<span class="rst working"><span class="rdot"></span>Working on it</span>'
     : attn
@@ -9912,6 +9936,7 @@ function saItemHtml(it, i){
     ${meta}
     ${reqSec}
     ${S('Open it', openBody)}
+    ${S('Run state', interruptedBody)}
     ${S('PR', prRepairBody)}
     ${S('Also carries', carriesBody)}
     ${S('What to check', checkBody)}
@@ -10396,6 +10421,11 @@ $('seeall').addEventListener('click', async (e) => {
   }
   const it = SA.items[i];
   if (!it) return;
+  if (k === 'resume') {
+    const taskId = it.interruptedTask?.id;
+    if (taskId != null) return fsRetryTask(taskId);
+    return;
+  }
   if (k === 'retrypr') return fsRetryPrGoal({ disabled: false, textContent: '', dataset: { fsprretry: String(it.goalId) } });
   if (k === 'glog') return toggleGoalDetail(it.goalId);
   // meta-row toggle (H23): tap the token chip to reveal the technique breakdown.

package/engine/server.mjs

@@ -2593,7 +2593,11 @@ async function startPreview(task) {
   // They never need a child process; treating them like a Manager app made
   // "Open it" fail even though the HTML artifact was ready to view.
   if (!cmd && typeof url === 'string' && url.startsWith('/upload/')) {
-    return { url, reused: false, started: false };
+    // The card opens this relative URL in a new tab. Carry the manager key in
+    // that URL; otherwise the new navigation loses the authenticated fetch
+    // context and the upload route correctly returns its 403 gate.
+    const keyedUrl = `${url}${url.includes('?') ? '&' : '?'}key=${encodeURIComponent(ACCESS_KEY)}`;
+    return { url: keyedUrl, reused: false, started: false };
   }
   const live = previews.get(task.id);
   if (live && live.proc.exitCode === null) {

package/package.json

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