pullfrog 0.1.0 → 0.1.2

package/dist/index.js

@@ -18198,7 +18198,7 @@ var require_summary = __commonJS({
     exports.summary = exports.markdownSummary = exports.SUMMARY_DOCS_URL = exports.SUMMARY_ENV_VAR = void 0;
     var os_1 = __require("os");
     var fs_1 = __require("fs");
-    var { access, appendFile, writeFile: writeFile3 } = fs_1.promises;
+    var { access, appendFile, writeFile: writeFile4 } = fs_1.promises;
     exports.SUMMARY_ENV_VAR = "GITHUB_STEP_SUMMARY";
     exports.SUMMARY_DOCS_URL = "https://docs.github.com/actions/using-workflows/workflow-commands-for-github-actions#adding-a-job-summary";
     var Summary = class {
@@ -18256,7 +18256,7 @@ var require_summary = __commonJS({
         return __awaiter(this, void 0, void 0, function* () {
           const overwrite = !!(options === null || options === void 0 ? void 0 : options.overwrite);
           const filePath = yield this.filePath();
-          const writeFunc = overwrite ? writeFile3 : appendFile;
+          const writeFunc = overwrite ? writeFile4 : appendFile;
           yield writeFunc(filePath, this._buffer, { encoding: "utf8" });
           return this.emptyBuffer();
         });
@@ -62662,8 +62662,8 @@ var require_snapshot_utils = __commonJS({
 var require_snapshot_recorder = __commonJS({
   "node_modules/.pnpm/undici@7.22.0/node_modules/undici/lib/mock/snapshot-recorder.js"(exports, module) {
     "use strict";
-    var { writeFile: writeFile3, readFile: readFile4, mkdir: mkdir2 } = __require("node:fs/promises");
-    var { dirname: dirname5, resolve: resolve3 } = __require("node:path");
+    var { writeFile: writeFile4, readFile: readFile5, mkdir: mkdir3 } = __require("node:fs/promises");
+    var { dirname: dirname6, resolve: resolve3 } = __require("node:path");
     var { setTimeout: setTimeout2, clearTimeout: clearTimeout2 } = __require("node:timers");
     var { InvalidArgumentError, UndiciError } = require_errors4();
     var { hashId, isUrlExcludedFactory, normalizeHeaders, createHeaderFilters } = require_snapshot_utils();
@@ -62864,7 +62864,7 @@ var require_snapshot_recorder = __commonJS({
           throw new InvalidArgumentError("Snapshot path is required");
         }
         try {
-          const data = await readFile4(resolve3(path3), "utf8");
+          const data = await readFile5(resolve3(path3), "utf8");
           const parsed2 = JSON.parse(data);
           if (Array.isArray(parsed2)) {
             this.#snapshots.clear();
@@ -62894,12 +62894,12 @@ var require_snapshot_recorder = __commonJS({
           throw new InvalidArgumentError("Snapshot path is required");
         }
         const resolvedPath = resolve3(path3);
-        await mkdir2(dirname5(resolvedPath), { recursive: true });
+        await mkdir3(dirname6(resolvedPath), { recursive: true });
         const data = Array.from(this.#snapshots.entries()).map(([hash2, snapshot2]) => ({
           hash: hash2,
           snapshot: snapshot2
         }));
-        await writeFile3(resolvedPath, JSON.stringify(data, null, 2), { flush: true });
+        await writeFile4(resolvedPath, JSON.stringify(data, null, 2), { flush: true });
       }
       /**
        * Clears all recorded snapshots
@@ -97475,14 +97475,14 @@ var require_turndown_cjs = __commonJS({
         } else if (node2.nodeType === 1) {
           replacement = replacementForNode.call(self2, node2);
         }
-        return join17(output, replacement);
+        return join18(output, replacement);
       }, "");
     }
     function postProcess(output) {
       var self2 = this;
       this.rules.forEach(function(rule) {
         if (typeof rule.append === "function") {
-          output = join17(output, rule.append(self2.options));
+          output = join18(output, rule.append(self2.options));
         }
       });
       return output.replace(/^[\t\r\n]+/, "").replace(/[\t\r\n\s]+$/, "");
@@ -97494,7 +97494,7 @@ var require_turndown_cjs = __commonJS({
       if (whitespace.leading || whitespace.trailing) content = content.trim();
       return whitespace.leading + rule.replacement(content, node2, this.options) + whitespace.trailing;
     }
-    function join17(output, replacement) {
+    function join18(output, replacement) {
       var s1 = trimTrailingNewlines(output);
       var s2 = trimLeadingNewlines(replacement);
       var nls = Math.max(output.length - s1.length, replacement.length - s2.length);
@@ -98926,8 +98926,8 @@ var require_fast_content_type_parse = __commonJS({
 // main.ts
 var core6 = __toESM(require_core(), 1);
 import { existsSync as existsSync7, readdirSync } from "node:fs";
-import { readFile as readFile3 } from "node:fs/promises";
-import { join as join16 } from "node:path";
+import { readFile as readFile4 } from "node:fs/promises";
+import { join as join17 } from "node:path";
 
 // node_modules/.pnpm/@ark+util@0.56.0/node_modules/@ark/util/out/arrays.js
 var liftArray = (data) => Array.isArray(data) ? data : [data];
@@ -107723,6 +107723,13 @@ function getApiUrl() {
   log.debug(`resolved API_URL: ${raw2}`);
   return raw2;
 }
+function isLocalApiUrl() {
+  try {
+    return isLocalUrl(new URL(getApiUrl()));
+  } catch {
+    return false;
+  }
+}
 
 // models.ts
 function provider(config3) {
@@ -108961,6 +108968,7 @@ function CreateCommentTool(ctx) {
         body: bodyWithFooter
       });
       ctx.toolState.wasUpdated = true;
+      log.info(`\xBB created comment ${result.data.id}`);
       if (commentType === "Plan") {
         if (result.data.node_id) {
           await patchWorkflowRunFields(ctx, { planCommentNodeId: result.data.node_id });
@@ -108974,6 +108982,7 @@ function CreateCommentTool(ctx) {
           comment_id: result.data.id,
           body: bodyWithPlanLink
         });
+        log.info(`\xBB updated comment ${updateResult.data.id}`);
         return {
           success: true,
           commentId: updateResult.data.id,
@@ -109007,6 +109016,7 @@ function EditCommentTool(ctx) {
         comment_id: commentId,
         body: bodyWithFooter
       });
+      log.info(`\xBB updated comment ${result.data.id}`);
       return {
         success: true,
         commentId: result.data.id,
@@ -109142,6 +109152,9 @@ ${collapsible}`;
           message: "progress recorded (no GitHub comment created - this may occur for workflow_dispatch events or when there is no associated issue/PR)"
         };
       }
+      if (result.commentId !== void 0) {
+        log.info(`\xBB ${result.action} comment ${result.commentId}`);
+      }
       if (!params.target_plan_comment) {
         ctx.toolState.finalSummaryWritten = true;
       }
@@ -109192,6 +109205,7 @@ function ReplyToReviewCommentTool(ctx) {
         comment_id,
         body: bodyWithFooter
       });
+      log.info(`\xBB created review comment ${result.data.id} (in reply to ${comment_id})`);
       ctx.toolState.wasUpdated = true;
       return {
         success: true,
@@ -109696,6 +109710,7 @@ async function spawn(options) {
   const startTime = performance3.now();
   let stdoutBuffer = "";
   let stderrBuffer = "";
+  const killGroup = options.killGroup ?? false;
   return new Promise((resolve3, reject) => {
     const child = nodeSpawn(options.cmd, options.args, {
       env: options.env || {
@@ -109703,9 +109718,20 @@ async function spawn(options) {
         HOME: process.env.HOME || ""
       },
       stdio: options.stdio || ["pipe", "pipe", "pipe"],
-      cwd: options.cwd || process.cwd()
+      cwd: options.cwd || process.cwd(),
+      detached: killGroup
     });
-    trackChild({ child });
+    const killSelf = (signal) => {
+      if (killGroup && child.pid) {
+        try {
+          process.kill(-child.pid, signal);
+          return;
+        } catch {
+        }
+      }
+      child.kill(signal);
+    };
+    trackChild({ child, killGroup });
     let timeoutId;
     let sigkillEscalatorId;
     let activityCheckIntervalId;
@@ -109716,10 +109742,10 @@ async function spawn(options) {
     if (options.timeout) {
       timeoutId = setTimeout(() => {
         isTimedOut = true;
-        child.kill("SIGTERM");
+        killSelf("SIGTERM");
         sigkillEscalatorId = setTimeout(() => {
           if (!child.killed) {
-            child.kill("SIGKILL");
+            killSelf("SIGKILL");
           }
         }, 5e3);
       }, options.timeout);
@@ -109738,9 +109764,9 @@ async function spawn(options) {
           killedAtIdleMs = idleMs;
           const idleSec = Math.round(idleMs / 1e3);
           log.info(
-            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process`
+            `no output for ${idleSec}s from pid=${child.pid} (${options.cmd}), killing process${killGroup ? " group" : ""}`
           );
-          child.kill("SIGKILL");
+          killSelf("SIGKILL");
           clearInterval(activityCheckIntervalId);
           try {
             options.onActivityTimeout?.();
@@ -142249,7 +142275,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.0",
+  version: "0.1.2",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -143193,6 +143219,10 @@ ${integrateStep}
       if (!pushed) {
         throw lastErr instanceof Error ? lastErr : new Error(String(lastErr));
       }
+      const pushedSha = $("git", ["rev-parse", "HEAD"], { log: false }).trim();
+      log.info(
+        `\xBB pushed branch ${branch} to ${pushDest.remoteName}/${pushDest.remoteBranch} (sha ${pushedSha})`
+      );
       return {
         success: true,
         branch,
@@ -143341,6 +143371,7 @@ function DeleteBranchTool(ctx) {
       await $git("push", ["origin", "--delete", `refs/heads/${params.branchName}`], {
         token: ctx.gitToken
       });
+      log.info(`\xBB deleted branch ${params.branchName}`);
       return { success: true, deleted: params.branchName };
     })
   });
@@ -143366,6 +143397,7 @@ function PushTagsTool(ctx) {
       await $git("push", pushArgs, {
         token: ctx.gitToken
       });
+      log.info(`\xBB pushed tag ${params.tag}`);
       return { success: true, tag: params.tag };
     })
   });
@@ -143690,6 +143722,7 @@ function CreatePullRequestReviewTool(ctx) {
       }
       const reviewId = result.data.id;
       const reviewNodeId = result.data.node_id;
+      log.info(`\xBB created review ${reviewId} on pull request #${pull_number}`);
       const actuallyReviewedSha = ctx.toolState.checkoutSha ?? params.commit_id;
       ctx.toolState.review = {
         id: reviewId,
@@ -144554,6 +144587,7 @@ function IssueTool(ctx) {
         labels: params.labels ?? [],
         assignees: params.assignees ?? []
       });
+      log.info(`\xBB created issue #${result.data.number} (id ${result.data.id})`);
       const nodeId = result.data.node_id;
       if (typeof nodeId === "string" && nodeId.length > 0) {
         await patchWorkflowRunFields(ctx, {
@@ -144745,6 +144779,7 @@ function AddLabelsTool(ctx) {
         issue_number,
         labels
       });
+      log.info(`\xBB added labels [${labels.join(", ")}] to issue #${issue_number}`);
       return {
         success: true,
         labels: result.data.map((label) => label.name)
@@ -144753,40 +144788,6 @@ function AddLabelsTool(ctx) {
   });
 }
 
-// mcp/learnings.ts
-var UpdateLearningsParams = type({
-  learnings: type.string.describe(
-    "the FULL merged learnings as a flat bullet list. each line starts with `- `. one discrete, actionable fact per bullet. combine existing bullets from the prompt with your new discoveries. deduplicate \u2014 if an existing bullet covers the same fact, update it in place rather than adding a new one. drop bullets that are clearly wrong or no longer relevant to the current codebase. keep the list focused and concise."
-  )
-});
-function UpdateLearningsTool(ctx) {
-  return tool({
-    name: "update_learnings",
-    description: "persist operational learnings about this repository (setup steps, test commands, key conventions, patterns). ONLY call this when you have high confidence the information is correct and broadly useful for future runs \u2014 not for one-off findings or uncertain observations. format: flat bullet list (`- ` per line, one fact per bullet). pass the FULL merged list \u2014 combine existing learnings from the prompt with new discoveries. deduplicate, and drop bullets that are clearly wrong or no longer relevant to the current codebase.",
-    parameters: UpdateLearningsParams,
-    execute: execute(async (params) => {
-      const response = await apiFetch({
-        path: `/api/repo/${ctx.repo.owner}/${ctx.repo.name}/learnings`,
-        method: "PATCH",
-        headers: {
-          authorization: `Bearer ${ctx.apiToken}`,
-          "content-type": "application/json"
-        },
-        body: JSON.stringify({
-          learnings: params.learnings,
-          model: ctx.toolState.model
-        }),
-        signal: AbortSignal.timeout(1e4)
-      });
-      if (!response.ok) {
-        const error49 = await response.text();
-        throw new Error(`failed to update learnings: ${error49}`);
-      }
-      return { success: true };
-    })
-  });
-}
-
 // mcp/output.ts
 var import_ajv3 = __toESM(require_ajv(), 1);
 var SetOutputParams = type({
@@ -144880,6 +144881,7 @@ function UpdatePullRequestBodyTool(ctx) {
         pull_number: params.pull_number,
         body: bodyWithFooter
       });
+      log.info(`\xBB updated pull request #${result.data.number}`);
       ctx.toolState.wasUpdated = true;
       return {
         success: true,
@@ -144907,6 +144909,7 @@ function CreatePullRequestTool(ctx) {
         base: params.base,
         draft: params.draft ?? false
       });
+      log.info(`\xBB created pull request #${result.data.number} (id ${result.data.id})`);
       const reviewer = ctx.payload.triggerer;
       if (reviewer) {
         try {
@@ -145458,7 +145461,7 @@ function ResolveReviewThreadTool(ctx) {
           threadId: params.thread_id
         });
         const thread = response.resolveReviewThread.thread;
-        log.debug(`resolved thread ${thread.id}, isResolved=${thread.isResolved}`);
+        log.info(`\xBB resolved review thread ${thread.id}`);
         return {
           thread_id: thread.id,
           is_resolved: thread.isResolved,
@@ -145930,6 +145933,7 @@ function UploadFileTool(ctx) {
       if (!uploadResponse.ok) {
         throw new Error(`failed to upload file: ${uploadResponse.statusText}`);
       }
+      log.info(`\xBB uploaded file ${publicUrl}`);
       return { success: true, publicUrl, filename, contentLength, contentType };
     })
   });
@@ -146023,8 +146027,7 @@ function buildOrchestratorTools(ctx, outputSchema) {
     PushTagsTool(ctx),
     DeleteBranchTool(ctx),
     CreatePullRequestTool(ctx),
-    UpdatePullRequestBodyTool(ctx),
-    UpdateLearningsTool(ctx)
+    UpdatePullRequestBodyTool(ctx)
   ];
 }
 async function tryStartMcpServer(ctx, tools, port) {
@@ -146181,9 +146184,6 @@ Rules:
 - Do NOT include a changelog section \u2014 the key changes list serves this purpose
 - Focus on *intent*, not *what* \u2014 the diff already shows what changed
 - Get the file count and commit count from the checkout_pr metadata, not by counting manually`;
-function learningsStep(t, n) {
-  return `${n}. **learnings** (only if high confidence): if you discovered something about repo setup, test commands, conventions, or patterns that you are confident is correct and would reliably help future runs, call \`${t("update_learnings")}\` to persist it. skip this step if you are unsure or the finding is speculative/one-off. format as a flat bullet list (\`- \` per line, one fact per bullet). merge with existing learnings from the prompt \u2014 pass the FULL merged list. deduplicate, and drop bullets that are clearly wrong or no longer relevant to the current codebase.`;
-}
 function computeModes(agentId) {
   const t = (toolName) => formatMcpToolRef(agentId, toolName);
   return [
@@ -146239,8 +146239,6 @@ function computeModes(agentId) {
    - create a PR via \`${t("create_pull_request")}\`
    - call \`${t("report_progress")}\` with the PR link or the exact error if push/PR failed
 
-${learningsStep(t, 6)}
-
 ### Notes
 
 For simple, well-defined tasks, skip the plan phase and go straight to build.`
@@ -146268,9 +146266,7 @@ For simple, well-defined tasks, skip the plan phase and go straight to build.`
    - confirm a clean working tree, then push via \`${t("push_branch")}\` (same push/prepush guidance as Build mode in *SYSTEM*)
    - reply to each comment using \`${t("reply_to_review_comment")}\`
    - resolve addressed threads via \`${t("resolve_review_thread")}\`
-   - call \`${t("report_progress")}\` with a brief summary (or the exact push error if push failed)
-
-${learningsStep(t, 6)}`
+   - call \`${t("report_progress")}\` with a brief summary (or the exact push error if push failed)`
     },
     // Review and IncrementalReview use the multi-lens orchestrator pattern
     // (canonical source: .claude/commands/anneal.md). The orchestrator does
@@ -146441,9 +146437,7 @@ ${PR_SUMMARY_FORMAT}`
 
 2. Produce a structured, actionable plan with clear milestones.
 
-3. Call \`${t("report_progress")}\` with the plan.
-
-${learningsStep(t, 4)}`
+3. Call \`${t("report_progress")}\` with the plan.`
     },
     {
       name: "Fix",
@@ -146465,9 +146459,7 @@ ${learningsStep(t, 4)}`
 
 5. Finalize:
    - confirm a clean working tree, then push via \`${t("push_branch")}\` (same push/prepush guidance as Build mode in *SYSTEM*)
-   - call \`${t("report_progress")}\` with the diagnosis and fix summary (or the exact push error if push failed)
-
-${learningsStep(t, 6)}`
+   - call \`${t("report_progress")}\` with the diagnosis and fix summary (or the exact push error if push failed)`
     },
     {
       name: "ResolveConflicts",
@@ -146511,9 +146503,7 @@ ${learningsStep(t, 6)}`
 3. Finalize:
    - if code changes were made, push to a pull request (new or existing) using \`${t("push_branch")}\` and \`${t("create_pull_request")}\` as needed. \`git status\` must be clean before you finish (see *SYSTEM* Git rules if push fails).
    - call \`${t("report_progress")}\` once with results \u2014 include exact tool errors if push or PR creation failed
-   - if the task involved labeling, commenting, or other GitHub operations, perform those directly
-
-${learningsStep(t, 4)}`
+   - if the task involved labeling, commenting, or other GitHub operations, perform those directly`
     }
   ];
 }
@@ -146613,6 +146603,17 @@ async function installFromNpmTarball(params) {
 // utils/providerErrors.ts
 var statusKey = `\\b(?:status[_ ]?code|http[_ ]?status|status)["']?\\s*[:=]\\s*["']?`;
 var PROVIDER_ERROR_PATTERNS = [
+  // auth patterns must come BEFORE rate-limit patterns. OpenRouter 401 error
+  // payloads carry `x-ratelimit-*` response headers in the dump, and the
+  // free-form rate-limit regex below would otherwise win on word-boundary
+  // matches inside header names. canonical 401 messages: OpenRouter returns
+  // `{"error":{"message":"User not found","code":401}}` for disabled or
+  // invalid keys (https://openai.luzhipeng.com/docs/api/reference/errors-and-debugging).
+  { regex: new RegExp(`${statusKey}401\\b`, "i"), label: "auth error (401)" },
+  { regex: new RegExp(`${statusKey}403\\b`, "i"), label: "auth error (403)" },
+  { regex: /\bUser not found\b/i, label: "auth error (invalid/disabled key)" },
+  { regex: /\bInvalid authentication\b/i, label: "auth error (invalid credentials)" },
+  { regex: /\bNo auth credentials found\b/i, label: "auth error (missing credentials)" },
   { regex: new RegExp(`${statusKey}429\\b`, "i"), label: "rate limited (429)" },
   { regex: new RegExp(`${statusKey}500\\b`, "i"), label: "provider 500 error" },
   { regex: new RegExp(`${statusKey}503\\b`, "i"), label: "provider unavailable (503)" },
@@ -146676,7 +146677,7 @@ function installBundledSkills(params) {
       writeFileSync6(join9(skillDir, "SKILL.md"), content);
     }
   }
-  log.info(`installed bundled skills: ${BUNDLED_SKILL_NAMES.join(", ")}`);
+  log.success(`installed bundled skills: ${BUNDLED_SKILL_NAMES.join(", ")}`);
 }
 function addSkill(params) {
   const result = spawnSync5(
@@ -146701,7 +146702,7 @@ function addSkill(params) {
     }
   );
   if (result.status === 0) {
-    log.info(`installed ${params.skill} skill (${params.agent})`);
+    log.success(`installed ${params.skill} skill (${params.agent})`);
   } else {
     const stderr = (result.stderr?.toString() || "").trim();
     const errorMsg = result.error ? result.error.message : stderr;
@@ -146835,18 +146836,17 @@ function buildPostRunPrompt(issues) {
   if (issues.summaryStale) parts.push(buildSummaryStalePrompt(issues.summaryStale.filePath));
   return parts.join("\n\n---\n\n");
 }
-function buildLearningsReflectionPrompt(agentId) {
-  const t = (name) => formatMcpToolRef(agentId, name);
+function buildLearningsReflectionPrompt(filePath) {
   return [
-    `REFLECTION \u2014 before you finish, think back over this task: did you discover anything about this repo's setup, test commands, conventions, or patterns that you are confident is correct and would reliably help future runs?`,
+    `REFLECTION \u2014 before you finish, think back over this task: did you discover anything about this repo's setup, test commands, conventions, or patterns that is high-confidence and would reliably help future runs?`,
     "",
-    `if so, call \`${t("update_learnings")}\` to persist it.`,
+    `the rolling learnings file is at \`${filePath}\`. read it first if you haven't already, then edit it in place using your native file tools. the server reads this file at end-of-run and persists any changes \u2014 there is no tool to call.`,
     "",
-    `rules:`,
-    `- only call \`${t("update_learnings")}\` when the finding is high-confidence and broadly useful. skip if unsure, speculative, or one-off.`,
-    `- pass the FULL merged list: existing learnings from the original prompt + your new discoveries. one fact per bullet, lines starting with \`- \`.`,
-    `- deduplicate, and drop bullets that are clearly wrong or no longer relevant to the current codebase.`,
-    `- if you already called \`${t("update_learnings")}\` earlier in this run, or nothing new is worth capturing, just reply "done" and stop \u2014 do not edit the repo for this reflection.`
+    `keep the file healthy:`,
+    `- only add bullets when the finding is high-confidence AND broadly useful. skip speculative, one-off, or "maybe" findings.`,
+    `- prune bullets that are clearly wrong, no longer relevant, or low-signal (rarely useful). a focused, accurate file beats a long stale one.`,
+    `- format: flat bullet list, one fact per line starting with \`- \`. deduplicate against existing entries \u2014 if a bullet covers the same fact, update it in place instead of adding a duplicate.`,
+    `- leave the file alone if you have nothing substantively new to add and the existing entries still look healthy. silence is a valid outcome \u2014 just reply "done" and stop.`
   ].join("\n");
 }
 async function runPostRunRetryLoop(params) {
@@ -147184,6 +147184,12 @@ async function runClaude(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // run claude in its own process group so SIGKILL on activity timeout /
+      // outer cancellation reaches any subprocesses it spawns (rg, file
+      // watchers, mcp transports, etc). claude itself is a node bundle so
+      // there's no shim-orphan issue like opencode-ai/bin/opencode, but
+      // detached + killGroup is the right default for any agent runtime.
+      killGroup: true,
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;
@@ -147423,7 +147429,7 @@ var claude = agent({
       stopScript: ctx.stopScript,
       summaryFilePath: ctx.summaryFilePath,
       summarySeed: ctx.summarySeed,
-      reflectionPrompt: buildLearningsReflectionPrompt("claude"),
+      reflectionPrompt: ctx.learningsFilePath ? buildLearningsReflectionPrompt(ctx.learningsFilePath) : void 0,
       canResume: (r) => Boolean(r.sessionId),
       resume: async (c) => {
         const sessionId = c.previousResult.sessionId;
@@ -147439,9 +147445,92 @@ var claude = agent({
 
 // agents/opencode.ts
 import { execFileSync as execFileSync4 } from "node:child_process";
-import { mkdirSync as mkdirSync5 } from "node:fs";
+import { mkdirSync as mkdirSync5, writeFileSync as writeFileSync8 } from "node:fs";
 import { join as join11 } from "node:path";
 import { performance as performance7 } from "node:perf_hooks";
+
+// agents/opencodePlugin.ts
+var PULLFROG_BUS_EVENT_TYPE = "pullfrog_bus_event";
+var PULLFROG_OPENCODE_PLUGIN_FILENAME = "pullfrog-events.ts";
+var PULLFROG_OPENCODE_PLUGIN_SOURCE = `// AUTOGENERATED by Pullfrog. do not edit; it'll be overwritten on the next run.
+// surfaces opencode subagent activity that the CLI's run-loop discards. see
+// action/agents/opencodePlugin.ts in pullfrog/app for why this exists. lives
+// inside the per-run tmpdir (XDG_CONFIG_HOME/opencode/plugin/), never inside
+// the user's working tree.
+
+const PULLFROG_BUS_EVENT_TYPE = ${JSON.stringify(PULLFROG_BUS_EVENT_TYPE)};
+
+// the first sessionID we see on a message.part.updated event is the
+// orchestrator \u2014 opencode's run command creates exactly one top-level session
+// before any subagent is dispatched, and the user-prompt text part fires
+// before the first task tool_use. we lock that sessionID in here and use it
+// to filter: the orchestrator's events are already streamed by the CLI's
+// run-loop, so we only forward (a) all subagent events, and (b) the
+// orchestrator's task tool dispatches at status="running". the CLI only
+// emits task tool_use at status=completed (after the subagent finishes), so
+// without the early announce the parent's labeler binds subagent sessions
+// before recordTaskDispatch fires and the lens label is lost.
+let orchestratorSessionID: string | undefined;
+
+function isOrchestratorTaskDispatch(part: {
+  type?: string;
+  tool?: string;
+  state?: { status?: string };
+}): boolean {
+  if (part.type !== "tool") return false;
+  if (part.tool !== "task") return false;
+  // only forward at status="running" (not "pending"). at pending the
+  // state.input is still {} \u2014 the orchestrator has emitted the part shell
+  // but the LLM hasn't filled in description/subagent_type/prompt yet. by
+  // running, input is populated and recordTaskDispatch can derive the lens
+  // label correctly.
+  return part.state?.status === "running";
+}
+
+export default async function pullfrogEventsPlugin() {
+  return {
+    event: async (input: {
+      event: {
+        type: string;
+        properties?: {
+          part?: {
+            sessionID?: string;
+            type?: string;
+            tool?: string;
+            state?: { status?: string };
+          };
+        };
+      };
+    }) => {
+      const event = input.event;
+      if (!event || typeof event !== "object") return;
+      if (event.type !== "message.part.updated") return;
+      const part = event.properties?.part;
+      const sessionID = part?.sessionID;
+      if (typeof sessionID !== "string" || sessionID.length === 0) return;
+      if (orchestratorSessionID === undefined) orchestratorSessionID = sessionID;
+
+      if (sessionID === orchestratorSessionID) {
+        // skip orchestrator events EXCEPT early task dispatches.
+        if (!part || !isOrchestratorTaskDispatch(part)) return;
+      }
+
+      try {
+        const line = JSON.stringify({
+          type: PULLFROG_BUS_EVENT_TYPE,
+          bus_event: event,
+        });
+        process.stdout.write(line + "\\n");
+      } catch {
+        // a circular reference or BigInt etc. would throw; swallow rather
+        // than letting a single bad event take down the plugin.
+      }
+    },
+  };
+}
+`;
+
+// agents/opencode.ts
 async function installOpencodeCli() {
   return await installFromNpmTarball({
     packageName: "opencode-ai",
@@ -147661,18 +147750,20 @@ async function runOpenCode(params) {
         return;
       }
       if (toolName === "task") {
-        const taskInput = event.part?.state?.input ?? {};
-        const dispatchedLabel = labeler.recordTaskDispatch(taskInput);
-        const dispatch = {
-          label: dispatchedLabel,
-          startedAt: performance7.now(),
-          toolUseCallID: toolId
-        };
-        taskDispatchByCallID.set(toolId, dispatch);
-        pendingTaskDispatches.push(dispatch);
-        log.info(
-          `\xBB dispatching subagent: ${dispatchedLabel}` + (taskInput.subagent_type ? ` (subagent_type=${taskInput.subagent_type})` : "")
-        );
+        if (!taskDispatchByCallID.has(toolId)) {
+          const taskInput = event.part?.state?.input ?? {};
+          const dispatchedLabel = labeler.recordTaskDispatch(taskInput);
+          const dispatch = {
+            label: dispatchedLabel,
+            startedAt: performance7.now(),
+            toolUseCallID: toolId
+          };
+          taskDispatchByCallID.set(toolId, dispatch);
+          pendingTaskDispatches.push(dispatch);
+          log.info(
+            `\xBB dispatching subagent: ${dispatchedLabel}` + (taskInput.subagent_type ? ` (subagent_type=${taskInput.subagent_type})` : "")
+          );
+        }
       } else {
         knownNonTaskCallIDs.add(toolId);
       }
@@ -147693,6 +147784,10 @@ async function runOpenCode(params) {
       if (event.part?.state?.status === "completed" && event.part.state.output) {
         log.debug(withLabel(label, `  output: ${event.part.state.output}`));
       }
+      if (event.part?.state?.status === "error") {
+        const errorMsg = event.part.state.output ?? "(no error message)";
+        log.info(withLabel(label, `\xBB tool call failed: ${errorMsg}`));
+      }
       if (toolName.includes("report_progress") && params.todoTracker) {
         log.debug("\xBB report_progress detected, disabling todo tracking");
         params.todoTracker.cancel();
@@ -147779,6 +147874,53 @@ async function runOpenCode(params) {
           tokensLogged = true;
         }
       }
+    },
+    [PULLFROG_BUS_EVENT_TYPE]: async (event) => {
+      const busEvent = event.bus_event;
+      if (!busEvent || busEvent.type !== "message.part.updated") return;
+      const part = busEvent.properties?.part;
+      if (!part || typeof part.sessionID !== "string") return;
+      const sessionID = part.sessionID;
+      const partType = part.type;
+      if (partType === "tool") {
+        const status = part.state?.status;
+        const partWithToolFields = part;
+        const isOrchestratorTaskDispatch = partWithToolFields.tool === "task" && status === "running";
+        if (isOrchestratorTaskDispatch) {
+          const callID = partWithToolFields.callID;
+          if (typeof callID === "string" && !taskDispatchByCallID.has(callID)) {
+            const taskInput = partWithToolFields.state?.input ?? {};
+            const dispatchedLabel = labeler.recordTaskDispatch(taskInput);
+            const dispatch = {
+              label: dispatchedLabel,
+              startedAt: performance7.now(),
+              toolUseCallID: callID
+            };
+            taskDispatchByCallID.set(callID, dispatch);
+            pendingTaskDispatches.push(dispatch);
+            log.info(
+              `\xBB dispatching subagent: ${dispatchedLabel}` + (taskInput.subagent_type ? ` (subagent_type=${taskInput.subagent_type})` : "")
+            );
+          }
+          return;
+        }
+        if (status !== "completed" && status !== "error") return;
+        await handlers2.tool_use({
+          type: "tool_use",
+          sessionID,
+          part
+        });
+        return;
+      }
+      if (partType === "step-start" || partType === "step-finish") return;
+      if (partType === "text" && part.time?.end !== void 0) {
+        await handlers2.text({
+          type: "text",
+          sessionID,
+          part
+        });
+        return;
+      }
     }
   };
   const recentStderr = [];
@@ -147795,6 +147937,20 @@ async function runOpenCode(params) {
       activityTimeout: 3e5,
       onActivityTimeout: params.onActivityTimeout,
       stdio: ["ignore", "pipe", "pipe"],
+      // node_modules/opencode-ai/bin/opencode is a Node shim that spawnSyncs
+      // the native opencode-<plat>-<arch> binary with stdio:"inherit". without
+      // a process-group kill, SIGKILL hits only the shim, the native binary
+      // is reparented to PID 1, holds our stdout pipe open, and `child.close`
+      // never fires — producing zombie runs. detached + killGroup nukes the
+      // whole tree.
+      killGroup: true,
+      // NB: we used to pass `isPausedExternally: isSubagentInFlight` to suspend
+      // the activity timer during subagent dispatches. unnecessary now that
+      // our injected plugin (action/agents/opencodePlugin.ts) re-emits
+      // subagent `message.part.updated` events on opencode's stdout — those
+      // arrive at child.stdout here, fire updateActivity(), and reset
+      // lastActivityTime naturally. verified empirically in PR #634
+      // (~3.3 plugin events/sec during a typical subagent run).
       onStdout: async (chunk) => {
         const text = chunk.toString();
         output += text;
@@ -147949,6 +148105,12 @@ var opencode = agent({
       XDG_CONFIG_HOME: join11(ctx.tmpdir, ".config")
     };
     mkdirSync5(join11(homeEnv.XDG_CONFIG_HOME, "opencode"), { recursive: true });
+    const opencodePluginDir = join11(homeEnv.XDG_CONFIG_HOME, "opencode", "plugin");
+    mkdirSync5(opencodePluginDir, { recursive: true });
+    writeFileSync8(
+      join11(opencodePluginDir, PULLFROG_OPENCODE_PLUGIN_FILENAME),
+      PULLFROG_OPENCODE_PLUGIN_SOURCE
+    );
     const agentBrowserVersion = getDevDependencyVersion("agent-browser");
     addSkill({
       ref: `vercel-labs/agent-browser@v${agentBrowserVersion}`,
@@ -147991,7 +148153,7 @@ var opencode = agent({
       stopScript: ctx.stopScript,
       summaryFilePath: ctx.summaryFilePath,
       summarySeed: ctx.summarySeed,
-      reflectionPrompt: buildLearningsReflectionPrompt("opencode"),
+      reflectionPrompt: ctx.learningsFilePath ? buildLearningsReflectionPrompt(ctx.learningsFilePath) : void 0,
       resume: async (c) => runOpenCode({
         ...runParams,
         args: [...baseArgs, "--continue", c.prompt]
@@ -152204,7 +152366,7 @@ ${ctx.error}` : ctx.error;
 
 // utils/gitAuthServer.ts
 import { randomUUID as randomUUID3 } from "node:crypto";
-import { writeFileSync as writeFileSync8 } from "node:fs";
+import { writeFileSync as writeFileSync9 } from "node:fs";
 import { createServer as createServer2 } from "node:http";
 import { join as join13 } from "node:path";
 var CODE_TTL_MS = 5 * 60 * 1e3;
@@ -152293,7 +152455,7 @@ async function startGitAuthServer(tmpdir3) {
       `try{require("fs").unlinkSync("${scriptPath.replace(/\\/g, "\\\\")}")}catch(e){}`,
       `})}).on("error",function(){process.exit(1)})}`
     ].join("\n");
-    writeFileSync8(scriptPath, content, { mode: 448 });
+    writeFileSync9(scriptPath, content, { mode: 448 });
     return scriptPath;
   }
   async function close() {
@@ -152567,9 +152729,9 @@ function buildPromptContext(ctx) {
   };
 }
 function assembleFullPrompt(ctx) {
-  const learningsSection = ctx.learnings ? `************* LEARNINGS *************
+  const learningsSection = ctx.learningsFilePath ? `************* LEARNINGS *************
 
-${ctx.learnings}` : "";
+Repo-level learnings accumulated by previous agent runs live at \`${ctx.learningsFilePath}\`. Read this file early and let the entries inform your approach (test commands, conventions, gotchas, etc.). The file may be empty if no learnings have been collected yet.` : "";
   const runtimeSection = `************* RUNTIME *************
 
 ${ctx.runtime}`;
@@ -152596,8 +152758,8 @@ function resolveInstructions(ctx) {
   if (eventContext)
     tocEntries.push({ label: "EVENT CONTEXT", description: "related PR/issue data" });
   tocEntries.push({ label: "SYSTEM", description: "persona, security, tools, workflow rules" });
-  if (pctx.learnings)
-    tocEntries.push({ label: "LEARNINGS", description: "repo-specific knowledge" });
+  if (pctx.learningsFilePath)
+    tocEntries.push({ label: "LEARNINGS", description: "repo-specific knowledge file path" });
   tocEntries.push({ label: "RUNTIME", description: "environment metadata" });
   const toc = buildToc(tocEntries);
   const full = assembleFullPrompt({
@@ -152606,7 +152768,7 @@ function resolveInstructions(ctx) {
     procedure,
     eventContext,
     system,
-    learnings: pctx.learnings,
+    learningsFilePath: pctx.learningsFilePath,
     runtime: pctx.runtime
   });
   const event = [pctx.eventTitle, pctx.eventMetadata].filter(Boolean).join("\n\n---\n\n");
@@ -152620,6 +152782,32 @@ function resolveInstructions(ctx) {
   };
 }
 
+// utils/learnings.ts
+import { mkdir, readFile as readFile2, writeFile as writeFile2 } from "node:fs/promises";
+import { dirname as dirname4, join as join14 } from "node:path";
+var LEARNINGS_FILE_NAME = "pullfrog-learnings.md";
+var MAX_LEARNINGS_LENGTH = 1e4;
+function learningsFilePath(tmpdir3) {
+  return join14(tmpdir3, LEARNINGS_FILE_NAME);
+}
+async function seedLearningsFile(params) {
+  const path3 = learningsFilePath(params.tmpdir);
+  await mkdir(dirname4(path3), { recursive: true });
+  await writeFile2(path3, params.current ?? "", "utf8");
+  return path3;
+}
+async function readLearningsFile(path3) {
+  let raw2;
+  try {
+    raw2 = await readFile2(path3, "utf8");
+  } catch {
+    return null;
+  }
+  const trimmed = raw2.trim();
+  if (trimmed.length > MAX_LEARNINGS_LENGTH) return trimmed.slice(0, MAX_LEARNINGS_LENGTH);
+  return trimmed;
+}
+
 // utils/normalizeEnv.ts
 function maskValue(value2) {
   if (value2 && typeof value2 === "string" && value2.trim().length > 0) {
@@ -152795,8 +152983,8 @@ function resolvePayload(resolvedPromptInput, repoSettings) {
 }
 
 // utils/prSummary.ts
-import { mkdir, readFile as readFile2, writeFile as writeFile2 } from "node:fs/promises";
-import { dirname as dirname4, join as join14 } from "node:path";
+import { mkdir as mkdir2, readFile as readFile3, writeFile as writeFile3 } from "node:fs/promises";
+import { dirname as dirname5, join as join15 } from "node:path";
 var SUMMARY_FILE_NAME = "pullfrog-summary.md";
 var SUMMARY_SCAFFOLD = `# PR summary
 
@@ -152806,19 +152994,19 @@ var SUMMARY_SCAFFOLD = `# PR summary
 var MIN_SNAPSHOT_LENGTH = 60;
 var MAX_SNAPSHOT_LENGTH = 32768;
 function summaryFilePath(tmpdir3) {
-  return join14(tmpdir3, SUMMARY_FILE_NAME);
+  return join15(tmpdir3, SUMMARY_FILE_NAME);
 }
 async function seedSummaryFile(params) {
   const path3 = summaryFilePath(params.tmpdir);
-  await mkdir(dirname4(path3), { recursive: true });
+  await mkdir2(dirname5(path3), { recursive: true });
   const seed = params.previousSnapshot && params.previousSnapshot.trim().length >= MIN_SNAPSHOT_LENGTH ? params.previousSnapshot : SUMMARY_SCAFFOLD;
-  await writeFile2(path3, seed, "utf8");
+  await writeFile3(path3, seed, "utf8");
   return path3;
 }
 async function readSummaryFile(path3) {
   let raw2;
   try {
-    raw2 = await readFile2(path3, "utf8");
+    raw2 = await readFile3(path3, "utf8");
   } catch {
     return null;
   }
@@ -153036,9 +153224,9 @@ async function resolveRunContextData(params) {
 import { execFileSync as execFileSync5, execSync as execSync3 } from "node:child_process";
 import { mkdtempSync } from "node:fs";
 import { tmpdir as tmpdir2 } from "node:os";
-import { join as join15 } from "node:path";
+import { join as join16 } from "node:path";
 function createTempDirectory() {
-  const sharedTempDir = mkdtempSync(join15(tmpdir2(), "pullfrog-"));
+  const sharedTempDir = mkdtempSync(join16(tmpdir2(), "pullfrog-"));
   process.env.PULLFROG_TEMP_DIR = sharedTempDir;
   log.info(`\xBB created temp dir at ${sharedTempDir}`);
   return sharedTempDir;
@@ -153440,15 +153628,12 @@ function formatTransientErrorSummary(error49, owner) {
 }
 async function mintProxyKey(ctx) {
   try {
-    process.env.ACTIONS_ID_TOKEN_REQUEST_URL = ctx.oidcCredentials.requestUrl;
-    process.env.ACTIONS_ID_TOKEN_REQUEST_TOKEN = ctx.oidcCredentials.requestToken;
-    const oidcToken = await core6.getIDToken("pullfrog-api");
-    delete process.env.ACTIONS_ID_TOKEN_REQUEST_URL;
-    delete process.env.ACTIONS_ID_TOKEN_REQUEST_TOKEN;
+    const headers = await buildProxyTokenHeaders(ctx);
+    if (!headers) return null;
     const response = await apiFetch({
       path: "/api/proxy-token",
       method: "POST",
-      headers: { Authorization: `Bearer ${oidcToken}` }
+      headers
     });
     if (response.status === 402) {
       const body = await response.json().catch(() => null);
@@ -153480,15 +153665,30 @@ async function mintProxyKey(ctx) {
     delete process.env.ACTIONS_ID_TOKEN_REQUEST_TOKEN;
   }
 }
+async function buildProxyTokenHeaders(ctx) {
+  if (ctx.oidcCredentials) {
+    process.env.ACTIONS_ID_TOKEN_REQUEST_URL = ctx.oidcCredentials.requestUrl;
+    process.env.ACTIONS_ID_TOKEN_REQUEST_TOKEN = ctx.oidcCredentials.requestToken;
+    const oidcToken = await core6.getIDToken("pullfrog-api");
+    delete process.env.ACTIONS_ID_TOKEN_REQUEST_URL;
+    delete process.env.ACTIONS_ID_TOKEN_REQUEST_TOKEN;
+    return { Authorization: `Bearer ${oidcToken}` };
+  }
+  if (isLocalApiUrl()) {
+    log.info(`\xBB proxy: dev bypass (x-dev-repo) for ${ctx.repo.owner}/${ctx.repo.name}`);
+    return { "x-dev-repo": `${ctx.repo.owner}/${ctx.repo.name}` };
+  }
+  return null;
+}
 async function resolveProxyModel(ctx) {
   if (process.env.PULLFROG_MODEL?.trim()) return;
   const needsProxy = isInfraCovered({ isOss: ctx.oss, plan: ctx.plan }) && ctx.proxyModel;
   if (!needsProxy) return;
-  if (!ctx.oidcCredentials) {
+  if (!ctx.oidcCredentials && !isLocalApiUrl()) {
     log.warning("\xBB proxy requested but no OIDC credentials available \u2014 skipping");
     return;
   }
-  const key = await mintProxyKey({ oidcCredentials: ctx.oidcCredentials });
+  const key = await mintProxyKey({ oidcCredentials: ctx.oidcCredentials, repo: ctx.repo });
   if (!key) return;
   process.env.OPENROUTER_API_KEY = key;
   core6.setSecret(key);
@@ -153512,6 +153712,45 @@ async function fetchPreviousSnapshot(ctx, prNumber) {
     return null;
   }
 }
+async function persistLearnings(ctx) {
+  const filePath = ctx.toolState.learningsFilePath;
+  if (!filePath) return;
+  if (ctx.toolState.learningsPersistAttempted) return;
+  ctx.toolState.learningsPersistAttempted = true;
+  const current = await readLearningsFile(filePath);
+  if (current === null) {
+    log.debug(`learnings tmpfile missing or unreadable at ${filePath} \u2014 skipping persist`);
+    return;
+  }
+  const seed = ctx.toolState.learningsSeed?.trim() ?? "";
+  if (current === seed) {
+    log.debug("learnings tmpfile unchanged from seed \u2014 skipping persist");
+    return;
+  }
+  try {
+    const response = await apiFetch({
+      path: `/api/repo/${ctx.repo.owner}/${ctx.repo.name}/learnings`,
+      method: "PATCH",
+      headers: {
+        authorization: `Bearer ${ctx.apiToken}`,
+        "content-type": "application/json"
+      },
+      body: JSON.stringify({
+        learnings: current,
+        model: ctx.toolState.model
+      }),
+      signal: AbortSignal.timeout(1e4)
+    });
+    if (!response.ok) {
+      const error49 = await response.text().catch(() => "(no body)");
+      log.debug(`learnings persist failed (${response.status}): ${error49}`);
+      return;
+    }
+    log.info("\xBB learnings updated");
+  } catch (err) {
+    log.debug(`learnings persist failed: ${err instanceof Error ? err.message : String(err)}`);
+  }
+}
 async function persistSummary(ctx) {
   const filePath = ctx.toolState.summaryFilePath;
   if (!filePath) return;
@@ -153593,7 +153832,8 @@ async function main() {
         oss: runContext.oss,
         plan: runContext.plan,
         proxyModel: runContext.proxyModel,
-        oidcCredentials
+        oidcCredentials,
+        repo: runContext.repo
       });
     } catch (error49) {
       if (error49 instanceof BillingError) {
@@ -153696,12 +153936,32 @@ async function main() {
         toolContext.mcpServerUrl = mcpHttpServer.url;
         log.info(`\xBB MCP server started at ${mcpHttpServer.url}`);
         timer.checkpoint("mcpServer");
+        try {
+          const learningsPath = await seedLearningsFile({
+            tmpdir: tmpdir3,
+            current: runContext.repoSettings.learnings
+          });
+          toolState.learningsFilePath = learningsPath;
+          try {
+            toolState.learningsSeed = await readFile4(learningsPath, "utf8");
+          } catch {
+          }
+          log.info(
+            `\xBB learnings seeded at ${learningsPath} (existing=${runContext.repoSettings.learnings ? "yes" : "no"})`
+          );
+          const ctxForExit = toolContext;
+          onExitSignal(() => persistLearnings(ctxForExit));
+        } catch (err) {
+          log.warning(
+            `\xBB learnings seed failed: ${err instanceof Error ? err.message : String(err)} \u2014 continuing without learnings file`
+          );
+        }
         if (payload.generateSummary && payload.event.is_pr && payload.event.issue_number) {
           const previousSnapshot = await fetchPreviousSnapshot(toolContext, payload.event.issue_number);
           const filePath = await seedSummaryFile({ tmpdir: tmpdir3, previousSnapshot });
           toolState.summaryFilePath = filePath;
           try {
-            toolState.summarySeed = await readFile3(filePath, "utf8");
+            toolState.summarySeed = await readFile4(filePath, "utf8");
           } catch {
           }
           log.info(
@@ -153725,7 +153985,7 @@ async function main() {
           modes: modes2,
           agentId,
           outputSchema,
-          learnings: runContext.repoSettings.learnings
+          learningsFilePath: toolState.learningsFilePath ?? null
         });
         const logParts = [
           instructions.eventInstructions ? `EVENT-LEVEL INSTRUCTIONS:
@@ -153741,7 +154001,7 @@ ${instructions.user}` : null,
           log.info(instructions.full);
         });
         if (agentId === "opencode") {
-          const pluginDir = join16(process.cwd(), ".opencode", "plugin");
+          const pluginDir = join17(process.cwd(), ".opencode", "plugin");
           const hasPlugins = existsSync7(pluginDir) && readdirSync(pluginDir).some((f) => /\.[jt]sx?$/.test(f));
           if (hasPlugins && toolState.dependencyInstallation?.promise) {
             log.info(
@@ -153802,6 +154062,7 @@ ${instructions.user}` : null,
           stopScript: runContext.repoSettings.stopScript,
           summaryFilePath: toolState.summaryFilePath,
           summarySeed: toolState.summarySeed,
+          learningsFilePath: toolState.learningsFilePath,
           onActivityTimeout: onInnerActivityTimeout,
           onToolUse: (event) => {
             const wasTracked = recordDiffReadFromToolUse({
@@ -153859,6 +154120,9 @@ ${instructions.user}` : null,
         if (toolContext) {
           await persistSummary(toolContext);
         }
+        if (toolContext) {
+          await persistLearnings(toolContext);
+        }
         if (toolContext && toolState.progressComment && !toolState.finalSummaryWritten) {
           await deleteProgressComment(toolContext).catch((error49) => {
             log.debug(`stranded progress comment cleanup failed: ${error49}`);
@@ -153911,6 +154175,9 @@ ${errorMessage}
       if (toolContext) {
         await persistSummary(toolContext);
       }
+      if (toolContext) {
+        await persistLearnings(toolContext);
+      }
       return {
         success: false,
         error: errorMessage