@memoryrelay/plugin-memoryrelay-ai 0.8.5 → 0.8.7

package/index.ts

@@ -1,6 +1,6 @@
 /**
  * OpenClaw Memory Plugin - MemoryRelay
- * Version: 0.8.5 (OpenClaw Security Compliance)
+ * Version: 0.8.7 (OpenClaw Security Compliance)
  *
  * Long-term memory with vector search using MemoryRelay API.
  * Provides auto-recall and auto-capture via lifecycle hooks.
@@ -9,7 +9,7 @@
  * API: https://api.memoryrelay.net
  * Docs: https://memoryrelay.ai
  *
- * ENHANCEMENTS (v0.8.5):
+ * ENHANCEMENTS (v0.8.7):
  * - Removed fs.writeFile from export command (stdout only now)
  * - No filesystem operations - passes OpenClaw security validation
  * - Export usage: openclaw memoryrelay export > memories.json
@@ -863,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.5 (removed for OpenClaw security compliance)
+  // Note: logFile is deprecated in v0.8.7 (removed for OpenClaw security compliance)
   // All debug logs are in-memory only. Use gateway methods to access logs.
   
   let debugLogger: DebugLogger | undefined;

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.5",
+  "version": "0.8.7",
   "uiHints": {
     "apiKey": {
       "label": "MemoryRelay API Key",
@@ -113,7 +113,7 @@
       },
       "logFile": {
         "type": "string",
-        "description": "DEPRECATED (v0.8.5): 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.7): 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,14 +1,13 @@
 {
   "name": "@memoryrelay/plugin-memoryrelay-ai",
-  "version": "0.8.5",
+  "version": "0.8.7",
   "description": "OpenClaw memory plugin for MemoryRelay API - sessions, decisions, patterns, projects & semantic search",
   "type": "module",
   "main": "index.ts",
   "scripts": {
     "test": "vitest run",
     "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
-    "postinstall": "node -e \"console.log('\\n⚠️  CONFIGURATION REQUIRED\\n\\nAdd config after installation:\\n\\ncat ~/.openclaw/openclaw.json | jq \\'.plugins.entries.\\\"plugin-memoryrelay-ai\\\".config = {\\\"apiKey\\\": \\\"YOUR_KEY\\\", \\\"agentId\\\": \\\"YOUR_AGENT\\\"}\\' > /tmp/config.json && mv /tmp/config.json ~/.openclaw/openclaw.json\\n\\nOr use environment variables:\\nexport MEMORYRELAY_API_KEY=\\\"mem_prod_...\\\"\\nexport MEMORYRELAY_AGENT_ID=\\\"your-agent\\\"\\n\\nThen restart: openclaw gateway restart\\n\\nGet your API key from: https://memoryrelay.ai\\n')\""
+    "test:coverage": "vitest run --coverage"
   },
   "keywords": [
     "openclaw",
@@ -51,7 +50,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");
-  });
-});