robot-resources 1.13.0 → 1.14.0

package/lib/install-node-shim.js

@@ -2,54 +2,31 @@ import { writeShellLine, hasShellLine } from './shell-config.js';
 import { readConfig } from './config.mjs';
 import { detectNodeAgent } from './detect.js';
 import { installRouterFiles } from './install-router-files.js';
+import { writePersistedNodeOptions } from './windows-env.js';
 
 const PLATFORM_URL = process.env.RR_PLATFORM_URL || 'https://api.robotresources.ai';
 
 /**
- * Install the Node shim into the user's shell config. Phase 3 entry for
- * the non-OC Node path.
+ * Install the Node shim into the user's shell config (POSIX) or user
+ * environment registry (Windows).
  *
- * Steps:
- *   1. Copy the bundled `@robot-resources/router` files to a stable absolute
- *      path under `~/.robot-resources/router/` (mirrors the OC plugin pattern
- *      at `~/.openclaw/extensions/`). Phase 8 fix: previously NODE_OPTIONS
- *      used the bare module name `@robot-resources/router/auto` which only
- *      resolved when the user was cd'd inside a project that had the
- *      package in its node_modules. From any other cwd, EVERY `node`
- *      command crashed with "Cannot find module".
- *   2. Append the marker block to detected rc files (zsh / bash / fish)
- *      with the ABSOLUTE PATH to the copied auto.cjs.
- *   3. Emit `node_shim_installed` telemetry.
+ * POSIX path (Phases 3 + 8):
+ *   1. Copy `@robot-resources/router` to ~/.robot-resources/router/
+ *      (absolute path; survives cwd changes + npm/npx cache cleanup).
+ *   2. Append marker block with `--require <abs path>` to detected rc files
+ *      (zsh / bash / fish).
  *
- * The user has to start a new shell (or `source` the file) for the
- * NODE_OPTIONS to take effect — we tell them this in the wizard's
- * post-install message.
+ * Windows path (Phase 9):
+ *   1. Same router-files copy — `homedir()` is platform-aware.
+ *   2. `setx NODE_OPTIONS "..."` writes to HKCU\\Environment so every new
+ *      cmd / PowerShell / Win+R-launched Node process inherits it.
  *
- * Windows: shell-config.writeShellLine returns no rc files on Windows
- * (we only support POSIX in P3). The wizard prints manual instructions
- * for Windows users in `non-oc-wizard.js`.
+ * Both paths emit `node_shim_installed` telemetry. The user has to open a
+ * new terminal for the change to take effect.
  *
  * Returns a UI-friendly result the wizard can format and print.
  */
 export async function installNodeShim({ cwd = process.cwd(), dryRun = false } = {}) {
-  if (process.platform === 'win32') {
-    await emit({
-      shell: 'unsupported',
-      shell_config_path: null,
-      sdks_detected: detectSdks(cwd),
-      dry_run: dryRun,
-      reason: 'windows_not_supported_yet',
-    });
-    return {
-      ok: false,
-      reason: 'windows_not_supported_yet',
-      message:
-        'Windows shell-config writing is not yet supported. Set ' +
-        'NODE_OPTIONS to point at ~/.robot-resources/router/auto.cjs manually ' +
-        'in your system environment variables, or wait for Phase 6.',
-    };
-  }
-
   const sdks = detectSdks(cwd);
 
   if (dryRun) {
@@ -60,18 +37,18 @@ export async function installNodeShim({ cwd = process.cwd(), dryRun = false } =
       dry_run: true,
       reason: null,
     });
-    return { ok: true, message: 'Dry-run: would have written NODE_OPTIONS to shell rc.' };
+    return { ok: true, message: 'Dry-run: would have written NODE_OPTIONS.' };
   }
 
   // Phase 8: copy router to an absolute path under ~/.robot-resources/router/
-  // before we wire the shell config. If the copy fails, we don't write a
-  // broken NODE_OPTIONS line.
+  // before we wire the env config. If the copy fails, we don't write a
+  // broken NODE_OPTIONS line on either platform.
   let autoPath;
   try {
     autoPath = installRouterFiles();
   } catch (err) {
     await emit({
-      shell: 'unknown',
+      shell: process.platform === 'win32' ? 'win32' : 'unknown',
       shell_config_path: null,
       sdks_detected: sdks,
       dry_run: false,
@@ -84,6 +61,48 @@ export async function installNodeShim({ cwd = process.cwd(), dryRun = false } =
     };
   }
 
+  // Windows branch — Phase 9. Mirrors the POSIX flow in shape: detect
+  // already-installed via the persisted registry value, write via setx,
+  // emit equivalent telemetry.
+  if (process.platform === 'win32') {
+    const winResult = writePersistedNodeOptions({ autoPath });
+    await emit({
+      shell: 'win32',
+      shell_config_path: 'HKCU\\Environment\\NODE_OPTIONS',
+      sdks_detected: sdks,
+      dry_run: false,
+      already_installed: !!winResult.already,
+      files_written: winResult.ok && !winResult.already ? 1 : 0,
+      files_with_errors: winResult.ok ? 0 : 1,
+      error_messages: winResult.ok ? [] : [winResult.error_message || winResult.reason || 'unknown'],
+      auto_path: autoPath,
+      win_node_options_length: winResult.length,
+      reason: winResult.ok ? null : winResult.reason,
+    });
+    if (winResult.ok && winResult.already) {
+      return {
+        ok: true,
+        already: true,
+        message: 'NODE_OPTIONS already includes the auto-attach line. No changes made.',
+      };
+    }
+    if (!winResult.ok) {
+      return {
+        ok: false,
+        reason: winResult.reason,
+        message: `Could not set NODE_OPTIONS via setx (${winResult.reason}): ${winResult.error_message || ''}`,
+      };
+    }
+    return {
+      ok: true,
+      written: ['HKCU\\Environment\\NODE_OPTIONS'],
+      errors: [],
+      message:
+        'Set NODE_OPTIONS in your user environment (HKCU\\Environment). ' +
+        'Open a new terminal for it to take effect. Existing terminals will not see the change.',
+    };
+  }
+
   const alreadyInstalled = hasShellLine();
   const result = writeShellLine({ autoPath });
 

package/lib/non-oc-wizard.js

@@ -107,12 +107,21 @@ async function showJsPath() {
     blank();
     info('Once your shell picks up the new NODE_OPTIONS, every Node agent on');
     info('this machine routes Anthropic SDK calls through Robot Resources.');
-    info('Open a new terminal — or run:  source ~/.zshrc   (or your shell rc)');
+    if (process.platform === 'win32') {
+      info('Open a new cmd / PowerShell window — current terminals will not see the change.');
+    } else {
+      info('Open a new terminal — or run:  source ~/.zshrc   (or your shell rc)');
+    }
   } else {
     warn(result.message);
     blank();
-    info('Manual install (paste into ~/.zshrc or ~/.bashrc):');
-    info('  export NODE_OPTIONS="${NODE_OPTIONS:-} --require @robot-resources/router/auto"');
+    if (process.platform === 'win32') {
+      info('Manual install: in a new cmd, run:');
+      info('  setx NODE_OPTIONS "--require %USERPROFILE%\\.robot-resources\\router\\auto.cjs"');
+    } else {
+      info('Manual install (paste into ~/.zshrc or ~/.bashrc):');
+      info('  export NODE_OPTIONS="${NODE_OPTIONS:-} --require ~/.robot-resources/router/auto.cjs"');
+    }
   }
   blank();
   info('Docs: https://robotresources.ai/docs/langchain');

package/lib/uninstall.js

@@ -5,6 +5,7 @@ import { stripJson5 } from './json5.js';
 import { removeShellLine } from './shell-config.js';
 import { detectVenv } from './venv-detect.js';
 import { spawnSync } from 'node:child_process';
+import { removePersistedNodeOptions } from './windows-env.js';
 
 /**
  * Single source of truth for `npx robot-resources --uninstall`.
@@ -86,18 +87,32 @@ export function runUninstall({ purge = false } = {}) {
     }
   }
 
-  // 3. Shell config — remove the NODE_OPTIONS marker block from any rc
-  //    files Phase 3's wizard wrote to. Idempotent: no-op if not present.
-  try {
-    const result = removeShellLine();
-    if (result.removed.length > 0) {
-      components_removed.push('shell_config_node_options');
+  // 3. Shell config / Windows registry — remove the NODE_OPTIONS line
+  //    Phase 3's wizard wrote to. Idempotent: no-op if not present.
+  if (process.platform === 'win32') {
+    // Phase 9: restore from backup or clear HKCU\Environment\NODE_OPTIONS.
+    try {
+      const result = removePersistedNodeOptions();
+      if (result.ok && result.action !== 'noop') {
+        components_removed.push(`win_node_options_${result.action}`);
+      } else if (!result.ok) {
+        errors.push({ component: 'win_node_options', message: 'reg_restore_failed' });
+      }
+    } catch (err) {
+      errors.push({ component: 'win_node_options', message: err.message });
     }
-    for (const e of result.errors) {
-      errors.push({ component: 'shell_config_node_options', message: `${e.path}: ${e.message}` });
+  } else {
+    try {
+      const result = removeShellLine();
+      if (result.removed.length > 0) {
+        components_removed.push('shell_config_node_options');
+      }
+      for (const e of result.errors) {
+        errors.push({ component: 'shell_config_node_options', message: `${e.path}: ${e.message}` });
+      }
+    } catch (err) {
+      errors.push({ component: 'shell_config_node_options', message: err.message });
     }
-  } catch (err) {
-    errors.push({ component: 'shell_config_node_options', message: err.message });
   }
 
   // 3b. Copied router dir at ~/.robot-resources/router/ (Phase 8). The shell

package/lib/windows-env.js

@@ -0,0 +1,202 @@
+import { spawnSync } from 'node:child_process';
+import { existsSync, readFileSync, writeFileSync, mkdirSync, unlinkSync } from 'node:fs';
+import { homedir } from 'node:os';
+import { join } from 'node:path';
+
+/**
+ * Windows NODE_OPTIONS persistence (Phase 9).
+ *
+ * The POSIX path uses a marker block in `~/.bashrc` / `~/.zshrc`. Windows
+ * has no equivalent shell rc that all Node-launched processes read — cmd,
+ * PowerShell, and Win+R-launched .exe all draw env vars from the user
+ * registry under HKCU\Environment.
+ *
+ * `setx NODE_OPTIONS "value"` writes there. New processes pick it up.
+ * The current shell does NOT see the change — user has to open a new
+ * terminal. Same UX caveat as POSIX.
+ *
+ * Why not edit PowerShell `$PROFILE`: ExecutionPolicy on locked-down
+ * corporate fleets often blocks unsigned `.ps1`. cmd-launched Node
+ * processes also miss it. setx is universal across both.
+ *
+ * Idempotency: we read the persisted NODE_OPTIONS via `reg query` (not
+ * `process.env.NODE_OPTIONS`, which is the current-shell value, not the
+ * persistent one). If our `--require <auto>` is already present, no-op.
+ *
+ * Uninstall: backup the user's PRE-modification value to
+ * `~/.robot-resources/windows-prior-node-options.txt`. On `--uninstall`,
+ * restore from backup; if the backup is missing or empty, clear the var.
+ *
+ * Truncation note: `setx` truncates values at 1024 chars. If a user
+ * already has a long NODE_OPTIONS plus other dev tooling, we may be
+ * close to the limit. We surface the merged length in telemetry so we
+ * can spot truncation in Supabase.
+ */
+
+const REG_PATH = 'HKCU\\Environment';
+const VAR_NAME = 'NODE_OPTIONS';
+const SETX_LIMIT = 1024;
+
+function backupFilePath(home = homedir()) {
+  return join(home, '.robot-resources', 'windows-prior-node-options.txt');
+}
+
+/**
+ * Read the persisted NODE_OPTIONS value from HKCU\Environment.
+ * Returns the string (possibly empty) or null if reading failed.
+ */
+export function readPersistedNodeOptions() {
+  const res = spawnSync(
+    'reg.exe',
+    ['query', REG_PATH, '/v', VAR_NAME],
+    { stdio: 'pipe', encoding: 'utf-8' },
+  );
+  if (res.status !== 0) {
+    // `reg query` returns non-zero when the value doesn't exist. That's a
+    // valid state — empty NODE_OPTIONS — not an error.
+    const stderr = (res.stderr || '').toLowerCase();
+    if (stderr.includes('unable to find') || stderr.includes('not exist')) {
+      return '';
+    }
+    return null;
+  }
+  // Output looks like:
+  //   HKEY_CURRENT_USER\Environment
+  //       NODE_OPTIONS    REG_SZ    --require ...
+  // We extract the value after REG_SZ. The value can contain spaces; take
+  // everything after the last "REG_SZ" occurrence on its line.
+  const lines = (res.stdout || '').split(/\r?\n/);
+  for (const line of lines) {
+    const m = line.match(/^\s*NODE_OPTIONS\s+REG_(?:SZ|EXPAND_SZ)\s+(.*)$/);
+    if (m) return m[1].trim();
+  }
+  return '';
+}
+
+/**
+ * Set NODE_OPTIONS in HKCU\Environment to the given value (idempotent
+ * append of `--require <autoPath>` to whatever was there).
+ *
+ * Returns:
+ *   { ok: true,  already: boolean, prior: string, written: string, length: number }
+ *   { ok: false, reason, error_message? }
+ */
+export function writePersistedNodeOptions({ autoPath, home = homedir() }) {
+  if (!autoPath) {
+    return { ok: false, reason: 'missing_auto_path' };
+  }
+
+  const prior = readPersistedNodeOptions();
+  if (prior === null) {
+    return { ok: false, reason: 'reg_query_failed' };
+  }
+
+  // Quote the path so a path with spaces survives Node's --require parser.
+  // Node accepts both quoted and unquoted forms; quoting is safer.
+  const ourArg = `--require "${autoPath}"`;
+
+  if (prior.includes(ourArg) || prior.includes(`--require ${autoPath}`)) {
+    return {
+      ok: true,
+      already: true,
+      prior,
+      written: prior,
+      length: prior.length,
+    };
+  }
+
+  const merged = prior ? `${prior} ${ourArg}` : ourArg;
+
+  if (merged.length > SETX_LIMIT) {
+    // setx truncates silently at 1024 chars. Refuse rather than write a
+    // broken value. User must shorten their existing NODE_OPTIONS first.
+    return {
+      ok: false,
+      reason: 'setx_limit_exceeded',
+      error_message: `merged value is ${merged.length} chars; setx truncates at ${SETX_LIMIT}`,
+      length: merged.length,
+    };
+  }
+
+  // Backup the prior value BEFORE writing, so --uninstall can restore.
+  // Even if backup write fails (disk full, permissions), we still proceed —
+  // the registry-level setx is the source of truth, the backup is just a
+  // convenience for restore.
+  try {
+    const backup = backupFilePath(home);
+    mkdirSync(join(home, '.robot-resources'), { recursive: true });
+    writeFileSync(backup, prior, { encoding: 'utf-8' });
+  } catch {
+    // Best-effort backup; continue.
+  }
+
+  const setxRes = spawnSync(
+    'setx.exe',
+    [VAR_NAME, merged],
+    { stdio: 'pipe', encoding: 'utf-8' },
+  );
+  if (setxRes.status !== 0) {
+    const stderr = (setxRes.stderr || '').toString().trim();
+    return {
+      ok: false,
+      reason: 'setx_failed',
+      error_message: stderr.slice(0, 200) || `exit ${setxRes.status}`,
+      length: merged.length,
+    };
+  }
+
+  return {
+    ok: true,
+    already: false,
+    prior,
+    written: merged,
+    length: merged.length,
+  };
+}
+
+/**
+ * Reverse `writePersistedNodeOptions`. Reads the backup file (if present)
+ * and restores that value via setx. If no backup is present or the backup
+ * is empty, clears the registry value entirely (`reg delete`).
+ *
+ * Returns { ok, restored_to: string, action: 'restored'|'cleared'|'noop' }
+ */
+export function removePersistedNodeOptions({ home = homedir() } = {}) {
+  const current = readPersistedNodeOptions();
+  if (current === null) return { ok: false, action: 'noop' };
+  if (current === '') return { ok: true, restored_to: '', action: 'noop' };
+
+  const backupPath = backupFilePath(home);
+  let priorValue = '';
+  if (existsSync(backupPath)) {
+    try { priorValue = readFileSync(backupPath, 'utf-8'); } catch { /* */ }
+  }
+
+  if (priorValue === '') {
+    // No prior value to restore — clear the var entirely.
+    const res = spawnSync(
+      'reg.exe',
+      ['delete', REG_PATH, '/v', VAR_NAME, '/f'],
+      { stdio: 'pipe', encoding: 'utf-8' },
+    );
+    if (existsSync(backupPath)) try { unlinkSync(backupPath); } catch { /* */ }
+    return {
+      ok: res.status === 0,
+      restored_to: '',
+      action: 'cleared',
+    };
+  }
+
+  // Restore the prior value.
+  const res = spawnSync(
+    'setx.exe',
+    [VAR_NAME, priorValue],
+    { stdio: 'pipe', encoding: 'utf-8' },
+  );
+  if (existsSync(backupPath)) try { unlinkSync(backupPath); } catch { /* */ }
+  return {
+    ok: res.status === 0,
+    restored_to: priorValue,
+    action: 'restored',
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "robot-resources",
-  "version": "1.13.0",
+  "version": "1.14.0",
   "description": "Robot Resources — AI agent tools. One command to install everything.",
   "type": "module",
   "bin": {