gsd-pi 2.59.0-dev.d77b3dd → 2.60.0-dev.2580e65

package/dist/resources/extensions/gsd/complexity-classifier.js

@@ -38,15 +38,18 @@ const UNIT_TYPE_TIERS = {
 export function classifyUnitComplexity(unitType, unitId, basePath, budgetPct, metadata) {
     // Hook units default to light
     if (unitType.startsWith("hook/")) {
-        const result = { tier: "light", reason: "hook unit", downgraded: false };
+        const result = { tier: "light", reason: "hook unit", downgraded: false, taskMetadata: undefined };
         return applyBudgetPressure(result, budgetPct);
     }
     // Start with the default tier for this unit type
     let tier = UNIT_TYPE_TIERS[unitType] ?? "standard";
     let reason = `unit type: ${unitType}`;
+    let taskMeta;
     // For execute-task, analyze task metadata for complexity signals
     if (unitType === "execute-task") {
-        const taskAnalysis = analyzeTaskComplexity(unitId, basePath, metadata);
+        // Extract metadata once and reuse throughout to avoid double-extraction
+        taskMeta = metadata ?? extractTaskMetadata(unitId, basePath);
+        const taskAnalysis = analyzeTaskComplexity(unitId, basePath, taskMeta);
         tier = taskAnalysis.tier;
         reason = taskAnalysis.reason;
     }
@@ -59,13 +62,14 @@ export function classifyUnitComplexity(unitType, unitId, basePath, budgetPct, me
         }
     }
     // Adaptive learning: check if history suggests bumping the tier
-    const tags = metadata?.tags ?? extractTaskMetadata(unitId, basePath).tags;
+    // Use already-extracted taskMeta.tags if available to avoid double-extraction
+    const tags = taskMeta?.tags ?? metadata?.tags;
     const adaptiveAdjustment = getAdaptiveTierAdjustment(unitType, tier, tags);
     if (adaptiveAdjustment && tierOrdinal(adaptiveAdjustment) > tierOrdinal(tier)) {
         reason = `${reason} (adaptive: high failure rate at ${tier})`;
         tier = adaptiveAdjustment;
     }
-    const result = { tier, reason, downgraded: false };
+    const result = { tier, reason, downgraded: false, taskMetadata: taskMeta };
     return applyBudgetPressure(result, budgetPct);
 }
 /**

package/dist/resources/extensions/gsd/custom-verification.js

@@ -16,6 +16,7 @@
  * - content-heuristic logs the specific failure (missing file, below minSize, pattern mismatch).
  * - The frozen DEFINITION.yaml on disk is the single source of truth for step policies.
  */
+import { logWarning } from "./workflow-logger.js";
 import { readFileSync, existsSync, statSync } from "node:fs";
 import { resolve, sep } from "node:path";
 import { spawnSync } from "node:child_process";
@@ -105,8 +106,8 @@ function handleContentHeuristic(runDir, step, verify) {
                     return "pause";
                 }
             }
-            catch {
-                // Invalid regex at runtime — treat as verification failure
+            catch (e) {
+                logWarning("engine", `content-heuristic regex failed: ${e.message}`);
                 return "pause";
             }
         }

package/dist/resources/extensions/gsd/gsd-db.js

@@ -8,7 +8,7 @@ import { createRequire } from "node:module";
 import { existsSync, copyFileSync, mkdirSync, realpathSync } from "node:fs";
 import { dirname } from "node:path";
 import { GSDError, GSD_STALE_STATE } from "./errors.js";
-import { logError } from "./workflow-logger.js";
+import { logError, logWarning } from "./workflow-logger.js";
 const _require = createRequire(import.meta.url);
 let providerName = null;
 let providerModule = null;
@@ -710,7 +710,9 @@ export function openDatabase(path) {
                 try {
                     adapter.close();
                 }
-                catch { /* swallow */ }
+                catch (e) {
+                    logWarning("db", `close after VACUUM failed: ${e.message}`);
+                }
                 throw retryErr;
             }
         }
@@ -718,7 +720,9 @@ export function openDatabase(path) {
             try {
                 adapter.close();
             }
-            catch { /* swallow */ }
+            catch (e) {
+                logWarning("db", `close after VACUUM failed: ${e.message}`);
+            }
             throw err;
         }
     }
@@ -730,7 +734,9 @@ export function openDatabase(path) {
         process.on("exit", () => { try {
             closeDatabase();
         }
-        catch { } });
+        catch (e) {
+            logWarning("db", `exit handler close failed: ${e.message}`);
+        } });
     }
     return true;
 }
@@ -739,17 +745,21 @@ export function closeDatabase() {
         try {
             currentDb.exec('PRAGMA wal_checkpoint(TRUNCATE)');
         }
-        catch { /* non-fatal — best effort before close */ }
+        catch (e) {
+            logWarning("db", `WAL checkpoint failed: ${e.message}`);
+        }
         try {
             // Incremental vacuum to reclaim space without blocking
             currentDb.exec('PRAGMA incremental_vacuum(64)');
         }
-        catch { /* non-fatal */ }
+        catch (e) {
+            logWarning("db", `incremental vacuum failed: ${e.message}`);
+        }
         try {
             currentDb.close();
         }
-        catch {
-            // swallow close errors
+        catch (e) {
+            logWarning("db", `database close failed: ${e.message}`);
         }
         currentDb = null;
         currentPath = null;
@@ -763,7 +773,9 @@ export function vacuumDatabase() {
     try {
         currentDb.exec('VACUUM');
     }
-    catch { /* non-fatal */ }
+    catch (e) {
+        logWarning("db", `VACUUM failed: ${e.message}`);
+    }
 }
 let _txDepth = 0;
 export function transaction(fn) {
@@ -961,7 +973,9 @@ export function clearArtifacts() {
     try {
         currentDb.exec("DELETE FROM artifacts");
     }
-    catch { /* cache clear is best effort */ }
+    catch (e) {
+        logWarning("db", `clearArtifacts failed: ${e.message}`);
+    }
 }
 export function insertArtifact(a) {
     if (!currentDb)
@@ -1488,7 +1502,9 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
         if (realpathSync(mainDbPath) === realpathSync(worktreeDbPath))
             return zero;
     }
-    catch { /* path resolution failed — fall through to existing checks */ }
+    catch (e) {
+        logWarning("db", `realpathSync failed: ${e.message}`);
+    }
     // Sanitize path: reject any characters that could break ATTACH syntax.
     // ATTACH DATABASE doesn't support parameterized paths in all providers,
     // so we use strict allowlist validation instead.
@@ -1610,7 +1626,9 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
                 try {
                     adapter.exec("ROLLBACK");
                 }
-                catch { /* best effort */ }
+                catch (e) {
+                    logWarning("db", `rollback failed: ${e.message}`);
+                }
                 throw txErr;
             }
             return { ...merged, conflicts };
@@ -1619,7 +1637,9 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
             try {
                 adapter.exec("DETACH DATABASE wt");
             }
-            catch { /* best effort */ }
+            catch (e) {
+                logWarning("db", `detach worktree DB failed: ${e.message}`);
+            }
         }
     }
     catch (err) {

package/dist/resources/extensions/gsd/guided-flow.js

@@ -37,6 +37,7 @@ import { selectAndApplyModel } from "./auto-model-selection.js";
 export { MILESTONE_ID_RE, generateMilestoneSuffix, nextMilestoneId, extractMilestoneSeq, parseMilestoneId, milestoneIdSort, maxMilestoneNum, findMilestoneIds, reserveMilestoneId, claimReservedId, getReservedMilestoneIds, clearReservedMilestoneIds, } from "./milestone-ids.js";
 export { showQueue, handleQueueReorder, showQueueAdd, buildExistingMilestonesContext, } from "./guided-flow-queue.js";
 import { getErrorMessage } from "./error-utils.js";
+import { logWarning } from "./workflow-logger.js";
 // ─── ID Generation with Reservation ─────────────────────────────────────────
 /**
  * Generate the next milestone ID, accounting for reserved IDs, and reserve it.
@@ -144,7 +145,9 @@ export function checkAutoStartAfterDiscuss() {
                 }
             }
         }
-        catch { /* non-fatal — PROJECT.md parsing failure shouldn't block auto-start */ }
+        catch (e) {
+            logWarning("guided", `PROJECT.md parsing failed: ${e.message}`);
+        }
     }
     // Gate 4: Discussion manifest process verification (multi-milestone only)
     // The LLM writes DISCUSSION-MANIFEST.json after each Phase 3 gate decision.
@@ -171,7 +174,9 @@ export function checkAutoStartAfterDiscuss() {
                 }
             }
         }
-        catch { /* malformed manifest — warn but don't block */ }
+        catch (e) {
+            logWarning("guided", `discussion manifest verification failed: ${e.message}`);
+        }
     }
     // Draft promotion cleanup: if a CONTEXT-DRAFT.md exists alongside the new
     // CONTEXT.md, delete the draft — it's been consumed by the discussion.
@@ -180,18 +185,21 @@ export function checkAutoStartAfterDiscuss() {
         if (draftFile)
             unlinkSync(draftFile);
     }
-    catch { /* non-fatal — stale draft doesn't break anything, CONTEXT.md wins */ }
+    catch (e) {
+        logWarning("guided", `CONTEXT-DRAFT.md unlink failed: ${e.message}`);
+    }
     // Cleanup: remove discussion manifest after auto-start (only needed during discussion)
     try {
         unlinkSync(manifestPath);
     }
-    catch { /* may not exist for single-milestone */ }
+    catch (e) {
+        logWarning("guided", `manifest unlink failed: ${e.message}`);
+    }
     pendingAutoStartMap.delete(basePath);
     ctx.ui.notify(`Milestone ${milestoneId} ready.`, "info");
     startAuto(ctx, pi, basePath, false, { step }).catch((err) => {
         ctx.ui.notify(`Auto-start failed: ${getErrorMessage(err)}`, "error");
-        if (process.env.GSD_DEBUG)
-            console.error('[gsd] auto start error:', err);
+        logWarning("guided", `auto start error: ${getErrorMessage(err)}`);
         debugLog("auto-start-failed", { error: getErrorMessage(err) });
     });
     return true;
@@ -744,8 +752,8 @@ function selfHealRuntimeRecords(basePath, ctx) {
         }
         return { cleared };
     }
-    catch {
-        // Non-fatal — self-heal should never block the wizard
+    catch (e) {
+        logWarning("guided", `self-heal stale runtime records failed: ${e.message}`);
         return { cleared: 0 };
     }
 }
@@ -952,7 +960,9 @@ export async function showSmartEntry(ctx, pi, basePath, options) {
                         return;
                     }
                 }
-                catch { /* directory exists but unreadable — fall through to normal flow */ }
+                catch (e) {
+                    logWarning("guided", `directory read failed: ${e.message}`);
+                }
             }
         }
         const uniqueMilestoneIds = !!loadEffectiveGSDPreferences()?.preferences?.unique_milestone_ids;

package/dist/resources/extensions/gsd/init-wizard.js

@@ -13,6 +13,7 @@ import { ensureGitignore, untrackRuntimeFiles } from "./gitignore.js";
 import { gsdRoot } from "./paths.js";
 import { assertSafeDirectory } from "./validate-directory.js";
 import { runSkillInstallStep } from "./skill-catalog.js";
+import { generateCodebaseMap, writeCodebaseMap } from "./codebase-generator.js";
 // ─── Defaults ───────────────────────────────────────────────────────────────────
 const DEFAULT_PREFS = {
     mode: "solo",
@@ -189,6 +190,17 @@ export async function showProjectInit(ctx, pi, basePath, detection) {
     // Ensure .gitignore
     ensureGitignore(basePath);
     untrackRuntimeFiles(basePath);
+    // Auto-generate codebase map for instant agent orientation
+    try {
+        const result = generateCodebaseMap(basePath);
+        if (result.fileCount > 0) {
+            writeCodebaseMap(basePath, result.content);
+            ctx.ui.notify(`Codebase map generated: ${result.fileCount} files`, "info");
+        }
+    }
+    catch {
+        // Non-fatal — codebase map generation failure should never block project init
+    }
     ctx.ui.notify("GSD initialized. Starting your first milestone...", "info");
     return { completed: true, bootstrapped: true };
 }

package/dist/resources/extensions/gsd/markdown-renderer.js

@@ -8,6 +8,7 @@
 // Critical invariant: rendered markdown must round-trip through
 // parseRoadmap(), parsePlan(), parseSummary() in files.ts.
 import { readFileSync, existsSync, mkdirSync } from "node:fs";
+import { logWarning } from "./workflow-logger.js";
 import { isClosedStatus } from "./status-guards.js";
 import { join, relative } from "node:path";
 import { createRequire } from "node:module";
@@ -56,7 +57,7 @@ function loadArtifactContent(artifactPath, absPath, opts) {
         content = readFileSync(absPath, "utf-8");
     }
     catch {
-        process.stderr.write(`markdown-renderer: cannot read file from disk: ${absPath}\n`);
+        logWarning("renderer", `cannot read file from disk: ${absPath}`);
         return null;
     }
     // Store in DB for future use (graceful degradation path)
@@ -72,7 +73,7 @@ function loadArtifactContent(artifactPath, absPath, opts) {
     }
     catch {
         // Non-fatal: we have the content, DB storage is best-effort
-        process.stderr.write(`markdown-renderer: warning — failed to store disk fallback in DB: ${artifactPath}\n`);
+        logWarning("renderer", `failed to store disk fallback in DB: ${artifactPath}`);
     }
     return content;
 }
@@ -93,7 +94,7 @@ async function writeAndStore(absPath, artifactPath, content, opts) {
     }
     catch {
         // Non-fatal: file is on disk, DB is best-effort
-        process.stderr.write(`markdown-renderer: warning — failed to update artifact in DB: ${artifactPath}\n`);
+        logWarning("renderer", `failed to update artifact in DB: ${artifactPath}`);
     }
     invalidateCaches();
 }
@@ -613,7 +614,8 @@ export function detectStaleRenders(basePath) {
         parseRoadmap = m.parseRoadmap;
         parsePlan = m.parsePlan;
     }
-    catch {
+    catch (e) {
+        logWarning("renderer", `parsers-legacy.ts require failed, falling back to .js: ${e.message}`);
         const m = _require("./parsers-legacy.js");
         parseRoadmap = m.parseRoadmap;
         parsePlan = m.parsePlan;
@@ -647,8 +649,8 @@ export function detectStaleRenders(basePath) {
                     }
                 }
             }
-            catch {
-                // Can't parse roadmap — skip silently
+            catch (e) {
+                logWarning("renderer", `roadmap parse failed: ${e.message}`);
             }
         }
         // ── Check plan checkbox state and summaries for each slice ────────
@@ -679,8 +681,8 @@ export function detectStaleRenders(basePath) {
                         }
                     }
                 }
-                catch {
-                    // Can't parse plan — skip silently
+                catch (e) {
+                    logWarning("renderer", `plan parse failed: ${e.message}`);
                 }
             }
             // Check missing task summary files
@@ -824,7 +826,7 @@ export async function repairStaleRenders(basePath) {
             }
         }
         catch (err) {
-            process.stderr.write(`markdown-renderer: repair failed for ${entry.path}: ${err.message}\n`);
+            logWarning("renderer", `repair failed for ${entry.path}: ${err.message}`);
         }
     }
     if (repairCount > 0) {

package/dist/resources/extensions/gsd/md-importer.js

@@ -10,6 +10,7 @@ import { resolveGsdRootFile, resolveMilestoneFile, resolveSliceFile, resolveTask
 import { findMilestoneIds } from './guided-flow.js';
 import { parseRoadmap, parsePlan } from './parsers-legacy.js';
 import { parseContextDependsOn } from './files.js';
+import { logWarning } from './workflow-logger.js';
 // ─── DECISIONS.md Parser ───────────────────────────────────────────────────
 const VALID_MADE_BY = new Set(['human', 'agent', 'collaborative']);
 /**
@@ -597,25 +598,25 @@ export function migrateFromMarkdown(gsdDir) {
             decisions = importDecisions(gsdDir);
         }
         catch (err) {
-            process.stderr.write(`gsd-migrate: skipping decisions import: ${err.message}\n`);
+            logWarning("migration", `skipping decisions import: ${err.message}`);
         }
         try {
             requirements = importRequirements(gsdDir);
         }
         catch (err) {
-            process.stderr.write(`gsd-migrate: skipping requirements import: ${err.message}\n`);
+            logWarning("migration", `skipping requirements import: ${err.message}`);
         }
         try {
             artifacts = importHierarchyArtifacts(gsdDir);
         }
         catch (err) {
-            process.stderr.write(`gsd-migrate: skipping artifacts import: ${err.message}\n`);
+            logWarning("migration", `skipping artifacts import: ${err.message}`);
         }
         try {
             hierarchy = migrateHierarchyToDb(gsdDir);
         }
         catch (err) {
-            process.stderr.write(`gsd-migrate: skipping hierarchy migration: ${err.message}\n`);
+            logWarning("migration", `skipping hierarchy migration: ${err.message}`);
         }
     });
     process.stderr.write(`gsd-migrate: imported ${decisions} decisions, ${requirements} requirements, ${artifacts} artifacts, ${hierarchy.milestones}M/${hierarchy.slices}S/${hierarchy.tasks}T hierarchy\n`);

package/dist/resources/extensions/gsd/milestone-actions.js

@@ -16,6 +16,7 @@ import { resolveMilestonePath, resolveMilestoneFile, buildMilestoneFileName, } f
 import { invalidateAllCaches } from "./cache.js";
 import { loadQueueOrder, saveQueueOrder } from "./queue-order.js";
 import { isDbAvailable, updateMilestoneStatus } from "./gsd-db.js";
+import { logWarning } from "./workflow-logger.js";
 // ─── Park ──────────────────────────────────────────────────────────────────
 /**
  * Park a milestone — creates a PARKED.md marker file with reason and timestamp.
@@ -51,7 +52,7 @@ export function parkMilestone(basePath, milestoneId, reason) {
             updateMilestoneStatus(milestoneId, "parked");
         }
         catch (err) {
-            process.stderr.write(`gsd: parkMilestone DB sync failed for ${milestoneId}: ${err.message}\n`);
+            logWarning("engine", `parkMilestone DB sync failed for ${milestoneId}: ${err.message}`);
         }
     }
     invalidateAllCaches();
@@ -76,7 +77,7 @@ export function unparkMilestone(basePath, milestoneId) {
             updateMilestoneStatus(milestoneId, "active");
         }
         catch (err) {
-            process.stderr.write(`gsd: unparkMilestone DB sync failed for ${milestoneId}: ${err.message}\n`);
+            logWarning("engine", `unparkMilestone DB sync failed for ${milestoneId}: ${err.message}`);
         }
     }
     invalidateAllCaches();

package/dist/resources/extensions/gsd/milestone-ids.js

@@ -5,6 +5,7 @@
  * Consumed by 15+ modules across the GSD extension. Zero side-effects.
  */
 import { randomInt } from "node:crypto";
+import { logWarning } from "./workflow-logger.js";
 import { readdirSync, existsSync } from "node:fs";
 import { milestonesDir } from "./paths.js";
 import { loadQueueOrder, sortByQueueOrder } from "./queue-order.js";
@@ -109,7 +110,7 @@ export function findMilestoneIds(basePath) {
     catch (err) {
         // Log why milestone scanning failed — silent [] here causes infinite loops (#456)
         if (existsSync(dir)) {
-            console.error(`[gsd] findMilestoneIds: .gsd/milestones/ exists but readdirSync failed — ${getErrorMessage(err)}`);
+            logWarning("engine", `findMilestoneIds: .gsd/milestones/ exists but readdirSync failed — ${getErrorMessage(err)}`);
         }
         return [];
     }