chapterhouse 0.4.2 → 0.5.0

package/dist/store/db.js

@@ -141,7 +141,7 @@ function rebuildMemoryTierTables(database) {
     }
 }
 function ensureMemoryTierColumns(database) {
-    for (const table of ["mem_entities", "mem_observations", "mem_decisions"]) {
+    for (const table of ["mem_entities", "mem_observations", "mem_decisions", "mem_action_items"]) {
         if (!hasColumn(database, table, "tier")) {
             database.exec(`ALTER TABLE ${table} ADD COLUMN tier TEXT DEFAULT 'warm'`);
         }
@@ -184,12 +184,24 @@ function ensureMemoryTierColumns(database) {
       ELSE 'warm'
     END
   `);
+    database.exec(`
+    UPDATE mem_action_items
+    SET tier = CASE
+      WHEN status IN ('done', 'dropped') THEN 'cold'
+      WHEN tier = 'glacier' THEN 'cold'
+      WHEN tier IN ('hot', 'warm', 'cold') THEN tier
+      WHEN status = 'open' AND due_at IS NOT NULL AND datetime(due_at) <= datetime('now', '+7 days') THEN 'hot'
+      ELSE 'warm'
+    END
+  `);
 }
 function ensureMemoryIndexes(database) {
     database.exec(`CREATE INDEX IF NOT EXISTS mem_entities_scope_kind_idx ON mem_entities(scope_id, kind)`);
     database.exec(`CREATE UNIQUE INDEX IF NOT EXISTS mem_entities_scope_kind_name_idx ON mem_entities(scope_id, kind, name)`);
     database.exec(`CREATE INDEX IF NOT EXISTS mem_observations_scope_idx ON mem_observations(scope_id)`);
     database.exec(`CREATE INDEX IF NOT EXISTS mem_decisions_scope_idx ON mem_decisions(scope_id)`);
+    database.exec(`CREATE INDEX IF NOT EXISTS idx_mem_action_items_scope_status ON mem_action_items(scope_id, status)`);
+    database.exec(`CREATE INDEX IF NOT EXISTS idx_mem_action_items_due ON mem_action_items(status, due_at)`);
 }
 const MEMORY_SCOPE_SEEDS = [
     {
@@ -561,6 +573,8 @@ export function getDb() {
         source TEXT NOT NULL DEFAULT 'unknown',
         session_key TEXT NOT NULL DEFAULT 'default',
         turn_id TEXT,
+        agent_slug TEXT,
+        agent_display_name TEXT,
         run_id TEXT,
         ts DATETIME DEFAULT CURRENT_TIMESTAMP
       )
@@ -587,6 +601,8 @@ export function getDb() {
             const oldConvColNames = new Set(oldConvCols.map((column) => column.name));
             const sessionKeySelect = oldConvColNames.has("session_key") ? "session_key" : "'default'";
             const turnIdSelect = oldConvColNames.has("turn_id") ? "turn_id" : "NULL";
+            const agentSlugSelect = oldConvColNames.has("agent_slug") ? "agent_slug" : "NULL";
+            const agentDisplayNameSelect = oldConvColNames.has("agent_display_name") ? "agent_display_name" : "NULL";
             const runIdSelect = oldConvColNames.has("run_id") ? "run_id" : "NULL";
             db.exec(`
         CREATE TABLE conversation_log (
@@ -596,13 +612,15 @@ export function getDb() {
           source TEXT NOT NULL DEFAULT 'unknown',
           session_key TEXT NOT NULL DEFAULT 'default',
           turn_id TEXT,
+          agent_slug TEXT,
+          agent_display_name TEXT,
           run_id TEXT,
           ts DATETIME DEFAULT CURRENT_TIMESTAMP
         )
       `);
             db.exec(`
-        INSERT INTO conversation_log (role, content, source, session_key, turn_id, run_id, ts)
-        SELECT role, content, source, ${sessionKeySelect}, ${turnIdSelect}, ${runIdSelect}, ts FROM conversation_log_old
+        INSERT INTO conversation_log (role, content, source, session_key, turn_id, agent_slug, agent_display_name, run_id, ts)
+        SELECT role, content, source, ${sessionKeySelect}, ${turnIdSelect}, ${agentSlugSelect}, ${agentDisplayNameSelect}, ${runIdSelect}, ts FROM conversation_log_old
       `);
             db.exec(`DROP TABLE conversation_log_old`);
         }
@@ -610,14 +628,40 @@ export function getDb() {
         db.exec(`
       CREATE TABLE IF NOT EXISTS copilot_sessions (
         session_key TEXT PRIMARY KEY,
-        mode TEXT NOT NULL CHECK(mode IN ('default', 'project')),
+        mode TEXT NOT NULL CHECK(mode IN ('default', 'project', 'agent')),
         project_root TEXT,
         copilot_session_id TEXT NOT NULL,
         model TEXT,
         updated_at DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP
       )
     `);
-        // Migrate: add session_key column to conversation_log if not present
+        try {
+            db.prepare(`
+        INSERT INTO copilot_sessions (session_key, mode, copilot_session_id)
+        VALUES ('__mode_probe__', 'agent', '__probe__')
+      `).run();
+            db.prepare(`DELETE FROM copilot_sessions WHERE session_key = '__mode_probe__'`).run();
+        }
+        catch {
+            db.exec(`ALTER TABLE copilot_sessions RENAME TO copilot_sessions_old`);
+            db.exec(`
+        CREATE TABLE copilot_sessions (
+          session_key TEXT PRIMARY KEY,
+          mode TEXT NOT NULL CHECK(mode IN ('default', 'project', 'agent')),
+          project_root TEXT,
+          copilot_session_id TEXT NOT NULL,
+          model TEXT,
+          updated_at DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP
+        )
+      `);
+            db.exec(`
+        INSERT INTO copilot_sessions (session_key, mode, project_root, copilot_session_id, model, updated_at)
+        SELECT session_key, mode, project_root, copilot_session_id, model, updated_at
+        FROM copilot_sessions_old
+      `);
+            db.exec(`DROP TABLE copilot_sessions_old`);
+        }
+        // Migrate: add metadata columns to conversation_log if not present
         const convCols = db.prepare(`PRAGMA table_info(conversation_log)`).all();
         if (!convCols.some((c) => c.name === 'session_key')) {
             db.exec(`ALTER TABLE conversation_log ADD COLUMN session_key TEXT NOT NULL DEFAULT 'default'`);
@@ -625,6 +669,12 @@ export function getDb() {
         if (!convCols.some((c) => c.name === 'turn_id')) {
             db.exec(`ALTER TABLE conversation_log ADD COLUMN turn_id TEXT`);
         }
+        if (!convCols.some((c) => c.name === 'agent_slug')) {
+            db.exec(`ALTER TABLE conversation_log ADD COLUMN agent_slug TEXT`);
+        }
+        if (!convCols.some((c) => c.name === 'agent_display_name')) {
+            db.exec(`ALTER TABLE conversation_log ADD COLUMN agent_display_name TEXT`);
+        }
         if (!convCols.some((c) => c.name === "run_id")) {
             db.exec(`ALTER TABLE conversation_log ADD COLUMN run_id TEXT`);
         }
@@ -785,6 +835,27 @@ export function getDb() {
         last_recalled_at DATETIME,
         created_at DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP
       )
+    `);
+        db.exec(`
+      CREATE TABLE IF NOT EXISTS mem_action_items (
+        id INTEGER PRIMARY KEY AUTOINCREMENT,
+        scope_id INTEGER NOT NULL REFERENCES mem_scopes(id),
+        entity_id INTEGER REFERENCES mem_entities(id),
+        title TEXT NOT NULL,
+        detail TEXT,
+        status TEXT NOT NULL DEFAULT 'open' CHECK(status IN ('open', 'done', 'dropped', 'snoozed')),
+        due_at TEXT,
+        snooze_until TEXT,
+        source TEXT,
+        created_at TEXT NOT NULL DEFAULT CURRENT_TIMESTAMP,
+        updated_at TEXT NOT NULL DEFAULT CURRENT_TIMESTAMP,
+        resolved_at TEXT,
+        resolution_reason TEXT,
+        tier TEXT NOT NULL DEFAULT 'warm' CHECK(tier IN ('hot', 'warm', 'cold')),
+        tier_pinned_at TEXT,
+        tier_reason TEXT,
+        last_recalled_at TEXT
+      )
     `);
         const decisionCols = db.prepare(`PRAGMA table_info(mem_decisions)`).all();
         if (!decisionCols.some((column) => column.name === "superseded_by")) {
@@ -856,6 +927,13 @@ export function getDb() {
           rationale,
           content_rowid='id'
         )
+      `);
+            db.exec(`
+        CREATE VIRTUAL TABLE IF NOT EXISTS mem_action_items_fts USING fts5(
+          title,
+          detail,
+          content_rowid='id'
+        )
       `);
             // Sync triggers
             db.exec(`DROP TRIGGER IF EXISTS memories_ai`);
@@ -867,6 +945,9 @@ export function getDb() {
             db.exec(`DROP TRIGGER IF EXISTS mem_decisions_ai`);
             db.exec(`DROP TRIGGER IF EXISTS mem_decisions_ad`);
             db.exec(`DROP TRIGGER IF EXISTS mem_decisions_au`);
+            db.exec(`DROP TRIGGER IF EXISTS mem_action_items_ai`);
+            db.exec(`DROP TRIGGER IF EXISTS mem_action_items_ad`);
+            db.exec(`DROP TRIGGER IF EXISTS mem_action_items_au`);
             db.exec(`
         CREATE TRIGGER memories_ai AFTER INSERT ON memories BEGIN
           INSERT INTO memories_fts(rowid, content) VALUES (new.id, new.content);
@@ -916,6 +997,24 @@ export function getDb() {
           INSERT INTO mem_decisions_fts(rowid, title, rationale)
           VALUES (new.id, new.title, new.rationale);
         END
+      `);
+            db.exec(`
+        CREATE TRIGGER mem_action_items_ai AFTER INSERT ON mem_action_items BEGIN
+          INSERT INTO mem_action_items_fts(rowid, title, detail)
+          VALUES (new.id, new.title, new.detail);
+        END
+      `);
+            db.exec(`
+        CREATE TRIGGER mem_action_items_ad AFTER DELETE ON mem_action_items BEGIN
+          DELETE FROM mem_action_items_fts WHERE rowid = old.id;
+        END
+      `);
+            db.exec(`
+        CREATE TRIGGER mem_action_items_au AFTER UPDATE ON mem_action_items BEGIN
+          DELETE FROM mem_action_items_fts WHERE rowid = old.id;
+          INSERT INTO mem_action_items_fts(rowid, title, detail)
+          VALUES (new.id, new.title, new.detail);
+        END
       `);
             // Backfill: check if FTS is in sync by comparing row counts
             const memCount = db.prepare(`SELECT COUNT(*) as c FROM memories`).get().c;
@@ -933,6 +1032,11 @@ export function getDb() {
             if (decisionCount > 0 && decisionFtsCount < decisionCount) {
                 db.exec(`INSERT INTO mem_decisions_fts(mem_decisions_fts) VALUES ('rebuild')`);
             }
+            const actionItemCount = db.prepare(`SELECT COUNT(*) as c FROM mem_action_items`).get().c;
+            const actionItemFtsCount = db.prepare(`SELECT COUNT(*) as c FROM mem_action_items_fts`).get().c;
+            if (actionItemCount > 0 && actionItemFtsCount < actionItemCount) {
+                db.exec(`INSERT INTO mem_action_items_fts(mem_action_items_fts) VALUES ('rebuild')`);
+            }
             fts5Available = true;
         }
         catch {
@@ -961,10 +1065,10 @@ export function deleteState(key) {
     db.prepare(`DELETE FROM max_state WHERE key = ?`).run(key);
 }
 /** Log a conversation turn (user, assistant, or system). */
-export function logConversation(role, content, source, sessionKey = "default", turnId) {
+export function logConversation(role, content, source, sessionKey = "default", metadata) {
     const db = getDb();
-    db.prepare(`INSERT INTO conversation_log (role, content, source, session_key, turn_id, run_id) VALUES (?, ?, ?, ?, ?, ?)`)
-        .run(role, content, source, sessionKey, turnId ?? null, getCurrentRunId());
+    db.prepare(`INSERT INTO conversation_log (role, content, source, session_key, turn_id, agent_slug, agent_display_name, run_id)
+     VALUES (?, ?, ?, ?, ?, ?, ?, ?)`).run(role, content, source, sessionKey, metadata?.turnId ?? null, metadata?.agentSlug ?? null, metadata?.agentDisplayName ?? null, getCurrentRunId());
     // Keep last 1000 entries to support context recovery after session loss
     logInsertCount++;
     if (logInsertCount % 50 === 0) {
@@ -1076,24 +1180,33 @@ export function getSessionMessages(sessionKey, limit, options = {}) {
     const runId = options.runId ?? getCurrentRunId();
     const rows = includeHistorical
         ? db
-            .prepare(`SELECT id, role, content, ts, turn_id FROM conversation_log
+            .prepare(`SELECT id, role, content, ts, turn_id, agent_slug, agent_display_name FROM conversation_log
          WHERE session_key = ? AND role IN ('user', 'assistant', 'agent_completion')
          ORDER BY id DESC LIMIT ?`)
             .all(sessionKey, effectiveLimit)
         : db
-            .prepare(`SELECT id, role, content, ts, turn_id FROM conversation_log
+            .prepare(`SELECT id, role, content, ts, turn_id, agent_slug, agent_display_name FROM conversation_log
          WHERE session_key = ? AND run_id = ? AND role IN ('user', 'assistant', 'agent_completion')
          ORDER BY id DESC LIMIT ?`)
             .all(sessionKey, runId, effectiveLimit);
     // Reverse so oldest is first (chronological order for the UI)
     rows.reverse();
-    return rows.map((r) => ({
-        id: r.id,
-        role: r.role === "agent_completion" ? "assistant" : r.role,
-        content: r.content,
-        ts: normalizeSqliteTsToIso(r.ts),
-        turn_id: r.turn_id,
-    }));
+    return rows.map((r) => {
+        const message = {
+            id: r.id,
+            role: r.role === "agent_completion" ? "assistant" : r.role,
+            content: r.content,
+            ts: normalizeSqliteTsToIso(r.ts),
+            turn_id: r.turn_id,
+        };
+        if (r.turn_id)
+            message.turnId = r.turn_id;
+        if (r.agent_slug)
+            message.agentSlug = r.agent_slug;
+        if (r.agent_display_name)
+            message.agentDisplayName = r.agent_display_name;
+        return message;
+    });
 }
 /**
  * Append one event to agent_task_events and return the new event.

package/dist/store/db.test.js

@@ -58,6 +58,53 @@ test("getDb initializes schema, state helpers, and conversation formatting", asy
         dbModule.closeDb();
     }
 });
+test("getDb initializes action-item memory schema and FTS shadow", async () => {
+    const dbModule = await loadDbModule();
+    try {
+        const db = dbModule.getDb();
+        const tables = db.prepare(`SELECT name FROM sqlite_master WHERE type = 'table'`).all();
+        const tableNames = new Set(tables.map((row) => row.name));
+        assert.equal(tableNames.has("mem_action_items"), true, "expected mem_action_items table");
+        assert.equal(tableNames.has("mem_action_items_fts"), true, "expected mem_action_items_fts virtual table");
+        const columns = db.prepare(`PRAGMA table_info(mem_action_items)`).all();
+        const columnNames = new Set(columns.map((column) => column.name));
+        for (const name of [
+            "id",
+            "scope_id",
+            "entity_id",
+            "title",
+            "detail",
+            "status",
+            "due_at",
+            "snooze_until",
+            "source",
+            "created_at",
+            "updated_at",
+            "resolved_at",
+            "resolution_reason",
+            "tier",
+            "tier_pinned_at",
+            "tier_reason",
+            "last_recalled_at",
+        ]) {
+            assert.equal(columnNames.has(name), true, `expected mem_action_items.${name}`);
+        }
+        const scope = db.prepare(`SELECT id FROM mem_scopes WHERE slug = 'chapterhouse'`).get();
+        const inserted = db.prepare(`
+      INSERT INTO mem_action_items (scope_id, title, detail, source)
+      VALUES (?, 'Action FTS sentinel', 'Searchable migration reminder', 'test')
+    `).run(scope.id);
+        const ftsHits = db.prepare(`
+      SELECT rowid
+      FROM mem_action_items_fts
+      WHERE mem_action_items_fts MATCH 'migration'
+    `).all();
+        assert.equal(ftsHits.some((hit) => hit.rowid === Number(inserted.lastInsertRowid)), true);
+    }
+    finally {
+        dbModule.closeDb();
+    }
+});
 test("getDb migrates legacy conversation_log tables to allow system messages", async () => {
     const seedDb = new Database(dbPath);
     seedDb.exec(`
@@ -559,12 +606,15 @@ test("getSessionMessages returns empty array for unknown session", async () => {
 test("getSessionMessages returns structured messages in chronological order, includes agent completions, excludes system rows, respects limit", async () => {
     const dbModule = await loadDbModule();
     try {
-        const db = dbModule.getDb();
+        dbModule.getDb();
         dbModule.logConversation("user", "hello", "web", "test-session");
         dbModule.logConversation("assistant", "hi there", "web", "test-session");
         dbModule.logConversation("system", "system noise", "worker", "test-session");
-        db.prepare(`INSERT INTO conversation_log (role, content, source, session_key, run_id)
-       VALUES ('agent_completion', ?, 'background', 'test-session', ?)`).run("[Agent task completed] @coder finished task task-1:\n\nDone", dbModule.getCurrentRunId());
+        dbModule.logConversation("agent_completion", "Done", "background", "test-session", {
+            agentSlug: "coder",
+            agentDisplayName: "Kaylee",
+            turnId: "agent-turn-1",
+        });
         dbModule.logConversation("user", "second message", "web", "test-session");
         dbModule.logConversation("user", "from other session", "web", "other-session");
         const all = dbModule.getSessionMessages("test-session");
@@ -574,14 +624,19 @@ test("getSessionMessages returns structured messages in chronological order, inc
         assert.equal(all[1].role, "assistant");
         assert.equal(all[1].content, "hi there");
         assert.equal(all[2].role, "assistant");
-        assert.equal(all[2].content, "[Agent task completed] @coder finished task task-1:\n\nDone");
+        assert.equal(all[2].content, "Done");
+        assert.equal(all[2].agentSlug, "coder");
+        assert.equal(all[2].agentDisplayName, "Kaylee");
+        assert.equal(all[2].turnId, "agent-turn-1");
         assert.equal(all[3].role, "user");
         assert.equal(all[3].content, "second message");
         // Limit clamping
         const limited = dbModule.getSessionMessages("test-session", 2);
         assert.equal(limited.length, 2, "limit=2 returns 2 most recent rows");
         // After reversal, these should be the 2 most-recent renderable rows.
-        assert.equal(limited[0].content, "[Agent task completed] @coder finished task task-1:\n\nDone");
+        assert.equal(limited[0].content, "Done");
+        assert.equal(limited[0].agentSlug, "coder");
+        assert.equal(limited[0].turnId, "agent-turn-1");
         assert.equal(limited[1].content, "second message");
         // Other session not leaked
         const other = dbModule.getSessionMessages("other-session");
@@ -605,6 +660,7 @@ test("getSessionMessages returns stable row id and turn_id for hydration reconci
         const message = messages[0];
         assert.equal(message.id, Number(result.lastInsertRowid));
         assert.equal(message.turn_id, "turn-stable-1");
+        assert.equal(message.turnId, "turn-stable-1");
     }
     finally {
         dbModule.closeDb();

package/package.json

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