@galda/cli 0.10.99 → 0.10.100

package/app/index.html

@@ -3475,6 +3475,28 @@ function restoreFocusedAnswerInput(container, attr, saved){
   el.focus();
   try { el.setSelectionRange(saved.start, saved.end); } catch { /* input type may not support it */ }
 }
+// The capture/restore pair above still loses an in-progress Japanese/Chinese/Korean
+// IME conversion: rebuilding the DOM node while the browser has an open composition
+// session aborts that session outright, so the half-typed conversion is dropped even
+// though the plain .value is restored. Track composition state at the document level
+// (delegated, since the actual input node gets destroyed and recreated every rebuild)
+// and let callers skip the destructive innerHTML rebuild while composing, retrying
+// once composition ends.
+let composingAnswerInput = null; // { attr, id } while a qansinput/fsansinput IME session is open
+document.addEventListener('compositionstart', (e) => {
+  const attr = e.target?.getAttribute?.('data-qansinput') != null ? 'data-qansinput'
+    : e.target?.getAttribute?.('data-fsansinput') != null ? 'data-fsansinput' : null;
+  if (attr) composingAnswerInput = { attr, id: e.target.getAttribute(attr) };
+});
+document.addEventListener('compositionend', (e) => {
+  if (!composingAnswerInput) return;
+  if (e.target?.getAttribute?.(composingAnswerInput.attr) !== composingAnswerInput.id) return;
+  composingAnswerInput = null;
+  // Catch up on whatever rebuild(s) were skipped mid-composition.
+  if (typeof renderTasks === 'function') renderTasks();
+  if (typeof renderFsFeed === 'function') renderFsFeed();
+});
+function answerInputComposing(attr){ return composingAnswerInput?.attr === attr; }
 
 const STATUS_LABEL = {
   sending: 'Sending…', stacked: 'Queued', planning: 'Planning…', queued: 'To do', running: 'Working…', needsInput: 'Needs input', needsApproval: 'Needs you',
@@ -4807,6 +4829,7 @@ function renderTasks(){
      <div class="cond">shelved — press Start to queue it</div></div>
      <button type="button" class="prmini pstart" data-pstart="${g.id}">Start</button>
      <button type="button" class="pdel" data-pdel="${g.id}" title="Delete">✕</button></div>`).join('');
+  if (answerInputComposing('data-qansinput')) return; // mid-IME conversion — rebuilding now would abort it; compositionend re-runs this
   const savedQAns = captureFocusedAnswerInput($('tasklist'), 'data-qansinput');
   $('tasklist').innerHTML =
     sec('Needs you', needsYou, 'needsyou')
@@ -6768,6 +6791,7 @@ function renderFsFeed(){
   $('fsRoot').classList.toggle('clawdhero', !hasWork);
   const sc = feed.parentNode;
   const nearBottom = sc.scrollHeight - sc.clientHeight - sc.scrollTop <= 80;
+  if (answerInputComposing('data-fsansinput')) return; // mid-IME conversion — rebuilding now would abort it; compositionend re-runs this
   const savedFsAns = captureFocusedAnswerInput(feed, 'data-fsansinput');
   feed.innerHTML = hasWork ? out : '';
   if (nearBottom) sc.scrollTop = sc.scrollHeight;
@@ -8517,15 +8541,40 @@ window.addEventListener('focus', () => { if (signinPending) refreshAfterSignin()
 
 // When the served UI is newer than the one this tab is running (hot-copy
 // or release), reload in place — drafts survive via localStorage.
+//
+// A new task/goal running elsewhere routinely touches app/index.html's mtime
+// (its own edit, or a hot-copy after another lane's merge) — that alone used
+// to force this reload on every open tab, mid-review, wiping the open Review
+// dialog with no way back in (Masa 2026-07-29). Two fixes: (1) never reload
+// while the Review dialog (SA.open) is up — remember a pending version and
+// apply it the moment the dialog closes instead; (2) whichever card was open
+// is saved to localStorage first, so if a reload does happen while reviewing,
+// boot() reopens the same card instead of dropping the user back at nothing.
+let _pendingUiVersion = null;
 function checkUiVersion(v){
   if (!v) return;
   if (window.__uiv && window.__uiv !== v) {
-    try { localStorage.setItem('draft', $('input').value ?? ''); } catch {}
+    if (typeof SA !== 'undefined' && SA.open) { _pendingUiVersion = v; return; }
+    try {
+      localStorage.setItem('draft', $('input').value ?? '');
+      if (typeof SA !== 'undefined' && SA.open) {
+        localStorage.setItem('reopenReview', JSON.stringify({ goalId: SA.solo || SA.focusGid || SA.cur, solo: !!SA.solo }));
+      } else {
+        localStorage.removeItem('reopenReview');
+      }
+    } catch {}
     location.reload();
     return;
   }
   window.__uiv = v;
 }
+// Called from saClose() so a version that arrived mid-review gets applied
+// (reloaded) right after the user is done, instead of never at all.
+function applyPendingUiVersionIfAny(){
+  if (_pendingUiVersion == null) return;
+  const v = _pendingUiVersion; _pendingUiVersion = null;
+  checkUiVersion(v);
+}
 
 async function refresh(){
   try {
@@ -8823,6 +8872,13 @@ async function boot(){
   if (window.__GALDA_DISCONNECTED) return bootDisconnected(window.__GALDA_DISCONNECTED);
   $('input').value = localStorage.getItem('draft') ?? '';
   await refresh();
+  // Reopen the Review dialog a version-triggered reload had to close mid-review
+  // (see checkUiVersion) so the user lands back where they were, not on the board.
+  try {
+    const reopen = JSON.parse(localStorage.getItem('reopenReview') ?? 'null');
+    localStorage.removeItem('reopenReview');
+    if (reopen && typeof saOpen === 'function') saOpen(reopen.goalId, reopen.solo);
+  } catch {}
   // The folder sits above the composer and is read on every screen, so its list is
   // fetched once at boot rather than on first open: without HOME the bar can only
   // print the absolute path (measured — it read "/var/folders/…/agent-manager"
@@ -9026,32 +9082,15 @@ function resolveComposer(raw){
     review: !!(msg.review || parsed.review),
   };
 }
-// Feature A (ONE-CONVERSATION §1.2/§1.4 "1本の会話"): the composer CONTINUES the
-// goal you're focused on (state.selectedGoal) instead of always making a new goal
-// — like Claude Code, where you keep talking to the same session. It binds only
-// when that goal is in the ACTIVE project AND in a repliable status: a review goal
-// routes to the AI-decide reply (continue/rescope/split/answer), an in-flight or
-// finished-rework goal takes a plain follow-up. Anything else (done, needsInput,
-// planning/stacked, or nothing selected) → new-goal mode, exactly as before.
-const REPLY_AUTO = new Set(['review']);
-const REPLY_PLAIN = new Set(['running', 'partial', 'failed', 'interrupted', 'blocked']);
+// The centre composer is always a NEW task. `selectedGoal` is viewing state: using
+// it as an implicit reply target made a request typed while reading Review #856 get
+// appended to that unfinished goal. Replying is intentionally available only from
+// the goal/review card's dedicated Reply / Fix controls, whose destination is shown
+// next to the control. Never infer a delivery destination from what happens to be
+// open in the centre lane.
 function replyBinding(){
-  const id = state.selectedGoal;
-  if (id == null) return null;
-  // Focus and reply-target are separate concepts. A Review reply keeps this goal
-  // selected so its live work remains in the centre feed, while the composer goes
-  // back to new-goal mode and shows no stale “Replying to” chip.
-  if (state.composerDetachedGoal === id) return null;
-  const g = state.goals.find((x) => x.id === id);
-  if (!g || g.projectId !== state.active) return null;   // stale after a project switch → new-goal mode
-  // needsInput has NO worker session yet (planGoal() sets it before any task/worker
-  // exists — see engine/server.mjs) — there's no "conversation" to resume via /reply
-  // (that endpoint 409s on needsInput; engine/lib.mjs:3010). Binding here routes to
-  // the SAME /answer endpoint the question card's own free-text box already uses
-  // (fsAnswerQuestion), so the composer becomes a second way to answer, not a new path.
-  if (g.status === 'needsInput' && g.question) return { goal: g, mode: 'answer' };
-  if (REPLY_AUTO.has(g.status)) return { goal: g, mode: 'auto' };
-  if (REPLY_PLAIN.has(g.status)) return { goal: g, mode: 'plain' };
+  // Deliberately do not bind the main composer. Keep this one routing boundary so
+  // a future UI change cannot accidentally reintroduce selectedGoal -> /reply.
   return null;
 }
 async function send(){
@@ -10529,7 +10568,7 @@ function saOpen(focusGoalId, solo = false){
   saFetchDiffs();
   saEnhanceShotBoxes();
 }
-function saClose(){ SA.open = false; SA.solo = null; SA.focusGid = null; SA.lastMarkup = null; if (SA.focusTimer) { clearTimeout(SA.focusTimer); SA.focusTimer = null; } $('seeall').classList.remove('open'); }
+function saClose(){ SA.open = false; SA.solo = null; SA.focusGid = null; SA.lastMarkup = null; if (SA.focusTimer) { clearTimeout(SA.focusTimer); SA.focusTimer = null; } $('seeall').classList.remove('open'); applyPendingUiVersionIfAny(); }
 // Continuous review (Masa 2026-07-08): move a persistent "current" card with
 // ↑↓ and auto-advance to the next pending after each verdict — same screen, same
 // Ledger, so approving flows without hunting for the next one.

package/engine/lib.mjs

@@ -2148,16 +2148,34 @@ export function wantsPullRequest(text) {
   return /プルリク|pull\s*request|(^|[^A-Za-z])PR([^A-Za-z]|$)/i.test(String(text ?? ''));
 }
 
+// Auto delivery must distinguish a request to CHANGE the connected project
+// from a request to THINK about it.  A project-level PR preference remains the
+// user's strongest default, and an explicit Deliverable choice below always
+// wins.  This only prevents the silent "implemented, locally previewed, but no
+// PR" outcome for ordinary implementation requests in projects that have not
+// configured that preference yet.
+export function isImplementationRequest(text) {
+  const value = String(text ?? '');
+  const implementation = /実装|修正|直して|直す|不具合|バグ|追加|変更|作って|できるように|直したい|fix\b|implement\b|build\b|add\b|change\b|bug\b/i;
+  const explorationOnly = /リサーチ|調査|分析|アイデア|案(?:を|が|は|ください)|提案|比較|相談|教えて|説明|要約|デザイン|モック|ワイヤーフレーム|資料|スライド|レポート|ドキュメント|research\b|analysis\b|brainstorm\b|ideas?\b|suggest(?:ion)?\b|design\b|mockup\b|wireframe\b|slides?\b|report\b/i;
+  // "実装案を出して" is exploration; "案3を実装して" is implementation.
+  return implementation.test(value) && !(explorationOnly.test(value) && !/実装|修正|直して|直す|追加|変更|fix\b|implement\b|build\b|add\b|change\b/i.test(value));
+}
+
 // Task 46: the "Deliverable" toggle (pr | none | auto) on task creation and
 // editing. An explicit 'pr'/'none' always wins over the text heuristic, so
 // a user's choice survives editing the goal's text afterwards. `projectDefault`
 // (task 45: a project's reviewDefinition.defaultWantsPR, edited from the same
 // settings panel as the review definition) is only consulted for 'auto' when
 // the text itself gives no signal either way — an explicit "PRを作って" in the
-// text still wins even if the project's default is off, matching the
-// pre-task-45 behavior that new callers who omit this argument keep exactly.
-export function resolveWantsPR(pr, text, projectDefault = false) {
-  return pr === 'pr' ? true : pr === 'none' ? false : (wantsPullRequest(text) || !!projectDefault);
+// text still wins even if the project's default is off. `preferenceKnown`
+// distinguishes an intentional per-project "No PR" from an older project
+// which has never made a delivery choice. Only the latter gets the helpful
+// implementation-request fallback.
+export function resolveWantsPR(pr, text, projectDefault = false, preferenceKnown = false) {
+  if (pr === 'pr') return true;
+  if (pr === 'none') return false;
+  return wantsPullRequest(text) || !!projectDefault || (!preferenceKnown && isImplementationRequest(text));
 }
 
 // Task 90: which entry point created a goal (Web chat / MCP / future API),

package/engine/server.mjs

@@ -25,6 +25,7 @@ import { parseStreamEvents, parseCodexEvents, buildCodexArgs, parsePlan, refineP
 import { migrateRequirementModel, validateRequirementModel } from './requirement-model.mjs';
 import { buildManagerVerificationChecks, buildRequirementVerificationEvidence, mergeRequirementVerificationEvidence } from './requirement-verification.mjs';
 import { createSerialQueue } from './lib.mjs';
+import { wantsPullRequest } from './lib.mjs';
 import { parseRequirementEvidence, unmappedChangedFiles, previewOpensSomethingElse } from './lib.mjs';
 import { isPrClosed } from './lib.mjs';
 import { collectProjectRules } from './lib.mjs';
@@ -2447,7 +2448,7 @@ function createSplitGoal(parent, text, identity, targetProjectId = parent.projec
   if (!targetProject) throw new Error(`unknown target project: ${targetProjectId}`);
   const goal = {
     id: nextId++, projectId: targetProjectId, text: text.slice(0, 8000), status: 'planning',
-    wantsPR: resolveWantsPR('auto', text, getReviewDefinition(targetProjectId).defaultWantsPR),
+    wantsPR: resolveWantsPR('auto', text, getReviewDefinition(targetProjectId).defaultWantsPR, !!targetProject.reviewPrefAsked),
     model: parent.model, effort: parent.effort, agent: parent.agent, mode: 'auto',
     skill: null, images: [], source: resolveGoalSource('app'), sourceRef: null,
     executionPlan: buildExecutionPlan({ projectId: targetProjectId, text, previousSession: false }),
@@ -4832,7 +4833,7 @@ const server = createServer(async (req, res) => {
           approvalDecision: null,
           reviewOnly: review ? true : undefined,
           note: review && typeof note === 'string' && note.trim() ? note.trim().slice(0, 8000) : undefined,
-          wantsPR: (review || approval) ? Boolean(importedPr) : resolveWantsPR(pr, text, getReviewDefinition(projectId).defaultWantsPR),
+          wantsPR: (review || approval) ? Boolean(importedPr) : resolveWantsPR(pr, text, getReviewDefinition(projectId).defaultWantsPR, !!project.reviewPrefAsked),
           model: workerModel(goalAgent, model),
           effort: workerEffort(goalAgent, effort),
           agent: goalAgent,
@@ -4950,7 +4951,8 @@ const server = createServer(async (req, res) => {
         if (promote) goal.prio = ++prioCounter;
         if (text?.trim()) {
           goal.text = text.trim().slice(0, 8000);
-          goal.wantsPR = resolveWantsPR(pr, goal.text, getReviewDefinition(goal.projectId).defaultWantsPR);
+          const project = projects.find((p) => p.id === goal.projectId);
+          goal.wantsPR = resolveWantsPR(pr, goal.text, getReviewDefinition(goal.projectId).defaultWantsPR, !!project?.reviewPrefAsked);
         }
         saveGoal(goal);
         json(res, 200, goal);
@@ -5521,7 +5523,7 @@ const server = createServer(async (req, res) => {
             send({ ev: 'review-definition', projectId: project.id, reviewDefinition: merged.definition });
           }
         }
-        goal.wantsPR = resolveWantsPR(undefined, goal.text, patch.defaultWantsPR);
+        goal.wantsPR = resolveWantsPR(undefined, goal.text, patch.defaultWantsPR, true);
         goal.question = null;
         goal.clarified = true;
         goal.status = 'stacked';
@@ -5587,7 +5589,7 @@ const server = createServer(async (req, res) => {
         // both asked twice and got no PR either time.
         // Upgrade only: this can turn the deliverable on, never off, so a later
         // unrelated reply cannot silently withdraw a PR the person already asked for.
-        if (resolveWantsPR('auto', text, false) && goal.wantsPR !== true) {
+        if (wantsPullRequest(text) && goal.wantsPR !== true) {
           goal.wantsPR = true;
           saveGoal(goal);
         }
@@ -5996,7 +5998,7 @@ async function answerGoalQuestion(goal, question) {
       // (rescope) or while asking a question (answer) is still a PR they asked
       // for, and it must survive until close-out rather than being dropped
       // because this particular reply happened not to wake a worker.
-      if (resolveWantsPR('auto', text, false)) goal.wantsPR = true;
+      if (wantsPullRequest(text)) goal.wantsPR = true;
       goal.status = result.status;
       saveGoal(goal);
       // rescope はワーカーを起こさない——「いったん止めて考え直す」なので、走り出したら

package/package.json

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