wrangler 2.0.16 → 2.0.19

package/bin/wrangler.js

@@ -3,9 +3,12 @@ const { spawn } = require("child_process");
 const path = require("path");
 const fs = require("fs");
 const os = require("os");
-const semiver = require("semiver");
 
 const MIN_NODE_VERSION = "16.7.0";
+const debug =
+	process.env["WRANGLER_LOG"] === "debug"
+		? (...args) => console.log(...args)
+		: () => {};
 
 let wranglerProcess;
 
@@ -17,8 +20,8 @@ function runWrangler() {
 		// Note Volta and nvm are also recommended in the official docs:
 		// https://developers.cloudflare.com/workers/get-started/guide#2-install-the-workers-cli
 		console.error(
-			`Wrangler requires at least Node.js v${MIN_NODE_VERSION}. You are using v${process.versions.node}.
-You should use the latest Node.js version if possible, as Cloudflare Workers use a very up-to-date version of V8.
+			`Wrangler requires at least node.js v${MIN_NODE_VERSION}. You are using v${process.versions.node}. Please update your version of node.js.
+
 Consider using a Node.js version manager such as https://volta.sh/ or https://github.com/nvm-sh/nvm.`
 		);
 		process.exitCode = 1;
@@ -84,9 +87,8 @@ function runDelegatedWrangler() {
 	} = JSON.parse(fs.readFileSync(packageJsonPath));
 	const resolvedBinaryPath = path.resolve(packageJsonPath, "..", binaryPath);
 
-	console.log(
-		`Delegating to locally-installed version of wrangler @ v${version}`
-	);
+	debug(`Delegating to locally-installed version of wrangler @ v${version}`);
+
 	// this call to `spawn` is simpler because the delegated version will do all
 	// of the other work.
 	return spawn(
@@ -125,10 +127,42 @@ async function main() {
 }
 
 process.on("SIGINT", () => {
-	wranglerProcess?.kill();
+	wranglerProcess && wranglerProcess.kill();
 });
 process.on("SIGTERM", () => {
-	wranglerProcess?.kill();
+	wranglerProcess && wranglerProcess.kill();
 });
 
+// semiver implementation via https://github.com/lukeed/semiver/blob/ae7eebe6053c96be63032b14fb0b68e2553fcac4/src/index.js
+
+/**
+MIT License
+
+Copyright (c) Luke Edwards <luke.edwards05@gmail.com> (lukeed.com)
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+ */
+
+var fn = new Intl.Collator(0, { numeric: 1 }).compare;
+
+function semiver(a, b, bool) {
+	a = a.split(".");
+	b = b.split(".");
+
+	return (
+		fn(a[0], b[0]) ||
+		fn(a[1], b[1]) ||
+		((b[2] = b.slice(2).join(".")),
+		(bool = /[.-]/.test((a[2] = a.slice(2).join(".")))),
+		bool == /[.-]/.test(b[2]) ? fn(a[2], b[2]) : bool ? -1 : 1)
+	);
+}
+
+// end semiver implementation
+
 void main();

package/miniflare-dist/index.mjs

@@ -4871,6 +4871,7 @@ async function main() {
   try {
     await mf.startServer();
     await mf.startScheduler();
+    process.send && process.send("ready");
   } catch (e) {
     mf.log.error(e);
     process.exitCode = 1;

package/package.json

@@ -1,18 +1,7 @@
 {
 	"name": "wrangler",
-	"version": "2.0.16",
-	"author": "wrangler@cloudflare.com",
+	"version": "2.0.19",
 	"description": "Command-line interface for all things Cloudflare Workers",
-	"bin": {
-		"wrangler": "./bin/wrangler.js",
-		"wrangler2": "./bin/wrangler.js"
-	},
-	"main": "wrangler-dist/cli.js",
-	"license": "MIT OR Apache-2.0",
-	"bugs": {
-		"url": "https://github.com/cloudflare/wrangler2/issues"
-	},
-	"homepage": "https://github.com/cloudflare/wrangler2#readme",
 	"keywords": [
 		"wrangler",
 		"cloudflare",
@@ -29,31 +18,86 @@
 		"webassembly",
 		"rust",
 		"emscripten",
-		"rust",
 		"typescript",
 		"graphql",
 		"router",
 		"http",
 		"cli"
 	],
+	"homepage": "https://github.com/cloudflare/wrangler2#readme",
+	"bugs": {
+		"url": "https://github.com/cloudflare/wrangler2/issues"
+	},
+	"license": "MIT OR Apache-2.0",
+	"author": "wrangler@cloudflare.com",
+	"main": "wrangler-dist/cli.js",
+	"types": "wrangler-dist/cli.d.ts",
+	"bin": {
+		"wrangler": "./bin/wrangler.js",
+		"wrangler2": "./bin/wrangler.js"
+	},
+	"files": [
+		"src",
+		"bin",
+		"miniflare-config-stubs",
+		"miniflare-dist",
+		"wrangler-dist",
+		"templates",
+		"vendor",
+		"import_meta_url.js",
+		"kv-asset-handler.js",
+		"Cloudflare_CA.pem"
+	],
+	"scripts": {
+		"build": "npm run clean && npm run bundle && npm run emit-types",
+		"bundle": "node -r esbuild-register scripts/bundle.ts",
+		"check:type": "tsc",
+		"clean": "rm -rf wrangler-dist miniflare-dist emitted-types",
+		"dev": "npm run clean && concurrently -c black,blue 'npm run bundle -- --watch' 'npm run check:type -- --watch --preserveWatchOutput'",
+		"emit-types": "tsc -p tsconfig.emit.json && node -r esbuild-register scripts/emit-types.ts",
+		"prepublishOnly": "SOURCEMAPS=false npm run build",
+		"start": "npm run bundle && NODE_OPTIONS=--enable-source-maps ./bin/wrangler.js",
+		"test": "jest --silent=false --verbose=true",
+		"test-watch": "npm run test -- --runInBand --testTimeout=50000 --watch"
+	},
+	"jest": {
+		"moduleNameMapper": {
+			"clipboardy": "<rootDir>/src/__tests__/helpers/clipboardy-mock.js",
+			"miniflare/cli": "<rootDir>/../../node_modules/miniflare/dist/src/cli.js"
+		},
+		"restoreMocks": true,
+		"setupFilesAfterEnv": [
+			"<rootDir>/src/__tests__/jest.setup.ts"
+		],
+		"testRegex": ".*.(test|spec)\\.[jt]sx?$",
+		"testTimeout": 30000,
+		"transform": {
+			"^.+\\.c?(t|j)sx?$": [
+				"esbuild-jest",
+				{
+					"sourcemap": true
+				}
+			]
+		},
+		"transformIgnorePatterns": [
+			"node_modules/(?!find-up|locate-path|p-locate|p-limit|p-timeout|p-queue|yocto-queue|path-exists|execa|strip-final-newline|npm-run-path|path-key|onetime|mimic-fn|human-signals|is-stream|get-port|supports-color|pretty-bytes)"
+		]
+	},
 	"dependencies": {
 		"@cloudflare/kv-asset-handler": "^0.2.0",
 		"@esbuild-plugins/node-globals-polyfill": "^0.1.1",
 		"@esbuild-plugins/node-modules-polyfill": "^0.1.4",
 		"blake3-wasm": "^2.1.5",
 		"esbuild": "0.14.47",
-		"miniflare": "^2.5.1",
+		"miniflare": "^2.6.0",
 		"nanoid": "^3.3.3",
 		"path-to-regexp": "^6.2.0",
 		"selfsigned": "^2.0.1",
-		"semiver": "^1.1.0",
 		"xxhash-wasm": "^1.0.1"
 	},
-	"optionalDependencies": {
-		"fsevents": "~2.3.2"
-	},
 	"devDependencies": {
 		"@iarna/toml": "^3.0.0",
+		"@microsoft/api-extractor": "^7.28.3",
 		"@types/command-exists": "^1.2.0",
 		"@types/glob-to-regexp": "0.4.1",
 		"@types/mime": "^2.0.3",
@@ -104,55 +148,13 @@
 		"undici": "^5.5.1",
 		"update-check": "^1.5.4",
 		"ws": "^8.5.0",
+		"xdg-app-paths": "^7.3.0",
 		"yargs": "^17.4.1"
 	},
-	"files": [
-		"src",
-		"bin",
-		"miniflare-config-stubs",
-		"miniflare-dist",
-		"wrangler-dist",
-		"templates",
-		"vendor",
-		"import_meta_url.js",
-		"kv-asset-handler.js",
-		"Cloudflare_CA.pem"
-	],
-	"scripts": {
-		"clean": "rm -rf wrangler-dist miniflare-dist",
-		"check:type": "tsc",
-		"bundle": "node -r esbuild-register scripts/bundle.ts",
-		"build": "npm run clean && npm run bundle",
-		"dev": "npm run clean && concurrently -c black,blue 'npm run bundle -- --watch' 'npm run check:type -- --watch --preserveWatchOutput'",
-		"prepublishOnly": "SOURCEMAPS=false npm run build",
-		"start": "npm run bundle && NODE_OPTIONS=--enable-source-maps ./bin/wrangler.js",
-		"test": "jest --silent=false --verbose=true",
-		"test-watch": "npm run test -- --runInBand --testTimeout=50000 --watch"
+	"optionalDependencies": {
+		"fsevents": "~2.3.2"
 	},
 	"engines": {
 		"node": ">=16.7.0"
-	},
-	"jest": {
-		"restoreMocks": true,
-		"testTimeout": 30000,
-		"testRegex": ".*.(test|spec)\\.[jt]sx?$",
-		"transformIgnorePatterns": [
-			"node_modules/(?!find-up|locate-path|p-locate|p-limit|p-timeout|p-queue|yocto-queue|path-exists|execa|strip-final-newline|npm-run-path|path-key|onetime|mimic-fn|human-signals|is-stream|get-port|supports-color|pretty-bytes)"
-		],
-		"moduleNameMapper": {
-			"clipboardy": "<rootDir>/src/__tests__/helpers/clipboardy-mock.js",
-			"miniflare/cli": "<rootDir>/../../node_modules/miniflare/dist/src/cli.js"
-		},
-		"transform": {
-			"^.+\\.c?(t|j)sx?$": [
-				"esbuild-jest",
-				{
-					"sourcemap": true
-				}
-			]
-		},
-		"setupFilesAfterEnv": [
-			"<rootDir>/src/__tests__/jest.setup.ts"
-		]
 	}
 }

package/src/__tests__/configuration.test.ts

@@ -40,6 +40,7 @@ describe("normalizeAndValidateConfig()", () => {
 			tsconfig: undefined,
 			kv_namespaces: [],
 			legacy_env: true,
+			send_metrics: undefined,
 			main: undefined,
 			migrations: [],
 			name: undefined,
@@ -76,6 +77,7 @@ describe("normalizeAndValidateConfig()", () => {
 		it("should override config defaults with provided values", () => {
 			const expectedConfig: Partial<ConfigFields<RawDevConfig>> = {
 				legacy_env: true,
+				send_metrics: false,
 				dev: {
 					ip: "255.255.255.255",
 					port: 9999,
@@ -98,6 +100,7 @@ describe("normalizeAndValidateConfig()", () => {
 		it("should error on invalid top level fields", () => {
 			const expectedConfig = {
 				legacy_env: "FOO",
+				send_metrics: "BAD",
 				dev: {
 					ip: 222,
 					port: "FOO",
@@ -117,13 +120,14 @@ describe("normalizeAndValidateConfig()", () => {
 			);
 			expect(diagnostics.hasWarnings()).toBe(false);
 			expect(diagnostics.renderErrors()).toMatchInlineSnapshot(`
-			        "Processing wrangler configuration:
-			          - Expected \\"legacy_env\\" to be of type boolean but got \\"FOO\\".
-			          - Expected \\"dev.ip\\" to be of type string but got 222.
-			          - Expected \\"dev.port\\" to be of type number but got \\"FOO\\".
-			          - Expected \\"dev.local_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"wss\\".
-			          - Expected \\"dev.upstream_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"ws\\"."
-		      `);
+			"Processing wrangler configuration:
+			  - Expected \\"legacy_env\\" to be of type boolean but got \\"FOO\\".
+			  - Expected \\"send_metrics\\" to be of type boolean but got \\"BAD\\".
+			  - Expected \\"dev.ip\\" to be of type string but got 222.
+			  - Expected \\"dev.port\\" to be of type number but got \\"FOO\\".
+			  - Expected \\"dev.local_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"wss\\".
+			  - Expected \\"dev.upstream_protocol\\" field to be one of [\\"http\\",\\"https\\"] but got \\"ws\\"."
+		`);
 		});
 
 		it("should warn on and remove unexpected top level fields", () => {

package/src/__tests__/helpers/run-in-tmp.ts

@@ -5,35 +5,34 @@ import { reinitialiseAuthTokens } from "../../user";
 
 const originalCwd = process.cwd();
 
-export function runInTempDir({ homedir }: { homedir?: string } = {}) {
+export function runInTempDir({ homedir } = { homedir: "./home" }) {
 	let tmpDir: string;
 
-	beforeAll(() => {
-		if (tmpDir !== undefined) {
-			process.chdir(originalCwd);
-			fs.rmSync(tmpDir, { recursive: true });
-		}
-	});
-
 	beforeEach(() => {
-		tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "wrangler-tests"));
+		// Use realpath because the temporary path can point to a symlink rather than the actual path.
+		tmpDir = fs.realpathSync(
+			fs.mkdtempSync(path.join(os.tmpdir(), "wrangler-tests"))
+		);
+
 		process.chdir(tmpDir);
-		if (homedir !== undefined) {
-			// Override where the home directory is so that we can write our own user config,
-			// without destroying the real thing.
-			fs.mkdirSync(homedir);
-			// Note it is very important that we use the "default" value from "node:os" (e.g. `import os from "node:os";`)
-			// rather than an alias to the module (e.g. `import * as os from "node:os";`).
-			// This is because the module gets transpiled so that the "method" `homedir()` gets converted to a
-			// getter that is not configurable (and so cannot be spied upon).
-			jest.spyOn(os, "homedir").mockReturnValue(homedir);
-			// Now that we have changed the home directory location, we must reinitialize the user auth state
-			reinitialiseAuthTokens();
-		}
+		// The path that is returned from `homedir()` should be absolute.
+		const absHomedir = path.resolve(tmpDir, homedir);
+		// Override where the home directory is so that we can write our own user config,
+		// without destroying the real thing.
+		fs.mkdirSync(absHomedir, { recursive: true });
+		// Note it is very important that we use the "default" value from "node:os" (e.g. `import os from "node:os";`)
+		// rather than an alias to the module (e.g. `import * as os from "node:os";`).
+		// This is because the module gets transpiled so that the "method" `homedir()` gets converted to a
+		// getter that is not configurable (and so cannot be spied upon).
+		jest.spyOn(os, "homedir")?.mockReturnValue(absHomedir);
+		// Now that we have changed the home directory location, we must reinitialize the user auth state
+		reinitialiseAuthTokens();
 	});
 
 	afterEach(() => {
-		process.chdir(originalCwd);
-		fs.rmSync(tmpDir, { recursive: true });
+		if (fs.existsSync(tmpDir)) {
+			process.chdir(originalCwd);
+			fs.rmSync(tmpDir, { recursive: true });
+		}
 	});
 }

package/src/__tests__/https-options.test.ts

@@ -1,24 +1,25 @@
 import fs from "node:fs";
 import os from "node:os";
-import { resolve } from "node:path";
+import path from "node:path";
+import { getGlobalWranglerConfigPath } from "../global-wrangler-config-path";
 import { getHttpsOptions } from "../https-options";
 import { mockConsoleMethods } from "./helpers/mock-console";
 import { runInTempDir } from "./helpers/run-in-tmp";
 
 describe("getHttpsOptions()", () => {
-	runInTempDir({ homedir: "./home" });
+	runInTempDir();
 	const std = mockConsoleMethods();
 
 	it("should use cached values if they have not expired", async () => {
-		fs.mkdirSync(resolve(os.homedir(), ".wrangler/local-cert"), {
+		fs.mkdirSync(path.resolve(getGlobalWranglerConfigPath(), "local-cert"), {
 			recursive: true,
 		});
 		fs.writeFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/key.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem"),
 			"PRIVATE KEY"
 		);
 		fs.writeFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/cert.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem"),
 			"PUBLIC KEY"
 		);
 		const result = await getHttpsOptions();
@@ -32,11 +33,11 @@ describe("getHttpsOptions()", () => {
 	it("should generate and cache new keys if none are cached", async () => {
 		const result = await getHttpsOptions();
 		const key = fs.readFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/key.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem"),
 			"utf8"
 		);
 		const cert = fs.readFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/cert.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem"),
 			"utf8"
 		);
 		expect(result.key).toEqual(key);
@@ -49,28 +50,28 @@ describe("getHttpsOptions()", () => {
 	});
 
 	it("should generate and cache new keys if cached files have expired", async () => {
-		fs.mkdirSync(resolve(os.homedir(), ".wrangler/local-cert"), {
+		fs.mkdirSync(path.resolve(getGlobalWranglerConfigPath(), "local-cert"), {
 			recursive: true,
 		});
 		const ORIGINAL_KEY = "EXPIRED PRIVATE KEY";
 		const ORIGINAL_CERT = "EXPIRED PUBLIC KEY";
 		fs.writeFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/key.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem"),
 			ORIGINAL_KEY
 		);
 		fs.writeFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/cert.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem"),
 			ORIGINAL_CERT
 		);
 		mockStatSync(/\.pem$/, { mtimeMs: new Date(2000).valueOf() });
 
 		const result = await getHttpsOptions();
 		const key = fs.readFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/key.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem"),
 			"utf8"
 		);
 		const cert = fs.readFileSync(
-			resolve(os.homedir(), ".wrangler/local-cert/cert.pem"),
+			path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem"),
 			"utf8"
 		);
 		expect(key).not.toEqual(ORIGINAL_KEY);
@@ -84,25 +85,57 @@ describe("getHttpsOptions()", () => {
 		expect(std.err).toMatchInlineSnapshot(`""`);
 	});
 
+	it("should warn if not able to write to the cache (legacy config path)", async () => {
+		fs.mkdirSync(path.join(os.homedir(), ".wrangler"));
+		mockWriteFileSyncThrow(/\.pem$/);
+		await getHttpsOptions();
+		expect(
+			fs.existsSync(
+				path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem")
+			)
+		).toBe(false);
+		expect(
+			fs.existsSync(
+				path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem")
+			)
+		).toBe(false);
+		expect(std.out).toMatchInlineSnapshot(
+			`"Generating new self-signed certificate..."`
+		);
+		expect(std.warn).toMatchInlineSnapshot(`
+		      "▲ [WARNING] Unable to cache generated self-signed certificate in home/.wrangler/local-cert.
+
+		        ERROR: Cannot write file
+
+		      "
+	    `);
+		expect(std.err).toMatchInlineSnapshot(`""`);
+		fs.rmSync(path.join(os.homedir(), ".wrangler"), { recursive: true });
+	});
+
 	it("should warn if not able to write to the cache", async () => {
 		mockWriteFileSyncThrow(/\.pem$/);
 		await getHttpsOptions();
 		expect(
-			fs.existsSync(resolve(os.homedir(), ".wrangler/local-cert/key.pem"))
+			fs.existsSync(
+				path.resolve(getGlobalWranglerConfigPath(), "local-cert/key.pem")
+			)
 		).toBe(false);
 		expect(
-			fs.existsSync(resolve(os.homedir(), ".wrangler/local-cert/cert.pem"))
+			fs.existsSync(
+				path.resolve(getGlobalWranglerConfigPath(), "local-cert/cert.pem")
+			)
 		).toBe(false);
 		expect(std.out).toMatchInlineSnapshot(
 			`"Generating new self-signed certificate..."`
 		);
 		expect(std.warn).toMatchInlineSnapshot(`
-      "▲ [WARNING] Unable to cache generated self-signed certificate in home/.wrangler/local-cert.
+		"▲ [WARNING] Unable to cache generated self-signed certificate in test-xdg-config/local-cert.
 
-        ERROR: Cannot write file
+		  ERROR: Cannot write file
 
-      "
-    `);
+		"
+	`);
 		expect(std.err).toMatchInlineSnapshot(`""`);
 	});
 });

package/src/__tests__/index.test.ts

@@ -35,7 +35,7 @@ describe("wrangler", () => {
 			  wrangler dev [script]      👂 Start a local server for developing your worker
 			  wrangler publish [script]  🆙 Publish your Worker to Cloudflare.
 			  wrangler tail [name]       🦚 Starts a log tailing session for a published Worker.
-			  wrangler secret            🤫 Generate a secret that can be referenced in the worker script
+			  wrangler secret            🤫 Generate a secret that can be referenced in a Worker
 			  wrangler kv:namespace      🗂️  Interact with your Workers KV Namespaces
 			  wrangler kv:key            🔑 Individually manage Workers KV key-value pairs
 			  wrangler kv:bulk           💪 Interact with multiple Workers KV key-value pairs at once
@@ -74,7 +74,7 @@ describe("wrangler", () => {
 			  wrangler dev [script]      👂 Start a local server for developing your worker
 			  wrangler publish [script]  🆙 Publish your Worker to Cloudflare.
 			  wrangler tail [name]       🦚 Starts a log tailing session for a published Worker.
-			  wrangler secret            🤫 Generate a secret that can be referenced in the worker script
+			  wrangler secret            🤫 Generate a secret that can be referenced in a Worker
 			  wrangler kv:namespace      🗂️  Interact with your Workers KV Namespaces
 			  wrangler kv:key            🔑 Individually manage Workers KV key-value pairs
 			  wrangler kv:bulk           💪 Interact with multiple Workers KV key-value pairs at once
@@ -123,20 +123,20 @@ describe("wrangler", () => {
 			await runWrangler("secret");
 			await endEventLoop();
 			expect(std.out).toMatchInlineSnapshot(`
-			        "wrangler secret
+			"wrangler secret
 
-			        🤫 Generate a secret that can be referenced in the worker script
+			🤫 Generate a secret that can be referenced in a Worker
 
-			        Commands:
-			          wrangler secret put <key>     Create or update a secret variable for a script
-			          wrangler secret delete <key>  Delete a secret variable from a script
-			          wrangler secret list          List all secrets for a script
+			Commands:
+			  wrangler secret put <key>     Create or update a secret variable for a Worker
+			  wrangler secret delete <key>  Delete a secret variable from a Worker
+			  wrangler secret list          List all secrets for a Worker
 
-			        Flags:
-			          -c, --config   Path to .toml configuration file  [string]
-			          -h, --help     Show help  [boolean]
-			          -v, --version  Show version number  [boolean]"
-		      `);
+			Flags:
+			  -c, --config   Path to .toml configuration file  [string]
+			  -h, --help     Show help  [boolean]
+			  -v, --version  Show version number  [boolean]"
+		`);
 		});
 
 		it("no subcommand 'kv:namespace' should display a list of available subcommands", async () => {

package/src/__tests__/jest.setup.ts

@@ -105,3 +105,16 @@ jest.mock("../user/generate-random-state", () => {
 		generateRandomState: jest.fn().mockImplementation(() => "MOCK_STATE_PARAM"),
 	};
 });
+
+jest.mock("xdg-app-paths", () => {
+	return {
+		__esModule: true,
+		default: jest.fn().mockImplementation(() => {
+			return {
+				config() {
+					return jest.requireActual("node:path").resolve("test-xdg-config");
+				},
+			};
+		}),
+	};
+});

package/src/__tests__/kv.test.ts

@@ -1,5 +1,5 @@
 import { writeFileSync } from "node:fs";
-import { Headers } from "undici";
+import { FormData, Headers } from "undici";
 import { mockAccountId, mockApiToken } from "./helpers/mock-account-id";
 import {
 	setMockResponse,
@@ -330,7 +330,15 @@ describe("wrangler", () => {
 						expect(accountId).toEqual("some-account-id");
 						expect(namespaceId).toEqual(expectedNamespaceId);
 						expect(key).toEqual(expectedKV.key);
-						expect(body).toEqual(expectedKV.value);
+						if (expectedKV.metadata) {
+							expect(body).toBeInstanceOf(FormData);
+							expect((body as FormData).get("value")).toEqual(expectedKV.value);
+							expect((body as FormData).get("metadata")).toEqual(
+								JSON.stringify(expectedKV.metadata)
+							);
+						} else {
+							expect(body).toEqual(expectedKV.value);
+						}
 						if (expectedKV.expiration !== undefined) {
 							expect(query.get("expiration")).toEqual(
 								`${expectedKV.expiration}`
@@ -488,6 +496,24 @@ describe("wrangler", () => {
 				expect(requests.count).toEqual(1);
 			});
 
+			it("should put a key with metadata", async () => {
+				const requests = mockKeyPutRequest("some-namespace-id", {
+					key: "dKey",
+					value: "dVal",
+					metadata: {
+						mKey: "mValue",
+					},
+				});
+				await runWrangler(
+					`kv:key put dKey dVal --namespace-id some-namespace-id --metadata {"mKey":"mValue"}`
+				);
+				expect(requests.count).toEqual(1);
+				expect(std.out).toMatchInlineSnapshot(
+					`"Writing the value \\"dVal\\" to key \\"dKey\\" on namespace some-namespace-id with metadata \\"{\\"mKey\\":\\"mValue\\"}\\"."`
+				);
+				expect(std.err).toMatchInlineSnapshot(`""`);
+			});
+
 			it("should error if no key is provided", async () => {
 				await expect(
 					runWrangler("kv:key put")
@@ -517,6 +543,7 @@ describe("wrangler", () => {
                 --preview       Interact with a preview namespace  [boolean]
                 --ttl           Time for which the entries should be visible  [number]
                 --expiration    Time since the UNIX epoch after which the entry expires  [number]
+                --metadata      Arbitrary JSON that is associated with a key  [string]
                 --path          Read value from the file at a given path  [string]"
         `);
 				expect(std.err).toMatchInlineSnapshot(`
@@ -555,6 +582,7 @@ describe("wrangler", () => {
                 --preview       Interact with a preview namespace  [boolean]
                 --ttl           Time for which the entries should be visible  [number]
                 --expiration    Time since the UNIX epoch after which the entry expires  [number]
+                --metadata      Arbitrary JSON that is associated with a key  [string]
                 --path          Read value from the file at a given path  [string]"
         `);
 				expect(std.err).toMatchInlineSnapshot(`
@@ -593,6 +621,7 @@ describe("wrangler", () => {
                 --preview       Interact with a preview namespace  [boolean]
                 --ttl           Time for which the entries should be visible  [number]
                 --expiration    Time since the UNIX epoch after which the entry expires  [number]
+                --metadata      Arbitrary JSON that is associated with a key  [string]
                 --path          Read value from the file at a given path  [string]"
         `);
 				expect(std.err).toMatchInlineSnapshot(`
@@ -631,6 +660,7 @@ describe("wrangler", () => {
                 --preview       Interact with a preview namespace  [boolean]
                 --ttl           Time for which the entries should be visible  [number]
                 --expiration    Time since the UNIX epoch after which the entry expires  [number]
+                --metadata      Arbitrary JSON that is associated with a key  [string]
                 --path          Read value from the file at a given path  [string]"
         `);
 				expect(std.err).toMatchInlineSnapshot(`
@@ -669,6 +699,7 @@ describe("wrangler", () => {
                 --preview       Interact with a preview namespace  [boolean]
                 --ttl           Time for which the entries should be visible  [number]
                 --expiration    Time since the UNIX epoch after which the entry expires  [number]
+                --metadata      Arbitrary JSON that is associated with a key  [string]
                 --path          Read value from the file at a given path  [string]"
         `);
 				expect(std.err).toMatchInlineSnapshot(`