@lark-codem/codem-core 0.8.80-dev.100103109015.0.g581b3b4 → 0.8.80-dev.100103168054.0.gb59abde

package/bin/lib/control-fd.js

@@ -20,9 +20,11 @@
 // "an old linco that never writes fd 3" (codem-bridge says so in a comment). Net
 // effect: `/space` reported success and the selection was silently lost.
 //
-// SCOPE — unix only. linco's writer is a `#[cfg(not(unix))]` no-op on Windows
+// The legacy /space channel is unix only. linco's writer is a
+// `#[cfg(not(unix))]` no-op on Windows
 // (crates/linco-cli/src/space_channel.rs), so there is nothing to forward there and
-// no reason to perturb Windows process creation.
+// no reason to forward that legacy channel on Windows. Workflow's required
+// pair is distinct and is inherited on every supported platform.
 
 /// The env var through which codem-bridge advertises the control fd.
 const CONTROL_FD_ENV = "CODEM_PROJECT_CHANNEL_FD";
@@ -30,6 +32,24 @@ const CONTROL_FD_ENV = "CODEM_PROJECT_CHANNEL_FD";
 /// stdio for a plain interactive run: the terminal and nothing else.
 const TERMINAL_STDIO = ["inherit", "inherit", "inherit"];
 
+// Workflow's private request/reply transport uses fd 3 and fd 4. Unlike
+// /space's optional notification, both channels are required for execution.
+const WORKFLOW_RESOURCE_PROVIDER_ARG = "--workflow-resource-provider-v1";
+
+function workflowProviderRequested(args = []) {
+  for (let i = 0; i < args.length; i++) {
+    // Core's prompt option allows hyphen-leading values, including this
+    // private flag or the option separator as literal user text.
+    if (args[i] === "--prompt" || args[i] === "-p") {
+      i++;
+      continue;
+    }
+    if (args[i] === "--") return false;
+    if (args[i] === WORKFLOW_RESOURCE_PROVIDER_ARG) return true;
+  }
+  return false;
+}
+
 /// Parse the advertised fd, or null when there is nothing to forward.
 ///
 /// Only fd 3 is expressible here: `stdio` slots are positional, so the 4th entry
@@ -50,7 +70,10 @@ function controlFdFromEnv(env) {
 
 /// Build the stdio array to hand the platform binary. Returns a fresh array —
 /// `launch` mutates it when degrading after an EBADF.
-function buildStdio({ env = process.env, platform = process.platform } = {}) {
+function buildStdio({ env = process.env, platform = process.platform, args = process.argv.slice(2) } = {}) {
+  if (workflowProviderRequested(args)) {
+    return [...TERMINAL_STDIO, "inherit", "inherit"];
+  }
   if (platform === "win32") {
     return TERMINAL_STDIO.slice();
   }
@@ -60,4 +83,4 @@ function buildStdio({ env = process.env, platform = process.platform } = {}) {
   return [...TERMINAL_STDIO, "inherit"];
 }
 
-module.exports = { buildStdio, controlFdFromEnv, CONTROL_FD_ENV, TERMINAL_STDIO };
+module.exports = { buildStdio, controlFdFromEnv, CONTROL_FD_ENV, TERMINAL_STDIO, workflowProviderRequested };

package/bin/lib/launch.js

@@ -11,7 +11,7 @@
 // naming an internal file instead of a sentence about their own machine. Both paths
 // must funnel into the same reporter.
 
-const { TERMINAL_STDIO } = require("./control-fd");
+const { TERMINAL_STDIO, workflowProviderRequested } = require("./control-fd");
 
 /// Human-readable follow-up for the errnos we can actually say something useful
 /// about. Anything else falls through to the bare message.
@@ -62,7 +62,8 @@ function reportFatal({ error, binary, platform, stderr, exit }) {
 /// `stdio` may carry a 4th slot for codem-bridge's /space control fd. If the host
 /// advertised that fd but did not actually pass it, spawning fails with EBADF —
 /// degrade to the terminal-only trio and start anyway rather than refusing to run
-/// over a non-essential side channel.
+/// over a non-essential side channel. Workflow's private fd 3/4 pair is
+/// required: missing channels terminate launch instead of using that fallback.
 function launch({
   binary,
   args,
@@ -80,7 +81,7 @@ function launch({
   try {
     child = attempt(stdio);
   } catch (error) {
-    if (stdio.length > 3 && error && error.code === "EBADF") {
+    if (stdio.length > 3 && error && error.code === "EBADF" && !workflowProviderRequested(args)) {
       stderr(
         "linco: the host advertised a /space control fd it did not pass; continuing without it"
       );

package/bin/linco

@@ -46,7 +46,7 @@ function main() {
     binary,
     args: process.argv.slice(2),
     env: environment,
-    stdio: buildStdio(),
+    stdio: buildStdio({ args: process.argv.slice(2) }),
   });
   if (!child) {
     // launch() already reported the failure and called exit(1).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lark-codem/codem-core",
-  "version": "0.8.80-dev.100103109015.0.g581b3b4",
+  "version": "0.8.80-dev.100103168054.0.gb59abde",
   "description": "CodeM CLI.",
   "license": "SEE LICENSE IN LICENSE",
   "bin": {
@@ -11,12 +11,12 @@
     "LICENSE"
   ],
   "optionalDependencies": {
-    "@lark-codem/codem-core-darwin-arm64": "0.8.80-dev.100103109015.0.g581b3b4",
-    "@lark-codem/codem-core-darwin-x64": "0.8.80-dev.100103109015.0.g581b3b4",
-    "@lark-codem/codem-core-linux-arm64-gnu": "0.8.80-dev.100103109015.0.g581b3b4",
-    "@lark-codem/codem-core-linux-x64-gnu": "0.8.80-dev.100103109015.0.g581b3b4",
-    "@lark-codem/codem-core-win32-arm64-gnu": "0.8.80-dev.100103109015.0.g581b3b4",
-    "@lark-codem/codem-core-win32-x64-msvc": "0.8.80-dev.100103109015.0.g581b3b4"
+    "@lark-codem/codem-core-darwin-arm64": "0.8.80-dev.100103168054.0.gb59abde",
+    "@lark-codem/codem-core-darwin-x64": "0.8.80-dev.100103168054.0.gb59abde",
+    "@lark-codem/codem-core-linux-arm64-gnu": "0.8.80-dev.100103168054.0.gb59abde",
+    "@lark-codem/codem-core-linux-x64-gnu": "0.8.80-dev.100103168054.0.gb59abde",
+    "@lark-codem/codem-core-win32-arm64-gnu": "0.8.80-dev.100103168054.0.gb59abde",
+    "@lark-codem/codem-core-win32-x64-msvc": "0.8.80-dev.100103168054.0.gb59abde"
   },
   "engines": {
     "node": ">=18"