@volter/supercode-terminal 0.2.41 → 0.2.42

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-terminal",
-  "version": "0.2.41",
+  "version": "0.2.42",
   "type": "module",
   "description": "Optional leak-safe terminal and tmux host adapter for Volter Harness frontends",
   "exports": {

package/substrate/tmux.js

@@ -75,16 +75,29 @@ export async function createSession({ command, cwd, env, name, owner, ownerOptio
             startArgs.push("-c", cwd);
         }
         startArgs.push(...paneBootstrapArgs(0));
-        await runTmuxSessionCreateAsync(startArgs, { cwd, operation: "create tmux session", session: name, socket });
-        if (owner) {
-            await runAsync("tmux", tmuxArgs(socket, ["set-option", "-t", name, ownershipOption, owner]));
-        }
+        // The session's own options ride the SAME invocation as its creation. A
+        // program that exits at once (a name not on PATH) otherwise takes the
+        // session with it before `remain-on-exit` or the owner stamp is set, and
+        // the next set-option's "no such session" replaces the program's own
+        // failure in what the launcher reports.
+        const setOnCreate = (option, value) => {
+            assertOptionValue(value);
+            // tmux splits commands at an argument that ends in `;`.
+            if (value.endsWith(";"))
+                throw new Error(`A tmux option value set at creation cannot end in ";": ${option}`);
+            startArgs.push(";", "set-option", "-t", name, option, value);
+        };
+        if (owner)
+            setOnCreate(ownershipOption, owner);
         for (const [option, value] of Object.entries(sessionOptions).sort(([left], [right]) => left.localeCompare(right))) {
-            await setSessionOptionAsync(name, option, value, socket);
+            assertUserOption(option);
+            setOnCreate(option, value);
         }
         for (const [option, value] of Object.entries(settings).sort(([left], [right]) => left.localeCompare(right))) {
-            await setSessionSettingAsync(name, option, value, socket);
+            assertSettingName(option);
+            setOnCreate(option, value);
         }
+        await runTmuxSessionCreateAsync(startArgs, { cwd, operation: "create tmux session", session: name, socket });
         await declareSessionColorEnvironmentAsync(socket);
         const target = targetForSession(name);
         for (let i = 1; i < panes; i += 1) {
@@ -150,11 +163,17 @@ export async function sessionOptionValuesAsync(option, socket) {
 export async function setSessionSettingAsync(session, option, value, socket) {
     if (!session)
         throw new Error("A session name is required.");
+    assertSettingName(option);
+    assertOptionValue(value);
+    await runAsync("tmux", tmuxArgs(socket, ["set-option", "-t", session, option, value]));
+}
+function assertSettingName(option) {
     if (!/^[a-z][a-z0-9-]*$/.test(option))
         throw new Error(`Not a tmux session option name: ${option}`);
+}
+function assertOptionValue(value) {
     if (/\0|\t|\r|\n/.test(value))
         throw new Error("A tmux option value cannot contain control separators.");
-    await runAsync("tmux", tmuxArgs(socket, ["set-option", "-t", session, option, value]));
 }
 /** Set one session-scoped user option without exposing it in the session name. */
 export async function setSessionOptionAsync(session, option, value, socket) {