pullfrog 0.1.56 → 0.1.57

package/dist/cli.mjs

@@ -106536,15 +106536,13 @@ function formatMcpToolRef(agentId, toolName) {
 function extractMcpToolRefs(agentId, text) {
   const prefix = formatMcpToolRef(agentId, "");
   const escaped = prefix.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-  return [...text.matchAll(new RegExp(`${escaped}([a-z0-9_]+)`, "g"))].map((match3) => match3[1]);
+  return [...text.matchAll(new RegExp(`(?<![A-Za-z0-9_])${escaped}([a-z0-9_]+)`, "g"))].map(
+    (match3) => match3[1]
+  );
 }
-function assertPromptToolRefs(params) {
+function findDanglingPromptToolRefs(params) {
   const registered = new Set(params.toolNames);
-  const dangling = [...new Set(extractMcpToolRefs(params.agentId, params.prompt))].filter((name) => !registered.has(name)).sort();
-  if (dangling.length === 0) return;
-  throw new Error(
-    `prompt advertises ${dangling.length} tool(s) the MCP server did not register: ${dangling.join(", ")}. registered: ${[...registered].sort().join(", ")}`
-  );
+  return [...new Set(extractMcpToolRefs(params.agentId, params.prompt))].filter((name) => !registered.has(name)).sort();
 }
 function hasSimilarIssues(params) {
   return params.repoIntelligence && !!params.event.issue_number && !params.event.is_pr;
@@ -107319,7 +107317,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.56",
+  version: "0.1.57",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -197698,11 +197696,16 @@ async function main() {
           xrepoLearningsFilePath: toolState.xrepoLearningsFilePath ?? null,
           xrepoLearningsHeadings: runContext.repoSettings.xrepoLearningsHeadings
         });
-        assertPromptToolRefs({
+        const danglingToolRefs = findDanglingPromptToolRefs({
           agentId,
           prompt: instructions.full,
           toolNames: mcpHttpServer.toolNames
         });
+        if (danglingToolRefs.length > 0) {
+          log2.warning(
+            `\xBB prompt advertises ${danglingToolRefs.length} tool(s) the MCP server did not register: ${danglingToolRefs.join(", ")}`
+          );
+        }
         const logParts = [
           instructions.eventInstructions ? `EVENT-LEVEL INSTRUCTIONS:
 ${instructions.eventInstructions}` : null,
@@ -198893,7 +198896,7 @@ async function runCli4(input) {
 }
 
 // cli.ts
-var VERSION10 = "0.1.56";
+var VERSION10 = "0.1.57";
 var bin = basename2(process.argv[1] || "");
 var PROG = bin === "pf" || bin === "pullfrog" ? bin : "pullfrog";
 var rawArgs = process.argv.slice(2);

package/dist/external.d.ts

@@ -25,7 +25,7 @@ export declare function formatMcpToolRef(agentId: AgentId, toolName: string): st
  */
 export declare function extractMcpToolRefs(agentId: AgentId, text: string): string[];
 /**
- * Fail the run if the assembled prompt advertises a tool the MCP server did not
+ * Report when the assembled prompt advertises a tool the MCP server did not
  * actually register.
  *
  * Prompt text and tool registration are decided in different files off
@@ -35,12 +35,19 @@ export declare function extractMcpToolRefs(agentId: AgentId, text: string): stri
  * never learns about one that is. `toolNames` comes straight back from
  * `startMcpHttpServer`, i.e. the list the server registered rather than a
  * second derivation of it, so there is nothing to keep in sync.
+ *
+ * WARNS rather than throws, and that is deliberate. The prompt embeds
+ * customer-authored text — repo instructions, an issue or PR body — so a
+ * customer who merely WRITES a prefixed token controls this predicate. Throwing
+ * handed them a way to fail every run on their repo, which is what happened to
+ * `arcainc/arca` on 0.1.56. A drift check that aborts a customer run is the
+ * same mistake 0.1.54 made; the drift still has to be visible, so it is logged.
  */
-export declare function assertPromptToolRefs(params: {
+export declare function findDanglingPromptToolRefs(params: {
     agentId: AgentId;
     prompt: string;
     toolNames: string[];
-}): void;
+}): string[];
 export type { EffortPosition } from "./effort.ts";
 export { DEFAULT_EFFORT_POSITION, EFFORT_ALIASES, isEffortPosition, offeredRungs, parseEffortPosition, resolveRung, rungLabel, rungPosition, } from "./effort.ts";
 export type { AutoTier, ModelAlias, ModelProvider, ProviderConfig } from "./models.ts";

package/dist/index.js

@@ -104369,15 +104369,13 @@ function formatMcpToolRef(agentId, toolName) {
 function extractMcpToolRefs(agentId, text) {
   const prefix = formatMcpToolRef(agentId, "");
   const escaped = prefix.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-  return [...text.matchAll(new RegExp(`${escaped}([a-z0-9_]+)`, "g"))].map((match3) => match3[1]);
+  return [...text.matchAll(new RegExp(`(?<![A-Za-z0-9_])${escaped}([a-z0-9_]+)`, "g"))].map(
+    (match3) => match3[1]
+  );
 }
-function assertPromptToolRefs(params) {
+function findDanglingPromptToolRefs(params) {
   const registered = new Set(params.toolNames);
-  const dangling = [...new Set(extractMcpToolRefs(params.agentId, params.prompt))].filter((name) => !registered.has(name)).sort();
-  if (dangling.length === 0) return;
-  throw new Error(
-    `prompt advertises ${dangling.length} tool(s) the MCP server did not register: ${dangling.join(", ")}. registered: ${[...registered].sort().join(", ")}`
-  );
+  return [...new Set(extractMcpToolRefs(params.agentId, params.prompt))].filter((name) => !registered.has(name)).sort();
 }
 function hasSimilarIssues(params) {
   return params.repoIntelligence && !!params.event.issue_number && !params.event.is_pr;
@@ -105152,7 +105150,7 @@ var import_semver = __toESM(require_semver2(), 1);
 // package.json
 var package_default = {
   name: "pullfrog",
-  version: "0.1.56",
+  version: "0.1.57",
   type: "module",
   bin: {
     pullfrog: "dist/cli.mjs",
@@ -195574,11 +195572,16 @@ async function main() {
           xrepoLearningsFilePath: toolState.xrepoLearningsFilePath ?? null,
           xrepoLearningsHeadings: runContext.repoSettings.xrepoLearningsHeadings
         });
-        assertPromptToolRefs({
+        const danglingToolRefs = findDanglingPromptToolRefs({
           agentId,
           prompt: instructions.full,
           toolNames: mcpHttpServer.toolNames
         });
+        if (danglingToolRefs.length > 0) {
+          log.warning(
+            `\xBB prompt advertises ${danglingToolRefs.length} tool(s) the MCP server did not register: ${danglingToolRefs.join(", ")}`
+          );
+        }
         const logParts = [
           instructions.eventInstructions ? `EVENT-LEVEL INSTRUCTIONS:
 ${instructions.eventInstructions}` : null,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pullfrog",
-  "version": "0.1.56",
+  "version": "0.1.57",
   "type": "module",
   "bin": {
     "pullfrog": "dist/cli.mjs",