gsd-pi 2.41.0 → 2.42.0-dev.eedc83f

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

@@ -0,0 +1,134 @@
+/**
+ * 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";
+
+// ─── Types ────────────────────────────────────────────────────────────────────
+
+/** Event types emitted by the auto-mode loop and phases. */
+export type JournalEventType =
+  | "iteration-start"
+  | "dispatch-match"
+  | "dispatch-stop"
+  | "pre-dispatch-hook"
+  | "unit-start"
+  | "unit-end"
+  | "post-unit-hook"
+  | "terminal"
+  | "guard-block"
+  | "milestone-transition"
+  | "stuck-detected"
+  | "sidecar-dequeue"
+  | "iteration-end";
+
+/** A single structured event in the journal. */
+export interface JournalEntry {
+  /** ISO-8601 timestamp */
+  ts: string;
+  /** UUID grouping all events from one iteration */
+  flowId: string;
+  /** Monotonically increasing sequence number within a flow */
+  seq: number;
+  /** The kind of event */
+  eventType: JournalEventType;
+  /** Name of the matched rule (from the unified registry), if applicable */
+  rule?: string;
+  /** Causal reference to a prior event in this or another flow */
+  causedBy?: { flowId: string; seq: number };
+  /** Arbitrary structured payload (e.g. unitId, status, action details) */
+  data?: Record<string, unknown>;
+}
+
+/** Filters for querying journal entries. */
+export interface JournalQueryFilters {
+  flowId?: string;
+  eventType?: string;
+  unitId?: string;
+  /** Filter by the rule name that produced the event */
+  rule?: string;
+  /** ISO-8601 lower bound (inclusive) */
+  after?: string;
+  /** ISO-8601 upper bound (inclusive) */
+  before?: string;
+}
+
+// ─── 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: string, entry: JournalEntry): void {
+  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: string,
+  filters?: JournalQueryFilters,
+): JournalEntry[] {
+  try {
+    const journalDir = join(gsdRoot(basePath), "journal");
+    const files = readdirSync(journalDir).filter(f => f.endsWith(".jsonl")).sort();
+
+    const entries: JournalEntry[] = [];
+    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) as JournalEntry;
+          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 as Record<string, unknown> | undefined)?.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/src/resources/extensions/gsd/md-importer.ts

@@ -25,6 +25,8 @@ 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.
@@ -64,6 +66,9 @@ export function parseDecisionsTable(content: string): Omit<Decision, 'seq'>[] {
     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') as import('./types.js').DecisionMadeBy;
 
     // Detect (amends DXXX) in the Decision column
     const amendsMatch = decisionText.match(/\(amends\s+(D\d+)\)/i);
@@ -79,6 +84,7 @@ export function parseDecisionsTable(content: string): Omit<Decision, 'seq'>[] {
       choice,
       rationale,
       revisable,
+      made_by,
       superseded_by: null,
     });
   }

package/src/resources/extensions/gsd/milestone-ids.ts

@@ -75,7 +75,7 @@ export function nextMilestoneId(milestoneIds: string[], uniqueEnabled?: boolean)
 /**
  * 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<string>();

package/src/resources/extensions/gsd/native-git-bridge.ts

@@ -808,7 +808,7 @@ export function nativeCheckoutBranch(basePath: string, branch: string): void {
     native.gitCheckoutBranch(basePath, branch);
     return;
   }
-  execSync(`git checkout ${branch}`, {
+  execFileSync("git", ["checkout", branch], {
     cwd: basePath,
     stdio: ["ignore", "pipe", "pipe"],
     encoding: "utf-8",
@@ -843,10 +843,11 @@ export function nativeMergeSquash(basePath: string, branch: string): GitMergeRes
   }
 
   try {
-    execSync(`git merge --squash ${branch}`, {
+    execFileSync("git", ["merge", "--squash", branch], {
       cwd: basePath,
       stdio: ["ignore", "pipe", "pipe"],
       encoding: "utf-8",
+      env: GIT_NO_PROMPT_ENV,
     });
     return { success: true, conflicts: [] };
   } catch (err: unknown) {