fix: prevent local link overwriting in smoke tests and improve runtime selection logic for cli runner

Author: soymadip

packages/cli/src/utils/runner.mjs

@@ -2,6 +2,7 @@ import fs from "fs";
 import path from "path";
 import { spawn } from "child_process";
 import { createRequire } from "module";
+import { hasCommand } from "@portosaur/core";
 
 /**
  * Generates a static Docusaurus config file by evaluating the Portosaur config
@@ -174,7 +175,11 @@ export async function runDocusaurus(
   args.push(...extraArgs);
 
   // Use bun when available for faster builds, fall back to node.
-  const runtime = typeof Bun !== "undefined" ? "bun" : "node";
+  // If node isn't installed, fallback to bun gracefully.
+  let runtime = "node";
+  if (typeof Bun !== "undefined" || !hasCommand("node")) {
+    runtime = "bun";
+  }
 
   // Skip actual execution in test mode
   if (process.env.PORTO_TEST_MODE === "true") {
@@ -186,10 +191,6 @@ export async function runDocusaurus(
 
   const childEnv = { ...process.env, FORCE_COLOR: "true" };
 
-  if (command === "build") {
-    childEnv.CI = "true";
-  }
-
   const child = spawn(runtime, args, {
     stdio: "inherit",
     cwd: UserRoot,

src/scripts/smoke-test.sh

@@ -23,6 +23,11 @@ bun run "$REPO_ROOT/packages/cli/bin/porto.mjs" init \
 # --- Link local packages into the new project ---
 echo "📦 Installing local packages..."
 cd "$SITE_DIR"
+
+# Prevent manual developer runs (e.g. bun run dev) from overwriting local links
+# by stripping the auto-install step from package scripts
+sed -i 's/$npm_execpath install && //' package.json
+
 bun install
 bun link @portosaur/cli @portosaur/theme @portosaur/core @portosaur/logger @portosaur/wizard