@vectorize-io/self-driving-agents 0.0.16 → 0.0.18

package/dist/cli.js

@@ -125,6 +125,33 @@ function enableKnowledgeTools() {
     pc.enableKnowledgeTools = true;
     writeFileSync(OPENCLAW_CONFIG_PATH, JSON.stringify(config, null, 2) + "\n");
 }
+// SDA agents are isolated per-agent only — no channel/user. The plugin's default
+// (["agent","channel","user"]) produces banks like "marketing-seo::unknown::anonymous"
+// for local sessions where channel/sender aren't populated.
+async function ensureOpenClawAgentGranularity() {
+    const config = readOpenClawConfig();
+    if (!config)
+        return;
+    const pc = config.plugins?.entries?.["hindsight-openclaw"]?.config;
+    if (!pc)
+        return;
+    const current = pc.dynamicBankGranularity;
+    const desired = ["agent"];
+    const matches = Array.isArray(current) && current.length === 1 && current[0] === "agent";
+    if (matches)
+        return;
+    if (current && current.length > 0) {
+        const ok = await p.confirm({
+            message: `Plugin has dynamicBankGranularity=${color.yellow(JSON.stringify(current))}. SDA agents need ${color.cyan('["agent"]')}. Override?`,
+            initialValue: true,
+        });
+        if (p.isCancel(ok) || !ok)
+            return;
+    }
+    pc.dynamicBankGranularity = desired;
+    writeFileSync(OPENCLAW_CONFIG_PATH, JSON.stringify(config, null, 2) + "\n");
+    p.log.success("Set dynamicBankGranularity = [\"agent\"] for per-agent bank isolation");
+}
 const MIN_PLUGIN_VERSION = "0.7.2";
 function getInstalledPluginVersion() {
     try {
@@ -178,7 +205,7 @@ function resolveFromPlugin(agentId) {
         bankId = pc.bankId;
     }
     else {
-        const granularity = pc.dynamicBankGranularity || ["agent", "channel", "user"];
+        const granularity = pc.dynamicBankGranularity || ["agent"];
         const fieldMap = {
             agent: agentId,
             channel: "unknown",
@@ -724,10 +751,12 @@ async function main() {
         if (harness === "openclaw") {
             await ensurePlugin();
             enableKnowledgeTools();
+            await ensureOpenClawAgentGranularity();
             ({ apiUrl, bankId, apiToken } = resolveFromPlugin(agentId));
         }
         else if (harness === "nemoclaw") {
             await ensureNemoClawPlugin(sandbox, agentId);
+            await ensureOpenClawAgentGranularity();
             ({ apiUrl, bankId, apiToken } = resolveFromPlugin(agentId));
         }
         else if (harness === "hermes") {
@@ -948,22 +977,24 @@ async function main() {
                     /* ignore */
                 }
             }
-            const allowedTools = userSettings.allowedTools || [];
+            const permissions = (userSettings.permissions ||= {});
+            const allow = permissions.allow || [];
             const toolsToAllow = [
-                "mcp__hindsight__*",
+                "mcp__plugin_hindsight-memory_hindsight__*",
                 "Skill(hindsight-memory:create-agent)",
                 `Bash(ls ~/.self-driving-agents/*)`,
                 `Bash(cat ~/.self-driving-agents/*)`,
             ];
             let updated = false;
             for (const tool of toolsToAllow) {
-                if (!allowedTools.includes(tool)) {
-                    allowedTools.push(tool);
+                if (!allow.includes(tool)) {
+                    allow.push(tool);
                     updated = true;
                 }
             }
             if (updated) {
-                userSettings.allowedTools = allowedTools;
+                permissions.allow = allow;
+                userSettings.permissions = permissions;
                 writeFileSync(userSettingsPath, JSON.stringify(userSettings, null, 2) + "\n");
                 p.log.success("Auto-approved hindsight tools in Claude Code");
             }

package/dist/tests/cli.test.js

@@ -580,7 +580,7 @@ describe("claude-code marketplace detection", () => {
     });
 });
 describe("claude-code allowed-tools merge", () => {
-    // Mirrors the auto-approve logic that merges entries into ~/.claude/settings.json's allowedTools.
+    // Mirrors the auto-approve logic that merges entries into ~/.claude/settings.json's permissions.allow.
     function mergeAllowed(existing, toAdd) {
         const merged = [...existing];
         let updated = false;
@@ -593,7 +593,7 @@ describe("claude-code allowed-tools merge", () => {
         return { merged, updated };
     }
     const HINDSIGHT_TOOLS = [
-        "mcp__hindsight__*",
+        "mcp__plugin_hindsight-memory_hindsight__*",
         "Skill(hindsight-memory:create-agent)",
         "Bash(ls ~/.self-driving-agents/*)",
         "Bash(cat ~/.self-driving-agents/*)",
@@ -607,7 +607,7 @@ describe("claude-code allowed-tools merge", () => {
         const { merged } = mergeAllowed(["Bash(npm *)", "Read"], HINDSIGHT_TOOLS);
         expect(merged).toContain("Bash(npm *)");
         expect(merged).toContain("Read");
-        expect(merged).toContain("mcp__hindsight__*");
+        expect(merged).toContain("mcp__plugin_hindsight-memory_hindsight__*");
     });
     it("does not duplicate when already present", () => {
         const existing = [...HINDSIGHT_TOOLS];
@@ -616,12 +616,36 @@ describe("claude-code allowed-tools merge", () => {
         expect(merged).toHaveLength(HINDSIGHT_TOOLS.length);
     });
     it("only adds missing entries", () => {
-        const existing = ["mcp__hindsight__*"];
+        const existing = ["mcp__plugin_hindsight-memory_hindsight__*"];
         const { merged, updated } = mergeAllowed(existing, HINDSIGHT_TOOLS);
         expect(updated).toBe(true);
         expect(merged).toHaveLength(HINDSIGHT_TOOLS.length);
     });
 });
+describe("openclaw dynamicBankGranularity enforcement", () => {
+    function decide(current) {
+        if (Array.isArray(current) && current.length === 1 && current[0] === "agent") {
+            return "noop";
+        }
+        if (Array.isArray(current) && current.length > 0) {
+            return "ask-confirm";
+        }
+        return "set-fresh";
+    }
+    it("sets [agent] when not configured (fresh install)", () => {
+        expect(decide(undefined)).toBe("set-fresh");
+        expect(decide(null)).toBe("set-fresh");
+        expect(decide([])).toBe("set-fresh");
+    });
+    it("noop when already [agent]", () => {
+        expect(decide(["agent"])).toBe("noop");
+    });
+    it("asks confirmation when set to a different value", () => {
+        expect(decide(["agent", "channel", "user"])).toBe("ask-confirm");
+        expect(decide(["channel"])).toBe("ask-confirm");
+        expect(decide(["agent", "project"])).toBe("ask-confirm");
+    });
+});
 describe("claude-code Hindsight config persistence", () => {
     // Mirrors the config-write logic: if existing config has a connection
     // (hindsightApiUrl or llmProvider), don't prompt; otherwise prompt.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vectorize-io/self-driving-agents",
-  "version": "0.0.16",
+  "version": "0.0.18",
   "description": "Install self-driving agents with portable memory on any harness",
   "type": "module",
   "main": "dist/cli.js",