fix(wrangler) wrangler kv:key put does not work as expected

.changeset/purple-cars-run.md

@@ -0,0 +1,7 @@
+---
+"wrangler": patch
+---
+
+fix: correctly handle non-text based files for kv put
+
+The current version of the kv:key put command with the --path argument will treat file contents as a string because it is not one of Blob or File when passed to the form helper library. We should turn it into a Blob so it's not mangling inputs.

packages/wrangler/src/__tests__/kv.test.ts

@@ -505,6 +505,29 @@ describe("wrangler", () => {
 				expect(std.err).toMatchInlineSnapshot(`""`);
 			});
 
+			it("should put a key with a binary value and metadata", async () => {
+				const buf = Buffer.from(
+					"iVBORw0KGgoAAAANSUhEUgAAAAoAAAAKCAYAAACNMs+9AAAACXBIWXMAAAsTAAALEwEAmpwYAAAAAXNSR0IArs4c6QAAAARnQU1BAACxjwv8YQUAAAAiSURBVHgB7coxEQAACMPAgH/PgAM6dGwu49fA/deIBXrgAj2cAhIFT4QxAAAAAElFTkSuQmCC",
+					"base64"
+				);
+				writeFileSync("test.png", buf);
+				const requests = mockKeyPutRequest("some-namespace-id", {
+					key: "dKey",
+					value: buf,
+					metadata: {
+						mKey: "mValue",
+					},
+				});
+				await runWrangler(
+					`"kv:key put my-key --namespace-id another-namespace-id --path test.png" '{"mKey":"mValue"}'`
+				);
+				expect(requests.count).toEqual(1);
+				expect(std.out).toMatchInlineSnapshot(
+					`"Writing the contents of test.png to the key \\"my-key\\" on namespace another-namespace-id."`
+				);
+				expect(std.err).toMatchInlineSnapshot(`""`);
+			});
+
 			it("should error if no key is provided", async () => {
 				await expect(
 					runWrangler("kv:key put")

packages/wrangler/src/kv/helpers.ts

@@ -199,7 +199,10 @@ export function unexpectedKVKeyValueProps(keyValue: KeyValue): string[] {
 function asFormData(fields: Record<string, unknown>): FormData {
 	const formData = new FormData();
 
-	for (const [name, value] of Object.entries(fields)) {
+	for (let [name, value] of Object.entries(fields)) {
+		if (Buffer.isBuffer(value)) {
+			value = new Blob([value]);
+		}
 		formData.append(name, value);
 	}