gsd-pi 2.43.0 → 2.44.0-dev.0b97ffd

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

@@ -10,8 +10,7 @@ import { resolveMilestoneFile, relMilestoneFile, resolveGsdRootFile } from './pa
 import { milestoneIdSort, findMilestoneIds } from './milestone-ids.js';
 
 import type {
-  Roadmap, BoundaryMapEntry,
-  SlicePlan, TaskPlanEntry, TaskPlanFile, TaskPlanFrontmatter,
+  TaskPlanFile, TaskPlanFrontmatter,
   Summary, SummaryFrontmatter, SummaryRequires, FileModified,
   Continue, ContinueFrontmatter, ContinueStatus,
   RequirementCounts,
@@ -21,9 +20,7 @@ import type {
 } from './types.js';
 
 import { checkExistingEnvKeys } from './env-utils.js';
-import { parseRoadmapSlices } from './roadmap-slices.js';
-import { nativeParseRoadmap, nativeExtractSection, nativeParsePlanFile, nativeParseSummaryFile, NATIVE_UNAVAILABLE } from './native-parser-bridge.js';
-import { debugTime, debugCount } from './debug-logger.js';
+import { nativeExtractSection, nativeParseSummaryFile, NATIVE_UNAVAILABLE } from './native-parser-bridge.js';
 import { CACHE_MAX } from './constants.js';
 import { splitFrontmatter, parseFrontmatterMap } from '../shared/frontmatter.js';
 
@@ -55,9 +52,22 @@ function cachedParse<T>(content: string, tag: string, parseFn: (c: string) => T)
   return result;
 }
 
-/** Clear the module-scoped parse cache. Call when files change on disk. */
+// ─── Cross-module cache clear registry ────────────────────────────────────
+// parsers-legacy.ts registers its cache-clear callback here at module init
+// to avoid circular imports. clearParseCache() calls all registered callbacks.
+const _cacheClearCallbacks: (() => void)[] = [];
+
+/** Register a callback to be invoked when clearParseCache() is called.
+ *  Used by parsers-legacy.ts to synchronously clear its own cache. */
+export function registerCacheClearCallback(cb: () => void): void {
+  _cacheClearCallbacks.push(cb);
+}
+
+/** Clear the module-scoped parse cache. Call when files change on disk.
+ *  Also clears any registered external caches (e.g. parsers-legacy.ts). */
 export function clearParseCache(): void {
   _parseCache.clear();
+  for (const cb of _cacheClearCallbacks) cb();
 }
 
 // ─── Helpers ───────────────────────────────────────────────────────────────
@@ -117,95 +127,6 @@ export function extractBoldField(text: string, key: string): string | null {
   return match ? match[1].trim() : null;
 }
 
-// ─── Roadmap Parser ────────────────────────────────────────────────────────
-
-export function parseRoadmap(content: string): Roadmap {
-  return cachedParse(content, 'roadmap', _parseRoadmapImpl);
-}
-
-function _parseRoadmapImpl(content: string): Roadmap {
-  const stopTimer = debugTime("parse-roadmap");
-  // Try native parser first for better performance
-  const nativeResult = nativeParseRoadmap(content);
-  if (nativeResult) {
-    stopTimer({ native: true, slices: nativeResult.slices.length, boundaryEntries: nativeResult.boundaryMap.length });
-    debugCount("parseRoadmapCalls");
-    return nativeResult;
-  }
-
-  const lines = content.split('\n');
-
-  const h1 = lines.find(l => l.startsWith('# '));
-  const title = h1 ? h1.slice(2).trim() : '';
-  const vision = extractBoldField(content, 'Vision') || '';
-
-  const scSection = extractSection(content, 'Success Criteria', 2) ||
-    (() => {
-      const idx = content.indexOf('**Success Criteria:**');
-      if (idx === -1) return '';
-      const rest = content.slice(idx);
-      const nextSection = rest.indexOf('\n---');
-      const block = rest.slice(0, nextSection === -1 ? undefined : nextSection);
-      const firstNewline = block.indexOf('\n');
-      return firstNewline === -1 ? '' : block.slice(firstNewline + 1);
-    })();
-  const successCriteria = scSection ? parseBullets(scSection) : [];
-
-  // Slices
-  const slices = parseRoadmapSlices(content);
-
-  // Boundary map
-  const boundaryMap: BoundaryMapEntry[] = [];
-  const bmSection = extractSection(content, 'Boundary Map');
-
-  if (bmSection) {
-    const h3Sections = extractAllSections(bmSection, 3);
-    for (const [heading, sectionContent] of h3Sections) {
-      const arrowMatch = heading.match(/^(\S+)\s*→\s*(\S+)/);
-      if (!arrowMatch) continue;
-
-      const fromSlice = arrowMatch[1];
-      const toSlice = arrowMatch[2];
-
-      let produces = '';
-      let consumes = '';
-
-      // Use indexOf-based parsing instead of [\s\S]*? regex to avoid
-      // catastrophic backtracking on content with code fences (#468).
-      const prodIdx = sectionContent.search(/^Produces:\s*$/m);
-      if (prodIdx !== -1) {
-        const afterProd = sectionContent.indexOf('\n', prodIdx);
-        if (afterProd !== -1) {
-          const consIdx = sectionContent.search(/^Consumes/m);
-          const endIdx = consIdx !== -1 && consIdx > afterProd ? consIdx : sectionContent.length;
-          produces = sectionContent.slice(afterProd + 1, endIdx).trim();
-        }
-      }
-
-      const consLineMatch = sectionContent.match(/^Consumes[^:]*:\s*(.+)$/m);
-      if (consLineMatch) {
-        consumes = consLineMatch[1].trim();
-      }
-      if (!consumes) {
-        const consIdx = sectionContent.search(/^Consumes[^:]*:\s*$/m);
-        if (consIdx !== -1) {
-          const afterCons = sectionContent.indexOf('\n', consIdx);
-          if (afterCons !== -1) {
-            consumes = sectionContent.slice(afterCons + 1).trim();
-          }
-        }
-      }
-
-      boundaryMap.push({ fromSlice, toSlice, produces, consumes });
-    }
-  }
-
-  const result = { title, vision, successCriteria, slices, boundaryMap };
-  stopTimer({ native: false, slices: slices.length, boundaryEntries: boundaryMap.length });
-  debugCount("parseRoadmapCalls");
-  return result;
-}
-
 // ─── Secrets Manifest Parser ───────────────────────────────────────────────
 
 const VALID_STATUSES = new Set<SecretsManifestEntryStatus>(['pending', 'collected', 'skipped']);
@@ -314,131 +235,6 @@ export function parseTaskPlanFile(content: string): TaskPlanFile {
   };
 }
 
-export function parsePlan(content: string): SlicePlan {
-  return cachedParse(content, 'plan', _parsePlanImpl);
-}
-
-function _parsePlanImpl(content: string): SlicePlan {
-  const stopTimer = debugTime("parse-plan");
-  const [, body] = splitFrontmatter(content);
-  // Try native parser first for better performance
-  const nativeResult = nativeParsePlanFile(body);
-  if (nativeResult) {
-    stopTimer({ native: true });
-    return {
-      id: nativeResult.id,
-      title: nativeResult.title,
-      goal: nativeResult.goal,
-      demo: nativeResult.demo,
-      mustHaves: nativeResult.mustHaves,
-      tasks: nativeResult.tasks.map(t => ({
-        id: t.id,
-        title: t.title,
-        description: t.description,
-        done: t.done,
-        estimate: t.estimate,
-        ...(t.files.length > 0 ? { files: t.files } : {}),
-        ...(t.verify ? { verify: t.verify } : {}),
-      })),
-      filesLikelyTouched: nativeResult.filesLikelyTouched,
-    };
-  }
-
-  const lines = body.split('\n');
-
-  const h1 = lines.find(l => l.startsWith('# '));
-  let id = '';
-  let title = '';
-  if (h1) {
-    const match = h1.match(/^#\s+(\w+):\s+(.+)/);
-    if (match) {
-      id = match[1];
-      title = match[2].trim();
-    } else {
-      title = h1.slice(2).trim();
-    }
-  }
-
-  const goal = extractBoldField(body, 'Goal') || '';
-  const demo = extractBoldField(body, 'Demo') || '';
-
-  const mhSection = extractSection(body, 'Must-Haves');
-  const mustHaves = mhSection ? parseBullets(mhSection) : [];
-
-  const tasksSection = extractSection(body, 'Tasks');
-  const tasks: TaskPlanEntry[] = [];
-
-  if (tasksSection) {
-    const taskLines = tasksSection.split('\n');
-    let currentTask: TaskPlanEntry | null = null;
-
-    for (const line of taskLines) {
-      const cbMatch = line.match(/^-\s+\[([ xX])\]\s+\*\*([\w.]+):\s+(.+?)\*\*\s*(.*)/);
-      // 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);
-
-        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*(.*)/);
-        if (filesMatch) {
-          currentTask.files = filesMatch[1]
-            .split(',')
-            .map(f => f.replace(/`/g, '').trim())
-            .filter(f => f.length > 0);
-        }
-      } else if (currentTask && line.match(/^\s*-\s+Verify:\s*(.*)/)) {
-        const verifyMatch = line.match(/^\s*-\s+Verify:\s*(.*)/);
-        if (verifyMatch) {
-          currentTask.verify = verifyMatch[1].trim();
-        }
-      } else if (currentTask && line.trim() && !line.startsWith('#')) {
-        const desc = line.trim();
-        if (desc) {
-          currentTask.description = currentTask.description
-            ? currentTask.description + ' ' + desc
-            : desc;
-        }
-      }
-    }
-    if (currentTask) tasks.push(currentTask);
-  }
-
-  const filesSection = extractSection(body, 'Files Likely Touched');
-  const filesLikelyTouched = filesSection ? parseBullets(filesSection) : [];
-
-  const result = { id, title, goal, demo, mustHaves, tasks, filesLikelyTouched };
-  stopTimer({ tasks: tasks.length });
-  debugCount("parsePlanCalls");
-  return result;
-}
-
 // ─── Summary Parser ────────────────────────────────────────────────────────
 
 export function parseSummary(content: string): Summary {

package/src/resources/extensions/gsd/git-service.ts

@@ -196,6 +196,10 @@ export const RUNTIME_EXCLUSION_PATHS: readonly string[] = [
   ".gsd/completed-units.json",
   ".gsd/STATE.md",
   ".gsd/gsd.db",
+  ".gsd/gsd.db-shm",   // SQLite WAL sidecar — always created alongside gsd.db (#2296)
+  ".gsd/gsd.db-wal",   // SQLite WAL sidecar — always created alongside gsd.db (#2296)
+  ".gsd/journal/",     // daily-rotated JSONL event journal (#2296)
+  ".gsd/doctor-history.jsonl", // doctor run history (#2296)
   ".gsd/DISCUSSION-MANIFEST.json",
 ];
 

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

@@ -29,6 +29,10 @@ const GSD_RUNTIME_PATTERNS = [
   ".gsd/completed-units.json",
   ".gsd/STATE.md",
   ".gsd/gsd.db",
+  ".gsd/gsd.db-shm",   // SQLite WAL sidecar — always created alongside gsd.db (#2296)
+  ".gsd/gsd.db-wal",   // SQLite WAL sidecar — always created alongside gsd.db (#2296)
+  ".gsd/journal/",     // daily-rotated JSONL event journal (#2296)
+  ".gsd/doctor-history.jsonl", // doctor run history (#2296)
   ".gsd/DISCUSSION-MANIFEST.json",
   ".gsd/milestones/**/*-CONTINUE.md",
   ".gsd/milestones/**/continue.md",