Invoke wrangler to check if it's installed, but don't auto-install through npx/bunx

.changeset/brown-spies-grab.md

@@ -0,0 +1,5 @@
+---
+"wrangler-action": minor
+---
+
+This unreverts #235 ensuring wrangler-action will re-use existing wrangler installations, thanks @AdiRishi! and ensures we don't automatically install wrangler when checking if it present

package-lock.json

@@ -1,12 +1,12 @@
 {
 	"name": "wrangler-action",
-	"version": "3.5.0",
+	"version": "3.6.1",
 	"lockfileVersion": 3,
 	"requires": true,
 	"packages": {
 		"": {
 			"name": "wrangler-action",
-			"version": "3.5.0",
+			"version": "3.6.1",
 			"license": "MIT OR Apache-2.0",
 			"dependencies": {
 				"@actions/core": "^1.10.1",

src/index.ts

@@ -90,7 +90,8 @@ async function installWrangler() {
 	let installedVersionSatisfiesRequirement = false;
 	try {
 		const { stdout } = await getExecOutput(
-			packageManager.exec,
+			// We want to simply invoke wrangler to check if it's installed, but don't want to auto-install it at this stage
+			packageManager.execNoInstall,
 			["wrangler", "--version"],
 			{
 				cwd: config["workingDirectory"],

src/packageManagers.test.ts

@@ -7,22 +7,25 @@ describe("getPackageManager", () => {
 			.toMatchInlineSnapshot(`
 				{
 				  "exec": "npx",
+				  "execNoInstall": "npx --no-install",
 				  "install": "npm i",
 				}
 			`);
 
 		expect(getPackageManager("yarn", { workingDirectory: "test/npm" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "yarn",
+				  "exec": "yarn",
-			  "install": "yarn add",
+				  "execNoInstall": "yarn",
-			}
+				  "install": "yarn add",
-		`);
+				}
+			`);
 
 		expect(getPackageManager("pnpm", { workingDirectory: "test/npm" }))
 			.toMatchInlineSnapshot(`
 				{
 				  "exec": "pnpm exec",
+				  "execNoInstall": "pnpm exec",
 				  "install": "pnpm add",
 				}
 			`);
@@ -31,6 +34,7 @@ describe("getPackageManager", () => {
 			.toMatchInlineSnapshot(`
 				{
 				  "exec": "bunx",
+				  "execNoInstall": "bun run",
 				  "install": "bun i",
 				}
 			`);
@@ -39,51 +43,56 @@ describe("getPackageManager", () => {
 	test("should use npm if no value provided and package-lock.json exists", () => {
 		expect(getPackageManager("", { workingDirectory: "test/npm" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "npx",
+				  "exec": "npx",
-			  "install": "npm i",
+				  "execNoInstall": "npx --no-install",
-			}
+				  "install": "npm i",
-		`);
+				}
+			`);
 	});
 
 	test("should use yarn if no value provided and yarn.lock exists", () => {
 		expect(getPackageManager("", { workingDirectory: "test/yarn" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "yarn",
+				  "exec": "yarn",
-			  "install": "yarn add",
+				  "execNoInstall": "yarn",
-			}
+				  "install": "yarn add",
-		`);
+				}
+			`);
 	});
 
 	test("should use pnpm if no value provided and pnpm-lock.yaml exists", () => {
 		expect(getPackageManager("", { workingDirectory: "test/pnpm" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "pnpm exec",
+				  "exec": "pnpm exec",
-			  "install": "pnpm add",
+				  "execNoInstall": "pnpm exec",
-			}
+				  "install": "pnpm add",
-		`);
+				}
+			`);
 	});
 
 	test("should use bun if no value provided and bun.lockb exists", () => {
 		expect(getPackageManager("", { workingDirectory: "test/bun" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "bunx",
+				  "exec": "bunx",
-			  "install": "bun i",
+				  "execNoInstall": "bun run",
-			}
+				  "install": "bun i",
+				}
 			`);
 	});
 
 	test("should use npm if no value provided and no lockfile is present", () => {
 		expect(getPackageManager("", { workingDirectory: "test/empty" }))
 			.toMatchInlineSnapshot(`
-			{
+				{
-			  "exec": "npx",
+				  "exec": "npx",
-			  "install": "npm i",
+				  "execNoInstall": "npx --no-install",
-			}
+				  "install": "npm i",
-		`);
+				}
+			`);
 	});
 
 	test("should throw if an invalid value is provided", () => {

src/packageManagers.ts

@@ -4,24 +4,29 @@ import * as path from "node:path";
 interface PackageManager {
 	install: string;
 	exec: string;
+	execNoInstall: string;
 }
 
 const PACKAGE_MANAGERS = {
 	npm: {
 		install: "npm i",
 		exec: "npx",
+		execNoInstall: "npx --no-install",
 	},
 	yarn: {
 		install: "yarn add",
 		exec: "yarn",
+		execNoInstall: "yarn",
 	},
 	pnpm: {
 		install: "pnpm add",
 		exec: "pnpm exec",
+		execNoInstall: "pnpm exec",
 	},
 	bun: {
 		install: "bun i",
 		exec: "bunx",
+		execNoInstall: "bun run",
 	},
 } as const satisfies Readonly<Record<string, PackageManager>>;