refactor: Extract package manager utility

rschristian · rschristian · commit 27015cf7d780 · 2025-03-27T23:08:15.000-05:00
diff --git a/jsconfig.json b/jsconfig.json
@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "target": "ESNext",
+    "module": "ESNext",
+    "moduleResolution": "NodeNext",
+    "allowJs": true,
+    "checkJs": true,
+    "resolveJsonModule": true,
+    "noEmit": true
+  }
+}
diff --git a/src/index.js b/src/index.js
@@ -1,9 +1,8 @@
-import path from 'path';
 import { getInput, setFailed, startGroup, endGroup, debug } from '@actions/core';
 import { context, getOctokit } from '@actions/github';
 import { exec } from '@actions/exec';
 import SizePlugin from 'size-plugin-core';
-import { fileExists, diffTable, toBool, stripHash } from './utils.js';
+import { getPackageManagerAndInstallScript, diffTable, toBool, stripHash } from './utils.js';
 
 /**
  * @typedef {ReturnType<typeof import("@actions/github").getOctokit>} Octokit
@@ -50,26 +49,7 @@ async function run(octokit, context, token) {
 	const buildScript = getInput('build-script') || 'build';
 	const cwd = process.cwd();
 
-	let yarnLock = await fileExists(path.resolve(cwd, 'yarn.lock'));
-	let pnpmLock = await fileExists(path.resolve(cwd, 'pnpm-lock.yaml'));
-	let bunLock = await fileExists(path.resolve(cwd, 'bun.lockb'));
-	let packageLock = await fileExists(path.resolve(cwd, 'package-lock.json'));
-
-	let packageManager = 'npm';
-	let installScript = 'npm install';
-	if (yarnLock) {
-		installScript = 'yarn --frozen-lockfile';
-		packageManager = 'yarn';
-	} else if (pnpmLock) {
-		installScript = 'pnpm install --frozen-lockfile';
-		packageManager = 'pnpm';
-	} else if (bunLock) {
-		installScript = 'bun install --frozen-lockfile';
-		packageManager = 'bun';
-	} else if (packageLock) {
-		installScript = 'npm ci';
-	}
-
+	let { packageManager, installScript } = await getPackageManagerAndInstallScript(cwd);
 	if (getInput('install-script')) {
 		installScript = getInput('install-script');
 	}
@@ -127,26 +107,8 @@ async function run(octokit, context, token) {
 
 	startGroup(`[base] Install Dependencies`);
 
-	yarnLock = await fileExists(path.resolve(cwd, 'yarn.lock'));
-	pnpmLock = await fileExists(path.resolve(cwd, 'pnpm-lock.yaml'));
-	bunLock = await fileExists(path.resolve(cwd, 'bun.lockb'));
-	packageLock = await fileExists(path.resolve(cwd, 'package-lock.json'));
-
-	packageManager = 'npm';
-	installScript = 'npm install';
-	if (yarnLock) {
-		installScript = `yarn --frozen-lockfile`;
-		packageManager = `yarn`;
-	} else if (pnpmLock) {
-		installScript = `pnpm install --frozen-lockfile`;
-		packageManager = `pnpm`;
-	} else if (bunLock) {
-		installScript = `bun install --frozen-lockfile`;
-		packageManager = `bun`;
-	} else if (packageLock) {
-		installScript = `npm ci`;
-	}
 
+	({ packageManager, installScript } = await getPackageManagerAndInstallScript(cwd));
 	if (getInput('install-script')) {
 		installScript = getInput('install-script');
 	}
diff --git a/src/utils.js b/src/utils.js
@@ -1,6 +1,37 @@
 import fs from 'fs';
+import path from 'path';
 import prettyBytes from 'pretty-bytes';
 
+/**
+ * @param {string} cwd
+ * @returns {Promise<{ packageManager: string, installScript: string }>}
+ */
+export async function getPackageManagerAndInstallScript(cwd) {
+	const [yarnLockExists, pnpmLockExists, bunLockExists, packageLockExists] = await Promise.all([
+		fileExists(path.resolve(cwd, 'yarn.lock')),
+		fileExists(path.resolve(cwd, 'pnpm-lock.yaml')),
+		fileExists(path.resolve(cwd, 'bun.lockb')),
+		fileExists(path.resolve(cwd, 'package-lock.json')),
+	]);
+
+	let packageManager = 'npm';
+	let installScript = 'npm install';
+	if (yarnLockExists) {
+		installScript = 'yarn --frozen-lockfile';
+		packageManager = 'yarn';
+	} else if (pnpmLockExists) {
+		installScript = 'pnpm install --frozen-lockfile';
+		packageManager = 'pnpm';
+	} else if (bunLockExists) {
+		installScript = 'bun install --frozen-lockfile';
+		packageManager = 'bun';
+	} else if (packageLockExists) {
+		installScript = 'npm ci';
+	}
+
+	return { packageManager, installScript };
+}
+
 /**
  * Check if a given file exists and can be accessed.
  * @param {string} filename
