@inetafrica/open-claudia 2.6.37 → 2.6.38

package/test-project-transcripts-smoke.js

@@ -0,0 +1,50 @@
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+const assert = require("assert");
+const { ProjectTranscripts, normalizeProjectPath, projectHash } = require("./project-transcripts");
+
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "open-claudia-transcripts-"));
+const project = path.join(tmp, "repo");
+fs.mkdirSync(project);
+
+const transcripts = new ProjectTranscripts({
+  configDir: tmp,
+  maxEntryChars: 100,
+  redact: (value) => String(value).replace(/SECRET/g, "[REDACTED]"),
+});
+
+const normalized = normalizeProjectPath(project);
+const hash = projectHash(project);
+assert.strictEqual(hash, projectHash(normalized));
+
+const result = transcripts.append({
+  role: "user",
+  text: "hello SECRET " + "x".repeat(100),
+  userId: "telegram:1",
+  chat: { transport: "telegram", id: "1" },
+  projectName: "repo",
+  projectPath: project,
+  backend: "codex",
+  sessionId: "sess-1",
+});
+
+assert.ok(result.transcriptPath.startsWith(path.join(tmp, "transcripts")));
+assert.ok(fs.existsSync(result.transcriptPath));
+const line = fs.readFileSync(result.transcriptPath, "utf8").trim();
+const entry = JSON.parse(line);
+assert.strictEqual(entry.project.path, normalized);
+// The stored hash is per-user (projectHash seeds with userId), so verify it
+// against the same user the entry was written for — not the path-only hash above.
+assert.strictEqual(entry.project.hash, projectHash(normalized, "telegram:1"));
+assert.strictEqual(entry.backend, "codex");
+assert.ok(entry.truncated);
+assert.ok(!entry.text.includes("SECRET"));
+assert.ok(entry.text.includes("[REDACTED]"));
+
+const note = transcripts.buildPointerNote(project, "repo", "telegram:1");
+assert.ok(note.includes(result.transcriptPath));
+assert.ok(note.includes("Do not read the whole file"));
+
+fs.rmSync(tmp, { recursive: true, force: true });
+console.log("project-transcripts smoke OK");

package/test-recall-engine.js

@@ -3,18 +3,20 @@ const assert = require("assert");
 const recall = require("./core/recall");
 
 // --- selector ---
-assert.strictEqual(recall.activeEngineName({}), "classic", "default → classic");
-assert.strictEqual(recall.activeEngineName({ recallEngine: "classic" }), "classic");
-assert.strictEqual(recall.activeEngineName({ recallEngine: "nope" }), "classic", "unknown → classic");
+assert.strictEqual(recall.DEFAULT_ENGINE, "discoverer", "discoverer is the product default");
+assert.strictEqual(recall.activeEngineName({}), "discoverer", "no setting → default (discoverer)");
+assert.strictEqual(recall.activeEngineName({ recallEngine: null }), "discoverer", "null setting → default");
+assert.strictEqual(recall.activeEngineName({ recallEngine: "classic" }), "classic", "classic still selectable as opt-out");
+assert.strictEqual(recall.activeEngineName({ recallEngine: "nope" }), "discoverer", "unknown → default");
 assert.strictEqual(recall.activeEngineName({ recallEngine: "CLASSIC" }), "classic", "case-insensitive");
 assert.ok(recall.listEngines().includes("classic"));
 assert.strictEqual(typeof recall.getEngine("classic").run, "function");
-assert.strictEqual(recall.getEngine("bogus").name, "classic", "getEngine falls back to classic");
+assert.strictEqual(recall.getEngine("bogus").name, "classic", "getEngine hard-floor stays classic (crash-proof)");
 
 // env override when no setting
 const prev = process.env.RECALL_ENGINE;
 process.env.RECALL_ENGINE = "classic";
-assert.strictEqual(recall.activeEngineName(null), "classic");
+assert.strictEqual(recall.activeEngineName(null), "classic", "env var overrides the default when no per-channel setting");
 if (prev === undefined) delete process.env.RECALL_ENGINE; else process.env.RECALL_ENGINE = prev;
 
 // --- classic engine orchestration: calls helpers and returns their blocks ---