gsd-pi 2.60.0-dev.d9052f5 → 2.61.0-dev.7aed0bf

package/dist/resources/extensions/gsd/rule-registry.js

@@ -5,6 +5,7 @@
 // and encapsulates mutable hook state as instance fields.
 //
 // A module-level singleton accessor allows existing code to migrate incrementally.
+import { logWarning } from "./workflow-logger.js";
 import { resolvePostUnitHooks, resolvePreDispatchHooks } from "./preferences.js";
 import { existsSync, readFileSync, writeFileSync, mkdirSync } from "node:fs";
 import { join } from "node:path";
@@ -310,8 +311,8 @@ export class RuleRegistry {
                 mkdirSync(dir, { recursive: true });
             writeFileSync(this._hookStatePath(basePath), JSON.stringify(state, null, 2), "utf-8");
         }
-        catch {
-            // Non-fatal — state is recreatable from artifacts
+        catch (e) {
+            logWarning("registry", `failed to persist hook state: ${e.message}`);
         }
     }
     /** Restore hook cycle counts from disk after a crash/restart. */
@@ -331,8 +332,8 @@ export class RuleRegistry {
                 }
             }
         }
-        catch {
-            // Non-fatal — fresh state is fine
+        catch (e) {
+            logWarning("registry", `failed to restore hook state: ${e.message}`);
         }
     }
     /** Clear persisted hook state file from disk. */
@@ -343,8 +344,8 @@ export class RuleRegistry {
                 writeFileSync(filePath, JSON.stringify({ cycleCounts: {}, savedAt: new Date().toISOString() }, null, 2), "utf-8");
             }
         }
-        catch {
-            // Non-fatal
+        catch (e) {
+            logWarning("registry", `failed to clear hook state: ${e.message}`);
         }
     }
     // ── Hook status reporting ───────────────────────────────────────────

package/dist/resources/extensions/gsd/safe-fs.js

@@ -1,8 +1,9 @@
 import { existsSync, mkdirSync, cpSync } from "node:fs";
 import { dirname } from "node:path";
+import { logWarning } from "./workflow-logger.js";
 /**
  * Safely creates a directory. Returns true if successful, false on error.
- * Logs to stderr when GSD_DEBUG is set.
+ * Logs warnings via workflow-logger on failure.
  */
 export function safeMkdir(dirPath) {
     try {
@@ -10,14 +11,13 @@ export function safeMkdir(dirPath) {
         return true;
     }
     catch (err) {
-        if (process.env.GSD_DEBUG)
-            console.error(`[gsd] mkdir failed: ${dirPath}`, err);
+        logWarning("fs", `mkdir failed: ${dirPath}: ${err.message}`);
         return false;
     }
 }
 /**
  * Safely copies src to dst. Returns true if successful, false if src doesn't exist or copy fails.
- * Logs to stderr when GSD_DEBUG is set.
+ * Logs warnings via workflow-logger on failure.
  */
 export function safeCopy(src, dst, opts) {
     if (!existsSync(src))
@@ -27,8 +27,7 @@ export function safeCopy(src, dst, opts) {
         return true;
     }
     catch (err) {
-        if (process.env.GSD_DEBUG)
-            console.error(`[gsd] copy failed: ${src} → ${dst}`, err);
+        logWarning("fs", `copy failed: ${src} → ${dst}: ${err.message}`);
         return false;
     }
 }
@@ -45,8 +44,7 @@ export function safeCopyRecursive(src, dst, opts) {
         return true;
     }
     catch (err) {
-        if (process.env.GSD_DEBUG)
-            console.error(`[gsd] recursive copy failed: ${src} → ${dst}`, err);
+        logWarning("fs", `recursive copy failed: ${src} → ${dst}: ${err.message}`);
         return false;
     }
 }

package/dist/resources/extensions/gsd/tools/complete-milestone.js

@@ -15,6 +15,7 @@ import { invalidateStateCache } from "../state.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";
 function renderMilestoneSummaryMarkdown(params) {
     const now = new Date().toISOString();
     const keyDecisionsYaml = params.keyDecisions.length > 0
@@ -140,7 +141,7 @@ export async function handleCompleteMilestone(params, basePath) {
     }
     catch (renderErr) {
         // Disk render failed — roll back DB status so state stays consistent
-        process.stderr.write(`gsd-db: complete_milestone — disk render failed, rolling back DB status: ${renderErr.message}\n`);
+        logWarning("tool", `complete_milestone — disk render failed, rolling back DB status: ${renderErr.message}`);
         updateMilestoneStatus(params.milestoneId, 'active', null);
         invalidateStateCache();
         return { error: `disk render failed: ${renderErr.message}` };
@@ -163,7 +164,7 @@ export async function handleCompleteMilestone(params, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: complete-milestone post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `complete-milestone post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/complete-slice.js

@@ -18,6 +18,7 @@ import { renderRoadmapCheckboxes } 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";
 /**
  * Render slice summary markdown matching the template format.
  * YAML frontmatter uses snake_case keys for parseSummary() compatibility.
@@ -240,7 +241,7 @@ export async function handleCompleteSlice(params, basePath) {
     }
     catch (renderErr) {
         // Disk render failed — roll back DB status so state stays consistent
-        process.stderr.write(`gsd-db: complete_slice — disk render failed, rolling back DB status: ${renderErr.message}\n`);
+        logWarning("tool", `complete_slice — disk render failed, rolling back DB status: ${renderErr.message}`);
         updateSliceStatus(params.milestoneId, params.sliceId, 'pending');
         invalidateStateCache();
         return { error: `disk render failed: ${renderErr.message}` };
@@ -265,7 +266,7 @@ export async function handleCompleteSlice(params, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: complete-slice post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `complete-slice post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         sliceId: params.sliceId,

package/dist/resources/extensions/gsd/tools/complete-task.js

@@ -18,6 +18,7 @@ import { renderPlanCheckboxes } from "../markdown-renderer.js";
 import { renderAllProjections, renderSummaryContent } from "../workflow-projections.js";
 import { writeManifest } from "../workflow-manifest.js";
 import { appendEvent } from "../workflow-events.js";
+import { logWarning } from "../workflow-logger.js";
 /**
  * Build a TaskRow-shaped object from CompleteTaskParams so the unified
  * renderSummaryContent() can be used at completion time (#2720).
@@ -165,7 +166,7 @@ export async function handleCompleteTask(params, basePath) {
     }
     catch (renderErr) {
         // Disk render failed — roll back DB status so state stays consistent
-        process.stderr.write(`gsd-db: complete_task — disk render failed, rolling back DB status: ${renderErr.message}\n`);
+        logWarning("tool", `complete_task — disk render failed, rolling back DB status: ${renderErr.message}`);
         // Delete orphaned verification_evidence rows first (FK constraint
         // references tasks, so evidence must go before status change).
         // Without this, retries accumulate duplicate evidence rows (#2724).
@@ -194,7 +195,7 @@ export async function handleCompleteTask(params, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: complete-task post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `complete-task post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         taskId: params.taskId,

package/dist/resources/extensions/gsd/tools/plan-milestone.js

@@ -7,6 +7,7 @@ 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";
 function validateRiskEntries(value) {
     if (!Array.isArray(value)) {
         throw new Error("keyRisks must be an array");
@@ -221,7 +222,7 @@ export async function handlePlanMilestone(rawParams, basePath) {
         roadmapPath = renderResult.roadmapPath;
     }
     catch (renderErr) {
-        process.stderr.write(`gsd-db: plan_milestone — render failed (DB rows preserved for debugging): ${renderErr.message}\n`);
+        logWarning("tool", `plan_milestone — render failed (DB rows preserved for debugging): ${renderErr.message}`);
         invalidateStateCache();
         return { error: `render failed: ${renderErr.message}` };
     }
@@ -241,7 +242,7 @@ export async function handlePlanMilestone(rawParams, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: plan-milestone post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `plan-milestone post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/plan-slice.js

@@ -7,6 +7,7 @@ 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";
 function validateTasks(value) {
     if (!Array.isArray(value) || value.length === 0) {
         throw new Error("tasks must be a non-empty array");
@@ -182,7 +183,7 @@ export async function handlePlanSlice(rawParams, basePath) {
             });
         }
         catch (hookErr) {
-            process.stderr.write(`gsd: plan-slice post-mutation hook warning: ${hookErr.message}\n`);
+            logWarning("tool", `plan-slice post-mutation hook warning: ${hookErr.message}`);
         }
         return {
             milestoneId: params.milestoneId,
@@ -192,7 +193,7 @@ export async function handlePlanSlice(rawParams, basePath) {
         };
     }
     catch (renderErr) {
-        process.stderr.write(`gsd-db: plan_slice — render failed (DB rows preserved for debugging): ${renderErr.message}\n`);
+        logWarning("tool", `plan_slice — render failed (DB rows preserved for debugging): ${renderErr.message}`);
         invalidateStateCache();
         return { error: `render failed: ${renderErr.message}` };
     }

package/dist/resources/extensions/gsd/tools/plan-task.js

@@ -7,6 +7,7 @@ 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";
 function validateParams(params) {
     if (!isNonEmptyString(params?.milestoneId))
         throw new Error("milestoneId is required");
@@ -106,7 +107,7 @@ export async function handlePlanTask(rawParams, basePath) {
             });
         }
         catch (hookErr) {
-            process.stderr.write(`gsd: plan-task post-mutation hook warning: ${hookErr.message}\n`);
+            logWarning("tool", `plan-task post-mutation hook warning: ${hookErr.message}`);
         }
         return {
             milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/reassess-roadmap.js

@@ -9,6 +9,7 @@ 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";
 function validateParams(params) {
     if (!isNonEmptyString(params?.milestoneId))
         throw new Error("milestoneId is required");
@@ -182,9 +183,8 @@ export async function handleReassessRoadmap(rawParams, basePath) {
                 if (existsSync(validationFile))
                     unlinkSync(validationFile);
             }
-            catch {
-                // Best-effort: DB row is already deleted, so state derivation
-                // will not see the file-based verdict as authoritative.
+            catch (e) {
+                logWarning("tool", `validation file cleanup failed: ${e.message}`);
             }
         }
         // ── Invalidate caches ─────────────────────────────────────────
@@ -204,7 +204,7 @@ export async function handleReassessRoadmap(rawParams, basePath) {
             });
         }
         catch (hookErr) {
-            process.stderr.write(`gsd: reassess-roadmap post-mutation hook warning: ${hookErr.message}\n`);
+            logWarning("tool", `reassess-roadmap post-mutation hook warning: ${hookErr.message}`);
         }
         return {
             milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/reopen-slice.js

@@ -15,6 +15,7 @@ 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 async function handleReopenSlice(params, basePath) {
     // ── Validate required fields ────────────────────────────────────────────
     if (!params.sliceId || typeof params.sliceId !== "string" || params.sliceId.trim() === "") {
@@ -77,7 +78,7 @@ export async function handleReopenSlice(params, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: reopen-slice post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `reopen-slice post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/reopen-task.js

@@ -14,6 +14,7 @@ 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 async function handleReopenTask(params, basePath) {
     // ── Validate required fields ────────────────────────────────────────────
     if (!params.taskId || typeof params.taskId !== "string" || params.taskId.trim() === "") {
@@ -81,7 +82,7 @@ export async function handleReopenTask(params, basePath) {
         });
     }
     catch (hookErr) {
-        process.stderr.write(`gsd: reopen-task post-mutation hook warning: ${hookErr.message}\n`);
+        logWarning("tool", `reopen-task post-mutation hook warning: ${hookErr.message}`);
     }
     return {
         milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/replan-slice.js

@@ -7,6 +7,7 @@ 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";
 function validateParams(params) {
     if (!isNonEmptyString(params?.milestoneId))
         throw new Error("milestoneId is required");
@@ -173,7 +174,7 @@ export async function handleReplanSlice(rawParams, basePath) {
             });
         }
         catch (hookErr) {
-            process.stderr.write(`gsd: replan-slice post-mutation hook warning: ${hookErr.message}\n`);
+            logWarning("tool", `replan-slice post-mutation hook warning: ${hookErr.message}`);
         }
         return {
             milestoneId: params.milestoneId,

package/dist/resources/extensions/gsd/tools/validate-milestone.js

@@ -15,6 +15,7 @@ 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";
 function renderValidationMarkdown(params) {
     let md = `---
 verdict: ${params.verdict}
@@ -95,7 +96,7 @@ export async function handleValidateMilestone(params, basePath) {
         await saveFile(validationPath, validationMd);
     }
     catch (renderErr) {
-        process.stderr.write(`gsd-db: validate_milestone — disk render failed, rolling back DB row: ${renderErr.message}\n`);
+        logWarning("tool", `validate_milestone — disk render failed, rolling back DB row: ${renderErr.message}`);
         deleteAssessmentByScope(params.milestoneId, 'milestone-validation');
         return { error: `disk render failed: ${renderErr.message}` };
     }

package/dist/resources/extensions/gsd/triage-resolution.js

@@ -113,10 +113,17 @@ export function executeReplan(basePath, mid, sid, capture) {
  */
 export function executeBacktrack(basePath, currentMilestoneId, capture) {
     try {
-        // 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;
+        // 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;
         const ts = new Date().toISOString();
         const triggerPath = join(gsdRoot(basePath), "BACKTRACK-TRIGGER.md");
         const content = [

package/dist/resources/extensions/gsd/workflow-events.js

@@ -2,6 +2,7 @@ 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 ───────────────────────────────────────────────────────────
 /**
  * Engine-generated session ID — stable for the lifetime of this process.
@@ -49,7 +50,7 @@ export function readEvents(logPath) {
             events.push(JSON.parse(line));
         }
         catch {
-            process.stderr.write(`workflow-events: skipping corrupted event line: ${line.slice(0, 80)}\n`);
+            logWarning("event-log", `skipping corrupted event line (${line.length} bytes)`);
         }
     }
     return events;

package/dist/resources/extensions/gsd/workflow-logger.js

@@ -2,7 +2,9 @@
 // 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.
-// All entries are also persisted to .gsd/audit-log.jsonl for post-mortem analysis.
+// 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.
 //
 // Stderr policy: every logWarning/logError call writes immediately to stderr
 // for terminal visibility. This is intentional — unlike debug-logger (which is
@@ -177,12 +179,15 @@ function _push(severity, component, message, context) {
     if (_buffer.length > MAX_BUFFER) {
         _buffer.shift();
     }
-    // Persist to .gsd/audit-log.jsonl so entries survive context resets
-    if (_auditBasePath) {
+    // 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") {
         try {
             const auditDir = join(_auditBasePath, ".gsd");
             mkdirSync(auditDir, { recursive: true });
-            appendFileSync(join(auditDir, "audit-log.jsonl"), JSON.stringify(entry) + "\n", "utf-8");
+            const sanitized = _sanitizeForAudit(entry);
+            appendFileSync(join(auditDir, "audit-log.jsonl"), JSON.stringify(sanitized) + "\n", "utf-8");
         }
         catch (auditErr) {
             // Best-effort — never let audit write failures bubble up
@@ -190,3 +195,31 @@ function _push(severity, component, message, context) {
         }
     }
 }
+/**
+ * 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) {
+    const sanitized = {
+        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 = {};
+        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/dist/resources/extensions/gsd/workflow-migration.js

@@ -6,6 +6,7 @@ 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 ───────────────────────────────────────────────────
 /**
  * Returns true when engine tables are empty AND a .gsd/milestones/ directory
@@ -22,8 +23,8 @@ export function needsAutoMigration(basePath) {
         if (row && row["cnt"] > 0)
             return false;
     }
-    catch {
-        // Table might not exist yet — that's fine, we can still migrate
+    catch (e) {
+        logWarning("migration", `DB probe failed: ${e.message}`);
         return false;
     }
     // Check if .gsd/milestones/ directory exists
@@ -66,7 +67,7 @@ export function migrateFromMarkdown(basePath) {
             .map(e => e.name);
     }
     catch {
-        process.stderr.write("workflow-migration: failed to read milestones directory\n");
+        logWarning("migration", "failed to read milestones directory");
         return;
     }
     if (milestoneDirs.length === 0) {
@@ -102,7 +103,7 @@ export function migrateFromMarkdown(basePath) {
                 }));
             }
             catch (err) {
-                process.stderr.write(`workflow-migration: failed to parse ROADMAP.md for ${mId}: ${err.message}\n`);
+                logWarning("migration", `failed to parse ROADMAP.md for ${mId}: ${err.message}`);
                 // Still add milestone with ID as title
                 milestoneInserts.push({ id: mId, title: mId, status: milestoneStatus });
             }
@@ -148,7 +149,7 @@ export function migrateFromMarkdown(basePath) {
                     }
                 }
                 catch (err) {
-                    process.stderr.write(`workflow-migration: failed to parse ${slice.id}-PLAN.md for ${mId}: ${err.message}\n`);
+                    logWarning("migration", `failed to parse ${slice.id}-PLAN.md for ${mId}: ${err.message}`);
                 }
             }
         }
@@ -163,8 +164,8 @@ export function migrateFromMarkdown(basePath) {
                 }
             }
         }
-        catch {
-            // Non-fatal
+        catch (e) {
+            logWarning("migration", `Orphaned summary check failed for ${mId}: ${e.message}`);
         }
     }
     // Execute all inserts atomically
@@ -245,19 +246,20 @@ export function validateMigration(basePath) {
                                 const plan = parsePlan(planContent);
                                 mdTaskCount += plan.tasks.length;
                             }
-                            catch {
-                                // Skip unreadable plan
+                            catch (e) {
+                                logWarning("migration", `Failed to read plan ${slice.id}-PLAN.md: ${e.message}`);
                             }
                         }
                     }
                 }
-                catch {
-                    // Skip unreadable roadmap
+                catch (e) {
+                    logWarning("migration", `Failed to read roadmap for ${mId}: ${e.message}`);
                 }
             }
         }
     }
-    catch {
+    catch (e) {
+        logWarning("migration", `Validation failed to read markdown: ${e.message}`);
         return { discrepancies: ["Failed to read markdown for validation"] };
     }
     // Compare counts

package/dist/resources/extensions/gsd/workflow-projections.js

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

package/dist/resources/extensions/gsd/workflow-reconcile.js

@@ -375,8 +375,8 @@ function parseEventBlock(block) {
                         try {
                             params = JSON.parse(paramsMatch[1]);
                         }
-                        catch {
-                            // Keep empty params on parse error
+                        catch (e) {
+                            logWarning("reconcile", `tool call params parse failed: ${e.message}`);
                         }
                         i++; // consume params line
                     }