@memoryrelay/plugin-memoryrelay-ai 0.8.4 → 0.8.6

package/index.ts

@@ -1,6 +1,6 @@
 /**
  * OpenClaw Memory Plugin - MemoryRelay
- * Version: 0.8.4 (OpenClaw Security Compliance)
+ * Version: 0.8.6 (OpenClaw Security Compliance)
  *
  * Long-term memory with vector search using MemoryRelay API.
  * Provides auto-recall and auto-capture via lifecycle hooks.
@@ -9,6 +9,11 @@
  * API: https://api.memoryrelay.net
  * Docs: https://memoryrelay.ai
  *
+ * ENHANCEMENTS (v0.8.6):
+ * - Removed fs.writeFile from export command (stdout only now)
+ * - No filesystem operations - passes OpenClaw security validation
+ * - Export usage: openclaw memoryrelay export > memories.json
+ *
  * ENHANCEMENTS (v0.8.4):
  * - Removed file logging feature to pass OpenClaw security validation
  * - All debug logs now in-memory only (circular buffer)
@@ -858,7 +863,7 @@ export default async function plugin(api: OpenClawPluginApi): Promise<void> {
   const verboseEnabled = cfg?.verbose || false;
   const maxLogEntries = cfg?.maxLogEntries || 100;
   
-  // Note: logFile is deprecated in v0.8.4 (removed for OpenClaw security compliance)
+  // Note: logFile is deprecated in v0.8.6 (removed for OpenClaw security compliance)
   // All debug logs are in-memory only. Use gateway methods to access logs.
   
   let debugLogger: DebugLogger | undefined;
@@ -3209,15 +3214,12 @@ export default async function plugin(api: OpenClawPluginApi): Promise<void> {
 
       mem
         .command("export")
-        .description("Export all memories to JSON file")
-        .option("--output <path>", "Output file path", "memories-export.json")
-        .action(async (opts) => {
+        .description("Export all memories to JSON (outputs to stdout)")
+        .action(async () => {
           try {
-            console.log("Exporting memories...");
             const memories = await client.export();
-            const fs = await import("fs/promises");
-            await fs.writeFile(opts.output, JSON.stringify(memories, null, 2));
-            console.log(`Exported ${memories.length} memories to ${opts.output}`);
+            console.log(JSON.stringify(memories, null, 2));
+            console.error(`\n# Exported ${memories.length} memories. Redirect stdout to save: memoryrelay export > memories.json`);
           } catch (err) {
             console.error(`Export failed: ${String(err)}`);
           }

package/openclaw.plugin.json

@@ -3,7 +3,7 @@
   "kind": "memory",
   "name": "MemoryRelay AI",
   "description": "AI memory service with sessions, decisions, patterns & projects (api.memoryrelay.net)",
-  "version": "0.8.4",
+  "version": "0.8.6",
   "uiHints": {
     "apiKey": {
       "label": "MemoryRelay API Key",
@@ -113,7 +113,7 @@
       },
       "logFile": {
         "type": "string",
-        "description": "DEPRECATED (v0.8.4): File logging removed for OpenClaw security compliance. This option is ignored. Use gateway methods to access in-memory logs (coming in v0.9.0)."
+        "description": "DEPRECATED (v0.8.6): File logging removed for OpenClaw security compliance. This option is ignored. Use gateway methods to access in-memory logs (coming in v0.9.0)."
       },
       "maxLogEntries": {
         "type": "number",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memoryrelay/plugin-memoryrelay-ai",
-  "version": "0.8.4",
+  "version": "0.8.6",
   "description": "OpenClaw memory plugin for MemoryRelay API - sessions, decisions, patterns, projects & semantic search",
   "type": "module",
   "main": "index.ts",
@@ -51,7 +51,8 @@
   },
   "files": [
     "index.ts",
-    "src/",
+    "src/*.ts",
+    "!src/*.test.ts",
     "bin/",
     "openclaw.plugin.json",
     "README.md",

package/src/debug-logger.test.ts

@@ -1,233 +0,0 @@
-/**
- * DebugLogger Tests (Corrected)
- * 
- * Tests matching actual implementation
- */
-
-import { describe, test, expect, beforeEach, vi } from "vitest";
-import { DebugLogger, type LogEntry } from "./debug-logger";
-import * as fs from "fs";
-
-vi.mock("fs");
-
-describe("DebugLogger", () => {
-  let logger: DebugLogger;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    logger = new DebugLogger({
-      enabled: true,
-      verbose: false,
-      maxEntries: 5,
-    });
-  });
-
-  test("logs are stored when enabled", () => {
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-
-    const logs = logger.getAllLogs();
-    expect(logs).toHaveLength(1);
-  });
-
-  test("logs are not stored when disabled", () => {
-    const disabledLogger = new DebugLogger({
-      enabled: false,
-      verbose: false,
-      maxEntries: 5,
-    });
-
-    disabledLogger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-
-    const logs = disabledLogger.getAllLogs();
-    expect(logs).toHaveLength(0);
-  });
-
-  test("respects circular buffer limit (FIFO)", () => {
-    for (let i = 0; i < 10; i++) {
-      logger.log({
-        timestamp: new Date(Date.now() + i).toISOString(),
-        tool: `tool_${i}`,
-        method: "GET",
-        path: `/test/${i}`,
-        duration: 100,
-        status: "success",
-      });
-    }
-
-    const logs = logger.getAllLogs();
-    expect(logs).toHaveLength(5); // maxEntries = 5
-    expect(logs[0].tool).toBe("tool_5"); // Oldest kept
-    expect(logs[4].tool).toBe("tool_9"); // Newest
-  });
-
-  test("getRecentLogs returns last N entries", () => {
-    for (let i = 0; i < 3; i++) {
-      logger.log({
-        timestamp: new Date(Date.now() + i).toISOString(),
-        tool: `tool_${i}`,
-        method: "GET",
-        path: `/test/${i}`,
-        duration: 100,
-        status: "success",
-      });
-    }
-
-    const logs = logger.getRecentLogs(2);
-    expect(logs).toHaveLength(2);
-    expect(logs[0].tool).toBe("tool_1"); // Second-to-last
-    expect(logs[1].tool).toBe("tool_2"); // Last
-  });
-
-  test("getToolLogs filters by tool name", () => {
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_recall",
-      method: "POST",
-      path: "/v1/memories/search",
-      duration: 78,
-      status: "success",
-    });
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 156,
-      status: "error",
-      error: "500",
-    });
-
-    const logs = logger.getToolLogs("memory_store", 10);
-    expect(logs).toHaveLength(2);
-    expect(logs.every(l => l.tool === "memory_store")).toBe(true);
-  });
-
-  test("getErrorLogs filters by error status", () => {
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_recall",
-      method: "POST",
-      path: "/v1/memories/search",
-      duration: 78,
-      status: "error",
-      error: "404",
-    });
-
-    const logs = logger.getErrorLogs(10);
-    expect(logs).toHaveLength(1);
-    expect(logs[0].status).toBe("error");
-  });
-
-  test("getStats calculates correctly", () => {
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 100,
-      status: "success",
-    });
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_recall",
-      method: "POST",
-      path: "/v1/memories/search",
-      duration: 200,
-      status: "success",
-    });
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 150,
-      status: "error",
-      error: "500",
-    });
-
-    const stats = logger.getStats();
-    expect(stats.total).toBe(3);
-    expect(stats.successful).toBe(2);
-    expect(stats.failed).toBe(1);
-    expect(stats.avgDuration).toBe(150); // (100+200+150)/3 = 150
-  });
-
-  test("clear() empties logs", () => {
-    logger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-
-    logger.clear();
-    const logs = logger.getAllLogs();
-    expect(logs).toHaveLength(0);
-  });
-
-  test("formatEntry creates human-readable output", () => {
-    const entry: LogEntry = {
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    };
-
-    const formatted = DebugLogger.formatEntry(entry);
-    expect(formatted).toContain("memory_store");
-    expect(formatted).toContain("142ms");
-    expect(formatted).toContain("✓");
-  });
-
-  test("formatEntry shows error", () => {
-    const entry: LogEntry = {
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 156,
-      status: "error",
-      error: "500 Internal Server Error",
-    };
-
-    const formatted = DebugLogger.formatEntry(entry);
-    expect(formatted).toContain("memory_store");
-    expect(formatted).toContain("156ms");
-    expect(formatted).toContain("✗");
-    expect(formatted).toContain("500 Internal Server Error");
-  });
-});

package/src/status-reporter.test.ts

@@ -1,230 +0,0 @@
-/**
- * StatusReporter Tests (Simplified)
- * 
- * Tests matching actual implementation
- */
-
-import { describe, test, expect, beforeEach } from "vitest";
-import { StatusReporter, type ConnectionStatus, type PluginConfig, type MemoryStats } from "./status-reporter";
-import { DebugLogger } from "./debug-logger";
-
-describe("StatusReporter", () => {
-  let reporter: StatusReporter;
-  let debugLogger: DebugLogger;
-  let mockConnection: ConnectionStatus;
-  let mockConfig: PluginConfig;
-  let mockStats: MemoryStats;
-
-  beforeEach(() => {
-    debugLogger = new DebugLogger({
-      enabled: true,
-      verbose: false,
-      maxEntries: 100,
-    });
-    reporter = new StatusReporter(debugLogger);
-
-    mockConnection = {
-      status: "connected",
-      endpoint: "https://api.memoryrelay.net",
-      lastCheck: new Date().toISOString(),
-      responseTime: 45,
-    };
-
-    mockConfig = {
-      agentId: "test-agent",
-      autoRecall: true,
-      autoCapture: false,
-      recallLimit: 5,
-      recallThreshold: 0.3,
-      excludeChannels: [],
-    };
-
-    mockStats = {
-      total_memories: 100,
-    };
-  });
-
-  test("records and clears tool failures", () => {
-    reporter.recordFailure("memory_store", "500 Error");
-    let issues = reporter.getIssues();
-    expect(issues).toHaveLength(1);
-    expect(issues[0].tool).toBe("memory_store");
-
-    reporter.recordSuccess("memory_store");
-    issues = reporter.getIssues();
-    expect(issues).toHaveLength(0);
-  });
-
-  test("buildReport creates status report", () => {
-    const toolGroups = {
-      "Core Memory": ["memory_store", "memory_recall"],
-      "Projects": ["project_list"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    expect(report.connection).toEqual(mockConnection);
-    expect(report.config).toEqual(mockConfig);
-    expect(report.stats).toEqual(mockStats);
-    expect(report.tools["Core Memory"]).toBeDefined();
-    expect(report.tools["Projects"]).toBeDefined();
-  });
-
-  test("buildReport includes tool status from debug logs", () => {
-    debugLogger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-
-    const toolGroups = {
-      "Core Memory": ["memory_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    const memoryTools = report.tools["Core Memory"];
-    expect(memoryTools.available).toBe(1);
-    expect(memoryTools.failed).toBe(0);
-    expect(memoryTools.tools[0].status).toBe("working");
-  });
-
-  test("buildReport shows failed tools", () => {
-    debugLogger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 156,
-      status: "error",
-      error: "500 Internal Server Error",
-    });
-
-    const toolGroups = {
-      "Core Memory": ["memory_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    const memoryTools = report.tools["Core Memory"];
-    expect(memoryTools.available).toBe(0);
-    expect(memoryTools.failed).toBe(1);
-    expect(memoryTools.tools[0].status).toBe("error");
-    expect(memoryTools.tools[0].error).toBe("500 Internal Server Error");
-  });
-
-  test("buildReport includes recent calls", () => {
-    debugLogger.log({
-      timestamp: new Date().toISOString(),
-      tool: "memory_store",
-      method: "POST",
-      path: "/v1/memories",
-      duration: 142,
-      status: "success",
-    });
-
-    const toolGroups = {
-      "Core Memory": ["memory_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    expect(report.recentCalls).toHaveLength(1);
-    expect(report.recentCalls[0].tool).toBe("memory_store");
-  });
-
-  test("formatReport creates human-readable output", () => {
-    const toolGroups = {
-      "Core Memory": ["memory_store", "memory_recall"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    const formatted = StatusReporter.formatReport(report);
-    expect(formatted).toContain("MemoryRelay Plugin Status");
-    expect(formatted).toContain("connected");
-    expect(formatted).toContain("Core Memory");
-  });
-
-  test("formatCompact creates brief output", () => {
-    const toolGroups = {
-      "Core Memory": ["memory_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    const compact = StatusReporter.formatCompact(report);
-    expect(compact).toContain("connected");
-    expect(compact.length).toBeLessThan(200); // Should be brief
-  });
-
-  test("handles disconnected status", () => {
-    mockConnection.status = "disconnected";
-
-    const toolGroups = {
-      "Core Memory": ["memory_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    expect(report.connection.status).toBe("disconnected");
-    const formatted = StatusReporter.formatReport(report);
-    expect(formatted).toContain("disconnected");
-  });
-
-  test("includes issues in report", () => {
-    reporter.recordFailure("memory_batch_store", "500 Error");
-
-    const toolGroups = {
-      "Core Memory": ["memory_batch_store"],
-    };
-
-    const report = reporter.buildReport(
-      mockConnection,
-      mockConfig,
-      mockStats,
-      toolGroups,
-    );
-
-    expect(report.issues).toHaveLength(1);
-    expect(report.issues[0].tool).toBe("memory_batch_store");
-  });
-});