gsd-pi 2.60.0-dev.2580e65 → 2.60.0-dev.d9052f5

package/src/resources/extensions/gsd/tools/plan-milestone.ts

@@ -15,7 +15,6 @@ import { renderRoadmapFromDb } from "../markdown-renderer.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface PlanMilestoneSliceInput {
   sliceId: string;
@@ -270,7 +269,9 @@ export async function handlePlanMilestone(
     const renderResult = await renderRoadmapFromDb(basePath, params.milestoneId);
     roadmapPath = renderResult.roadmapPath;
   } catch (renderErr) {
-    logWarning("tool", `plan_milestone — render failed (DB rows preserved for debugging): ${(renderErr as Error).message}`);
+    process.stderr.write(
+      `gsd-db: plan_milestone — render failed (DB rows preserved for debugging): ${(renderErr as Error).message}\n`,
+    );
     invalidateStateCache();
     return { error: `render failed: ${(renderErr as Error).message}` };
   }
@@ -291,7 +292,9 @@ export async function handlePlanMilestone(
       trigger_reason: params.triggerReason,
     });
   } catch (hookErr) {
-    logWarning("tool", `plan-milestone post-mutation hook warning: ${(hookErr as Error).message}`);
+    process.stderr.write(
+      `gsd: plan-milestone post-mutation hook warning: ${(hookErr as Error).message}\n`,
+    );
   }
 
   return {

package/src/resources/extensions/gsd/tools/plan-slice.ts

@@ -16,7 +16,6 @@ import { renderPlanFromDb } from "../markdown-renderer.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface PlanSliceTaskInput {
   taskId: string;
@@ -230,7 +229,9 @@ export async function handlePlanSlice(
         trigger_reason: params.triggerReason,
       });
     } catch (hookErr) {
-      logWarning("tool", `plan-slice post-mutation hook warning: ${(hookErr as Error).message}`);
+      process.stderr.write(
+        `gsd: plan-slice post-mutation hook warning: ${(hookErr as Error).message}\n`,
+      );
     }
 
     return {
@@ -240,7 +241,9 @@ export async function handlePlanSlice(
       taskPlanPaths: renderResult.taskPlanPaths,
     };
   } catch (renderErr) {
-    logWarning("tool", `plan_slice — render failed (DB rows preserved for debugging): ${(renderErr as Error).message}`);
+    process.stderr.write(
+      `gsd-db: plan_slice — render failed (DB rows preserved for debugging): ${(renderErr as Error).message}\n`,
+    );
     invalidateStateCache();
     return { error: `render failed: ${(renderErr as Error).message}` };
   }

package/src/resources/extensions/gsd/tools/plan-task.ts

@@ -7,7 +7,6 @@ import { renderTaskPlanFromDb } from "../markdown-renderer.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface PlanTaskParams {
   milestoneId: string;
@@ -136,7 +135,9 @@ export async function handlePlanTask(
         trigger_reason: params.triggerReason,
       });
     } catch (hookErr) {
-      logWarning("tool", `plan-task post-mutation hook warning: ${(hookErr as Error).message}`);
+      process.stderr.write(
+        `gsd: plan-task post-mutation hook warning: ${(hookErr as Error).message}\n`,
+      );
     }
 
     return {

package/src/resources/extensions/gsd/tools/reassess-roadmap.ts

@@ -19,7 +19,6 @@ import { renderRoadmapFromDb, renderAssessmentFromDb } from "../markdown-rendere
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface SliceChangeInput {
   sliceId: string;
@@ -249,8 +248,9 @@ export async function handleReassessRoadmap(
       );
       try {
         if (existsSync(validationFile)) unlinkSync(validationFile);
-      } catch (e) {
-        logWarning("tool", `validation file cleanup failed: ${(e as Error).message}`);
+      } catch {
+        // Best-effort: DB row is already deleted, so state derivation
+        // will not see the file-based verdict as authoritative.
       }
     }
 
@@ -271,7 +271,9 @@ export async function handleReassessRoadmap(
         trigger_reason: params.triggerReason,
       });
     } catch (hookErr) {
-      logWarning("tool", `reassess-roadmap post-mutation hook warning: ${(hookErr as Error).message}`);
+      process.stderr.write(
+        `gsd: reassess-roadmap post-mutation hook warning: ${(hookErr as Error).message}\n`,
+      );
     }
 
     return {

package/src/resources/extensions/gsd/tools/reopen-slice.ts

@@ -24,7 +24,6 @@ import { isClosedStatus } from "../status-guards.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface ReopenSliceParams {
   milestoneId: string;
@@ -114,7 +113,9 @@ export async function handleReopenSlice(
       trigger_reason: params.triggerReason,
     });
   } catch (hookErr) {
-    logWarning("tool", `reopen-slice post-mutation hook warning: ${(hookErr as Error).message}`);
+    process.stderr.write(
+      `gsd: reopen-slice post-mutation hook warning: ${(hookErr as Error).message}\n`,
+    );
   }
 
   return {

package/src/resources/extensions/gsd/tools/reopen-task.ts

@@ -22,7 +22,6 @@ import { isClosedStatus } from "../status-guards.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface ReopenTaskParams {
   milestoneId: string;
@@ -118,7 +117,9 @@ export async function handleReopenTask(
       trigger_reason: params.triggerReason,
     });
   } catch (hookErr) {
-    logWarning("tool", `reopen-task post-mutation hook warning: ${(hookErr as Error).message}`);
+    process.stderr.write(
+      `gsd: reopen-task post-mutation hook warning: ${(hookErr as Error).message}\n`,
+    );
   }
 
   return {

package/src/resources/extensions/gsd/tools/replan-slice.ts

@@ -16,7 +16,6 @@ import { renderPlanFromDb, renderReplanFromDb } from "../markdown-renderer.js";
 import { renderAllProjections } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface ReplanSliceTaskInput {
   taskId: string;
@@ -227,7 +226,9 @@ export async function handleReplanSlice(
         trigger_reason: params.triggerReason,
       });
     } catch (hookErr) {
-      logWarning("tool", `replan-slice post-mutation hook warning: ${(hookErr as Error).message}`);
+      process.stderr.write(
+        `gsd: replan-slice post-mutation hook warning: ${(hookErr as Error).message}\n`,
+      );
     }
 
     return {

package/src/resources/extensions/gsd/tools/validate-milestone.ts

@@ -22,7 +22,6 @@ import { saveFile, clearParseCache } from "../files.js";
 import { invalidateStateCache } from "../state.js";
 import { VALIDATION_VERDICTS, isValidMilestoneVerdict } from "../verdict-parser.js";
 import { insertMilestoneValidationGates } from "../milestone-validation-gates.js";
-import { logWarning } from "../workflow-logger.js";
 
 export interface ValidateMilestoneParams {
   milestoneId: string;
@@ -138,7 +137,9 @@ export async function handleValidateMilestone(
   try {
     await saveFile(validationPath, validationMd);
   } catch (renderErr) {
-    logWarning("tool", `validate_milestone — disk render failed, rolling back DB row: ${(renderErr as Error).message}`);
+    process.stderr.write(
+      `gsd-db: validate_milestone — disk render failed, rolling back DB row: ${(renderErr as Error).message}\n`,
+    );
     deleteAssessmentByScope(params.milestoneId, 'milestone-validation');
     return { error: `disk render failed: ${(renderErr as Error).message}` };
   }

package/src/resources/extensions/gsd/triage-resolution.ts

@@ -148,17 +148,10 @@ export function executeBacktrack(
   capture: CaptureEntry,
 ): string | null {
   try {
-    // Extract target milestone from capture text or resolution.
-    // Filter out the current milestone ID to avoid picking it as the backtrack target
-    // when the text mentions both current and target milestones (e.g. "backtrack from M004 to M003").
-    const sourceText = capture.resolution ?? capture.text;
-    const allMatches = [...sourceText.matchAll(/\b(M\d{3}(?:-[a-z0-9]{6})?)\b/g)]
-      .map(m => m[1])
-      .filter(id => id !== currentMilestoneId);
-    // Reject ambiguous multi-target strings — if more than one distinct target remains,
-    // don't guess; let the user clarify.
-    const uniqueTargets = [...new Set(allMatches)];
-    const targetMilestoneId = uniqueTargets.length === 1 ? uniqueTargets[0] : null;
+    // Extract target milestone from capture text or resolution
+    const targetMatch = (capture.resolution ?? capture.text)
+      .match(/\b(M\d{3}(?:-[a-z0-9]{6})?)\b/);
+    const targetMilestoneId = targetMatch?.[1] ?? null;
 
     const ts = new Date().toISOString();
     const triggerPath = join(gsdRoot(basePath), "BACKTRACK-TRIGGER.md");

package/src/resources/extensions/gsd/types.ts

@@ -316,7 +316,6 @@ export interface ClassificationResult {
   tier: ComplexityTier;
   reason: string;
   downgraded: boolean;
-  taskMetadata?: TaskMetadata;
 }
 
 export interface TaskMetadata {

package/src/resources/extensions/gsd/workflow-events.ts

@@ -2,7 +2,6 @@ import { createHash, randomUUID } from "node:crypto";
 import { appendFileSync, readFileSync, existsSync, mkdirSync } from "node:fs";
 import { join } from "node:path";
 import { atomicWriteSync } from "./atomic-write.js";
-import { logWarning } from "./workflow-logger.js";
 
 // ─── Session ID ───────────────────────────────────────────────────────────
 
@@ -75,7 +74,7 @@ export function readEvents(logPath: string): WorkflowEvent[] {
     try {
       events.push(JSON.parse(line) as WorkflowEvent);
     } catch {
-      logWarning("event-log", `skipping corrupted event line (${line.length} bytes)`);
+      process.stderr.write(`workflow-events: skipping corrupted event line: ${line.slice(0, 80)}\n`);
     }
   }
 

package/src/resources/extensions/gsd/workflow-logger.ts

@@ -2,9 +2,7 @@
 // Centralized warning/error accumulator for the workflow engine pipeline.
 // Captures structured entries that the auto-loop can drain after each unit
 // to surface root causes for stuck loops, silent degradation, and blocked writes.
-// Error-severity entries are persisted to .gsd/audit-log.jsonl (sanitized) for
-// post-mortem analysis. Warnings are ephemeral (stderr + buffer only) to avoid
-// log amplification from expected-control-flow catch paths.
+// All entries are also persisted to .gsd/audit-log.jsonl for post-mortem analysis.
 //
 // Stderr policy: every logWarning/logError call writes immediately to stderr
 // for terminal visibility. This is intentional — unlike debug-logger (which is
@@ -35,20 +33,7 @@ export type LogComponent =
   | "compaction"    // Event compaction
   | "reconcile"     // Worktree reconciliation
   | "db"            // Database operations (gsd-db)
-  | "dispatch"      // Auto-dispatch rule evaluation
-  | "recovery"      // Auto-recovery and timeout recovery
-  | "session"       // Session lock and session state I/O
-  | "prompt"        // Prompt construction and context injection
-  | "dashboard"     // Auto-dashboard rendering
-  | "timer"         // Auto-timers (idle watchdog, hard timeout)
-  | "worktree"      // Worktree lifecycle (create, sync, merge)
-  | "command"       // Slash command execution and maintenance
-  | "parallel"      // Parallel orchestrator and merge
-  | "fs"            // Safe filesystem operations
-  | "bootstrap"     // Extension bootstrap (system-context, agent-end)
-  | "guided"        // Guided flow (discuss, plan wizards)
-  | "registry"      // Rule registry hook state
-  | "renderer";     // Markdown renderer and projections
+  | "dispatch";     // Auto-dispatch rule evaluation
 
 export interface LogEntry {
   ts: string;
@@ -245,47 +230,15 @@ function _push(
     _buffer.shift();
   }
 
-  // Persist errors to .gsd/audit-log.jsonl so they survive context resets.
-  // Only error-severity entries are persisted — warnings are ephemeral (stderr + buffer)
-  // to avoid log amplification from expected-control-flow catch paths.
-  if (_auditBasePath && severity === "error") {
+  // Persist to .gsd/audit-log.jsonl so entries survive context resets
+  if (_auditBasePath) {
     try {
       const auditDir = join(_auditBasePath, ".gsd");
       mkdirSync(auditDir, { recursive: true });
-      const sanitized = _sanitizeForAudit(entry);
-      appendFileSync(join(auditDir, "audit-log.jsonl"), JSON.stringify(sanitized) + "\n", "utf-8");
+      appendFileSync(join(auditDir, "audit-log.jsonl"), JSON.stringify(entry) + "\n", "utf-8");
     } catch (auditErr) {
       // Best-effort — never let audit write failures bubble up
       process.stderr.write(`[gsd:audit] failed to persist log entry: ${(auditErr as Error).message}\n`);
     }
   }
 }
-
-/**
- * Sanitize a log entry before persisting to the audit JSONL file.
- * Strips potentially sensitive context (raw paths, cwd, full error text)
- * to avoid leaking local environment details into durable telemetry.
- */
-function _sanitizeForAudit(entry: LogEntry): LogEntry {
-  const sanitized: LogEntry = {
-    ts: entry.ts,
-    severity: entry.severity,
-    component: entry.component,
-    // Truncate message to avoid persisting oversized raw error dumps
-    message: entry.message.length > 200 ? entry.message.slice(0, 200) + "…[truncated]" : entry.message,
-  };
-  if (entry.context) {
-    // Allowlist: only persist known-safe structured keys
-    const SAFE_KEYS = new Set(["fn", "tool", "mid", "sid", "tid", "worktree"]);
-    const filtered: Record<string, string> = {};
-    for (const [k, v] of Object.entries(entry.context)) {
-      if (SAFE_KEYS.has(k)) {
-        filtered[k] = v;
-      }
-    }
-    if (Object.keys(filtered).length > 0) {
-      sanitized.context = filtered;
-    }
-  }
-  return sanitized;
-}

package/src/resources/extensions/gsd/workflow-migration.ts

@@ -7,7 +7,6 @@ import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import { _getAdapter, transaction } from "./gsd-db.js";
 import { parseRoadmap, parsePlan } from "./parsers-legacy.js";
-import { logWarning } from "./workflow-logger.js";
 
 // ─── needsAutoMigration ───────────────────────────────────────────────────
 
@@ -24,8 +23,8 @@ export function needsAutoMigration(basePath: string): boolean {
   try {
     const row = db.prepare("SELECT COUNT(*) as cnt FROM milestones").get();
     if (row && (row["cnt"] as number) > 0) return false;
-  } catch (e) {
-    logWarning("migration", `DB probe failed: ${(e as Error).message}`);
+  } catch {
+    // Table might not exist yet — that's fine, we can still migrate
     return false;
   }
 
@@ -72,7 +71,7 @@ export function migrateFromMarkdown(basePath: string): void {
       .filter(e => e.isDirectory())
       .map(e => e.name);
   } catch {
-    logWarning("migration", "failed to read milestones directory");
+    process.stderr.write("workflow-migration: failed to read milestones directory\n");
     return;
   }
 
@@ -142,7 +141,7 @@ export function migrateFromMarkdown(basePath: string): void {
           risk: s.risk || "low",
         }));
       } catch (err) {
-        logWarning("migration", `failed to parse ROADMAP.md for ${mId}: ${(err as Error).message}`);
+        process.stderr.write(`workflow-migration: failed to parse ROADMAP.md for ${mId}: ${(err as Error).message}\n`);
         // Still add milestone with ID as title
         milestoneInserts.push({ id: mId, title: mId, status: milestoneStatus });
       }
@@ -192,7 +191,7 @@ export function migrateFromMarkdown(basePath: string): void {
             });
           }
         } catch (err) {
-          logWarning("migration", `failed to parse ${slice.id}-PLAN.md for ${mId}: ${(err as Error).message}`);
+          process.stderr.write(`workflow-migration: failed to parse ${slice.id}-PLAN.md for ${mId}: ${(err as Error).message}\n`);
         }
       }
     }
@@ -207,8 +206,8 @@ export function migrateFromMarkdown(basePath: string): void {
           process.stderr.write(`workflow-migration: orphaned summary file ${summaryFile} in ${mId} (slice not found in ROADMAP.md), skipping\n`);
         }
       }
-    } catch (e) {
-      logWarning("migration", `Orphaned summary check failed for ${mId}: ${(e as Error).message}`);
+    } catch {
+      // Non-fatal
     }
   }
 
@@ -309,18 +308,17 @@ export function validateMigration(basePath: string): { discrepancies: string[] }
                 const planContent = readFileSync(planPath, "utf-8");
                 const plan = parsePlan(planContent);
                 mdTaskCount += plan.tasks.length;
-              } catch (e) {
-                logWarning("migration", `Failed to read plan ${slice.id}-PLAN.md: ${(e as Error).message}`);
+              } catch {
+                // Skip unreadable plan
               }
             }
           }
-        } catch (e) {
-          logWarning("migration", `Failed to read roadmap for ${mId}: ${(e as Error).message}`);
+        } catch {
+          // Skip unreadable roadmap
         }
       }
     }
-  } catch (e) {
-    logWarning("migration", `Validation failed to read markdown: ${(e as Error).message}`);
+  } catch {
     return { discrepancies: ["Failed to read markdown for validation"] };
   }
 

package/src/resources/extensions/gsd/workflow-projections.ts

@@ -423,7 +423,7 @@ export function regenerateIfMissing(
           renderSummaryProjection(basePath, milestoneId, sliceId, task.id);
           regenerated++;
         } catch (err) {
-          logWarning("projection", `regenerateIfMissing SUMMARY failed for ${task.id}: ${(err as Error).message}`);
+          console.error(`[projections] regenerateIfMissing SUMMARY failed for ${task.id}:`, err);
         }
       }
     }
@@ -452,7 +452,7 @@ export function regenerateIfMissing(
     }
     return true;
   } catch (err) {
-    logWarning("projection", `regenerateIfMissing ${fileType} failed: ${(err as Error).message}`);
+    console.error(`[projections] regenerateIfMissing ${fileType} failed:`, err);
     return false;
   }
 }

package/src/resources/extensions/gsd/workflow-reconcile.ts

@@ -455,8 +455,8 @@ function parseEventBlock(block: string): WorkflowEvent[] {
           if (paramsMatch) {
             try {
               params = JSON.parse(paramsMatch[1]!) as Record<string, unknown>;
-            } catch (e) {
-              logWarning("reconcile", `tool call params parse failed: ${(e as Error).message}`);
+            } catch {
+              // Keep empty params on parse error
             }
             i++; // consume params line
           }

package/src/resources/extensions/gsd/worktree-manager.ts

@@ -95,8 +95,8 @@ export function resolveGitDir(basePath: string): string {
     if (content.startsWith("gitdir: ")) {
       return resolve(basePath, content.slice(8));
     }
-  } catch (e) {
-    logWarning("worktree", `.git file read failed: ${(e as Error).message}`);
+  } catch {
+    // Not a file or unreadable — fall through to default
   }
   return join(basePath, ".git");
 }
@@ -308,9 +308,8 @@ export function findNestedGitDirs(rootPath: string): string[] {
     let entries: string[];
     try {
       entries = readdirSync(dir);
-    } catch (e) {
-      logWarning("worktree", `readdirSync failed: ${(e as Error).message}`);
-      return;
+    } catch {
+      return; // Permission denied, broken symlink, etc.
     }
 
     for (const entry of entries) {
@@ -322,8 +321,7 @@ export function findNestedGitDirs(rootPath: string): string[] {
       let stat;
       try {
         stat = lstatSync(fullPath);
-      } catch (e) {
-        logWarning("worktree", `lstatSync failed for ${fullPath}: ${(e as Error).message}`);
+      } catch {
         continue;
       }
       if (!stat.isDirectory()) continue;
@@ -339,8 +337,8 @@ export function findNestedGitDirs(rootPath: string): string[] {
           // Don't recurse into the nested repo — we found what we need
           continue;
         }
-      } catch (e) {
-        logWarning("worktree", `existsSync/.git check failed for ${fullPath}: ${(e as Error).message}`);
+      } catch {
+        // No .git here — continue scanning
       }
 
       walk(fullPath, depth + 1);
@@ -376,7 +374,7 @@ export function removeWorktree(
     if (entry?.path) {
       wtPath = entry.path;
     }
-  } catch (e) { logWarning("worktree", `nativeWorktreeList parse failed: ${(e as Error).message}`); }
+  } catch { /* fall back to computed path */ }
 
   const resolvedWtPath = existsSync(wtPath) ? realpathSync(wtPath) : wtPath;
 
@@ -390,7 +388,7 @@ export function removeWorktree(
   if (!existsSync(wtPath)) {
     nativeWorktreePrune(basePath);
     if (deleteBranch) {
-      try { nativeBranchDelete(basePath, branch, true); } catch (e) { logWarning("worktree", `nativeBranchDelete failed: ${(e as Error).message}`); }
+      try { nativeBranchDelete(basePath, branch, true); } catch { /* branch may not exist */ }
     }
     return;
   }
@@ -424,8 +422,8 @@ export function removeWorktree(
           logWarning("reconcile", `Submodule changes detected — stash failed, changes may be lost during force removal`, { worktree: name, path: resolvedWtPath });
         }
       }
-    } catch (e) {
-      logWarning("worktree", `submodule status check failed: ${(e as Error).message}`);
+    } catch {
+      // submodule status failed — proceed with normal removal
     }
   }
 
@@ -456,11 +454,11 @@ export function removeWorktree(
   // Remove worktree: try non-force first when submodules have changes,
   // falling back to force only after submodule state has been preserved.
   const useForce = hasSubmoduleChanges ? false : force;
-  try { nativeWorktreeRemove(basePath, resolvedWtPath, useForce); } catch (e) { logWarning("worktree", `nativeWorktreeRemove failed: ${(e as Error).message}`); }
+  try { nativeWorktreeRemove(basePath, resolvedWtPath, useForce); } catch { /* may fail */ }
 
   // If the directory is still there (e.g. locked), try harder with force
   if (existsSync(resolvedWtPath)) {
-    try { nativeWorktreeRemove(basePath, resolvedWtPath, true); } catch (e) { logWarning("worktree", `nativeWorktreeRemove (force) failed: ${(e as Error).message}`); }
+    try { nativeWorktreeRemove(basePath, resolvedWtPath, true); } catch { /* may fail */ }
   }
 
   // (#2821) If the worktree directory STILL exists after both native removal
@@ -490,7 +488,7 @@ export function removeWorktree(
   nativeWorktreePrune(basePath);
 
   if (deleteBranch) {
-    try { nativeBranchDelete(basePath, branch, true); } catch (e) { logWarning("worktree", `final branch delete failed: ${(e as Error).message}`); }
+    try { nativeBranchDelete(basePath, branch, true); } catch { /* branch may not exist */ }
   }
 }
 

package/src/resources/extensions/shared/interview-ui.ts

@@ -298,9 +298,7 @@ export async function showInterviewRound(
 			// Auto-open the notes field when "None of the above" is selected
 			// so the user can immediately provide a free-text explanation
 			// instead of being trapped in a re-asking loop (bug #2715).
-			// Only auto-open if the user hasn't already provided notes —
-			// otherwise Enter from notes mode loops back here endlessly.
-			if (!isMultiSelect(currentIdx) && states[currentIdx].cursorIndex === noneOrDoneIdx(currentIdx) && !states[currentIdx].notes) {
+			if (!isMultiSelect(currentIdx) && states[currentIdx].cursorIndex === noneOrDoneIdx(currentIdx)) {
 				states[currentIdx].notesVisible = true;
 				focusNotes = true;
 				loadStateToEditor();

package/packages/pi-coding-agent/dist/core/lsp/lsp-legacy-alias.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=lsp-legacy-alias.test.d.ts.map

package/packages/pi-coding-agent/dist/core/lsp/lsp-legacy-alias.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"lsp-legacy-alias.test.d.ts","sourceRoot":"","sources":["../../../src/core/lsp/lsp-legacy-alias.test.ts"],"names":[],"mappings":""}

package/packages/pi-coding-agent/dist/core/lsp/lsp-legacy-alias.test.js

@@ -1,47 +0,0 @@
-// GSD2 — Regression test for LSP legacy server key aliases
-// Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>
-/**
- * When a default server key is renamed (e.g., kotlin-language-server → kotlin-lsp),
- * user overrides referencing the old key must still merge correctly via LEGACY_ALIASES.
- *
- * This test exercises the merge path through loadConfig() with a temp project
- * containing an lsp.json that uses the legacy key.
- */
-import { describe, it, beforeEach, afterEach } from "node:test";
-import assert from "node:assert/strict";
-import * as fs from "node:fs";
-import * as path from "node:path";
-import * as os from "node:os";
-import { loadConfig } from "./config.js";
-describe("LSP legacy server key aliases", () => {
-    let tmpDir;
-    beforeEach(() => {
-        tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "lsp-alias-test-"));
-    });
-    afterEach(() => {
-        fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-    it("merges user override with legacy key 'kotlin-language-server' into 'kotlin-lsp'", () => {
-        // Write an lsp.json that uses the old key name with a command that exists (node)
-        // so resolveCommand doesn't filter it out.
-        const overrideConfig = {
-            servers: {
-                "kotlin-language-server": {
-                    command: "node",
-                },
-            },
-        };
-        fs.writeFileSync(path.join(tmpDir, "lsp.json"), JSON.stringify(overrideConfig));
-        // Also add root markers so the server is detected
-        fs.writeFileSync(path.join(tmpDir, "build.gradle.kts"), "");
-        const config = loadConfig(tmpDir);
-        // The merged config should have kotlin-lsp (new key) with the user's command override
-        const kotlinServer = config.servers["kotlin-lsp"];
-        assert.ok(kotlinServer, "kotlin-lsp should exist in merged config");
-        assert.equal(kotlinServer.command, "node", "command should be overridden from user config via legacy alias");
-        assert.ok(kotlinServer.fileTypes.includes(".kt"), "fileTypes should be inherited from defaults");
-        // The old key should NOT appear as a separate entry
-        assert.equal(config.servers["kotlin-language-server"], undefined, "legacy key should not appear as separate server");
-    });
-});
-//# sourceMappingURL=lsp-legacy-alias.test.js.map

package/packages/pi-coding-agent/dist/core/lsp/lsp-legacy-alias.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"lsp-legacy-alias.test.js","sourceRoot":"","sources":["../../../src/core/lsp/lsp-legacy-alias.test.ts"],"names":[],"mappings":"AAAA,2DAA2D;AAC3D,4DAA4D;AAE5D;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAChE,OAAO,MAAM,MAAM,oBAAoB,CAAC;AACxC,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAClC,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAEzC,QAAQ,CAAC,+BAA+B,EAAE,GAAG,EAAE;IAC9C,IAAI,MAAc,CAAC;IAEnB,UAAU,CAAC,GAAG,EAAE;QACf,MAAM,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,iBAAiB,CAAC,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iFAAiF,EAAE,GAAG,EAAE;QAC1F,iFAAiF;QACjF,2CAA2C;QAC3C,MAAM,cAAc,GAAG;YACtB,OAAO,EAAE;gBACR,wBAAwB,EAAE;oBACzB,OAAO,EAAE,MAAM;iBACf;aACD;SACD,CAAC;QACF,EAAE,CAAC,aAAa,CACf,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,CAC9B,CAAC;QAEF,kDAAkD;QAClD,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,kBAAkB,CAAC,EAAE,EAAE,CAAC,CAAC;QAE5D,MAAM,MAAM,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC;QAElC,sFAAsF;QACtF,MAAM,YAAY,GAAG,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;QAClD,MAAM,CAAC,EAAE,CAAC,YAAY,EAAE,0CAA0C,CAAC,CAAC;QACpE,MAAM,CAAC,KAAK,CACX,YAAY,CAAC,OAAO,EACpB,MAAM,EACN,gEAAgE,CAChE,CAAC;QACF,MAAM,CAAC,EAAE,CACR,YAAY,CAAC,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,EACtC,6CAA6C,CAC7C,CAAC;QAEF,oDAAoD;QACpD,MAAM,CAAC,KAAK,CACX,MAAM,CAAC,OAAO,CAAC,wBAAwB,CAAC,EACxC,SAAS,EACT,iDAAiD,CACjD,CAAC;IACH,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC","sourcesContent":["// GSD2 — Regression test for LSP legacy server key aliases\n// Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>\n\n/**\n * When a default server key is renamed (e.g., kotlin-language-server → kotlin-lsp),\n * user overrides referencing the old key must still merge correctly via LEGACY_ALIASES.\n *\n * This test exercises the merge path through loadConfig() with a temp project\n * containing an lsp.json that uses the legacy key.\n */\n\nimport { describe, it, beforeEach, afterEach } from \"node:test\";\nimport assert from \"node:assert/strict\";\nimport * as fs from \"node:fs\";\nimport * as path from \"node:path\";\nimport * as os from \"node:os\";\nimport { loadConfig } from \"./config.js\";\n\ndescribe(\"LSP legacy server key aliases\", () => {\n\tlet tmpDir: string;\n\n\tbeforeEach(() => {\n\t\ttmpDir = fs.mkdtempSync(path.join(os.tmpdir(), \"lsp-alias-test-\"));\n\t});\n\n\tafterEach(() => {\n\t\tfs.rmSync(tmpDir, { recursive: true, force: true });\n\t});\n\n\tit(\"merges user override with legacy key 'kotlin-language-server' into 'kotlin-lsp'\", () => {\n\t\t// Write an lsp.json that uses the old key name with a command that exists (node)\n\t\t// so resolveCommand doesn't filter it out.\n\t\tconst overrideConfig = {\n\t\t\tservers: {\n\t\t\t\t\"kotlin-language-server\": {\n\t\t\t\t\tcommand: \"node\",\n\t\t\t\t},\n\t\t\t},\n\t\t};\n\t\tfs.writeFileSync(\n\t\t\tpath.join(tmpDir, \"lsp.json\"),\n\t\t\tJSON.stringify(overrideConfig),\n\t\t);\n\n\t\t// Also add root markers so the server is detected\n\t\tfs.writeFileSync(path.join(tmpDir, \"build.gradle.kts\"), \"\");\n\n\t\tconst config = loadConfig(tmpDir);\n\n\t\t// The merged config should have kotlin-lsp (new key) with the user's command override\n\t\tconst kotlinServer = config.servers[\"kotlin-lsp\"];\n\t\tassert.ok(kotlinServer, \"kotlin-lsp should exist in merged config\");\n\t\tassert.equal(\n\t\t\tkotlinServer.command,\n\t\t\t\"node\",\n\t\t\t\"command should be overridden from user config via legacy alias\",\n\t\t);\n\t\tassert.ok(\n\t\t\tkotlinServer.fileTypes.includes(\".kt\"),\n\t\t\t\"fileTypes should be inherited from defaults\",\n\t\t);\n\n\t\t// The old key should NOT appear as a separate entry\n\t\tassert.equal(\n\t\t\tconfig.servers[\"kotlin-language-server\"],\n\t\t\tundefined,\n\t\t\t\"legacy key should not appear as separate server\",\n\t\t);\n\t});\n});\n"]}