@chatpanel/events 0.102.0 → 0.103.1

package/voice-speaker.js

@@ -1,98 +1 @@
-// WHOSE VOICE IS THIS — the gate that keeps a conversation to the person having it.
-//
-// In a voice conversation every finalized sentence is SENT, so anything the microphone hears
-// becomes a question: a television, a colleague at the next desk, someone answering their own
-// phone behind you. The engine transcribes them all perfectly and correctly, and the
-// assistant answers the room.
-//
-// The gateway already computes a 512-d speaker fingerprint per committed segment and clusters
-// it into a stable label for the session (`diarize-engine.js`). All that is missing is the
-// DECISION, which is this file: the first voice in a conversation is the person who started
-// it, and later sentences from a different voice are not their turn.
-//
-// THE FAILURE DIRECTION MATTERS MORE THAN THE FEATURE. An assistant that occasionally answers
-// the television is annoying; one that ignores YOU is broken, and from the outside the two
-// look identical — a mic that is open and going nowhere. So every uncertain case sends:
-//
-//   · no speaker on the final (diarization off, model missing, embedding failed) → send;
-//   · no primary enrolled yet → this is the first voice, enroll it and send;
-//   · the primary has not been heard for a long time → adopt whoever is talking now and
-//     send, because the phone may have been handed over, or the first voice may have been
-//     the television while the user was drawing breath.
-//
-// Only one case holds: a DIFFERENT voice, while the person having the conversation is still
-// in it. That is the case the user asked for and the only one we can be confident about.
-//
-// The engine's own honesty note applies here too: embeddings separate different speakers
-// well, but similar voices can merge — so this gate can let a very similar voice through. It
-// never claims to be security, only to keep the room out of the conversation.
-
-/** How long the primary must be silent before another voice may take over the conversation. */
-export const RE_ENROLL_MS = 45_000;
-
-/** A label the gateway pins to the microphone channel; never a guess, so always the primary. */
-export const PINNED_SELF = 'You';
-
-const labelOf = (speaker) => {
-  if (!speaker) return '';
-  if (typeof speaker === 'string') return speaker;
-  return String(speaker.label || speaker.id || '');
-};
-
-/**
- * A per-conversation speaker gate.
- *
- * @param {object} [opts]
- * @param {number} [opts.reEnrollMs]  silence after which another voice may take over
- * @param {() => number} [opts.now]   injected clock, so the rules are testable without waiting
- */
-export function createSpeakerGate({ reEnrollMs = RE_ENROLL_MS, now = Date.now } = {}) {
-  let primary = '';        // the label of the person having this conversation
-  let lastHeard = 0;       // when the primary last said something
-  let held = 0;            // sentences kept out, for the UI to report honestly
-
-  return {
-    /** The enrolled voice, or '' before anyone has spoken. */
-    primary: () => primary,
-    /** How many sentences this gate has held back. */
-    heldCount: () => held,
-
-    /**
-     * Should this finalized sentence become a turn?
-     *
-     * @param {{ speaker?: any }} [final]
-     * @returns {{ send: boolean, reason: 'no-speaker'|'enrolled'|'primary'|'adopted'|'other', speaker: string }}
-     */
-    admit(final = {}) {
-      const label = labelOf(final.speaker);
-      // Diarization is optional and fails open — a sentence with no speaker is always sent.
-      if (!label) return { send: true, reason: 'no-speaker', speaker: '' };
-
-      const t = now();
-      if (!primary) {
-        primary = label;
-        lastHeard = t;
-        return { send: true, reason: 'enrolled', speaker: label };
-      }
-      if (label === primary) {
-        lastHeard = t;
-        return { send: true, reason: 'primary', speaker: label };
-      }
-      // A different voice. Only take over when the conversation has clearly moved on.
-      if (t - lastHeard >= reEnrollMs) {
-        primary = label;
-        lastHeard = t;
-        return { send: true, reason: 'adopted', speaker: label };
-      }
-      held += 1;
-      return { send: false, reason: 'other', speaker: label };
-    },
-
-    /**
-     * Forget who was talking. Called when a conversation starts, and by a user who wants the
-     * gate to hear them again — a voice it merged or mislabelled must be recoverable without
-     * ending the session.
-     */
-    reset() { primary = ''; lastHeard = 0; held = 0; },
-  };
-}
+const i=45e3,d="You",l=r=>r?typeof r=="string"?r:String(r.label||r.id||""):"";function p({reEnrollMs:r=45e3,now:s=Date.now}={}){let t="",n=0,o=0;return{primary:()=>t,heldCount:()=>o,admit(u={}){const e=l(u.speaker);if(!e)return{send:!0,reason:"no-speaker",speaker:""};const a=s();return t?e===t?(n=a,{send:!0,reason:"primary",speaker:e}):a-n>=r?(t=e,n=a,{send:!0,reason:"adopted",speaker:e}):(o+=1,{send:!1,reason:"other",speaker:e}):(t=e,n=a,{send:!0,reason:"enrolled",speaker:e})},reset(){t="",n=0,o=0}}}export{d as PINNED_SELF,i as RE_ENROLL_MS,p as createSpeakerGate};

package/weather-tool.js

@@ -1,65 +1 @@
-// `weather` as a TOOL — the spec, the guidance, and the answer shape, without the fetch.
-//
-// The extension answers "how is the weather in X" with this rather than with a web search,
-// because weather sites are rendered by scripts and a fetched page is navigation and no
-// temperature — which is exactly what a desktop turn without this tool produced: "I couldn't
-// retrieve reliable live weather". The engine (`weather.js`: the wttr.in query, the parse,
-// the ambiguity check) is already shared; this is the tool every client arms in front of it.
-//
-// `fetchJson(url, { timeoutMs })` is injected: each host owns its network guard and applies
-// it to a URL a MODEL supplied — attacker-influenced by construction.
-
-import { getWeather } from './weather.js';
-
-export const WEATHER_TOOL_NAME = 'weather';
-
-export const WEATHER_TOOL_SYSTEM =
-  'For weather, call `weather` FIRST — it answers the whole question in one request. Only '
-  + 'fall back to web_search if it tells you to. Report the location it says it resolved, '
-  + 'because a bare town name can geocode to the wrong place.';
-
-export function weatherToolProvider({ fetchJson } = {}) {
-  if (typeof fetchJson !== 'function') throw new Error('weatherToolProvider: fetchJson required');
-  return {
-    specs: [
-      {
-        name: 'weather',
-        // A read, declared: the round runner overlaps reads and serialises everything it
-        // cannot classify, and "weather" is not a verb its name heuristic knows.
-        annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
-        description:
-          'Current conditions and a short forecast for one place, in a single request. Use this '
-          + 'for any weather question instead of searching. Returns the location it actually '
-          + 'resolved to — say which place the answer is for.',
-        parameters: {
-          type: 'object',
-          properties: {
-            location: {
-              type: 'string',
-              description: 'A place, as the user said it. Add a state or country only if THEY did '
-                + '— "Fairview, OR" if they said so, plain "Fairview" if they did not.',
-            },
-          },
-          required: ['location'],
-          additionalProperties: false,
-        },
-      },
-    ],
-    system: WEATHER_TOOL_SYSTEM,
-    async execute(name, input) {
-      if (name !== 'weather') return JSON.stringify({ error: `Unknown tool: ${name}` });
-      const location = String(input?.location || '').trim();
-      if (!location) return 'No location provided to weather.';
-      const got = await getWeather(location, { fetchJson });
-      // THE FALLBACK IS AN INSTRUCTION, not an error string. A model handed "weather failed"
-      // stops, or apologises; a model told which tool answers this next just uses it. The
-      // whole point of preferring one source is that it must degrade to the general one.
-      if (!got.ok) {
-        return `The weather service could not answer for "${location}" (${got.reason}). `
-          + `Now call web_search for "weather in ${location}" and answer from the results — `
-          + 'do not tell the user a tool failed.';
-      }
-      return { text: got.text, note: 'ChatPanel · wttr.in' };
-    },
-  };
-}
+import{getWeather as a}from"./weather.js";const l="weather",i="For weather, call `weather` FIRST — it answers the whole question in one request. Only fall back to web_search if it tells you to. Report the location it says it resolved, because a bare town name can geocode to the wrong place.";function c({fetchJson:o}={}){if(typeof o!="function")throw new Error("weatherToolProvider: fetchJson required");return{specs:[{name:"weather",annotations:{readOnlyHint:!0,idempotentHint:!0,openWorldHint:!0},description:"Current conditions and a short forecast for one place, in a single request. Use this for any weather question instead of searching. Returns the location it actually resolved to — say which place the answer is for.",parameters:{type:"object",properties:{location:{type:"string",description:'A place, as the user said it. Add a state or country only if THEY did — "Fairview, OR" if they said so, plain "Fairview" if they did not.'}},required:["location"],additionalProperties:!1}}],system:i,async execute(r,n){if(r!=="weather")return JSON.stringify({error:`Unknown tool: ${r}`});const e=String(n?.location||"").trim();if(!e)return"No location provided to weather.";const t=await a(e,{fetchJson:o});return t.ok?{text:t.text,note:"ChatPanel · wttr.in"}:`The weather service could not answer for "${e}" (${t.reason}). Now call web_search for "weather in ${e}" and answer from the results — do not tell the user a tool failed.`}}}export{l as WEATHER_TOOL_NAME,i as WEATHER_TOOL_SYSTEM,c as weatherToolProvider};

package/weather.js

@@ -1,211 +1,2 @@
-// Weather — one deterministic source, asked first, with search as the fallback.
-//
-// There was no weather capability at all, so every "how is the weather in X" went to
-// web_search and the model assembled an answer from whatever three sites it happened to
-// scrape. That is slow (several page fetches for one number), inconsistent between asks, and
-// it failed in the way scraping fails: a repeated search got refused by the loop guard, the
-// reworded retry returned nothing, and the model announced it had no access to weather.
-//
-// wttr.in answers the whole question in ONE request as JSON, which is why it is the default.
-// It is not authoritative and it is not always up, so the tool falls back to search rather
-// than pretending — but the common case costs one round trip and reads the same every time.
-//
-// THE GEOCODER IS THE KNOWN WEAKNESS, and it is handled by disclosure rather than cleverness.
-// "Issaquah" once resolved to somewhere obscure and the answer looked plausible and was for
-// the wrong place. wttr.in reports what it actually resolved to, so `formatWeather` always
-// prints that line: a wrong place is then visible instead of silent. Guessing a region on the
-// user's behalf would replace a visible error with an invisible one.
-//
-// Pure and platform-free, like every other module here: the fetch is INJECTED, so the
-// extension passes its SSRF-guarded secureFetch, the gateway passes its own, and neither has
-// a second copy of the parsing.
-
-export const WEATHER_HOST = 'https://wttr.in';
-
-/** Long enough for a cold CDN edge, short enough that a hung host still falls back. */
-export const WEATHER_TIMEOUT_MS = 7000;
-
-export class WeatherError extends Error {
-  constructor(code, message) { super(message); this.name = 'WeatherError'; this.code = code; }
-}
-
-/**
- * The URL for one location.
- *
- * The location goes in the PATH, so it is percent-encoded whole — a bare `encodeURIComponent`
- * of "Seattle, WA" gives `Seattle%2C%20WA`, which wttr.in reads correctly, while leaving the
- * comma raw would let a location containing `/` or `?` reshape the request. Never interpolate
- * an unencoded user string into a path.
- */
-export function weatherUrl(location, { host = WEATHER_HOST } = {}) {
-  const q = String(location ?? '').trim();
-  if (!q) throw new WeatherError('NO_LOCATION', 'a location is required');
-  if (q.length > 120) throw new WeatherError('BAD_LOCATION', 'location is too long to be one');
-  return `${host}/${encodeURIComponent(q)}?format=j1`;
-}
-
-const num = (v) => {
-  const n = Number(v);
-  return Number.isFinite(n) ? n : null;
-};
-const str = (v) => String(v ?? '').trim();
-// wttr.in nests half its scalars as [{ value: "…" }], which is a shape, not data.
-const firstValue = (v) => (Array.isArray(v) ? str(v[0]?.value) : str(v));
-
-/**
- * wttr.in's `format=j1` body → a shape that does not mention wttr.in.
- *
- * Returns null for anything unusable rather than throwing: a malformed body and an outage are
- * the same event to the caller, and both mean "fall back to search".
- */
-export function parseWeather(json, { query = '' } = {}) {
-  if (!json || typeof json !== 'object') return null;
-  const cur = Array.isArray(json.current_condition) ? json.current_condition[0] : null;
-  if (!cur) return null;
-  const tempC = num(cur.temp_C);
-  const tempF = num(cur.temp_F);
-  // A reading with no temperature is not a weather reading. Guarded because an error page
-  // served as JSON still parses into an object, and half-empty output read as real data is
-  // worse than no data.
-  if (tempC === null && tempF === null) return null;
-
-  const areaRaw = Array.isArray(json.nearest_area) ? json.nearest_area[0] : null;
-  const area = areaRaw ? {
-    name: firstValue(areaRaw.areaName),
-    region: firstValue(areaRaw.region),
-    country: firstValue(areaRaw.country),
-    lat: num(areaRaw.latitude),
-    lon: num(areaRaw.longitude),
-  } : null;
-
-  const days = (Array.isArray(json.weather) ? json.weather : []).slice(0, 3).map((d) => ({
-    date: str(d.date),
-    maxC: num(d.maxtempC),
-    maxF: num(d.maxtempF),
-    minC: num(d.mintempC),
-    minF: num(d.mintempF),
-    sunrise: str(d.astronomy?.[0]?.sunrise),
-    sunset: str(d.astronomy?.[0]?.sunset),
-    // The midday slot is the day's headline condition; the array is 3-hourly.
-    condition: firstValue(d.hourly?.[4]?.weatherDesc) || firstValue(d.hourly?.[0]?.weatherDesc),
-    chanceOfRain: num(d.hourly?.[4]?.chanceofrain),
-  }));
-
-  return {
-    query: str(query),
-    area,
-    now: {
-      tempC,
-      tempF,
-      feelsLikeC: num(cur.FeelsLikeC),
-      feelsLikeF: num(cur.FeelsLikeF),
-      condition: firstValue(cur.weatherDesc),
-      humidity: num(cur.humidity),
-      windKph: num(cur.windspeedKmph),
-      windMph: num(cur.windspeedMiles),
-      windDir: str(cur.winddir16Point),
-      precipMm: num(cur.precipMM),
-      uv: num(cur.uvIndex),
-      observedAt: str(cur.localObsDateTime),
-    },
-    days,
-  };
-}
-
-/** "Seattle, Washington, United States" — whatever of it exists, deduped. */
-export function areaLabel(area, fallback = '') {
-  if (!area) return fallback;
-  const parts = [area.name, area.region, area.country].map(str).filter(Boolean);
-  const seen = new Set();
-  const uniq = parts.filter((p) => {
-    const k = p.toLowerCase();
-    if (seen.has(k)) return false;
-    seen.add(k);
-    return true;
-  });
-  return uniq.join(', ') || fallback;
-}
-
-/**
- * True when the user named a bare place with no region or country.
- *
- * Not used to REWRITE the query — that is how you turn a wrong answer into a confidently
- * wrong one. It is used to add a line telling the model to say which place it got, so the
- * user can correct it. Dozens of towns are called Fairview.
- */
-export function isAmbiguousLocation(location) {
-  const q = str(location);
-  if (!q) return false;
-  return !/[,]/.test(q) && q.split(/\s+/).length <= 2;
-}
-
-const c2f = (c) => Math.round((c * 9) / 5 + 32);
-const temp = (c, f) => {
-  const cc = c === null && f !== null ? Math.round(((f - 32) * 5) / 9) : c;
-  const ff = f === null && c !== null ? c2f(c) : f;
-  if (cc === null && ff === null) return '';
-  if (cc === null) return `${ff}°F`;
-  if (ff === null) return `${cc}°C`;
-  return `${ff}°F / ${cc}°C`;
-};
-
-/** What the model reads. Compact — this is one fact, not a report. */
-export function formatWeather(w) {
-  if (!w) return '';
-  const where = areaLabel(w.area, w.query);
-  const lines = [];
-  // FIRST LINE NAMES THE PLACE IT ACTUALLY RESOLVED. The geocoder picks obscure matches for
-  // bare town names, and an answer that does not say where it is from cannot be caught.
-  lines.push(`Weather for ${where}${w.query && where.toLowerCase() !== w.query.toLowerCase() ? ` (asked: "${w.query}")` : ''}`);
-  const n = w.now;
-  const now = [`Now: ${temp(n.tempC, n.tempF)}`];
-  if (n.feelsLikeC !== null || n.feelsLikeF !== null) now.push(`feels like ${temp(n.feelsLikeC, n.feelsLikeF)}`);
-  if (n.condition) now.push(n.condition.toLowerCase());
-  lines.push(now.join(', '));
-  const detail = [];
-  if (n.humidity !== null) detail.push(`humidity ${n.humidity}%`);
-  if (n.windMph !== null || n.windKph !== null) {
-    detail.push(`wind ${n.windMph !== null ? `${n.windMph} mph` : `${n.windKph} km/h`}${n.windDir ? ` ${n.windDir}` : ''}`);
-  }
-  if (n.precipMm !== null) detail.push(`precip ${n.precipMm} mm`);
-  if (n.uv !== null) detail.push(`UV ${n.uv}`);
-  if (detail.length) lines.push(detail.join(' · '));
-  for (const d of w.days) {
-    const range = `${temp(d.maxC, d.maxF)} high / ${temp(d.minC, d.minF)} low`;
-    const rain = d.chanceOfRain !== null ? `, ${d.chanceOfRain}% rain` : '';
-    lines.push(`${d.date}: ${range}${d.condition ? `, ${d.condition.toLowerCase()}` : ''}${rain}`);
-  }
-  if (n.observedAt) lines.push(`Observed ${n.observedAt} local. Source: wttr.in`);
-  if (isAmbiguousLocation(w.query)) {
-    lines.push('The place name was ambiguous — TELL THE USER which location this is for, and '
-      + 'offer to re-check with a state or country if it is the wrong one.');
-  }
-  return lines.join('\n');
-}
-
-/**
- * Ask for one location's weather.
- *
- * @param fetchJson async (url, { timeoutMs }) -> parsed JSON. Injected, so the caller's
- *        SSRF-guarded fetch is the only one that runs and this module stays runnable in Node,
- *        a browser and a mobile runtime.
- * @returns { ok: true, weather, text } | { ok: false, reason }
- *
- * NEVER THROWS. Every failure is a reason the caller can hand to the model as "use search
- * instead", because an exception here would surface as a tool error and stop the turn on a
- * question the fallback can still answer.
- */
-export async function getWeather(location, { fetchJson, timeoutMs = WEATHER_TIMEOUT_MS, host = WEATHER_HOST } = {}) {
-  let url;
-  try { url = weatherUrl(location, { host }); } catch (e) { return { ok: false, reason: e.message }; }
-  if (typeof fetchJson !== 'function') return { ok: false, reason: 'no fetch was provided' };
-  let json;
-  try {
-    json = await fetchJson(url, { timeoutMs });
-  } catch (e) {
-    return { ok: false, reason: e?.message || 'the weather service did not answer' };
-  }
-  const weather = parseWeather(json, { query: location });
-  if (!weather) return { ok: false, reason: 'the weather service returned nothing usable' };
-  return { ok: true, weather, text: formatWeather(weather) };
-}
+const f="https://wttr.in",L=7e3;class m extends Error{constructor(r,n){super(n),this.name="WeatherError",this.code=r}}function d(e,{host:r=f}={}){const n=String(e??"").trim();if(!n)throw new m("NO_LOCATION","a location is required");if(n.length>120)throw new m("BAD_LOCATION","location is too long to be one");return`${r}/${encodeURIComponent(n)}?format=j1`}const s=e=>{const r=Number(e);return Number.isFinite(r)?r:null},l=e=>String(e??"").trim(),h=e=>Array.isArray(e)?l(e[0]?.value):l(e);function w(e,{query:r=""}={}){if(!e||typeof e!="object")return null;const n=Array.isArray(e.current_condition)?e.current_condition[0]:null;if(!n)return null;const t=s(n.temp_C),a=s(n.temp_F);if(t===null&&a===null)return null;const i=Array.isArray(e.nearest_area)?e.nearest_area[0]:null,o=i?{name:h(i.areaName),region:h(i.region),country:h(i.country),lat:s(i.latitude),lon:s(i.longitude)}:null,c=(Array.isArray(e.weather)?e.weather:[]).slice(0,3).map(u=>({date:l(u.date),maxC:s(u.maxtempC),maxF:s(u.maxtempF),minC:s(u.mintempC),minF:s(u.mintempF),sunrise:l(u.astronomy?.[0]?.sunrise),sunset:l(u.astronomy?.[0]?.sunset),condition:h(u.hourly?.[4]?.weatherDesc)||h(u.hourly?.[0]?.weatherDesc),chanceOfRain:s(u.hourly?.[4]?.chanceofrain)}));return{query:l(r),area:o,now:{tempC:t,tempF:a,feelsLikeC:s(n.FeelsLikeC),feelsLikeF:s(n.FeelsLikeF),condition:h(n.weatherDesc),humidity:s(n.humidity),windKph:s(n.windspeedKmph),windMph:s(n.windspeedMiles),windDir:l(n.winddir16Point),precipMm:s(n.precipMM),uv:s(n.uvIndex),observedAt:l(n.localObsDateTime)},days:c}}function y(e,r=""){if(!e)return r;const n=[e.name,e.region,e.country].map(l).filter(Boolean),t=new Set;return n.filter(i=>{const o=i.toLowerCase();return t.has(o)?!1:(t.add(o),!0)}).join(", ")||r}function $(e){const r=l(e);return r?!/[,]/.test(r)&&r.split(/\s+/).length<=2:!1}const C=e=>Math.round(e*9/5+32),p=(e,r)=>{const n=e===null&&r!==null?Math.round((r-32)*5/9):e,t=r===null&&e!==null?C(e):r;return n===null&&t===null?"":n===null?`${t}°F`:t===null?`${n}°C`:`${t}°F / ${n}°C`};function g(e){if(!e)return"";const r=y(e.area,e.query),n=[];n.push(`Weather for ${r}${e.query&&r.toLowerCase()!==e.query.toLowerCase()?` (asked: "${e.query}")`:""}`);const t=e.now,a=[`Now: ${p(t.tempC,t.tempF)}`];(t.feelsLikeC!==null||t.feelsLikeF!==null)&&a.push(`feels like ${p(t.feelsLikeC,t.feelsLikeF)}`),t.condition&&a.push(t.condition.toLowerCase()),n.push(a.join(", "));const i=[];t.humidity!==null&&i.push(`humidity ${t.humidity}%`),(t.windMph!==null||t.windKph!==null)&&i.push(`wind ${t.windMph!==null?`${t.windMph} mph`:`${t.windKph} km/h`}${t.windDir?` ${t.windDir}`:""}`),t.precipMm!==null&&i.push(`precip ${t.precipMm} mm`),t.uv!==null&&i.push(`UV ${t.uv}`),i.length&&n.push(i.join(" · "));for(const o of e.days){const c=`${p(o.maxC,o.maxF)} high / ${p(o.minC,o.minF)} low`,u=o.chanceOfRain!==null?`, ${o.chanceOfRain}% rain`:"";n.push(`${o.date}: ${c}${o.condition?`, ${o.condition.toLowerCase()}`:""}${u}`)}return t.observedAt&&n.push(`Observed ${t.observedAt} local. Source: wttr.in`),$(e.query)&&n.push("The place name was ambiguous — TELL THE USER which location this is for, and offer to re-check with a state or country if it is the wrong one."),n.join(`
+`)}async function x(e,{fetchJson:r,timeoutMs:n=7e3,host:t=f}={}){let a;try{a=d(e,{host:t})}catch(c){return{ok:!1,reason:c.message}}if(typeof r!="function")return{ok:!1,reason:"no fetch was provided"};let i;try{i=await r(a,{timeoutMs:n})}catch(c){return{ok:!1,reason:c?.message||"the weather service did not answer"}}const o=w(i,{query:e});return o?{ok:!0,weather:o,text:g(o)}:{ok:!1,reason:"the weather service returned nothing usable"}}export{f as WEATHER_HOST,L as WEATHER_TIMEOUT_MS,m as WeatherError,y as areaLabel,g as formatWeather,x as getWeather,$ as isAmbiguousLocation,w as parseWeather,d as weatherUrl};

package/web-search-tool.js

@@ -1,107 +1,19 @@
-// `web_search` as a TOOL — the spec, the guidance, and how results are put in front of a
-// model, without the search itself.
-//
-// Running a search is platform work: the extension fetches SERPs from a service worker
-// with DOMParser, the desktop's main process reads anchors with a tokenizer, a gateway that
-// grew one would use its own fetch. What is identical everywhere is what the model is told
-// the tool does, what it is told about citing, and how a result list becomes text it can
-// cite from — so that is what lives here, and `search` is injected.
-//
-// The citation rules are in the RESULT, not only in the system prompt, on purpose: the
-// model reads them at the moment it has sources in hand, which is when it is most likely to
-// follow them, and a turn that never searches never pays for them.
+const a="web_search",u='You can call web_search to look up current information from the web — prices, news, recent events, documentation, or anything time-sensitive or that may have changed since your training. Call it whenever the user asks about such things instead of guessing or saying you are unsure. Cite results inline as markdown links — e.g. ([1](https://…)) — never HTML, <sup>, or bare numbers, and finish with a "Sources" list of the links you used.',c=Object.freeze({name:a,annotations:{readOnlyHint:!0,idempotentHint:!0,openWorldHint:!0},description:"Search the web and return ranked result snippets with their source URLs. Use this for current events, live prices/quotes, news, product/library docs, or any fact you are unsure about or that may have changed since training — prefer it over guessing.",parameters:{type:"object",properties:{query:{type:"string",description:"The search query — a few keywords work best."}},required:["query"],additionalProperties:!1}});function l(e){if(!e?.results?.length){const t=(e?.engines||[]).join(", ");return`No web results for "${e?.query||""}"`+(t?` (searched: ${t}).`:".")+" This may mean the engine blocked the request rather than that nothing exists — do NOT conclude the information is unavailable. Try a shorter, more general query (drop dates and qualifiers), and tell the user they can enable another search engine in ChatPanel settings if it keeps failing."}const r=Array.isArray(e.engines)?e.engines:[],o=e.results.map(t=>`[${t.rank}] [${t.title}](${t.url})`).join(`
+`),s=e.results[0].url,n=`Web search results for "${e.query}" (engines: ${r.join(", ")}).
 
-export const WEB_SEARCH_TOOL_NAME = 'web_search';
+Citation rules: when a claim draws on a result below, cite it inline as a markdown link to that result's URL — e.g. ([1](${s})). Cite multiple sources as separate links: ([1](url)) ([3](url)). Do NOT output HTML, <sup>, or bare bracket numbers like [1] — every citation must be a clickable markdown link. Finish with a "Sources" section that repeats, as markdown links, each source you cited.
 
-export const WEB_SEARCH_TOOL_SYSTEM =
-  'You can call web_search to look up current information from the web — prices, news, recent '
-  + 'events, documentation, or anything time-sensitive or that may have changed since your training. '
-  + 'Call it whenever the user asks about such things instead of guessing or saying you are unsure. '
-  + 'Cite results inline as markdown links — e.g. ([1](https://…)) — never HTML, <sup>, or bare '
-  + 'numbers, and finish with a "Sources" list of the links you used.';
+Sources:
+${o}`,i=e.results.map(t=>`### [${t.rank}] ${t.title}
+<${t.url}>
 
-export const WEB_SEARCH_SPEC = Object.freeze({
-  name: WEB_SEARCH_TOOL_NAME,
-  annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
-  description:
-    'Search the web and return ranked result snippets with their source URLs. Use this for '
-    + 'current events, live prices/quotes, news, product/library docs, or any fact you are unsure '
-    + 'about or that may have changed since training — prefer it over guessing.',
-  parameters: {
-    type: 'object',
-    properties: {
-      query: { type: 'string', description: 'The search query — a few keywords work best.' },
-    },
-    required: ['query'],
-    additionalProperties: false,
-  },
-});
+${t.text}`).join(`
 
-/**
- * Flatten a search result into one readable blob the model can cite from.
- *
- * `res` is `{ query, engines: [ids], results: [{ rank, title, url, text }] }`. A citation
- * index of markdown links sits at the TOP so it survives truncation.
- */
-export function searchResultsToText(res) {
-  if (!res?.results?.length) {
-    // Name the engines HERE especially. The success path already lists them; the failure
-    // path did not, so "no web results" looked like "the web has nothing" when it usually
-    // means "the one engine we were allowed to ask returned nothing" — a search engine
-    // blocking us and a query with no answer are completely different problems, and the
-    // model cannot tell them apart without this.
-    const tried = (res?.engines || []).join(', ');
-    return `No web results for "${res?.query || ''}"`
-      + (tried ? ` (searched: ${tried}).` : '.')
-      + ' This may mean the engine blocked the request rather than that nothing exists —'
-      + ' do NOT conclude the information is unavailable. Try a shorter, more general query'
-      + ' (drop dates and qualifiers), and tell the user they can enable another search'
-      + ' engine in ChatPanel settings if it keeps failing.';
-  }
-  const engines = Array.isArray(res.engines) ? res.engines : [];
-  const sources = res.results.map((r) => `[${r.rank}] [${r.title}](${r.url})`).join('\n');
-  const example = res.results[0].url;
-  const head =
-    `Web search results for "${res.query}" (engines: ${engines.join(', ')}).\n\n`
-    + 'Citation rules: when a claim draws on a result below, cite it inline as a markdown '
-    + `link to that result's URL — e.g. ([1](${example})). Cite multiple sources as separate `
-    + 'links: ([1](url)) ([3](url)). Do NOT output HTML, <sup>, or bare bracket numbers like '
-    + '[1] — every citation must be a clickable markdown link. Finish with a "Sources" section '
-    + 'that repeats, as markdown links, each source you cited.\n\n'
-    + `Sources:\n${sources}`;
-  const body = res.results
-    .map((r) => `### [${r.rank}] ${r.title}\n<${r.url}>\n\n${r.text}`)
-    .join('\n\n---\n\n');
-  return `${head}\n\n---\nResult details:\n\n${body}`;
-}
+---
 
-/**
- * The tool provider — `{ specs, system, execute }` for `buildToolset`.
- *
- * @param search `(query) => Promise<{ query, engines, results }>` — the host's search. It
- *               may throw; the model gets the message rather than a dead turn.
- */
-export function webSearchToolProvider({ search } = {}) {
-  if (typeof search !== 'function') throw new Error('webSearchToolProvider: search required');
-  return {
-    specs: [WEB_SEARCH_SPEC],
-    system: WEB_SEARCH_TOOL_SYSTEM,
-    async execute(name, input) {
-      if (name !== WEB_SEARCH_TOOL_NAME) return JSON.stringify({ error: `Unknown tool: ${name}` });
-      const q = String(input?.query || '').trim();
-      if (!q) return 'No query provided to web_search.';
-      try {
-        const res = await search(q);
-        // Return an OBJECT so the step can name the ENGINE that actually served the results.
-        // "web_search" alone doesn't tell the user whether Startpage or DuckDuckGo answered —
-        // which matters, because engines differ in coverage, and because a CLI agent may have
-        // run its OWN search instead of this one. `note` becomes the step's badge; `text` is
-        // what the model reads, unchanged.
-        const engines = (res?.engines || []).join(', ');
-        return { text: searchResultsToText(res), note: engines ? `ChatPanel · ${engines}` : 'ChatPanel' };
-      } catch (e) {
-        return `web_search failed: ${e?.message || e}`;
-      }
-    },
-  };
-}
+`);return`${n}
+
+---
+Result details:
+
+${i}`}function h({search:e}={}){if(typeof e!="function")throw new Error("webSearchToolProvider: search required");return{specs:[c],system:u,async execute(r,o){if(r!==a)return JSON.stringify({error:`Unknown tool: ${r}`});const s=String(o?.query||"").trim();if(!s)return"No query provided to web_search.";try{const n=await e(s),i=(n?.engines||[]).join(", ");return{text:l(n),note:i?`ChatPanel · ${i}`:"ChatPanel"}}catch(n){return`web_search failed: ${n?.message||n}`}}}}export{c as WEB_SEARCH_SPEC,a as WEB_SEARCH_TOOL_NAME,u as WEB_SEARCH_TOOL_SYSTEM,l as searchResultsToText,h as webSearchToolProvider};