fix: remove duplicate memory_compact registration + redundant schema required (OpenClaw 2026.4.5 compat) (#541)

* fix: remove duplicate memory_compact tool registration in index.ts

memory_compact was registered twice when enableManagementTools is true:
once via registerAllMemoryTools() → registerMemoryCompactTool() in
src/tools.ts, and again in a standalone block in index.ts (lines
2116–2202). OpenClaw 2026.4.5 now throws on duplicate tool name
registration, causing plugin initialization failure.

The standalone block in index.ts is the stale duplicate — the canonical
implementation lives in src/tools.ts and is already wired through
registerAllMemoryTools().

* fix: remove redundant required: ["embedding"] from configSchema root

OpenClaw 2026.4.5 changed how config values are passed to the AJV
validator during framework-level validation. The root-level
required: ["embedding"] constraint fails because the config object
reaching the validator may not have the embedding key at the time of
framework validation (timing/default-handling change).

This constraint was always redundant: the plugin's own
parsePluginConfig() already validates embedding.apiKey presence at
load time, and embedding.required: ["apiKey"] in the sub-schema
still catches misconfiguration.

---------

Co-authored-by: Max Eschbach <maxeschbach@macbookair.local>

Author: Banger455

index.ts

@@ -2113,94 +2113,6 @@ const memoryLanceDBProPlugin = {
       }
     );
 
-    // ========================================================================
-    // Memory Compaction (Progressive Summarization)
-    // ========================================================================
-
-    if (config.enableManagementTools) {
-      api.registerTool({
-        name: "memory_compact",
-        description:
-          "Consolidate semantically similar old memories into refined single entries " +
-          "(progressive summarization). Reduces noise and improves retrieval quality over time. " +
-          "Use dry_run:true first to preview the compaction plan without making changes.",
-        inputSchema: {
-          type: "object" as const,
-          properties: {
-            dry_run: {
-              type: "boolean",
-              description: "Preview clusters without writing changes. Default: false.",
-            },
-            min_age_days: {
-              type: "number",
-              description: "Only compact memories at least this many days old. Default: 7.",
-            },
-            similarity_threshold: {
-              type: "number",
-              description: "Cosine similarity threshold for clustering [0-1]. Default: 0.88.",
-            },
-            scopes: {
-              type: "array",
-              items: { type: "string" },
-              description: "Scope filter. Omit to compact all scopes.",
-            },
-          },
-          required: [],
-        },
-        execute: async (args: Record<string, unknown>) => {
-          const compactionCfg: CompactionConfig = {
-            enabled: true,
-            minAgeDays:
-              typeof args.min_age_days === "number"
-                ? args.min_age_days
-                : (config.memoryCompaction?.minAgeDays ?? 7),
-            similarityThreshold:
-              typeof args.similarity_threshold === "number"
-                ? Math.max(0, Math.min(1, args.similarity_threshold))
-                : (config.memoryCompaction?.similarityThreshold ?? 0.88),
-            minClusterSize: config.memoryCompaction?.minClusterSize ?? 2,
-            maxMemoriesToScan: config.memoryCompaction?.maxMemoriesToScan ?? 200,
-            dryRun: args.dry_run === true,
-            cooldownHours: config.memoryCompaction?.cooldownHours ?? 24,
-          };
-          const scopes =
-            Array.isArray(args.scopes) && args.scopes.length > 0
-              ? (args.scopes as string[])
-              : undefined;
-
-          const result = await runCompaction(
-            store,
-            embedder,
-            compactionCfg,
-            scopes,
-            api.logger,
-          );
-
-          return {
-            content: [
-              {
-                type: "text",
-                text: JSON.stringify(
-                  {
-                    scanned: result.scanned,
-                    clustersFound: result.clustersFound,
-                    memoriesDeleted: result.memoriesDeleted,
-                    memoriesCreated: result.memoriesCreated,
-                    dryRun: result.dryRun,
-                    summary: result.dryRun
-                      ? `Dry run: found ${result.clustersFound} cluster(s) in ${result.scanned} memories — no changes made.`
-                      : `Compacted ${result.memoriesDeleted} memories into ${result.memoriesCreated} consolidated entries.`,
-                  },
-                  null,
-                  2,
-                ),
-              },
-            ],
-          };
-        },
-      });
-    }
-
     // Auto-compaction at gateway_start (if enabled, respects cooldown)
     if (config.memoryCompaction?.enabled) {
       api.on("gateway_start", () => {

openclaw.plugin.json

@@ -854,10 +854,7 @@
           }
         }
       }
-    },
-    "required": [
-      "embedding"
-    ]
+    }
   },
   "uiHints": {
     "embedding.apiKey": {