@inetafrica/open-claudia 2.6.47 → 2.6.49

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "2.6.47",
+  "version": "2.6.49",
   "description": "Your always-on AI coding assistant — Claude Code, Cursor Agent, and OpenAI Codex via Telegram or Kazee Chat",
   "main": "bot.js",
   "bin": {
@@ -9,7 +9,7 @@
   "scripts": {
     "setup": "node setup.js",
     "start": "node bot.js",
-    "test": "OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node -e \"require('./vault'); console.log('OK')\" && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-usage-accounting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-engine.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-graph.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-discoverer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-project-transcripts-smoke.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-abilities.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-extraction.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-couse.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-transfer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-tiers.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-merge-guard.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-learning-e2e.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-pack-nesting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-read-signal.js"
+    "test": "OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node -e \"require('./vault'); console.log('OK')\" && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-usage-accounting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-engine.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-graph.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-recall-discoverer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-project-transcripts-smoke.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-abilities.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-extraction.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-couse.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-transfer.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-tiers.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-ability-merge-guard.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-learning-e2e.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-pack-nesting.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-read-signal.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-tools.js && OPEN_CLAUDIA_TEST=1 WORKSPACE=/tmp/open-claudia-test CLAUDE_PATH=node node test-tool-graph.js"
   },
   "files": [
     "bot.js",
@@ -43,7 +43,9 @@
     "test-ability-merge-guard.js",
     "test-learning-e2e.js",
     "test-pack-nesting.js",
-    "test-read-signal.js"
+    "test-read-signal.js",
+    "test-tools.js",
+    "test-tool-graph.js"
   ],
   "keywords": [
     "claude",

package/test-tool-graph.js

@@ -0,0 +1,97 @@
+// Directed tool-graph: "tool B follows tool A". Unlike the recall graph this is
+// DIRECTED (A→B carries info B→A does not) and has NO structural floor, so an
+// unused chain decays to zero and is pruned. This exercises core/tool-graph.js
+// end to end against an isolated DB so nothing touches the real graph.
+const assert = require("assert");
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "tool-graph-test-"));
+process.env.TOOL_GRAPH_DB = path.join(tmp, "tool-graph.db");
+
+const graph = require("./core/tool-graph");
+
+// node:sqlite arrived in node 22; on older runtimes the graph is a no-op. Skip
+// cleanly so the suite stays green rather than failing on the environment.
+if (!graph.available()) {
+  console.log("tool-graph OK (node:sqlite unavailable — skipped)");
+  process.exit(0);
+}
+
+// ── reinforceSequence lays directed edges for IMMEDIATE succession only ──
+graph.reinforceSequence(["auth", "list", "download"]);
+let after = graph.followers("auth");
+assert.strictEqual(after.length, 1, "auth has exactly one follower");
+assert.strictEqual(after[0].name, "list", "list follows auth");
+assert.strictEqual(after[0].weight, 1, "a single co-occurrence weighs 1 (no structural floor)");
+
+// non-adjacent pairs are NOT linked — auth→download must not exist
+assert.ok(!graph.followers("auth").some((r) => r.name === "download"), "auth→download skipped (not adjacent)");
+
+// ── directionality: the edge is auth→list, never list→auth ──
+assert.ok(graph.followers("list").some((r) => r.name === "download"), "download follows list");
+assert.ok(!graph.followers("list").some((r) => r.name === "auth"), "auth does NOT follow list (directed)");
+assert.strictEqual(graph.predecessors("download")[0].name, "list", "list precedes download");
+assert.ok(graph.predecessors("list").some((r) => r.name === "auth"), "auth precedes list");
+assert.ok(!graph.predecessors("list").some((r) => r.name === "download"), "download does NOT precede list (directed)");
+
+// ── Hebbian bump: re-running the chain strengthens the same edges ──
+graph.reinforceSequence(["auth", "list", "download"]);
+assert.strictEqual(graph.followers("auth")[0].weight, 2, "second co-use bumps auth→list to 2");
+
+// ── repeated names in a row collapse: a tool re-run in a loop must not self-link ──
+graph.reinforceSequence(["loop", "loop", "loop", "end"]);
+assert.ok(!graph.followers("loop").some((r) => r.name === "loop"), "no loop→loop self-edge");
+assert.strictEqual(graph.followers("loop")[0].name, "end", "loop→end is the only edge from a repeated run");
+
+// ── addFollow guards: src===dst is rejected ──
+assert.strictEqual(graph.addFollow("x", "x"), false, "self-edge refused");
+
+// ── a pure 'ensure' edge (no bump) has no last_reinforced and so survives decay ──
+graph.addFollow("ensure-src", "ensure-dst"); // weight 1, last_reinforced=null
+const ensured = graph.followers("ensure-src");
+assert.strictEqual(ensured[0].weight, 1, "ensure edge weighs 1");
+
+// ── decay: backdate a reinforced edge far into the past, then it prunes (no floor) ──
+const db = graph.openDb();
+const longAgo = new Date(Date.now() - 365 * 86400000).toISOString();
+db.prepare("UPDATE tool_edges SET last_reinforced=? WHERE src=? AND dst=?").run(longAgo, "auth", "list");
+let res = graph.decay({ halfLifeDays: 1, pruneBelow: 0.15 });
+assert.ok(res.pruned >= 1, "a year-old edge under the floor is pruned");
+assert.ok(!graph.followers("auth").some((r) => r.name === "list"), "pruned edge is gone");
+
+// the ensure edge (last_reinforced=null) was skipped by decay, not pruned
+assert.ok(graph.followers("ensure-src").some((r) => r.name === "ensure-dst"), "un-reinforced edge survives decay");
+
+// ── decay that reduces but keeps an edge above the floor ──
+graph.addFollow("warm-a", "warm-b", { bump: 10 });
+const recent = new Date(Date.now() - 30 * 86400000).toISOString();
+db.prepare("UPDATE tool_edges SET last_reinforced=? WHERE src=? AND dst=?").run(recent, "warm-a", "warm-b");
+graph.decay({ halfLifeDays: 30, pruneBelow: 0.15 }); // one half-life → ~5
+const warm = graph.followers("warm-a")[0];
+assert.ok(warm && warm.name === "warm-b", "warm edge survives a single half-life");
+assert.ok(warm.weight < 10 && warm.weight > 0.15, `warm edge decayed but kept (${warm.weight})`);
+
+// ── removeEdge ──
+assert.strictEqual(graph.removeEdge("warm-a", "warm-b"), true, "removeEdge succeeds");
+assert.ok(!graph.followers("warm-a").length, "removed edge gone");
+
+// ── pruneOrphans drops edges whose endpoints are no longer registered tools ──
+graph.reinforceSequence(["live", "dead"]); // 'dead' will not be in the live set
+const fakeLib = { listTools: () => [{ name: "live" }, { name: "end" }, { name: "loop" }, { name: "ensure-src" }, { name: "ensure-dst" }] };
+const removed = graph.pruneOrphans(fakeLib);
+assert.ok(removed >= 1, "edge pointing at an unregistered tool ('dead') is pruned");
+assert.ok(!graph.followers("live").length, "live→dead removed because dead is not a live tool");
+
+// ── tend() is the nightly entry: decays + prunes orphans + returns stats ──
+const t = graph.tend(fakeLib);
+assert.ok(typeof t.edges === "number" && typeof t.nodes === "number", "tend returns stats");
+assert.ok(typeof t.pruned === "number" && typeof t.decayed === "number", "tend returns decay/prune counts");
+
+// ── stats / allEdges sanity ──
+const s = graph.stats();
+assert.strictEqual(s.edges, graph.allEdges().length, "stats edge count matches allEdges");
+
+graph._resetForTest();
+console.log("tool-graph OK");

package/test-tools.js

@@ -0,0 +1,92 @@
+// Reusable tools: a script the agent crystallises must register with a parseable
+// header, run with the keyring merged into its env (preauth), and be findable by
+// the always-on tool index. This exercises the core/tools.js library end to end
+// in an isolated TOOLS_DIR so nothing touches the real ~/.open-claudia/tools.
+const assert = require("assert");
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), "tools-test-"));
+process.env.TOOLS_DIR = path.join(tmp, "tools");
+
+const tools = require("./core/tools");
+
+// ── add a bare script (no header): a header is synthesised from opts, shebang
+//    is preserved on line 1, and the file is made executable ──
+const srcA = path.join(tmp, "hello.sh");
+fs.writeFileSync(srcA, "#!/usr/bin/env bash\necho \"hi $1\"\n");
+const a = tools.addTool(srcA, {
+  name: "hello",
+  description: "say hi",
+  pack: "greetings",
+  requires: ["api_key", "api_secret"],
+  usage: "hello <name>",
+});
+assert.strictEqual(a.name, "hello", "synthesised tool keeps requested name");
+assert.strictEqual(a.description, "say hi", "description parsed back from synthesised header");
+assert.strictEqual(a.pack, "greetings", "pack link parsed back");
+assert.deepStrictEqual(a.requires, ["api_key", "api_secret"], "requires parsed back as a list");
+assert.strictEqual(a.usage, "hello <name>", "usage parsed back");
+assert.ok(a.executable, "stored tool is chmod +x");
+assert.ok(a.content.startsWith("#!/usr/bin/env bash\n"), "shebang stays on line 1");
+assert.ok(/# open-claudia-tool: hello/.test(a.content), "marker line injected with '#' prefix for a shell script");
+
+// ── findTool / listTools see it ──
+assert.strictEqual(tools.findTool("hello").name, "hello", "findTool resolves by name");
+assert.strictEqual(tools.findTool("HELLO").name, "hello", "findTool is case-insensitive (sanitized)");
+assert.ok(tools.listTools().some((t) => t.name === "hello"), "listTools includes the new tool");
+
+// ── a JS source with a shebang gets a '//' comment header, not '#' ──
+const srcB = path.join(tmp, "fetch.js");
+fs.writeFileSync(srcB, "#!/usr/bin/env node\nconsole.log('x');\n");
+const b = tools.addTool(srcB, { name: "fetch", description: "fetch a thing" });
+assert.ok(/\/\/ open-claudia-tool: fetch/.test(b.content), "JS script gets '//' marker prefix");
+assert.strictEqual(b.requires.length, 0, "no requires when none supplied");
+
+// ── a source that already declares its own header is respected; a --pack passed
+//    in is grafted on without clobbering the existing marker ──
+const srcC = path.join(tmp, "self-doc.sh");
+fs.writeFileSync(
+  srcC,
+  "#!/usr/bin/env bash\n# open-claudia-tool: selfdoc\n# description: pre-documented\necho hi\n"
+);
+const c = tools.addTool(srcC, { pack: "linked-pack" });
+assert.strictEqual(c.name, "selfdoc", "existing marker name wins");
+assert.strictEqual(c.description, "pre-documented", "existing description preserved");
+assert.strictEqual(c.pack, "linked-pack", "--pack grafted onto a self-documented script");
+
+// ── missingRequires: keys absent from keyring AND env are reported; present env
+//    keys are not ──
+process.env.api_key = "present";
+delete process.env.api_secret;
+const missing = tools.missingRequires(tools.findTool("hello"));
+assert.ok(!missing.includes("api_key"), "a key present in env is not 'missing'");
+assert.ok(missing.includes("api_secret"), "a key absent everywhere is 'missing'");
+delete process.env.api_key;
+
+// ── runEnv returns an env object (keyring merged or graceful fallback) ──
+const env = tools.runEnv();
+assert.ok(env && typeof env === "object" && env.PATH, "runEnv yields an env object carrying PATH");
+
+// ── toolNameFromPath only recognises a direct child of TOOLS_DIR ──
+assert.strictEqual(
+  tools.toolNameFromPath(path.join(process.env.TOOLS_DIR, "hello")),
+  "hello",
+  "a file directly in TOOLS_DIR resolves to its tool name"
+);
+assert.strictEqual(tools.toolNameFromPath("/etc/passwd"), null, "a path outside TOOLS_DIR → null");
+assert.strictEqual(
+  tools.toolNameFromPath(path.join(process.env.TOOLS_DIR, "sub", "deep")),
+  null,
+  "a nested path under TOOLS_DIR → null"
+);
+
+// ── parseHeader returns null for a script with no marker line ──
+assert.strictEqual(tools.parseHeader("#!/bin/sh\necho hi\n"), null, "no marker → not a tool");
+
+// ── remove ──
+assert.ok(tools.removeTool("hello"), "removeTool returns the removed tool");
+assert.strictEqual(tools.findTool("hello"), null, "removed tool is gone");
+
+console.log("tools OK");