@cubicecho/agent-core 2.0.4 → 2.0.6

package/dist/side-task.js

@@ -61,7 +61,16 @@ function rejectedTheRequest(error) {
  * model that spends it deliberating is cut off mid-scratchpad and the closing tag never
  * arrives — and the whole deliberation was then returned to the caller as the answer.
  */
-const stripThinking = (text) => text.replace(/<think>[\s\S]*?<\/think>/gi, "").replace(/<think>[\s\S]*$/i, "");
+const stripThinking = (text) => text
+    .replace(/<think>[\s\S]*?<\/think>/gi, "")
+    .replace(/<think>[\s\S]*$/i, "")
+    // And the fence that never opens. Several chat templates put the opening tag at the end of
+    // the prompt rather than leaving the model to write it, so what comes back is deliberation
+    // first and only the closing tag to mark where it stops. Neither pattern above matches that,
+    // and the whole scratchpad went to the caller as the answer — a session title, a tool
+    // preselection, a suggestion list. Every real `<think>` is gone by this point, so a `</think>`
+    // still here opened in the prompt; the first one is taken, which keeps the most text.
+    .replace(/^[\s\S]*?<\/think>/i, "");
 /**
  * Runs a side task and returns the reply text, thinking stripped. Throws like any request.
  *

package/dist/tool-loading.d.ts

@@ -88,6 +88,7 @@ export declare function expandNames(requested: string[], catalog: CatalogServer[
         name: string;
         hits: string[];
     }[];
+    deferred: string[];
 };
 /**
  * What `load_tools` reports back: the descriptions, now that they are worth their tokens.
@@ -95,7 +96,7 @@ export declare function expandNames(requested: string[], catalog: CatalogServer[
  * @param expanded What `expandNames` resolved: the matches, the misses, and the over-broad asks.
  * @param catalog The servers, read for the descriptions now worth their tokens.
  */
-export declare function loadResult({ matched, unknown, overBroad }: ReturnType<typeof expandNames>, catalog: CatalogServer[]): string;
+export declare function loadResult({ matched, unknown, overBroad, deferred }: ReturnType<typeof expandNames>, catalog: CatalogServer[]): string;
 /**
  * Whether the catalogue holds a tool by this name.
  *

package/dist/tool-loading.js

@@ -137,6 +137,7 @@ export function expandNames(requested, catalog) {
     const matched = new Set();
     const unknown = [];
     const overBroad = [];
+    const deferred = [];
     const known = new Set(all.map((tool) => tool.name));
     const resolve = (name) => {
         if (name.endsWith("*")) {
@@ -172,13 +173,21 @@ export function expandNames(requested, catalog) {
         // model that is meant to be choosing from twelve. Already-matched names are free, so a
         // name that only repeats an earlier one never spends budget.
         const fresh = hits.filter((hit) => !matched.has(hit));
-        if (matched.size + fresh.length > MAX_PER_LOAD)
+        // Two different refusals, and answering both with "narrow it down" made one of them
+        // impossible to act on. A name that would not fit an empty call is over-broad, and the
+        // names are what the model needs. A name that only does not fit *this* call is precise
+        // enough already — telling a model that asked for one exact tool that it matches one tool,
+        // "more than the twelve one call may load", and to choose from a list holding just that
+        // name, leaves it nothing to do but send the identical call again.
+        if (fresh.length > MAX_PER_LOAD)
             overBroad.push({ name, hits });
+        else if (matched.size + fresh.length > MAX_PER_LOAD)
+            deferred.push(name);
         else
             for (const hit of hits)
                 matched.add(hit);
     }
-    return { matched: [...matched], unknown, overBroad };
+    return { matched: [...matched], unknown, overBroad, deferred };
 }
 /**
  * What `load_tools` reports back: the descriptions, now that they are worth their tokens.
@@ -186,7 +195,7 @@ export function expandNames(requested, catalog) {
  * @param expanded What `expandNames` resolved: the matches, the misses, and the over-broad asks.
  * @param catalog The servers, read for the descriptions now worth their tokens.
  */
-export function loadResult({ matched, unknown, overBroad }, catalog) {
+export function loadResult({ matched, unknown, overBroad, deferred }, catalog) {
     const byName = new Map(flatten(catalog).map((tool) => [tool.name, tool.description]));
     const lines = [];
     if (matched.length) {
@@ -199,6 +208,11 @@ export function loadResult({ matched, unknown, overBroad }, catalog) {
             lines.push("");
         lines.push(`\`${name}\` matches ${hits.length} tools, more than the ${MAX_PER_LOAD} one call may load.`, "Name the ones you need from:", ...hits.map((hit) => `  ${hit}`));
     }
+    if (deferred.length) {
+        if (lines.length)
+            lines.push("");
+        lines.push(`This call is full at ${MAX_PER_LOAD} tools, so these were not loaded: ${deferred.join(", ")}.`, "Ask for them on your next step.");
+    }
     if (unknown.length) {
         if (lines.length)
             lines.push("");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cubicecho/agent-core",
-  "version": "2.0.4",
+  "version": "2.0.6",
   "description": "The endpoint-agnostic half of an OpenAI-compatible agent loop: tool-schema compatibility, on-demand tool loading, one-shot side tasks, run events, and a pooled client.",
   "keywords": [
     "openai",