lsd-pi 1.0.0 → 1.1.0

package/src/resources/extensions/github-sync/templates.ts

@@ -1,183 +0,0 @@
-/**
- * Markdown formatters for GitHub issue bodies, PR descriptions,
- * and summary comments.
- *
- * All functions produce GitHub-flavored markdown strings ready
- * for the `gh` CLI body parameters.
- */
-
-// ─── Milestone Issue Body ───────────────────────────────────────────────────
-
-export interface MilestoneData {
-  id: string;
-  title: string;
-  vision?: string;
-  successCriteria?: string[];
-  slices?: Array<{ id: string; title: string; taskCount?: number }>;
-}
-
-export function formatMilestoneIssueBody(data: MilestoneData): string {
-  const lines: string[] = [];
-
-  lines.push(`# ${data.id}: ${data.title}`);
-  lines.push("");
-
-  if (data.vision) {
-    lines.push("## Vision");
-    lines.push(data.vision);
-    lines.push("");
-  }
-
-  if (data.successCriteria?.length) {
-    lines.push("## Success Criteria");
-    for (const criterion of data.successCriteria) {
-      lines.push(`- [ ] ${criterion}`);
-    }
-    lines.push("");
-  }
-
-  if (data.slices?.length) {
-    lines.push("## Slices");
-    lines.push("");
-    lines.push("| Slice | Title | Tasks |");
-    lines.push("|-------|-------|-------|");
-    for (const slice of data.slices) {
-      lines.push(`| ${slice.id} | ${slice.title} | ${slice.taskCount ?? "—"} |`);
-    }
-    lines.push("");
-  }
-
-  lines.push("---");
-  lines.push("*Auto-generated by GSD GitHub Sync*");
-
-  return lines.join("\n");
-}
-
-// ─── Slice PR Body ──────────────────────────────────────────────────────────
-
-export interface SliceData {
-  id: string;
-  title: string;
-  goal?: string;
-  mustHaves?: string[];
-  demoCriterion?: string;
-  tasks?: Array<{ id: string; title: string; issueNumber?: number }>;
-}
-
-export function formatSlicePRBody(data: SliceData): string {
-  const lines: string[] = [];
-
-  lines.push(`## ${data.id}: ${data.title}`);
-  lines.push("");
-
-  if (data.goal) {
-    lines.push("### Goal");
-    lines.push(data.goal);
-    lines.push("");
-  }
-
-  if (data.mustHaves?.length) {
-    lines.push("### Must-Haves");
-    for (const item of data.mustHaves) {
-      lines.push(`- ${item}`);
-    }
-    lines.push("");
-  }
-
-  if (data.demoCriterion) {
-    lines.push("### Demo Criterion");
-    lines.push(data.demoCriterion);
-    lines.push("");
-  }
-
-  if (data.tasks?.length) {
-    lines.push("### Tasks");
-    for (const task of data.tasks) {
-      const ref = task.issueNumber ? ` (#${task.issueNumber})` : "";
-      lines.push(`- [ ] ${task.id}: ${task.title}${ref}`);
-    }
-    lines.push("");
-  }
-
-  lines.push("---");
-  lines.push("*Auto-generated by GSD GitHub Sync*");
-
-  return lines.join("\n");
-}
-
-// ─── Task Issue Body ────────────────────────────────────────────────────────
-
-export interface TaskData {
-  id: string;
-  title: string;
-  description?: string;
-  files?: string[];
-  verifyCriteria?: string[];
-}
-
-export function formatTaskIssueBody(data: TaskData): string {
-  const lines: string[] = [];
-
-  lines.push(`## ${data.id}: ${data.title}`);
-  lines.push("");
-
-  if (data.description) {
-    lines.push(data.description);
-    lines.push("");
-  }
-
-  if (data.files?.length) {
-    lines.push("### Files");
-    for (const file of data.files) {
-      lines.push(`- \`${file}\``);
-    }
-    lines.push("");
-  }
-
-  if (data.verifyCriteria?.length) {
-    lines.push("### Verification");
-    for (const criterion of data.verifyCriteria) {
-      lines.push(`- [ ] ${criterion}`);
-    }
-    lines.push("");
-  }
-
-  return lines.join("\n");
-}
-
-// ─── Summary Comment ────────────────────────────────────────────────────────
-
-export interface SummaryData {
-  oneLiner?: string;
-  body?: string;
-  frontmatter?: Record<string, unknown>;
-}
-
-export function formatSummaryComment(data: SummaryData): string {
-  const lines: string[] = [];
-
-  if (data.oneLiner) {
-    lines.push(`**Summary:** ${data.oneLiner}`);
-    lines.push("");
-  }
-
-  if (data.body) {
-    lines.push(data.body);
-    lines.push("");
-  }
-
-  if (data.frontmatter && Object.keys(data.frontmatter).length > 0) {
-    lines.push("<details>");
-    lines.push("<summary>Metadata</summary>");
-    lines.push("");
-    lines.push("```yaml");
-    for (const [key, value] of Object.entries(data.frontmatter)) {
-      lines.push(`${key}: ${JSON.stringify(value)}`);
-    }
-    lines.push("```");
-    lines.push("");
-    lines.push("</details>");
-  }
-
-  return lines.join("\n");
-}

package/src/resources/extensions/github-sync/tests/cli.test.ts

@@ -1,20 +0,0 @@
-import test, { describe, it, beforeEach } from "node:test";
-import assert from "node:assert/strict";
-import { ghIsAvailable, _resetGhCache } from "../cli.ts";
-
-describe("cli", () => {
-  beforeEach(() => {
-    _resetGhCache();
-  });
-
-  it("ghIsAvailable returns boolean", () => {
-    const result = ghIsAvailable();
-    assert.equal(typeof result, "boolean");
-  });
-
-  it("ghIsAvailable caches result", () => {
-    const first = ghIsAvailable();
-    const second = ghIsAvailable();
-    assert.equal(first, second);
-  });
-});

package/src/resources/extensions/github-sync/tests/commit-linking.test.ts

@@ -1,43 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-import { buildTaskCommitMessage } from "../../gsd/git-service.ts";
-
-describe("commit linking", () => {
-  it("appends Resolves #N when issueNumber is set", () => {
-    const msg = buildTaskCommitMessage({
-      taskId: "S01/T02",
-      taskTitle: "implement auth",
-      issueNumber: 43,
-    });
-    assert.ok(msg.includes("Resolves #43"), "should include Resolves trailer");
-    assert.ok(msg.startsWith("feat:"), "subject line has no scope");
-    assert.ok(msg.includes("GSD-Task: S01/T02"), "GSD-Task trailer present");
-  });
-
-  it("includes both key files and Resolves #N", () => {
-    const msg = buildTaskCommitMessage({
-      taskId: "S01/T02",
-      taskTitle: "implement auth",
-      keyFiles: ["src/auth.ts"],
-      issueNumber: 43,
-    });
-    assert.ok(msg.includes("- src/auth.ts"), "key files present");
-    assert.ok(msg.includes("Resolves #43"), "Resolves trailer present");
-    assert.ok(msg.includes("GSD-Task: S01/T02"), "GSD-Task trailer present");
-    // GSD-Task should come after key files but before Resolves
-    const keyFilesIdx = msg.indexOf("- src/auth.ts");
-    const taskIdx = msg.indexOf("GSD-Task: S01/T02");
-    const resolvesIdx = msg.indexOf("Resolves #43");
-    assert.ok(taskIdx > keyFilesIdx, "GSD-Task after key files");
-    assert.ok(resolvesIdx > taskIdx, "Resolves after GSD-Task");
-  });
-
-  it("no Resolves trailer when issueNumber is not set", () => {
-    const msg = buildTaskCommitMessage({
-      taskId: "S01/T02",
-      taskTitle: "implement auth",
-    });
-    assert.ok(!msg.includes("Resolves"), "no Resolves when no issueNumber");
-    assert.ok(msg.includes("GSD-Task: S01/T02"), "GSD-Task trailer still present");
-  });
-});

package/src/resources/extensions/github-sync/tests/mapping.test.ts

@@ -1,104 +0,0 @@
-import { describe, it, beforeEach, afterEach } from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, mkdirSync, rmSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import {
-  loadSyncMapping,
-  saveSyncMapping,
-  createEmptyMapping,
-  getMilestoneRecord,
-  getSliceRecord,
-  getTaskRecord,
-  getTaskIssueNumber,
-  setMilestoneRecord,
-  setSliceRecord,
-  setTaskRecord,
-} from "../mapping.ts";
-import type { SyncMapping, MilestoneSyncRecord, SliceSyncRecord, SyncEntityRecord } from "../types.ts";
-
-describe("mapping", () => {
-  let tmpDir: string;
-
-  beforeEach(() => {
-    tmpDir = mkdtempSync(join(tmpdir(), "gsd-sync-test-"));
-    mkdirSync(join(tmpDir, ".gsd"), { recursive: true });
-  });
-
-  afterEach(() => {
-    rmSync(tmpDir, { recursive: true, force: true });
-  });
-
-  it("loadSyncMapping returns null when no file exists", () => {
-    const result = loadSyncMapping(tmpDir);
-    assert.equal(result, null);
-  });
-
-  it("round-trips save/load", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    saveSyncMapping(tmpDir, mapping);
-    const loaded = loadSyncMapping(tmpDir);
-    assert.deepEqual(loaded, mapping);
-  });
-
-  it("createEmptyMapping has correct structure", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    assert.equal(mapping.version, 1);
-    assert.equal(mapping.repo, "owner/repo");
-    assert.deepEqual(mapping.milestones, {});
-    assert.deepEqual(mapping.slices, {});
-    assert.deepEqual(mapping.tasks, {});
-  });
-
-  it("milestone record accessors work", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    assert.equal(getMilestoneRecord(mapping, "M001"), null);
-
-    const record: MilestoneSyncRecord = {
-      issueNumber: 42,
-      ghMilestoneNumber: 1,
-      lastSyncedAt: "2025-01-01T00:00:00Z",
-      state: "open",
-    };
-    setMilestoneRecord(mapping, "M001", record);
-    assert.deepEqual(getMilestoneRecord(mapping, "M001"), record);
-  });
-
-  it("slice record accessors work", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    assert.equal(getSliceRecord(mapping, "M001", "S01"), null);
-
-    const record: SliceSyncRecord = {
-      issueNumber: 0,
-      prNumber: 50,
-      branch: "milestone/M001/S01",
-      lastSyncedAt: "2025-01-01T00:00:00Z",
-      state: "open",
-    };
-    setSliceRecord(mapping, "M001", "S01", record);
-    assert.deepEqual(getSliceRecord(mapping, "M001", "S01"), record);
-  });
-
-  it("task record accessors work", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    assert.equal(getTaskRecord(mapping, "M001", "S01", "T01"), null);
-    assert.equal(getTaskIssueNumber(mapping, "M001", "S01", "T01"), null);
-
-    const record: SyncEntityRecord = {
-      issueNumber: 43,
-      lastSyncedAt: "2025-01-01T00:00:00Z",
-      state: "open",
-    };
-    setTaskRecord(mapping, "M001", "S01", "T01", record);
-    assert.deepEqual(getTaskRecord(mapping, "M001", "S01", "T01"), record);
-    assert.equal(getTaskIssueNumber(mapping, "M001", "S01", "T01"), 43);
-  });
-
-  it("rejects mapping with wrong version", () => {
-    const mapping = createEmptyMapping("owner/repo");
-    (mapping as any).version = 2;
-    saveSyncMapping(tmpDir, mapping);
-    const loaded = loadSyncMapping(tmpDir);
-    assert.equal(loaded, null);
-  });
-});

package/src/resources/extensions/github-sync/tests/templates.test.ts

@@ -1,110 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-import {
-  formatMilestoneIssueBody,
-  formatSlicePRBody,
-  formatTaskIssueBody,
-  formatSummaryComment,
-} from "../templates.ts";
-
-describe("templates", () => {
-  describe("formatMilestoneIssueBody", () => {
-    it("includes title and vision", () => {
-      const body = formatMilestoneIssueBody({
-        id: "M001",
-        title: "Build Auth",
-        vision: "Secure authentication for all users",
-      });
-      assert.ok(body.includes("M001: Build Auth"));
-      assert.ok(body.includes("Secure authentication"));
-    });
-
-    it("renders success criteria as checkboxes", () => {
-      const body = formatMilestoneIssueBody({
-        id: "M001",
-        title: "Auth",
-        successCriteria: ["Users can log in", "OAuth works"],
-      });
-      assert.ok(body.includes("- [ ] Users can log in"));
-      assert.ok(body.includes("- [ ] OAuth works"));
-    });
-
-    it("renders slice table", () => {
-      const body = formatMilestoneIssueBody({
-        id: "M001",
-        title: "Auth",
-        slices: [
-          { id: "S01", title: "Core types", taskCount: 3 },
-          { id: "S02", title: "OAuth", taskCount: 5 },
-        ],
-      });
-      assert.ok(body.includes("| S01 | Core types | 3 |"));
-      assert.ok(body.includes("| S02 | OAuth | 5 |"));
-    });
-  });
-
-  describe("formatSlicePRBody", () => {
-    it("includes goal and must-haves", () => {
-      const body = formatSlicePRBody({
-        id: "S01",
-        title: "Core Auth Types",
-        goal: "Define all auth types",
-        mustHaves: ["User type", "Session type"],
-      });
-      assert.ok(body.includes("Define all auth types"));
-      assert.ok(body.includes("- User type"));
-      assert.ok(body.includes("- Session type"));
-    });
-
-    it("renders task checklist with issue links", () => {
-      const body = formatSlicePRBody({
-        id: "S01",
-        title: "Auth",
-        tasks: [
-          { id: "T01", title: "Types", issueNumber: 43 },
-          { id: "T02", title: "Schema" },
-        ],
-      });
-      assert.ok(body.includes("- [ ] T01: Types (#43)"));
-      assert.ok(body.includes("- [ ] T02: Schema"));
-      assert.ok(!body.includes("T02: Schema (#"));
-    });
-  });
-
-  describe("formatTaskIssueBody", () => {
-    it("includes files and verification", () => {
-      const body = formatTaskIssueBody({
-        id: "T01",
-        title: "Add types",
-        files: ["src/types.ts"],
-        verifyCriteria: ["Types compile"],
-      });
-      assert.ok(body.includes("`src/types.ts`"));
-      assert.ok(body.includes("- [ ] Types compile"));
-    });
-  });
-
-  describe("formatSummaryComment", () => {
-    it("includes one-liner and body", () => {
-      const comment = formatSummaryComment({
-        oneLiner: "Added retry logic",
-        body: "Implemented exponential backoff",
-      });
-      assert.ok(comment.includes("**Summary:** Added retry logic"));
-      assert.ok(comment.includes("Implemented exponential backoff"));
-    });
-
-    it("wraps frontmatter in details block", () => {
-      const comment = formatSummaryComment({
-        frontmatter: { duration: "45m", key_files: ["a.ts"] },
-      });
-      assert.ok(comment.includes("<details>"));
-      assert.ok(comment.includes("duration:"));
-    });
-
-    it("handles empty data gracefully", () => {
-      const comment = formatSummaryComment({});
-      assert.equal(typeof comment, "string");
-    });
-  });
-});

package/src/resources/extensions/github-sync/types.ts

@@ -1,47 +0,0 @@
-/**
- * Type definitions for the GitHub Sync extension.
- *
- * Config shape (stored in GSD preferences under `github` key) and
- * sync mapping records (stored in `.gsd/github-sync.json`).
- */
-
-// ─── Configuration ──────────────────────────────────────────────────────────
-
-export interface GitHubSyncConfig {
-  enabled: boolean;
-  /** "owner/repo" — auto-detected from git remote if omitted. */
-  repo?: string;
-  /** GitHub Projects v2 number (optional). */
-  project?: number;
-  /** Labels applied to all created issues. */
-  labels?: string[];
-  /** Append "Resolves #N" to task commits. Default: true. */
-  auto_link_commits?: boolean;
-  /** Create per-slice draft PRs. Default: true. */
-  slice_prs?: boolean;
-}
-
-// ─── Sync Mapping ───────────────────────────────────────────────────────────
-
-export interface SyncEntityRecord {
-  issueNumber: number;
-  lastSyncedAt: string;
-  state: "open" | "closed";
-}
-
-export interface MilestoneSyncRecord extends SyncEntityRecord {
-  ghMilestoneNumber: number;
-}
-
-export interface SliceSyncRecord extends SyncEntityRecord {
-  prNumber: number;
-  branch: string;
-}
-
-export interface SyncMapping {
-  version: 1;
-  repo: string;
-  milestones: Record<string, MilestoneSyncRecord>;
-  slices: Record<string, SliceSyncRecord>;
-  tasks: Record<string, SyncEntityRecord>;
-}

package/src/resources/extensions/gsd/activity-log.ts

@@ -1,163 +0,0 @@
-/**
- * GSD Activity Log — Save raw chat sessions to .gsd/activity/
- *
- * Before each context wipe in auto-mode, dumps the full session
- * as JSONL. No formatting, no truncation, no information loss.
- * These are debug artifacts — only read when summaries aren't enough.
- *
- * Diagnostic extraction is handled by session-forensics.ts.
- */
-
-import { writeFileSync, writeSync, mkdirSync, readdirSync, unlinkSync, statSync, openSync, closeSync, constants } from "node:fs";
-import { createHash } from "node:crypto";
-import { join } from "node:path";
-import { GSDError, GSD_IO_ERROR } from "./errors.js";
-
-const SEQ_PREFIX_RE = /^(\d+)-/;
-import type { ExtensionContext } from "@gsd/pi-coding-agent";
-import { gsdRoot } from "./paths.js";
-
-interface ActivityLogState {
-  nextSeq: number;
-  lastSnapshotKeyByUnit: Map<string, string>;
-}
-
-const activityLogState = new Map<string, ActivityLogState>();
-
-/**
- * Clear accumulated activity log state (#611).
- * Call when auto-mode stops to prevent unbounded memory growth
- * from lastSnapshotKeyByUnit maps accumulating across units.
- */
-export function clearActivityLogState(): void {
-  activityLogState.clear();
-}
-
-function scanNextSequence(activityDir: string): number {
-  let maxSeq = 0;
-  try {
-    for (const f of readdirSync(activityDir)) {
-      const match = f.match(SEQ_PREFIX_RE);
-      if (match) maxSeq = Math.max(maxSeq, parseInt(match[1], 10));
-    }
-  } catch (e) {
-    void e; /* directory not readable — start at 1 */
-    return 1;
-  }
-  return maxSeq + 1;
-}
-
-function getActivityState(activityDir: string): ActivityLogState {
-  let state = activityLogState.get(activityDir);
-  if (!state) {
-    state = { nextSeq: scanNextSequence(activityDir), lastSnapshotKeyByUnit: new Map() };
-    activityLogState.set(activityDir, state);
-  }
-  return state;
-}
-
-/**
- * Build a lightweight dedup key from session entries without serializing
- * the entire content to a string (#611). Uses entry count + hash of
- * the last few entries as a fingerprint instead of hashing megabytes.
- */
-function snapshotKey(unitType: string, unitId: string, entries: unknown[]): string {
-  const hash = createHash("sha1");
-  hash.update(`${unitType}\0${unitId}\0${entries.length}\0`);
-  // Hash only the last 3 entries as a fingerprint — if the session grew,
-  // the count change alone detects it; if content changed, the tail hash catches it.
-  const tail = entries.slice(-3);
-  for (const entry of tail) {
-    hash.update(JSON.stringify(entry));
-  }
-  return hash.digest("hex");
-}
-
-function nextActivityFilePath(
-  activityDir: string,
-  state: ActivityLogState,
-  unitType: string,
-  safeUnitId: string,
-): string {
-  // Use O_CREAT | O_EXCL for atomic "create if absent" — no directory scan needed.
-  for (let attempts = 0; attempts < 1000; attempts++) {
-    const seq = String(state.nextSeq).padStart(3, "0");
-    const filePath = join(activityDir, `${seq}-${unitType}-${safeUnitId}.jsonl`);
-    try {
-      const fd = openSync(filePath, constants.O_CREAT | constants.O_EXCL | constants.O_WRONLY);
-      closeSync(fd);
-      return filePath;
-    } catch (err: any) {
-      if (err?.code === "EEXIST") {
-        state.nextSeq++;
-        continue;
-      }
-      throw err;
-    }
-  }
-  // Fallback: should never reach here in practice
-  throw new GSDError(GSD_IO_ERROR, `Failed to find available activity log sequence in ${activityDir}`);
-}
-
-export function saveActivityLog(
-  ctx: ExtensionContext,
-  basePath: string,
-  unitType: string,
-  unitId: string,
-): string | null {
-  try {
-    const entries = ctx.sessionManager.getEntries();
-    if (!entries || entries.length === 0) return null;
-
-    const activityDir = join(gsdRoot(basePath), "activity");
-    mkdirSync(activityDir, { recursive: true });
-
-    const safeUnitId = unitId.replace(/\//g, "-");
-    const state = getActivityState(activityDir);
-    const unitKey = `${unitType}\0${safeUnitId}`;
-    // Use lightweight fingerprint instead of serializing all entries (#611)
-    const key = snapshotKey(unitType, safeUnitId, entries);
-    if (state.lastSnapshotKeyByUnit.get(unitKey) === key) return null;
-
-    const filePath = nextActivityFilePath(activityDir, state, unitType, safeUnitId);
-    // Stream entries to disk line-by-line instead of building one massive string (#611).
-    // For large sessions, the single-string approach allocated hundreds of MB.
-    const fd = openSync(filePath, "w");
-    try {
-      for (const entry of entries) {
-        writeSync(fd, JSON.stringify(entry) + "\n");
-      }
-    } finally {
-      closeSync(fd);
-    }
-    state.nextSeq += 1;
-    state.lastSnapshotKeyByUnit.set(unitKey, key);
-    return filePath;
-  } catch (e) {
-    // Don't let logging failures break auto-mode
-    void e;
-    return null;
-  }
-}
-
-export function pruneActivityLogs(activityDir: string, retentionDays: number): void {
-  try {
-    const files = readdirSync(activityDir);
-    const entries: { seq: number; filePath: string }[] = [];
-    for (const f of files) {
-      const match = f.match(SEQ_PREFIX_RE);
-      if (match) entries.push({ seq: parseInt(match[1], 10), filePath: join(activityDir, f) });
-    }
-    if (entries.length === 0) return;
-    const maxSeq = Math.max(...entries.map(e => e.seq));
-    const cutoff = Date.now() - retentionDays * 86_400_000;
-    for (const entry of entries) {
-      if (entry.seq === maxSeq) continue;  // always preserve highest-seq
-      if (retentionDays === 0) { try { unlinkSync(entry.filePath); } catch { /* skip */ } continue; }
-      try {
-        const mtime = statSync(entry.filePath).mtimeMs;
-        if (Math.floor(mtime) <= cutoff) unlinkSync(entry.filePath);
-      } catch { /* file vanished or stat failed — skip */ }
-    }
-  } catch { /* empty dir or readdirSync failure — skip */ }
-}