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

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

@@ -38,18 +38,15 @@ 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, taskMetadata: undefined };
+        const result = { tier: "light", reason: "hook unit", downgraded: false };
         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") {
-        // Extract metadata once and reuse throughout to avoid double-extraction
-        taskMeta = metadata ?? extractTaskMetadata(unitId, basePath);
-        const taskAnalysis = analyzeTaskComplexity(unitId, basePath, taskMeta);
+        const taskAnalysis = analyzeTaskComplexity(unitId, basePath, metadata);
         tier = taskAnalysis.tier;
         reason = taskAnalysis.reason;
     }
@@ -62,14 +59,13 @@ export function classifyUnitComplexity(unitType, unitId, basePath, budgetPct, me
         }
     }
     // Adaptive learning: check if history suggests bumping the tier
-    // Use already-extracted taskMeta.tags if available to avoid double-extraction
-    const tags = taskMeta?.tags ?? metadata?.tags;
+    const tags = metadata?.tags ?? extractTaskMetadata(unitId, basePath).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, taskMetadata: taskMeta };
+    const result = { tier, reason, downgraded: false };
     return applyBudgetPressure(result, budgetPct);
 }
 /**

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

@@ -16,7 +16,6 @@
  * - 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";
@@ -106,8 +105,8 @@ function handleContentHeuristic(runDir, step, verify) {
                     return "pause";
                 }
             }
-            catch (e) {
-                logWarning("engine", `content-heuristic regex failed: ${e.message}`);
+            catch {
+                // Invalid regex at runtime — treat as verification failure
                 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, logWarning } from "./workflow-logger.js";
+import { logError } from "./workflow-logger.js";
 const _require = createRequire(import.meta.url);
 let providerName = null;
 let providerModule = null;
@@ -710,9 +710,7 @@ export function openDatabase(path) {
                 try {
                     adapter.close();
                 }
-                catch (e) {
-                    logWarning("db", `close after VACUUM failed: ${e.message}`);
-                }
+                catch { /* swallow */ }
                 throw retryErr;
             }
         }
@@ -720,9 +718,7 @@ export function openDatabase(path) {
             try {
                 adapter.close();
             }
-            catch (e) {
-                logWarning("db", `close after VACUUM failed: ${e.message}`);
-            }
+            catch { /* swallow */ }
             throw err;
         }
     }
@@ -734,9 +730,7 @@ export function openDatabase(path) {
         process.on("exit", () => { try {
             closeDatabase();
         }
-        catch (e) {
-            logWarning("db", `exit handler close failed: ${e.message}`);
-        } });
+        catch { } });
     }
     return true;
 }
@@ -745,21 +739,17 @@ export function closeDatabase() {
         try {
             currentDb.exec('PRAGMA wal_checkpoint(TRUNCATE)');
         }
-        catch (e) {
-            logWarning("db", `WAL checkpoint failed: ${e.message}`);
-        }
+        catch { /* non-fatal — best effort before close */ }
         try {
             // Incremental vacuum to reclaim space without blocking
             currentDb.exec('PRAGMA incremental_vacuum(64)');
         }
-        catch (e) {
-            logWarning("db", `incremental vacuum failed: ${e.message}`);
-        }
+        catch { /* non-fatal */ }
         try {
             currentDb.close();
         }
-        catch (e) {
-            logWarning("db", `database close failed: ${e.message}`);
+        catch {
+            // swallow close errors
         }
         currentDb = null;
         currentPath = null;
@@ -773,9 +763,7 @@ export function vacuumDatabase() {
     try {
         currentDb.exec('VACUUM');
     }
-    catch (e) {
-        logWarning("db", `VACUUM failed: ${e.message}`);
-    }
+    catch { /* non-fatal */ }
 }
 let _txDepth = 0;
 export function transaction(fn) {
@@ -973,9 +961,7 @@ export function clearArtifacts() {
     try {
         currentDb.exec("DELETE FROM artifacts");
     }
-    catch (e) {
-        logWarning("db", `clearArtifacts failed: ${e.message}`);
-    }
+    catch { /* cache clear is best effort */ }
 }
 export function insertArtifact(a) {
     if (!currentDb)
@@ -1502,9 +1488,7 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
         if (realpathSync(mainDbPath) === realpathSync(worktreeDbPath))
             return zero;
     }
-    catch (e) {
-        logWarning("db", `realpathSync failed: ${e.message}`);
-    }
+    catch { /* path resolution failed — fall through to existing checks */ }
     // 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.
@@ -1626,9 +1610,7 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
                 try {
                     adapter.exec("ROLLBACK");
                 }
-                catch (e) {
-                    logWarning("db", `rollback failed: ${e.message}`);
-                }
+                catch { /* best effort */ }
                 throw txErr;
             }
             return { ...merged, conflicts };
@@ -1637,9 +1619,7 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
             try {
                 adapter.exec("DETACH DATABASE wt");
             }
-            catch (e) {
-                logWarning("db", `detach worktree DB failed: ${e.message}`);
-            }
+            catch { /* best effort */ }
         }
     }
     catch (err) {

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

@@ -37,7 +37,6 @@ 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.
@@ -145,9 +144,7 @@ export function checkAutoStartAfterDiscuss() {
                 }
             }
         }
-        catch (e) {
-            logWarning("guided", `PROJECT.md parsing failed: ${e.message}`);
-        }
+        catch { /* non-fatal — PROJECT.md parsing failure shouldn't block auto-start */ }
     }
     // Gate 4: Discussion manifest process verification (multi-milestone only)
     // The LLM writes DISCUSSION-MANIFEST.json after each Phase 3 gate decision.
@@ -174,9 +171,7 @@ export function checkAutoStartAfterDiscuss() {
                 }
             }
         }
-        catch (e) {
-            logWarning("guided", `discussion manifest verification failed: ${e.message}`);
-        }
+        catch { /* malformed manifest — warn but don't block */ }
     }
     // Draft promotion cleanup: if a CONTEXT-DRAFT.md exists alongside the new
     // CONTEXT.md, delete the draft — it's been consumed by the discussion.
@@ -185,21 +180,18 @@ export function checkAutoStartAfterDiscuss() {
         if (draftFile)
             unlinkSync(draftFile);
     }
-    catch (e) {
-        logWarning("guided", `CONTEXT-DRAFT.md unlink failed: ${e.message}`);
-    }
+    catch { /* non-fatal — stale draft doesn't break anything, CONTEXT.md wins */ }
     // Cleanup: remove discussion manifest after auto-start (only needed during discussion)
     try {
         unlinkSync(manifestPath);
     }
-    catch (e) {
-        logWarning("guided", `manifest unlink failed: ${e.message}`);
-    }
+    catch { /* may not exist for single-milestone */ }
     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");
-        logWarning("guided", `auto start error: ${getErrorMessage(err)}`);
+        if (process.env.GSD_DEBUG)
+            console.error('[gsd] auto start error:', err);
         debugLog("auto-start-failed", { error: getErrorMessage(err) });
     });
     return true;
@@ -752,8 +744,8 @@ function selfHealRuntimeRecords(basePath, ctx) {
         }
         return { cleared };
     }
-    catch (e) {
-        logWarning("guided", `self-heal stale runtime records failed: ${e.message}`);
+    catch {
+        // Non-fatal — self-heal should never block the wizard
         return { cleared: 0 };
     }
 }
@@ -960,9 +952,7 @@ export async function showSmartEntry(ctx, pi, basePath, options) {
                         return;
                     }
                 }
-                catch (e) {
-                    logWarning("guided", `directory read failed: ${e.message}`);
-                }
+                catch { /* directory exists but unreadable — fall through to normal flow */ }
             }
         }
         const uniqueMilestoneIds = !!loadEffectiveGSDPreferences()?.preferences?.unique_milestone_ids;

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

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

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

@@ -10,7 +10,6 @@ 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']);
 /**
@@ -598,25 +597,25 @@ export function migrateFromMarkdown(gsdDir) {
             decisions = importDecisions(gsdDir);
         }
         catch (err) {
-            logWarning("migration", `skipping decisions import: ${err.message}`);
+            process.stderr.write(`gsd-migrate: skipping decisions import: ${err.message}\n`);
         }
         try {
             requirements = importRequirements(gsdDir);
         }
         catch (err) {
-            logWarning("migration", `skipping requirements import: ${err.message}`);
+            process.stderr.write(`gsd-migrate: skipping requirements import: ${err.message}\n`);
         }
         try {
             artifacts = importHierarchyArtifacts(gsdDir);
         }
         catch (err) {
-            logWarning("migration", `skipping artifacts import: ${err.message}`);
+            process.stderr.write(`gsd-migrate: skipping artifacts import: ${err.message}\n`);
         }
         try {
             hierarchy = migrateHierarchyToDb(gsdDir);
         }
         catch (err) {
-            logWarning("migration", `skipping hierarchy migration: ${err.message}`);
+            process.stderr.write(`gsd-migrate: skipping hierarchy migration: ${err.message}\n`);
         }
     });
     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,7 +16,6 @@ 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.
@@ -52,7 +51,7 @@ export function parkMilestone(basePath, milestoneId, reason) {
             updateMilestoneStatus(milestoneId, "parked");
         }
         catch (err) {
-            logWarning("engine", `parkMilestone DB sync failed for ${milestoneId}: ${err.message}`);
+            process.stderr.write(`gsd: parkMilestone DB sync failed for ${milestoneId}: ${err.message}\n`);
         }
     }
     invalidateAllCaches();
@@ -77,7 +76,7 @@ export function unparkMilestone(basePath, milestoneId) {
             updateMilestoneStatus(milestoneId, "active");
         }
         catch (err) {
-            logWarning("engine", `unparkMilestone DB sync failed for ${milestoneId}: ${err.message}`);
+            process.stderr.write(`gsd: unparkMilestone DB sync failed for ${milestoneId}: ${err.message}\n`);
         }
     }
     invalidateAllCaches();

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

@@ -5,7 +5,6 @@
  * 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";
@@ -110,7 +109,7 @@ export function findMilestoneIds(basePath) {
     catch (err) {
         // Log why milestone scanning failed — silent [] here causes infinite loops (#456)
         if (existsSync(dir)) {
-            logWarning("engine", `findMilestoneIds: .gsd/milestones/ exists but readdirSync failed — ${getErrorMessage(err)}`);
+            console.error(`[gsd] findMilestoneIds: .gsd/milestones/ exists but readdirSync failed — ${getErrorMessage(err)}`);
         }
         return [];
     }