salmon-loop 0.3.2 → 0.5.0

package/dist/core/permission-gate/default-gate.js

@@ -93,19 +93,16 @@ class DefaultPermissionGate {
         };
     }
     mapAuthorizationDecision(decision) {
+        const source = decision.source === 'auto' || decision.source === 'allowlist' ? 'policy' : decision.source;
         if (decision.outcome === 'deny') {
-            return {
-                kind: 'deny',
-                reason: decision.reason ?? 'denied',
-                source: decision.source === 'auto' || decision.source === 'allowlist'
-                    ? 'policy'
-                    : decision.source,
-            };
+            return { kind: 'deny', reason: decision.reason ?? 'denied', source };
         }
         return {
             kind: 'allow',
             reason: decision.reason,
-            source: decision.source === 'auto' || decision.source === 'allowlist' ? 'policy' : decision.source,
+            source,
+            ttlMs: decision.ttlMs,
+            persist: decision.persist,
         };
     }
 }

package/dist/core/plan/storage.js

@@ -1,6 +1,7 @@
 import { randomBytes, createHash } from 'crypto';
 import path from 'path';
 import { mkdir, readFile, rename, stat, writeFile } from '../adapters/fs/node-fs.js';
+import { getLogger } from '../observability/logger.js';
 const SESSION_ID_RE = /^[a-zA-Z0-9_-]{6,64}$/;
 export function assertValidSessionId(sessionId) {
     if (!SESSION_ID_RE.test(sessionId)) {
@@ -31,8 +32,8 @@ async function resolveGitDir(repoRoot) {
         if (st.isDirectory())
             return dotGit;
     }
-    catch {
-        // ignore
+    catch (error) {
+        getLogger().debug(`[PlanStorage] .git stat failed: ${error instanceof Error ? error.message : String(error)}`);
     }
     try {
         const raw = await readFile(dotGit, 'utf-8');
@@ -43,7 +44,8 @@ async function resolveGitDir(repoRoot) {
         const gitdir = m[1].trim();
         return path.isAbsolute(gitdir) ? gitdir : path.resolve(repoRoot, gitdir);
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[PlanStorage] .git read failed: ${error instanceof Error ? error.message : String(error)}`);
         return null;
     }
 }
@@ -61,7 +63,8 @@ export async function ensureSalmonloopIgnored(repoRoot) {
     try {
         existing = await readFile(excludePath, 'utf-8');
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[PlanStorage] Failed to read git exclude file: ${error instanceof Error ? error.message : String(error)}`);
         existing = '';
     }
     if (existing.split('\n').some((l) => l.trim() === '.salmonloop/'))

package/dist/core/plugin/loader.js

@@ -1,7 +1,9 @@
 import { join } from 'path';
+import { pythonPlugin } from '../../languages/python/index.js';
 import { typescriptPlugin, tsxPlugin, javascriptPlugin } from '../../languages/typescript/index.js';
 import { readdir } from '../adapters/fs/node-fs.js';
 import { getLogger } from '../observability/logger.js';
+import { errorMessage } from '../utils/error.js';
 import { validateQueryPack } from './validator.js';
 // Import built-in plugins (Phase 1: explicit import)
 export class PluginLoader {
@@ -18,10 +20,11 @@ export class PluginLoader {
         try {
             // Phase 1: Manually register TypeScript/JavaScript plugins
             getLogger().debug('Loading built-in plugins...');
+            this.registerWithValidation(registry, pythonPlugin);
             this.registerWithValidation(registry, typescriptPlugin);
             this.registerWithValidation(registry, tsxPlugin);
             this.registerWithValidation(registry, javascriptPlugin);
-            getLogger().debug(`Plugins loaded: ${typescriptPlugin.meta.name}, ${tsxPlugin.meta.name}, ${javascriptPlugin.meta.name}`);
+            getLogger().debug(`Plugins loaded: ${pythonPlugin.meta.name}, ${typescriptPlugin.meta.name}, ${tsxPlugin.meta.name}, ${javascriptPlugin.meta.name}`);
             // Phase 2: Load user plugins from .salmonloop/languages/
             if (repoPath) {
                 await this.loadUserPlugins(registry, repoPath);
@@ -31,11 +34,11 @@ export class PluginLoader {
         catch (error) {
             // In test environment, we want to know why it failed
             if (process.env.NODE_ENV === 'test') {
-                const errorMsg = error instanceof Error ? error.message : String(error);
+                const errorMsg = errorMessage(error);
                 getLogger().error(`CRITICAL: Failed to load plugins: ${errorMsg}`);
                 throw error;
             }
-            getLogger().error(`Failed to load plugins: ${error instanceof Error ? error.message : String(error)}`);
+            getLogger().error(`Failed to load plugins: ${errorMessage(error)}`);
         }
     }
     /**
@@ -70,7 +73,7 @@ export class PluginLoader {
                 }
                 catch (err) {
                     if (err && typeof err === 'object' && 'code' in err && err.code !== 'ENOENT') {
-                        getLogger().warn(`Failed to load user plugin from ${dirName}: ${err instanceof Error ? (err instanceof Error ? err.message : String(err)) : String(err)}`);
+                        getLogger().warn(`Failed to load user plugin from ${dirName}: ${errorMessage(err)}`);
                     }
                 }
             }
@@ -78,7 +81,7 @@ export class PluginLoader {
         catch (err) {
             // Ignore if directory doesn't exist
             if (err && typeof err === 'object' && 'code' in err && err.code !== 'ENOENT') {
-                getLogger().debug(`Error scanning for user plugins: ${err instanceof Error ? err.message : String(err)}`);
+                getLogger().debug(`Error scanning for user plugins: ${errorMessage(err)}`);
             }
         }
     }

package/dist/core/prompts/registry.js

@@ -2,6 +2,7 @@ import { fileURLToPath } from 'node:url';
 import Handlebars from 'handlebars';
 import { z } from 'zod';
 import { readFile } from '../adapters/fs/node-fs.js';
+import { unwrapZodSchema } from '../utils/zod.js';
 const TEMPLATE_URLS = {
     'system/_tool_defs.hbs': new URL('./templates/system/_tool_defs.hbs', import.meta.url),
     'system/main_system.hbs': new URL('./templates/system/main_system.hbs', import.meta.url),
@@ -60,8 +61,8 @@ export class PromptRegistry {
         if (!url) {
             throw new Error(`Unknown prompt template path: ${relativePath}`);
         }
-        const bunAny = globalThis;
-        const bun = bunAny.Bun;
+        const bunGlobal = globalThis;
+        const bun = bunGlobal.Bun;
         if (bun?.file) {
             return bun.file(url).text();
         }
@@ -113,34 +114,15 @@ export class PromptRegistry {
         if (!zodSchema) {
             return { type: 'object', description: 'Schema details unavailable' };
         }
-        const unwrapForJsonSchema = (schema) => {
-            let current = schema;
-            for (let depth = 0; depth < 20; depth++) {
-                const ZodEffects = z.ZodEffects;
-                if (ZodEffects && current instanceof ZodEffects) {
-                    current = current._def.schema;
-                    continue;
-                }
-                if (current instanceof z.ZodPipe) {
-                    current = current._def.out;
-                    continue;
-                }
-                if (current instanceof z.ZodOptional ||
-                    current instanceof z.ZodNullable ||
-                    current instanceof z.ZodDefault) {
-                    current = current._def.innerType;
-                    continue;
-                }
-                break;
-            }
-            return current;
-        };
         try {
-            const unwrapped = unwrapForJsonSchema(zodSchema);
-            const schema = z.toJSONSchema(unwrapped);
-            if (schema && typeof schema === 'object') {
-                const { $schema: _ignored, ...rest } = schema;
-                return rest;
+            const unwrapped = unwrapZodSchema(zodSchema);
+            const zWithJson = z;
+            if (zWithJson.toJSONSchema) {
+                const schema = zWithJson.toJSONSchema(unwrapped);
+                if (schema && typeof schema === 'object') {
+                    const { $schema: _ignored, ...rest } = schema;
+                    return rest;
+                }
             }
         }
         catch (_e) {
@@ -174,7 +156,7 @@ export class PromptRegistry {
         return this.render('explore_system', { tools: this.getToolsForTemplate() });
     }
     renderAutopilotSystem() {
-        return this.render('autopilot_system', {});
+        return this.render('autopilot_system', { tools: this.getToolsForTemplate() });
     }
     renderAnswerSystem() {
         return this.render('answer_system', {});

package/dist/core/prompts/runtime.js

@@ -1,3 +1,4 @@
+import { getLogger } from '../observability/logger.js';
 import { resolvePhaseVisibleTools } from '../tools/tool-visibility.js';
 import { Phase } from '../types/runtime.js';
 import { getPromptRegistry } from './registry.js';
@@ -16,7 +17,8 @@ function extractTargetFiles(plan) {
         const files = parsed.files.filter((f) => typeof f === 'string');
         return files.length > 0 ? files.join(', ') : undefined;
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[PromptRuntime] Failed to extract target files from plan JSON: ${error instanceof Error ? error.message : String(error)}`);
         return undefined;
     }
 }

package/dist/core/prompts/templates/system/autopilot_system.hbs

@@ -1,11 +1,35 @@
 You are a coding assistant running in "autopilot" mode.
-Drive the task forward autonomously and answer in the same language as the user.
-Use the repository context available in the current turn when present.
-If no repository action is possible yet, explain the next best action succinctly.
-Prefer taking a reasonable repository action over asking the user for confirmation.
+Your job is to complete the given task by using the available tools to inspect and modify the repository.
+You MUST use tools to accomplish the task — do not just describe what needs to be done, actually do it.
+
+## Problem-Solving Strategy
+
+1. **Understand**: Read the problem statement carefully. Identify what is failing and what the expected behavior should be.
+2. **Locate**: Find the source file to fix AND all test files that will be affected. Use `code.search` to search for the rule name, function name, or error message across the entire codebase. Pay special attention to CLI integration tests, snapshot tests, and end-to-end tests that assert on the behavior you are changing. Read ALL affected test files before writing any code.
+3. **Diagnose**: Understand the root cause before writing any code. Read surrounding context, not just the error site.
+4. **Fix**: Write the minimal change that fixes the issue. Update ALL affected test files to match the new behavior. Do not refactor unrelated code.
+5. **Verify**: Run the FULL test suite with `shell.exec` (e.g. `pytest`). If tests fail, analyze the output and iterate. Do NOT run only targeted tests — behavioral changes cascade to other tests.
+
+## Efficiency Rules
+- Do NOT reproduce the bug. The problem statement already describes it. Go straight from understanding to fixing.
+- Do NOT run multiple exploratory shell commands. Read the source code, understand the logic, write the fix.
+- If you understand the root cause after reading the source, write the fix immediately. Do not verify the bug first.
+- IMPORTANT: The Locate step is not optional. Spending time finding all affected tests upfront saves rounds later.
+
+## Rules
+- After writing code, you MUST run tests with `shell.exec` to verify your changes. This is not optional — a fix without verification is incomplete.
+- If tests fail, read the failure output carefully, diagnose the issue, and fix it. Do not stop until tests pass. A failing test means your fix is incomplete — find all affected tests and update them.
+- Read test files before modifying code — tests define the expected behavior.
+- Do not modify test files unless the test itself is incorrect. If your fix changes behavior correctly, update the test's expected output to match.
+- Before writing any code, use `code.search` to find ALL tests that reference the changed behavior. Search for: the rule name (e.g. "L031"), the function name, error messages, and expected test output. CLI integration tests and snapshot tests often contain hardcoded expected output that must be updated when behavior changes.
+- Make the smallest possible change. Avoid unrelated refactoring or style changes.
+- Drive the task forward autonomously and answer in the same language as the user.
+
 Treat simple repo-relative paths like "smoke.txt", "README.md", or "src/index.ts" as valid paths from the repository root.
 Do not ask the user to validate a path that is already a valid relative path or can be inferred directly from the instruction.
 Use `interaction.ask_user` only when the task is genuinely ambiguous and you cannot safely infer the next action.
 When calling `interaction.ask_user`, the arguments MUST be valid JSON with exactly this shape: {"questions":[{"header":"Short header","question":"One clear question","options":[{"label":"Option 1","description":"Why pick it"},{"label":"Option 2","description":"Why pick it"}],"multiSelect":false}]}.
 Do not call `interaction.ask_user` with free-form strings, a single question field, string-only options, or empty options arrays.
 For simple file creation or edits, call `fs.write_file` directly with {"file":"relative/path.ext","content":"..."} instead of asking the user for the path again.
+
+{{> tool_defs}}

package/dist/core/protocols/a2a/sdk/executor.js

@@ -1,4 +1,5 @@
 import { InMemoryTaskStore } from '@a2a-js/sdk/server';
+import { getLogger } from '../../../observability/logger.js';
 import { buildCanonicalExecutionRequest, buildInstructionFromParts, } from '../../shared/execution-request.js';
 import { parseA2ASkillFlowMode } from '../../shared/flow-mode-mapping.js';
 export function createA2AInteractionExecutor(deps) {
@@ -346,7 +347,8 @@ export function createA2AInteractionExecutor(deps) {
                     data: JSON.parse(artifact.content),
                 };
             }
-            catch {
+            catch (error) {
+                getLogger().debug(`[A2AExecutor] Failed to parse artifact content as JSON: ${error instanceof Error ? error.message : String(error)}`);
                 // Fall through to text
             }
         }

package/dist/core/protocols/a2a/sdk/server.js

@@ -406,9 +406,9 @@ const normalizeA2AExtensionHeadersByVersion = (req, res, next) => {
     });
     next();
 };
-function isObjectRecord(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
+import { getLogger } from '../../../observability/logger.js';
+import { isRecord } from '../../../utils/serialize.js';
+const isObjectRecord = isRecord;
 function looksLikeTask(result) {
     return typeof result.id === 'string' && 'status' in result;
 }
@@ -826,7 +826,8 @@ function normalizeSseJsonRpcChunkByMethod(chunk, method) {
             const payload = JSON.parse(line.slice('data: '.length));
             return `data: ${JSON.stringify(normalizeJsonRpcPayloadByMethod(method, payload))}`;
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[A2AServer] Failed to parse SSE JSON-RPC chunk: ${error instanceof Error ? error.message : String(error)}`);
             return line;
         }
     })

package/dist/core/protocols/acp/acp-command-runner.js

@@ -1,3 +1,4 @@
+import { getLogger } from '../../observability/logger.js';
 function sleep(ms) {
     if (ms <= 0)
         return Promise.resolve();
@@ -72,8 +73,8 @@ async function safeRelease(terminal) {
     try {
         await terminal.release();
     }
-    catch {
-        // Ignore release errors.
+    catch (error) {
+        getLogger().debug(`[AcpCommandRunner] Failed to release terminal: ${error instanceof Error ? error.message : String(error)}`);
     }
 }
 export function createAcpCommandRunner(params) {
@@ -134,8 +135,8 @@ export function createAcpCommandRunner(params) {
                         try {
                             await terminal.kill();
                         }
-                        catch {
-                            // Ignore kill errors.
+                        catch (error) {
+                            getLogger().debug(`[AcpCommandRunner] Failed to kill terminal on abort: ${error instanceof Error ? error.message : String(error)}`);
                         }
                         break;
                     }
@@ -144,8 +145,8 @@ export function createAcpCommandRunner(params) {
                         try {
                             await terminal.kill();
                         }
-                        catch {
-                            // Ignore kill errors.
+                        catch (error) {
+                            getLogger().debug(`[AcpCommandRunner] Failed to kill terminal on timeout: ${error instanceof Error ? error.message : String(error)}`);
                         }
                         break;
                     }

package/dist/core/protocols/acp/acp-session-persistence.js

@@ -1,6 +1,7 @@
 import { mkdir, open, readFile, rename, stat, unlink, writeFile, } from '../../adapters/fs/node-fs.js';
 import { defaultPathAdapter } from '../../adapters/path/path-adapter.js';
 import { recordAuditEvent } from '../../observability/audit-trail.js';
+import { getLogger } from '../../observability/logger.js';
 import { hashRepoPath, isPermissionPolicyValue, parseTimestamp, } from './acp-types.js';
 export function createAcpSessionPersistence(options) {
     const deletedSessionIds = new Map();
@@ -219,7 +220,8 @@ export function createAcpSessionPersistence(options) {
                     phase: 'PREFLIGHT',
                 });
             }
-            catch {
+            catch (error) {
+                getLogger().debug(`[AcpSessionPersistence] Failed to parse stale lock file: ${error instanceof Error ? error.message : String(error)}`);
                 try {
                     const lockStat = await stat(lockPath);
                     const ageMs = Date.now() - lockStat.mtimeMs;
@@ -231,8 +233,8 @@ export function createAcpSessionPersistence(options) {
                         }, { source: 'acp', severity: 'medium', scope: 'session', phase: 'PREFLIGHT' });
                     }
                 }
-                catch {
-                    // ignore
+                catch (innerError) {
+                    getLogger().debug(`[AcpSessionPersistence] Failed to stat lock file for age check: ${innerError instanceof Error ? innerError.message : String(innerError)}`);
                 }
             }
         };
@@ -246,7 +248,8 @@ export function createAcpSessionPersistence(options) {
                     await lockHandle.writeFile(JSON.stringify({ pid: process.pid, createdAtMs: Date.now() }), 'utf8');
                     break;
                 }
-                catch {
+                catch (error) {
+                    getLogger().debug(`[AcpSessionPersistence] Lock acquire attempt failed: ${error instanceof Error ? error.message : String(error)}`);
                     await tryClearStaleLock();
                     const delayMs = Math.min(250, 20 * (attempt + 1));
                     await new Promise((resolve) => setTimeout(resolve, delayMs));
@@ -271,8 +274,8 @@ export function createAcpSessionPersistence(options) {
                     const existingRaw = await readFile(options.path, 'utf8');
                     existing = normalizePersistedSessionStore(JSON.parse(existingRaw));
                 }
-                catch {
-                    // ignore read failure; writing fresh payload is acceptable
+                catch (error) {
+                    getLogger().debug(`[AcpSessionPersistence] Failed to read existing session store for merge: ${error instanceof Error ? error.message : String(error)}`);
                 }
                 const merged = new Map();
                 const mergedDeletedSessions = pruneDeletedSessionRecords([
@@ -311,14 +314,14 @@ export function createAcpSessionPersistence(options) {
                 try {
                     await lockHandle.close();
                 }
-                catch {
-                    // ignore
+                catch (error) {
+                    getLogger().debug(`[AcpSessionPersistence] Failed to close lock handle: ${error instanceof Error ? error.message : String(error)}`);
                 }
                 try {
                     await unlink(lockPath);
                 }
-                catch {
-                    // ignore
+                catch (error) {
+                    getLogger().debug(`[AcpSessionPersistence] Failed to unlink lock file: ${error instanceof Error ? error.message : String(error)}`);
                 }
             }
         }

package/dist/core/protocols/acp/formal-agent.js

@@ -3,10 +3,12 @@ import { text } from '../../../locales/index.js';
 import { defaultPathAdapter } from '../../adapters/path/path-adapter.js';
 import { inferTurnStopReasonFromFailure } from '../../interaction/turn-stop-reason.js';
 import { recordAuditEvent } from '../../observability/audit-trail.js';
+import { getLogger } from '../../observability/logger.js';
 import { toAcpPublicModes } from '../../public-capabilities/projections.js';
 import { buildPublicCapabilityRegistry } from '../../public-capabilities/registry.js';
 import { parseSlashInput } from '../../slash/parser.js';
 import { Phase } from '../../types/runtime.js';
+import { isRecord } from '../../utils/serialize.js';
 import { buildCanonicalExecutionRequest } from '../shared/execution-request.js';
 import { parseAcpFlowMode } from '../shared/flow-mode-mapping.js';
 import { probeCheckpoint, probeCheckpointForNewSession, } from './acp-checkpoint-probe.js';
@@ -18,7 +20,7 @@ import { createAcpSessionStore, isTerminalTaskEvent } from './handlers.js';
 import { createAcpToolAuthorizationProvider } from './permission-provider.js';
 import { mapToolKind } from './tool-kind-mapping.js';
 function formatInputRequiredMessage(inputRequired) {
-    if (!inputRequired || !Array.isArray(inputRequired.questions))
+    if (!inputRequired || !isRecord(inputRequired) || !Array.isArray(inputRequired.questions))
         return null;
     const questions = inputRequired.questions;
     if (questions.length === 0)
@@ -525,6 +527,7 @@ function acpMcpServersToExtensions(mcpServers) {
         mcpServers: resolvedServers,
         toolPlugins: [],
         skillDiscovery: { paths: [], scope: 'repo' },
+        agentProfiles: [],
     };
 }
 function validateAcpMcpServers(mcpServers) {
@@ -569,12 +572,13 @@ function isPersistableSession(session) {
 async function awaitTerminalEvent(params) {
     if (!params.eventBus)
         return null;
-    const history = params.eventBus.list(params.taskId);
+    const { eventBus } = params;
+    const history = eventBus.list(params.taskId);
     const terminal = history.find(isTerminalTaskEvent);
     if (terminal)
         return terminal;
     return await new Promise((resolve) => {
-        const unsubscribe = params.eventBus.subscribe((event) => {
+        const unsubscribe = eventBus.subscribe((event) => {
             if (event.taskId !== params.taskId)
                 return;
             if (!isTerminalTaskEvent(event))
@@ -638,8 +642,8 @@ export function createAcpFormalAgent(deps) {
         try {
             await emitSessionUpdate(sessionId, update);
         }
-        catch {
-            // Best-effort: do not fail the request due to notification delivery issues.
+        catch (error) {
+            getLogger().debug(`[AcpFormalAgent] Best-effort session info update failed: ${error instanceof Error ? error.message : String(error)}`);
         }
     }
     async function emitRuntimePlanUpdateIfNeeded(params) {
@@ -1092,7 +1096,10 @@ export function createAcpFormalAgent(deps) {
                         ...current,
                         history: [
                             ...current.history,
-                            { role: 'assistant', content: [buildTextContentBlock(responseText)] },
+                            {
+                                role: 'assistant',
+                                content: [buildTextContentBlock(responseText)],
+                            },
                         ],
                     }));
                     await sessionPersistence.persist();

package/dist/core/protocols/acp/permission-provider.js

@@ -1,4 +1,5 @@
 import { text } from '../../../locales/index.js';
+import { getLogger } from '../../observability/logger.js';
 import { mapToolKind } from './tool-kind-mapping.js';
 function buildPermissionOptions() {
     return [
@@ -45,8 +46,8 @@ export function createAcpToolAuthorizationProvider(params) {
             };
             await params.conn.sessionUpdate({ sessionId: params.sessionId, update });
         }
-        catch {
-            // Best-effort: do not block authorization if the client can't accept updates.
+        catch (error) {
+            getLogger().debug(`[AcpPermissionProvider] Best-effort in_progress update failed: ${error instanceof Error ? error.message : String(error)}`);
         }
     };
     return {

package/dist/core/protocols/acp/stdio-server.js

@@ -1,6 +1,8 @@
 import { Readable, Writable } from 'node:stream';
 import { AgentSideConnection } from '@agentclientprotocol/sdk';
 import { tryGetLogger } from '../../observability/logger.js';
+import { errorMessage } from '../../utils/error.js';
+import { isRecord } from '../../utils/serialize.js';
 const INVALID_REQUEST = {
     jsonrpc: '2.0',
     id: null,
@@ -11,9 +13,7 @@ const PARSE_ERROR = {
     id: null,
     error: { code: -32700, message: 'Parse error' },
 };
-function isJsonObject(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
+const isJsonObject = isRecord;
 function hasOwn(value, key) {
     return Object.prototype.hasOwnProperty.call(value, key);
 }
@@ -50,9 +50,9 @@ function createNdjsonWriter(output) {
                 .catch(() => undefined)
                 .then(() => writer.write(data))
                 .catch((error) => {
-                const detail = error instanceof Error ? error.message : String(error);
+                const detail = errorMessage(error);
                 safeWarn(`ACP stdio failed to write NDJSON line. reason="${detail}"`);
-                lastError = error instanceof Error ? new Error(detail) : new Error(detail);
+                lastError = new Error(detail);
             });
             await tail;
         },
@@ -83,7 +83,7 @@ async function processStdioLine(line, ndjson, controller) {
         controller.enqueue(parsed);
     }
     catch (error) {
-        const detail = error instanceof Error ? error.message : String(error);
+        const detail = errorMessage(error);
         safeWarn(`ACP stdio failed to parse JSON line. reason="${detail}"`);
         await ndjson.write(PARSE_ERROR);
     }