fix: use file URL for dynamic imports to support Windows (#156)

* fix: use file URL for dynamic imports to support Windows

Node's ESM dynamic `import()` rejects bare absolute paths on Windows
because the drive letter (e.g. `C:`) is interpreted as an unsupported
URL scheme. Wrap the path with `pathToFileURL()` so it works
cross-platform.

Fixes #155

* fix: comprehensive Windows compatibility fixes

- Add cross-platform spawn helper (shell: true on Windows for .cmd shims)
- Fix URL.pathname in bin.ts (use fileURLToPath for Windows drive letters)
- Fix node_modules/.bin resolution to prefer .cmd on Windows
- Fix all env file parsers to handle CRLF line endings
- Replace opn with open for better Windows URL handling
- Add SIGBREAK handler for Windows signal handling
- Fix Prettier step to use spawn with array args (safe path handling)
- Fix package install to use spawn with array args (safe special chars)
- Fix Gradle wrapper resolution for Windows (gradlew.bat)
- Fix Goose agent config dir to use %APPDATA% on Windows
- Fix vercel spawn calls with shell option

Ref #155

* chore: formatting

Author: nicknisi

package.json

@@ -52,18 +52,18 @@
     "@hono/node-server": "^1",
     "@napi-rs/keyring": "^1.2.0",
     "@workos-inc/node": "^8.7.0",
+    "@workos/migrations": "^2.0.0",
     "@workos/openapi-spec": "^0.1.0",
     "@workos/skills": "0.6.0",
     "chalk": "^5.6.2",
     "diff": "^8.0.3",
     "fast-glob": "^3.3.3",
     "hono": "^4",
     "ink": "^6.8.0",
-    "opn": "^5.4.0",
+    "open": "^11.0.0",
     "react": "^19.2.4",
     "semver": "^7.7.4",
     "uuid": "^13.0.0",
-    "@workos/migrations": "^2.0.0",
     "xstate": "^5.28.0",
     "yaml": "^2.8.2",
     "yargs": "^18.0.0",

pnpm-lock.yaml

@@ -4,7 +4,8 @@
 if (process.argv.includes('--local') || process.env.INSTALLER_DEV) {
   const { config } = await import('dotenv');
   // bin.ts compiles to dist/bin.js, so go up one level to find .env.local
-  config({ path: new URL('../.env.local', import.meta.url).pathname });
+  const { fileURLToPath } = await import('node:url');
+  config({ path: fileURLToPath(new URL('../.env.local', import.meta.url)) });
 }
 
 import { satisfies } from 'semver';

src/bin.ts

@@ -6,9 +6,8 @@ vi.mock('../utils/debug.js', () => ({
   logError: vi.fn(),
 }));
 
-// Mock opn (browser open)
 const mockOpen = vi.fn().mockResolvedValue(undefined);
-vi.mock('opn', () => ({ default: mockOpen }));
+vi.mock('open', () => ({ default: mockOpen }));
 
 // Mock clack
 const mockSpinner = {

src/commands/claim.spec.ts

@@ -6,7 +6,7 @@
  * until the environment is claimed.
  */
 
-import open from 'opn';
+import open from 'open';
 import clack from '../utils/clack.js';
 import { getActiveEnvironment, isUnclaimedEnvironment, markEnvironmentClaimed } from '../lib/config-store.js';
 import { createClaimNonce, UnclaimedEnvApiError } from '../lib/unclaimed-env-api.js';

src/commands/claim.ts

@@ -5,6 +5,7 @@ import { readFileSync, existsSync } from 'node:fs';
 import { resolve } from 'node:path';
 import { parse as parseYaml } from 'yaml';
 import chalk from 'chalk';
+import { IS_WINDOWS, SPAWN_OPTS } from '../utils/platform.js';
 
 export interface DevArgs {
   port: number;
@@ -122,6 +123,7 @@ export async function runDev(argv: DevArgs): Promise<void> {
         ...process.env,
         ...buildDevEnv(emulator.url, emulator.apiKey),
       },
+      ...SPAWN_OPTS,
     });
   } catch {
     console.error(chalk.red(`Failed to start: ${devCmd.command} ${devCmd.args.join(' ')}`));
@@ -149,6 +151,9 @@ export async function runDev(argv: DevArgs): Promise<void> {
   };
   process.on('SIGINT', () => shutdown('SIGINT'));
   process.on('SIGTERM', () => shutdown('SIGTERM'));
+  if (IS_WINDOWS) {
+    process.on('SIGBREAK', () => shutdown('SIGINT'));
+  }
 
   // 6. If child exits, close emulator and exit with same code
   child.on('exit', (code) => {

src/commands/dev.ts

@@ -3,6 +3,7 @@ import { readFileSync, existsSync } from 'node:fs';
 import { resolve } from 'node:path';
 import { parse as parseYaml } from 'yaml';
 import chalk from 'chalk';
+import { IS_WINDOWS } from '../utils/platform.js';
 
 export interface EmulateArgs {
   port: number;
@@ -75,4 +76,7 @@ export async function runEmulate(argv: EmulateArgs): Promise<void> {
   };
   process.once('SIGINT', shutdown);
   process.once('SIGTERM', shutdown);
+  if (IS_WINDOWS) {
+    process.once('SIGBREAK', shutdown);
+  }
 }

src/commands/emulate.ts

@@ -4,6 +4,7 @@ import { existsSync } from 'fs';
 import { mkdir, mkdtemp, cp, rename, rm, readdir, readFile, stat, access, writeFile } from 'fs/promises';
 import chalk from 'chalk';
 import { getSkillsDir as getSkillsPackageDir } from '@workos/skills';
+import { IS_WINDOWS } from '../utils/platform.js';
 
 export const SKILL_VERSION_MARKER_FILENAME = '.workos-skill-version';
 
@@ -48,30 +49,31 @@ export interface AgentConfig {
 }
 
 export function createAgents(home: string): Record<string, AgentConfig> {
+  const appData = process.env.APPDATA ?? join(home, 'AppData', 'Roaming');
   return {
     'claude-code': {
       name: 'claude-code',
       displayName: 'Claude Code',
-      globalSkillsDir: join(home, '.claude/skills'),
+      globalSkillsDir: join(home, '.claude', 'skills'),
       detect: () => existsSync(join(home, '.claude')),
     },
     codex: {
       name: 'codex',
       displayName: 'Codex',
-      globalSkillsDir: join(home, '.codex/skills'),
+      globalSkillsDir: join(home, '.codex', 'skills'),
       detect: () => existsSync(join(home, '.codex')),
     },
     cursor: {
       name: 'cursor',
       displayName: 'Cursor',
-      globalSkillsDir: join(home, '.cursor/skills'),
+      globalSkillsDir: join(home, '.cursor', 'skills'),
       detect: () => existsSync(join(home, '.cursor')),
     },
     goose: {
       name: 'goose',
       displayName: 'Goose',
-      globalSkillsDir: join(home, '.config/goose/skills'),
-      detect: () => existsSync(join(home, '.config/goose')),
+      globalSkillsDir: IS_WINDOWS ? join(appData, 'goose', 'skills') : join(home, '.config', 'goose', 'skills'),
+      detect: () => (IS_WINDOWS ? existsSync(join(appData, 'goose')) : existsSync(join(home, '.config', 'goose'))),
     },
   };
 }

src/commands/install-skill.ts

@@ -4,7 +4,7 @@ import { join } from 'node:path';
 import { tmpdir } from 'node:os';
 
 const mockOpen = vi.fn();
-vi.mock('opn', () => ({ default: (...args: unknown[]) => mockOpen(...args) }));
+vi.mock('open', () => ({ default: (...args: unknown[]) => mockOpen(...args) }));
 
 class MockDeviceAuthTimeoutError extends Error {}
 const mockRequestDeviceCode = vi.fn();

src/commands/login.spec.ts

@@ -1,4 +1,4 @@
-import open from 'opn';
+import open from 'open';
 import chalk from 'chalk';
 import clack from '../utils/clack.js';
 import { saveCredentials, getCredentials, getAccessToken, isTokenExpired, updateTokens } from '../lib/credentials.js';