@galda/cli 0.10.76 → 0.10.77

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/package.json

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