bun-ready 0.2.0 → 0.2.3

package/dist/cli.js

@@ -1,11 +1,14 @@
-// src/cli.ts
-import { promises as fs3 } from "node:fs";
-import path5 from "node:path";
-
-// src/analyze.ts
-import path4 from "node:path";
-import os from "node:os";
-import { promises as fs2 } from "node:fs";
+var __defProp = Object.defineProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, {
+      get: all[name],
+      enumerable: true,
+      configurable: true,
+      set: (newValue) => all[name] = () => newValue
+    });
+};
+var __esm = (fn, res) => () => (fn && (res = fn(fn = 0)), res);
 
 // src/spawn.ts
 import { spawn } from "node:child_process";
@@ -18,8 +21,7 @@ var runNodeSpawn = async (cmd, args, cwd) => {
     child.stderr.on("data", (d) => err += String(d));
     child.on("close", (code) => resolve({ code: code ?? 1, stdout: out, stderr: err }));
   });
-};
-var runBunSpawn = async (cmd, args, cwd) => {
+}, runBunSpawn = async (cmd, args, cwd) => {
   const BunAny = globalThis;
   const bun = BunAny.Bun;
   if (!bun)
@@ -46,10 +48,53 @@ var runBunSpawn = async (cmd, args, cwd) => {
   };
   const [stdout, stderr, code] = await Promise.all([readAll(proc.stdout), readAll(proc.stderr), proc.exited]);
   return { code, stdout, stderr };
-};
-var exec = async (cmd, args, cwd) => {
+}, exec = async (cmd, args, cwd) => {
   return await runBunSpawn(cmd, args, cwd);
 };
+var init_spawn = () => {};
+
+// src/bun_check.ts
+var exports_bun_check = {};
+__export(exports_bun_check, {
+  checkBunAvailable: () => checkBunAvailable
+});
+async function checkBunAvailable() {
+  try {
+    const res = await exec("bun", ["--version"], process.cwd());
+    if (res.code === 0 && res.stdout.includes("bun")) {
+      return { available: true };
+    }
+    return {
+      available: false,
+      error: `Bun command returned unexpected output (exit ${res.code})`
+    };
+  } catch (error) {
+    const msg = error instanceof Error ? error.message : String(error);
+    if (msg.includes("ENOENT") || msg.includes("spawn bun ENOENT")) {
+      return {
+        available: false,
+        error: "Bun is not installed. Please install Bun from https://bun.sh or use --no-install and --no-test flags."
+      };
+    }
+    return {
+      available: false,
+      error: `Failed to check Bun availability: ${msg}`
+    };
+  }
+}
+var init_bun_check = __esm(() => {
+  init_spawn();
+});
+
+// src/cli.ts
+import { promises as fs3 } from "node:fs";
+import path5 from "node:path";
+
+// src/analyze.ts
+init_spawn();
+import path4 from "node:path";
+import os from "node:os";
+import { promises as fs2 } from "node:fs";
 
 // src/util.ts
 import { promises as fs } from "node:fs";
@@ -399,7 +444,15 @@ var detectNativeAddonRiskV2 = (repo, config) => {
   const suspects = names.filter((n) => {
     if (allowlist.includes(n))
       return false;
-    return NATIVE_SUSPECTS_V2.includes(n) || includesAny(n, ["napi", "node-gyp", "prebuild", "ffi", "bindings", "native", "native-module"]);
+    const inList = NATIVE_SUSPECTS_V2.includes(n);
+    if (inList)
+      return true;
+    const keywords = ["@napi-rs/", "napi-rs", "node-napi", "neon", "node-gyp", "prebuild", "ffi", "bindings", "native", "native-module"];
+    const keywordMatch = includesAny(n, keywords);
+    if (keywordMatch) {
+      return true;
+    }
+    return false;
   });
   const scriptNames = Object.keys(repo.scripts);
   const hasNodeGypRebuild = scriptNames.some((k) => {
@@ -693,6 +746,18 @@ async function copyIfExists(from, to) {
   }
 }
 async function runBunInstallDryRun(packagePath) {
+  const { checkBunAvailable: checkBunAvailable2 } = await Promise.resolve().then(() => (init_bun_check(), exports_bun_check));
+  const bunCheck = await checkBunAvailable2();
+  if (!bunCheck.available) {
+    const skipReason = bunCheck.error;
+    return {
+      ok: false,
+      summary: `Skipped: ${skipReason}`,
+      logs: [],
+      installAnalysis: { blockedDeps: [], trustedDepsMentioned: [], notes: [] },
+      skipReason
+    };
+  }
   const base = await fs2.mkdtemp(path4.join(os.tmpdir(), "bun-ready-"));
   const cleanup = async () => {
     try {
@@ -726,6 +791,17 @@ function shouldRunBunTest(scripts) {
   return t.toLowerCase().includes("bun test") || t.toLowerCase().trim() === "bun test";
 }
 async function runBunTest(packagePath) {
+  const { checkBunAvailable: checkBunAvailable2 } = await Promise.resolve().then(() => (init_bun_check(), exports_bun_check));
+  const bunCheck = await checkBunAvailable2();
+  if (!bunCheck.available) {
+    const skipReason = bunCheck.error;
+    return {
+      ok: false,
+      summary: `Skipped: ${skipReason}`,
+      logs: [],
+      skipReason
+    };
+  }
   const res = await exec("bun", ["test"], packagePath);
   const combined = [...res.stdout ? res.stdout.split(`
 `) : [], ...res.stderr ? res.stderr.split(`
@@ -758,7 +834,8 @@ async function analyzeSinglePackage(packagePath, opts, config, pkgName) {
     install = {
       ok: installResult.ok,
       summary: installResult.summary,
-      logs: installResult.logs
+      logs: installResult.logs,
+      ...installResult.skipReason !== undefined ? { skipReason: installResult.skipReason } : {}
     };
     installOk = installResult.ok;
     if (installResult.installAnalysis.blockedDeps.length > 0) {
@@ -794,7 +871,8 @@ async function analyzeSinglePackage(packagePath, opts, config, pkgName) {
     test = {
       ok: testResult.ok,
       summary: testResult.summary,
-      logs: testResult.logs
+      logs: testResult.logs,
+      ...testResult.skipReason !== undefined ? { skipReason: testResult.skipReason } : {}
     };
     testOk = testResult.ok;
   }
@@ -1283,6 +1361,21 @@ var main = async () => {
     scanOpts.failOn = opts.failOn;
   }
   const res = await analyzeRepoOverall(scanOpts);
+  if (res.install?.skipReason || res.test?.skipReason) {
+    const skipWarnings = [];
+    if (res.install?.skipReason) {
+      skipWarnings.push(`Install check skipped: ${res.install.skipReason}`);
+    }
+    if (res.test?.skipReason) {
+      skipWarnings.push(`Test run skipped: ${res.test.skipReason}`);
+    }
+    if (skipWarnings.length > 0) {
+      process.stderr.write(`WARNING:
+${skipWarnings.map((w) => `  - ${w}`).join(`
+`)}
+`);
+    }
+  }
   const out = opts.format === "json" ? renderJson(res) : renderMarkdown(res);
   const target = opts.outFile ?? (opts.format === "json" ? "bun-ready.json" : "bun-ready.md");
   const resolved = path5.resolve(process.cwd(), target);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bun-ready",
-  "version": "0.2.0",
+  "version": "0.2.3",
   "description": "CLI that estimates how painful migrating a Node.js repo to Bun might be. Generates a green/yellow/red Markdown report with reasons.",
   "author": "Pas7 Studio",
   "license": "Apache-2.0",