@memtensor/memos-local-openclaw-plugin 1.0.4-beta.4 → 1.0.4-beta.6

package/src/ingest/providers/index.ts

@@ -1,13 +1,47 @@
 import * as fs from "fs";
 import * as path from "path";
-import type { SummarizerConfig, Logger, OpenClawAPI } from "../../types";
-import { summarizeOpenAI, summarizeTaskOpenAI, generateTaskTitleOpenAI, judgeNewTopicOpenAI, filterRelevantOpenAI, judgeDedupOpenAI, parseFilterResult, parseDedupResult } from "./openai";
+import type { SummarizerConfig, SummaryProvider, Logger } from "../../types";
+import { summarizeOpenAI, summarizeTaskOpenAI, generateTaskTitleOpenAI, judgeNewTopicOpenAI, filterRelevantOpenAI, judgeDedupOpenAI } from "./openai";
 import type { FilterResult, DedupResult } from "./openai";
 export type { FilterResult, DedupResult } from "./openai";
 import { summarizeAnthropic, summarizeTaskAnthropic, generateTaskTitleAnthropic, judgeNewTopicAnthropic, filterRelevantAnthropic, judgeDedupAnthropic } from "./anthropic";
 import { summarizeGemini, summarizeTaskGemini, generateTaskTitleGemini, judgeNewTopicGemini, filterRelevantGemini, judgeDedupGemini } from "./gemini";
 import { summarizeBedrock, summarizeTaskBedrock, generateTaskTitleBedrock, judgeNewTopicBedrock, filterRelevantBedrock, judgeDedupBedrock } from "./bedrock";
 
+/**
+ * Detect provider type from provider key name or base URL.
+ */
+function detectProvider(
+  providerKey: string | undefined,
+  baseUrl: string,
+): SummaryProvider {
+  const key = providerKey?.toLowerCase() ?? "";
+  const url = baseUrl.toLowerCase();
+  if (key.includes("anthropic") || url.includes("anthropic")) return "anthropic";
+  if (key.includes("gemini") || url.includes("generativelanguage.googleapis.com")) {
+    return "gemini";
+  }
+  if (key.includes("bedrock") || url.includes("bedrock")) return "bedrock";
+  return "openai_compatible";
+}
+
+/**
+ * Return the correct endpoint for a given provider and base URL.
+ */
+function normalizeEndpointForProvider(
+  provider: SummaryProvider,
+  baseUrl: string,
+): string {
+  const stripped = baseUrl.replace(/\/+$/, "");
+  if (provider === "anthropic") {
+    if (stripped.endsWith("/v1/messages")) return stripped;
+    return `${stripped}/v1/messages`;
+  }
+  if (stripped.endsWith("/chat/completions")) return stripped;
+  if (stripped.endsWith("/completions")) return stripped;
+  return `${stripped}/chat/completions`;
+}
+
 /**
  * Build a SummarizerConfig from OpenClaw's native model configuration (openclaw.json).
  * This serves as the final fallback when both strongCfg and plugin summarizer fail or are absent.
@@ -15,7 +49,8 @@ import { summarizeBedrock, summarizeTaskBedrock, generateTaskTitleBedrock, judge
 function loadOpenClawFallbackConfig(log: Logger): SummarizerConfig | undefined {
   try {
     const home = process.env.HOME ?? process.env.USERPROFILE ?? "";
-    const cfgPath = path.join(home, ".openclaw", "openclaw.json");
+    const ocHome = process.env.OPENCLAW_STATE_DIR || path.join(home, ".openclaw");
+    const cfgPath = path.join(ocHome, "openclaw.json");
     if (!fs.existsSync(cfgPath)) return undefined;
 
     const raw = JSON.parse(fs.readFileSync(cfgPath, "utf-8"));
@@ -36,13 +71,12 @@ function loadOpenClawFallbackConfig(log: Logger): SummarizerConfig | undefined {
     const apiKey: string | undefined = providerCfg.apiKey;
     if (!baseUrl || !apiKey) return undefined;
 
-    const endpoint = baseUrl.endsWith("/chat/completions")
-      ? baseUrl
-      : baseUrl.replace(/\/+$/, "") + "/chat/completions";
+    const provider = detectProvider(providerKey, baseUrl);
+    const endpoint = normalizeEndpointForProvider(provider, baseUrl);
 
-    log.debug(`OpenClaw fallback model: ${modelId} via ${baseUrl}`);
+    log.debug(`OpenClaw fallback model: ${modelId} via ${baseUrl} (${provider})`);
     return {
-      provider: "openai_compatible",
+      provider,
       endpoint,
       apiKey,
       model: modelId,
@@ -120,7 +154,6 @@ export class Summarizer {
   constructor(
     private cfg: SummarizerConfig | undefined,
     private log: Logger,
-    private openclawAPI?: OpenClawAPI,
     strongCfg?: SummarizerConfig,
   ) {
     this.strongCfg = strongCfg;
@@ -130,20 +163,11 @@ export class Summarizer {
   /**
    * Ordered config chain: strongCfg → cfg → fallbackCfg (OpenClaw native model).
    * Returns configs that are defined, in priority order.
-   * Openclaw configs without hostCompletion capability or without openclawAPI are excluded.
    */
   private getConfigChain(): SummarizerConfig[] {
     const chain: SummarizerConfig[] = [];
     if (this.strongCfg) chain.push(this.strongCfg);
-    if (this.cfg) {
-      if (this.cfg.provider === "openclaw") {
-        if (this.cfg.capabilities?.hostCompletion === true && this.openclawAPI) {
-          chain.push(this.cfg);
-        }
-      } else {
-        chain.push(this.cfg);
-      }
-    }
+    if (this.cfg) chain.push(this.cfg);
     if (this.fallbackCfg) chain.push(this.fallbackCfg);
     return chain;
   }
@@ -227,9 +251,7 @@ export class Summarizer {
       return taskFallback(text);
     }
 
-    const result = await this.tryChain("summarizeTask", (cfg) =>
-      cfg.provider === "openclaw" ? this.summarizeTaskOpenClaw(text) : callSummarizeTask(cfg, text, this.log),
-    );
+    const result = await this.tryChain("summarizeTask", (cfg) => callSummarizeTask(cfg, text, this.log));
     return result ?? taskFallback(text);
   }
 
@@ -268,11 +290,7 @@ export class Summarizer {
     if (!this.cfg && !this.fallbackCfg) return null;
     if (candidates.length === 0) return { relevant: [], sufficient: true };
 
-    const result = await this.tryChain("filterRelevant", (cfg) =>
-      cfg.provider === "openclaw"
-        ? this.filterRelevantOpenClaw(query, candidates)
-        : callFilterRelevant(cfg, query, candidates, this.log),
-    );
+    const result = await this.tryChain("filterRelevant", (cfg) => callFilterRelevant(cfg, query, candidates, this.log));
     return result ?? null;
   }
 
@@ -283,144 +301,13 @@ export class Summarizer {
     if (!this.cfg && !this.fallbackCfg) return null;
     if (candidates.length === 0) return null;
 
-    const result = await this.tryChain("judgeDedup", (cfg) =>
-      cfg.provider === "openclaw"
-        ? this.judgeDedupOpenClaw(newSummary, candidates)
-        : callJudgeDedup(cfg, newSummary, candidates, this.log),
-    );
+    const result = await this.tryChain("judgeDedup", (cfg) => callJudgeDedup(cfg, newSummary, candidates, this.log));
     return result ?? { action: "NEW", reason: "all_models_failed" };
   }
 
   getStrongConfig(): SummarizerConfig | undefined {
     return this.strongCfg;
   }
-
-  // ─── OpenClaw API Implementation ───
-
-  private requireOpenClawAPI(): void {
-    if (!this.openclawAPI) {
-      throw new Error(
-        "OpenClaw API not available. Ensure sharing.capabilities.hostCompletion is enabled in config."
-      );
-    }
-  }
-
-  private async summarizeOpenClaw(text: string): Promise<string> {
-    this.requireOpenClawAPI();
-    const prompt = [
-      `Summarize the text in ONE concise sentence (max 120 characters). IMPORTANT: Use the SAME language as the input text — if the input is Chinese, write Chinese; if English, write English. Preserve exact names, commands, error codes. No bullet points, no preamble — output only the sentence.`,
-      ``,
-      text.slice(0, 2000),
-    ].join("\n");
-
-    const response = await this.openclawAPI!.complete({
-      prompt,
-      maxTokens: 100,
-      temperature: 0,
-      model: this.cfg?.model,
-    });
-
-    return response.text.trim().slice(0, 200);
-  }
-
-  private async summarizeTaskOpenClaw(text: string): Promise<string> {
-    this.requireOpenClawAPI();
-    const prompt = [
-      OPENCLAW_TASK_SUMMARY_PROMPT,
-      ``,
-      text,
-    ].join("\n");
-
-    const response = await this.openclawAPI!.complete({
-      prompt,
-      maxTokens: 4096,
-      temperature: 0.1,
-      model: this.cfg?.model,
-    });
-
-    return response.text.trim();
-  }
-
-  private async judgeNewTopicOpenClaw(currentContext: string, newMessage: string): Promise<boolean> {
-    this.requireOpenClawAPI();
-    const prompt = [
-      OPENCLAW_TOPIC_JUDGE_PROMPT,
-      ``,
-      `CURRENT CONVERSATION SUMMARY:`,
-      currentContext,
-      ``,
-      `NEW USER MESSAGE:`,
-      newMessage,
-    ].join("\n");
-
-    const response = await this.openclawAPI!.complete({
-      prompt,
-      maxTokens: 10,
-      temperature: 0,
-      model: this.cfg?.model,
-    });
-
-    const answer = response.text.trim().toUpperCase();
-    this.log.debug(`Topic judge result: "${answer}"`);
-    return answer.startsWith("NEW");
-  }
-
-  private async filterRelevantOpenClaw(
-    query: string,
-    candidates: Array<{ index: number; role: string; content: string; time?: string }>,
-  ): Promise<FilterResult> {
-    this.requireOpenClawAPI();
-    const candidateText = candidates
-      .map((c) => `${c.index}. [${c.role}] ${c.content}`)
-      .join("\n");
-
-    const prompt = [
-      OPENCLAW_FILTER_RELEVANT_PROMPT,
-      ``,
-      `QUERY: ${query}`,
-      ``,
-      `CANDIDATES:`,
-      candidateText,
-    ].join("\n");
-
-    const response = await this.openclawAPI!.complete({
-      prompt,
-      maxTokens: 200,
-      temperature: 0,
-      model: this.cfg?.model,
-    });
-
-    return parseFilterResult(response.text.trim(), this.log);
-  }
-
-  private async judgeDedupOpenClaw(
-    newSummary: string,
-    candidates: Array<{ index: number; summary: string; chunkId: string }>,
-  ): Promise<DedupResult> {
-    this.requireOpenClawAPI();
-    const candidateText = candidates
-      .map((c) => `${c.index}. ${c.summary}`)
-      .join("\n");
-
-    const prompt = [
-      OPENCLAW_DEDUP_JUDGE_PROMPT,
-      ``,
-      `NEW MEMORY:`,
-      newSummary,
-      ``,
-      `EXISTING MEMORIES:`,
-      candidateText,
-    ].join("\n");
-
-    const response = await this.openclawAPI!.complete({
-      prompt,
-      maxTokens: 300,
-      temperature: 0,
-      model: this.cfg?.model,
-    });
-
-    return parseDedupResult(response.text.trim(), this.log);
-  }
 }
 
 // ─── Dispatch helpers ───
@@ -596,90 +483,3 @@ function wordCount(text: string): number {
   return count;
 }
 
-// ─── OpenClaw Prompt Templates ───
-
-const OPENCLAW_TASK_SUMMARY_PROMPT = `You create a DETAILED task summary from a multi-turn conversation. This summary will be the ONLY record of this conversation, so it must preserve ALL important information.
-
-CRITICAL LANGUAGE RULE: You MUST write in the SAME language as the user's messages. Chinese input → Chinese output. English input → English output. NEVER mix languages.
-
-Output EXACTLY this structure:
-
-📌 Title
-A short, descriptive title (10-30 characters). Like a chat group name.
-
-🎯 Goal
-One sentence: what the user wanted to accomplish.
-
-📋 Key Steps
-- Describe each meaningful step in detail
-- Include the ACTUAL content produced: code snippets, commands, config blocks, formulas, key paragraphs
-- For code: include the function signature and core logic (up to ~30 lines per block), use fenced code blocks
-- For configs: include the actual config values and structure
-- For lists/instructions: include the actual items, not just "provided a list"
-- Merge only truly trivial back-and-forth (like "ok" / "sure")
-- Do NOT over-summarize: "provided a function" is BAD; show the actual function
-
-✅ Result
-What was the final outcome? Include the final version of any code/config/content produced.
-
-💡 Key Details
-- Decisions made, trade-offs discussed, caveats noted, alternative approaches mentioned
-- Specific values: numbers, versions, thresholds, URLs, file paths, model names
-- Omit this section only if there truly are no noteworthy details
-
-RULES:
-- This summary is a KNOWLEDGE BASE ENTRY, not a brief note. Be thorough.
-- PRESERVE verbatim: code, commands, URLs, file paths, error messages, config values, version numbers, names, amounts
-- DISCARD only: greetings, filler, the assistant explaining what it will do before doing it
-- Replace secrets (API keys, tokens, passwords) with [REDACTED]
-- Target length: 30-50% of the original conversation length. Longer conversations need longer summaries.
-- Output summary only, no preamble.`;
-
-const OPENCLAW_TOPIC_JUDGE_PROMPT = `You are a conversation topic boundary detector. Given a summary of the CURRENT conversation and a NEW user message, determine if the new message starts a DIFFERENT topic/task.
-
-Answer ONLY "NEW" or "SAME".
-
-Rules:
-- "NEW" = the new message is about a completely different subject, project, or task
-- "SAME" = the new message continues, follows up on, or is closely related to the current topic
-- Follow-up questions, clarifications, refinements, bug fixes, or next steps on the same task = SAME
-- Greetings or meta-questions like "你好" or "谢谢" without new substance = SAME
-- A clearly unrelated request (e.g., current topic is deployment, new message asks about cooking) = NEW
-
-Output exactly one word: NEW or SAME`;
-
-const OPENCLAW_FILTER_RELEVANT_PROMPT = `You are a memory relevance judge. Given a user's QUERY and a list of CANDIDATE memory summaries, do two things:
-
-1. Select ALL candidates that could be useful for answering the query. When in doubt, INCLUDE the candidate.
-   - For questions about lists, history, or "what/where/who" across multiple items, include ALL matching items.
-   - For factual lookups, a single direct answer is enough.
-2. Judge whether the selected memories are SUFFICIENT to fully answer the query WITHOUT fetching additional context.
-
-IMPORTANT for "sufficient" judgment:
-- sufficient=true ONLY when the memories contain a concrete ANSWER, fact, decision, or actionable information that directly addresses the query.
-- sufficient=false when the memories only repeat the question, show related topics but lack the specific detail, or contain partial information.
-
-Output a JSON object with exactly two fields:
-{"relevant":[1,3,5],"sufficient":true}
-
-- "relevant": array of candidate numbers that are useful. Empty array [] if none are relevant.
-- "sufficient": true ONLY if the memories contain a direct answer; false otherwise.
-
-Output ONLY the JSON object, nothing else.`;
-
-const OPENCLAW_DEDUP_JUDGE_PROMPT = `You are a memory deduplication system. Given a NEW memory summary and several EXISTING memory summaries, determine the relationship.
-
-For each EXISTING memory, the NEW memory is either:
-- "DUPLICATE": NEW is fully covered by an EXISTING memory — no new information at all
-- "UPDATE": NEW contains information that supplements or updates an EXISTING memory (new data, status change, additional detail)
-- "NEW": NEW is a different topic/event despite surface similarity
-
-Pick the BEST match among all candidates. If none match well, choose "NEW".
-
-Output a single JSON object:
-- If DUPLICATE: {"action":"DUPLICATE","targetIndex":2,"reason":"..."}
-- If UPDATE: {"action":"UPDATE","targetIndex":3,"reason":"...","mergedSummary":"a combined summary preserving all info from both old and new, same language as input"}
-- If NEW: {"action":"NEW","reason":"..."}
-
-CRITICAL: mergedSummary must use the SAME language as the input. Output ONLY the JSON object.`;
-

package/src/ingest/providers/openai.ts

@@ -316,7 +316,7 @@ export async function filterRelevantOpenAI(
   return parseFilterResult(raw, log);
 }
 
-export function parseFilterResult(raw: string, log: Logger): FilterResult {
+function parseFilterResult(raw: string, log: Logger): FilterResult {
   try {
     const match = raw.match(/\{[\s\S]*\}/);
     if (match) {

package/src/ingest/task-processor.ts

@@ -39,7 +39,7 @@ export class TaskProcessor {
     private ctx: PluginContext,
   ) {
     const strongCfg = ctx.config.skillEvolution?.summarizer;
-    this.summarizer = new Summarizer(ctx.config.summarizer, ctx.log, ctx.openclawAPI, strongCfg);
+    this.summarizer = new Summarizer(ctx.config.summarizer, ctx.log, strongCfg);
   }
 
   onTaskCompleted(cb: (task: Task) => void): void {

package/src/recall/engine.ts

@@ -246,7 +246,7 @@ export class RecallEngine {
     if (candidateSkills.length === 0) return [];
 
     // LLM relevance judgment
-    const summarizer = new Summarizer(this.ctx.config.summarizer, this.ctx.log, this.ctx.openclawAPI);
+    const summarizer = new Summarizer(this.ctx.config.summarizer, this.ctx.log);
     const relevantIndices = await this.judgeSkillRelevance(summarizer, query, candidateSkills);
 
     return relevantIndices.map((idx) => {

package/src/shared/llm-call.ts

@@ -1,6 +1,35 @@
 import * as fs from "fs";
 import * as path from "path";
-import type { SummarizerConfig, Logger, PluginContext, OpenClawAPI } from "../types";
+import type { SummarizerConfig, SummaryProvider, Logger, PluginContext } from "../types";
+
+/**
+ * Detect provider type from provider key name or base URL.
+ */
+function detectProvider(providerKey: string | undefined, baseUrl: string): SummaryProvider {
+  const key = providerKey?.toLowerCase() ?? "";
+  const url = baseUrl.toLowerCase();
+  if (key.includes("anthropic") || url.includes("anthropic")) return "anthropic";
+  if (key.includes("gemini") || url.includes("generativelanguage.googleapis.com")) {
+    return "gemini";
+  }
+  if (key.includes("bedrock") || url.includes("bedrock")) return "bedrock";
+  return "openai_compatible";
+}
+
+/**
+ * Return the correct default endpoint for a given provider.
+ */
+function defaultEndpointForProvider(provider: SummaryProvider, baseUrl: string): string {
+  const stripped = baseUrl.replace(/\/+$/, "");
+  if (provider === "anthropic") {
+    if (stripped.endsWith("/v1/messages")) return stripped;
+    return `${stripped}/v1/messages`;
+  }
+  // OpenAI-compatible providers
+  if (stripped.endsWith("/chat/completions")) return stripped;
+  if (stripped.endsWith("/completions")) return stripped;
+  return `${stripped}/chat/completions`;
+}
 
 /**
  * Build a SummarizerConfig from OpenClaw's native model configuration (openclaw.json).
@@ -30,13 +59,12 @@ export function loadOpenClawFallbackConfig(log: Logger): SummarizerConfig | unde
     const apiKey: string | undefined = providerCfg.apiKey;
     if (!baseUrl || !apiKey) return undefined;
 
-    const endpoint = baseUrl.endsWith("/chat/completions")
-      ? baseUrl
-      : baseUrl.replace(/\/+$/, "") + "/chat/completions";
+    const provider = detectProvider(providerKey, baseUrl);
+    const endpoint = defaultEndpointForProvider(provider, baseUrl);
 
-    log.debug(`OpenClaw fallback model: ${modelId} via ${baseUrl}`);
+    log.debug(`OpenClaw fallback model: ${modelId} via ${baseUrl} (${provider})`);
     return {
-      provider: "openai_compatible",
+      provider,
       endpoint,
       apiKey,
       model: modelId,
@@ -66,42 +94,86 @@ export interface LLMCallOptions {
   maxTokens?: number;
   temperature?: number;
   timeoutMs?: number;
-  /** Pass ctx.openclawAPI so callLLMOnce can handle provider === "openclaw" */
-  openclawAPI?: OpenClawAPI;
 }
 
-function normalizeEndpoint(url: string): string {
+function normalizeOpenAIEndpoint(url: string): string {
   const stripped = url.replace(/\/+$/, "");
   if (stripped.endsWith("/chat/completions")) return stripped;
   if (stripped.endsWith("/completions")) return stripped;
   return `${stripped}/chat/completions`;
 }
 
+function normalizeAnthropicEndpoint(url: string): string {
+  const stripped = url.replace(/\/+$/, "");
+  if (stripped.endsWith("/v1/messages")) return stripped;
+  if (stripped.endsWith("/messages")) return stripped;
+  return `${stripped}/v1/messages`;
+}
+
+function isAnthropicProvider(cfg: SummarizerConfig): boolean {
+  return cfg.provider === "anthropic";
+}
+
 /**
  * Make a single LLM call with the given config. Throws on failure.
- * When cfg.provider === "openclaw", delegates to the OpenClaw host completion API.
+ * Dispatches to Anthropic or OpenAI-compatible format based on provider.
  */
 export async function callLLMOnce(
   cfg: SummarizerConfig,
   prompt: string,
   opts: LLMCallOptions = {},
 ): Promise<string> {
-  // Handle openclaw provider via host completion API
-  if (cfg.provider === "openclaw") {
-    const api = opts.openclawAPI;
-    if (!api) {
-      throw new Error("OpenClaw API not available. Ensure sharing.capabilities.hostCompletion is enabled.");
-    }
-    const response = await api.complete({
-      prompt,
-      maxTokens: opts.maxTokens ?? 1024,
+  if (isAnthropicProvider(cfg)) {
+    return callLLMOnceAnthropic(cfg, prompt, opts);
+  }
+  return callLLMOnceOpenAI(cfg, prompt, opts);
+}
+
+async function callLLMOnceAnthropic(
+  cfg: SummarizerConfig,
+  prompt: string,
+  opts: LLMCallOptions = {},
+): Promise<string> {
+  const endpoint = normalizeAnthropicEndpoint(
+    cfg.endpoint ?? "https://api.anthropic.com/v1/messages",
+  );
+  const model = cfg.model ?? "claude-3-haiku-20240307";
+  const headers: Record<string, string> = {
+    "Content-Type": "application/json",
+    "x-api-key": cfg.apiKey ?? "",
+    "anthropic-version": "2023-06-01",
+    ...cfg.headers,
+  };
+
+  const resp = await fetch(endpoint, {
+    method: "POST",
+    headers,
+    body: JSON.stringify({
+      model,
       temperature: opts.temperature ?? 0.1,
-      model: cfg.model,
-    });
-    return response.text.trim();
+      max_tokens: opts.maxTokens ?? 1024,
+      messages: [{ role: "user", content: prompt }],
+    }),
+    signal: AbortSignal.timeout(opts.timeoutMs ?? 30_000),
+  });
+
+  if (!resp.ok) {
+    const body = await resp.text();
+    throw new Error(`LLM call failed (${resp.status}): ${body}`);
   }
 
-  const endpoint = normalizeEndpoint(cfg.endpoint ?? "https://api.openai.com/v1/chat/completions");
+  const json = (await resp.json()) as { content: Array<{ type: string; text: string }> };
+  return json.content.find((c) => c.type === "text")?.text?.trim() ?? "";
+}
+
+async function callLLMOnceOpenAI(
+  cfg: SummarizerConfig,
+  prompt: string,
+  opts: LLMCallOptions = {},
+): Promise<string> {
+  const endpoint = normalizeOpenAIEndpoint(
+    cfg.endpoint ?? "https://api.openai.com/v1/chat/completions",
+  );
   const model = cfg.model ?? "gpt-4o-mini";
   const headers: Record<string, string> = {
     "Content-Type": "application/json",

package/src/skill/evaluator.ts

@@ -145,7 +145,7 @@ export class SkillEvaluator {
       .replace("{SUMMARY}", task.summary.slice(0, 3000));
 
     try {
-      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillEvaluator.create", { openclawAPI: this.ctx.openclawAPI });
+      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillEvaluator.create");
       return this.parseJSON<CreateEvalResult>(raw, {
         shouldGenerate: false, reason: "parse failed", suggestedName: "", suggestedTags: [], confidence: 0,
       });
@@ -169,7 +169,7 @@ export class SkillEvaluator {
       .replace("{SUMMARY}", task.summary.slice(0, 3000));
 
     try {
-      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillEvaluator.upgrade", { openclawAPI: this.ctx.openclawAPI });
+      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillEvaluator.upgrade");
       return this.parseJSON<UpgradeEvalResult>(raw, {
         shouldUpgrade: false, upgradeType: "refine", dimensions: [], reason: "parse failed", mergeStrategy: "", confidence: 0,
       });
@@ -179,7 +179,6 @@ export class SkillEvaluator {
     }
   }
 
-
   private parseJSON<T>(raw: string, fallback: T): T {
     const jsonMatch = raw.match(/\{[\s\S]*\}/);
     if (!jsonMatch) return fallback;

package/src/skill/evolver.ts

@@ -12,6 +12,8 @@ import { SkillUpgrader } from "./upgrader";
 import { SkillInstaller } from "./installer";
 import { buildSkillConfigChain, callLLMWithFallback } from "../shared/llm-call";
 
+export type SkillEvolvedCallback = (skillName: string, upgradeType: "created" | "upgraded") => void;
+
 export class SkillEvolver {
   private evaluator: SkillEvaluator;
   private generator: SkillGenerator;
@@ -19,6 +21,7 @@ export class SkillEvolver {
   private installer: SkillInstaller;
   private processing = false;
   private queue: Task[] = [];
+  onSkillEvolved: SkillEvolvedCallback | null = null;
 
   constructor(
     private store: SqliteStore,
@@ -279,6 +282,7 @@ Use selectedIndex 0 when none is highly relevant.`;
       if (upgraded) {
         this.store.linkTaskSkill(task.id, freshSkill.id, "evolved_from", freshSkill.version + 1);
         this.installer.syncIfInstalled(freshSkill.name);
+        this.onSkillEvolved?.(freshSkill.name, "upgraded");
       } else {
         this.store.linkTaskSkill(task.id, freshSkill.id, "applied_to", freshSkill.version);
       }
@@ -307,6 +311,7 @@ Use selectedIndex 0 when none is highly relevant.`;
       this.markChunksWithSkill(chunks, skill.id);
       this.store.linkTaskSkill(task.id, skill.id, "generated_from", 1);
       this.store.setTaskSkillMeta(task.id, { skillStatus: "generated", skillReason: evalResult.reason });
+      this.onSkillEvolved?.(skill.name, "created");
 
       const autoInstall = this.ctx.config.skillEvolution?.autoInstall ?? DEFAULTS.skillAutoInstall;
       if (autoInstall && skill.status === "active") {

package/src/skill/generator.ts

@@ -356,7 +356,7 @@ export class SkillGenerator {
       .replace("{CONVERSATION}", conversationText.slice(0, 12000))
       + langInstruction;
 
-    const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.step1", { maxTokens: 6000, temperature: 0.2, timeoutMs: 120_000, openclawAPI: this.ctx.openclawAPI });
+    const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.step1", { maxTokens: 6000, temperature: 0.2, timeoutMs: 120_000 });
 
     const trimmed = raw.trim();
     if (trimmed.startsWith("---")) return trimmed;
@@ -379,7 +379,7 @@ export class SkillGenerator {
       .replace("{CONVERSATION}", conversationText.slice(0, 6000));
 
     try {
-      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.scripts", { maxTokens: 3000, temperature: 0.1, timeoutMs: 120_000, openclawAPI: this.ctx.openclawAPI });
+      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.scripts", { maxTokens: 3000, temperature: 0.1, timeoutMs: 120_000 });
       return this.parseJSONArray<{ filename: string; content: string }>(raw);
     } catch (err) {
       this.ctx.log.warn(`SkillGenerator: script extraction failed: ${err}`);
@@ -401,7 +401,7 @@ export class SkillGenerator {
       .replace("{CONVERSATION}", conversationText.slice(0, 6000));
 
     try {
-      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.refs", { maxTokens: 3000, temperature: 0.1, timeoutMs: 120_000, openclawAPI: this.ctx.openclawAPI });
+      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.refs", { maxTokens: 3000, temperature: 0.1, timeoutMs: 120_000 });
       return this.parseJSONArray<{ filename: string; content: string }>(raw);
     } catch (err) {
       this.ctx.log.warn(`SkillGenerator: reference extraction failed: ${err}`);
@@ -423,7 +423,7 @@ export class SkillGenerator {
       + `\n\n⚠️ LANGUAGE: Write test prompts and expectations in ${lang}, matching the skill's language.\n`;
 
     try {
-      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.evals", { maxTokens: 2000, temperature: 0.3, timeoutMs: 120_000, openclawAPI: this.ctx.openclawAPI });
+      const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillGenerator.evals", { maxTokens: 2000, temperature: 0.3, timeoutMs: 120_000 });
       return this.parseJSONArray(raw);
     } catch (err) {
       this.ctx.log.warn(`SkillGenerator: eval generation failed: ${err}`);
@@ -467,7 +467,6 @@ export class SkillGenerator {
     return { hitCount, results };
   }
 
-
   // ─── Helpers ───
 
   private parseJSONArray<T>(raw: string): T[] {
@@ -500,5 +499,4 @@ export class SkillGenerator {
     return "";
   }
 
-
 }

package/src/skill/upgrader.ts

@@ -190,7 +190,7 @@ export class SkillUpgrader {
       .replace("{TASK_ID}", task.id)
       + langInstruction;
 
-    const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillUpgrader.upgrade", { maxTokens: 6000, temperature: 0.2, timeoutMs: 90_000, openclawAPI: this.ctx.openclawAPI });
+    const raw = await callLLMWithFallback(chain, prompt, this.ctx.log, "SkillUpgrader.upgrade", { maxTokens: 6000, temperature: 0.2, timeoutMs: 90_000 });
 
     const changelogSep = raw.indexOf("---CHANGELOG---");
     if (changelogSep !== -1) {