@galda/cli 0.10.86 → 0.10.87

package/app/index.html

@@ -9859,6 +9859,11 @@ function saItemHtml(it, i){
   // verification visible together while prior attempts stay in Conversation.
   const attempt = it.attempt;
   const activeText = String(attempt?.activeRequirement?.text || '').replace(/\s+/g, ' ').trim();
+  const requestText = String(it.goalText || '').replace(/\s+/g, ' ').trim();
+  // A single requirement is commonly recorded verbatim from the Request. Do
+  // not make the review read it back as a second section; keep this section for
+  // a narrowed retry requirement or concrete run evidence instead.
+  const reviewText = activeText && activeText !== requestText ? activeText : '';
   const attemptFiles = Array.isArray(attempt?.changedFiles) ? attempt.changedFiles.filter(Boolean) : [];
   const attemptTests = attempt?.verification?.tests;
   const attemptProof = attempt?.verification?.proof;
@@ -9867,8 +9872,8 @@ function saItemHtml(it, i){
     attemptTests?.ran ? `Tests: ${attemptTests.passed ?? 0} passed${attemptTests.failed ? ` / ${attemptTests.failed} failed` : ''}` : '',
     attemptProof?.pass === true ? 'Proof recorded' : '',
   ].filter(Boolean).join(' · ');
-  const attemptSec = activeText || attemptEvidence
-    ? `<section class="rsec"><div class="rsec-h">This review</div>${activeText ? `<p class="rbody rask">${esc(activeText)}</p>` : ''}${attemptEvidence ? `<p class="rbody v">${esc(attemptEvidence)}</p>` : ''}</section>`
+  const attemptSec = reviewText || attemptEvidence
+    ? `<section class="rsec"><div class="rsec-h">This review</div>${reviewText ? `<p class="rbody rask">${esc(reviewText)}</p>` : ''}${attemptEvidence ? `<p class="rbody v">${esc(attemptEvidence)}</p>` : ''}</section>`
     : '';
   const reqVerify = it.requirementVerification;
   const reqVerifyParts = [
@@ -9877,10 +9882,15 @@ function saItemHtml(it, i){
     Array.isArray(reqVerify?.changedFiles) && reqVerify.changedFiles.length ? `Files: ${reqVerify.changedFiles.join(' · ')}` : '',
   ].filter(Boolean);
   const reqVerifySec = reqVerifyParts.length ? `<section class="rsec"><div class="rsec-h">Requirement verification</div><p class="rbody v">${esc(reqVerifyParts.join(' · '))}</p></section>` : '';
-  const coverageRows = (it.requirementCoverage || []).filter((r) => r.text).map((r) => {
+  const coverage = (it.requirementCoverage || []).filter((r) => r.text);
+  // For one requirement copied from Request, its status is useful but the full
+  // sentence is not. Multiple or narrowed requirements retain their text.
+  const coverageIsSingleRequest = coverage.length === 1
+    && String(coverage[0].text).replace(/\s+/g, ' ').trim() === requestText;
+  const coverageRows = coverage.map((r) => {
     const status = r.evidenceStatus || r.taskStatus;
     const tone = status === 'addressed' || status === 'done' ? 'ok' : status === 'partial' ? '' : 'dim';
-    return `<p class="rbody v"><b class="${tone}">${esc(status)}</b> · ${esc(r.text)}</p>`;
+    return `<p class="rbody v"><b class="${tone}">${esc(status)}</b>${coverageIsSingleRequest ? '' : ` · ${esc(r.text)}`}</p>`;
   }).join('');
   const coverageSec = coverageRows ? `<section class="rsec"><div class="rsec-h">Requirement coverage</div>${coverageRows}</section>` : '';
   // Worker-declared evidence is displayed verbatim as an auditable label, not
@@ -9890,8 +9900,6 @@ function saItemHtml(it, i){
     return `<p class="rbody v"><b class="${tone}">${esc(entry.status)}</b> · ${esc(entry.evidence || entry.id)}</p>`;
   }).join('');
   const evidenceSec = evidenceRows ? `<section class="rsec"><div class="rsec-h">Requirement evidence</div>${evidenceRows}</section>` : '';
-  const userCoverageRows = (it.requirementCoverage || []).filter((r) => r.text).map((r) => `<p class="rbody v"><b class="${r.status === 'done' ? 'ok' : 'dim'}">${esc(r.status)}</b> · ${esc(r.text)}</p>`).join('');
-  const userCoverageSec = userCoverageRows ? `<section class="rsec"><div class="rsec-h">User requirements</div>${userCoverageRows}</section>` : '';
   // ── Open it (Masa 2026-07-22): a plain section in the card's own typography — the same
   // heading + body shape as Request and Result, not a chip in another vocabulary. The
   // address the worker declared IS the control (press it → the process starts on demand →
@@ -10122,7 +10130,6 @@ function saItemHtml(it, i){
     ${coverageSec}
     ${evidenceSec}
     ${reqVerifySec}
-    ${userCoverageSec}
     ${S('Open it', openBody)}
     ${S('Run state', interruptedBody)}
     ${S('PR', prRepairBody)}

package/engine/server.mjs

@@ -5520,6 +5520,12 @@ async function answerGoalQuestion(goal, question) {
         addGoalMessage(goal, { from: 'you', via: 'review', text });
         goal.text = text.slice(0, 8000);
       }
+      // A PR request added during Review changes the deliverable, not the
+      // worker's permissions. Workers deliberately cannot git add/commit; the
+      // Manager close-out below owns commit/push/PR creation. Keep this
+      // monotonic: an explicit later PR request upgrades a No-PR/auto goal,
+      // while ordinary feedback never silently downgrades an existing PR goal.
+      if (result.spawnsWorker && resolveWantsPR('auto', text, false)) goal.wantsPR = true;
       goal.status = result.status;
       saveGoal(goal);
       // rescope はワーカーを起こさない——「いったん止めて考え直す」なので、走り出したら

package/package.json

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