@castlekit/castle 0.4.0 → 0.4.2

package/src/cli/onboarding.ts

@@ -329,6 +329,7 @@ export async function runOnboarding(): Promise<void> {
   }
 
   // Step 1: Check for OpenClaw
+  let openclawSkipped = false;
   const openclawSpinner = p.spinner();
   openclawSpinner.start("Checking for OpenClaw...");
 
@@ -355,32 +356,30 @@ export async function runOnboarding(): Promise<void> {
       const installSpinner = p.spinner();
       installSpinner.start("Installing OpenClaw...");
 
+      const installCmd = process.platform === "win32"
+        ? 'powershell -NoProfile -ExecutionPolicy Bypass -Command "iwr -useb https://openclaw.ai/install.ps1 -OutFile $env:TEMP\\openclaw-install.ps1; & $env:TEMP\\openclaw-install.ps1 -NoOnboard"'
+        : 'curl -fsSL --proto "=https" --tlsv1.2 https://openclaw.ai/install.sh | bash -s -- --no-onboard --no-prompt';
+
       try {
-        execSync(
-          'curl -fsSL --proto "=https" --tlsv1.2 https://openclaw.ai/install.sh | bash -s -- --no-onboard --no-prompt',
-          { stdio: "pipe", timeout: 120000 }
-        );
+        execSync(installCmd, { stdio: "inherit", timeout: 300000 });
         installSpinner.stop(BLUE("✔ OpenClaw installed"));
       } catch (error) {
+        openclawSkipped = true;
         installSpinner.stop(pc.red("OpenClaw installation failed"));
-        p.note(
-          `Install OpenClaw manually:\n${BLUE_LIGHT(
-            "curl -fsSL https://openclaw.ai/install.sh | bash"
-          )}\n\nThen run: ${BLUE_LIGHT("castle setup")}`,
-          BLUE_BOLD("Manual Install")
+        const manualCmd = process.platform === "win32"
+          ? "iwr -useb https://openclaw.ai/install.ps1 | iex"
+          : "curl -fsSL https://openclaw.ai/install.sh | bash";
+        p.log.warn(
+          `You can install OpenClaw later:\n${BLUE_LIGHT(manualCmd)}`
         );
-        p.outro("Come back when OpenClaw is installed!");
-        process.exit(1);
       }
     } else {
-      p.note(
-        `Install OpenClaw:\n${BLUE_LIGHT(
-          "curl -fsSL https://openclaw.ai/install.sh | bash"
-        )}\n\nThen come back and run:\n${BLUE_LIGHT("castle setup")}`,
-        BLUE_BOLD("Install OpenClaw First")
-      );
-      p.outro("See you soon!");
-      process.exit(0);
+      openclawSkipped = true;
+    }
+
+    if (openclawSkipped && !isOpenClawInstalled()) {
+      // Skip Gateway config — use defaults and continue to build/start Castle
+      p.log.message(pc.dim("Using default settings — you can reconfigure later with castle setup"));
     }
   } else {
     // Auto-detect token and agents in one go
@@ -413,34 +412,49 @@ export async function runOnboarding(): Promise<void> {
   let token = readOpenClawToken();
   let gatewayUrl: string | undefined;
   let isRemote = false;
+  let primaryAgent = "assistant";
+  let agents: DiscoveredAgent[] = [];
+
+  if (!openclawSkipped) {
+    // If we have auto-detected config, offer a choice
+    const hasLocalConfig = !!readOpenClawPort() || isOpenClawInstalled();
+
+    if (hasLocalConfig && token) {
+      // Both auto-detect and manual are available
+      const connectionMode = await p.select({
+        message: "How would you like to connect?",
+        options: [
+          {
+            value: "auto",
+            label: `Auto-detected local Gateway ${pc.dim(`(port ${port})`)}`,
+            hint: "Recommended for local setups",
+          },
+          {
+            value: "manual",
+            label: "Enter Gateway details manually",
+            hint: "For remote, Tailscale, or custom setups",
+          },
+        ],
+      });
 
-  // If we have auto-detected config, offer a choice
-  const hasLocalConfig = !!readOpenClawPort() || isOpenClawInstalled();
-
-  if (hasLocalConfig && token) {
-    // Both auto-detect and manual are available
-    const connectionMode = await p.select({
-      message: "How would you like to connect?",
-      options: [
-        {
-          value: "auto",
-          label: `Auto-detected local Gateway ${pc.dim(`(port ${port})`)}`,
-          hint: "Recommended for local setups",
-        },
-        {
-          value: "manual",
-          label: "Enter Gateway details manually",
-          hint: "For remote, Tailscale, or custom setups",
-        },
-      ],
-    });
-
-    if (p.isCancel(connectionMode)) {
-      p.cancel("Setup cancelled.");
-      process.exit(0);
-    }
+      if (p.isCancel(connectionMode)) {
+        p.cancel("Setup cancelled.");
+        process.exit(0);
+      }
 
-    if (connectionMode === "manual") {
+      if (connectionMode === "manual") {
+        const manualResult = await promptManualGateway();
+        if (!manualResult) {
+          p.cancel("Setup cancelled.");
+          process.exit(0);
+        }
+        port = manualResult.port;
+        token = manualResult.token;
+        gatewayUrl = manualResult.gatewayUrl;
+        isRemote = manualResult.isRemote;
+      }
+    } else if (!token) {
+      // No auto-detected token — fall through to manual entry
       const manualResult = await promptManualGateway();
       if (!manualResult) {
         p.cancel("Setup cancelled.");
@@ -451,57 +465,44 @@ export async function runOnboarding(): Promise<void> {
       gatewayUrl = manualResult.gatewayUrl;
       isRemote = manualResult.isRemote;
     }
-  } else if (!token) {
-    // No auto-detected token — fall through to manual entry
-    const manualResult = await promptManualGateway();
-    if (!manualResult) {
-      p.cancel("Setup cancelled.");
-      process.exit(0);
-    }
-    port = manualResult.port;
-    token = manualResult.token;
-    gatewayUrl = manualResult.gatewayUrl;
-    isRemote = manualResult.isRemote;
-  }
 
-  // Step 3: Agent Discovery (use URL if remote, port if local)
-  const agentTarget = gatewayUrl || port;
-  const agents = await discoverAgents(agentTarget, token);
+    // Step 3: Agent Discovery (use URL if remote, port if local)
+    const agentTarget = gatewayUrl || port;
+    agents = await discoverAgents(agentTarget, token);
 
-  let primaryAgent: string;
+    if (agents.length > 0) {
 
-  if (agents.length > 0) {
+      const selectedAgent = await p.select({
+        message: "Choose your primary agent",
+        options: agents.map((a) => ({
+          value: a.id,
+          label: `${a.name} ${pc.dim(`<${a.id}>`)}`,
+          hint: a.description || undefined,
+        })),
+      });
 
-    const selectedAgent = await p.select({
-      message: "Choose your primary agent",
-      options: agents.map((a) => ({
-        value: a.id,
-        label: `${a.name} ${pc.dim(`<${a.id}>`)}`,
-        hint: a.description || undefined,
-      })),
-    });
+      if (p.isCancel(selectedAgent)) {
+        p.cancel("Setup cancelled.");
+        process.exit(0);
+      }
 
-    if (p.isCancel(selectedAgent)) {
-      p.cancel("Setup cancelled.");
-      process.exit(0);
-    }
+      primaryAgent = selectedAgent as string;
+    } else {
+      const setPrimary = await p.text({
+        message: "Enter the name of your primary agent",
+        initialValue: "assistant",
+        validate(value: string | undefined) {
+          if (!value?.trim()) return "Agent name is required";
+        },
+      });
 
-    primaryAgent = selectedAgent as string;
-  } else {
-    const setPrimary = await p.text({
-      message: "Enter the name of your primary agent",
-      initialValue: "assistant",
-      validate(value: string | undefined) {
-        if (!value?.trim()) return "Agent name is required";
-      },
-    });
+      if (p.isCancel(setPrimary)) {
+        p.cancel("Setup cancelled.");
+        process.exit(0);
+      }
 
-    if (p.isCancel(setPrimary)) {
-      p.cancel("Setup cancelled.");
-      process.exit(0);
+      primaryAgent = setPrimary as string;
     }
-
-    primaryAgent = setPrimary as string;
   }
 
   // Step 5: Create Castle config
@@ -619,12 +620,37 @@ export async function runOnboarding(): Promise<void> {
 
   // Write PID file helper
   const pidFile = join(castleDir, "server.pid");
+  const isWin = process.platform === "win32";
+
+  // Stop existing service FIRST — otherwise the service manager respawns
+  // the old server immediately after we kill it, stealing the port.
+  if (process.platform === "darwin") {
+    const plistPath = join(home(), "Library", "LaunchAgents", "com.castlekit.castle.plist");
+    try {
+      execSyncChild(`launchctl unload "${plistPath}" 2>/dev/null`, { stdio: "ignore", timeout: 10000 });
+    } catch { /* no existing service */ }
+  } else if (process.platform === "linux") {
+    try {
+      execSyncChild("systemctl --user stop castle.service 2>/dev/null", { stdio: "ignore", timeout: 10000 });
+    } catch { /* no existing service */ }
+  } else if (isWin) {
+    try {
+      execSyncChild('schtasks /End /TN "CastleServer" 2>nul', { stdio: "ignore", timeout: 10000 });
+    } catch { /* no existing task */ }
+    try {
+      execSyncChild('schtasks /Delete /TN "CastleServer" /F 2>nul', { stdio: "ignore", timeout: 10000 });
+    } catch { /* no existing task */ }
+  }
 
   // Kill any existing Castle server (by PID file)
   try {
     const existingPid = parseInt(readF(pidFile, "utf-8").trim(), 10);
     if (Number.isInteger(existingPid) && existingPid > 0) {
-      process.kill(existingPid);
+      if (isWin) {
+        try { execSyncChild(`taskkill /PID ${existingPid} /F 2>nul`, { stdio: "ignore", timeout: 5000 }); } catch { /* ignore */ }
+      } else {
+        process.kill(existingPid);
+      }
       for (let i = 0; i < 30; i++) {
         try {
           process.kill(existingPid, 0);
@@ -639,14 +665,35 @@ export async function runOnboarding(): Promise<void> {
   }
 
   // Kill anything else on the target port
-  try {
-    execSyncChild(`lsof -ti:${castlePort} | xargs kill -9 2>/dev/null`, {
-      stdio: "ignore",
-      timeout: 5000,
-    });
-    await new Promise((r) => setTimeout(r, 500));
-  } catch {
-    // Nothing on port or lsof not available
+  if (isWin) {
+    try {
+      const netstatOut = execSyncChild(
+        `netstat -ano | findstr ":${castlePort} " | findstr "LISTENING"`,
+        { encoding: "utf-8", timeout: 5000 }
+      ).toString();
+      const pids = new Set<string>();
+      for (const line of netstatOut.split("\n")) {
+        const parts = line.trim().split(/\s+/);
+        const pid = parts[parts.length - 1];
+        if (pid && /^\d+$/.test(pid) && pid !== "0") pids.add(pid);
+      }
+      for (const pid of pids) {
+        try { execSyncChild(`taskkill /PID ${pid} /F 2>nul`, { stdio: "ignore", timeout: 5000 }); } catch { /* ignore */ }
+      }
+      if (pids.size > 0) await new Promise((r) => setTimeout(r, 500));
+    } catch {
+      // Nothing on port or netstat not available
+    }
+  } else {
+    try {
+      execSyncChild(`lsof -ti:${castlePort} | xargs kill -9 2>/dev/null`, {
+        stdio: "ignore",
+        timeout: 5000,
+      });
+      await new Promise((r) => setTimeout(r, 500));
+    } catch {
+      // Nothing on port or lsof not available
+    }
   }
 
   // Escape XML special characters for plist values
@@ -693,9 +740,7 @@ ${plistEnvEntries}
     </dict>
 </dict>
 </plist>`;
-    try {
-      execSyncChild(`launchctl unload "${plistPath}" 2>/dev/null`, { stdio: "ignore", timeout: 10000 });
-    } catch { /* ignore */ }
+    // Service was already unloaded above — just write new plist and load
     writeFile(plistPath, plist);
     try {
       execSyncChild(`launchctl load "${plistPath}"`, { stdio: "ignore", timeout: 10000 });
@@ -730,6 +775,32 @@ WantedBy=default.target
     } catch {
       // Non-fatal
     }
+  } else if (isWin) {
+    // Windows: use Task Scheduler to run Castle at logon
+    // Build a batch wrapper that sets environment variables and starts the server
+    const batPath = join(castleDir, "start-server.bat");
+    const envLines = Object.entries(serverEnv)
+      .map(([k, v]) => `set "${k}=${v}"`)
+      .join("\r\n");
+    const batContent = `@echo off\r\n${envLines}\r\ncd /d "${PROJECT_ROOT}"\r\n"${nodePath}" ${serverArgs.map((a) => `"${a}"`).join(" ")}\r\n`;
+    writeFile(batPath, batContent);
+
+    // Task was already deleted above — create fresh
+    try {
+      execSyncChild(
+        `schtasks /Create /TN "CastleServer" /TR "\\"${batPath}\\"" /SC ONLOGON /RL HIGHEST /F 2>nul`,
+        { stdio: "ignore", timeout: 10000 }
+      );
+    } catch {
+      // Non-fatal — fall back to spawning directly
+    }
+
+    // Also start the task now
+    try {
+      execSyncChild('schtasks /Run /TN "CastleServer" 2>nul', { stdio: "ignore", timeout: 10000 });
+    } catch {
+      // Non-fatal
+    }
   }
 
   // If no service manager started it, spawn directly
@@ -805,6 +876,11 @@ WantedBy=default.target
   }
 
   p.outro(pc.dim(`Opening ${BLUE(`http://localhost:${castlePort}`)}...`));
-  const open = (await import("open")).default;
-  await open(`http://localhost:${castlePort}`);
+  const url = `http://localhost:${castlePort}`;
+  if (process.platform === "win32") {
+    execSync(`start "" "${url}"`, { stdio: "ignore" });
+  } else {
+    const open = (await import("open")).default;
+    await open(url);
+  }
 }