chapterhouse 0.3.22 → 0.3.24

package/dist/copilot/orchestrator.js

@@ -574,10 +574,16 @@ async function executeOnSession(manager, item) {
         });
         const unsubSubDoneDb = session.on("subagent.completed", (event) => {
             try {
-                spawnArgsMap.delete(event.data.toolCallId);
-                activeSubagentTaskIds.delete(event.data.toolCallId);
-                db.prepare(`UPDATE agent_tasks SET status = 'completed', completed_at = CURRENT_TIMESTAMP WHERE task_id = ?`).run(event.data.toolCallId);
-                const taskId = event.data.toolCallId;
+                const doneData = event.data;
+                const taskId = String(doneData.toolCallId ?? "");
+                const finalResult = typeof doneData.result?.detailedContent === "string"
+                    ? doneData.result.detailedContent
+                    : typeof doneData.result?.content === "string"
+                        ? doneData.result.content
+                        : null;
+                spawnArgsMap.delete(taskId);
+                activeSubagentTaskIds.delete(taskId);
+                db.prepare(`UPDATE agent_tasks SET status = 'completed', result = ?, completed_at = CURRENT_TIMESTAMP WHERE task_id = ?`).run(finalResult?.slice(0, 10000) ?? null, taskId);
                 const taskRow = db.prepare(`SELECT agent_slug FROM agent_tasks WHERE task_id = ?`).get(taskId);
                 void agentEventBus.emit({
                     type: "session:destroyed",
@@ -587,7 +593,6 @@ async function executeOnSession(manager, item) {
                     timestamp: new Date(),
                 });
                 // Emit turn:delta with subagent completed part (coexistence — #130)
-                const doneData = event.data;
                 const donePart = {
                     type: "subagent",
                     toolCallId: String(doneData.toolCallId ?? ""),

package/dist/copilot/orchestrator.test.js

@@ -914,7 +914,7 @@ test("S5-01: subagent.started event inserts an adhoc row into agent_tasks", asyn
     // Resolve the pending sendAndWait so the test can clean up
     state.pendingReject?.(new Error("test teardown"));
 });
-test("S5-01: subagent.completed event updates agent_tasks status to completed", async (t) => {
+test("S5-01: subagent.completed event persists final result text to agent_tasks", async (t) => {
     const { orchestrator, state, client } = await loadOrchestratorModule(t, {
         sendResult: "__PENDING__",
     });
@@ -934,10 +934,15 @@ test("S5-01: subagent.completed event updates agent_tasks status to completed",
         agentName: "Wash",
         agentDisplayName: "Wash — Frontend Dev",
         durationMs: 1234,
+        result: {
+            detailedContent: "Workers tab refreshes with persisted final output",
+        },
     });
     const updateWrite = state.dbWrites.find((w) => w.sql.includes("UPDATE") && w.sql.includes("agent_tasks") && w.sql.includes("completed"));
     assert.ok(updateWrite, "subagent.completed must UPDATE agent_tasks to completed");
+    assert.ok(updateWrite.sql.includes("result = ?"), "completed subagent updates must persist the final result text");
     assert.ok(JSON.stringify(updateWrite.args).includes("subagent-call-002"), "UPDATE must target the correct task_id");
+    assert.ok(JSON.stringify(updateWrite.args).includes("Workers tab refreshes with persisted final output"), "UPDATE must store the final result text in agent_tasks.result");
     state.pendingReject?.(new Error("test teardown"));
 });
 test("S5-01: subagent.failed event updates agent_tasks status to error", async (t) => {

package/dist/copilot/task-event-log.js

@@ -7,11 +7,12 @@
  * in sync without the caller needing to wire anything extra.
  *
  * Consumers:
- *  1. `GET /api/workers/:taskId/events` — REST hydration on page-load / SSE reconnect.
- *     The ring buffer is checked first (fast path, in-memory); SQLite is the fallback
- *     for completed tasks whose ring buffer has been cleared.
- *  2. Per-task SSE subscribers — `subscribeTaskLog` delivers events as they arrive
- *     so the SSE frame fires immediately (no SQLite round-trip).
+ *  1. `GET /api/workers/:taskId/events` — SSE backlog replay on connect and
+ *     reconnect. The ring buffer is checked first (fast path, in-memory);
+ *     SQLite is the fallback for completed tasks whose ring buffer has been
+ *     cleared.
+ *  2. Per-task SSE subscribers — `subscribeTaskLog` delivers live events as
+ *     they arrive so the SSE frame fires immediately (no SQLite round-trip).
  *
  * Lifecycle:
  *  - `initTaskEventLog()` must be called once from `initOrchestrator()`.

package/dist/store/db.js

@@ -44,6 +44,14 @@ export function getDb() {
         started_at DATETIME DEFAULT CURRENT_TIMESTAMP,
         completed_at DATETIME
       )
+    `);
+        db.exec(`
+      CREATE TABLE IF NOT EXISTS projects (
+        slug TEXT PRIMARY KEY,
+        cwd TEXT NOT NULL,
+        created_at DATETIME DEFAULT CURRENT_TIMESTAMP,
+        updated_at DATETIME DEFAULT CURRENT_TIMESTAMP
+      )
     `);
         db.exec(`
       CREATE TABLE IF NOT EXISTS max_state (

package/dist/store/db.test.js

@@ -32,6 +32,7 @@ test("getDb initializes schema, state helpers, and conversation formatting", asy
             "worker_sessions",
             "agent_sessions",
             "agent_tasks",
+            "projects",
             "max_state",
             "conversation_log",
             "memories",

package/dist/wiki/project-registry.js

@@ -1,47 +1,91 @@
 import { isAbsolute } from "node:path";
-import { readPage, writePage } from "./fs.js";
-const PROJECTS_INDEX_PATH = "pages/projects/index.md";
+import { getDb } from "../store/db.js";
+import { childLogger } from "../util/logger.js";
+import { deletePage, pageExists, readPage } from "./fs.js";
+const log = childLogger("project-registry");
+const LEGACY_PROJECTS_INDEX_PATH = "pages/projects/index.md";
 const REGISTRY_HEADING = "## Project Registry";
 const OPENING_FENCE = "```yaml";
 const CLOSING_FENCE = "```";
 const SLUG_RE = /^[a-z0-9][a-z0-9-]*$/;
 export function loadRegistry() {
-    const content = readPage(PROJECTS_INDEX_PATH);
-    if (!content)
-        return {};
-    const section = parseRegistrySection(content);
-    if (!section)
-        return {};
-    return parseRegistryBlock(section.blockLines);
+    ensureRegistryMigrated();
+    const rows = getDb()
+        .prepare("SELECT slug, cwd FROM projects ORDER BY slug")
+        .all();
+    return Object.fromEntries(rows.map(({ slug, cwd }) => [slug, cwd]));
 }
 export function saveRegistry(registry) {
     validateRegistry(registry);
-    const renderedSection = renderRegistrySection(registry);
-    const current = readPage(PROJECTS_INDEX_PATH);
-    if (!current) {
-        writePage(PROJECTS_INDEX_PATH, `${renderedSection}\n`);
+    ensureRegistryMigrated();
+    const entries = Object.entries(registry).sort(([left], [right]) => left.localeCompare(right));
+    const db = getDb();
+    const save = db.transaction(() => {
+        db.prepare("DELETE FROM projects").run();
+        const insert = db.prepare(`
+      INSERT INTO projects (slug, cwd, created_at, updated_at)
+      VALUES (?, ?, CURRENT_TIMESTAMP, CURRENT_TIMESTAMP)
+    `);
+        for (const [slug, cwd] of entries) {
+            insert.run(slug, cwd);
+        }
+    });
+    save();
+    removeLegacyRegistryFile("Removed legacy wiki registry after SQLite save");
+}
+export function assertValidProjectSlug(slug) {
+    if (!SLUG_RE.test(slug)) {
+        throw new Error(`Project registry has invalid project slug '${slug}'. Expected a lowercase slug.`);
+    }
+}
+function ensureRegistryMigrated() {
+    const db = getDb();
+    let migratedRegistry;
+    const migrate = db.transaction(() => {
+        const row = db.prepare("SELECT COUNT(*) AS count FROM projects").get();
+        if (row.count > 0 || !pageExists(LEGACY_PROJECTS_INDEX_PATH)) {
+            return;
+        }
+        const legacyContent = readPage(LEGACY_PROJECTS_INDEX_PATH);
+        if (!legacyContent) {
+            return;
+        }
+        const registry = parseLegacyRegistry(legacyContent);
+        if (!registry) {
+            log.warn({ path: LEGACY_PROJECTS_INDEX_PATH }, "Legacy projects page had no registry section; skipping SQLite migration");
+            return;
+        }
+        const insert = db.prepare(`
+      INSERT INTO projects (slug, cwd, created_at, updated_at)
+      VALUES (?, ?, CURRENT_TIMESTAMP, CURRENT_TIMESTAMP)
+    `);
+        for (const [slug, cwd] of Object.entries(registry).sort(([left], [right]) => left.localeCompare(right))) {
+            insert.run(slug, cwd);
+        }
+        migratedRegistry = registry;
+    });
+    migrate.immediate();
+    if (!migratedRegistry) {
         return;
     }
-    const section = parseRegistrySection(current);
-    if (!section) {
-        const trimmed = stripTrailingBlankLines(normalizeLineEndings(current));
-        const prefix = trimmed ? `${trimmed}\n\n` : "";
-        writePage(PROJECTS_INDEX_PATH, `${prefix}${renderedSection}\n`);
+    removeLegacyRegistryFile("Migrated project registry from wiki to SQLite", Object.keys(migratedRegistry).length);
+}
+function removeLegacyRegistryFile(message, count) {
+    if (!deletePage(LEGACY_PROJECTS_INDEX_PATH)) {
         return;
     }
-    const before = stripTrailingBlankLines(section.before.join("\n"));
-    const after = stripTrailingBlankLines(stripLeadingBlankLines(section.after).join("\n"));
-    const pieces = [];
-    if (before) {
-        pieces.push(before);
-        pieces.push("");
+    if (typeof count === "number") {
+        log.info({ count, path: LEGACY_PROJECTS_INDEX_PATH }, message);
+        return;
     }
-    pieces.push(renderedSection);
-    if (after) {
-        pieces.push("");
-        pieces.push(after);
+    log.info({ path: LEGACY_PROJECTS_INDEX_PATH }, message);
+}
+function parseLegacyRegistry(content) {
+    const section = parseRegistrySection(content);
+    if (!section) {
+        return undefined;
     }
-    writePage(PROJECTS_INDEX_PATH, `${pieces.join("\n")}\n`);
+    return parseRegistryBlock(section.blockLines);
 }
 function parseRegistrySection(content) {
     const normalized = normalizeLineEndings(content);
@@ -81,11 +125,7 @@ function parseRegistrySection(content) {
             throw new Error("Project registry is malformed: unexpected content after the fenced block.");
         }
     }
-    return {
-        before: lines.slice(0, headingIndex),
-        blockLines,
-        after: lines.slice(sectionEnd),
-    };
+    return { blockLines };
 }
 function parseRegistryBlock(lines) {
     const registry = {};
@@ -114,28 +154,11 @@ function validateRegistry(registry) {
         validatePath(path);
     }
 }
-export function assertValidProjectSlug(slug) {
-    if (!SLUG_RE.test(slug)) {
-        throw new Error(`Project registry has invalid project slug '${slug}'. Expected a lowercase slug.`);
-    }
-}
 function validatePath(path) {
     if (!path || !isAbsolute(path)) {
         throw new Error(`Project registry path '${path}' must be an absolute path.`);
     }
 }
-function renderRegistrySection(registry) {
-    const lines = [
-        REGISTRY_HEADING,
-        "",
-        OPENING_FENCE,
-        ...Object.keys(registry)
-            .sort()
-            .map((slug) => `${slug}: ${registry[slug]}`),
-        CLOSING_FENCE,
-    ];
-    return lines.join("\n");
-}
 function findNextHeading(lines, startIndex) {
     for (let index = startIndex; index < lines.length; index += 1) {
         if (/^##\s+/.test(lines[index])) {
@@ -147,14 +170,4 @@ function findNextHeading(lines, startIndex) {
 function normalizeLineEndings(content) {
     return content.replace(/\r\n/g, "\n");
 }
-function stripTrailingBlankLines(content) {
-    return content.replace(/\n+$/g, "");
-}
-function stripLeadingBlankLines(lines) {
-    let start = 0;
-    while (start < lines.length && lines[start].trim() === "") {
-        start += 1;
-    }
-    return lines.slice(start);
-}
 //# sourceMappingURL=project-registry.js.map

package/dist/wiki/project-registry.test.js

@@ -6,67 +6,115 @@ const repoRoot = process.cwd();
 const sandboxRoot = join(repoRoot, ".test-work", `wiki-project-registry-${process.pid}`);
 process.env.CHAPTERHOUSE_HOME = sandboxRoot;
 async function loadModules() {
-    const nonce = `${Date.now()}-${Math.random()}`;
-    const projectRegistry = await import(new URL(`./project-registry.js?case=${nonce}`, import.meta.url).href);
-    const wikiFs = await import(new URL(`./fs.js?case=${nonce}`, import.meta.url).href);
-    return { projectRegistry, wikiFs };
+    const projectRegistry = await import(new URL("./project-registry.js", import.meta.url).href);
+    const wikiFs = await import(new URL("./fs.js", import.meta.url).href);
+    const dbModule = await import(new URL("../store/db.js", import.meta.url).href);
+    return { projectRegistry, wikiFs, dbModule };
 }
 function resetSandbox() {
     mkdirSync(join(repoRoot, ".test-work"), { recursive: true });
     rmSync(sandboxRoot, { recursive: true, force: true });
 }
-test.beforeEach(() => {
+test.beforeEach(async () => {
+    const dbModule = await import(new URL("../store/db.js", import.meta.url).href);
+    dbModule.closeDb();
     resetSandbox();
 });
-test.after(() => {
+test.after(async () => {
+    const dbModule = await import(new URL("../store/db.js", import.meta.url).href);
+    dbModule.closeDb();
     rmSync(sandboxRoot, { recursive: true, force: true });
 });
-test("loadRegistry returns an empty object when pages/projects/index.md is absent", async () => {
-    const { projectRegistry } = await loadModules();
-    assert.deepEqual(projectRegistry.loadRegistry(), {});
+test("loadRegistry returns an empty object when the projects table is empty", async () => {
+    const { projectRegistry, dbModule } = await loadModules();
+    try {
+        assert.deepEqual(projectRegistry.loadRegistry(), {});
+    }
+    finally {
+        dbModule.closeDb();
+    }
 });
-test("loadRegistry returns an empty object when the projects page has no registry section", async () => {
-    const { projectRegistry, wikiFs } = await loadModules();
-    wikiFs.writePage("pages/projects/index.md", "---\ntitle: Projects\nsummary: Project pages live here.\nupdated: 2026-05-12\n---\n\n# Projects\n\nTracked project pages.\n");
-    assert.deepEqual(projectRegistry.loadRegistry(), {});
+test("saveRegistry persists the registry in SQLite", async () => {
+    const { projectRegistry, dbModule, wikiFs } = await loadModules();
+    try {
+        projectRegistry.saveRegistry({
+            "docs-site": "/home/bjk/projects/docs-site",
+            chapterhouse: "/home/bjk/projects/chapterhouse",
+        });
+        assert.deepEqual(projectRegistry.loadRegistry(), {
+            chapterhouse: "/home/bjk/projects/chapterhouse",
+            "docs-site": "/home/bjk/projects/docs-site",
+        });
+        assert.equal(wikiFs.readPage("pages/projects/index.md"), undefined);
+        assert.deepEqual(dbModule.getDb().prepare("SELECT slug, cwd FROM projects ORDER BY slug").all(), [
+            { slug: "chapterhouse", cwd: "/home/bjk/projects/chapterhouse" },
+            { slug: "docs-site", cwd: "/home/bjk/projects/docs-site" },
+        ]);
+    }
+    finally {
+        dbModule.closeDb();
+    }
 });
-test("loadRegistry parses the fenced yaml registry block", async () => {
-    const { projectRegistry, wikiFs } = await loadModules();
-    wikiFs.writePage("pages/projects/index.md", "---\ntitle: Projects\nsummary: Canonical project registry.\nupdated: 2026-05-12\n---\n\n# Projects\n\n## Project Registry\n\n```yaml\nchapterhouse: /home/bjk/projects/chapterhouse\ndocs-site: /home/bjk/Documents/docs site\n```\n");
-    assert.deepEqual(projectRegistry.loadRegistry(), {
-        chapterhouse: "/home/bjk/projects/chapterhouse",
-        "docs-site": "/home/bjk/Documents/docs site",
-    });
+test("saveRegistry replaces prior SQLite-backed registry contents", async () => {
+    const { projectRegistry, dbModule } = await loadModules();
+    try {
+        projectRegistry.saveRegistry({
+            alpha: "/srv/alpha",
+            zeta: "/srv/zeta",
+        });
+        projectRegistry.saveRegistry({
+            beta: "/srv/beta",
+            alpha: "/srv/alpha",
+        });
+        assert.deepEqual(projectRegistry.loadRegistry(), {
+            alpha: "/srv/alpha",
+            beta: "/srv/beta",
+        });
+        assert.deepEqual(dbModule.getDb().prepare("SELECT slug, cwd FROM projects ORDER BY slug").all(), [
+            { slug: "alpha", cwd: "/srv/alpha" },
+            { slug: "beta", cwd: "/srv/beta" },
+        ]);
+    }
+    finally {
+        dbModule.closeDb();
+    }
 });
-test("loadRegistry rejects malformed registry content", async () => {
-    const { projectRegistry, wikiFs } = await loadModules();
-    wikiFs.writePage("pages/projects/index.md", "# Projects\n\n## Project Registry\n\n```yaml\nChapterHouse: /home/bjk/projects/chapterhouse\nrelative: ./docs-site\nbroken line\nchapterhouse: /home/bjk/projects/chapterhouse\n```\n");
-    assert.throws(() => projectRegistry.loadRegistry(), /invalid project slug|absolute path|malformed registry line|duplicate project slug/);
-});
-test("saveRegistry creates a new registry page with deterministic ordering", async () => {
-    const { projectRegistry, wikiFs } = await loadModules();
-    projectRegistry.saveRegistry({
-        "docs-site": "/home/bjk/projects/docs-site",
-        chapterhouse: "/home/bjk/projects/chapterhouse",
-    });
-    assert.equal(wikiFs.readPage("pages/projects/index.md"), "## Project Registry\n\n```yaml\nchapterhouse: /home/bjk/projects/chapterhouse\ndocs-site: /home/bjk/projects/docs-site\n```\n");
-});
-test("saveRegistry rewrites only the registry section and preserves surrounding content", async () => {
-    const { projectRegistry, wikiFs } = await loadModules();
-    wikiFs.writePage("pages/projects/index.md", "---\ntitle: Projects\nsummary: Canonical project registry.\nupdated: 2026-05-12\n---\n\n# Projects\n\nIntro paragraph.\n\n## Project Registry\n\n```yaml\nzeta: /srv/zeta\nalpha: /srv/alpha\n```\n\n## Notes\n\nKeep this section untouched.\n");
-    projectRegistry.saveRegistry({
-        beta: "/srv/beta",
-        alpha: "/srv/alpha",
-    });
-    assert.equal(wikiFs.readPage("pages/projects/index.md"), "---\ntitle: Projects\nsummary: Canonical project registry.\nupdated: 2026-05-12\n---\n\n# Projects\n\nIntro paragraph.\n\n## Project Registry\n\n```yaml\nalpha: /srv/alpha\nbeta: /srv/beta\n```\n\n## Notes\n\nKeep this section untouched.\n");
-    assert.deepEqual(projectRegistry.loadRegistry(), {
-        alpha: "/srv/alpha",
-        beta: "/srv/beta",
-    });
+test("loadRegistry migrates the legacy wiki registry into SQLite and removes the file", async () => {
+    const { projectRegistry, wikiFs, dbModule } = await loadModules();
+    try {
+        wikiFs.writePage("pages/projects/index.md", "---\n"
+            + "title: Projects\n"
+            + "summary: Canonical project registry.\n"
+            + "updated: 2026-05-12\n"
+            + "---\n\n"
+            + "# Projects\n\n"
+            + "## Project Registry\n\n"
+            + "```yaml\n"
+            + "chapterhouse: /home/bjk/projects/chapterhouse\n"
+            + "docs-site: /home/bjk/Documents/docs site\n"
+            + "```\n");
+        assert.deepEqual(projectRegistry.loadRegistry(), {
+            chapterhouse: "/home/bjk/projects/chapterhouse",
+            "docs-site": "/home/bjk/Documents/docs site",
+        });
+        assert.equal(wikiFs.pageExists("pages/projects/index.md"), false);
+        assert.deepEqual(dbModule.getDb().prepare("SELECT slug, cwd FROM projects ORDER BY slug").all(), [
+            { slug: "chapterhouse", cwd: "/home/bjk/projects/chapterhouse" },
+            { slug: "docs-site", cwd: "/home/bjk/Documents/docs site" },
+        ]);
+    }
+    finally {
+        dbModule.closeDb();
+    }
 });
 test("saveRegistry rejects invalid slugs and non-absolute paths", async () => {
-    const { projectRegistry } = await loadModules();
-    assert.throws(() => projectRegistry.saveRegistry({ ChapterHouse: "/home/bjk/projects/chapterhouse" }), /invalid project slug/);
-    assert.throws(() => projectRegistry.saveRegistry({ chapterhouse: "./relative" }), /absolute path/);
+    const { projectRegistry, dbModule } = await loadModules();
+    try {
+        assert.throws(() => projectRegistry.saveRegistry({ ChapterHouse: "/home/bjk/projects/chapterhouse" }), /invalid project slug/);
+        assert.throws(() => projectRegistry.saveRegistry({ chapterhouse: "./relative" }), /absolute path/);
+    }
+    finally {
+        dbModule.closeDb();
+    }
 });
 //# sourceMappingURL=project-registry.test.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chapterhouse",
-  "version": "0.3.22",
+  "version": "0.3.24",
   "description": "Chapterhouse — a team-level AI assistant for engineering teams, built on the GitHub Copilot SDK. Web UI only.",
   "bin": {
     "chapterhouse": "dist/cli.js"