@wangyaoshen/remux 0.3.9-dev.390cb29 → 0.3.10-dev.574c4d2

package/tests/pty-daemon.test.js

@@ -7,6 +7,7 @@ import net from "net";
 import fs from "fs";
 import path from "path";
 import { fileURLToPath } from "url";
+import pty from "node-pty";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const DAEMON_SCRIPT = path.join(__dirname, "..", "pty-daemon.js");
@@ -21,6 +22,24 @@ const TAG_SNAPSHOT_REQ = 0x06;
 const TAG_SNAPSHOT_RES = 0x07;
 const TAG_SHUTDOWN = 0xff;
 
+function supportsNodePty() {
+  try {
+    const proc = pty.spawn("/bin/sh", ["-lc", "exit 0"], {
+      name: "xterm-256color",
+      cols: 80,
+      rows: 24,
+      cwd: "/tmp",
+      env: process.env,
+    });
+    proc.kill();
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+const describePtyLifecycle = supportsNodePty() ? describe : describe.skip;
+
 function encodeFrame(tag, payload) {
   const data =
     typeof payload === "string" ? Buffer.from(payload, "utf8") : payload;
@@ -137,7 +156,7 @@ describe("TLV Frame Codec", () => {
   });
 });
 
-describe("PTY Daemon Lifecycle", () => {
+describePtyLifecycle("PTY Daemon Lifecycle", () => {
   let daemon;
   let socketPath;
 

package/tests/server.test.js

@@ -11,6 +11,7 @@ import WebSocket from "ws";
 import fs from "fs";
 import path from "path";
 import { homedir } from "os";
+import pty from "node-pty";
 
 const PORT = 19876 + Math.floor(Math.random() * 1000); // randomized test port
 const TOKEN = "test-token-" + Date.now();
@@ -20,6 +21,24 @@ const PERSIST_FILE = path.join(PERSIST_DIR, `sessions-${INSTANCE_ID}.json`);
 const DB_FILE = path.join(PERSIST_DIR, `remux-${INSTANCE_ID}.db`);
 let serverProc;
 
+function supportsNodePty() {
+  try {
+    const proc = pty.spawn("/bin/sh", ["-lc", "exit 0"], {
+      name: "xterm-256color",
+      cols: 80,
+      rows: 24,
+      cwd: "/tmp",
+      env: process.env,
+    });
+    proc.kill();
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+const PTY_SUPPORTED = supportsNodePty();
+
 function httpGet(urlPath) {
   return new Promise((resolve, reject) => {
     http.get(`http://localhost:${PORT}${urlPath}`, (res) => {
@@ -113,7 +132,7 @@ beforeAll(async () => {
   // Explicitly remove REMUX_PASSWORD to avoid env leaking from parent
   const cleanEnv = { ...process.env };
   delete cleanEnv.REMUX_PASSWORD;
-  serverProc = spawn("node", ["server.js"], {
+  serverProc = spawn(process.execPath, ["server.js"], {
     env: {
       ...cleanEnv,
       PORT: String(PORT),
@@ -185,6 +204,15 @@ describe("HTTP", () => {
     expect(res.body).toContain("<title>Remux</title>");
   });
 
+  it("serves emergency mode shell without workspace or device chrome", async () => {
+    const res = await httpGet(`/?token=${TOKEN}`);
+    expect(res.status).toBe(200);
+    expect(res.body).toContain("cmd-input");
+    expect(res.body).not.toContain("Workspace");
+    expect(res.body).not.toContain("Devices");
+    expect(res.body).not.toContain("Enable Notifications");
+  });
+
   it("serves ghostty-web JS", async () => {
     const res = await httpGet("/dist/ghostty-web.js");
     expect(res.status).toBe(200);
@@ -513,16 +541,18 @@ describe("client connection state", () => {
     const ws2 = await connectAuthed();
 
     // ws1 attaches first (active)
-    await sendAndCollect(
+    const msgs1 = await sendAndCollect(
       ws1,
       { type: "attach_first", session: "main", cols: 80, rows: 24 },
       { timeout: 3000 },
     );
+    const att1 = msgs1.find((m) => m.type === "attached");
+    expect(att1).toBeDefined();
 
     // ws2 attaches to same tab (observer)
     const msgs2 = await sendAndCollect(
       ws2,
-      { type: "attach_first", session: "main", cols: 80, rows: 24 },
+      { type: "attach_tab", tabId: att1.tabId, cols: 80, rows: 24 },
       { timeout: 3000 },
     );
     const att2 = msgs2.find((m) => m.type === "attached");
@@ -552,11 +582,12 @@ describe("client connection state", () => {
     const att1 = msgs1.find((m) => m.type === "attached");
     expect(att1.role).toBe("active");
     const clientId1 = att1.clientId;
+    const tabId = att1.tabId;
 
     // ws2 attaches (observer)
     const msgs2 = await sendAndCollect(
       ws2,
-      { type: "attach_first", session: "main", cols: 80, rows: 24 },
+      { type: "attach_tab", tabId, cols: 80, rows: 24 },
       { timeout: 3000 },
     );
     const att2 = msgs2.find((m) => m.type === "attached");
@@ -617,11 +648,12 @@ describe("client connection state", () => {
     const att1 = msgs1.find((m) => m.type === "attached");
     expect(att1.role).toBe("active");
     const clientId1 = att1.clientId;
+    const tabId = att1.tabId;
 
     // ws2 attaches (observer)
     const msgs2 = await sendAndCollect(
       ws2,
-      { type: "attach_first", session: "main", cols: 80, rows: 24 },
+      { type: "attach_tab", tabId, cols: 80, rows: 24 },
       { timeout: 3000 },
     );
     const att2 = msgs2.find((m) => m.type === "attached");
@@ -669,16 +701,18 @@ describe("client connection state", () => {
     const ws2 = await connectAuthed();
 
     // ws1 attaches (active)
-    await sendAndCollect(
+    const msgs1 = await sendAndCollect(
       ws1,
       { type: "attach_first", session: "main", cols: 80, rows: 24 },
       { timeout: 3000 },
     );
+    const att1 = msgs1.find((m) => m.type === "attached");
+    expect(att1).toBeDefined();
 
     // ws2 attaches (observer)
     const msgs2 = await sendAndCollect(
       ws2,
-      { type: "attach_first", session: "main", cols: 80, rows: 24 },
+      { type: "attach_tab", tabId: att1.tabId, cols: 80, rows: 24 },
       { timeout: 3000 },
     );
     const att2 = msgs2.find((m) => m.type === "attached");
@@ -720,15 +754,15 @@ describe("multi-client", () => {
       { type: "attach_first", session: "main", cols: 100, rows: 30 },
       { timeout: 3000 },
     );
+    const att1 = msgs1.find((m) => m.type === "attached");
+    expect(att1).toBeDefined();
     const msgs2 = await sendAndCollect(
       ws2,
-      { type: "attach_first", session: "main", cols: 80, rows: 24 },
+      { type: "attach_tab", tabId: att1.tabId, cols: 80, rows: 24 },
       { timeout: 3000 },
     );
 
-    const att1 = msgs1.find((m) => m.type === "attached");
     const att2 = msgs2.find((m) => m.type === "attached");
-    expect(att1).toBeDefined();
     expect(att2).toBeDefined();
     expect(att1.tabId).toBe(att2.tabId);
 
@@ -840,7 +874,11 @@ describe("inspect", () => {
     const result = msgs.find((m) => m.type === "inspect_result");
     expect(result).toBeDefined();
     expect(typeof result.text).toBe("string");
-    expect(result.text).toContain("inspect-test-output");
+    if (PTY_SUPPORTED) {
+      expect(result.text).toContain("inspect-test-output");
+    } else {
+      expect(result.text).toContain("Shell unavailable on this machine");
+    }
     expect(result.meta).toBeDefined();
     expect(result.meta.session).toBe("main");
     expect(typeof result.meta.cols).toBe("number");

package/vitest.config.js

@@ -2,6 +2,7 @@ import { defineConfig } from "vitest/config";
 
 export default defineConfig({
   test: {
+    fileParallelism: false,
     testTimeout: 20000,
     hookTimeout: 40000,
     include: ["tests/**/*.test.{js,ts}"],