@anchrd/intel-api 0.6.7 → 0.7.0

package/dist/adapters/db/db-indexing.js

@@ -3,20 +3,21 @@ function mapTarget(row, contentKeys) {
         nodeId: row.node_id,
         versionId: row.version_id,
         title: row.title,
+        description: row.description,
         contentKeys,
         mediaType: row.media_type,
         kind: row.kind,
         updatedAt: row.updated_at,
     };
 }
-export function createKnowledgeIndexRepository(db) {
+export function createNodeIndexRepository(db) {
     return {
         async getTarget(versionId) {
             const row = await db
-                .prepare(`SELECT n.id AS node_id, v.id AS version_id, n.title, v.content_key,
+                .prepare(`SELECT n.id AS node_id, v.id AS version_id, n.title, n.description, v.content_key,
           v.media_type, n.kind, n.updated_at
-          FROM knowledge_versions v
-          JOIN knowledge_nodes n ON n.id = v.node_id
+          FROM node_versions v
+          JOIN nodes n ON n.id = v.node_id
           WHERE v.id = ? AND n.current_version_id = v.id AND n.archived_at IS NULL`)
                 .bind(versionId)
                 .first();
@@ -24,27 +25,33 @@ export function createKnowledgeIndexRepository(db) {
                 return null;
             if (row.kind !== "table")
                 return mapTarget(row, [row.content_key]);
-            // A table's rows live in every version, so indexing the newest one alone would make a search
-            // find only what the last append added (#40).
+            // A table's rows live in every segment since the newest snapshot, so indexing the newest one
+            // alone would make a search find only what the last append added (#40) — and indexing the
+            // segments before a snapshot would make it find rows a mutation removed (#135). The same
+            // reading rule as `listVersionContentKeys`, because the index must describe what a read sees.
             const segments = await db
-                .prepare(`SELECT content_key FROM knowledge_versions
-          WHERE node_id = ? ORDER BY sequence`)
-                .bind(row.node_id)
+                .prepare(`SELECT content_key FROM node_versions
+          WHERE node_id = ? AND sequence >= COALESCE((
+            SELECT MAX(sequence) FROM node_versions
+            WHERE node_id = ? AND segment = 'snapshot'
+          ), 0)
+          ORDER BY sequence`)
+                .bind(row.node_id, row.node_id)
                 .all();
             return mapTarget(row, (segments.results ?? []).map((segment) => segment.content_key));
         },
         async replace(target, content) {
             await db.batch([
                 db
-                    .prepare(`DELETE FROM knowledge_fts WHERE node_id = ? AND EXISTS (
-            SELECT 1 FROM knowledge_nodes
+                    .prepare(`DELETE FROM node_fts WHERE node_id = ? AND EXISTS (
+            SELECT 1 FROM nodes
             WHERE id = ? AND current_version_id = ? AND archived_at IS NULL
           )`)
                     .bind(target.nodeId, target.nodeId, target.versionId),
                 db
-                    .prepare(`INSERT INTO knowledge_fts (version_id, node_id, title, content)
+                    .prepare(`INSERT INTO node_fts (version_id, node_id, title, content)
             SELECT ?, ?, ?, ? WHERE EXISTS (
-              SELECT 1 FROM knowledge_nodes
+              SELECT 1 FROM nodes
               WHERE id = ? AND current_version_id = ? AND archived_at IS NULL
             )`)
                     .bind(target.versionId, target.nodeId, target.title, content, target.nodeId, target.versionId),
@@ -52,12 +59,12 @@ export function createKnowledgeIndexRepository(db) {
         },
         async markIndexed(versionId, occurredAt) {
             await db
-                .prepare(`UPDATE knowledge_index_state
+                .prepare(`UPDATE node_index_state
           SET status = 'indexed', attempt_count = attempt_count + 1,
             last_error = NULL, updated_at = ?
           WHERE version_id = ? AND EXISTS (
-            SELECT 1 FROM knowledge_versions version
-            JOIN knowledge_nodes node ON node.id = version.node_id
+            SELECT 1 FROM node_versions version
+            JOIN nodes node ON node.id = version.node_id
             WHERE version.id = ? AND node.current_version_id = version.id
               AND node.archived_at IS NULL
           )`)
@@ -66,11 +73,11 @@ export function createKnowledgeIndexRepository(db) {
         },
         async markError(versionId, message, occurredAt) {
             await db
-                .prepare(`UPDATE knowledge_index_state
+                .prepare(`UPDATE node_index_state
           SET status = 'error', attempt_count = attempt_count + 1,
             last_error = ?, updated_at = ? WHERE version_id = ? AND EXISTS (
-              SELECT 1 FROM knowledge_versions version
-              JOIN knowledge_nodes node ON node.id = version.node_id
+              SELECT 1 FROM node_versions version
+              JOIN nodes node ON node.id = version.node_id
               WHERE version.id = ? AND node.current_version_id = version.id
                 AND node.archived_at IS NULL
             )`)

package/dist/adapters/db/db.d.ts

@@ -1,6 +1,6 @@
-import type { KnowledgeRepository } from "../../knowledge/knowledge.types.js";
+import type { NodeRepository } from "../../nodes/nodes.types.js";
 import type { D1Database } from "./db.types.js";
-export declare function createKnowledgeRepository(deps: {
+export declare function createNodeRepository(deps: {
     db: D1Database;
     now(): Date;
-}): KnowledgeRepository;
+}): NodeRepository;