remodex-windows-fix 1.0.2 → 1.0.4

package/README.md

@@ -1,183 +1,186 @@
-# remodex-windows-fix
-
-Windows-friendly mirror of `remodex` with a launcher fix for `codex app-server`.
-
-## What Changed
-
-- Fixes Windows startup when `remodex up` tries to spawn `codex` and fails with `spawn codex ENOENT`.
-- Prefers `codex.cmd` or `codex.bat` on Windows and runs them with shell support.
-- Falls back to `codex.exe` when a wrapper script is not present.
-- Adds `REMODEX_CODEX_BIN` and `PHODEX_CODEX_BIN` overrides for explicit Codex binary selection.
-- Prevents immediate uncaught startup crashes by routing launcher failures through Remodex error handling.
-
-## Why This Exists
-
-On Windows, `codex` is often installed through npm as a `.cmd` shim. A plain Node `spawn("codex", ["app-server"])` call can fail even when `codex` works in PowerShell. This fork resolves the executable explicitly before launching the bridge.
-
-## Install
-
-Install from this GitHub repository:
-
-```bash
-npm install -g github:needitem/remodex-windows-fix
-```
-
-Global CLI command:
-
-```bash
-remodex-windows-fix
-```
-
-## Usage
-
-Start the bridge:
-
-```bash
-remodex-windows-fix up
-```
-
-Resume the last active thread:
-
-```bash
-remodex-windows-fix resume
-```
-
-Watch rollout output:
-
-```bash
-remodex-windows-fix watch [threadId]
-```
-
-## Windows Override
-
-If you want to force a specific Codex binary, set one of these environment variables before running `remodex up`:
-
-```powershell
-$env:REMODEX_CODEX_BIN = "C:\Users\th072\AppData\Roaming\npm\codex.cmd"
-```
-
-Legacy alias:
-
-```powershell
-$env:PHODEX_CODEX_BIN = "C:\Users\th072\AppData\Roaming\npm\codex.cmd"
-```
-
-## Self-Hosted Relay
-
-This repository now includes the upstream-compatible relay code and a local runner.
-
-Start your own relay:
-
-```bash
-npm run relay
-```
-
-Or use the bundled binary directly:
-
-```bash
-remodex-relay
-```
-
-Optional relay host/port overrides:
-
-```powershell
-$env:REMODEX_RELAY_HOST = "0.0.0.0"
-$env:REMODEX_RELAY_PORT = "9000"
-```
-
-Point the bridge at your relay:
-
-```powershell
-$env:REMODEX_RELAY = "ws://YOUR_HOST:9000/relay"
-remodex up
-```
-
-For TLS/reverse-proxy setups, use the public `wss://YOUR_DOMAIN/relay` URL instead.
-
-Health endpoint:
-
-```text
-GET /health
-```
-
-## Cloudflare Deploy
-
-This repository also includes a Cloudflare Workers relay implementation in [cloudflare/worker.mjs](cloudflare/worker.mjs) with Durable Objects configured in [wrangler.toml](wrangler.toml).
-
-GitHub itself still does not host persistent WebSocket servers, but Cloudflare Workers can deploy this relay directly from your GitHub repository without running anything locally.
-
-Import this repository in Cloudflare:
-
-1. Open Cloudflare Workers & Pages.
-2. Create or import a Worker from your GitHub repository.
-3. Use the worker name `remodex-relay` so it matches [wrangler.toml](wrangler.toml).
-4. Deploy the repository as-is.
-5. After deploy, use the public Worker URL as the relay base:
-
-```powershell
-$env:REMODEX_RELAY = "wss://YOUR-WORKER.YOUR-SUBDOMAIN.workers.dev/relay"
-remodex-windows-fix up
-```
-
-Current deployed example for this repository:
-
-```powershell
-$env:REMODEX_RELAY = "wss://remodex-relay.th07290828.workers.dev/relay"
-remodex-windows-fix up
-```
-
-If you are using `cmd.exe` instead of PowerShell:
-
-```cmd
-set REMODEX_RELAY=wss://remodex-relay.th07290828.workers.dev/relay
-remodex-windows-fix up
-```
-
-Health check:
-
-```text
-https://remodex-relay.th07290828.workers.dev/health
-```
-
-Cloudflare health endpoint:
-
-```text
-GET /health
-```
-
-## Alternative GitHub Deploy
-
-If you prefer a normal Node web service instead of Workers, this repository still includes the Render-compatible runner in [render.yaml](render.yaml).
-
-```powershell
-$env:REMODEX_RELAY = "wss://YOUR-SERVICE.onrender.com/relay"
-remodex-windows-fix up
-```
-
-## Validation
-
-Typical local verification:
-
-```bash
-codex --version
-codex app-server --help
-remodex-windows-fix up
-```
-
-Expected Windows resolution after the fix:
-
-```text
-C:\Users\th072\AppData\Roaming\npm\codex.cmd app-server
-```
-
-## Project Layout
-
-```text
-bin/
-src/
-package.json
-```
-
-## Attribution
-
-This repository preserves the upstream Remodex package structure and credits the original package author in `package.json`. This fork adds a Windows launcher compatibility patch and accompanying documentation.
+# remodex-windows-fix
+
+Windows-friendly mirror of `remodex` with a launcher fix for `codex app-server`.
+
+## What Changed
+
+- Fixes Windows startup when `remodex up` tries to spawn `codex` and fails with `spawn codex ENOENT`.
+- Prefers `codex.cmd` or `codex.bat` on Windows and runs them with shell support.
+- Falls back to `codex.exe` when a wrapper script is not present.
+- Adds `REMODEX_CODEX_BIN` and `PHODEX_CODEX_BIN` overrides for explicit Codex binary selection.
+- Prevents immediate uncaught startup crashes by routing launcher failures through Remodex error handling.
+
+## Why This Exists
+
+On Windows, `codex` is often installed through npm as a `.cmd` shim. A plain Node `spawn("codex", ["app-server"])` call can fail even when `codex` works in PowerShell. This fork resolves the executable explicitly before launching the bridge.
+
+## Install
+
+Install from npm:
+
+```bash
+npm install -g remodex-windows-fix
+```
+
+Global CLI command:
+
+```bash
+remodex-windows-fix
+```
+
+## Usage
+
+Start the bridge:
+
+```bash
+remodex-windows-fix up
+```
+
+Resume the last active thread:
+
+```bash
+remodex-windows-fix resume
+```
+
+Watch rollout output:
+
+```bash
+remodex-windows-fix watch [threadId]
+```
+
+## Windows Override
+
+If you want to force a specific Codex binary, set one of these environment variables before running `remodex up`:
+
+```powershell
+$env:REMODEX_CODEX_BIN = "C:\Users\th072\AppData\Roaming\npm\codex.cmd"
+```
+
+Legacy alias:
+
+```powershell
+$env:PHODEX_CODEX_BIN = "C:\Users\th072\AppData\Roaming\npm\codex.cmd"
+```
+
+## Self-Hosted Relay
+
+This repository now includes the upstream-compatible relay code and a local runner.
+
+Start your own relay:
+
+```bash
+npm run relay
+```
+
+Or use the bundled binary directly:
+
+```bash
+remodex-relay
+```
+
+Optional relay host/port overrides:
+
+```powershell
+$env:REMODEX_RELAY_HOST = "0.0.0.0"
+$env:REMODEX_RELAY_PORT = "9000"
+```
+
+Point the bridge at your relay:
+
+```powershell
+$env:REMODEX_RELAY = "ws://YOUR_HOST:9000/relay"
+remodex-windows-fix up
+```
+
+For TLS/reverse-proxy setups, use the public `wss://YOUR_DOMAIN/relay` URL instead.
+
+Health endpoint:
+
+```text
+GET /health
+```
+
+## Cloudflare Deploy
+
+This repository also includes a Cloudflare Workers relay implementation in [cloudflare/worker.mjs](cloudflare/worker.mjs) with Durable Objects configured in [wrangler.toml](wrangler.toml).
+
+GitHub itself still does not host persistent WebSocket servers, but Cloudflare Workers can deploy this relay directly from your GitHub repository without running anything locally.
+
+Import this repository in Cloudflare:
+
+1. Open Cloudflare Workers & Pages.
+2. Create or import a Worker from your GitHub repository.
+3. Use the worker name `remodex-relay` so it matches [wrangler.toml](wrangler.toml).
+4. Deploy the repository as-is.
+5. After deploy, use the public Worker URL as the relay base:
+
+```powershell
+$env:REMODEX_RELAY = "wss://YOUR-WORKER.YOUR-SUBDOMAIN.workers.dev/relay"
+remodex-windows-fix up
+```
+
+Current deployed example for this repository:
+
+```powershell
+$env:REMODEX_RELAY = "wss://remodex-relay.th07290828.workers.dev/relay"
+remodex-windows-fix up
+```
+
+If you are using `cmd.exe` instead of PowerShell:
+
+```cmd
+set REMODEX_RELAY=wss://remodex-relay.th07290828.workers.dev/relay
+remodex-windows-fix up
+```
+
+Health check:
+
+```text
+https://remodex-relay.th07290828.workers.dev/health
+```
+
+Cloudflare health endpoint:
+
+```text
+GET /health
+```
+
+## Alternative GitHub Deploy
+
+If you prefer a normal Node web service instead of Workers, this repository still includes the Render-compatible runner in [render.yaml](render.yaml).
+
+```powershell
+$env:REMODEX_RELAY = "wss://YOUR-SERVICE.onrender.com/relay"
+remodex-windows-fix up
+```
+
+## Validation
+
+Typical local verification:
+
+```bash
+codex --version
+codex app-server --help
+remodex-windows-fix up
+```
+
+Expected Windows resolution after the fix:
+
+```text
+C:\Users\th072\AppData\Roaming\npm\codex.cmd app-server
+```
+
+## Project Layout
+
+```text
+bin/
+cloudflare/
+relay/
+src/
+wrangler.toml
+package.json
+```
+
+## Attribution
+
+This repository preserves the upstream Remodex package structure and credits the original package author in `package.json`. This fork adds a Windows launcher compatibility patch and accompanying documentation.

package/bin/phodex.js

@@ -1,8 +1,8 @@
-#!/usr/bin/env node
-// FILE: phodex.js
-// Purpose: Backward-compatible wrapper that forwards legacy `phodex up` usage to `remodex up`.
-// Layer: CLI binary
-// Exports: none
-// Depends on: ./remodex
-
-require("./remodex");
+#!/usr/bin/env node
+// FILE: phodex.js
+// Purpose: Backward-compatible wrapper that forwards legacy `phodex up` usage to `remodex up`.
+// Layer: CLI binary
+// Exports: none
+// Depends on: ./remodex
+
+require("./remodex");

package/bin/remodex-relay.js

@@ -1,19 +1,19 @@
 #!/usr/bin/env node
-// FILE: remodex-relay.js
-// Purpose: CLI entrypoint for running the bundled Remodex relay locally.
-// Layer: CLI binary
-// Exports: none
-// Depends on: ../src/relay-server
-
-const { startRelayServer } = require("../src/relay-server");
-
-if (process.argv.includes("--help") || process.argv.includes("-h")) {
-  console.log("Usage: remodex-relay [port]");
-  console.log("Environment: REMODEX_RELAY_HOST, REMODEX_RELAY_PORT");
-  process.exit(0);
-}
-
-const portArg = process.argv[2];
-startRelayServer({
-  port: portArg ? Number.parseInt(portArg, 10) : undefined,
-});
+// FILE: remodex-relay.js
+// Purpose: CLI entrypoint for running the bundled Remodex relay locally.
+// Layer: CLI binary
+// Exports: none
+// Depends on: ../src/relay-server
+
+const { startRelayServer } = require("../src/relay-server");
+
+if (process.argv.includes("--help") || process.argv.includes("-h")) {
+  console.log("Usage: remodex-relay [port]");
+  console.log("Environment: REMODEX_RELAY_HOST, REMODEX_RELAY_PORT");
+  process.exit(0);
+}
+
+const portArg = process.argv[2];
+startRelayServer({
+  port: portArg ? Number.parseInt(portArg, 10) : undefined,
+});

package/bin/remodex.js

@@ -1,44 +1,44 @@
 #!/usr/bin/env node
-// FILE: remodex.js
-// Purpose: CLI surface for starting the local Remodex bridge, reopening the latest active thread, and tailing its rollout file.
-// Layer: CLI binary
-// Exports: none
-// Depends on: ../src
-
-const { startBridge, openLastActiveThread, watchThreadRollout } = require("../src");
-
-const command = process.argv[2] || "up";
-
-if (command === "up") {
-  startBridge();
-  return;
-}
-
-if (command === "resume") {
-  try {
-    const state = openLastActiveThread();
-    console.log(
-      `[remodex] Opened last active thread: ${state.threadId} (${state.source || "unknown"})`
-    );
-  } catch (error) {
-    console.error(`[remodex] ${(error && error.message) || "Failed to reopen the last thread."}`);
-    process.exit(1);
-  }
-  return;
-}
-
-if (command === "watch") {
-  try {
-    watchThreadRollout(process.argv[3] || "");
-  } catch (error) {
-    console.error(`[remodex] ${(error && error.message) || "Failed to watch the thread rollout."}`);
-    process.exit(1);
-  }
-  return;
-}
-
-if (command !== "up") {
-  console.error(`Unknown command: ${command}`);
-  console.error("Usage: remodex up | remodex resume | remodex watch [threadId]");
-  process.exit(1);
-}
+// FILE: remodex.js
+// Purpose: CLI surface for starting the local Remodex bridge, reopening the latest active thread, and tailing its rollout file.
+// Layer: CLI binary
+// Exports: none
+// Depends on: ../src
+
+const { startBridge, openLastActiveThread, watchThreadRollout } = require("../src");
+
+const command = process.argv[2] || "up";
+
+if (command === "up") {
+  startBridge();
+  return;
+}
+
+if (command === "resume") {
+  try {
+    const state = openLastActiveThread();
+    console.log(
+      `[remodex] Opened last active thread: ${state.threadId} (${state.source || "unknown"})`
+    );
+  } catch (error) {
+    console.error(`[remodex] ${(error && error.message) || "Failed to reopen the last thread."}`);
+    process.exit(1);
+  }
+  return;
+}
+
+if (command === "watch") {
+  try {
+    watchThreadRollout(process.argv[3] || "");
+  } catch (error) {
+    console.error(`[remodex] ${(error && error.message) || "Failed to watch the thread rollout."}`);
+    process.exit(1);
+  }
+  return;
+}
+
+if (command !== "up") {
+  console.error(`Unknown command: ${command}`);
+  console.error("Usage: remodex up | remodex resume | remodex watch [threadId]");
+  process.exit(1);
+}

package/cloudflare/README.md

@@ -1,16 +1,16 @@
-# Cloudflare Relay
-
-This folder contains a Cloudflare Workers + Durable Objects implementation of the Remodex relay protocol.
-
-It preserves the same session path and headers used by the bridge and iPhone client:
-
-- path: `/relay/{sessionId}`
-- required header: `x-role: mac` or `x-role: iphone`
-- close code `4000`: invalid session or role
-- close code `4001`: previous Mac connection replaced
-- close code `4002`: session unavailable / Mac disconnected
-- close code `4003`: previous iPhone connection replaced
-
-The public health endpoint is:
-
-- `GET /health`
+# Cloudflare Relay
+
+This folder contains a Cloudflare Workers + Durable Objects implementation of the Remodex relay protocol.
+
+It preserves the same session path and headers used by the bridge and iPhone client:
+
+- path: `/relay/{sessionId}`
+- required header: `x-role: mac` or `x-role: iphone`
+- close code `4000`: invalid session or role
+- close code `4001`: previous Mac connection replaced
+- close code `4002`: session unavailable / Mac disconnected
+- close code `4003`: previous iPhone connection replaced
+
+The public health endpoint is:
+
+- `GET /health`

package/cloudflare/worker.mjs

@@ -1,6 +1,5 @@
 import { DurableObject } from "cloudflare:workers";
 
-const MAX_HISTORY = 500;
 const CLOSE_CODE_INVALID_SESSION = 4000;
 const CLOSE_CODE_MAC_REPLACED = 4001;
 const CLOSE_CODE_SESSION_UNAVAILABLE = 4002;
@@ -36,19 +35,15 @@ export class SessionRelay extends DurableObject {
     this.env = env;
     this.mac = null;
     this.clients = new Set();
-    this.history = [];
-
-    this.ctx.blockConcurrencyWhile(async () => {
-      this.history = (await this.ctx.storage.get("history")) || [];
-      for (const socket of this.ctx.getWebSockets()) {
-        const metadata = socket.deserializeAttachment() || {};
-        if (metadata.role === "mac") {
-          this.mac = socket;
-        } else if (metadata.role === "iphone") {
-          this.clients.add(socket);
-        }
+
+    for (const socket of this.ctx.getWebSockets()) {
+      const metadata = socket.deserializeAttachment() || {};
+      if (metadata.role === "mac") {
+        this.mac = socket;
+      } else if (metadata.role === "iphone") {
+        this.clients.add(socket);
       }
-    });
+    }
   }
 
   async fetch(request) {
@@ -104,10 +99,6 @@ export class SessionRelay extends DurableObject {
       }
       this.clients = new Set([serverSocket]);
       console.log(`[relay] iPhone connected -> session ${sessionId} (1 client)`);
-
-      for (const msg of this.history) {
-        this.safeSend(serverSocket, msg);
-      }
     }
 
     return new Response(null, { status: 101, webSocket: clientSocket });
@@ -118,12 +109,6 @@ export class SessionRelay extends DurableObject {
     const text = normalizeMessage(message);
 
     if (metadata.role === "mac") {
-      this.history.push(text);
-      if (this.history.length > MAX_HISTORY) {
-        this.history.shift();
-      }
-      await this.ctx.storage.put("history", this.history);
-
       for (const client of this.clients) {
         this.safeSend(client, text);
       }
@@ -158,11 +143,6 @@ export class SessionRelay extends DurableObject {
     } else if (metadata.role === "iphone") {
       this.clients.delete(socket);
     }
-
-    if (!this.isSocketOpen(this.mac) && this.clients.size === 0) {
-      this.history = [];
-      await this.ctx.storage.deleteAll();
-    }
   }
 
   isSocketOpen(socket) {