npm - conductor-remote - Versions diffs - 1.131.4 → 1.131.6 - Mend

conductor-remote 1.131.4 → 1.131.6

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (16) hide show

package/README.md +4 -4
package/dist/assets/{AutoModelSettings-SBdRr_mv.js → AutoModelSettings-BV17IdxP.js} +1 -1
package/dist/assets/{PierrePatch-BmY2b7Ei.js → PierrePatch-Dfh5TW8u.js} +1 -1
package/dist/assets/{index-BAEVNqQy.js → index-BY3vTp3J.js} +4 -4
package/dist/index.html +1 -1
package/dist/sw.js +1 -1
package/dist-node/scripts/service/commands.js +2 -2
package/dist-node/scripts/service/flags.js +8 -0
package/dist-node/scripts/service/installation.js +53 -3
package/dist-node/scripts/service/launch-agent.js +14 -0
package/dist-node/scripts/service.js +1 -1
package/dist-node/src/host/ports.js +82 -0
package/dist-node/src/server.js +16 -0
package/dist-node/src/writes/actuator.js +18 -7
package/dist-node/src/writes/sidecar.js +15 -4
package/package.json +1 -1

package/dist/index.html CHANGED Viewed

@@ -25,7 +25,7 @@
 		<title>Conductor Remote</title>
 		<!-- Runs before the module bundle so it can catch a stale shell that fails to boot. -->
 		<script src="/self-heal.js"></script>
-		<script type="module" crossorigin src="/assets/index-BAEVNqQy.js"></script>
+		<script type="module" crossorigin src="/assets/index-BY3vTp3J.js"></script>
 		<link rel="stylesheet" crossorigin href="/assets/index-Dbu5DOed.css">
 	<link rel="manifest" href="/manifest.webmanifest"></head>
 	<body>

package/dist/sw.js CHANGED Viewed

	@@ -1 +1 @@
1	- if(!self.define){let e,s={};const i=(i,n)=>(i=new URL(i+".js",n).href,s[i]\|\|new Promise(s=>{if("document"in self){const e=document.createElement("script");e.src=i,e.onload=s,document.head.appendChild(e)}else e=i,importScripts(i),s()}).then(()=>{let e=s[i];if(!e)throw new Error(`Module ${i} didn’t register its module`);return e}));self.define=(n,r)=>{const l=e\|\|("document"in self?document.currentScript.src:"")\|\|location.href;if(s[l])return;let o={};const t=e=>i(e,l),a={module:{uri:l},exports:o,require:t};s[l]=Promise.all(n.map(e=>a[e]\|\|t(e))).then(e=>(r(...e),o))}}define(["./workbox-dcde9eb3"],function(e){"use strict";importScripts("/push-sw.js"),self.addEventListener("message",e=>{e.data&&"SKIP_WAITING"===e.data.type&&self.skipWaiting()}),e.clientsClaim(),e.precacheAndRoute([{url:"self-heal.js",revision:"49bd63adb25a09341f8d2610e8bd3c76"},{url:"push-sw.js",revision:"4e0b714c0ea53e85c167e6d8f5b767aa"},{url:"index.html",revision:"~~e644d471ce76c93e68869452f62ead76~~"},{url:"assets/workbox-window.prod.es5-BBnX5xw4.js",revision:null},{url:"assets/pierre-shiki-wasm-C8ETeZem.js",revision:null},{url:"assets/pierre-light-480U9XYS.js",revision:null},{url:"assets/pierre-dark-CyvmCCZW.js",revision:null},{url:"assets/index-Dbu5DOed.css",revision:null},{url:"assets/index-~~BAEVNqQy~~.js",revision:null},{url:"assets/PierrePatch-~~BmY2b7Ei~~.js",revision:null},{url:"assets/AutoModelSettings-~~SBdRr_mv~~.js",revision:null},{url:"apple-touch-icon.png",revision:"1127bb396b4648add53dce3f22c92aee"},{url:"icon-192.png",revision:"c5e01ac58768627e18ee7b8b6a9239ef"},{url:"icon-512.png",revision:"a40638c55e310312457a621c9a0002c8"},{url:"icon-maskable-512.png",revision:"a9b0d962686287452216492cd2247499"},{url:"icon.svg",revision:"c1aee186821798733dd477e69a0ef243"},{url:"manifest.webmanifest",revision:"cf88fbc5755108a7fe0616fa160a8a15"}],{}),e.cleanupOutdatedCaches(),e.registerRoute(new e.NavigationRoute(e.createHandlerBoundToURL("/index.html"),{denylist:[/^\/api\//]})),e.registerRoute(({url:e})=>e.pathname.startsWith("/assets/diff-syntax/"),new e.CacheFirst({cacheName:"diff-syntax",plugins:[new e.ExpirationPlugin({maxEntries:32,maxAgeSeconds:31536e3})]}),"GET")});
1	+ if(!self.define){let e,s={};const i=(i,n)=>(i=new URL(i+".js",n).href,s[i]\|\|new Promise(s=>{if("document"in self){const e=document.createElement("script");e.src=i,e.onload=s,document.head.appendChild(e)}else e=i,importScripts(i),s()}).then(()=>{let e=s[i];if(!e)throw new Error(`Module ${i} didn’t register its module`);return e}));self.define=(n,r)=>{const l=e\|\|("document"in self?document.currentScript.src:"")\|\|location.href;if(s[l])return;let o={};const t=e=>i(e,l),a={module:{uri:l},exports:o,require:t};s[l]=Promise.all(n.map(e=>a[e]\|\|t(e))).then(e=>(r(...e),o))}}define(["./workbox-dcde9eb3"],function(e){"use strict";importScripts("/push-sw.js"),self.addEventListener("message",e=>{e.data&&"SKIP_WAITING"===e.data.type&&self.skipWaiting()}),e.clientsClaim(),e.precacheAndRoute([{url:"self-heal.js",revision:"49bd63adb25a09341f8d2610e8bd3c76"},{url:"push-sw.js",revision:"4e0b714c0ea53e85c167e6d8f5b767aa"},{url:"index.html",revision:"ae7f5f4a576844959923e8df23dafac7"},{url:"assets/workbox-window.prod.es5-BBnX5xw4.js",revision:null},{url:"assets/pierre-shiki-wasm-C8ETeZem.js",revision:null},{url:"assets/pierre-light-480U9XYS.js",revision:null},{url:"assets/pierre-dark-CyvmCCZW.js",revision:null},{url:"assets/index-Dbu5DOed.css",revision:null},{url:"assets/index-BY3vTp3J.js",revision:null},{url:"assets/PierrePatch-Dfh5TW8u.js",revision:null},{url:"assets/AutoModelSettings-BV17IdxP.js",revision:null},{url:"apple-touch-icon.png",revision:"1127bb396b4648add53dce3f22c92aee"},{url:"icon-192.png",revision:"c5e01ac58768627e18ee7b8b6a9239ef"},{url:"icon-512.png",revision:"a40638c55e310312457a621c9a0002c8"},{url:"icon-maskable-512.png",revision:"a9b0d962686287452216492cd2247499"},{url:"icon.svg",revision:"c1aee186821798733dd477e69a0ef243"},{url:"manifest.webmanifest",revision:"cf88fbc5755108a7fe0616fa160a8a15"}],{}),e.cleanupOutdatedCaches(),e.registerRoute(new e.NavigationRoute(e.createHandlerBoundToURL("/index.html"),{denylist:[/^\/api\//]})),e.registerRoute(({url:e})=>e.pathname.startsWith("/assets/diff-syntax/"),new e.CacheFirst({cacheName:"diff-syntax",plugins:[new e.ExpirationPlugin({maxEntries:32,maxAgeSeconds:31536e3})]}),"GET")});

package/dist-node/scripts/service/commands.js CHANGED Viewed

@@ -2,11 +2,11 @@ import { config } from "./configuration.js";
 import { FLAG_ENV } from "./flags.js";
 import { install, restart, uninstall } from "./installation.js";
 import { logs, status } from "./presentation.js";
-export function runServiceCommand() {
+export async function runServiceCommand() {
     const cmd = process.argv[2] ?? 'status';
     switch (cmd) {
         case 'install':
-            install();
+            await install();
             break;
         case 'uninstall':
             uninstall();

package/dist-node/scripts/service/flags.js CHANGED Viewed

@@ -1,3 +1,4 @@
+import { parsePortValue } from "../../src/host/ports.js";
 /**
  * Install-time knobs are accepted as documented CLI flags OR the matching env var — a flag wins over the
  * ambient env. Parsed flags are folded back into process.env so everything downstream (and the plist we
@@ -40,6 +41,13 @@ export function applyFlags(argv) {
             console.error(`flag ${name} must be on or off`);
             process.exit(1);
         }
+        // Named here as well as in the install preflight because a typo is worth catching against the
+        // argument that carries it. The preflight still runs: an ambient RELAY_PORT reaches install
+        // without passing through any flag, and only the preflight sees both ports at once.
+        if ((name === '--port' || name === '--voice-port') && parsePortValue(value) === null) {
+            console.error(`flag ${name} must be a whole number from 1 to 65535, not "${value}"`);
+            process.exit(1);
+        }
         process.env[envKey] = value;
     }
 }

package/dist-node/scripts/service/installation.js CHANGED Viewed

@@ -1,9 +1,10 @@
 import fs from 'node:fs';
 import path from 'node:path';
+import { bindFailureLines, listenersOn, parsePortValue, probeListen } from "../../src/host/ports.js";
 import { exposeStorePath } from "../../src/host/tailscale.js";
 import { readVoiceConfig } from "../../src/voice/config.js";
-import { distBuilt, domain, LABEL, logDir, persistPinnedToken, plistPath, projectDir, readPlistEnv } from "./environment.js";
-import { buildPlist, launchctl, reloadAgent } from "./launch-agent.js";
+import { distBuilt, domain, LABEL, logDir, persistPinnedToken, plistPath, projectDir, RELAY_PORT, readPlistEnv, VOICE_PORT } from "./environment.js";
+import { agentPid, buildPlist, launchctl, reloadAgent } from "./launch-agent.js";
 import { ensureTailscale, resolveExposeMode } from "./network.js";
 import { printUrl, printVoiceRoute } from "./presentation.js";
 import { ensureVoiceFunnel } from "./voice.js";
@@ -11,7 +12,55 @@ import { ensureVoiceFunnel } from "./voice.js";
 function isEphemeralInstall(dir) {
     return /[\\/]_npx[\\/]|[\\/]\.npm[\\/]_npx[\\/]/.test(dir);
 }
-export function install() {
+/**
+ * Refuse a port the relay cannot have, before the plist is written and launchd is reloaded.
+ *
+ * The alternative is the daemon's own crash: `server.listen` reaches launchd as an unhandled error
+ * event, KeepAlive respawns it, and the loop's only trace is a stack trace in relay.err.log — while
+ * install itself prints a success line, a phone URL and a QR code for a relay that is not running.
+ * A port our own daemon holds is not a conflict; a re-install boots it out moments later.
+ */
+async function assertPortFree(what, host, port, setting) {
+    const code = await probeListen(host, port);
+    if (code === null)
+        return;
+    if (code === 'EADDRINUSE') {
+        const ours = agentPid();
+        const holders = listenersOn(port);
+        if (ours !== null && holders.length > 0 && holders.every(holder => holder.pid === ours))
+            return;
+    }
+    for (const line of bindFailureLines(what, host, port, code, setting))
+        console.error(line);
+    process.exit(1);
+}
+/** Both ports, resolved from flags or the ambient environment, checked together and against the machine. */
+async function assertPortsUsable() {
+    const relay = parsePortValue(RELAY_PORT);
+    if (relay === null) {
+        console.error(`✗ the relay port must be a whole number from 1 to 65535, not "${RELAY_PORT}".`);
+        console.error('  Set it with `--port <n>` (or `conductor-remote config set port <n>`).');
+        process.exit(1);
+    }
+    const voice = parsePortValue(VOICE_PORT);
+    if (voice === null) {
+        console.error(`✗ the voice port must be a whole number from 1 to 65535, not "${VOICE_PORT}".`);
+        console.error('  Set it with `--voice-port <n>` (or `conductor-remote config set voice-port <n>`).');
+        process.exit(1);
+    }
+    // The two servers live in one process, so equal ports are a collision the relay finds at boot rather
+    // than a configuration the machine can report as busy. 8788 is the voice default and a habitual
+    // second relay port, which is how this pair meets by accident.
+    if (relay === voice) {
+        console.error(`✗ the relay and the voice listener would both bind :${relay}.`);
+        console.error('  They are two servers in one process; give them different ports.');
+        process.exit(1);
+    }
+    await assertPortFree('the relay', process.env.RELAY_HOST ?? '127.0.0.1', relay, 'port');
+    // The voice listener is loopback-only by construction (src/server.ts); Funnel is what publishes it.
+    await assertPortFree('the voice listener', '127.0.0.1', voice, 'voice-port');
+}
+export async function install() {
     if (isEphemeralInstall(projectDir)) {
         console.error(`✗ refusing to install from an npx cache path:\n    ${projectDir}\n` +
             '  That directory is temporary and gets purged, which would break the LaunchAgent.\n' +
@@ -22,6 +71,7 @@ export function install() {
         console.error('✗ dist/ not built. Run `yarn build` first (or use `yarn deploy`, which builds).');
         process.exit(1);
     }
+    await assertPortsUsable();
     persistPinnedToken();
     // Resolve (and persist) the expose posture *before* the plist is written and launchd is reloaded, so
     // the daemon starts with an explicit EXPOSE in its own environment rather than racing the file

package/dist-node/scripts/service/launch-agent.js CHANGED Viewed

@@ -27,6 +27,20 @@ function serviceLoaded() {
         return false;
     }
 }
+/**
+ * The pid of the running LaunchAgent, or null when it is not loaded or not running. Used by the install
+ * preflight to tell "the port is taken" from "the port is taken by the very relay we are about to replace".
+ */
+export function agentPid() {
+    try {
+        const out = execFileSync('launchctl', ['print', `${domain}/${LABEL}`], { encoding: 'utf8', stdio: 'pipe' });
+        const pid = Number(out.match(/pid = (\d+)/)?.[1]);
+        return Number.isInteger(pid) ? pid : null;
+    }
+    catch {
+        return null;
+    }
+}
 /**
  * Reload the agent from the freshly written plist. `bootout` of a *running* instance is asynchronous,
  * so we wait for it to fully unload before `bootstrap` — otherwise bootstrap races the teardown and

package/dist-node/scripts/service.js CHANGED Viewed

@@ -14,4 +14,4 @@ if ((process.argv[2] ?? 'status') !== 'logs')
     applyFlags(process.argv.slice(3));
 // Import only after parsing flags: environment.ts freezes the relay/voice ports.
 const { runServiceCommand } = await import("./service/commands.js");
-runServiceCommand();
+await runServiceCommand();

package/dist-node/src/host/ports.js ADDED Viewed

@@ -0,0 +1,82 @@
+import { execFileSync } from 'node:child_process';
+import net from 'node:net';
+/**
+ * Port knobs arrive as free text — a `--port` flag, a plist entry, an ambient shell variable — and every
+ * one of them is handed straight to `server.listen`, which rejects a non-number with a bare
+ * ERR_SOCKET_BAD_PORT inside the daemon. Reject it here instead, where a person is reading the output.
+ */
+export function parsePortValue(raw) {
+    if (raw === undefined || raw === null)
+        return null;
+    const text = raw.trim();
+    if (!/^\d+$/.test(text))
+        return null;
+    const port = Number(text);
+    return port >= 1 && port <= 65535 ? port : null;
+}
+/**
+ * Parse `lsof -Fpcn` field output into one entry per listening process. The field form is used rather
+ * than the default table because lsof's COMMAND column carries spaces ("Google Chrome H"), so a column
+ * split names the wrong holder. Process fields are printed once per process, ahead of its file blocks,
+ * which is what keeps a process holding several matching descriptors from being reported several times.
+ */
+export function parseLsofListeners(out) {
+    const holders = [];
+    let pid = null;
+    for (const line of out.split('\n')) {
+        if (line.startsWith('p')) {
+            const parsed = Number(line.slice(1));
+            pid = Number.isInteger(parsed) ? parsed : null;
+        }
+        else if (line.startsWith('c') && pid !== null) {
+            holders.push({ command: line.slice(1), pid });
+            pid = null;
+        }
+    }
+    return holders;
+}
+/** Who is listening on `port`, best-effort — this only ever decorates a message that has already failed. */
+export function listenersOn(port) {
+    try {
+        const out = execFileSync('lsof', ['-nP', `-iTCP:${port}`, '-sTCP:LISTEN', '-Fpcn'], {
+            encoding: 'utf8',
+            stdio: 'pipe'
+        });
+        return parseLsofListeners(out);
+    }
+    catch {
+        // lsof exits non-zero when nothing matches, and it may not be installed at all.
+        return [];
+    }
+}
+/**
+ * Can this process bind `host:port`? Answered by binding it, because that is the only test that agrees
+ * with the real listen in every case that matters: a wildcard listener blocks a loopback bind, a
+ * privileged port is refused rather than busy, and a host the machine does not hold fails on its own.
+ * Resolves the error code, or null when the port is free. The socket never accepts, so it leaves no
+ * TIME_WAIT behind and the port is free again the moment this resolves.
+ */
+export function probeListen(host, port) {
+    return new Promise(resolve => {
+        const probe = net.createServer();
+        probe.once('error', (err) => resolve(err.code ?? 'EUNKNOWN'));
+        probe.listen(port, host, () => probe.close(() => resolve(null)));
+    });
+}
+export function describeHolders(holders) {
+    if (!holders.length)
+        return 'an unidentified process';
+    return holders.map(holder => `${holder.command} (pid ${holder.pid})`).join(', ');
+}
+/**
+ * The two lines a failed bind prints, shared by the install preflight and the running relay so both name
+ * the same port and the same knob. `config set` is the hint in both because it is the one lever that
+ * works against an installed daemon, which is where this failure is otherwise invisible.
+ */
+export function bindFailureLines(what, host, port, code, setting) {
+    const cause = code === 'EADDRINUSE' ? `held by ${describeHolders(listenersOn(port))}` : (code ?? 'unknown error');
+    return [
+        `✗ ${what} cannot bind ${host}:${port} — ${cause}.`,
+        `  Free that port, or move it: conductor-remote config set ${setting} <port>`
+    ];
+}

package/dist-node/src/server.js CHANGED Viewed

@@ -1,6 +1,7 @@
 import { startAutoUpdate } from "./host/autoupdate.js";
 import { startFunnelWatchdog } from "./host/funnel-watchdog.js";
 import { watchNoSleepExpiry } from "./host/nosleep.js";
+import { bindFailureLines } from "./host/ports.js";
 import { driftWarningLines, tailscaleBin } from "./host/tailscale.js";
 import { createRelayServer } from "./http/router.js";
 import { createRelayServices } from "./http/services.js";
@@ -9,6 +10,21 @@ import { voicePort } from "./voice/config.js";
 const services = createRelayServices();
 const { cfg, voiceServer, voiceBroker, voiceHistory, orchestration, relayIdentity, recoverUiLease, wakeWorkflows, actuator, firstPrompts, autoModels, sweepStagedAttachments, STAGED_ATTACHMENT_SWEEP_MS, parkedPrompts, delegationQueue, liveDelegationStores, devServers, reads, sessionPoller } = services;
 const server = createRelayServer(services);
+// A `listen` failure arrives as an 'error' event, and with nobody listening for it the process throws.
+// Under launchd that is a KeepAlive restart loop whose only trace is a stack trace in relay.err.log with
+// the port nowhere in it, while `service status` says only "loaded but not running". Name the port.
+server.on('error', (err) => {
+    for (const line of bindFailureLines('the relay', cfg.host, cfg.port, err.code ?? null, 'port'))
+        console.error(line);
+    process.exit(1);
+});
+// The voice listener is opt-in and the control panel is the product, so its own bind failure is loud
+// rather than fatal: taking the phone's relay down over an unconfigured second server is the worse trade.
+// `service status` reports the voice route separately, so a listener that never came up still shows there.
+voiceServer.on('error', (err) => {
+    for (const line of bindFailureLines('the voice listener', '127.0.0.1', voicePort(), err.code ?? null, 'voice-port'))
+        console.error(line);
+});
 server.listen(cfg.port, cfg.host, () => {
     voiceServer.listen(voicePort(), '127.0.0.1', () => {
         console.info(`  voice:      127.0.0.1:${voicePort()}${voiceBroker ? '' : ' (waiting for OpenAI config)'}`);

package/dist-node/src/writes/actuator.js CHANGED Viewed

@@ -7,14 +7,16 @@ import { exec, uiTurn } from "./ui-lock.js";
  * `sessionId` over Conductor's own dispatch socket (see src/writes/sidecar.ts), so it needs
  * no window focus and the app UI reflects the turn correctly.
  *
- * Opt-in (WRITE_STRATEGY=sidecar) because it speaks a private, versioned IPC and
- * hasn't been validated by an automated live send (that would inject a prompt
- * into a running agent). It is the intended default once you've confirmed it on
- * your setup.
+ * Opt-in (WRITE_STRATEGY=sidecar) because it speaks a private, versioned IPC.
+ * BROKEN against current Conductor builds — the send payload is rejected
+ * upstream (schema drift, see src/writes/sidecar.ts and issue #97). Do not
+ * select it expecting sends to land; AppleScript is the working path. There is
+ * no fallback to AppleScript on send failure: a failed sidecar send reports
+ * failure. Fixes welcome.
  */
 export class SidecarActuator {
     name = 'sidecar';
-    caveat = 'Delivered straight to the target session over Conductor’s dispatch socket — precise per-workspace targeting.';
+    caveat = 'Broken against current Conductor builds (see issue #97) — AppleScript is the working path.';
     precise = true;
     available() {
         return sidecarAvailable();
@@ -25,11 +27,20 @@ export class SidecarActuator {
         if (!sessionId)
             return { ok: false, strategy: this.name, error: 'no session id to target' };
         try {
-            await sidecarSendUserMessage(sessionId, text, options.queue ? 'queue' : 'default');
+            // `options.queue` has no sidecar equivalent: the live schema accepts
+            // only 'default' | 'steering' and rejects the old 'queue' value, so
+            // always send 'default' rather than a known-invalid enum.
+            void options.queue;
+            await sidecarSendUserMessage(sessionId, text, 'default');
             return { ok: true, strategy: this.name };
         }
         catch (err) {
-            return { ok: false, strategy: this.name, error: err instanceof Error ? err.message : String(err) };
+            const raw = err instanceof Error ? err.message : String(err);
+            return {
+                ok: false,
+                strategy: this.name,
+                error: `${raw} (sidecar sends are broken against current Conductor builds — see issue #97; AppleScript is the working path)`
+            };
         }
     }
 }

package/dist-node/src/writes/sidecar.js CHANGED Viewed

@@ -13,14 +13,20 @@ import path from 'node:path';
  * `sessionId`, so there is no window focus or AppleScript involved and the app's
  * UI updates correctly because it's the real dispatch path. It is also the most
  * update-fragile surface (a private, versioned IPC), which is why it lives
- * behind the Actuator interface and falls back to AppleScript when the socket
- * can't be reached (see src/writes/actuator.ts).
+ * behind the Actuator interface as an opt-in (`WRITE_STRATEGY=sidecar`).
+ *
+ * BROKEN against current Conductor builds: the send payload below is rejected
+ * by the live sidecar (`ok:false`, schema drift since the 0.76 recon) and the
+ * `type:"query"` shape validates then silently drops the prompt for sessions
+ * that aren't live in the sidecar's store. AppleScript is the working send
+ * path. Fixes welcome — see issue #97 for the probe handover.
  *
  * Reverse-engineered from conductor-runtime (Conductor 0.76):
  *   - socket:      `$TMPDIR/conductor-sidecar-v2-<sidecarPid>.sock`
  *   - transport:   newline-delimited JSON-RPC 2.0 (`{jsonrpc,id,method,params}`)
  *   - local auth:  the literal `{ userId: 'local', auth: 'local' }`
- *   - send prompt: method `query`, params `{ type: 'sendUserMessageRequest', … }`
+ *   - send prompt (STALE — rejected by current builds, see above):
+ *     method `query`, params `{ type: 'sendUserMessageRequest', … }`
  *   - safe read:   method `contextUsage`, params `{ sessionId, …auth }`
  *
  * Stale socket files from exited sidecars linger in `$TMPDIR`, so discovery is
@@ -129,7 +135,9 @@ async function rpc(method, params, timeoutMs = 8000) {
     }
     throw lastErr instanceof Error ? lastErr : new Error('sidecar unreachable');
 }
-/** Resolve a connectable sidecar socket, or null. Used to decide write strategy. */
+/** True when a sidecar socket accepts connections. Note: a reachable socket
+ * does NOT mean sends work — the send payload is currently rejected upstream
+ * (see header). Used to decide write strategy. */
 export function sidecarSocket(timeoutMs = 800) {
     const candidates = listSidecarSockets();
     return (async () => {
@@ -165,6 +173,9 @@ export async function sidecarAvailable() {
 /**
  * Deliver a prompt to a specific session — the real send path, precisely
  * targeted. Resolves once the sidecar has accepted (queued/sent) the message.
+ *
+ * Currently BROKEN upstream (see header): expect rejection against current
+ * Conductor builds. Failures must surface, never report success.
  */
 export async function sidecarSendUserMessage(sessionId, text, deliveryMode = 'default') {
     await rpc('query', {

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
 	"name": "conductor-remote",
-	"version": "1.131.4",
+	"version": "1.131.6",
 	"type": "module",
 	"packageManager": "yarn@4.15.0",
 	"description": "Phone control panel for local Conductor agents. Reads ride SQLite + git; prompts ride Conductor's own dispatch path.",