railcode 0.1.16 → 0.1.18

package/README.md

@@ -78,7 +78,7 @@ token is cleared and you're prompted to `railcode login` again.
 ```json
 {
   "app": "dashboard",
-  "build": "pnpm build",
+  "build": "npm run build",
   "dist": "dist"
 }
 ```
@@ -89,5 +89,7 @@ Deploy output resolution:
 
 1. `railcode.json` `dist` wins.
 2. Otherwise `railcode.json` `build` runs and uploads `dist/`.
-3. Otherwise a `package.json` with a `build` script runs `pnpm run build` and uploads `dist/`.
+3. Otherwise a `package.json` with a `build` script runs `<pm> run build` and uploads `dist/`,
+   where `<pm>` is the project's package manager — detected from a `packageManager` field or
+   lockfile (pnpm/yarn/bun), defaulting to `npm` when there's no lockfile.
 4. Otherwise a root `index.html` can be deployed interactively; for CI set `"dist": "."`.

package/dist/index.js

@@ -555,7 +555,13 @@ function shouldSkipDeployPath(root, appRoot, relPath) {
     const deployingAppRoot = resolve(root) === resolve(appRoot);
     if (!deployingAppRoot)
         return false;
-    return first === MANIFEST_NAME || first === "package.json" || first === "pnpm-lock.yaml";
+    return (first === MANIFEST_NAME ||
+        first === "package.json" ||
+        first === "pnpm-lock.yaml" ||
+        first === "package-lock.json" ||
+        first === "yarn.lock" ||
+        first === "bun.lockb" ||
+        first === "bun.lock");
 }
 function appLiveUrl(config, appSlug) {
     const api = new URL(config.apiUrl ?? DEFAULT_API_URL);
@@ -595,6 +601,35 @@ async function resolveDeployPlan(root, manifest) {
     }
     throw new CliError(`Could not find deploy output. Set "dist" in ${MANIFEST_NAME}, add a package.json build script, or put index.html at the app root.`, 2);
 }
+// Which package manager to drive for an app dir. We never want to force a
+// particular one onto users, so we detect what the project already declares —
+// the corepack `packageManager` field first, then a lockfile — and fall back to
+// npm, the only manager guaranteed to ship with Node. A fresh scaffold (no
+// lockfile) therefore uses npm; a user who adopts pnpm/yarn/bun is respected via
+// their lockfile.
+function detectPackageManager(root) {
+    const packagePath = join(root, "package.json");
+    if (existsSync(packagePath)) {
+        try {
+            const pkg = JSON.parse(readFileSync(packagePath, "utf8"));
+            if (typeof pkg.packageManager === "string") {
+                const name = pkg.packageManager.split("@")[0];
+                if (name === "npm" || name === "pnpm" || name === "yarn" || name === "bun")
+                    return name;
+            }
+        }
+        catch {
+            // Malformed package.json — fall through to lockfile / npm default.
+        }
+    }
+    if (existsSync(join(root, "pnpm-lock.yaml")))
+        return "pnpm";
+    if (existsSync(join(root, "yarn.lock")))
+        return "yarn";
+    if (existsSync(join(root, "bun.lockb")) || existsSync(join(root, "bun.lock")))
+        return "bun";
+    return "npm";
+}
 function packageBuildCommand(root) {
     const packagePath = join(root, "package.json");
     if (!existsSync(packagePath))
@@ -609,7 +644,7 @@ function packageBuildCommand(root) {
     if (typeof pkg.scripts?.build !== "string") {
         throw new CliError(`Found package.json but no build script. Add "build", or set "dist" in ${MANIFEST_NAME}.`, 2);
     }
-    return "pnpm run build";
+    return `${detectPackageManager(root)} run build`;
 }
 // ---------------------------------------------------------------------------
 // init
@@ -635,7 +670,7 @@ async function commandInit(args) {
     console.log("Next:");
     console.log(`  cd ${app}`);
     if (template === "react") {
-        console.log("  pnpm install");
+        console.log("  npm install");
     }
     console.log("  railcode deploy");
 }
@@ -654,7 +689,7 @@ function writeStaticStarter(target, app) {
     writeFileSync(join(target, "index.html"), starterIndexHtml(app));
 }
 function writeReactStarter(target, app) {
-    writeFileSync(join(target, MANIFEST_NAME), `${JSON.stringify({ app, build: "pnpm run build", dist: "dist" }, null, 2)}\n`);
+    writeFileSync(join(target, MANIFEST_NAME), `${JSON.stringify({ app, build: "npm run build", dist: "dist" }, null, 2)}\n`);
     writeFileSync(join(target, "package.json"), reactPackageJson(app));
     writeFileSync(join(target, "index.html"), reactIndexHtml(app));
     writeFileSync(join(target, "tsconfig.json"), reactTsConfig());
@@ -1606,8 +1641,6 @@ const LLM_LOGOS: LogoItem[] = [
 const DATA_LOGO_ITEMS: LogoItem[] = [
   { name: "postgres", src: postgresLogo },
   { name: "bigquery", src: bigqueryLogo },
-  { name: "snowflake" },
-  { name: "saved queries" },
 ];
 
 const CONNECTOR_LOGO_ITEMS: LogoItem[] = [
@@ -4595,7 +4628,7 @@ function assetServerPort(args, manifest) {
 // Print the install command and stop — never mutate the app dir on `railcode dev`.
 function ensureAppDeps(cwd) {
     if (!existsSync(join(cwd, "node_modules"))) {
-        throw new CliError("Dependencies are not installed. Run `pnpm install` here, then `railcode dev` again.", 1);
+        throw new CliError(`Dependencies are not installed. Run \`${detectPackageManager(cwd)} install\` here, then \`railcode dev\` again.`, 1);
     }
 }
 async function startAssetServer(plan, cwd, startPort, devProxyPort) {
@@ -4605,11 +4638,14 @@ async function startAssetServer(plan, cwd, startPort, devProxyPort) {
     const reservation = await reserveAvailablePort(startPort, "asset");
     const assetPort = reservation.port;
     // For the detected Vite case, run the local vite binary directly with pinned
-    // host/port so the proxy target is known. (Going through `pnpm dev -- …` would
-    // forward the `--` separator to vite, which then ignores the flags.) A custom
-    // command gets the ports via env and is expected to honour PORT.
+    // host/port so the proxy target is known. `npx` resolves the vite already in
+    // the app's node_modules (deps are guaranteed present — ensureAppDeps runs
+    // first) and is package-manager-agnostic, so we don't force pnpm on the user.
+    // (Going through `<pm> dev -- …` would forward the `--` separator to vite,
+    // which then ignores the flags.) A custom command gets the ports via env and
+    // is expected to honour PORT.
     const command = plan.vite
-        ? `pnpm exec vite --host 127.0.0.1 --port ${assetPort} --strictPort --clearScreen false`
+        ? `npx vite --host 127.0.0.1 --port ${assetPort} --strictPort --clearScreen false`
         : plan.command;
     console.log(`  asset server: ${command}`);
     try {
@@ -4738,7 +4774,7 @@ function resolveDevPlan(cwd, manifest) {
         try {
             const pkg = JSON.parse(readFileSync(pkgPath, "utf8"));
             if (typeof pkg.scripts?.dev === "string") {
-                return { mode: "asset", root, command: "pnpm dev", vite: true };
+                return { mode: "asset", root, command: `${detectPackageManager(cwd)} run dev`, vite: true };
             }
         }
         catch {
@@ -5322,11 +5358,16 @@ function serveStatic(root, pathname, res) {
     }
     serveFile(res, fp, inferContentType(fp));
 }
-function listenLoopback(server, port, limit = 50) {
+export function listenLoopback(server, port, limit = 50) {
     return new Promise((resolvePromise, reject) => {
         let attempt = 0;
         const tryListen = (p) => {
+            const onListening = () => {
+                server.removeListener("error", onError);
+                resolvePromise(p);
+            };
             const onError = (err) => {
+                server.removeListener("listening", onListening);
                 if (err.code === "EADDRINUSE" && attempt < limit) {
                     attempt += 1;
                     tryListen(p + 1);
@@ -5336,10 +5377,8 @@ function listenLoopback(server, port, limit = 50) {
                 }
             };
             server.once("error", onError);
-            server.listen(p, "127.0.0.1", () => {
-                server.removeListener("error", onError);
-                resolvePromise(p);
-            });
+            server.once("listening", onListening);
+            server.listen(p, "127.0.0.1");
         };
         tryListen(port);
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "railcode",
-  "version": "0.1.16",
+  "version": "0.1.18",
   "description": "Developer CLI for the multi-tenant Railcode platform: log in, scaffold, and deploy static apps.",
   "license": "MIT",
   "type": "module",