gsd-pi 2.41.0-dev.9446b20 → 2.41.0-dev.b832948

package/dist/resources/extensions/gsd/detection.js

@@ -11,7 +11,7 @@ import { homedir } from "node:os";
 import { gsdRoot } from "./paths.js";
 const gsdHome = process.env.GSD_HOME || join(homedir(), ".gsd");
 // ─── Project File Markers ───────────────────────────────────────────────────────
-const PROJECT_FILES = [
+export const PROJECT_FILES = [
     "package.json",
     "Cargo.toml",
     "go.mod",

package/dist/resources/extensions/gsd/doctor.js

@@ -745,6 +745,7 @@ export async function runGSDDoctor(basePath, options) {
             }
             catch { /* non-fatal */ }
             let allTasksDone = plan.tasks.length > 0;
+            let taskUncheckedByDoctor = false;
             for (const task of plan.tasks) {
                 const taskUnitId = `${unitId}/${task.id}`;
                 const summaryPath = resolveTaskFile(basePath, milestoneId, slice.id, task.id, "SUMMARY");
@@ -762,6 +763,7 @@ export async function runGSDDoctor(basePath, options) {
                     dryRunCanFix("task_done_missing_summary", `uncheck ${task.id} in plan for ${taskUnitId}`);
                     if (shouldFix("task_done_missing_summary")) {
                         await markTaskUndoneInPlan(basePath, milestoneId, slice.id, task.id, fixesApplied);
+                        taskUncheckedByDoctor = true;
                     }
                 }
                 if (!task.done && hasSummary) {
@@ -822,6 +824,14 @@ export async function runGSDDoctor(basePath, options) {
                 }
                 allTasksDone = allTasksDone && task.done;
             }
+            // ── #1850: cascade slice uncheck when task_done_missing_summary fires ──
+            // When doctor unchecks tasks inside a done slice, the slice must also be
+            // unchecked so the state machine re-enters the executing phase. Without
+            // this, state.ts skips done slices and the unchecked tasks never run,
+            // causing doctor to fire again on every start (infinite loop).
+            if (taskUncheckedByDoctor && slice.done) {
+                await markSliceUndoneInRoadmap(basePath, milestoneId, slice.id, fixesApplied);
+            }
             // Blocker-without-replan detection
             const replanPath = resolveSliceFile(basePath, milestoneId, slice.id, "REPLAN");
             if (!replanPath) {
@@ -898,7 +908,7 @@ export async function runGSDDoctor(basePath, options) {
                     fixable: true,
                 });
                 dryRunCanFix("all_tasks_done_roadmap_not_checked", `mark ${slice.id} done in roadmap`);
-                if (shouldFix("all_tasks_done_roadmap_not_checked") && (hasSliceSummary || issues.some(issue => issue.code === "all_tasks_done_missing_slice_summary" && issue.unitId === unitId))) {
+                if (shouldFix("all_tasks_done_roadmap_not_checked") && (hasSliceSummary || existsSync(join(slicePath, `${slice.id}-SUMMARY.md`)))) {
                     await markSliceDoneInRoadmap(basePath, milestoneId, slice.id, fixesApplied);
                 }
             }

package/dist/resources/extensions/gsd/exit-command.js

@@ -4,8 +4,18 @@ export function registerExitCommand(pi, deps = {}) {
         description: "Exit GSD gracefully",
         handler: async (_args, ctx) => {
             // Stop auto-mode first so locks and activity state are cleaned up before shutdown.
-            const stopAuto = deps.stopAuto ?? (await importExtensionModule(import.meta.url, "./auto.js")).stopAuto;
-            await stopAuto(ctx, pi, "Graceful exit");
+            // Wrapped in try/catch: if gsd-pi was updated on disk mid-session, the dynamic
+            // import may resolve a new auto-worktree.js whose static imports reference
+            // exports absent from the process-cached native-git-bridge.js (ESM cache is
+            // immutable). The user's work is already saved — this is cleanup only.
+            try {
+                const stopAuto = deps.stopAuto ?? (await importExtensionModule(import.meta.url, "./auto.js")).stopAuto;
+                await stopAuto(ctx, pi, "Graceful exit");
+            }
+            catch (e) {
+                const msg = e instanceof Error ? e.message : String(e);
+                ctx.ui?.notify?.(`Auto-mode cleanup skipped (module version mismatch): ${msg}`, "warning");
+            }
             ctx.shutdown();
         },
     });

package/dist/resources/extensions/gsd/export.js

@@ -2,7 +2,7 @@
 // Generate shareable reports of milestone work in JSON or markdown format.
 import { writeFileSync, mkdirSync } from "node:fs";
 import { join, basename } from "node:path";
-import { exec } from "node:child_process";
+import { execFile } from "node:child_process";
 import { getLedger, getProjectTotals, aggregateByPhase, aggregateBySlice, aggregateByModel, formatCost, formatTokenCount, loadLedgerFromDisk, } from "./metrics.js";
 import { gsdRoot } from "./paths.js";
 import { formatDuration, fileLink } from "../shared/format-utils.js";
@@ -13,18 +13,14 @@ import { getErrorMessage } from "./error-utils.js";
  * Non-blocking, non-fatal — failures are silently ignored.
  */
 export function openInBrowser(filePath) {
-    const cmd = process.platform === "darwin" ? "open" :
-        process.platform === "win32" ? "start" :
-            "xdg-open";
-    // On Windows, `start` needs an empty title argument when the path has spaces
-    const args = process.platform === "win32"
-        ? `"" "${filePath}"`
-        : `"${filePath}"`;
-    exec(`${cmd} ${args}`, (err) => {
-        // Non-fatal — if the browser can't be opened, the file path is still shown
-        if (err)
-            void err;
-    });
+    if (process.platform === "win32") {
+        // PowerShell's Start-Process handles paths with '&' and spaces safely.
+        execFile("powershell", ["-c", `Start-Process '${filePath.replace(/'/g, "''")}'`], () => { });
+    }
+    else {
+        const cmd = process.platform === "darwin" ? "open" : "xdg-open";
+        execFile(cmd, [filePath], () => { });
+    }
 }
 /**
  * Write an export file directly, without requiring an ExtensionCommandContext.

package/dist/resources/extensions/gsd/extension-manifest.json

@@ -8,8 +8,8 @@
   "provides": {
     "tools": [
       "bash", "write", "read", "edit",
-      "gsd_save_decision", "gsd_save_summary",
-      "gsd_update_requirement", "gsd_generate_milestone_id"
+      "gsd_decision_save", "gsd_summary_save",
+      "gsd_requirement_update", "gsd_milestone_generate_id"
     ],
     "commands": ["gsd", "kill", "worktree", "exit"],
     "hooks": ["session_start"],

package/dist/resources/extensions/gsd/files.js

@@ -309,19 +309,36 @@ function _parsePlanImpl(content) {
         let currentTask = null;
         for (const line of taskLines) {
             const cbMatch = line.match(/^-\s+\[([ xX])\]\s+\*\*([\w.]+):\s+(.+?)\*\*\s*(.*)/);
-            if (cbMatch) {
+            // Heading-style: ### T01 -- Title, ### T01: Title, ### T01 — Title
+            const hdMatch = !cbMatch ? line.match(/^#{2,4}\s+([\w.]+)\s*(?:--|—|:)\s*(.+)/) : null;
+            if (cbMatch || hdMatch) {
                 if (currentTask)
                     tasks.push(currentTask);
-                const rest = cbMatch[4] || '';
-                const estMatch = rest.match(/`est:([^`]+)`/);
-                const estimate = estMatch ? estMatch[1] : '';
-                currentTask = {
-                    id: cbMatch[2],
-                    title: cbMatch[3],
-                    description: '',
-                    done: cbMatch[1].toLowerCase() === 'x',
-                    estimate,
-                };
+                if (cbMatch) {
+                    const rest = cbMatch[4] || '';
+                    const estMatch = rest.match(/`est:([^`]+)`/);
+                    const estimate = estMatch ? estMatch[1] : '';
+                    currentTask = {
+                        id: cbMatch[2],
+                        title: cbMatch[3],
+                        description: '',
+                        done: cbMatch[1].toLowerCase() === 'x',
+                        estimate,
+                    };
+                }
+                else {
+                    const rest = hdMatch[2] || '';
+                    const titleEstMatch = rest.match(/^(.+?)\s*`est:([^`]+)`\s*$/);
+                    const title = titleEstMatch ? titleEstMatch[1].trim() : rest.trim();
+                    const estimate = titleEstMatch ? titleEstMatch[2] : '';
+                    currentTask = {
+                        id: hdMatch[1],
+                        title,
+                        description: '',
+                        done: false,
+                        estimate,
+                    };
+                }
             }
             else if (currentTask && line.match(/^\s*-\s+Files:\s*(.*)/)) {
                 const filesMatch = line.match(/^\s*-\s+Files:\s*(.*)/);

package/dist/resources/extensions/gsd/forensics.js

@@ -10,6 +10,7 @@
 import { existsSync, mkdirSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import { join, dirname, relative } from "node:path";
 import { fileURLToPath } from "node:url";
+import { homedir } from "node:os";
 import { extractTrace } from "./session-forensics.js";
 import { nativeParseJsonlTail } from "./native-parser-bridge.js";
 import { MAX_JSONL_BYTES, parseJSONL } from "./jsonl-utils.js";
@@ -46,9 +47,15 @@ export async function handleForensics(args, ctx, pi) {
     ctx.ui.notify("Building forensic report...", "info");
     const report = await buildForensicReport(basePath);
     const savedPath = saveForensicReport(basePath, report, problemDescription);
-    // Derive GSD source dir for prompt
-    const __extensionDir = dirname(fileURLToPath(import.meta.url));
-    const gsdSourceDir = __extensionDir;
+    // Derive GSD source dir for prompt — fall back to ~/.gsd/agent/extensions/gsd/
+    // when import.meta.url resolves to the npm-global install path (Windows).
+    let gsdSourceDir = dirname(fileURLToPath(import.meta.url));
+    if (!existsSync(join(gsdSourceDir, "prompts"))) {
+        const gsdHome = process.env.GSD_HOME || join(homedir(), ".gsd");
+        const fallback = join(gsdHome, "agent", "extensions", "gsd");
+        if (existsSync(join(fallback, "prompts")))
+            gsdSourceDir = fallback;
+    }
     const forensicData = formatReportForPrompt(report);
     const content = loadPrompt("forensics", {
         problemDescription,

package/dist/resources/extensions/gsd/git-service.js

@@ -508,7 +508,11 @@ export class GitServiceImpl {
  */
 export function createDraftPR(basePath, milestoneId, title, body) {
     try {
-        const result = execSync(`gh pr create --draft --title ${JSON.stringify(title)} --body ${JSON.stringify(body)}`, { cwd: basePath, encoding: "utf8", timeout: 30000, env: GIT_NO_PROMPT_ENV });
+        const result = execFileSync("gh", [
+            "pr", "create", "--draft",
+            "--title", title,
+            "--body", body,
+        ], { cwd: basePath, encoding: "utf8", timeout: 30000, env: GIT_NO_PROMPT_ENV });
         return result.trim();
     }
     catch {

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

@@ -118,7 +118,7 @@ function openRawDb(path) {
     return new Database(path);
 }
 // ─── Schema ────────────────────────────────────────────────────────────────
-const SCHEMA_VERSION = 3;
+const SCHEMA_VERSION = 4;
 function initSchema(db, fileBacked) {
     // WAL mode for file-backed databases (must be outside transaction)
     if (fileBacked) {
@@ -142,6 +142,7 @@ function initSchema(db, fileBacked) {
         choice TEXT NOT NULL DEFAULT '',
         rationale TEXT NOT NULL DEFAULT '',
         revisable TEXT NOT NULL DEFAULT '',
+        made_by TEXT NOT NULL DEFAULT 'agent',
         superseded_by TEXT DEFAULT NULL
       )
     `);
@@ -273,6 +274,15 @@ function migrateSchema(db) {
             db.exec("CREATE VIEW active_memories AS SELECT * FROM memories WHERE superseded_by IS NULL");
             db.prepare("INSERT INTO schema_version (version, applied_at) VALUES (:version, :applied_at)").run({ ":version": 3, ":applied_at": new Date().toISOString() });
         }
+        // v3 → v4: add made_by column to decisions table
+        if (currentVersion < 4) {
+            // Add made_by column — default 'agent' for existing rows (pre-attribution decisions)
+            db.exec(`ALTER TABLE decisions ADD COLUMN made_by TEXT NOT NULL DEFAULT 'agent'`);
+            // Recreate views to pick up new columns (SQLite expands SELECT * at view creation time)
+            db.exec("DROP VIEW IF EXISTS active_decisions");
+            db.exec("CREATE VIEW active_decisions AS SELECT * FROM decisions WHERE superseded_by IS NULL");
+            db.prepare("INSERT INTO schema_version (version, applied_at) VALUES (:version, :applied_at)").run({ ":version": 4, ":applied_at": new Date().toISOString() });
+        }
         db.exec("COMMIT");
     }
     catch (err) {
@@ -375,8 +385,8 @@ export function insertDecision(d) {
     if (!currentDb)
         throw new GSDError(GSD_STALE_STATE, "gsd-db: No database open");
     currentDb
-        .prepare(`INSERT INTO decisions (id, when_context, scope, decision, choice, rationale, revisable, superseded_by)
-     VALUES (:id, :when_context, :scope, :decision, :choice, :rationale, :revisable, :superseded_by)`)
+        .prepare(`INSERT INTO decisions (id, when_context, scope, decision, choice, rationale, revisable, made_by, superseded_by)
+     VALUES (:id, :when_context, :scope, :decision, :choice, :rationale, :revisable, :made_by, :superseded_by)`)
         .run({
         ":id": d.id,
         ":when_context": d.when_context,
@@ -385,6 +395,7 @@ export function insertDecision(d) {
         ":choice": d.choice,
         ":rationale": d.rationale,
         ":revisable": d.revisable,
+        ":made_by": d.made_by ?? "agent",
         ":superseded_by": d.superseded_by,
     });
 }
@@ -406,6 +417,7 @@ export function getDecisionById(id) {
         choice: row["choice"],
         rationale: row["rationale"],
         revisable: row["revisable"],
+        made_by: row["made_by"] ?? "agent",
         superseded_by: row["superseded_by"] ?? null,
     };
 }
@@ -425,6 +437,7 @@ export function getActiveDecisions() {
         choice: row["choice"],
         rationale: row["rationale"],
         revisable: row["revisable"],
+        made_by: row["made_by"] ?? "agent",
         superseded_by: null,
     }));
 }
@@ -537,8 +550,8 @@ export function upsertDecision(d) {
     if (!currentDb)
         throw new GSDError(GSD_STALE_STATE, "gsd-db: No database open");
     currentDb
-        .prepare(`INSERT OR REPLACE INTO decisions (id, when_context, scope, decision, choice, rationale, revisable, superseded_by)
-     VALUES (:id, :when_context, :scope, :decision, :choice, :rationale, :revisable, :superseded_by)`)
+        .prepare(`INSERT OR REPLACE INTO decisions (id, when_context, scope, decision, choice, rationale, revisable, made_by, superseded_by)
+     VALUES (:id, :when_context, :scope, :decision, :choice, :rationale, :revisable, :made_by, :superseded_by)`)
         .run({
         ":id": d.id,
         ":when_context": d.when_context,
@@ -547,6 +560,7 @@ export function upsertDecision(d) {
         ":choice": d.choice,
         ":rationale": d.rationale,
         ":revisable": d.revisable,
+        ":made_by": d.made_by ?? "agent",
         ":superseded_by": d.superseded_by ?? null,
     });
 }
@@ -649,8 +663,11 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
     try {
         adapter.exec(`ATTACH DATABASE '${worktreeDbPath}' AS wt`);
         try {
+            // Check if attached wt database has the made_by column (legacy v3 worktrees won't)
+            const wtInfo = adapter.prepare("PRAGMA wt.table_info('decisions')").all();
+            const hasMadeBy = wtInfo.some((col) => col["name"] === "made_by");
             const decConf = adapter
-                .prepare(`SELECT m.id FROM decisions m INNER JOIN wt.decisions w ON m.id = w.id WHERE m.decision != w.decision OR m.choice != w.choice OR m.rationale != w.rationale OR m.superseded_by IS NOT w.superseded_by`)
+                .prepare(`SELECT m.id FROM decisions m INNER JOIN wt.decisions w ON m.id = w.id WHERE m.decision != w.decision OR m.choice != w.choice OR m.rationale != w.rationale OR ${hasMadeBy ? "m.made_by != w.made_by" : "'agent' != 'agent'"} OR m.superseded_by IS NOT w.superseded_by`)
                 .all();
             for (const row of decConf)
                 conflicts.push(`decision ${row["id"]}: modified in both`);
@@ -665,10 +682,10 @@ export function reconcileWorktreeDb(mainDbPath, worktreeDbPath) {
                 const dR = adapter
                     .prepare(`
           INSERT OR REPLACE INTO decisions (
-            id, when_context, scope, decision, choice, rationale, revisable, superseded_by
+            id, when_context, scope, decision, choice, rationale, revisable, made_by, superseded_by
           )
           SELECT
-            id, when_context, scope, decision, choice, rationale, revisable, superseded_by
+            id, when_context, scope, decision, choice, rationale, revisable, ${hasMadeBy ? "made_by" : "'agent'"}, superseded_by
           FROM wt.decisions
         `)
                     .run();

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

@@ -130,7 +130,7 @@ export async function showQueueAdd(ctx, pi, basePath, state) {
     // ── Build existing milestones context for the prompt ────────────────
     const existingContext = await buildExistingMilestonesContext(basePath, milestoneIds, state);
     // ── Determine next milestone ID ─────────────────────────────────────
-    // Note: the LLM will use the gsd_generate_milestone_id tool to get IDs
+    // Note: the LLM will use the gsd_milestone_generate_id tool to get IDs
     // at creation time, but we still mention the next ID in the preamble
     // for context about where the sequence is.
     const uniqueEnabled = !!loadEffectiveGSDPreferences()?.preferences?.unique_milestone_ids;

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

@@ -20,6 +20,7 @@ import { join } from "node:path";
 import { readFileSync, existsSync, mkdirSync, readdirSync, unlinkSync } from "node:fs";
 import { readSessionLockData, isSessionLockProcessAlive } from "./session-lock.js";
 import { nativeIsRepo, nativeInit } from "./native-git-bridge.js";
+import { isInheritedRepo } from "./repo-identity.js";
 import { ensureGitignore, ensurePreferences, untrackRuntimeFiles } from "./gitignore.js";
 import { loadEffectiveGSDPreferences } from "./preferences.js";
 import { detectProjectState } from "./detection.js";
@@ -37,7 +38,7 @@ import { getErrorMessage } from "./error-utils.js";
 // ─── ID Generation with Reservation ─────────────────────────────────────────
 /**
  * Generate the next milestone ID, accounting for reserved IDs, and reserve it.
- * Ensures any preview ID shown in the UI matches what `gsd_generate_milestone_id`
+ * Ensures any preview ID shown in the UI matches what `gsd_milestone_generate_id`
  * will later return.
  */
 function nextMilestoneIdReserved(existingIds, uniqueEnabled) {
@@ -277,7 +278,7 @@ function buildHeadlessDiscussPrompt(nextId, seedContext, _basePath) {
  * Ensures git repo, .gsd/ structure, gitignore, and preferences all exist.
  */
 function bootstrapGsdProject(basePath) {
-    if (!nativeIsRepo(basePath)) {
+    if (!nativeIsRepo(basePath) || isInheritedRepo(basePath)) {
         const mainBranch = loadEffectiveGSDPreferences()?.preferences?.git?.main_branch || "main";
         nativeInit(basePath, mainBranch);
     }
@@ -721,7 +722,10 @@ export async function showSmartEntry(ctx, pi, basePath, options) {
         // which will detect "no milestones" and start the discuss prompt
     }
     // ── Ensure git repo exists — GSD needs it for worktree isolation ──────
-    if (!nativeIsRepo(basePath)) {
+    // Also handle inherited repos: if basePath is a subdirectory of another
+    // git repo that has no .gsd, create a fresh repo to prevent cross-project
+    // state leaks (#1639).
+    if (!nativeIsRepo(basePath) || isInheritedRepo(basePath)) {
         const mainBranch = loadEffectiveGSDPreferences()?.preferences?.git?.main_branch || "main";
         nativeInit(basePath, mainBranch);
     }

package/dist/resources/extensions/gsd/journal.js

@@ -0,0 +1,85 @@
+/**
+ * GSD Event Journal — structured JSONL event log for auto-mode iterations.
+ *
+ * Writes daily-rotated JSONL files to `.gsd/journal/YYYY-MM-DD.jsonl`.
+ * Zero imports from `auto/` — depends only on node:fs, node:path, and paths.ts.
+ *
+ * Observability:
+ * - Each line in the JSONL file is a self-contained JournalEntry
+ * - Events are grouped by flowId (one per iteration) with monotonic seq numbers
+ * - causedBy references enable causal chain reconstruction
+ * - queryJournal() enables programmatic filtering by flowId, eventType, unitId, time range
+ * - Silent failure: journal writes never throw — absence of events is the failure signal
+ */
+import { appendFileSync, mkdirSync, readdirSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+import { gsdRoot } from "./paths.js";
+// ─── Emit ─────────────────────────────────────────────────────────────────────
+/**
+ * Append a journal event to the daily JSONL file.
+ *
+ * File path: `<gsdRoot>/journal/<YYYY-MM-DD>.jsonl`
+ * where the date is extracted from `entry.ts.slice(0, 10)`.
+ *
+ * Never throws — all errors are silently caught.
+ */
+export function emitJournalEvent(basePath, entry) {
+    try {
+        const journalDir = join(gsdRoot(basePath), "journal");
+        mkdirSync(journalDir, { recursive: true });
+        const dateStr = entry.ts.slice(0, 10);
+        const filePath = join(journalDir, `${dateStr}.jsonl`);
+        appendFileSync(filePath, JSON.stringify(entry) + "\n");
+    }
+    catch {
+        // Silent failure — journal must never break auto-mode
+    }
+}
+// ─── Query ────────────────────────────────────────────────────────────────────
+/**
+ * Read and filter journal entries from all daily JSONL files.
+ *
+ * Returns an empty array on any error (missing directory, corrupt files, etc.).
+ */
+export function queryJournal(basePath, filters) {
+    try {
+        const journalDir = join(gsdRoot(basePath), "journal");
+        const files = readdirSync(journalDir).filter(f => f.endsWith(".jsonl")).sort();
+        const entries = [];
+        for (const file of files) {
+            const raw = readFileSync(join(journalDir, file), "utf-8");
+            for (const line of raw.split("\n")) {
+                if (!line.trim())
+                    continue;
+                try {
+                    const entry = JSON.parse(line);
+                    entries.push(entry);
+                }
+                catch {
+                    // Skip malformed lines
+                }
+            }
+        }
+        if (!filters)
+            return entries;
+        return entries.filter(e => {
+            if (filters.flowId && e.flowId !== filters.flowId)
+                return false;
+            if (filters.eventType && e.eventType !== filters.eventType)
+                return false;
+            if (filters.rule && e.rule !== filters.rule)
+                return false;
+            if (filters.unitId && e.data?.unitId !== filters.unitId)
+                return false;
+            if (filters.after && e.ts < filters.after)
+                return false;
+            if (filters.before && e.ts > filters.before)
+                return false;
+            return true;
+        });
+    }
+    catch {
+        // Missing directory, permission errors, etc. — return empty
+        return [];
+    }
+}

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

@@ -9,6 +9,7 @@ import { upsertDecision, upsertRequirement, insertArtifact, openDatabase, transa
 import { resolveGsdRootFile, milestonesDir, gsdRoot, resolveTaskFiles, } from './paths.js';
 import { findMilestoneIds } from './guided-flow.js';
 // ─── DECISIONS.md Parser ───────────────────────────────────────────────────
+const VALID_MADE_BY = new Set(['human', 'agent', 'collaborative']);
 /**
  * Parse a DECISIONS.md markdown table into Decision objects (without seq).
  * Detects `(amends DXXX)` in the Decision column to build supersession info.
@@ -49,6 +50,9 @@ export function parseDecisionsTable(content) {
         const choice = cells[4].trim();
         const rationale = cells[5].trim();
         const revisable = cells[6].trim();
+        // Made By column is optional for backward compatibility — defaults to 'agent'
+        const rawMadeBy = cells.length >= 8 ? cells[7].trim().toLowerCase() : 'agent';
+        const made_by = (VALID_MADE_BY.has(rawMadeBy) ? rawMadeBy : 'agent');
         // Detect (amends DXXX) in the Decision column
         const amendsMatch = decisionText.match(/\(amends\s+(D\d+)\)/i);
         if (amendsMatch) {
@@ -62,6 +66,7 @@ export function parseDecisionsTable(content) {
             choice,
             rationale,
             revisable,
+            made_by,
             superseded_by: null,
         });
     }

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

@@ -62,7 +62,7 @@ export function nextMilestoneId(milestoneIds, uniqueEnabled) {
 /**
  * Module-level set of milestone IDs that have been previewed/promised to the
  * user but not yet materialised on disk. Both guided-flow (preview) and
- * gsd_generate_milestone_id (tool) share this set so the ID shown in the UI
+ * gsd_milestone_generate_id (tool) share this set so the ID shown in the UI
  * matches the one the tool returns.
  */
 const reservedMilestoneIds = new Set();

package/dist/resources/extensions/gsd/native-git-bridge.js

@@ -647,7 +647,7 @@ export function nativeCheckoutBranch(basePath, branch) {
         native.gitCheckoutBranch(basePath, branch);
         return;
     }
-    execSync(`git checkout ${branch}`, {
+    execFileSync("git", ["checkout", branch], {
         cwd: basePath,
         stdio: ["ignore", "pipe", "pipe"],
         encoding: "utf-8",
@@ -679,7 +679,7 @@ export function nativeMergeSquash(basePath, branch) {
         return native.gitMergeSquash(basePath, branch);
     }
     try {
-        execSync(`git merge --squash ${branch}`, {
+        execFileSync("git", ["merge", "--squash", branch], {
             cwd: basePath,
             stdio: ["ignore", "pipe", "pipe"],
             encoding: "utf-8",