mixdog 1.2.3 → 1.2.4

package/src/runtime/computer-bridge/action-schema.mjs

@@ -155,198 +155,198 @@ const captureProperties = {
 // schema below derives its `input` from it. It is not part of the advertised
 // root: provider tool schemas must be an object root without anyOf/oneOf/allOf.
 export const COMPUTER_ACTION_CONTRACTS = [
-    branch(
-      'wait_for_user',
-      input({
-        timeout_ms: {
+  branch(
+    'wait_for_user',
+    input({
+      timeout_ms: {
+        type: 'integer',
+        minimum: 0,
+        maximum: 120000,
+        description: 'Wait for manual or host-configured idle resume; default 60000.',
+      },
+    }),
+    false
+  ),
+  branch(
+    'list',
+    input(
+      {
+        kind: {
+          type: 'string',
+          enum: ['windows', 'apps', 'history'],
+          description:
+            "history returns this session's own executed commands with their verdicts, for reviewing what a run already did.",
+        },
+        query: {
+          type: 'string',
+          minLength: 1,
+          maxLength: MAX_TARGET_TOKEN_LENGTH,
+          description:
+            'With kind="apps", also lists installed apps whose name or id matches, including apps that are not running.',
+        },
+        limit: {
           type: 'integer',
-          minimum: 0,
-          maximum: 120000,
-          description: 'Wait for manual or host-configured idle resume; default 60000.',
+          minimum: 1,
+          maximum: 60,
+          description: 'kind="history" only: how many of the most recent records to return; default 60.',
         },
-      }),
-      false
-    ),
-    branch(
-      'list',
-      input(
-        {
-          kind: {
-            type: 'string',
-            enum: ['windows', 'apps', 'history'],
-            description:
-              "history returns this session's own executed commands with their verdicts, for reviewing what a run already did.",
-          },
-          query: {
-            type: 'string',
-            minLength: 1,
-            maxLength: MAX_TARGET_TOKEN_LENGTH,
-            description:
-              'With kind="apps", also lists installed apps whose name or id matches, including apps that are not running.',
-          },
-          limit: {
-            type: 'integer',
-            minimum: 1,
-            maximum: 60,
-            description: 'kind="history" only: how many of the most recent records to return; default 60.',
-          },
+      },
+      ['kind']
+    )
+  ),
+  branch(
+    'diagnose',
+    input({
+      ...windowTarget,
+      ocr_language: {
+        ...ocrLanguage,
+        description: 'Optional OS OCR language tag to verify, e.g. ko or en-US.',
+      },
+    }),
+    false
+  ),
+  branch('capture', input(captureProperties), false),
+  branch(
+    'act',
+    input(
+      {
+        ...windowTarget,
+        ...appTarget,
+        frame_id: {
+          type: 'string',
+          minLength: 1,
+          maxLength: MAX_TARGET_TOKEN_LENGTH,
+          description: 'Latest unexpired frame id shared by coordinate actions in this act call.',
         },
-        ['kind']
-      )
-    ),
-    branch(
-      'diagnose',
-      input({
+        actions: {
+          type: 'array',
+          items: COMPUTER_CORE_ACTION_SCHEMA,
+          minItems: 1,
+          maxItems: 6,
+          description:
+            'First: an input action. Then type/key/wait reusing focus, or click/double_click/triple_click/scroll/set_value/type/key on a ref from the same observation (no marks or coordinates). Waits total ≤10s; transition/failure or a changed, disabled, or off-screen ref stops the rest.',
+        },
+        observe: {
+          type: 'string',
+          enum: ['state', 'ax'],
+          description:
+            'Returned observation. Default: ax when the first action targets a ref (full state if the tree is unusable), otherwise state. state includes the frame for coordinates or visual checks; ax skips the screenshot.',
+        },
+        ...delivery,
+      },
+      ['actions']
+    )
+  ),
+  branch(
+    'window',
+    input(
+      {
         ...windowTarget,
-        ocr_language: {
-          ...ocrLanguage,
-          description: 'Optional OS OCR language tag to verify, e.g. ko or en-US.',
+        ...appTarget,
+        operation: {
+          type: 'string',
+          enum: ['focus', 'move', 'minimize', 'maximize', 'restore', 'close', 'terminate'],
         },
-      }),
-      false
-    ),
-    branch('capture', input(captureProperties), false),
-    branch(
-      'act',
-      input(
-        {
-          ...windowTarget,
-          ...appTarget,
-          frame_id: {
-            type: 'string',
-            minLength: 1,
-            maxLength: MAX_TARGET_TOKEN_LENGTH,
-            description: 'Latest unexpired frame id shared by coordinate actions in this act call.',
-          },
-          actions: {
-            type: 'array',
-            items: COMPUTER_CORE_ACTION_SCHEMA,
-            minItems: 1,
-            maxItems: 6,
-            description:
-              'First: an input action. Then type/key/wait reusing focus, or click/double_click/triple_click/scroll/set_value/type/key on a ref from the same observation (no marks or coordinates). Waits total ≤10s; transition/failure or a changed, disabled, or off-screen ref stops the rest.',
-          },
-          observe: {
-            type: 'string',
-            enum: ['state', 'ax'],
-            description:
-              'Returned observation. Default: ax when the first action targets a ref (full state if the tree is unusable), otherwise state. state includes the frame for coordinates or visual checks; ax skips the screenshot.',
-          },
-          ...delivery,
+        confirm: {
+          type: 'string',
+          enum: ['terminate'],
+          description:
+            'terminate only, and required for it: kills the process behind the window, losing unsaved work. Ask the user first.',
         },
-        ['actions']
-      )
-    ),
-    branch(
-      'window',
-      input(
-        {
-          ...windowTarget,
-          ...appTarget,
-          operation: {
-            type: 'string',
-            enum: ['focus', 'move', 'minimize', 'maximize', 'restore', 'close', 'terminate'],
-          },
-          confirm: {
-            type: 'string',
-            enum: ['terminate'],
-            description:
-              'terminate only, and required for it: kills the process behind the window, losing unsaved work. Ask the user first.',
-          },
-          x: {
-            type: 'integer',
-            description: 'move only: new left edge; y the top; width/height an optional new size.',
-          },
-          y: { type: 'integer' },
-          width: { type: 'integer', minimum: 1 },
-          height: { type: 'integer', minimum: 1 },
+        x: {
+          type: 'integer',
+          description: 'move only: new left edge; y the top; width/height an optional new size.',
         },
-        ['operation']
-      )
-    ),
-    branch(
-      'menu',
-      input(
-        {
-          ...windowTarget,
-          ...appTarget,
-          path: {
-            type: 'array',
-            items: { type: 'string', minLength: 1, maxLength: 512 },
-            minItems: 1,
-            maxItems: 8,
-            description:
-              'Exact labels from the bar down, e.g. ["File","Save As"]. Missing, ambiguous, or disabled entries fail closed.',
-          },
+        y: { type: 'integer' },
+        width: { type: 'integer', minimum: 1 },
+        height: { type: 'integer', minimum: 1 },
+      },
+      ['operation']
+    )
+  ),
+  branch(
+    'menu',
+    input(
+      {
+        ...windowTarget,
+        ...appTarget,
+        path: {
+          type: 'array',
+          items: { type: 'string', minLength: 1, maxLength: 512 },
+          minItems: 1,
+          maxItems: 8,
+          description:
+            'Exact labels from the bar down, e.g. ["File","Save As"]. Missing, ambiguous, or disabled entries fail closed.',
         },
-        ['path']
-      )
-    ),
-    branch(
-      'verify',
-      input(
-        {
-          ...windowTarget,
-          ...appTarget,
-          expect: {
-            type: 'array',
-            minItems: 1,
-            maxItems: 8,
-            description: 'AND-combined predicates. Reads state only: no pixels, and prior refs stay valid.',
-            items: {
-              type: 'object',
-              properties: {
-                present: {
-                  type: 'string',
-                  maxLength: MAX_TARGET_TOKEN_LENGTH,
-                  description: 'Text an element name or value must contain.',
-                },
-                absent: { type: 'string', maxLength: MAX_TARGET_TOKEN_LENGTH },
-                title_contains: { type: 'string', maxLength: MAX_TARGET_TOKEN_LENGTH },
-                window_exists: { type: 'boolean' },
+      },
+      ['path']
+    )
+  ),
+  branch(
+    'verify',
+    input(
+      {
+        ...windowTarget,
+        ...appTarget,
+        expect: {
+          type: 'array',
+          minItems: 1,
+          maxItems: 8,
+          description: 'AND-combined predicates. Reads state only: no pixels, and prior refs stay valid.',
+          items: {
+            type: 'object',
+            properties: {
+              present: {
+                type: 'string',
+                maxLength: MAX_TARGET_TOKEN_LENGTH,
+                description: 'Text an element name or value must contain.',
               },
-              additionalProperties: false,
+              absent: { type: 'string', maxLength: MAX_TARGET_TOKEN_LENGTH },
+              title_contains: { type: 'string', maxLength: MAX_TARGET_TOKEN_LENGTH },
+              window_exists: { type: 'boolean' },
             },
-          },
-          timeout_ms: { type: 'integer', minimum: 0, maximum: 30000 },
-          stable_samples: {
-            type: 'integer',
-            minimum: 1,
-            maximum: 5,
-            description: 'Consecutive satisfied samples required. Default 2.',
+            additionalProperties: false,
           },
         },
-        ['expect']
-      )
-    ),
-    branch(
-      'clipboard',
-      input(
-        {
-          operation: { type: 'string', enum: ['read', 'write'] },
-          text: {
-            type: 'string',
-            maxLength: MAX_CLIPBOARD_TEXT_LENGTH,
-            description: 'Required for operation="write".',
-          },
+        timeout_ms: { type: 'integer', minimum: 0, maximum: 30000 },
+        stable_samples: {
+          type: 'integer',
+          minimum: 1,
+          maximum: 5,
+          description: 'Consecutive satisfied samples required. Default 2.',
         },
-        ['operation']
-      )
-    ),
-    branch(
-      'launch',
-      input(
-        {
-          app: {
-            type: 'string',
-            minLength: 1,
-            maxLength: MAX_TARGET_TOKEN_LENGTH,
-            description: 'Executable name, exact path, file, or URL.',
-          },
+      },
+      ['expect']
+    )
+  ),
+  branch(
+    'clipboard',
+    input(
+      {
+        operation: { type: 'string', enum: ['read', 'write'] },
+        text: {
+          type: 'string',
+          maxLength: MAX_CLIPBOARD_TEXT_LENGTH,
+          description: 'Required for operation="write".',
+        },
+      },
+      ['operation']
+    )
+  ),
+  branch(
+    'launch',
+    input(
+      {
+        app: {
+          type: 'string',
+          minLength: 1,
+          maxLength: MAX_TARGET_TOKEN_LENGTH,
+          description: 'Executable name, exact path, file, or URL.',
         },
-        ['app']
-      )
-    ),
+      },
+      ['app']
+    )
+  ),
 ];
 
 // A JSON Schema 2020-12 object root with no root combinator (Anthropic

package/src/runtime/computer-bridge/linux-readiness.d.mts

@@ -46,7 +46,5 @@ export function linuxComputerPreflight(options?: Record<string, unknown>): Promi
 export function resetLinuxComputerPreflightCache(): void;
 export const GNOME_EXTENSION_UUID: 'mixdog-windows@mixdog.io';
 export function gnomeExtensionDirs(env?: Record<string, string | undefined>): string[];
-export const FIX_MESSAGES: Readonly<Record<'allow_input' | 'relogin' | 'enable_window_helper' | 'install_package', string>>;
-export const REFUSAL_MESSAGE: string;
 export const CAPTURE_FALLBACK_NOTE: string;
 export function linuxCaptureFallback(options?: Record<string, unknown>): Promise<{ command: Record<string, unknown>; note: string } | null>;

package/src/runtime/computer-bridge/linux-readiness.mjs

@@ -32,22 +32,6 @@ export const CLIPBOARD_HINT =
 export const DISPLAY_HINT =
   'display_unavailable: no graphical session to drive; run inside a desktop session with DISPLAY (X11) or WAYLAND_DISPLAY set';
 
-/**
- * User-facing English phrases, one per fix a capability can ask for. They are
- * the keys of the UI catalogs (apps/desktop/src/renderer/locales), so a UI that
- * renders a capability's `fix` looks the phrase up there; `{{package}}` is the
- * capability's `package`. The tool result text for the model stays English.
- */
-export const FIX_MESSAGES = Object.freeze({
-  allow_input:
-    'Allow Mixdog to send keyboard and mouse input: grant access to /dev/uinput (add your user to the input group), then sign in again.',
-  relogin: 'Sign out and sign in again to finish setting up window control for Mixdog.',
-  enable_window_helper:
-    'Enable the Mixdog Windows extension so Mixdog can list and control windows (gnome-extensions enable mixdog-windows@mixdog.io).',
-  install_package: 'Install {{package}} to enable this Computer Use feature.',
-});
-export const REFUSAL_MESSAGE = 'Computer Use cannot run this action because a required capability is not ready.';
-
 /** Stable code of the whole refusal; the per-capability codes are the native ones. */
 export const CAPABILITY_UNAVAILABLE_CODE = 'computer_capability_unavailable';
 

package/src/runtime/effort-judge/judge-client.mjs

@@ -1,7 +1,8 @@
-// Effort-judge client. The judge loads on the first turn that uses it (a model
-// with Auto effort support) and then stays resident until the feature is
-// turned off or a model update replaces its files, so users who never send
-// such a turn never hold it in memory. A turn waits for a still-loading judge
+// Effort-judge client. The judge loads when a session that uses it (a model
+// with Auto effort support) is created or opened, or on the first judged turn,
+// and then stays resident until the feature is turned off or a model update
+// replaces its files, so users who never open such a session never hold it in
+// memory. A turn waits for a still-loading judge
 // only briefly; when the model is missing, still loading, slow, or failing,
 // `judgeTurn` returns a `skipped` reason and the turn keeps its default
 // effort. Decisions are logged locally (lengths and probabilities only, never
@@ -237,6 +238,22 @@ export function refreshEffortJudge() {
   });
 }
 
+/**
+ * Starts the judge worker ahead of the first judged turn so its cold load
+ * (about 2.3 s) is over by then. Fire-and-forget: it spawns nothing when the
+ * platform is unsupported, the model is missing or being installed, or a
+ * failed load is still inside LOAD_RETRY_MS. Returns whether a worker is
+ * running or loading.
+ */
+export function prewarmEffortJudge() {
+  if (!onnxRuntimeSupported() || installing) return false;
+  const dir = effortJudgeModelDir();
+  if (!effortJudgeAvailable(dir)) return false;
+  if (!worker && lastLoadError && Date.now() - lastLoadErrorAt < LOAD_RETRY_MS) return false;
+  ensureWorker(dir);
+  return true;
+}
+
 export function effortJudgeReady() {
   return ready;
 }

package/src/runtime/effort-judge/judge-worker.mjs

@@ -94,7 +94,10 @@ async function load() {
     executionProviders: ['cpu'],
     graphOptimizationLevel: 'all',
     // One judgment runs at a time; 4 threads halve the long-input tail
-    // (p95 ~280 -> ~110 ms measured) for a ~0.1 s burst per turn.
+    // (p95 ~280 -> ~110 ms measured) for a ~0.1 s burst per turn. More threads
+    // did not help where agents share the CPU: with 3-4 agent-like loads on
+    // 20 logical cores, 6 threads timed out 8-16% (3 loads) and 78-87% (4 loads) of judgments against
+    // 0-5% for 4, so the count stays fixed.
     intraOpNumThreads: 4,
   });
   const settings = calibration();

package/src/runtime/github/tool.mjs

@@ -1,4 +1,4 @@
-import { GITHUB_ACTIONS } from './contract.mjs';
+import { DEFAULT_GITHUB_HOSTNAME, GITHUB_ACTIONS } from './contract.mjs';
 import { executeGithubRequest } from './client.mjs';
 import { GITHUB_OUTPUT_MAX_BYTES } from '../shared/tool-output-limit.mjs';
 import { persistToolResultArtifactSync } from '../agent/orchestrator/session/tool-result-offload.mjs';
@@ -6,19 +6,19 @@ import { normalizeOutputPath } from '../agent/orchestrator/tools/builtin/path-ut
 
 // Model-facing reduction only; executeGithubRequest (also used by Desktop)
 // keeps the raw REST response, and the agent's copy is preserved as `raw`.
-const KEPT_URL_KEYS = new Set([
-  'html_url',
-  'browser_download_url',
-  'diff_url',
-  'patch_url',
-  'clone_url',
-  'ssh_url',
-  'tarball_url',
-  'zipball_url',
-  'target_url',
-  'details_url',
+// html_url is the one page link the model can hand to a user; the asset,
+// status and check links are the only other URLs not derivable from ids.
+const KEPT_URL_KEYS = new Set(['html_url', 'browser_download_url', 'target_url', 'details_url']);
+const DROPPED_KEYS = new Set([
+  'url',
+  'node_id',
+  '_links',
+  'gravatar_id',
+  'performed_via_github_app',
+  'author_association',
+  'sub_issues_summary',
+  'issue_dependencies_summary',
 ]);
-const DROPPED_KEYS = new Set(['node_id', '_links', 'gravatar_id']);
 const USER_KEYS = new Set([
   'user',
   'owner',
@@ -55,12 +55,22 @@ function reduceGithubData(value, key, parentKey) {
   if (isNestedRepository(key, parentKey) && typeof value.full_name === 'string') return value.full_name;
   const reduced = {};
   for (const [name, item] of Object.entries(value)) {
-    if (DROPPED_KEYS.has(name) || (name.endsWith('_url') && !KEPT_URL_KEYS.has(name))) continue;
+    if (DROPPED_KEYS.has(name) || name.endsWith('node_id') || (name.endsWith('_url') && !KEPT_URL_KEYS.has(name))) {
+      continue;
+    }
+    // Numbered objects (issues, PRs, milestones) are addressed by number; their id is noise.
+    if (name === 'id' && typeof value.number === 'number') continue;
+    if (name === 'reactions' && isEmptyReactions(item)) continue;
     reduced[name] = reduceGithubData(item, name, key);
   }
   return reduced;
 }
 
+function isEmptyReactions(value) {
+  if (!value || typeof value !== 'object' || Array.isArray(value)) return false;
+  return Object.values(value).every((item) => typeof item !== 'number' || item === 0);
+}
+
 const EXCERPT_MAX = 140;
 const day = (value) => (typeof value === 'string' ? value.slice(0, 10) : undefined);
 const login = (value) => userLogin(value) ?? (typeof value === 'string' ? value : undefined);
@@ -151,6 +161,89 @@ function projectList(action, data) {
   return project && Array.isArray(data) ? data.map(project) : undefined;
 }
 
+const BODY_MAX = 1500;
+
+function bodyText(body) {
+  if (typeof body !== 'string') return undefined;
+  const text = body.replace(/\r\n/g, '\n').trim();
+  if (!text) return undefined;
+  if (text.length <= BODY_MAX) return text;
+  const cut = /[\ud800-\udbff]/.test(text[BODY_MAX - 1]) ? BODY_MAX - 1 : BODY_MAX;
+  return `${text.slice(0, cut)}… ${text.length - cut} more chars in raw`;
+}
+
+function logins(list) {
+  const names = (Array.isArray(list) ? list : []).map(login).filter(Boolean);
+  return names.length ? names : undefined;
+}
+
+const stamp = (value) => (typeof value === 'string' ? value : undefined);
+const repoName = (repo) => (typeof repo === 'string' ? repo : repo?.full_name);
+
+// Single-item projections: the decision fields of one issue/PR. Nulls that
+// mean "not yet" (open PR closed_at) are dropped; `mergeable`/`mergeable_state`
+// stay as returned because null there means GitHub is still computing.
+const VIEW_PROJECTORS = {
+  'pr.view': (pr) => ({
+    number: pr.number,
+    title: pr.title,
+    state: pr.state,
+    draft: pr.draft,
+    merged: pr.merged,
+    mergeable: pr.mergeable,
+    mergeable_state: pr.mergeable_state,
+    user: login(pr.user),
+    labels: labelNames(pr.labels),
+    assignees: logins(pr.assignees),
+    requested_reviewers: logins(pr.requested_reviewers),
+    head: pr.head && { ref: pr.head.ref, sha: pr.head.sha, repo: repoName(pr.head.repo) },
+    base: pr.base && { ref: pr.base.ref, sha: pr.base.sha },
+    commits: pr.commits,
+    additions: pr.additions,
+    deletions: pr.deletions,
+    changed_files: pr.changed_files,
+    comments: pr.comments,
+    review_comments: pr.review_comments,
+    html_url: pr.html_url,
+    created_at: stamp(pr.created_at),
+    updated_at: stamp(pr.updated_at),
+    closed_at: stamp(pr.closed_at),
+    merged_at: stamp(pr.merged_at),
+    body: bodyText(pr.body),
+  }),
+  'issue.view': (issue) => ({
+    number: issue.number,
+    title: issue.title,
+    state: issue.state,
+    state_reason: issue.state_reason || undefined,
+    user: login(issue.user),
+    labels: labelNames(issue.labels),
+    assignees: logins(issue.assignees),
+    milestone: issue.milestone?.title,
+    comments: issue.comments,
+    html_url: issue.html_url,
+    created_at: stamp(issue.created_at),
+    updated_at: stamp(issue.updated_at),
+    closed_at: stamp(issue.closed_at),
+    body: bodyText(issue.body),
+  }),
+};
+
+function projectView(action, data) {
+  const project = VIEW_PROJECTORS[action];
+  return project && data && typeof data === 'object' && !Array.isArray(data) ? project(data) : undefined;
+}
+
+// Rows whose excerpt/description was shortened ("…") need raw for the rest.
+function rowsWereCut(rows) {
+  return rows.some(
+    (row) =>
+      row &&
+      typeof row === 'object' &&
+      Object.values(row).some((value) => typeof value === 'string' && value.endsWith('…'))
+  );
+}
+
 const fits = (text) => Buffer.byteLength(text, 'utf8') <= GITHUB_OUTPUT_MAX_BYTES;
 
 function clampStrings(value, max) {
@@ -303,7 +396,7 @@ export const GITHUB_TOOL_DEF = {
   },
   trigger: 'GitHub repos, issues, PRs, Actions, releases; not local Git history, diffs or staging (git).',
   description:
-    'GitHub repositories, issues, PRs and reviews, Actions/logs, releases, and notifications through the signed-in GitHub CLI. Local Git history/diffs/staging belong to git. One action per call; writes are serialized and never retried. Obtain user approval before writes; workflow runs and published releases may deploy. repo is owner/name; omit to resolve the current Project. Lists use page/limit and return compact rows (issues/PRs: number, title, state, author, updated, labels, 140-char excerpt; other lists: ids and status fields); open a row with its *.view action for bodies and details. Review/merge requires the current PR head sha. Install/connect in Extensions → Plugin → Git & GitHub. Results omit API-only URLs and node ids and name users/repos by login/full_name; `raw` is a readable file with the full response, and long results state what is shown.',
+    'GitHub repositories, issues, PRs and reviews, Actions/logs, releases, and notifications through the signed-in GitHub CLI. Local Git history/diffs/staging belong to git. One action per call; writes are serialized and never retried. Obtain user approval before writes; workflow runs and published releases may deploy. repo is owner/name; omit to resolve the current Project. Lists use page/limit and return compact rows (issues/PRs: number, title, state, author, updated, labels, 140-char excerpt; other lists: ids and status fields); open a row with its *.view action for bodies and details. Review/merge requires the current PR head sha. Install/connect in Extensions → Plugin → Git & GitHub. Results omit API-only URLs and node ids and name users/repos by login/full_name; issue/PR views keep decision fields and cap the body; `raw`, present whenever rows, text or fields were cut or dropped, is a readable file with the full response, and long results state what is shown.',
   inputSchema: {
     type: 'object',
     properties: {
@@ -357,9 +450,17 @@ export const GITHUB_TOOL_DEF = {
 export async function executeGithubTool(args, cwd, options = {}) {
   try {
     const result = await executeGithubRequest(args, cwd, options);
-    const reduced = { ...result, data: projectList(args.action, result.data) ?? reduceGithubData(result.data) };
+    const data =
+      projectList(args.action, result.data) ?? projectView(args.action, result.data) ?? reduceGithubData(result.data);
+    const reduced = { ...result, data };
+    if (reduced.hostname === DEFAULT_GITHUB_HOSTNAME) delete reduced.hostname;
     const text = JSON.stringify(reduced);
-    if (text === JSON.stringify(result) && fits(text)) return text;
+    // Lists point at raw only when a row or excerpt was cut; they are projected
+    // by design. Everything else needs raw whenever the reduction dropped data.
+    const lossy = GITHUB_ACTIONS[args.action]?.list
+      ? Array.isArray(data) && rowsWereCut(data)
+      : JSON.stringify(data) !== JSON.stringify(result.data);
+    if (!lossy && fits(text)) return text;
     const raw = persistToolResultArtifactSync({
       sessionId: options.sessionId,
       toolCallId: options.toolCallId,