feat(api,ui): revision-checked knowledge edit, delete and restore

packages/core/src/curator.ts

@@ -1052,9 +1052,9 @@ async function runInner(input: {
       try {
         input.signal?.throwIfAborted();
         const promotion = ltm.promoteCrossProject({ dryRun: false });
-        if (promotion.promoted > 0) {
+        if (promotion.promoted > 0 || promotion.conflicts.length > 0) {
           log.info(
-            `cross-project promotion: promoted ${promotion.promoted} entries across ${promotion.clusters.length} cluster(s)`,
+            `cross-project promotion: promoted ${promotion.promoted} entries, skipped ${promotion.conflicts.length} title conflict(s) across ${promotion.clusters.length} cluster(s)`,
           );
         }
       } catch (err) {

packages/core/src/data.ts

@@ -1834,13 +1834,22 @@ export function reassignKnowledge(
   // across projects (reviewer NIT).
   // When moving from cross_project to a project, clear the flag.
   const clearCrossProject = oldProjectId === null || oldProjectId === "";
+  const nextCrossProject = clearCrossProject ? 0 : 1;
   database.query("BEGIN IMMEDIATE").run();
   try {
+    if (nextCrossProject === 1) {
+      const conflict = ltm.findSharedTitleConflict(
+        entry.logical_id,
+        entry.title,
+      );
+      if (conflict) throw new ltm.TitleConflictError(entry.title, conflict);
+    }
+
     database
       .query(
         "UPDATE knowledge SET project_id = ?, cross_project = ? WHERE logical_id = ?",
       )
-      .run(toId, clearCrossProject ? 0 : 1, entry.logical_id);
+      .run(toId, nextCrossProject, entry.logical_id);
 
     // Clean up knowledge_transfers that became self-referential after the move.
     if (clearCrossProject) {

packages/core/src/import/structured.ts

@@ -18,6 +18,7 @@
 import * as ltm from "../ltm";
 import { db, ensureProject } from "../db";
 import { parseImportDoc, type LoreImportDoc } from "./schema";
+import { currentTenantId } from "../tenant";
 
 export type StructuredImportOptions = {
   /** Fallback project path for entries without an explicit `project`. */
@@ -64,25 +65,41 @@ function clampConfidence(v: number | undefined): number {
  * project pool then the cross-project pool. Returns the resolved current-row id.
  */
 function findExactTitle(title: string, pid: string | null): string | null {
+  const tenantId = currentTenantId();
   const inProject =
     pid !== null
       ? (db()
           .query(
-            "SELECT id FROM knowledge_current WHERE project_id = ? AND LOWER(title) = LOWER(?) AND confidence > 0 LIMIT 1",
+            `SELECT id FROM knowledge_current
+             WHERE tenant_id = ? AND project_id = ?
+               AND ${ltm.titleKeySql("title")} = ${ltm.titleKeySql("?")}
+               AND confidence > 0
+             ORDER BY logical_id
+             LIMIT 1`,
           )
-          .get(pid, title) as { id: string } | null)
+          .get(tenantId, pid, title) as { id: string } | null)
       : (db()
           .query(
-            "SELECT id FROM knowledge_current WHERE project_id IS NULL AND LOWER(title) = LOWER(?) AND confidence > 0 LIMIT 1",
+            `SELECT id FROM knowledge_current
+             WHERE tenant_id = ? AND project_id IS NULL
+               AND ${ltm.titleKeySql("title")} = ${ltm.titleKeySql("?")}
+               AND confidence > 0
+             ORDER BY logical_id
+             LIMIT 1`,
           )
-          .get(title) as { id: string } | null);
+          .get(tenantId, title) as { id: string } | null);
   if (inProject) return inProject.id;
 
   const crossProject = db()
     .query(
-      "SELECT id FROM knowledge_current WHERE cross_project = 1 AND LOWER(title) = LOWER(?) AND confidence > 0 LIMIT 1",
+      `SELECT id FROM knowledge_current
+       WHERE tenant_id = ? AND cross_project = 1
+         AND ${ltm.titleKeySql("title")} = ${ltm.titleKeySql("?")}
+         AND confidence > 0
+       ORDER BY logical_id
+       LIMIT 1`,
     )
-    .get(title) as { id: string } | null;
+    .get(tenantId, title) as { id: string } | null;
   return crossProject?.id ?? null;
 }
 

packages/core/src/index.ts

@@ -11,6 +11,7 @@
 
 export * as temporal from "./temporal";
 export * as ltm from "./ltm";
+export * as knowledgeEdit from "./knowledge-edit";
 // Client-side encryption primitives (C, #825). Namespaced `crypto` — consumers that
 // also need the WebCrypto/Node global should import that from `node:crypto` to avoid
 // shadowing this named import.