@adhdev/daemon-core 0.6.75 → 0.6.77

package/dist/index.d.ts

@@ -285,7 +285,7 @@ interface ManagedCliEntry {
     cliType: string;
     cliName: string;
     status: string;
-    mode: 'terminal' | 'chat';
+    mode: 'terminal';
     workspace: string;
     activeChat: ActiveChatData | null;
 }
@@ -1075,6 +1075,7 @@ declare class ProviderLoader {
         icon: string;
         command: string;
         category: string;
+        versionCommand?: string;
     }[];
     /**
     * List providers by category
@@ -1245,6 +1246,7 @@ interface CLIInfo {
     displayName: string;
     icon: string;
     command: string;
+    versionCommand?: string;
     installed: boolean;
     version?: string;
     path?: string;
@@ -2710,6 +2712,9 @@ declare class ProviderCliAdapter implements CliAdapter {
     private lastApprovalResolvedAt;
     private approvalTransitionBuffer;
     private approvalExitTimeout;
+    private pendingScriptStatus;
+    private pendingScriptStatusSince;
+    private pendingScriptStatusTimer;
     private settleTimer;
     private settledBuffer;
     private submitPendingUntil;
@@ -2741,6 +2746,7 @@ declare class ProviderCliAdapter implements CliAdapter {
     private readonly sendDelayMs;
     private readonly sendKey;
     private readonly submitStrategy;
+    private static readonly SCRIPT_STATUS_DEBOUNCE_MS;
     constructor(provider: CliProviderModule, workingDir: string, extraArgs?: string[]);
     /** Inject CLI scripts after construction (e.g. when resolved by ProviderLoader) */
     setCliScripts(scripts: CliScripts): void;

package/dist/index.js

@@ -692,6 +692,12 @@ __export(provider_cli_adapter_exports, {
 function stripAnsi(str) {
   return str.replace(/\x1B\[\d*[A-HJKSTfG]/g, " ").replace(/\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])/g, "").replace(/\x1B\][^\x07]*\x07/g, "").replace(/\x1B\][^\x1B]*\x1B\\/g, "").replace(/  +/g, " ");
 }
+function stripTerminalNoise(str) {
+  return String(str || "").replace(/[\x00-\x08\x0B\x0C\x0E-\x1F\x7F-\x9F]/g, "").replace(/(^|[\s([])(?:\??\d{1,4}(?:;\d{1,4})*[A-Za-z])(?=$|[\s)\]])/g, "$1").replace(/(^|[\s([])(?:\[\??\d{1,4}(?:;\d{1,4})*[A-Za-z])(?=$|[\s)\]])/g, "$1").replace(/(^|[\s([])(?:\d{1,4};\?)(?=$|[\s)\]])/g, "$1").replace(/\r+/g, "\n").replace(/[ \t]+\n/g, "\n").replace(/\n{3,}/g, "\n\n").replace(/ {2,}/g, " ");
+}
+function sanitizeTerminalText(str) {
+  return stripTerminalNoise(stripAnsi(str));
+}
 function findBinary(name) {
   const isWin = os11.platform() === "win32";
   try {
@@ -924,6 +930,9 @@ var init_provider_cli_adapter = __esm({
       // Approval state machine
       approvalTransitionBuffer = "";
       approvalExitTimeout = null;
+      pendingScriptStatus = null;
+      pendingScriptStatusSince = 0;
+      pendingScriptStatusTimer = null;
       // Output settle debounce — fires after PTY output goes quiet
       settleTimer = null;
       settledBuffer = "";
@@ -989,6 +998,7 @@ var init_provider_cli_adapter = __esm({
       sendDelayMs;
       sendKey;
       submitStrategy;
+      static SCRIPT_STATUS_DEBOUNCE_MS = 1e3;
       /** Inject CLI scripts after construction (e.g. when resolved by ProviderLoader) */
       setCliScripts(scripts) {
         this.cliScripts = scripts;
@@ -1115,7 +1125,7 @@ var init_provider_cli_adapter = __esm({
       handleOutput(rawData) {
         this.terminalScreen.write(rawData);
         this.terminalHistory = mergeTerminalHistory(this.terminalHistory, this.terminalScreen.getText());
-        const cleanData = stripAnsi(rawData);
+        const cleanData = sanitizeTerminalText(rawData);
         if (this.isWaitingForResponse && cleanData) {
           this.responseBuffer = (this.responseBuffer + cleanData).slice(-8e3);
         }
@@ -1212,24 +1222,39 @@ var init_provider_cli_adapter = __esm({
         const scriptStatus = rawScriptStatus;
         if (!scriptStatus) return;
         const prevStatus = this.currentStatus;
-        if (scriptStatus === "waiting_approval") {
-          const modalMessage = modal?.message || "";
-          const screenText = this.terminalScreen.getText() || this.accumulatedBuffer;
-          const autoAcceptPatterns = [
-            /be able to read, edit, and execute/i,
-            /Security guide/i,
-            /Enter to confirm/i,
-            /Quick safety check/i,
-            /Do you trust the files/i,
-            /Is this a project/i
-          ];
-          if (autoAcceptPatterns.some((p) => p.test(modalMessage) || p.test(screenText))) {
-            LOG.info("CLI", `[${this.cliType}] Auto-accepting startup dialog: ${modalMessage.slice(0, 80)}`);
-            setTimeout(() => this.ptyProcess?.write("\r"), 200);
-            this.lastApprovalResolvedAt = Date.now();
-            this.activeModal = null;
+        const clearPendingScriptStatus = () => {
+          this.pendingScriptStatus = null;
+          this.pendingScriptStatusSince = 0;
+          if (this.pendingScriptStatusTimer) {
+            clearTimeout(this.pendingScriptStatusTimer);
+            this.pendingScriptStatusTimer = null;
+          }
+        };
+        const armPendingScriptStatus = (delayMs) => {
+          if (this.pendingScriptStatusTimer) clearTimeout(this.pendingScriptStatusTimer);
+          this.pendingScriptStatusTimer = setTimeout(() => {
+            this.pendingScriptStatusTimer = null;
+            this.settledBuffer = this.recentOutputBuffer;
+            this.evaluateSettled();
+          }, delayMs);
+        };
+        const shouldDebouncePromotion = (status) => prevStatus === "idle" && !this.isWaitingForResponse && !this.currentTurnScope && (status === "generating" || status === "waiting_approval");
+        if (shouldDebouncePromotion(scriptStatus)) {
+          if (this.pendingScriptStatus !== scriptStatus) {
+            this.pendingScriptStatus = scriptStatus;
+            this.pendingScriptStatusSince = now;
+            armPendingScriptStatus(_ProviderCliAdapter.SCRIPT_STATUS_DEBOUNCE_MS);
+            return;
+          }
+          const elapsed = now - this.pendingScriptStatusSince;
+          if (elapsed < _ProviderCliAdapter.SCRIPT_STATUS_DEBOUNCE_MS) {
+            armPendingScriptStatus(_ProviderCliAdapter.SCRIPT_STATUS_DEBOUNCE_MS - elapsed);
             return;
           }
+        } else {
+          clearPendingScriptStatus();
+        }
+        if (scriptStatus === "waiting_approval") {
           const inCooldown = this.lastApprovalResolvedAt && Date.now() - this.lastApprovalResolvedAt < this.timeouts.approvalCooldown;
           if (!inCooldown) {
             this.isWaitingForResponse = true;
@@ -1242,6 +1267,12 @@ var init_provider_cli_adapter = __esm({
           }
         }
         if (scriptStatus === "generating") {
+          const screenText = this.terminalScreen.getText() || this.accumulatedBuffer;
+          const noActiveTurn = !this.currentTurnScope;
+          const looksIdleChrome = /(^|\n)\s*[❯›>]\s*(?:\n|$)/m.test(screenText) || /accept edits on/i.test(screenText) && (/Update available!/i.test(screenText) || /\/effort/i.test(screenText) || /^.*➜\s+\S+/m.test(screenText));
+          if (prevStatus === "idle" && !this.isWaitingForResponse && noActiveTurn && !modal && looksIdleChrome) {
+            return;
+          }
           if (prevStatus === "waiting_approval") {
             if (this.approvalExitTimeout) {
               clearTimeout(this.approvalExitTimeout);
@@ -1699,7 +1730,7 @@ ${data.message || ""}`.trim();
           committedMessages: this.committedMessages.slice(-20),
           structuredMessages: this.structuredMessages.slice(-20),
           messageCount: this.committedMessages.length,
-          screenText: this.terminalScreen.getText().slice(-4e3),
+          screenText: sanitizeTerminalText(this.terminalScreen.getText()).slice(-4e3),
           terminalHistory: this.terminalHistory.slice(-8e3),
           currentTurnScope: this.currentTurnScope,
           startupBuffer: this.startupBuffer.slice(-4e3),
@@ -1708,6 +1739,7 @@ ${data.message || ""}`.trim();
           accumulatedBufferLength: this.accumulatedBuffer.length,
           accumulatedRawBufferLength: this.accumulatedRawBuffer.length,
           rawBufferPreview: this.accumulatedRawBuffer.slice(-1e3),
+          sanitizedRawPreview: sanitizeTerminalText(this.accumulatedRawBuffer).slice(-1e3),
           responseBuffer: this.responseBuffer.slice(-1e3),
           isWaitingForResponse: this.isWaitingForResponse,
           activeModal: this.activeModal,
@@ -1946,6 +1978,10 @@ async function detectIDEs() {
 // src/detection/cli-detector.ts
 var import_child_process2 = require("child_process");
 var os2 = __toESM(require("os"));
+function parseVersion(raw) {
+  const match = raw.match(/v?(\d+\.\d+(?:\.\d+)?(?:-[a-zA-Z0-9.]+)?)/);
+  return match ? match[1] : raw.split("\n")[0].slice(0, 100);
+}
 function execAsync(cmd, timeoutMs = 5e3) {
   return new Promise((resolve8) => {
     const child = (0, import_child_process2.exec)(cmd, { encoding: "utf-8", timeout: timeoutMs }, (err, stdout) => {
@@ -1970,10 +2006,18 @@ async function detectCLIs(providerLoader) {
         const firstPath = pathResult.split("\n")[0];
         let version;
         try {
-          const versionResult = await execAsync(`${cli.command} --version 2>/dev/null`, 3e3);
-          if (versionResult) {
-            const match = versionResult.match(/(\d+\.\d+[\.\d]*)/);
-            version = match ? match[1] : versionResult.split("\n")[0].slice(0, 30);
+          const versionCommands = [
+            cli.versionCommand,
+            `${cli.command} --version 2>/dev/null`,
+            `${cli.command} -V 2>/dev/null`,
+            `${cli.command} -v 2>/dev/null`
+          ].filter((v) => !!v);
+          for (const versionCommand of versionCommands) {
+            const versionResult = await execAsync(versionCommand, 3e3);
+            if (versionResult) {
+              version = parseVersion(versionResult);
+              break;
+            }
           }
         } catch {
         }
@@ -4479,7 +4523,7 @@ function buildManagedClis(cliStates) {
     cliType: s.type,
     cliName: s.name,
     status: s.status,
-    mode: s.mode,
+    mode: "terminal",
     workspace: s.workspace || "",
     activeChat: s.activeChat
   }));
@@ -6035,8 +6079,13 @@ var DaemonCommandHandler = class {
   getCliAdapter(type) {
     const target = type || this._currentIdeType;
     if (!target || !this._ctx.adapters) return null;
+    let normalizedTarget = target;
+    const colonIdx = normalizedTarget.lastIndexOf(":");
+    if (colonIdx >= 0) normalizedTarget = normalizedTarget.substring(colonIdx + 1);
+    const direct = this._ctx.adapters.get(normalizedTarget);
+    if (direct) return direct;
     for (const [key, adapter] of this._ctx.adapters.entries()) {
-      if (adapter.cliType === target || key.startsWith(target)) {
+      if (adapter.cliType === target || adapter.cliType === normalizedTarget || key === normalizedTarget || key.startsWith(target) || key.startsWith(normalizedTarget)) {
         return adapter;
       }
     }
@@ -6549,12 +6598,15 @@ var ProviderLoader = class _ProviderLoader {
     const result = [];
     for (const p of this.providers.values()) {
       if ((p.category === "cli" || p.category === "acp") && p.spawn?.command) {
+        const verCmdConfig = p.versionCommand;
+        const versionCommand = typeof verCmdConfig === "object" && verCmdConfig !== null ? verCmdConfig[process.platform] : verCmdConfig;
         result.push({
           id: p.type,
           displayName: p.displayName || p.name,
           icon: p.icon || "\u{1F527}",
           command: p.spawn.command,
-          category: p.category
+          category: p.category,
+          ...typeof versionCommand === "string" && versionCommand.trim() ? { versionCommand: versionCommand.trim() } : {}
         });
       }
     }
@@ -8472,19 +8524,6 @@ var CliProviderInstance = class {
   getState() {
     const adapterStatus = this.adapter.getStatus();
     const dirName = this.workingDir.split("/").filter(Boolean).pop() || "session";
-    const recentMessages = adapterStatus.messages.slice(-50).map((m) => {
-      const content = typeof m.content === "string" && m.content.length > 8e3 ? m.content.slice(0, 8e3) + "\n... (truncated)" : m.content;
-      return { ...m, content };
-    });
-    if (recentMessages.length > 0) {
-      const dirName2 = this.workingDir.split("/").filter(Boolean).pop() || "session";
-      this.historyWriter.appendNewMessages(
-        this.type,
-        recentMessages,
-        `${this.provider.name} \xB7 ${dirName2}`,
-        this.instanceId
-      );
-    }
     if (adapterStatus.terminalHistory?.trim()) {
       this.historyWriter.appendTerminalHistory(
         this.type,
@@ -8498,12 +8537,12 @@ var CliProviderInstance = class {
       name: this.provider.name,
       category: "cli",
       status: adapterStatus.status,
-      mode: this.settings.mode || "terminal",
+      mode: "terminal",
       activeChat: {
         id: `${this.type}_${this.workingDir}`,
         title: `${this.provider.name} \xB7 ${dirName}`,
         status: adapterStatus.status,
-        messages: recentMessages,
+        messages: [],
         activeModal: adapterStatus.activeModal,
         terminalHistory: adapterStatus.terminalHistory,
         inputContent: ""
@@ -8517,11 +8556,15 @@ var CliProviderInstance = class {
   }
   onEvent(event, data) {
     if (event === "send_message" && data?.text) {
-      this.adapter.sendMessage(data.text);
+      void this.adapter.sendMessage(data.text).catch((e) => {
+        LOG.warn("CLI", `[${this.type}] send_message failed: ${e?.message || e}`);
+      });
     } else if (event === "server_connected" && data?.serverConn) {
       this.adapter.setServerConn(data.serverConn);
     } else if (event === "resolve_action" && data) {
-      this.adapter.resolveAction(data);
+      void this.adapter.resolveAction(data).catch((e) => {
+        LOG.warn("CLI", `[${this.type}] resolve_action failed: ${e?.message || e}`);
+      });
     }
   }
   dispose() {
@@ -10635,18 +10678,18 @@ function findBinary2(name) {
   const result = runCommand(cmd, 5e3);
   return result ? result.split("\n")[0] : null;
 }
-function parseVersion(raw) {
+function parseVersion2(raw) {
   const match = raw.match(/v?(\d+\.\d+(?:\.\d+)?(?:-[a-zA-Z0-9.]+)?)/);
   return match ? match[1] : raw.split("\n")[0].substring(0, 100);
 }
 function getVersion(binary, versionCommand) {
   if (versionCommand) {
     const raw = runCommand(versionCommand);
-    return raw ? parseVersion(raw) : null;
+    return raw ? parseVersion2(raw) : null;
   }
   for (const flag of ["--version", "-V", "-v"]) {
     const raw = runCommand(`"${binary}" ${flag}`);
-    if (raw && raw.length < 500) return parseVersion(raw);
+    if (raw && raw.length < 500) return parseVersion2(raw);
   }
   return null;
 }
@@ -12907,16 +12950,16 @@ var DevServer = class _DevServer {
   resolveAutoImplWritableProviderDir(category, type, requestedDir) {
     const canonicalUserDir = path12.resolve(this.providerLoader.getUserProviderDir(category, type));
     const desiredDir = requestedDir ? path12.resolve(requestedDir) : canonicalUserDir;
-    if (desiredDir !== canonicalUserDir) {
-      return null;
+    const upstreamRoot = path12.resolve(this.providerLoader.getUpstreamDir());
+    if (desiredDir === upstreamRoot || desiredDir.startsWith(`${upstreamRoot}${path12.sep}`)) {
+      return { dir: null, reason: `Refusing to write into upstream provider directory: ${desiredDir}` };
     }
-    const userRoot = path12.resolve(this.providerLoader.getUserDir());
-    if (desiredDir !== userRoot && !desiredDir.startsWith(`${userRoot}${path12.sep}`)) {
-      return null;
+    if (path12.basename(desiredDir) !== type) {
+      return { dir: null, reason: `Requested writable provider directory must end with '${type}': ${desiredDir}` };
     }
     const sourceDir = this.findProviderDir(type);
     if (!sourceDir) {
-      return null;
+      return { dir: null, reason: `Provider source directory not found for '${type}'` };
     }
     if (!fs9.existsSync(desiredDir)) {
       fs9.mkdirSync(path12.dirname(desiredDir), { recursive: true });
@@ -12925,7 +12968,7 @@ var DevServer = class _DevServer {
     }
     const providerJson = path12.join(desiredDir, "provider.json");
     if (!fs9.existsSync(providerJson)) {
-      return null;
+      return { dir: null, reason: `provider.json not found in writable provider directory: ${desiredDir}` };
     }
     try {
       const providerData = JSON.parse(fs9.readFileSync(providerJson, "utf-8"));
@@ -12933,10 +12976,13 @@ var DevServer = class _DevServer {
         providerData.disableUpstream = true;
         fs9.writeFileSync(providerJson, JSON.stringify(providerData, null, 2));
       }
-    } catch {
-      return null;
+    } catch (error) {
+      return {
+        dir: null,
+        reason: `Failed to update provider.json in writable provider directory: ${error.message}`
+      };
     }
-    return desiredDir;
+    return { dir: desiredDir };
   }
   loadAutoImplReferenceScripts(referenceType) {
     if (!referenceType) return {};
@@ -12971,13 +13017,14 @@ var DevServer = class _DevServer {
       this.json(res, 404, { error: `Provider not found: ${type}` });
       return;
     }
-    const providerDir = this.resolveAutoImplWritableProviderDir(provider.category, type, requestedProviderDir);
-    if (!providerDir) {
+    const writableProvider = this.resolveAutoImplWritableProviderDir(provider.category, type, requestedProviderDir);
+    if (!writableProvider.dir) {
       this.json(res, 409, {
-        error: `Auto-implement only writes to the canonical user provider directory for '${type}'.`
+        error: writableProvider.reason || `Auto-implement only writes to the canonical user provider directory for '${type}'.`
       });
       return;
     }
+    const providerDir = writableProvider.dir;
     try {
       const resolvedReference = this.resolveAutoImplReference(provider.category, reference, type);
       this.sendAutoImplSSE({
@@ -13478,14 +13525,16 @@ var DevServer = class _DevServer {
     lines.push("| Status | When to use | How to detect |");
     lines.push("|---|---|---|");
     lines.push("| `idle` | AI is NOT generating, no approval needed | Default state. No stop button, no spinners, no approval pills/buttons |");
-    lines.push("| `generating` | AI is actively streaming/thinking | ANY of: (1) Stop/Cancel button visible, (2) CSS animation (animate-spin/pulse/bounce), (3) floating state text like Thinking/Generating/Sailing, (4) streaming indicator class |");
+    lines.push('| `generating` | AI is actively streaming/thinking | ANY of: (1) Submit button icon SVG changes (e.g. arrow\u2192stop square, fill="none"\u2192fill="currentColor"), (2) Stop/Cancel button visible, (3) CSS animation, (4) Structural markers (aria-labels that only appear during generation) |');
     lines.push("| `waiting_approval` | AI stopped and needs user action | Actionable buttons like Run/Skip/Accept/Reject are visible AND clickable |");
     lines.push("");
     lines.push("### \u26A0\uFE0F Status Detection Gotchas (MUST READ!)");
-    lines.push('1. **FALSE POSITIVES from old messages**: Chat history may contain text like "Command Awaiting Approval" from PAST turns. If you search the entire chat panel for this text, you will get false matches from parent divs whose innerText includes ALL child text. ONLY match small leaf elements (under 80 chars) or use explicit button/pill selectors.');
-    lines.push('2. **Awaiting Approval pill without actions**: Some IDEs show a floating pill/banner saying "Awaiting Approval" that is just a scroll-to indicator (not an actual approval dialog). If this pill exists but NO actionable buttons (Run/Skip/Accept/Reject) exist anywhere in the panel, the status should be `idle`, NOT `waiting_approval`.');
-    lines.push("3. **generating detection must be multi-signal**: Do NOT rely on just one indicator. Check ALL of: stop buttons, CSS animations, floating state labels, streaming classes. IDEs differ widely.");
-    lines.push("4. **activeModal must include actions**: When `status` is `waiting_approval`, the `activeModal` object MUST include a non-empty `actions` array listing the button labels. If you cannot find any action buttons, the status is NOT `waiting_approval`.");
+    lines.push(`1. **DO NOT rely on button text/labels in the user's language.** OS locale may be Korean, Japanese, etc. Button text like "Cancel" or "Stop" will be localized. Instead, detect STRUCTURAL indicators: SVG icon changes, CSS classes, aria-labels from the extension's own React/Radix UI (which stay in English regardless of OS locale).`);
+    lines.push('2. **Use sendMessage to CREATE a generating state, then CAPTURE the DOM.** Send a LONG prompt (e.g. "Write an extremely detailed 5000-word essay...") so the AI takes 10+ seconds. Then periodically capture the DOM during generation to find which elements appear/change. Compare idle vs generating DOM snapshots to find reliable structural markers.');
+    lines.push("3. **Look for SVG icon changes in the submit button.** Many IDEs change the submit button icon from an arrow (send) to a square (stop) during generation. Check the SVG `fill` attribute or path data.");
+    lines.push('4. **FALSE POSITIVES from old messages**: Chat history may contain text like "Command Awaiting Approval" from PAST turns. ONLY match small leaf elements (under 80 chars) or use explicit button/pill selectors.');
+    lines.push("5. **Awaiting Approval pill without actions**: Some IDEs show a floating pill/banner that is just a scroll-to indicator. If NO actionable buttons exist, the status should be `idle`, NOT `waiting_approval`.");
+    lines.push("6. **activeModal must include actions**: When `status` is `waiting_approval`, the `activeModal` object MUST include a non-empty `actions` array.");
     lines.push("");
     lines.push("## Action");
     lines.push("1. Edit the script files to implement working code");
@@ -13543,10 +13592,10 @@ var DevServer = class _DevServer {
     lines.push(`echo "$RESULT" | python3 -c "import sys,json; d=json.load(sys.stdin); r=d.get('result',d); r=json.loads(r) if isinstance(r,str) else r; assert r.get('status')=='idle', f'Expected idle, got {r.get(chr(34)+chr(115)+chr(116)+chr(97)+chr(116)+chr(117)+chr(115)+chr(34))}'; print('Step 1 PASS: status=idle')"`);
     lines.push("```");
     lines.push("");
-    lines.push("### Step 2: Send a message that triggers generation");
+    lines.push("### Step 2: Send a LONG message that triggers extended generation (10+ seconds)");
     lines.push("```bash");
-    lines.push(`curl -sS -X POST http://127.0.0.1:${DEV_SERVER_PORT}/api/scripts/run -H "Content-Type: application/json" -d '{"script": "sendMessage", "type": "${type}", "ideType": "${type}", "args": {"message": "Say hello in one word"}}'`);
-    lines.push("sleep 2");
+    lines.push(`curl -sS -X POST http://127.0.0.1:${DEV_SERVER_PORT}/api/scripts/run -H "Content-Type: application/json" -d '{"script": "sendMessage", "type": "${type}", "ideType": "${type}", "args": {"message": "Write an extremely detailed 5000-word essay about the history of artificial intelligence from Alan Turing to 2025. Be very thorough and verbose."}}'`);
+    lines.push("sleep 3");
     lines.push("```");
     lines.push("");
     lines.push("### Step 3: Check generating OR completed");
@@ -13670,6 +13719,9 @@ var DevServer = class _DevServer {
     lines.push("8. Keep exports compatible with the existing `scripts.js` router (`module.exports = function ...`).");
     lines.push("9. Do not rewrite unrelated provider config. Only touch the scripts needed for this task unless a tiny supporting change is required.");
     lines.push("10. When the verification API returns `instanceId`, keep using that exact instance for follow-up `send`, `resolve`, `raw`, and `stop` calls. Do not assume type-only routing is safe if multiple sessions exist.");
+    lines.push("11. Do NOT repeatedly dump the same target files. Read the target scripts once, reproduce the bug, then move directly to patching.");
+    lines.push("12. If the user instructions include concrete screen text, raw PTY snippets, or a specific repro, treat that as the primary acceptance criteria.");
+    lines.push("13. After the first successful live repro, stop broad diagnosis. Edit the scripts, reload, and verify. Do not burn tokens on repeated re-inspection without code changes.");
     lines.push("");
     lines.push("## Task");
     lines.push(`Edit files in \`${providerDir}\` to implement: **${functions.join(", ")}**`);
@@ -13711,6 +13763,9 @@ var DevServer = class _DevServer {
     lines.push("");
     lines.push("Use `resolve` when the parsed modal buttons are correct. Use `raw` when the CLI expects a literal keystroke like `1`, `y`, or Enter. Repeat until idle.");
     lines.push("");
+    lines.push("### Patch Discipline");
+    lines.push("Once the repro is confirmed, immediately edit the target files. Avoid loops where you keep re-reading long files or re-running the same debug commands without changing code.");
+    lines.push("");
     lines.push("### 5. Verify the side effects outside the CLI");
     lines.push("```bash");
     lines.push("test -f tmp/adhdev_provider_fix_test.py");