fix: update snapshot for error test

chore: bump dependencies

.changeset/weak-lobsters-march.md

@@ -0,0 +1,9 @@
+---
+"wrangler-action": patch
+---
+
+Fixes issues with semver comparison, where version parts were treated lexicographically instead of numerically.
+
+Bulk secret uploading was introduced in wrangler `3.4.0`, and this action tries to check if the version used is greater than `3.4.0`, and then if so, using the new bulk secret API which is faster. Due to a bug in the semver comparison, `3.19.0` was being considered less than `3.4.0`, and then using an older and slower method for uploading secrets.
+
+Now the semver comparison is fixed, the faster bulk method is used for uploading secrets when available.

CHANGELOG.md

@@ -1,31 +1,5 @@
 # Changelog
 
-## 3.4.1
-
-### Patch Changes
-
-- [#227](https://github.com/cloudflare/wrangler-action/pull/227) [`bbedd8e54f256d36f81f81f1f05b90937d533bb7`](https://github.com/cloudflare/wrangler-action/commit/bbedd8e54f256d36f81f81f1f05b90937d533bb7) Thanks [@AdiRishi](https://github.com/AdiRishi)! - Surface inner exception when secret:bulk upload command fails
-
-## 3.4.0
-
-### Minor Changes
-
-- [#213](https://github.com/cloudflare/wrangler-action/pull/213) [`d13856dfc92816473ebf47f66e263a2668a97896`](https://github.com/cloudflare/wrangler-action/commit/d13856dfc92816473ebf47f66e263a2668a97896) Thanks [@GrantBirki](https://github.com/GrantBirki)! - This change introduces three new GitHub Actions output variables. These variables are as follows:
-
-  - `command-output` - contains the string results of `stdout`
-  - `command-stderr` - contains the string results of `stderr`
-  - `deployment-url` - contains the string results of the URL that was deployed (ex: `https://<your_pages_site>.pages.dev`)
-
-  These output variables are intended to be used by more advanced workflows that require the output results or deployment url from Wrangler commands in subsequent workflow steps.
-
-### Patch Changes
-
-- [#216](https://github.com/cloudflare/wrangler-action/pull/216) [`9aba9c34daabca23a88191a5fe1b81fa721c1f11`](https://github.com/cloudflare/wrangler-action/commit/9aba9c34daabca23a88191a5fe1b81fa721c1f11) Thanks [@Cherry](https://github.com/Cherry)! - Fixes issues with semver comparison, where version parts were treated lexicographically instead of numerically.
-
-  Bulk secret uploading was introduced in wrangler `3.4.0`, and this action tries to check if the version used is greater than `3.4.0`, and then if so, using the new bulk secret API which is faster. Due to a bug in the semver comparison, `3.19.0` was being considered less than `3.4.0`, and then using an older and slower method for uploading secrets.
-
-  Now the semver comparison is fixed, the faster bulk method is used for uploading secrets when available.
-
 ## 3.3.2
 
 ### Patch Changes

README.md

@@ -27,7 +27,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy
         uses: cloudflare/wrangler-action@v3
         with:
@@ -36,7 +36,7 @@ jobs:
 
 ## Authentication
 
-You'll need to configure Wrangler using GitHub's Secrets feature - go to "Settings -> Secrets" and add your Cloudflare API token (for help finding this, see the [Workers documentation](https://developers.cloudflare.com/workers/wrangler/ci-cd/#api-token)). Your API token is encrypted by GitHub, and the action won't print it into logs, so it should be safe!
+You'll need to configure Wrangler using GitHub's Secrets feature - go to "Settings -> Secrets" and add your Cloudflare API token (for help finding this, see the [Workers documentation](https://developers.cloudflare.com/workers/quickstart/#api-token)). Your API token is encrypted by GitHub, and the action won't print it into logs, so it should be safe!
 
 With your API token set as a secret for your repository, pass it to the action in the `with` block of your workflow. Below, I've set the secret name to `CLOUDFLARE_API_TOKEN`:
 
@@ -138,7 +138,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy
         uses: cloudflare/wrangler-action@v3
         with:
@@ -159,7 +159,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy
         uses: cloudflare/wrangler-action@v3
         with:
@@ -182,7 +182,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy app
         uses: cloudflare/wrangler-action@v3
         with:
@@ -208,7 +208,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy app
         uses: cloudflare/wrangler-action@v3
         with:
@@ -218,91 +218,6 @@ jobs:
 
 For more advanced usage or to programmatically trigger the workflow from scripts, refer to [the GitHub documentation](https://docs.github.com/en/rest/reference/actions#create-a-workflow-dispatch-event) for making API calls.
 
-### Upload a Worker Version
-
-To create a new version of your Worker that is not deployed immediately, use the `wrangler versions upload --experimental-versions` command. Worker versions created in this way can then be deployed all at once at a later time or gradually deployed using the `wranger versions deploy --experimental-versions` command or via the Cloudflare dashboard under the Deployments tab. For now, the `--experimental-versions` flag and wrangler v3.40.0 or above is required to use this feature.
-
-```yaml
-jobs:
-  upload:
-    runs-on: ubuntu-latest
-    name: Deploy
-    steps:
-      - uses: actions/checkout@v4
-      - name: Upload Worker Version
-        uses: cloudflare/wrangler-action@v3
-        with:
-          apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
-          accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
-          command: versions upload --experimental-versions
-```
-
-## Advanced Usage
-
-### Using Wrangler Command Output in Subsequent Steps
-
-More advanced workflows may need to parse the resulting output of Wrangler commands. To do this, you can use the `command-output` output variable in subsequent steps. For example, if you want to print the output of the Wrangler command, you can do the following:
-
-```yaml
-- name: Deploy
-  id: deploy
-  uses: cloudflare/wrangler-action@v3
-  with:
-    apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
-    accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
-    command: pages deploy --project-name=example
-
-- name: print wrangler command output
-  env:
-    CMD_OUTPUT: ${{ steps.deploy.outputs.command-output }}
-  run: echo $CMD_OUTPUT
-```
-
-Now when you run your workflow, you will see the full output of the Wrangler command in your workflow logs. You can also use this output in subsequent workflow steps to parse the output for specific values.
-
-> Note: the `command-stderr` output variable is also available if you need to parse the standard error output of the Wrangler command.
-
-### Using the `deployment-url` Output Variable
-
-If you are executing a Wrangler command that results in either a Workers or Pages deployment, you can utilize the `deployment-url` output variable to get the URL of the deployment. For example, if you want to print the deployment URL after deploying your application, you can do the following:
-
-```yaml
-- name: Deploy
-  id: deploy
-  uses: cloudflare/wrangler-action@v3
-  with:
-    apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
-    accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
-    command: pages deploy --project-name=example
-
-- name: print deployment-url
-  env:
-    DEPLOYMENT_URL: ${{ steps.deploy.outputs.deployment-url }}
-  run: echo $DEPLOYMENT_URL
-```
-
-The resulting output will look something like this:
-
-```text
-https://<your_pages_site>.pages.dev
-```
-
-### Using a different package manager
-
-By default, this action will detect which package manager to use, based on the presence of a `package-lock.json`, `yarn.lock`, `pnpm-lock.yaml`, or `bun.lockb` file.
-
-If you need to use a specific package manager for your application, you can set the `packageManager` input to `npm`, `yarn`, `pnpm`, or `bun`. You don't need to set this option unless you want to override the default behavior.
-
-```yaml
-jobs:
-  deploy:
-    steps:
-      uses: cloudflare/wrangler-action@v3
-      with:
-        apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
-        packageManager: pnpm
-```
-
 ## Troubleshooting
 
 ### "I just started using Workers/Wrangler and I don't know what this is!"
@@ -321,7 +236,7 @@ jobs:
     runs-on: ubuntu-latest
     name: Deploy
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Deploy app
         uses: cloudflare/wrangler-action@v3
         with:

action.yml

@@ -44,10 +44,3 @@ inputs:
   packageManager:
     description: "The package manager you'd like to use to install and run wrangler. If not specified, the preferred package manager will be inferred based on the presence of a lockfile or fallback to using npm if no lockfile is found. Valid values are `npm` | `pnpm` | `yarn` | `bun`."
     required: false
-outputs:
-  command-output:
-    description: "The output of the Wrangler command (comes from stdout)"
-  command-stderr:
-    description: "The error output of the Wrangler command (comes from stderr)"
-  deployment-url:
-    description: "If the command was a Workers or Pages deployment, this will be the URL of the deployment"

package-lock.json

@@ -1,12 +1,12 @@
 {
 	"name": "wrangler-action",
-	"version": "3.4.1",
+	"version": "3.3.2",
 	"lockfileVersion": 3,
 	"requires": true,
 	"packages": {
 		"": {
 			"name": "wrangler-action",
-			"version": "3.4.1",
+			"version": "3.3.2",
 			"license": "MIT OR Apache-2.0",
 			"dependencies": {
 				"@actions/core": "^1.10.1",
@@ -4548,9 +4548,9 @@
 			}
 		},
 		"node_modules/undici": {
-			"version": "5.28.3",
-			"resolved": "https://registry.npmjs.org/undici/-/undici-5.28.3.tgz",
-			"integrity": "sha512-3ItfzbrhDlINjaP0duwnNsKpDQk3acHI3gVJ1z4fmwMK31k5G9OVIAMLSIaP6w4FaGkaAkN6zaQO9LUvZ1t7VA==",
+			"version": "5.28.2",
+			"resolved": "https://registry.npmjs.org/undici/-/undici-5.28.2.tgz",
+			"integrity": "sha512-wh1pHJHnUeQV5Xa8/kyQhO7WFa8M34l026L5P/+2TYiakvGy5Rdc8jWZVyG7ieht/0WgJLEd3kcU5gKx+6GC8w==",
 			"dependencies": {
 				"@fastify/busboy": "^2.0.0"
 			},
@@ -4592,9 +4592,9 @@
 			}
 		},
 		"node_modules/vite": {
-			"version": "5.0.12",
-			"resolved": "https://registry.npmjs.org/vite/-/vite-5.0.12.tgz",
-			"integrity": "sha512-4hsnEkG3q0N4Tzf1+t6NdN9dg/L3BM+q8SWgbSPnJvrgH2kgdyzfVJwbR1ic69/4uMJJ/3dqDZZE5/WwqW8U1w==",
+			"version": "5.0.7",
+			"resolved": "https://registry.npmjs.org/vite/-/vite-5.0.7.tgz",
+			"integrity": "sha512-B4T4rJCDPihrQo2B+h1MbeGL/k/GMAHzhQ8S0LjQ142s6/+l3hHTT095ORvsshj4QCkoWu3Xtmob5mazvakaOw==",
 			"dev": true,
 			"dependencies": {
 				"esbuild": "^0.19.3",

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wrangler-action",
-	"version": "3.4.1",
+	"version": "3.3.2",
 	"description": "GitHub Action to use [Wrangler](https://developers.cloudflare.com/workers/cli-wrangler/).",
 	"author": "wrangler@cloudflare.com",
 	"license": "MIT OR Apache-2.0",

src/index.ts

@@ -5,10 +5,8 @@ import {
 	endGroup as originalEndGroup,
 	error as originalError,
 	info as originalInfo,
-	debug,
 	startGroup as originalStartGroup,
 	setFailed,
-	setOutput,
 } from "@actions/core";
 import { exec, execShell } from "./exec";
 import { checkWorkingDirectory, semverCompare } from "./utils";
@@ -205,11 +203,7 @@ async function uploadSecrets() {
 				),
 			),
 		});
-	} catch (err: unknown) {
-		if (err instanceof Error) {
-			error(err.message);
-			err.stack && debug(err.stack);
-		}
+	} catch (err) {
 		throw new Error(`Failed to upload secrets.`);
 	} finally {
 		endGroup();
@@ -248,46 +242,10 @@ async function wranglerCommands() {
 				}
 			}
 
-			// Used for saving the wrangler output
-			let stdOut = "";
-			let stdErr = "";
-
-			// Construct the options for the exec command
-			const options = {
+			await exec(`${packageManager.exec} wrangler ${command}`, args, {
 				cwd: config["workingDirectory"],
 				silent: config["QUIET_MODE"],
-				listeners: {
-					stdout: (data: Buffer) => {
-						stdOut += data.toString();
-					},
-					stderr: (data: Buffer) => {
-						stdErr += data.toString();
-					},
-				},
-			};
-
-			// Execute the wrangler command
-			await exec(`${packageManager.exec} wrangler ${command}`, args, options);
-
-			// Set the outputs for the command
-			setOutput("command-output", stdOut);
-			setOutput("command-stderr", stdErr);
-
-			// Check if this command is a workers or pages deployment
-			if (
-				command.startsWith("deploy") ||
-				command.startsWith("publish") ||
-				command.startsWith("pages publish") ||
-				command.startsWith("pages deploy")
-			) {
-				// If this is a workers or pages deployment, try to extract the deployment URL
-				let deploymentUrl = "";
-				const deploymentUrlMatch = stdOut.match(/https?:\/\/[a-zA-Z0-9-./]+/);
-				if (deploymentUrlMatch && deploymentUrlMatch[0]) {
-					deploymentUrl = deploymentUrlMatch[0].trim();
-					setOutput("deployment-url", deploymentUrl);
-				}
-			}
+			});
 		}
 	} finally {
 		endGroup();