mindkeeper-openclaw 0.2.30 → 0.2.32

package/dist/index.js

@@ -718,11 +718,66 @@ var Tracker = class {
     await this.ensureConfigLoaded();
     await this.store.init();
     await this.ensureGitignore();
+    this.log?.info?.(
+      `[mindkeeper] init: commitMessage.mode=${this.config.commitMessage.mode}, llmProvider=${this.llmProvider ? "yes" : "no"}`
+    );
     const changed = await this.getTrackedChangedFiles();
     if (changed.length > 0) {
-      await this.store.addFiles(changed.map((e) => e.filepath));
-      const msg = generateTemplateMessage(changed.map((e) => e.filepath));
-      await this.store.commit(msg);
+      const filesToCommit = changed.map((e) => e.filepath);
+      await this.store.addFiles(filesToCommit);
+      let message = null;
+      if (this.config.commitMessage.mode === "llm" && this.llmProvider) {
+        let headOid = null;
+        try {
+          headOid = await this.resolveHead();
+        } catch {
+        }
+        if (headOid) {
+          const diffs = [];
+          for (const file of filesToCommit) {
+            try {
+              const oldContent = await this.store.readFile(file, headOid) ?? "";
+              let newContent;
+              try {
+                newContent = await import_promises3.default.readFile(
+                  import_node_path3.default.join(this.workDir, file),
+                  "utf-8"
+                );
+              } catch {
+                newContent = "";
+              }
+              const d = computeDiff({
+                file,
+                fromVersion: headOid.slice(0, 8),
+                toVersion: "(staged)",
+                oldContent,
+                newContent
+              });
+              if (d.additions > 0 || d.deletions > 0) diffs.push(d);
+            } catch {
+            }
+          }
+          if (diffs.length > 0) {
+            message = await generateLlmMessage(diffs, this.llmProvider, this.log);
+            if (message) {
+              this.log?.info?.(
+                `[mindkeeper] init: LLM-generated commit \u2014 "${message.slice(0, 50)}${message.length > 50 ? "\u2026" : ""}"`
+              );
+            }
+          } else {
+            this.log?.info?.("[mindkeeper] init: diffs empty (all files unchanged vs HEAD)");
+          }
+        } else {
+          this.log?.info?.("[mindkeeper] init: no HEAD yet (first commit), LLM skipped");
+        }
+      }
+      if (!message) {
+        message = generateTemplateMessage(filesToCommit);
+        this.log?.info?.(`[mindkeeper] init: using template \u2014 "${message}"`);
+      }
+      await this.store.commit(message);
+    } else {
+      this.log?.info?.("[mindkeeper] init: no changed files");
     }
     return { initialFiles: changed.map((e) => e.filepath) };
   }
@@ -823,6 +878,9 @@ var Tracker = class {
       headOid = await this.resolveHead();
     } catch {
     }
+    this.log?.info?.(
+      `[mindkeeper] autoSnapshot: files=${filesToCommit.length}, headOid=${headOid ? headOid.slice(0, 8) : "none"}, mode=${this.config.commitMessage.mode}, llmProvider=${this.llmProvider ? "yes" : "no"}`
+    );
     if (headOid) {
       for (const file of filesToCommit) {
         try {
@@ -846,21 +904,42 @@ var Tracker = class {
           if (d.additions > 0 || d.deletions > 0) {
             diffs.push(d);
           }
-        } catch {
+        } catch (err) {
+          this.log?.warn?.(
+            `[mindkeeper] autoSnapshot: diff failed for ${file}: ${err instanceof Error ? err.message : String(err)}`
+          );
         }
       }
     }
+    this.log?.info?.(`[mindkeeper] autoSnapshot: computed ${diffs.length} non-empty diff(s)`);
     let message = null;
+    let usedLlm = false;
+    let templateReason = null;
     if (this.config.commitMessage.mode === "llm" && diffs.length > 0) {
-      message = await generateLlmMessage(diffs, this.llmProvider, this.log);
+      if (this.llmProvider) {
+        message = await generateLlmMessage(diffs, this.llmProvider, this.log);
+        if (message) usedLlm = true;
+        else templateReason = "LLM API call failed (check logs above for error details)";
+      } else {
+        templateReason = "llmProvider is null (createOpenClawLlmProvider returned null \u2014 check startup logs)";
+      }
+    } else if (this.config.commitMessage.mode === "llm" && diffs.length === 0) {
+      templateReason = headOid ? `diffs empty (${filesToCommit.length} file(s) changed but no textual diff vs HEAD)` : "no HEAD commit yet (first commit), cannot compute diff";
+    } else {
+      templateReason = `commitMessage.mode="${this.config.commitMessage.mode}" (not "llm")`;
     }
     if (!message) {
-      if (this.config.commitMessage.mode === "llm" && this.log?.warn) {
-        const reason = diffs.length === 0 ? "No diff available (first commit or no file changes), using template" : "LLM failed, falling back to template";
-        this.log.warn(`[mindkeeper] ${reason}`);
-      }
       message = generateTemplateMessage(filesToCommit);
     }
+    if (usedLlm) {
+      this.log?.info?.(
+        `[mindkeeper] Commit message: LLM \u2713 \u2014 "${message.slice(0, 60)}${message.length > 60 ? "\u2026" : ""}"`
+      );
+    } else {
+      this.log?.warn?.(
+        `[mindkeeper] Commit message: TEMPLATE \u2014 reason: ${templateReason}`
+      );
+    }
     const oid = await this.store.commit(message);
     return {
       oid,
@@ -1172,7 +1251,7 @@ function readEnvApiKey(provider) {
 // src/llm-provider.ts
 var MAX_DIFF_CHARS = 4e3;
 async function createOpenClawLlmProvider(api) {
-  const modelSpec = resolveModelFromConfig(api.config);
+  const modelSpec = resolveModelFromConfig(api.config, api.log);
   if (!modelSpec) {
     api.log?.warn?.(
       "[mindkeeper] No default model configured in OpenClaw \u2014 LLM commit messages disabled."
@@ -1188,25 +1267,32 @@ async function createOpenClawLlmProvider(api) {
   const apiKey = await resolveApiKey(modelSpec.provider);
   if (!apiKey) {
     api.log?.warn?.(
-      `[mindkeeper] No API key found for provider "${modelSpec.provider}" \u2014 LLM commit messages disabled.`
+      `[mindkeeper] No API key found for provider "${modelSpec.provider}" \u2014 LLM commit messages disabled. Checked: auth-profiles.json + env vars.`
     );
     return null;
   }
   api.log?.info?.(
-    `[mindkeeper] LLM commit messages enabled (${modelSpec.provider}/${modelSpec.model}).`
+    `[mindkeeper] LLM commit messages enabled (${modelSpec.provider}/${modelSpec.model}${modelSpec.baseUrl ? `, baseUrl=${modelSpec.baseUrl}` : ""}).`
   );
   return {
     async generateCommitMessage(diffs) {
       const diffText = diffs.map((d) => `--- ${d.file} ---
 ${d.unified}`).join("\n").slice(0, MAX_DIFF_CHARS);
+      api.log?.info?.(
+        `[mindkeeper] Calling LLM for commit message (${diffs.length} diff(s), ${diffText.length} chars)\u2026`
+      );
       const { callLlm } = await import("./llm-client.cjs");
-      return callLlm({
+      const result = await callLlm({
         provider: modelSpec.provider,
         model: modelSpec.model,
         apiKey,
         userPrompt: diffText,
         baseUrl: modelSpec.baseUrl
       });
+      api.log?.info?.(
+        `[mindkeeper] LLM returned: "${(result ?? "").slice(0, 60)}${(result ?? "").length > 60 ? "\u2026" : ""}"`
+      );
+      return result;
     }
   };
 }
@@ -1214,10 +1300,13 @@ function isOAuthProvider(provider) {
   const normalized = normalizeProvider(provider);
   return normalized.includes("portal") || normalized.includes("oauth");
 }
-function resolveModelFromConfig(config) {
+function resolveModelFromConfig(config, log) {
   const agents = config?.agents;
   const defaults = agents?.defaults;
   const raw = defaults?.model;
+  log?.info?.(
+    `[mindkeeper] resolveModel: config.agents=${agents ? "present" : "missing"}, config.agents.defaults=${defaults ? "present" : "missing"}, config.agents.defaults.model=${raw === void 0 ? "undefined" : typeof raw === "string" ? `"${raw}"` : JSON.stringify(raw)}`
+  );
   let spec;
   if (typeof raw === "string") {
     spec = raw.trim();
@@ -1225,7 +1314,12 @@ function resolveModelFromConfig(config) {
     const primary = raw.primary;
     if (typeof primary === "string") spec = primary.trim();
   }
-  if (!spec?.includes("/")) return null;
+  if (!spec?.includes("/")) {
+    log?.warn?.(
+      `[mindkeeper] resolveModel: no valid "provider/model" spec found (got: ${spec === void 0 ? "undefined" : `"${spec}"`})`
+    );
+    return null;
+  }
   const slashIdx = spec.indexOf("/");
   const provider = spec.slice(0, slashIdx);
   const model = spec.slice(slashIdx + 1);
@@ -1295,10 +1389,16 @@ function createWatcherService(api, trackerRef) {
         return;
       }
       ensureWorkspaceSkillMirror(workspaceDir, { log: api.log });
+      log.info(
+        `[mindkeeper] Service starting: ctx.config=${ctx.config ? "present" : "missing"}, pluginConfig=${api.pluginConfig ? JSON.stringify(api.pluginConfig) : "none"}`
+      );
       const llmProvider = await createOpenClawLlmProvider({
         config: ctx.config,
         log
       });
+      log.info(
+        `[mindkeeper] LLM provider: ${llmProvider ? "created successfully" : "null (will use template)"}`
+      );
       const tracker = new Tracker({
         workDir: workspaceDir,
         llmProvider: llmProvider ?? void 0,
@@ -1307,6 +1407,9 @@ function createWatcherService(api, trackerRef) {
       });
       await tracker.init();
       trackerRef.current = tracker;
+      log.info(
+        `[mindkeeper] Tracker config: commitMessage.mode=${tracker.getConfig().commitMessage.mode}`
+      );
       watcher = new Watcher({
         tracker,
         onSnapshot: (commit) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mindkeeper-openclaw",
-  "version": "0.2.30",
+  "version": "0.2.32",
   "description": "OpenClaw plugin for mindkeeper: auto-snapshot, diff, and rollback for agent context files",
   "keywords": [
     "openclaw",