chapterhouse 0.4.0 → 0.4.2

package/dist/memory/housekeeping.test.js

@@ -45,30 +45,30 @@ test("dedupObservationsPass supersedes similar observations in scope determinist
     const team = getScope("team");
     assert.ok(chapterhouse && team);
     const first = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "The worker event stream uses server sent events for live task output.",
         source: "test",
         confidence: 0.4,
     });
     const keeper = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "Worker event streams use server sent events for live task output.",
         source: "test",
         confidence: 0.9,
     });
     const third = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "The worker event stream uses server sent events for live task output today.",
         source: "test",
         confidence: 0.9,
     });
     const otherScope = recordObservation({
-        scope_id: team.id,
+        scope_id: chapterhouse.id,
         content: "Worker event streams use server sent events for live task output.",
         source: "test",
         confidence: 0.1,
     });
-    const summary = housekeepingModule.dedupObservationsPass(chapterhouse.id);
+    const summary = housekeepingModule.dedupObservationsPass(team.id);
     assert.equal(summary.pass, "dedupObservationsPass");
     assert.equal(summary.examined, 3);
     assert.equal(summary.modified, 2);
@@ -79,7 +79,7 @@ test("dedupObservationsPass supersedes similar observations in scope determinist
         { id: third.id, superseded_by: keeper.id },
     ]);
     assert.equal(db.prepare(`SELECT superseded_by FROM mem_observations WHERE id = ?`).get(otherScope.id).superseded_by, null);
-    const second = housekeepingModule.dedupObservationsPass(chapterhouse.id);
+    const second = housekeepingModule.dedupObservationsPass(team.id);
     assert.equal(second.modified, 0);
 });
 test("dedupDecisionsPass supersedes similar active decisions within scope and keeps the latest decision", async () => {
@@ -88,39 +88,39 @@ test("dedupDecisionsPass supersedes similar active decisions within scope and ke
     const getScope = getFunction(memoryModule, "getScope");
     const upsertEntity = getFunction(memoryModule, "upsertEntity");
     const recordDecision = getFunction(memoryModule, "recordDecision");
-    const chapterhouse = getScope("chapterhouse");
     const team = getScope("team");
-    assert.ok(chapterhouse && team);
-    const api = upsertEntity({ scope_id: chapterhouse.id, kind: "component", name: "api" });
-    const web = upsertEntity({ scope_id: chapterhouse.id, kind: "component", name: "web" });
+    const infra = getScope("infra");
+    assert.ok(team && infra);
+    const api = upsertEntity({ scope_id: team.id, kind: "component", name: "api" });
+    const web = upsertEntity({ scope_id: team.id, kind: "component", name: "web" });
     const oldDecision = recordDecision({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         entity_id: api.id,
         title: "Use SQLite FTS5 for memory recall",
         rationale: "Initial choice.",
         decided_at: "2026-05-11",
     });
     const keeper = recordDecision({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         entity_id: api.id,
         title: "Use SQLite FTS5 for scoped memory recall",
         rationale: "Latest choice.",
         decided_at: "2026-05-13",
     });
     const otherEntity = recordDecision({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         entity_id: web.id,
         title: "Use SQLite FTS5 for memory recall",
         rationale: "Same title, different entity context, but newer scope-level decision.",
         decided_at: "2026-05-14",
     });
     const otherScope = recordDecision({
-        scope_id: team.id,
+        scope_id: infra.id,
         title: "Use SQLite FTS5 for memory recall",
         rationale: "Same title, different scope.",
         decided_at: "2026-05-14",
     });
-    const summary = housekeepingModule.dedupDecisionsPass(chapterhouse.id);
+    const summary = housekeepingModule.dedupDecisionsPass(team.id);
     assert.equal(summary.pass, "dedupDecisionsPass");
     assert.equal(summary.examined, 3);
     assert.equal(summary.modified, 2);
@@ -129,7 +129,7 @@ test("dedupDecisionsPass supersedes similar active decisions within scope and ke
     assert.equal(db.prepare(`SELECT superseded_by FROM mem_decisions WHERE id = ?`).get(keeper.id).superseded_by, otherEntity.id);
     assert.equal(db.prepare(`SELECT superseded_by FROM mem_decisions WHERE id = ?`).get(otherEntity.id).superseded_by, null);
     assert.equal(db.prepare(`SELECT superseded_by FROM mem_decisions WHERE id = ?`).get(otherScope.id).superseded_by, null);
-    const second = housekeepingModule.dedupDecisionsPass(chapterhouse.id);
+    const second = housekeepingModule.dedupDecisionsPass(team.id);
     assert.equal(second.modified, 0);
 });
 test("orphanCleanupPass clears missing observation entity references without touching valid or out-of-scope rows", async () => {
@@ -225,18 +225,18 @@ test("tieringPass promotes and demotes rows from lifecycle signals and is idempo
     const upsertEntity = getFunction(memoryModule, "upsertEntity");
     const recordObservation = getFunction(memoryModule, "recordObservation");
     const recordDecision = getFunction(memoryModule, "recordDecision");
-    const chapterhouse = getScope("chapterhouse");
-    assert.ok(chapterhouse);
-    const entity = upsertEntity({ scope_id: chapterhouse.id, kind: "component", name: "memory", tier: "warm" });
+    const team = getScope("team");
+    assert.ok(team);
+    const entity = upsertEntity({ scope_id: team.id, kind: "component", name: "memory", tier: "warm" });
     const referencedObservation = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         entity_id: entity.id,
         content: "Referenced by a recent decision through its entity.",
         source: "test",
         tier: "warm",
     });
     const recentDecision = recordDecision({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         entity_id: entity.id,
         title: "Restate memory tiering decision",
         rationale: "Recent entity-linked decisions keep related observations hot.",
@@ -244,20 +244,20 @@ test("tieringPass promotes and demotes rows from lifecycle signals and is idempo
         tier: "warm",
     });
     const oldHot = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "Old hot row with no recall activity should cool down.",
         source: "test",
         tier: "hot",
     });
     const staleLowConfidence = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "Low confidence stale row should go cold.",
         source: "test",
         tier: "warm",
         confidence: 0.2,
     });
     const archived = recordObservation({
-        scope_id: chapterhouse.id,
+        scope_id: team.id,
         content: "Archived row should always be cold.",
         source: "test",
         tier: "hot",
@@ -265,7 +265,7 @@ test("tieringPass promotes and demotes rows from lifecycle signals and is idempo
     db.prepare(`UPDATE mem_observations SET created_at = datetime('now', '-45 days') WHERE id = ?`).run(oldHot.id);
     db.prepare(`UPDATE mem_observations SET created_at = datetime('now', '-61 days') WHERE id = ?`).run(staleLowConfidence.id);
     db.prepare(`UPDATE mem_observations SET archived_at = CURRENT_TIMESTAMP WHERE id = ?`).run(archived.id);
-    const summary = housekeepingModule.tieringPass(chapterhouse.id);
+    const summary = housekeepingModule.tieringPass(team.id);
     assert.equal(summary.pass, "tieringPass");
     assert.equal(summary.modified, 5);
     assert.deepEqual(summary.errors, []);
@@ -274,7 +274,7 @@ test("tieringPass promotes and demotes rows from lifecycle signals and is idempo
     assert.equal(db.prepare(`SELECT tier FROM mem_observations WHERE id = ?`).get(oldHot.id).tier, "warm");
     assert.equal(db.prepare(`SELECT tier FROM mem_observations WHERE id = ?`).get(staleLowConfidence.id).tier, "cold");
     assert.equal(db.prepare(`SELECT tier FROM mem_observations WHERE id = ?`).get(archived.id).tier, "cold");
-    const second = housekeepingModule.tieringPass(chapterhouse.id);
+    const second = housekeepingModule.tieringPass(team.id);
     assert.equal(second.modified, 0);
 });
 //# sourceMappingURL=housekeeping.test.js.map

package/dist/memory/recall.js

@@ -1,6 +1,17 @@
 import { config } from "../config.js";
 import { getDb, isFts5Available } from "../store/db.js";
 import { getActiveScope } from "./active-scope.js";
+function quoteFts5QueryTerms(query) {
+    return query
+        .trim()
+        .split(/\s+/)
+        .filter((term) => term.length > 0)
+        .map((term) => {
+        const unquoted = term.replace(/^["']|["']$/g, "");
+        return `"${unquoted.replace(/"/g, "\"\"")}"`;
+    })
+        .join(" ");
+}
 function recallHotTier(scopeId, options = {}) {
     const rows = getDb().prepare(`
     SELECT 'observation' AS kind, id, content
@@ -25,6 +36,7 @@ function recallHotTier(scopeId, options = {}) {
 }
 function recallObservationHits(query, scopeId, options = {}) {
     if (isFts5Available()) {
+        const ftsQuery = quoteFts5QueryTerms(query);
         const rows = getDb().prepare(`
       SELECT
         o.id,
@@ -43,7 +55,7 @@ function recallObservationHits(query, scopeId, options = {}) {
         AND (? = 1 OR o.superseded_by IS NULL)
         AND (? = 1 OR o.archived_at IS NULL)
       ORDER BY score DESC, o.id DESC
-    `).all(config.memoryHotRecallBoost, query, scopeId ?? null, scopeId ?? null, options.includeCold ? 1 : 0, options.includeSuperseded ? 1 : 0, options.includeArchived ? 1 : 0);
+    `).all(config.memoryHotRecallBoost, ftsQuery, scopeId ?? null, scopeId ?? null, options.includeCold ? 1 : 0, options.includeSuperseded ? 1 : 0, options.includeArchived ? 1 : 0);
         return rows.map((row) => ({
             kind: "observation",
             id: row.id,
@@ -78,6 +90,7 @@ function recallObservationHits(query, scopeId, options = {}) {
 }
 function recallDecisionHits(query, scopeId, options = {}) {
     if (isFts5Available()) {
+        const ftsQuery = quoteFts5QueryTerms(query);
         const rows = getDb().prepare(`
       SELECT
         d.id,
@@ -99,7 +112,7 @@ function recallDecisionHits(query, scopeId, options = {}) {
         AND (? = 1 OR d.superseded_by IS NULL)
         AND (? = 1 OR d.archived_at IS NULL)
       ORDER BY score DESC, d.id DESC
-    `).all(config.memoryHotRecallBoost, query, scopeId ?? null, scopeId ?? null, options.includeCold ? 1 : 0, options.includeSuperseded ? 1 : 0, options.includeArchived ? 1 : 0);
+    `).all(config.memoryHotRecallBoost, ftsQuery, scopeId ?? null, scopeId ?? null, options.includeCold ? 1 : 0, options.includeSuperseded ? 1 : 0, options.includeArchived ? 1 : 0);
         return rows.map((row) => ({
             kind: "decision",
             id: row.id,

package/dist/memory/recall.test.js

@@ -193,4 +193,46 @@ test("recall boosts hot rows and excludes cold rows unless includeCold is set",
     const withCold = recall({ query: "tierboost sentinel", scope_id: chapterhouse.id, kinds: ["observation"], limit: 10, includeCold: true });
     assert.equal(withCold.hits.some((hit) => hit.id === cold.id), true);
 });
+test("recall matches multi-word observation queries with exact tokens", async () => {
+    const { dbModule, memoryModule } = await loadModules();
+    dbModule.getDb();
+    const getScope = getFunction(memoryModule, "getScope");
+    const recordObservation = getFunction(memoryModule, "recordObservation");
+    const recall = getFunction(memoryModule, "recall");
+    const chapterhouse = getScope("chapterhouse");
+    assert.ok(chapterhouse);
+    const observation = recordObservation({
+        scope_id: chapterhouse.id,
+        content: "Chapterhouse memory P1 shipped on 2026-05-13",
+        source: "test",
+    });
+    const result = recall({
+        query: "memory P1 shipped",
+        scope_id: chapterhouse.id,
+        kinds: ["observation"],
+        limit: 10,
+    });
+    assert.equal(result.hits.some((hit) => hit.id === observation.id), true);
+});
+test("recall treats hyphenated observation query terms literally", async () => {
+    const { dbModule, memoryModule } = await loadModules();
+    dbModule.getDb();
+    const getScope = getFunction(memoryModule, "getScope");
+    const recordObservation = getFunction(memoryModule, "recordObservation");
+    const recall = getFunction(memoryModule, "recall");
+    const chapterhouse = getScope("chapterhouse");
+    assert.ok(chapterhouse);
+    const observation = recordObservation({
+        scope_id: chapterhouse.id,
+        content: "Chapterhouse agent-memory recall shipped safely",
+        source: "test",
+    });
+    const result = recall({
+        query: "agent-memory",
+        scope_id: chapterhouse.id,
+        kinds: ["observation"],
+        limit: 10,
+    });
+    assert.equal(result.hits.some((hit) => hit.id === observation.id), true);
+});
 //# sourceMappingURL=recall.test.js.map