fix: register retry with Map + resetRegistration clear (#448)

Author: jlin53882

index.ts

@@ -1615,10 +1615,14 @@ const pluginVersion = getPluginVersion();
 // Plugin Definition
 // ============================================================================
 
-// WeakSet keyed by API instance — each distinct API object tracks its own initialized state.
-// Using WeakSet instead of a module-level boolean avoids the "second register() call skips
-// hook/tool registration for the new API instance" regression that rwmjhb identified.
-const _registeredApis = new WeakSet<OpenClawPluginApi>();
+// Map keyed by API instance — each distinct API object tracks its own initialized state.
+// Using Map instead of WeakSet allows tracking per-instance state AND enables clear() for reset.
+const _registeredApis = new Map<OpenClawPluginApi, boolean>();
+
+// Helper for tests to check registered APIs
+export function _getRegisteredApisForTest(): Map<OpenClawPluginApi, boolean> {
+  return _registeredApis;
+}
 
 const memoryLanceDBProPlugin = {
   id: "memory-lancedb-pro",
@@ -1633,7 +1637,7 @@ const memoryLanceDBProPlugin = {
       api.logger.debug?.("memory-lancedb-pro: register() called again — skipping re-init (idempotent)");
       return;
     }
-    _registeredApis.add(api);
+    // Note: Map.set(api, true) is called AFTER successful init in try block
 
     // Parse and validate configuration
     const config = parsePluginConfig(api.pluginConfig);