@vectorize-io/self-driving-agents 0.0.17 → 0.0.19

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",
@@ -627,7 +654,7 @@ async function promptClaudeConfig(agentId, opts = { askBankId: true }) {
         apiUrl = urlInput;
         p.log.warn("Make sure your Hindsight instance is publicly accessible from Claude's servers.");
     }
-    const tokenInput = await p.text({ message: "Hindsight API token:" });
+    const tokenInput = await p.password({ message: "Hindsight API token:" });
     if (p.isCancel(tokenInput)) {
         p.cancel("Cancelled.");
         process.exit(0);
@@ -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") {
@@ -779,7 +808,7 @@ async function main() {
                         p.cancel("Cancelled.");
                         process.exit(0);
                     }
-                    const tokenInput = await p.text({ message: "Hindsight API token:" });
+                    const tokenInput = await p.password({ message: "Hindsight API token:" });
                     if (p.isCancel(tokenInput)) {
                         p.cancel("Cancelled.");
                         process.exit(0);
@@ -794,7 +823,7 @@ async function main() {
                     p.cancel("Cancelled.");
                     process.exit(0);
                 }
-                const tokenInput = await p.text({ message: "Hindsight API token:" });
+                const tokenInput = await p.password({ message: "Hindsight API token:" });
                 if (p.isCancel(tokenInput)) {
                     p.cancel("Cancelled.");
                     process.exit(0);
@@ -990,7 +1019,6 @@ async function main() {
         p.log.info([
             `Agent:     ${color.bold(agentId)}`,
             `Source:    ${color.dim(source)}`,
-            `Bank:      ${color.dim(bankId)}`,
             `API:       ${color.dim(apiUrl)}`,
             `Workspace: ${color.dim(workspaceDir)}`,
         ].join("\n"));

package/dist/tests/cli.test.js

@@ -43,7 +43,7 @@ function resolveFromPluginConfig(agentId, pc) {
         bankId = pc.bankId;
     }
     else {
-        const granularity = pc.dynamicBankGranularity || ["agent", "channel", "user"];
+        const granularity = pc.dynamicBankGranularity || ["agent"];
         const fieldMap = {
             agent: agentId,
             channel: "unknown",
@@ -221,9 +221,11 @@ describe("resolveFromPluginConfig", () => {
         });
         expect(result.bankId).toBe("my-agent::unknown::anonymous");
     });
-    it("uses default granularity when not specified", () => {
+    it("uses default granularity (agent-only) when not specified", () => {
+        // SDA forces ["agent"] in plugin config, so the dry-run preview default
+        // mirrors that. Multi-field granularity must be opted into explicitly.
         const result = resolveFromPluginConfig("my-agent", {});
-        expect(result.bankId).toBe("my-agent::unknown::anonymous");
+        expect(result.bankId).toBe("my-agent");
     });
     it("uses static bankId when dynamicBankId is false", () => {
         const result = resolveFromPluginConfig("my-agent", {
@@ -243,8 +245,8 @@ describe("resolveFromPluginConfig", () => {
         expect(result.apiUrl).toBe("https://api.hindsight.vectorize.io");
         expect(result.apiToken).toBe("hsk_abc");
         // dynamicBankId=false but no bankId set, so falls through to dynamic path
-        // with bankIdPrefix
-        expect(result.bankId).toBe("my-sandbox-marketing-seo::unknown::anonymous");
+        // with bankIdPrefix and the agent-only default granularity
+        expect(result.bankId).toBe("my-sandbox-marketing-seo");
     });
     it("resolves nemoclaw-style config with static bankId", () => {
         const result = resolveFromPluginConfig("marketing-seo", {
@@ -622,6 +624,30 @@ describe("claude-code allowed-tools merge", () => {
         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.17",
+  "version": "0.0.19",
   "description": "Install self-driving agents with portable memory on any harness",
   "type": "module",
   "main": "dist/cli.js",