@galda/cli 0.10.76 → 0.10.78

package/app/index.html

@@ -733,6 +733,12 @@
   @keyframes saFinishDraw{to{stroke-dashoffset:0}}
   @keyframes saFinishFade{0%,72%{opacity:1}100%{opacity:0}}
   @media (prefers-reduced-motion:reduce){.sa-finish,.sa-finish-mark,.sa-finish-mark path{animation:none}.sa-finish{background:transparent}}
+  /* A PR-less approval is just as complete as a merged PR. Keep this transient
+     celebration fixed above the board so it never changes document height or
+     scroll position. */
+  .approve-finish{position:fixed;inset:0;z-index:10000;display:grid;place-items:center;pointer-events:none;
+    background:rgba(0,0,0,.28);backdrop-filter:blur(2px);animation:saFinishFade .9s ease both}
+  .approve-finish .sa-finish-mark{width:112px;height:112px}
 
   /* connect modal */
   /* A confirm dialog (deleteConfirmOverlay etc.) must always sit above other
@@ -5594,6 +5600,20 @@ function finishOpenReview(goal){
   const delay = reduce ? 350 : 950;
   setTimeout(() => { layer.remove(); saClose(); render(); }, delay);
 }
+// PR-less review approval: use the same high-signal green completion mark as
+// the merged-PR path, but do not close or reflow the review surface. The canvas
+// burst starts at the actual Approve control, making the reward feel causal.
+function celebrateDirectApprove(cx, cy){
+  const reduce = matchMedia('(prefers-reduced-motion: reduce)').matches;
+  if (!reduce) confettiBurst(cx, cy);
+  const layer = document.createElement('div');
+  layer.className = 'approve-finish';
+  layer.setAttribute('role', 'status');
+  layer.setAttribute('aria-label', 'Approved');
+  layer.innerHTML = '<div class="sa-finish-mark"><svg viewBox="0 0 64 64" aria-hidden="true"><path d="M17 33l10 10 21-23"/></svg></div>';
+  document.body.appendChild(layer);
+  setTimeout(() => layer.remove(), reduce ? 350 : 950);
+}
 async function setRequirementStatus(goalId, taskId, status){
   const approvals = (state.reviewApprovals[goalId] ??= {});
   approvals[taskId] = approvals[taskId] === status ? undefined : status; // click again to undo
@@ -10415,7 +10435,7 @@ $('seeall').addEventListener('click', async (e) => {
     // instead of celebrating/advancing; saCall already upserted the still-'review' goal.
     const g = state.goals.find((x) => x.id === it.goalId);
     if (g && g.status === 'review' && g.pr) { confirmMerge(g); return; }
-    confettiBurst(bx.left + bx.width / 2, bx.top + bx.height / 2); saAdvanceFocus(it.goalId); }); }
+    celebrateDirectApprove(bx.left + bx.width / 2, bx.top + bx.height / 2); saAdvanceFocus(it.goalId); }); }
   if (k === 'dis') return saAct(it, 2, 'reject').then((ok) => { if (ok) saAdvanceFocus(it.goalId); }); // 07-20c: Reject → Rejected column
   if (k === 'arch') return saAct(it, 4, 'archive').then((ok) => { if (ok) saAdvanceFocus(it.goalId); });
   if (k === 'rev') return saAct(it, 5, 'revert').then((ok) => { if (ok) saAdvanceFocus(it.goalId); });
@@ -10478,7 +10498,7 @@ document.addEventListener('keydown', (e) => {
       if (g && g.status === 'review' && g.pr) { confirmMerge(g); return; } // 07-20c: PR goal → merge dialog
       const el = $('seeall').querySelector(`.sa-item[data-gid="${gid}"] .sa-app`);
       const box = el?.getBoundingClientRect();
-      confettiBurst(box ? box.left + box.width / 2 : innerWidth / 2, box ? box.top + box.height / 2 : innerHeight / 3);
+      celebrateDirectApprove(box ? box.left + box.width / 2 : innerWidth / 2, box ? box.top + box.height / 2 : innerHeight / 3);
       saAdvanceFocus(gid);
     });
   } else saAct(it, 2, 'reject').then((ok) => { if (ok) saAdvanceFocus(gid); }); // 07-20c: D = Reject

package/engine/lib.mjs

@@ -2843,7 +2843,7 @@ export function replayQueueLog(rawText, reviewDefinitions = {}) {
     try {
       const e = JSON.parse(line);
       if (e.kind === 'goal') { goals = goals.filter((g) => g.id !== e.id); if (!e.deleted) goals.push(e); }
-      else { tasks = tasks.filter((t) => t.id !== e.id); tasks.push(e); }
+      else if (e.kind === 'task') { tasks = tasks.filter((t) => t.id !== e.id); if (!e.deleted) tasks.push(e); }
     } catch { /* skip a corrupt line rather than lose the whole log */ }
   }
   for (const t of tasks) {

package/engine/server.mjs

@@ -2589,6 +2589,12 @@ function installPreviewReaper() {
 async function startPreview(task) {
   installPreviewReaper();
   const { cmd, url, dir } = task.run;
+  // Attached static artifacts are already served by this Manager at /upload.
+  // 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 };
+  }
   const live = previews.get(task.id);
   if (live && live.proc.exitCode === null) {
     live.expires = Date.now() + PREVIEW_IDLE_MS;
@@ -4159,6 +4165,16 @@ const server = createServer(async (req, res) => {
         || tasks.some((t) => t.projectId === project.id && ['running', 'queued'].includes(t.status));
       if (active) return json(res, 409, { error: 'cannot delete a project with active work' });
       const idx = projects.findIndex((p) => p.id === project.id);
+      // A project id can be reused after deletion (for example "New project").
+      // Remove its complete history before freeing that id, otherwise its old
+      // Review cards appear in the newly created project.
+      const removedGoals = goals.filter((g) => g.projectId === project.id);
+      const removedGoalIds = new Set(removedGoals.map((g) => g.id));
+      const removedTasks = tasks.filter((t) => t.projectId === project.id || removedGoalIds.has(t.goalId));
+      goals = goals.filter((g) => g.projectId !== project.id);
+      tasks = tasks.filter((t) => t.projectId !== project.id && !removedGoalIds.has(t.goalId));
+      for (const g of removedGoals) logAppend(JSON.stringify({ kind: 'goal', id: g.id, deleted: true }));
+      for (const t of removedTasks) logAppend(JSON.stringify({ kind: 'task', id: t.id, deleted: true }));
       projects.splice(idx, 1);
       queues.delete(project.id);
       saveProjects();

package/package.json

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