cortad 0.3.1-rc.13 → 0.3.1-rc.14

package/lib/proof.mjs

@@ -547,8 +547,12 @@ const sentence = ({ model, site }) => (model ? " The model said it."
 // The tools the request's last model call asked for, when they ran inside this request and no model
 // call came after them: the app closed the request before the model could answer from their result
 // (an AI SDK route left at one step). A tool the page runs itself and sends back is not this.
+// A last call that wrote words answered: its request carries the tool calls before it in its history
+// (MIT Learn AI's answering call resent search_courses), which is not the request stopping there.
 function endedOnTool(ex) {
-  const asked = [...new Set(arr(generation(ex).at(-1)?.called).map((c) => str(c?.name)).filter(Boolean))];
+  const last = generation(ex).at(-1);
+  if (str(last?.reply).trim()) return null;
+  const asked = [...new Set(arr(last?.called).map((c) => str(c?.name)).filter(Boolean))];
   if (!asked.length) return null;
   const inApp = new Set(ex.deps.flatMap((d) => arr(d.called).map((c) => str(c?.name))));
   const streamed = eventsOf(str(ex.reply)).events.some((e) => e && typeof e === "object" && /tool.*(?:output|result)/i.test(str(e.type)));

package/lib/pyhook/sitecustomize.py

@@ -171,7 +171,14 @@ def _install():
     # What the app's tools answered, as the prompt of the next model call carries them (chat tool
     # messages, responses function outputs, Anthropic tool_result blocks, Gemini functionResponse
     # parts): the material a reply's facts rest on. Bounded per call.
-    tool_text, tools_max = 3000, 12
+    # A tool's answer is kept to the cap the reader takes, and one longer says it was cut (wire.cjs toolText).
+    tool_text, tools_max = 12000, 12
+
+    def tool_text_row(name, full, **extra):
+        out = {"name": str(name or "")[:80], "text": full[:tool_text], **extra}
+        if len(full) > tool_text:
+            out["cut"] = True
+        return out
 
     def text_of(v):
         if isinstance(v, str):
@@ -214,9 +221,10 @@ def _install():
         out, names = [], {}
 
         def add(name, text):
-            t = text_of(text)[:tool_text]
+            full = text_of(text)
+            t = full[:tool_text]
             if t.strip() and len(out) < tools_max and all(o["text"] != t for o in out):
-                out.append({"name": str(name or "")[:80], "text": t})
+                out.append(tool_text_row(name, full))
 
         messages = items(body.get("messages"))
         for m in messages:
@@ -645,9 +653,10 @@ def _install():
         calls, names, answers = {}, {}, []
 
         def answer(name, said):
-            t = text_of(said)[:tool_text]
+            full = text_of(said)
+            t = full[:tool_text]
             if name and t.strip() and len(answers) < tools_max and not any(a["name"] == name and a["text"] == t for a in answers):
-                answers.append({"name": str(name)[:80], "text": t, "provider": True})
+                answers.append(tool_text_row(name, full, provider=True))
 
         for e in events:
             if not isinstance(e, dict):
@@ -1731,7 +1740,7 @@ def _install():
             text = "raised %s: %s" % (type(error).__name__, error) if error is not None else returned(value)
             row = {"at": int(time.time() * 1000), "host": "in-app", "status": 200,
                    "called": [{"name": name, "arguments": args_text(plain_of(named))}],
-                   "tools": [{"name": name, "text": str(text)[:tool_text]}]}
+                   "tools": [tool_text_row(name, str(text))]}
             row.update(inside_of(req))
             write({"dep": row})
         except Exception:

package/lib/stores.mjs

@@ -126,6 +126,8 @@ function fromParts(values) {
   for (const [p, hostName] of hosts) {
     // A host setting left blank is a store the app is set up not to use (QDRANT_HOST= beside QDRANT_PATH).
     if (hostName && !String(values[hostName] ?? "").trim()) continue;
+    // A host setting that holds a whole address (REDIS_HOST=redis://...) is fromUrl's, not a host.
+    if (hostName && String(values[hostName]).includes("://")) continue;
     const parts = {};
     for (const name of Object.keys(values)) {
       if (!name.startsWith(p) || name === hostName) continue;

package/lib/trace.cjs

@@ -17,7 +17,7 @@ if (FILE) {
     const http = require("node:http");
     const https = require("node:https");
     const als = new AsyncLocalStorage();
-    const { MAX, REPLY_MAX, EMBEDDING, TOOL_TEXT, scrubbed, list, clipped, argsText, turnOf, isModelCall, isRetrieval, passagesFrom, decoded, cookieNames, makeRecorder } = require("./wire.cjs");
+    const { MAX, REPLY_MAX, EMBEDDING, TOOL_TEXT, toolText, scrubbed, list, clipped, argsText, turnOf, isModelCall, isRetrieval, passagesFrom, decoded, cookieNames, makeRecorder } = require("./wire.cjs");
     const row = (value) => { try { fs.appendFileSync(FILE, JSON.stringify(scrubbed(value)) + "\n", { mode: 0o600 }); } catch { /* the command is gone */ } };
     // What the app writes into its own folder while a run is on: each file copied once before its
     // first write, so the command puts every one back when the run ends (lib/writes.mjs). Trials
@@ -611,7 +611,7 @@ if (FILE) {
       args.forEach((a, i) => { named[IDENT.test(names[i] || "") ? names[i] : String(i)] = plainOf(a); });
       let text;
       try { text = error ? `raised ${(error && error.name) || "an error"}: ${(error && error.message) || String(error)}` : returned(value); } catch { text = "returned a value this hook cannot read"; }
-      row({ dep: { at: Date.now(), host: "in-app", status: 200, ...(ctx ? { ex: ctx.id } : {}), ...(turn ? { turn } : {}), called: [{ name, arguments: argsText(named) }], tools: [{ name, text: String(text).slice(0, TOOL_TEXT) }] } });
+      row({ dep: { at: Date.now(), host: "in-app", status: 200, ...(ctx ? { ex: ctx.id } : {}), ...(turn ? { turn } : {}), called: [{ name, arguments: argsText(named) }], tools: [toolText(name, String(text))] } });
     };
     const TOOL = Symbol.for("cortad.wrapped");
     const wrappers = new WeakMap();

package/lib/wire.cjs

@@ -55,7 +55,11 @@ const partsOf = (text) => text.split(/\{[^}]*\}|\$\{[^}]*\}|%[sd]|<[^>]{1,40}>/)
 // messages of a chat body, the function outputs of a responses body, the tool_result blocks of
 // an Anthropic one, the functionResponse parts of a Gemini one. They are the material a reply's
 // facts rest on, and without them a real ticket id read as invented. Bounded per call.
-const TOOL_TEXT = 3000, TOOLS_MAX = 12;
+// A tool's answer is kept to the cap the reader takes (12,000 characters), and one longer says it was
+// cut: MIT Learn AI's 8,000-character search results were kept at 3,000, and a podcast episode in
+// the part cut off was graded as made up. A cut answer stands the passage questions down.
+const TOOL_TEXT = 12000, TOOLS_MAX = 12;
+const toolText = (name, full, extra = {}) => ({ name: String(name || "").slice(0, 80), text: full.slice(0, TOOL_TEXT), ...extra, ...(full.length > TOOL_TEXT ? { cut: true } : {}) });
 const textOf = (v) => (typeof v === "string" ? v : Array.isArray(v) ? v.map((p) => (p && typeof p === "object" ? (p.text || p.content || "") : String(p || ""))).filter(Boolean).join("\n") : v && typeof v === "object" ? JSON.stringify(v) : "");
 // Only what came after the person's latest message belongs to this turn: a thread the app
 // resends whole carries every earlier turn's tool answers too.
@@ -71,7 +75,7 @@ const toolsIn = (sent) => {
   if (!body || typeof body !== "object") return undefined;
   const out = [];
   const names = new Map();
-  const add = (name, text) => { const t = textOf(text).slice(0, TOOL_TEXT); if (t.trim() && out.length < TOOLS_MAX && !out.some((o) => o.text === t)) out.push({ name: String(name || "").slice(0, 80), text: t }); };
+  const add = (name, text) => { const full = textOf(text); const t = full.slice(0, TOOL_TEXT); if (t.trim() && out.length < TOOLS_MAX && !out.some((o) => o.text === t)) out.push(toolText(name, full)); };
   for (const m of list(body.messages)) {
     for (const c of list(m.tool_calls)) if (c.id && c.function) names.set(c.id, c.function.name);
     for (const c of list(m.content)) if (c.type === "tool_use" && c.id) names.set(c.id, c.name);
@@ -323,7 +327,7 @@ const providerIn = (sent, events) => {
   const offered = list(body && body.tools).map((t) => t.type).filter((t) => typeof t === "string" && t !== "function" && t !== "custom");
   const named = (kind) => offered.find((t) => t.startsWith(kind)) || kind;
   const calls = new Map(), names = new Map(), answers = [];
-  const answer = (name, text) => { const t = textOf(text).slice(0, TOOL_TEXT); if (name && t.trim() && answers.length < TOOLS_MAX && !answers.some((a) => a.name === name && a.text === t)) answers.push({ name: String(name).slice(0, 80), text: t, provider: true }); };
+  const answer = (name, text) => { const full = textOf(text); const t = full.slice(0, TOOL_TEXT); if (name && t.trim() && answers.length < TOOLS_MAX && !answers.some((a) => a.name === name && a.text === t)) answers.push(toolText(name, full, { provider: true })); };
   for (const e of events) {
     if (!e || typeof e !== "object") continue;
     const order = [];
@@ -813,6 +817,6 @@ function makeRecorder({ row, rulesFile, onTurn = () => {}, onEnded = () => {} })
 
 module.exports = {
   MAX, REPLY_MAX, EMBEDDING, TOOL_TEXT, MODEL_HOST,
-  scrubbed, list, clipped, argsText, turnOf, isModelCall, isRetrieval, passagesFrom, decoded, cookieNames,
+  scrubbed, list, clipped, argsText, turnOf, toolText, isModelCall, isRetrieval, passagesFrom, decoded, cookieNames,
   makeRecorder,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cortad",
-  "version": "0.3.1-rc.13",
+  "version": "0.3.1-rc.14",
   "description": "Connects the AI app on your machine to Cortad for test conversations, and gives your coding agent the MCP and skill to run them. No dependencies.",
   "bin": {
     "cortad": "local.mjs"