fix: log warn on initFailed in service.start() and fix stale test comment

When secret resolution fails, service.start() now logs a visible warn
explaining why memory hooks are disabled, instead of silently skipping.

Update stale comment in session-summary test: register() returns
initPromise so awaiting it is meaningful, not just a sync guard.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Coke1120

index.ts

@@ -1653,7 +1653,8 @@ const memoryLanceDBProPlugin = {
     // embedder, retriever, and smartExtractor require async secret resolution.
     // They are initialized in initPromise and must not be used before it resolves.
     // Every hook that uses them awaits initPromise at its top.
-    // If initPromise rejects, initFailed is set and hooks silently skip.
+    // If initPromise rejects, initFailed is set; service.start() logs a warn so
+    // the user knows why memory features are unavailable.
     let embedder!: ReturnType<typeof createEmbedder>;
     let retriever!: ReturnType<typeof createRetriever>;
     let smartExtractor: SmartExtractor | null = null;
@@ -3671,7 +3672,14 @@ const memoryLanceDBProPlugin = {
       start: async () => {
         // Wait for async secret resolution before running startup checks.
         await initPromise;
-        if (initFailed) return;
+        if (initFailed) {
+          api.logger.warn(
+            "memory-lancedb-pro: secret resolution failed during startup — " +
+            "embedding, recall, and capture hooks are disabled. " +
+            "Check your embedding.apiKey / retrieval.rerankApiKey config.",
+          );
+          return;
+        }
 
         // IMPORTANT: Do not block gateway startup on external network calls.
         // If embedding/retrieval tests hang (bad network / slow provider), the gateway

test/session-summary-before-reset.test.mjs

@@ -102,8 +102,9 @@ describe("systemSessionMemory before_reset", () => {
     const dbPath = path.join(workDir, "db");
     const api = createApiHarness({ dbPath, embeddingBaseURL });
 
-    memoryLanceDBProPlugin.register(api);
-    // register() is sync; before_reset is available immediately.
+    await memoryLanceDBProPlugin.register(api);
+    // register() returns initPromise; awaiting it ensures secrets are resolved
+    // and hooks (including before_reset) are fully initialized.
     assert.equal(typeof api.hooks.before_reset, "function");
     assert.equal(api.hooks["command:new"], undefined);