feat(dev): auto-switch ports on dev:all when defaults are busy

Adds a small launcher (scripts/dev-all.mjs) that probes free ports
for the daemon (OD_PORT, default 7456) and Vite (VITE_PORT, default
5173) before invoking concurrently, so a stray process holding
either port no longer breaks the boot. The resolved ports are
exported into the child env; vite.config.ts now reads VITE_PORT to
keep its dev server and /api proxy aligned with the daemon's actual
port.

Made-with: Cursor

daemon/agents.js

@@ -7,12 +7,7 @@ import path from 'node:path';
 const execFileP = promisify(execFile);
 
 // Each entry defines how to invoke the agent in non-interactive "one-shot" mode.
-// `buildArgs(prompt, imagePaths, extraAllowedDirs)` returns argv for the child
-// process. `extraAllowedDirs` is a list of absolute directories the agent must
-// be permitted to read files from (skill seeds, design-system specs) that live
-// outside the project cwd. Currently only Claude Code wires this through
-// (`--add-dir`); other agents either inherit broader access or run with cwd
-// boundaries we can't widen via flags.
+// `buildArgs(prompt, imagePaths)` returns argv for the child process.
 // `streamFormat` hints to the daemon how to interpret stdout:
 //   - 'claude-stream-json' : line-delimited JSON emitted by Claude Code's
 //     `--output-format stream-json`. Daemon parses it into typed events
@@ -24,23 +19,14 @@ export const AGENT_DEFS = [
     name: 'Claude Code',
     bin: 'claude',
     versionArgs: ['--version'],
-    buildArgs: (prompt, _imagePaths, extraAllowedDirs = []) => {
-      const args = [
-        '-p',
-        prompt,
-        '--output-format',
-        'stream-json',
-        '--verbose',
-        '--include-partial-messages',
-      ];
-      const dirs = (extraAllowedDirs || []).filter(
-        (d) => typeof d === 'string' && d.length > 0,
-      );
-      if (dirs.length > 0) {
-        args.push('--add-dir', ...dirs);
-      }
-      return args;
-    },
+    buildArgs: (prompt) => [
+      '-p',
+      prompt,
+      '--output-format',
+      'stream-json',
+      '--verbose',
+      '--include-partial-messages',
+    ],
     streamFormat: 'claude-stream-json',
   },
   {
@@ -125,13 +111,3 @@ export async function detectAgents() {
 export function getAgentDef(id) {
   return AGENT_DEFS.find((a) => a.id === id) || null;
 }
-
-// Resolve the absolute path of an agent's binary on the current PATH.
-// Used by the chat handler so spawn() gets the same executable that
-// detection reported as available — fixes Windows ENOENT when the bare
-// bin name isn't on the child process's PATH (issue #10).
-export function resolveAgentBin(id) {
-  const def = getAgentDef(id);
-  if (!def?.bin) return null;
-  return resolveOnPath(def.bin);
-}

daemon/server.js

@@ -6,7 +6,7 @@ import { fileURLToPath } from 'node:url';
 import path from 'node:path';
 import fs from 'node:fs';
 import os from 'node:os';
-import { detectAgents, getAgentDef, resolveAgentBin } from './agents.js';
+import { detectAgents, getAgentDef } from './agents.js';
 import { listSkills } from './skills.js';
 import { listDesignSystems, readDesignSystem } from './design-systems.js';
 import { createClaudeStreamHandler } from './claude-stream.js';
@@ -769,17 +769,7 @@ export async function startServer({ port = 7456 } = {}) {
       safeImages.length ? `\n\n${safeImages.map((p) => `@${p}`).join(' ')}` : '',
     ].join('');
 
-    // Skill seeds (`skills/<id>/assets/template.html`) and design-system
-    // specs (`design-systems/<id>/DESIGN.md`) live outside the project cwd.
-    // The composed system prompt asks the agent to Read them via absolute
-    // paths in the skill-root preamble — without an explicit allowlist,
-    // Claude Code blocks those reads (issue #6: "no permission to read
-    // skills template"). We surface both roots so any agent that honours
-    // `--add-dir` can resolve those side files.
-    const extraAllowedDirs = [SKILLS_DIR, DESIGN_SYSTEMS_DIR].filter(
-      (d) => fs.existsSync(d),
-    );
-    const args = def.buildArgs(composed, safeImages, extraAllowedDirs);
+    const args = def.buildArgs(composed, safeImages);
 
     res.setHeader('Content-Type', 'text/event-stream');
     res.setHeader('Cache-Control', 'no-cache, no-transform');
@@ -792,20 +782,9 @@ export async function startServer({ port = 7456 } = {}) {
       res.write(`data: ${JSON.stringify(data)}\n\n`);
     };
 
-    // Resolve the agent's bin to its absolute path. Detection (`/api/agents`)
-    // already locates the executable via PATH, but spawning the bare name here
-    // fails on Windows (ENOENT) when the child process's PATH doesn't contain
-    // the user's npm-global / shim directory — see issue #10.
-    const resolvedBin = resolveAgentBin(agentId) || def.bin;
-    // npm shims on Windows are .cmd/.bat files; Node ≥21 refuses to spawn
-    // those without `shell: true` (CVE-2024-27980). When `shell: true` is set
-    // on Windows, Node escapes args automatically for the cmd.exe shell.
-    const useShell =
-      process.platform === 'win32' && /\.(cmd|bat)$/i.test(resolvedBin);
-
     send('start', {
       agentId,
-      bin: resolvedBin,
+      bin: def.bin,
       streamFormat: def.streamFormat ?? 'plain',
       projectId: typeof projectId === 'string' ? projectId : null,
       cwd,
@@ -813,11 +792,10 @@ export async function startServer({ port = 7456 } = {}) {
 
     let child;
     try {
-      child = spawn(resolvedBin, args, {
+      child = spawn(def.bin, args, {
         env: { ...process.env },
         stdio: ['ignore', 'pipe', 'pipe'],
         cwd: cwd || undefined,
-        shell: useShell,
       });
     } catch (err) {
       send('error', { message: `spawn failed: ${err.message}` });

src/App.tsx

@@ -116,7 +116,6 @@ export function App() {
     const withOnboarding: AppConfig = { ...next, onboardingCompleted: true };
     saveConfig(withOnboarding);
     setConfig(withOnboarding);
-    setSettingsOpen(false);
   }, []);
 
   const handleModeChange = useCallback(

src/components/SettingsDialog.tsx

@@ -276,7 +276,10 @@ export function SettingsDialog({
             type="button"
             className="primary"
             disabled={!canSave}
-            onClick={() => onSave(cfg)}
+            onClick={() => {
+              onSave(cfg);
+              onClose();
+            }}
           >
             {welcome ? t('settings.getStarted') : t('common.save')}
           </button>