@dmsdc-ai/aigentry-telepty 0.6.8 → 0.6.10

package/CHANGELOG.md

@@ -4,6 +4,8 @@ All notable changes to `@dmsdc-ai/aigentry-telepty` are documented here.
 
 ## [Unreleased]
 
+## [0.6.10] - 2026-07-05
+
 ### Added — seamless cross-machine on Tailscale (auto bind + auto trust) (#672)
 
 - **On a Tailscale host, a fresh install is cross-machine-ready with zero manual env.** At
@@ -31,6 +33,22 @@ All notable changes to `@dmsdc-ai/aigentry-telepty` are documented here.
   Zero-config cross-machine is Tailscale-specific; other topologies use the manual
   `TELEPTY_BIND` path. Addressing stays IP-free — use MagicDNS names with `<id>@<host>`.
 
+### Fixed — busy-target `--submit` latency (busy-dispatch fast-path) (#694)
+
+- **A gated `--submit` to a BUSY (mid-turn) recipient no longer burns the full gate timeout.**
+  A busy claude sits in `working`/`thinking` — neither is a ready state — so the render-gate
+  (`awaitReplReady`) could never pass mid-turn and waited out the entire `gate_timeout_ms`
+  (default 10s) before dispatching best-effort. A new busy-dispatch fast-path detects a
+  **genuine ongoing turn** (`working`/`thinking` held ≥ `TELEPTY_SUBMIT_BUSY_GRACE_MS`, default
+  250ms — which excludes the transient `working` a target emits while echoing our own
+  just-injected text) and dispatches after only the existing echo+micro-quiet settle
+  (`awaitInputSettled`), cutting the busy path from ~10s to **sub-second**.
+- **Never fires blindly, idle path unchanged.** The downstream `gatedTerminalSubmit` still
+  runs its own echo+quiet gate before the `\r`; a CR into a busy composer merely queues and the
+  #617 hold-and-redeliver loop re-fires it on idle (delivery reliability unchanged). `idle`/
+  `waiting` targets never match the fast-path, so the proven idle behavior is byte-for-byte
+  identical. Rollback via `TELEPTY_SUBMIT_BUSY_DISPATCH=off`.
+
 ## [0.6.6] - 2026-06-14
 
 ### Fixed — duplicate same-id `allow` flap loop + `kill` doesn't stick (#56)

package/README.md

@@ -299,4 +299,4 @@ npm run test:watch    # Watch mode
 
 ## License
 
-ISC
+MIT

package/cli.js

@@ -2375,7 +2375,14 @@ async function main() {
       const target = await resolveSessionTarget(sessionId);
       if (!target) {
         console.error(`❌ Session '${sessionId}' was not found on any discovered host.`);
-        process.exit(1);
+        // #691: drain-exit, do NOT process.exit() here. resolveSessionTarget →
+        // discoverSessions() leaves undici keep-alive sockets and AbortSignal.timeout
+        // timers still mid-close; a hard process.exit() races that teardown and trips a
+        // libuv double-close assertion (src/win/async.c:76 UV_HANDLE_CLOSING) on Windows,
+        // which wedges the session. Setting exitCode + returning lets the event loop close
+        // each handle exactly once. Same exit code (1); success path unchanged.
+        process.exitCode = 1;
+        return;
       }
 
       let injectPrompt = prompt;
@@ -2388,7 +2395,8 @@ async function main() {
         const ent = checkEntitlement({ feature: 'telepty.remote_sessions' });
         if (!ent.allowed) {
           console.error(`⚠️  ${ent.reason}\n   Upgrade: ${ent.upgrade_url}`);
-          process.exit(1);
+          process.exitCode = 1; // #691: drain-exit (see not-found above), not process.exit() mid-teardown
+          return;
         }
 
         if (useRef) {

package/daemon.js

@@ -53,6 +53,15 @@ const REDELIVER_ENABLED = String(process.env.TELEPTY_REDELIVER || '').toLowerCas
 const REDELIVER_MAX_ATTEMPTS = Math.max(1, Number(process.env.TELEPTY_REDELIVER_MAX_ATTEMPTS || 3));
 const REDELIVER_TOTAL_TIMEOUT_MS = Math.max(1000, Number(process.env.TELEPTY_REDELIVER_TOTAL_TIMEOUT_MS || 600000));
 const REDELIVER_IDLE_WAIT_MS = Math.max(1000, Number(process.env.TELEPTY_REDELIVER_IDLE_WAIT_MS || 120000));
+// #694: busy-dispatch fast-path — on a busy (mid-turn) recipient the render-gate (awaitReplReady)
+// can never reach idle/waiting, so it burns the full gate timeout before best-effort dispatch.
+// When enabled, a genuine ongoing turn (working/thinking held ≥ grace) dispatches after only the
+// echo+micro-quiet settle instead of the full timeout. `off` restores the pure-idle-gate behavior.
+const SUBMIT_BUSY_DISPATCH_ENABLED = String(process.env.TELEPTY_SUBMIT_BUSY_DISPATCH || '').toLowerCase() !== 'off';
+// Grace floor separating a REAL ongoing turn from the transient `working` a target emits while
+// echoing our OWN just-injected text (duration_ms ≈ 0). Default 250ms: above the echo transient,
+// below any human-perceptible latency.
+const SUBMIT_BUSY_GRACE_MS = Math.max(0, Number(process.env.TELEPTY_SUBMIT_BUSY_GRACE_MS || 250));
 
 // Session state machine manager — auto-detects session state from PTY output
 const sessionStateManager = new SessionStateManager({
@@ -1790,6 +1799,26 @@ function forceSubmitDeliveredToSurface(session, strategy) {
   return !!strategy;
 }
 
+// #678: which submit render-gate outcomes are TERMINAL (hard-fail 504, never dispatch the CR)
+// vs best-effort DISPATCH (fire the CR anyway). A missing render-state — `no_state` (no state
+// machine registered for the session) or `no_state_manager` — is NOT terminal: it says nothing
+// about PTY/ownerWs liveness. A wrapped session restored across a daemon restart has no machine
+// yet a perfectly live bridge/PTY, so we dispatch best-effort exactly like `timeout` (and if the
+// PTY/ownerWs really is gone, submitViaPty returns false → a clean 503, not a false success).
+// Only genuinely-terminal PTY states — session_dead / session_error / session_restarting — still
+// short-circuit, because writing a CR to a dead/errored/restarting PTY is pointless.
+// #694: `busy_settled` (busy-dispatch fast-path) joins the best-effort DISPATCH set — the target is
+// alive and mid-turn, so firing the CR is correct (it queues + #617 redelivers), never a hard-fail.
+const NON_TERMINAL_GATE_REASONS = new Set(['timeout', 'no_state', 'no_state_manager', 'busy_settled']);
+function isTerminalGateFailure(gateResult) {
+  return !!(
+    gateResult &&
+    gateResult.ready !== true &&
+    gateResult.reason &&
+    !NON_TERMINAL_GATE_REASONS.has(gateResult.reason)
+  );
+}
+
 async function deliverInjectionToSession(id, session, prompt, options = {}) {
   const now = Date.now();
   if (!options.bypassBootstrapQueue && shouldQueueBootstrapOperation(session)) {
@@ -2066,6 +2095,11 @@ for (const [id, meta] of Object.entries(_persisted)) {
   if (!restored) continue;
   sessions[id] = restored;
   initializeBootstrapState(sessions[id]);
+  // #678: a session restored across a daemon restart must get a render-state machine,
+  // else the submit gate reads getState()=null → no_state and never fires the CR. The
+  // machine is otherwise created only at first register(), which the reconnecting bridge's
+  // idempotent re-register skips (see the /api/sessions/register early-return below).
+  sessionStateManager.register(id);
   console.log(`[PERSIST] Restored session ${id} (awaiting reconnect)`);
 }
 const STRIPPED_SESSION_ENV_KEYS = [
@@ -2282,6 +2316,12 @@ app.post('/api/sessions/register', (req, res) => {
     // on a metadata re-register, or a session's delivered bytes would change mid-flight.
     if (req.body.provenance_capable === true) existing.provenanceCapable = true;
     existing.provenanceNonce = ensureSessionNonce(session_id);
+    // #678: a bridge reconnecting to an already-known session (e.g. one restored across a
+    // daemon restart, or a re-`allow` of the same id) must still end up with a render-state
+    // machine. register() is idempotent (returns the existing machine untouched), so this is
+    // safe for the normal re-register too — and it is the ONLY thing that recreates the
+    // machine for a same-id reconnect, since a daemon restart is not required to reach here.
+    sessionStateManager.register(session_id);
     console.log(`[REGISTER] Re-registered session ${session_id} (type: ${existing.type}, updated metadata)`);
     return res.status(200).json({ session_id, type: existing.type, command: existing.command, cwd: existing.cwd, reregistered: true, session_token: mintSessionToken(session_id), session_nonce: existing.provenanceNonce, provenance_capable: !!existing.provenanceCapable });
   }
@@ -2814,8 +2854,10 @@ function submitViaOsascript(sessionId, keyCombo) {
 // POST /api/sessions/:id/submit — render-gated CLI-aware submit
 //
 // Default behavior (0.3.0+): wait for the target REPL to be ready (sessionStateManager
-// reports `idle`/`waiting` with confidence ≥ 0.85) before firing Enter. When the
-// caller passes `injected_body`, also verify the body has been consumed (i.e.
+// reports `idle`/`waiting` with confidence ≥ 0.5, the awaitReplReady default) before firing
+// Enter. #694: a BUSY recipient (working/thinking) is never idle/waiting, so the busy-dispatch
+// fast-path below dispatches best-effort after an echo+quiet settle instead of burning the gate
+// timeout. When the caller passes `injected_body`, also verify the body has been consumed (i.e.
 // disappeared from the input box) by polling the session output ring; if still
 // visible, perform one bounded retry.
 //
@@ -3015,12 +3057,40 @@ app.post('/api/sessions/:id/submit', async (req, res) => {
   // Hard-fail reasons (session_dead/error/restarting/no_state/no_state_manager)
   // still short-circuit to 504 because dispatching to a dead/missing PTY is
   // pointless. See spec §1.3 / §3.3.
-  const gateResult = await submitGate.awaitReplReady(id, sessionStateManager, {
-    timeoutMs: gateTimeoutMs,
-    ...(minConfidence !== undefined ? { minConfidence } : {}),
-  });
+  //
+  // #694 busy-dispatch fast-path: a BUSY (mid-turn) recipient sits in working/thinking — neither
+  // is a READY_STATE — so awaitReplReady can never pass mid-turn and would burn the FULL
+  // gateTimeoutMs (up to 10s) before best-effort dispatch. When the target is a GENUINE ongoing
+  // turn (isBusyDispatchState: working/thinking held ≥ SUBMIT_BUSY_GRACE_MS, which excludes the
+  // transient `working` from echoing our OWN just-injected text, duration_ms ≈ 0), wait ONLY for
+  // the input to settle (body echoed + micro-quiet, via awaitInputSettled) and dispatch
+  // best-effort — the same downstream path as a gate timeout, but in ~hundreds of ms. idle/waiting
+  // never match isBusyDispatchState, so the idle path is byte-unchanged. gatedTerminalSubmit below
+  // still runs its OWN echo+quiet gate before the \r (never fires blindly), and a CR into a busy
+  // composer merely queues → #617 redeliver fires it on idle. Rollback: TELEPTY_SUBMIT_BUSY_DISPATCH=off.
+  let gateResult = null;
+  if (injectedBody && SUBMIT_BUSY_DISPATCH_ENABLED) {
+    const cur = sessionStateManager.getState(id);
+    if (submitGate.isBusyDispatchState(cur, SUBMIT_BUSY_GRACE_MS)) {
+      const settle = await submitGate.awaitInputSettled(session, injectedBody, {
+        timeoutMs: 1200,
+        quietWindowMs: 100,
+        echoGraceMs: 400,
+        pollIntervalMs: 30,
+        stripAnsi: stripAnsiState,
+      });
+      gateResult = { ready: false, reason: 'busy_settled', last_state: cur.state, waited_ms: settle.waited_ms };
+      console.log(`[SUBMIT] busy-dispatch fast-path ${id}: state=${cur.state} (${cur.duration_ms}ms) settle=${settle.reason} (${settle.waited_ms}ms) — dispatching without idle-gate burn`);
+    }
+  }
+  if (!gateResult) {
+    gateResult = await submitGate.awaitReplReady(id, sessionStateManager, {
+      timeoutMs: gateTimeoutMs,
+      ...(minConfidence !== undefined ? { minConfidence } : {}),
+    });
+  }
   const gatedDispatchAfterTimeout = !gateResult.ready;
-  if (gatedDispatchAfterTimeout && gateResult.reason && gateResult.reason !== 'timeout') {
+  if (isTerminalGateFailure(gateResult)) {
     console.log(`[SUBMIT] gate hard-fail ${id}: ${gateResult.reason} (last_state=${gateResult.last_state})`);
     return res.status(504).json({
       error: 'Submit gated-timeout — target REPL not in a dispatchable state',
@@ -3034,7 +3104,9 @@ app.post('/api/sessions/:id/submit', async (req, res) => {
     });
   }
   if (gatedDispatchAfterTimeout) {
-    console.log(`[SUBMIT] gate timeout ${id}: dispatching anyway (last_state=${gateResult.last_state})`);
+    // #678: `no_state`/`no_state_manager` join `timeout` here — dispatch the CR best-effort
+    // rather than hard-fail, so a live-bridge session with no render-state machine still submits.
+    console.log(`[SUBMIT] gate ${gateResult.reason || 'not-ready'} ${id}: dispatching anyway (last_state=${gateResult.last_state})`);
   }
 
   // Step 2: dispatch Enter via the PTY/context path, render-gated (#568).
@@ -4523,6 +4595,7 @@ module.exports = {
   fireAutoReport,                 // #32: provenance-tagged auto-report (deps DI: now/deliver/...)
   maybeRecordInjectConsumption,   // #619: durable early-consumption fact capture (idle-gate decay-proofing)
   forceSubmitDeliveredToSurface,  // #544/#537/Bug B: PTY-native force-confirm (pty_cr = delivered)
+  isTerminalGateFailure,          // #678: submit-gate disposition — no_state is best-effort dispatch, not hard-fail
   terminalLevelSubmit,            // #544: PTY-only submit path (pty_cr | null)
   submitViaPty,                   // #544: bare-0x0D submit into the innermost node-pty
   runSubmitAll,                   // #546: submit-all via PTY for every backend (no cmux send-key)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dmsdc-ai/aigentry-telepty",
-  "version": "0.6.8",
+  "version": "0.6.10",
   "main": "daemon.js",
   "bin": {
     "aigentry-telepty": "install.js",
@@ -37,9 +37,9 @@
   "scripts": {
     "postinstall": "node scripts/postinstall.js",
     "preuninstall": "node scripts/preuninstall.js",
-    "test": "node --require ./test-support/setup-env.js --test test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js && git diff --exit-code tests/snippet-protocol/v1/",
-    "test:watch": "node --require ./test-support/setup-env.js --test --watch test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js",
-    "test:ci": "node --require ./test-support/setup-env.js --test --test-reporter=spec test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js && git diff --exit-code tests/snippet-protocol/v1/",
+    "test": "node --require ./test-support/setup-env.js --test test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/submit-gate-restore-register-678.test.js test/submit-busy-dispatch-694.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js test/inject-notfound-teardown-691.test.js && git diff --exit-code tests/snippet-protocol/v1/",
+    "test:watch": "node --require ./test-support/setup-env.js --test --watch test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/submit-gate-restore-register-678.test.js test/submit-busy-dispatch-694.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js test/inject-notfound-teardown-691.test.js",
+    "test:ci": "node --require ./test-support/setup-env.js --test --test-reporter=spec test/auth.test.js test/http-auth.test.js test/tailnet-autobind.test.js test/win-firewall.test.js test/broker-protocol.test.js test/broker-auth.test.js test/broker-server.test.js test/broker-client.test.js test/daemon-broker-wiring.test.js test/broker-cli.test.js test/broker-integration.test.js test/daemon.test.js test/daemon-singleton.test.js test/daemon-harness-port.test.js test/daemon-bind-default.test.js test/integration/daemon-launch.test.js test/cli.test.js test/subcommand-help.test.js test/telepty-kill.test.js test/dupid-flap-kill-stick-56.test.js test/idle-ttl.test.js test/telepty-clean-older-than.test.js test/lifecycle-transport-agnostic.test.js test/skill-installer.test.js test/interactive-terminal.test.js test/runtime-info.test.js test/session-routing.test.js test/session-state.test.js test/session-store-persistence.test.js test/mailbox-lock.test.js test/report-enforcement.test.js test/enforce-report.test.js test/peer-inject-validator.test.js test/enforce-submit-gate.test.js test/submit-gate.test.js test/submit-via-pty.test.js test/submit-render-gate.test.js test/submit-gate-restore-register-678.test.js test/submit-busy-dispatch-694.test.js test/prompt-symbol-registry.test.js test/inject-submit-flags.test.js test/inject-submit-force-env.test.js test/inject-consumption-evidence.test.js test/host-spec.test.js test/cross-host-inject.test.js test/cross-machine-ssh-routing.test.js test/init.test.js test/install-service-generation.test.js test/install-broker-service.test.js test/uninstall.test.js test/win-resolve-executable.test.js test/version-handshake.test.js test/ensure-daemon-running.test.js test/daemon-restart-fallback-15.test.js test/win-kill-process.test.js test/daemon-control-port-owner.test.js test/daemon-lifecycle-55.test.js test/banner-stderr-jq-safety.test.js test/bridge-supervisor-ipc.test.js test/bridge-j3-shim.test.js test/bridge-e2e.test.js test/release-0.4.5-bugfixes.test.js test/idle-unconfirmed-settle.test.js test/idle-unconfirmed-consumption.test.js test/idle-unconfirmed-decayed-619.test.js test/inject-redeliver.test.js test/provenance.test.js test/inject-audit-broker-seam.test.js test/inject-provenance-daemon.test.js test/inject-audit-log.test.js test/inject-audit-daemon.test.js test/inject-audit-cli.test.js test/inject-notfound-teardown-691.test.js && git diff --exit-code tests/snippet-protocol/v1/",
     "typecheck": "tsc --noEmit",
     "regen-fixtures": "node scripts/regen-snippet-fixtures.js"
   },

package/src/submit-gate.js

@@ -35,6 +35,19 @@ function isFailed(state) {
   return !!(state && FAIL_STATES.has(state.state));
 }
 
+// #694: a "busy-dispatch" state — the recipient is mid-turn (working/thinking, the
+// ACCEPTED_AFTER_SUBMIT_STATES) and has been for at least graceMs. Used by the submit gate to
+// short-circuit the idle-wait on a genuinely-busy target: working/thinking are NOT READY_STATES,
+// so awaitReplReady can never pass mid-turn and would burn its full timeout before best-effort
+// dispatch. The graceMs floor is what distinguishes a REAL ongoing turn from the transient
+// `working` a target emits while echoing our OWN just-injected text (duration_ms ≈ 0) — so a
+// fresh inject never trips the fast-path prematurely, and idle/waiting never match at all.
+function isBusyDispatchState(state, graceMs) {
+  if (!state || !ACCEPTED_AFTER_SUBMIT_STATES.has(state.state)) return false;
+  const grace = Number.isFinite(graceMs) ? graceMs : 0;
+  return Number.isFinite(state.duration_ms) && state.duration_ms >= grace;
+}
+
 /**
  * Wait until the session's REPL is ready to accept Enter.
  *
@@ -784,6 +797,7 @@ module.exports = {
   defaultReadScreen,
   isReady,
   isFailed,
+  isBusyDispatchState,
   READY_STATES,
   FAIL_STATES,
   ACCEPTED_AFTER_SUBMIT_STATES,