@buoy-gg/agent-core 7.0.41 → 7.0.42

package/lib/module/catalog/normalizeParams.js

@@ -1,327 +1,3 @@
-"use strict";
+"use strict";const f=i=>!!i&&typeof i==="object"&&!Array.isArray(i);export function hashQueryKey(i){return JSON.stringify(i,(e,t)=>f(t)?Object.keys(t).sort().reduce((a,r)=>{a[r]=t[r];return a},{}):t)}function n(i,e,t,a){if(i[t]!==void 0||i[e]===void 0)return false;i[t]=i[e];delete i[e];a.push(`Accepted \`${e}\` as \`${t}\` \u2014 the parameter is named \`${t}\`.`);return true}const o=new Set(["getQueryData","refetch","invalidate","reset","remove","triggerError","restoreError","triggerLoading","restoreLoading"]);const l=["id","urlPattern","enabled","name","methods","kind","status","statusText","headers","body","bodyPatch","bodyPath","bodyValue","failKind","delayMs","times","alternate","force","bodyOmitted"];function h(i,e,t){if(o.has(i)&&e.queryHash===void 0){const a=f(e.filters)?e.filters:void 0;const r=e.queryKey??e.key??e.hash??a?.queryKey??a?.queryHash;if(Array.isArray(r)){e.queryHash=hashQueryKey(r);t.push(`Accepted the queryKey array as the query's hash (${e.queryHash}) \u2014 the parameter is \`queryHash\`, the string from listQueries.`)}else if(typeof r==="string"){e.queryHash=r;t.push(`Accepted it as \`queryHash\` \u2014 that is the parameter name.`)}delete e.queryKey;delete e.key;delete e.hash;delete e.filters}if(i==="setQueryData"){if(f(e.data)){for(const a of["merge","force"]){if(typeof e.data[a]==="boolean"&&e[a]===void 0){e[a]=e.data[a];delete e.data[a];t.push(`Moved \`${a}\` out of \`data\` to the top level \u2014 it is a write flag, not part of the value.`)}}}n(e,"newData","data",t)||e.path===void 0&&n(e,"value","data",t)||n(e,"patch","data",t);if(e.queryKey===void 0&&typeof e.queryHash==="string"){try{const a=JSON.parse(e.queryHash);if(Array.isArray(a)){e.queryKey=a;t.push("Accepted the queryHash as the key \u2014 setQueryData takes `queryKey`, the array.")}}catch{}}if(e.patch!==void 0||e.merge===void 0){if(t.some(a=>a.includes("`patch`")))e.merge=true}}}function c(i,e,t){if(i==="getEventBody"||i==="setPinned"||i==="setSaved"){n(e,"requestId","id",t)||n(e,"eventId","id",t)}if(i==="deleteOverrideRule"||i==="setOverrideRuleEnabled"||i==="getOverrideRuleBody"){n(e,"ruleId","id",t)}if(i==="upsertOverrideRule"){const a=f(e.rule)?{...e.rule}:{};if(e.fromRequestId===void 0&&typeof a.fromRequestId==="string"){e.fromRequestId=a.fromRequestId;t.push("Accepted `rule.fromRequestId` \u2014 it is a top-level parameter, beside `rule`.")}delete a.fromRequestId;y(a,t);m(e,a,t);const r=[];for(const d of l){if(e[d]!==void 0&&a[d]===void 0){a[d]=e[d];r.push(d)}delete e[d]}if(r.length)t.push(`Accepted ${r.map(d=>`\`${d}\``).join(", ")} at the top level \u2014 rule fields belong inside \`rule\`.`);if(Object.keys(a).length)e.rule=a}}function y(i,e){for(const[t,a]of[["match",["urlPattern","methods","method","url","pattern","urlContains","contains"]],["response",["status","statusText","body","headers","bodyPatch"]]]){const r=i[t];if(!f(r))continue;const d=[];for(const s of a){if(r[s]===void 0)continue;const u=s==="method"?"methods":s==="url"||s==="pattern"||s==="urlContains"||s==="contains"?"urlPattern":s;if(i[u]!==void 0)continue;i[u]=u==="methods"&&!Array.isArray(r[s])?[r[s]]:r[s];delete r[s];d.push(s)}if(!d.length)continue;if(!Object.keys(r).length)delete i[t];e.push(`Read \`rule.${t}.{${d.join(", ")}}\` as top-level rule fields \u2014 this rule is flat, unlike msw/nock-style mocks.`)}}function m(i,e,t){if(e.bodyPath===void 0&&typeof e.path==="string"){e.bodyPath=e.path;if(e.bodyValue===void 0&&e.value!==void 0)e.bodyValue=e.value;delete e.path;delete e.value;t.push("Read `path`/`value` as `bodyPath`/`bodyValue` \u2014 on a rule they address the response body.")}const a=s=>e[s]!==void 0?e[s]:i[s];const r=s=>{delete e[s];delete i[s]};const d=a("mode");if(typeof d==="string"&&["respond","fail","delay"].includes(d)&&e.kind===void 0){e.kind=d;t.push(`Read \`mode: "${d}"\` as \`kind: "${d}"\`.`)}r("mode");if(a("fail")===true&&e.kind===void 0){e.kind="fail";t.push('Read `fail: true` as `kind: "fail"` \u2014 a transport failure, as if offline.')}r("fail");if(a("failOnce")===true){if(e.kind===void 0)e.kind="fail";if(e.times===void 0)e.times=1;t.push('Read `failOnce: true` as `kind: "fail"` with `times: 1` \u2014 it fires once, then the rule disables itself.')}r("failOnce");if(a("once")===true&&e.times===void 0){e.times=1;t.push("Read `once: true` as `times: 1` \u2014 the rule auto-disables after one match.")}r("once");for(const s of["maxHits","hits","count","maxTimes","applyTimes"]){const u=a(s);if(typeof u==="number"&&Number.isFinite(u)&&u>0&&e.times===void 0){e.times=u;t.push(`Read \`${s}: ${u}\` as \`times: ${u}\` \u2014 the rule auto-disables after ${u} match(es).`)}r(s)}}function b(i,e,t){n(e,"name","storeName",t)||n(e,"store","storeName",t);if(i==="setState"){n(e,"statePatch","state",t)||n(e,"partial","state",t)||n(e,"patch","state",t)||n(e,"newState","state",t)||n(e,"nextState","state",t)||e.path===void 0&&n(e,"value","state",t);if(e.replace===void 0){e.replace=false;t.push("Applied replace:false (merge) \u2014 a store keeps action functions that can't be re-sent, and replacing would wipe them (the buttons that call them would then crash). Send replace:true only to deliberately reset the whole store.")}}}function g(i,e,t){if(i==="navigate"){n(e,"route","path",t)||n(e,"to","path",t)||n(e,"url","path",t)||n(e,"pathname","path",t)}}export function normalizeParams(i,e,t){const a={...t};const r=[];switch(i){case"query":h(e,a,r);break;case"network":c(e,a,r);break;case"zustand":b(e,a,r);break;case"route-events":g(e,a,r);break;default:break}return{params:a,notes:r}}export function aliasNote(i){return i.length?`
 
-/**
- * Parameter aliases — accept what models actually send.
- *
- * The provider sees ONE tool per Buoy tool with `params` as an open object
- * (see toProviderTools.ts), so the per-action parameter names live in prose
- * and every model has to reproduce them from memory. A sweep of 18 models
- * against the live app showed the same handful of misses over and over, on
- * cheap models AND on gpt-5 and Gemini 2.5 Pro:
- *
- *   queryKey: [...]   where the action wants queryHash   (by far the #1 miss —
- *                     the RIGHT NOW block shows keys as arrays, and
- *                     setQueryData takes queryKey, so the model reuses it)
- *   requestId         where network.getEventBody wants id
- *   newState/patch    where zustand.setState wants state
- *   name              where zustand wants storeName
- *   bodyPatch/body/…  at the top level of upsertOverrideRule instead of inside `rule`
- *
- * A validation error names the right field, but a cheap model repeats the same
- * wrong call or gives up, and the user sees "nothing happened". Every one of
- * these is mechanically unambiguous, so it is fixed HERE, before validation,
- * and the result carries a one-line note so the model learns the real name
- * without failing the call. Nothing here guesses: an alias only fires when the
- * canonical field is absent and the alias is present.
- */
-
-const isPlainObject = v => !!v && typeof v === "object" && !Array.isArray(v);
-
-/**
- * TanStack's default `hashKey`: JSON.stringify with plain-object keys sorted,
- * so `["todos", {page:1, sort:"a"}]` hashes the same whatever order the model
- * wrote the object in.
- */
-export function hashQueryKey(key) {
-  return JSON.stringify(key, (_, val) => isPlainObject(val) ? Object.keys(val).sort().reduce((acc, k) => {
-    acc[k] = val[k];
-    return acc;
-  }, {}) : val);
-}
-
-/** Rename `from` to `to` when `to` is absent and `from` is present. */
-function alias(p, from, to, notes) {
-  if (p[to] !== undefined || p[from] === undefined) return false;
-  p[to] = p[from];
-  delete p[from];
-  notes.push(`Accepted \`${from}\` as \`${to}\` — the parameter is named \`${to}\`.`);
-  return true;
-}
-const QUERY_HASH_ACTIONS = new Set(["getQueryData", "refetch", "invalidate", "reset", "remove", "triggerError", "restoreError", "triggerLoading", "restoreLoading"]);
-
-/** Fields that belong inside `rule` for network.upsertOverrideRule. */
-const RULE_FIELDS = ["id", "urlPattern", "enabled", "name", "methods", "kind", "status", "statusText", "headers", "body", "bodyPatch", "bodyPath", "bodyValue", "failKind", "delayMs", "times", "alternate", "force", "bodyOmitted"];
-function normalizeQuery(action, p, notes) {
-  if (QUERY_HASH_ACTIONS.has(action) && p.queryHash === undefined) {
-    // `filters: { queryKey }` — the TanStack client-API shape.
-    const filters = isPlainObject(p.filters) ? p.filters : undefined;
-    const key = p.queryKey ?? p.key ?? p.hash ?? filters?.queryKey ?? filters?.queryHash;
-    if (Array.isArray(key)) {
-      p.queryHash = hashQueryKey(key);
-      notes.push(`Accepted the queryKey array as the query's hash (${p.queryHash}) — the parameter is \`queryHash\`, the string from listQueries.`);
-    } else if (typeof key === "string") {
-      // A hash string under another name, or a loose "a,b,c" spelling the
-      // adapter resolves against the live cache.
-      p.queryHash = key;
-      notes.push(`Accepted it as \`queryHash\` — that is the parameter name.`);
-    }
-    delete p.queryKey;
-    delete p.key;
-    delete p.hash;
-    delete p.filters;
-  }
-  if (action === "setQueryData") {
-    // Models routinely tuck the write FLAGS inside `data` — `data: {item:{…},
-    // merge:true}` — which turns a merge into a replace and leaves a junk
-    // `merge`/`force` key in the cache. Lift a boolean flag out of `data` to
-    // the top level when the top level doesn't already set it. Guarded to
-    // booleans so a real data field named "merge" is never disturbed.
-    if (isPlainObject(p.data)) {
-      for (const flag of ["merge", "force"]) {
-        if (typeof p.data[flag] === "boolean" && p[flag] === undefined) {
-          p[flag] = p.data[flag];
-          delete p.data[flag];
-          notes.push(`Moved \`${flag}\` out of \`data\` to the top level — it is a write flag, not part of the value.`);
-        }
-      }
-    }
-    // `value` normally means "the patch, spelled differently". But alongside
-    // `path` it means the OPPOSITE — the single value that path is set to — so
-    // rewriting it there would turn `path:"item.name", value:"x"` into a
-    // top-level patch of `"x"`, silently. Only the path form reserves it.
-    alias(p, "newData", "data", notes) || p.path === undefined && alias(p, "value", "data", notes) || alias(p, "patch", "data", notes);
-    if (p.queryKey === undefined && typeof p.queryHash === "string") {
-      try {
-        const parsed = JSON.parse(p.queryHash);
-        if (Array.isArray(parsed)) {
-          p.queryKey = parsed;
-          notes.push("Accepted the queryHash as the key — setQueryData takes `queryKey`, the array.");
-        }
-      } catch {
-        /* leave it; validation will say what is missing */
-      }
-    }
-    if (p.patch !== undefined || p.merge === undefined) {
-      // A caller that said "patch" meant merge.
-      if (notes.some(n => n.includes("`patch`"))) p.merge = true;
-    }
-  }
-}
-function normalizeNetwork(action, p, notes) {
-  if (action === "getEventBody" || action === "setPinned" || action === "setSaved") {
-    alias(p, "requestId", "id", notes) || alias(p, "eventId", "id", notes);
-  }
-  if (action === "deleteOverrideRule" || action === "setOverrideRuleEnabled" || action === "getOverrideRuleBody") {
-    alias(p, "ruleId", "id", notes);
-  }
-  if (action === "upsertOverrideRule") {
-    const rule = isPlainObject(p.rule) ? {
-      ...p.rule
-    } : {};
-    // fromRequestId is a TOP-LEVEL field; models tuck it into the rule.
-    if (p.fromRequestId === undefined && typeof rule.fromRequestId === "string") {
-      p.fromRequestId = rule.fromRequestId;
-      notes.push("Accepted `rule.fromRequestId` — it is a top-level parameter, beside `rule`.");
-    }
-    delete rule.fromRequestId;
-    unwrapRuleSections(rule, notes);
-    ruleIntentWords(p, rule, notes);
-    // Rule fields sent flat at the top level move inside `rule`.
-    const moved = [];
-    for (const f of RULE_FIELDS) {
-      if (p[f] !== undefined && rule[f] === undefined) {
-        rule[f] = p[f];
-        moved.push(f);
-      }
-      delete p[f];
-    }
-    if (moved.length) notes.push(`Accepted ${moved.map(f => `\`${f}\``).join(", ")} at the top level — rule fields belong inside \`rule\`.`);
-    if (Object.keys(rule).length) p.rule = rule;
-  }
-}
-
-/**
- * Flatten the `match` / `response` wrappers models borrow from other mocking
- * libraries.
- *
- * Our rule is flat — `{urlPattern, methods, status, body, times}`. Nearly every
- * HTTP-mocking library a model has read is not: msw, nock and WireMock all group
- * the matcher and the reply into separate objects. So a model that has never
- * seen our schema (and it has not — `params` is an unconstrained object) writes
- * the convention it knows. Measured: Claude sent
- * `{times: 1, match: {method, urlPattern}, response: {status, body}}` — every
- * value correct, in the shape the rest of the world uses.
- *
- * Only these two wrapper names, and only fields that are really ours: an
- * unrecognised key inside one is left where it is rather than hoisted, so this
- * cannot quietly invent a rule field that does not exist.
- */
-function unwrapRuleSections(rule, notes) {
-  for (const [wrapper, fields] of [["match", ["urlPattern", "methods", "method", "url", "pattern", "urlContains", "contains"]], ["response", ["status", "statusText", "body", "headers", "bodyPatch"]]]) {
-    const inner = rule[wrapper];
-    if (!isPlainObject(inner)) continue;
-    const moved = [];
-    for (const f of fields) {
-      if (inner[f] === undefined) continue;
-      // `method` and `url` are that library's spelling of ours.
-      const target = f === "method" ? "methods" : f === "url" || f === "pattern" || f === "urlContains" || f === "contains" ? "urlPattern" : f;
-      if (rule[target] !== undefined) continue;
-      rule[target] = target === "methods" && !Array.isArray(inner[f]) ? [inner[f]] : inner[f];
-      delete inner[f];
-      moved.push(f);
-    }
-    if (!moved.length) continue;
-    // Everything recognised moved out; drop the wrapper only if nothing is left,
-    // so a leftover key still surfaces as a schema error rather than vanishing.
-    if (!Object.keys(inner).length) delete rule[wrapper];
-    notes.push(`Read \`rule.${wrapper}.{${moved.join(", ")}}\` as top-level rule fields — this rule is flat, unlike msw/nock-style mocks.`);
-  }
-}
-
-/**
- * The words models reach for when describing an override rule, mapped to the
- * fields the rule actually has.
- *
- * MEASURED, not invented. On `api-sim/next-call-fails-once` — "make the next
- * menu request fail, just the next one" — every model that acted picked the
- * right tool and explained the right reason, then wrote the rule like this:
- *
- *   grok    ×3   {fail: true, once: true}
- *   composer     {fail: true}  and  {maxHits: 1, fail: true}
- *   opus         {status: 500, enabled: true}
- *
- * The schema wants `{kind: "fail", times: 1}`. Ten of fourteen attempts on that
- * case graded `acceptable` rather than `best` for no reason except this — the
- * budget was expressed as `once`/`maxHits` and silently dropped, leaving a rule
- * that never expires.
- *
- * It is NOT a missing instruction. `times` is documented as "Use 1 for 'make the
- * next call fail' so it cannot linger" — very nearly the user's own sentence —
- * and every model read past it. That is the same finding as the nesting-depth
- * work: the fix has to be structural, not a better sentence.
- *
- * Each mapping only ever makes a rule NARROWER (a budget where there was none,
- * a transport failure where the kind was unstated), so a misread costs a rule
- * that stops too early rather than one that runs forever.
- */
-function ruleIntentWords(p, rule, notes) {
-  // `path`/`value` is what the state tools call this, and a model that learned
-  // it on `zustand.setState` or `query.setQueryData` writes the same words here.
-  // A rule has no other meaning for either name, so accept them.
-  if (rule.bodyPath === undefined && typeof rule.path === "string") {
-    rule.bodyPath = rule.path;
-    if (rule.bodyValue === undefined && rule.value !== undefined) rule.bodyValue = rule.value;
-    delete rule.path;
-    delete rule.value;
-    notes.push("Read `path`/`value` as `bodyPath`/`bodyValue` — on a rule they address the response body.");
-  }
-  const read = key => rule[key] !== undefined ? rule[key] : p[key];
-  const drop = key => {
-    delete rule[key];
-    delete p[key];
-  };
-
-  // `mode` is the other library's name for `kind`, and models use it verbatim:
-  // measured on `api-sim/api-down`, alongside `match.urlContains`.
-  const mode = read("mode");
-  if (typeof mode === "string" && ["respond", "fail", "delay"].includes(mode) && rule.kind === undefined) {
-    rule.kind = mode;
-    notes.push(`Read \`mode: "${mode}"\` as \`kind: "${mode}"\`.`);
-  }
-  drop("mode");
-
-  // "fail" as a flag, rather than kind:"fail".
-  if (read("fail") === true && rule.kind === undefined) {
-    rule.kind = "fail";
-    notes.push("Read `fail: true` as `kind: \"fail\"` — a transport failure, as if offline.");
-  }
-  drop("fail");
-
-  // "just once" / "only the next one", in the several ways it gets written.
-  if (read("failOnce") === true) {
-    if (rule.kind === undefined) rule.kind = "fail";
-    if (rule.times === undefined) rule.times = 1;
-    notes.push("Read `failOnce: true` as `kind: \"fail\"` with `times: 1` — it fires once, then the rule disables itself.");
-  }
-  drop("failOnce");
-  if (read("once") === true && rule.times === undefined) {
-    rule.times = 1;
-    notes.push("Read `once: true` as `times: 1` — the rule auto-disables after one match.");
-  }
-  drop("once");
-
-  // A numeric budget under any of its other names. `limit` is deliberately NOT
-  // here: it means "how many to list" on half the actions in this catalog, and
-  // a wrong guess about a budget is a rule that stops early on a request the
-  // caller still needed.
-  for (const key of ["maxHits", "hits", "count", "maxTimes", "applyTimes"]) {
-    const v = read(key);
-    if (typeof v === "number" && Number.isFinite(v) && v > 0 && rule.times === undefined) {
-      rule.times = v;
-      notes.push(`Read \`${key}: ${v}\` as \`times: ${v}\` — the rule auto-disables after ${v} match(es).`);
-    }
-    drop(key);
-  }
-}
-function normalizeZustand(action, p, notes) {
-  alias(p, "name", "storeName", notes) || alias(p, "store", "storeName", notes);
-  if (action === "setState") {
-    alias(p, "statePatch", "state", notes) || alias(p, "partial", "state", notes) || alias(p, "patch", "state", notes) || alias(p, "newState", "state", notes) || alias(p, "nextState", "state", notes) ||
-    // See the same guard on setQueryData: with `path`, `value` is the value
-    // being set, not another spelling of the patch.
-    p.path === undefined && alias(p, "value", "state", notes);
-    // ALWAYS merge, never replace, unless the caller explicitly said replace.
-    // The adapter defaults replace:true, which throws away everything not sent
-    // — including the store's ACTION FUNCTIONS (setQty, removeLine, …), which
-    // can't cross the JSON wire, so a remote caller never has them to re-send.
-    // The data then looks right but the buttons that call those methods crash.
-    // A merge (replace:false) shallow-merges the fields sent and keeps the
-    // functions and untouched keys intact.
-    if (p.replace === undefined) {
-      p.replace = false;
-      notes.push("Applied replace:false (merge) — a store keeps action functions that can't be re-sent, and replacing would wipe them (the buttons that call them would then crash). Send replace:true only to deliberately reset the whole store.");
-    }
-  }
-}
-function normalizeRoutes(action, p, notes) {
-  if (action === "navigate") {
-    alias(p, "route", "path", notes) || alias(p, "to", "path", notes) || alias(p, "url", "path", notes) || alias(p, "pathname", "path", notes);
-  }
-}
-
-/**
- * Fix the unambiguous parameter-name misses before validation. Returns the
- * same object shape validation expects plus the notes to append to the result.
- */
-export function normalizeParams(toolId, action, raw) {
-  const p = {
-    ...raw
-  };
-  const notes = [];
-  switch (toolId) {
-    case "query":
-      normalizeQuery(action, p, notes);
-      break;
-    case "network":
-      normalizeNetwork(action, p, notes);
-      break;
-    case "zustand":
-      normalizeZustand(action, p, notes);
-      break;
-    case "route-events":
-      normalizeRoutes(action, p, notes);
-      break;
-    default:
-      break;
-  }
-  return {
-    params: p,
-    notes
-  };
-}
-
-/** The one-line trailer appended to a tool result when aliases fired. */
-export function aliasNote(notes) {
-  return notes.length ? `\n\n[Buoy] ${notes.join(" ")}` : "";
-}
-//# sourceMappingURL=normalizeParams.js.map
+[Buoy] ${i.join(" ")}`:""}

package/lib/module/catalog/signature.js

@@ -1,63 +1 @@
-"use strict";
-
-/**
- * Rendering a JSON Schema as the one-line signature a model reads.
- *
- * Lifted out of `blocks/uiTool.ts`, where it was written for `buoy_ui` and
- * where its own comment records the result that justifies using it everywhere:
- *
- *   > `uiProviderTool` used to send `params: { type: "object" }` and 26 lines
- *   > of prose about what each kind is FOR — and not one field name. The model
- *   > was being asked to guess `images` vs `items` and `chips` vs `options`,
- *   > and it guessed wrong three rounds running.
- *
- * The other 24 tools still send exactly that: an open `params` object and a
- * sentence promising the error will explain the shape. `normalizeParams.ts`
- * exists to catch the five misses that produced most often. This renders the
- * field names instead, from the SAME schema `validateParams` enforces, so what
- * the model reads can never drift from what the validator accepts.
- */
-
-/** `{ url*, alt?, caption? }` — `*` marks required. */
-export function signature(schema, depth = 0) {
-  if (!schema || typeof schema !== "object") return "any";
-  const s = schema;
-  if (Array.isArray(s.anyOf)) {
-    return s.anyOf.map(b => signature(b, depth)).join(" | ");
-  }
-  if (s.type === "array") {
-    const cap = typeof s.maxItems === "number" ? ` ≤${s.maxItems}` : "";
-    return `[${signature(s.items, depth + 1)}${cap}]`;
-  }
-  if (s.type === "object" || s.properties) {
-    const props = s.properties ?? {};
-    const names = Object.keys(props);
-    if (!names.length) return "{…}";
-    // Three levels is every field a model has to name; below that lives the
-    // action's `dispatch`, which the prose already describes.
-    if (depth >= 3) return "{…}";
-    const required = new Set(s.required ?? []);
-    const fields = names.map(name => {
-      const inner = signature(props[name], depth + 1);
-      const mark = required.has(name) ? "*" : "";
-      return inner === "any" ? `${name}${mark}` : `${name}${mark}: ${inner}`;
-    });
-    return `{ ${fields.join(", ")} }`;
-  }
-  if (Array.isArray(s.enum)) return s.enum.map(v => `"${String(v)}"`).join("|");
-  return typeof s.type === "string" ? s.type : "any";
-}
-
-/**
- * The `params:` line under an action, or "" when the action takes none.
- *
- * Actions with no declared properties (71 of 192) get nothing rather than an
- * empty `{}` — a line that says the action takes no arguments is already said
- * by its absence, and 71 of them would be pure noise.
- */
-export function paramsLine(schema) {
-  const props = schema?.properties;
-  if (!props || Object.keys(props).length === 0) return "";
-  return `params: ${signature(schema)}`;
-}
-//# sourceMappingURL=signature.js.map
+"use strict";export function signature(n,e=0){if(!n||typeof n!=="object")return"any";const r=n;if(Array.isArray(r.anyOf)){return r.anyOf.map(t=>signature(t,e)).join(" | ")}if(r.type==="array"){const t=typeof r.maxItems==="number"?` \u2264${r.maxItems}`:"";return`[${signature(r.items,e+1)}${t}]`}if(r.type==="object"||r.properties){const t=r.properties??{};const s=Object.keys(t);if(!s.length)return"{\u2026}";if(e>=3)return"{\u2026}";const u=new Set(r.required??[]);const a=s.map(i=>{const o=signature(t[i],e+1);const p=u.has(i)?"*":"";return o==="any"?`${i}${p}`:`${i}${p}: ${o}`});return`{ ${a.join(", ")} }`}if(Array.isArray(r.enum))return r.enum.map(t=>`"${String(t)}"`).join("|");return typeof r.type==="string"?r.type:"any"}export function paramsLine(n){const e=n?.properties;if(!e||Object.keys(e).length===0)return"";return`params: ${signature(n)}`}

package/lib/module/catalog/snapshotReads.js

@@ -1,259 +1 @@
-"use strict";
-
-/**
- * The other half of Buoy's read surface.
- *
- * Every Buoy adapter answers in two ways, and only one of them is an action.
- * `listQueries`, `async.getAllKeys` and `getState` are actions; console
- * entries, network events, the route sitemap and the env table are NOT — they
- * exist only inside the adapter's `getSnapshot`, which the sync layer polls and
- * puts on the wire for MCP's `get_snapshot` / `get_console` /
- * `get_network_requests` / `get_routes`.
- *
- * `dispatchToolAction` reaches only the first half. So an agent driving tools
- * on the device could read storage, stores and the query cache, and was blind
- * to the app's logs, its HTTP traffic, its routes and its environment — the
- * four things a QA person is most likely to ask about, and two of which the
- * empty state advertises as suggestions. The catalog even tells the model to
- * "use get_console", which is an MCP tool that does not exist on this path.
- *
- * This module closes that half. `getSnapshot` is a RESERVED action name: the
- * engine intercepts it before dispatch and routes it to the registry's
- * snapshot reader instead.
- *
- * WHY THE PROJECTIONS. A raw snapshot is the heaviest payload in the system —
- * measured at ~1.1KB per console entry with up to 1000 retained, and network
- * events carry bodies. Handing that to a token-metered model truncated at 24K
- * chars produces a torn JSON object and a wasted turn. Each projector here
- * does what the matching MCP tool does: newest-first, filtered, and only the
- * fields worth spending tokens on. Bodies stay behind `network.getEventBody`,
- * which already exists.
- */
-
-import { isSelfTraffic } from "../policy/redact";
-
-/** Reserved action name. Not a real adapter action anywhere. */
-export const SNAPSHOT_ACTION = "getSnapshot";
-const SEVERITY = {
-  verbose: 0,
-  debug: 0,
-  trace: 0,
-  log: 1,
-  info: 1,
-  warn: 2,
-  warning: 2,
-  error: 3
-};
-function asArray(value) {
-  return Array.isArray(value) ? value : [];
-}
-function rec(value) {
-  return value && typeof value === "object" ? value : {};
-}
-function num(value, fallback) {
-  return typeof value === "number" && value > 0 ? value : fallback;
-}
-
-/**
- * The newest N. Every Buoy event store (BaseEventStore, which network,
- * console, sentry and route events all sit on) PREPENDS, so the list arrives
- * newest-FIRST and the newest N are the first N. An earlier version took the
- * LAST N here — the oldest — so once more than N were captured, RIGHT NOW
- * showed the stalest requests and a crash that had just happened fell out of
- * the console projection. Order is preserved: newest first.
- */
-function newest(items, limit) {
-  return items.length <= limit ? items : items.slice(0, limit);
-}
-function projectConsole(payload, params) {
-  const entries = asArray(rec(payload).entries);
-  const minSeverity = typeof params.level === "string" ? SEVERITY[params.level] ?? 0 : 0;
-  const pattern = typeof params.pattern === "string" ? params.pattern.toLowerCase() : undefined;
-  const matched = entries.filter(raw => {
-    const e = rec(raw);
-    const level = String(e.level ?? "log");
-    if ((SEVERITY[level] ?? 1) < minSeverity) return false;
-    if (!pattern) return true;
-    return String(e.message ?? "").toLowerCase().includes(pattern);
-  });
-  const shown = newest(matched, num(params.limit, 40));
-  return {
-    totalCaptured: entries.length,
-    shown: shown.length,
-    // Pre-rendered message only. Full args and stacks stay on the device;
-    // `get_snapshot('console')` is the MCP escape hatch and has no equivalent
-    // here on purpose — it is the payload that blows the prompt.
-    entries: shown.map(raw => {
-      const e = rec(raw);
-      return {
-        at: e.timestamp,
-        level: e.level,
-        message: e.message
-      };
-    })
-  };
-}
-function projectNetwork(payload, params) {
-  const p = rec(payload);
-  // The agent's own POSTs to the model endpoint are dropped HERE, not only by
-  // the engine's later pass, so the counts the model is given match the rows
-  // it is given. Reporting "4 requests" above a list of 3 is its own small lie.
-  const events = asArray(p.events).filter(raw => !isSelfTraffic(rec(raw).url));
-  const failedOnly = params.failedOnly === true;
-  const pattern = typeof params.pattern === "string" ? params.pattern.toLowerCase() : undefined;
-  const matched = events.filter(raw => {
-    const e = rec(raw);
-    const status = typeof e.status === "number" ? e.status : undefined;
-    if (failedOnly && !(e.error || status !== undefined && status >= 400)) return false;
-    if (!pattern) return true;
-    return String(e.url ?? "").toLowerCase().includes(pattern);
-  });
-  const shown = newest(matched, num(params.limit, 25));
-  return {
-    totalCaptured: events.length,
-    shown: shown.length,
-    // No bodies. `network.getEventBody` fetches one on demand, which is the
-    // difference between a readable list and a truncated blob.
-    requests: shown.map(raw => {
-      const e = rec(raw);
-      return {
-        id: e.id,
-        method: e.method,
-        url: e.url,
-        status: e.status,
-        durationMs: e.duration,
-        error: e.error,
-        // An override rule served this response. Carried through so neither
-        // the model nor the receipt can present mocked data as live.
-        overridden: e.override ? true : undefined
-      };
-    }),
-    hint: "Read one response with network.getEventBody({ id })."
-  };
-}
-
-/**
- * The sitemap arrives as a NESTED RouteInfo tree, not a list. A model asked
- * "what screens exist" needs the leaf paths, so flatten and keep the paths.
- */
-function flattenRoutes(nodes, out = [], cap = 80) {
-  for (const raw of asArray(nodes)) {
-    if (out.length >= cap) break;
-    const node = rec(raw);
-    if (node.isInternal !== true && typeof node.path === "string") out.push(node.path);
-    flattenRoutes(node.children, out, cap);
-  }
-  return out;
-}
-function projectRoutes(payload, params) {
-  const p = rec(payload);
-  const sitemap = rec(p.sitemap);
-
-  // Route events are stored newest-FIRST, and a stack item's screen path is
-  // `pathname` (`path` does not exist on it). Getting either wrong is not a
-  // crash, it is a confident wrong answer: an early version of this read told
-  // the user they were on the Pokedex while the shop was on screen.
-  const stackRaw = asArray(p.stack).map(rec);
-  const eventsRaw = asArray(p.events).map(rec);
-  const stack = stackRaw.map(s => ({
-    name: s.name,
-    path: s.pathname,
-    focused: s.isFocused === true ? true : undefined
-  }));
-  const recentNavigations = eventsRaw.slice(0, num(params.limit, 15)).map(e => ({
-    at: e.timestamp,
-    from: e.previousPathname,
-    to: e.pathname
-  }));
-
-  // Mirrors the adapter's own `getCurrentRoute`: the newest navigation event
-  // wins, and with no history yet — a freshly launched app — the FOCUSED stack
-  // item does, which is not always the last one in the array.
-  const focused = stackRaw.find(s => s.isFocused === true) ?? stackRaw[stackRaw.length - 1];
-  const currentRoute = eventsRaw.length ? {
-    path: eventsRaw[0].pathname,
-    source: "event"
-  } : typeof focused?.pathname === "string" ? {
-    path: focused.pathname,
-    source: "stack"
-  } : undefined;
-  return {
-    currentRoute,
-    stack,
-    routes: flattenRoutes(sitemap.routes),
-    sitemapSource: sitemap.source,
-    recentNavigations
-  };
-}
-function projectEnv(payload) {
-  const p = rec(payload);
-  return {
-    env: p.env ?? p.variables,
-    requiredEnvVars: p.requiredEnvVars
-  };
-}
-
-/**
- * Sentry envelopes are the fattest payloads any Buoy tool captures (a single
- * event with breadcrumbs is tens of KB), so only the item SUMMARIES travel —
- * the one-line human read the adapter already computes. The status field is
- * load-bearing: an empty list with "sdk-not-found" means the app has no
- * Sentry client, which is a different answer from "nothing errored".
- */
-function projectSentry(payload, params) {
-  const p = rec(payload);
-  const envelopes = asArray(p.envelopes).map(rec);
-  const typeFilter = typeof params.type === "string" ? params.type : undefined;
-  const pattern = typeof params.pattern === "string" ? params.pattern.toLowerCase() : undefined;
-  const matched = envelopes.filter(e => {
-    const items = asArray(e.items).map(rec);
-    if (typeFilter && !items.some(i => i.type === typeFilter)) return false;
-    if (!pattern) return true;
-    return items.some(i => String(i.summary ?? "").toLowerCase().includes(pattern));
-  });
-  const shown = newest(matched, num(params.limit, 10));
-  return {
-    status: p.status,
-    totalCaptured: envelopes.length,
-    shown: shown.length,
-    envelopes: shown.map(e => ({
-      id: e.id,
-      at: e.timestamp,
-      origin: e.origin,
-      eventId: e.eventId,
-      totalBytes: e.totalBytes,
-      items: asArray(e.items).map(raw => {
-        const i = rec(raw);
-        return {
-          type: i.type,
-          summary: i.summary,
-          spanCount: i.spanCount,
-          bytes: i.bytes
-        };
-      })
-    })),
-    hint: "For crash stack traces, console's [FATAL] entries carry the component stacks."
-  };
-}
-
-/**
- * Trim one snapshot to something worth sending. Unknown tools pass through —
- * the engine's 24K truncation is still the backstop.
- */
-export function projectSnapshot(toolId, payload, params = {}) {
-  switch (toolId) {
-    case "console":
-      return projectConsole(payload, params);
-    case "network":
-      return projectNetwork(payload, params);
-    case "route-events":
-      return projectRoutes(payload, params);
-    case "env":
-      return projectEnv(payload);
-    case "sentry":
-      return projectSentry(payload, params);
-    default:
-      return payload;
-  }
-}
-//# sourceMappingURL=snapshotReads.js.map
+"use strict";import{isSelfTraffic as y}from"../policy/redact";export const SNAPSHOT_ACTION="getSnapshot";const m={verbose:0,debug:0,trace:0,log:1,info:1,warn:2,warning:2,error:3};function l(e){return Array.isArray(e)?e:[]}function u(e){return e&&typeof e==="object"?e:{}}function d(e,t){return typeof e==="number"&&e>0?e:t}function h(e,t){return e.length<=t?e:e.slice(0,t)}function v(e,t){const o=l(u(e).entries);const c=typeof t.level==="string"?m[t.level]??0:0;const i=typeof t.pattern==="string"?t.pattern.toLowerCase():void 0;const a=o.filter(f=>{const n=u(f);const r=String(n.level??"log");if((m[r]??1)<c)return false;if(!i)return true;return String(n.message??"").toLowerCase().includes(i)});const p=h(a,d(t.limit,40));return{totalCaptured:o.length,shown:p.length,entries:p.map(f=>{const n=u(f);return{at:n.timestamp,level:n.level,message:n.message}})}}function w(e,t){const o=u(e);const c=l(o.events).filter(n=>!y(u(n).url));const i=t.failedOnly===true;const a=typeof t.pattern==="string"?t.pattern.toLowerCase():void 0;const p=c.filter(n=>{const r=u(n);const s=typeof r.status==="number"?r.status:void 0;if(i&&!(r.error||s!==void 0&&s>=400))return false;if(!a)return true;return String(r.url??"").toLowerCase().includes(a)});const f=h(p,d(t.limit,25));return{totalCaptured:c.length,shown:f.length,requests:f.map(n=>{const r=u(n);return{id:r.id,method:r.method,url:r.url,status:r.status,durationMs:r.duration,error:r.error,overridden:r.override?true:void 0}}),hint:"Read one response with network.getEventBody({ id })."}}function g(e,t=[],o=80){for(const c of l(e)){if(t.length>=o)break;const i=u(c);if(i.isInternal!==true&&typeof i.path==="string")t.push(i.path);g(i.children,t,o)}return t}function C(e,t){const o=u(e);const c=u(o.sitemap);const i=l(o.stack).map(u);const a=l(o.events).map(u);const p=i.map(s=>({name:s.name,path:s.pathname,focused:s.isFocused===true?true:void 0}));const f=a.slice(0,d(t.limit,15)).map(s=>({at:s.timestamp,from:s.previousPathname,to:s.pathname}));const n=i.find(s=>s.isFocused===true)??i[i.length-1];const r=a.length?{path:a[0].pathname,source:"event"}:typeof n?.pathname==="string"?{path:n.pathname,source:"stack"}:void 0;return{currentRoute:r,stack:p,routes:g(c.routes),sitemapSource:c.source,recentNavigations:f}}function S(e){const t=u(e);return{env:t.env??t.variables,requiredEnvVars:t.requiredEnvVars}}function k(e,t){const o=u(e);const c=l(o.envelopes).map(u);const i=typeof t.type==="string"?t.type:void 0;const a=typeof t.pattern==="string"?t.pattern.toLowerCase():void 0;const p=c.filter(n=>{const r=l(n.items).map(u);if(i&&!r.some(s=>s.type===i))return false;if(!a)return true;return r.some(s=>String(s.summary??"").toLowerCase().includes(a))});const f=h(p,d(t.limit,10));return{status:o.status,totalCaptured:c.length,shown:f.length,envelopes:f.map(n=>({id:n.id,at:n.timestamp,origin:n.origin,eventId:n.eventId,totalBytes:n.totalBytes,items:l(n.items).map(r=>{const s=u(r);return{type:s.type,summary:s.summary,spanCount:s.spanCount,bytes:s.bytes}})})),hint:"For crash stack traces, console's [FATAL] entries carry the component stacks."}}export function projectSnapshot(e,t,o={}){switch(e){case"console":return v(t,o);case"network":return w(t,o);case"route-events":return C(t,o);case"env":return S(t);case"sentry":return k(t,o);default:return t}}