wrangler 2.1.13 → 2.1.15

package/README.md

@@ -82,7 +82,3 @@ For more commands and options, refer to the [documentation](https://developers.c
 ## Documentation
 
 For the latest Wrangler documentation, [click here](https://developers.cloudflare.com/workers/wrangler/).
-
-```
-
-```

package/miniflare-dist/index.mjs

@@ -746,10 +746,11 @@ import {
   DurableObjectStub
 } from "@miniflare/durable-objects";
 import {
-  Log,
+  Log as MiniflareLog,
+  LogLevel as MiniflareLogLevel,
   Miniflare,
-  Response as MiniflareResponse3,
-  Request as MiniflareRequest3
+  Request as MiniflareRequest3,
+  Response as MiniflareResponse3
 } from "miniflare";
 
 // ../../node_modules/yargs/lib/platform-shims/esm.mjs
@@ -6145,7 +6146,7 @@ var getRequestContextCheckOptions = async () => {
 };
 
 // src/miniflare-cli/index.ts
-var NoOpLog = class extends Log {
+var MiniflareNoOpLog = class extends MiniflareLog {
   log() {
   }
   error(message) {
@@ -6159,9 +6160,12 @@ async function main() {
     ...JSON.parse(args._[0] ?? "{}"),
     ...requestContextCheckOptions
   };
-  const logLevel = config.logLevel.toUpperCase();
-  config.log = config.logLevel === "none" ? new NoOpLog() : new Log(logLevel, config.logOptions);
-  if (logLevel === "DEBUG" || logLevel === "VERBOSE") {
+  let logLevelString = config.logLevel.toUpperCase();
+  if (logLevelString === "LOG")
+    logLevelString = "INFO";
+  const logLevel = MiniflareLogLevel[logLevelString];
+  config.log = logLevel === MiniflareLogLevel.NONE ? new MiniflareNoOpLog() : new MiniflareLog(logLevel, config.logOptions);
+  if (logLevel === MiniflareLogLevel.DEBUG) {
     console.log("MINIFLARE OPTIONS:\n", JSON.stringify(config, null, 2));
   }
   config.bindings = {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wrangler",
-	"version": "2.1.13",
+	"version": "2.1.15",
 	"description": "Command-line interface for all things Cloudflare Workers",
 	"keywords": [
 		"wrangler",
@@ -160,6 +160,7 @@
 		"jest-fetch-mock": "^3.0.3",
 		"jest-websocket-mock": "^2.3.0",
 		"mime": "^3.0.0",
+		"minimatch": "^5.1.0",
 		"msw": "^0.47.1",
 		"npx-import": "^1.1.3",
 		"open": "^8.4.0",

package/src/__tests__/delete.test.ts

@@ -5,6 +5,7 @@ import { mockConfirm } from "./helpers/mock-dialogs";
 import { runInTempDir } from "./helpers/run-in-tmp";
 import { runWrangler } from "./helpers/run-wrangler";
 import writeWranglerToml from "./helpers/write-wrangler-toml";
+import type { KVNamespaceInfo } from "../kv/helpers";
 
 describe("delete", () => {
 	mockAccountId();
@@ -22,6 +23,7 @@ describe("delete", () => {
 			text: `Are you sure you want to delete my-script? This action cannot be undone.`,
 			result: true,
 		});
+		mockListKVNamespacesRequest();
 		mockDeleteWorkerRequest({ name: "my-script" });
 		await runWrangler("delete --name my-script");
 
@@ -41,6 +43,7 @@ describe("delete", () => {
 			result: true,
 		});
 		writeWranglerToml();
+		mockListKVNamespacesRequest();
 		mockDeleteWorkerRequest();
 		await runWrangler("delete");
 
@@ -84,6 +87,105 @@ describe("delete", () => {
 		}
 	`);
 	});
+
+	it("should delete a site namespace associated with a worker", async () => {
+		const kvNamespaces = [
+			{
+				title: "__my-script-workers_sites_assets",
+				id: "id-for-my-script-site-ns",
+			},
+			// this one isn't associated with the worker
+			{
+				title: "__test-name-workers_sites_assets",
+				id: "id-for-another-site-ns",
+			},
+		];
+
+		mockConfirm({
+			text: `Are you sure you want to delete my-script? This action cannot be undone.`,
+			result: true,
+		});
+		mockListKVNamespacesRequest(...kvNamespaces);
+		// it should only try to delete the site namespace associated with this worker
+		setMockResponse(
+			"/accounts/:accountId/storage/kv/namespaces/id-for-my-script-site-ns",
+			"DELETE",
+			([_url, accountId]) => {
+				expect(accountId).toEqual("some-account-id");
+				return null;
+			}
+		);
+		mockDeleteWorkerRequest({ name: "my-script" });
+		await runWrangler("delete --name my-script");
+		expect(std).toMatchInlineSnapshot(`
+		Object {
+		  "debug": "",
+		  "err": "",
+		  "out": "🌀 Deleted asset namespace for Workers Site \\"__my-script-workers_sites_assets\\"
+		Successfully deleted my-script",
+		  "warn": "",
+		}
+	`);
+	});
+
+	it("should delete a site namespace associated with a worker, including it's preview namespace", async () => {
+		// This is the same test as the previous one, but it includes a preview namespace
+		const kvNamespaces = [
+			{
+				title: "__my-script-workers_sites_assets",
+				id: "id-for-my-script-site-ns",
+			},
+			// this is the preview namespace
+			{
+				title: "__my-script-workers_sites_assets_preview",
+				id: "id-for-my-script-site-preview-ns",
+			},
+
+			// this one isn't associated with the worker
+			{
+				title: "__test-name-workers_sites_assets",
+				id: "id-for-another-site-ns",
+			},
+		];
+
+		mockConfirm({
+			text: `Are you sure you want to delete my-script? This action cannot be undone.`,
+			result: true,
+		});
+		mockListKVNamespacesRequest(...kvNamespaces);
+		// it should only try to delete the site namespace associated with this worker
+
+		setMockResponse(
+			"/accounts/:accountId/storage/kv/namespaces/id-for-my-script-site-ns",
+			"DELETE",
+			([_url, accountId]) => {
+				expect(accountId).toEqual("some-account-id");
+				return null;
+			}
+		);
+
+		setMockResponse(
+			"/accounts/:accountId/storage/kv/namespaces/id-for-my-script-site-preview-ns",
+			"DELETE",
+			([_url, accountId]) => {
+				expect(accountId).toEqual("some-account-id");
+				return null;
+			}
+		);
+
+		mockDeleteWorkerRequest({ name: "my-script" });
+		await runWrangler("delete --name my-script");
+		expect(std).toMatchInlineSnapshot(`
+		Object {
+		  "debug": "",
+		  "err": "",
+		  "out": "🌀 Deleted asset namespace for Workers Site \\"__my-script-workers_sites_assets\\"
+		🌀 Deleted asset namespace for Workers Site \\"__my-script-workers_sites_assets_preview\\"
+		Successfully deleted my-script",
+		  "warn": "",
+		}
+	`);
+	});
 });
 
 /** Create a mock handler for the request to upload a worker script. */
@@ -114,3 +216,15 @@ function mockDeleteWorkerRequest(
 		}
 	);
 }
+
+/** Create a mock handler for the request to get a list of all KV namespaces. */
+function mockListKVNamespacesRequest(...namespaces: KVNamespaceInfo[]) {
+	setMockResponse(
+		"/accounts/:accountId/storage/kv/namespaces",
+		"GET",
+		([_url, accountId]) => {
+			expect(accountId).toEqual("some-account-id");
+			return namespaces;
+		}
+	);
+}

package/src/__tests__/deployments.test.ts

@@ -0,0 +1,91 @@
+// import * as fs from "fs";
+// import * as TOML from "@iarna/toml";
+import { rest } from "msw";
+import { mockAccountId, mockApiToken } from "./helpers/mock-account-id";
+import { mockConsoleMethods } from "./helpers/mock-console";
+import {
+	msw,
+	mswSuccessOauthHandlers,
+	mswSuccessUserHandlers,
+} from "./helpers/msw";
+import { mswSuccessDeployments } from "./helpers/msw";
+import { runInTempDir } from "./helpers/run-in-tmp";
+import { runWrangler } from "./helpers/run-wrangler";
+
+describe("deployments", () => {
+	const std = mockConsoleMethods();
+	runInTempDir();
+	mockAccountId();
+	mockApiToken();
+	runInTempDir();
+
+	beforeEach(() => {
+		msw.use(
+			...mswSuccessDeployments,
+			...mswSuccessOauthHandlers,
+			...mswSuccessUserHandlers,
+			rest.get(
+				"*/accounts/:accountId/workers/services/:scriptName",
+				(request, response, context) => {
+					expect(["undefined", "somethingElse"]).toContain(
+						request.params.scriptName
+					);
+
+					return response.once(
+						context.status(200),
+						context.json({
+							success: true,
+							errors: [],
+							messages: [],
+							result: {
+								default_environment: {
+									script: {
+										tag: "MOCK-TAG",
+									},
+								},
+							},
+						})
+					);
+				}
+			)
+		);
+	});
+
+	it("should log deployments", async () => {
+		await runWrangler("deployments");
+		expect(std.out).toMatchInlineSnapshot(`
+		"
+		Version ID: Galaxy-Class
+		Version number: 1701-E
+		Created on: 2021-01-01T00:00:00.000000Z
+		Author email: Jean-Luc-Picard@federation.org
+		Latest deploy: true
+
+		Version ID: Intrepid-Class
+		Version number: NCC-74656
+		Created on: 2021-02-02T00:00:00.000000Z
+		Author email: Kathryn-Janeway@federation.org
+		Latest deploy: false
+		"
+	`);
+	});
+
+	it("should log deployments for script with passed in name option", async () => {
+		await runWrangler("deployments --name somethingElse");
+		expect(std.out).toMatchInlineSnapshot(`
+		"
+		Version ID: Galaxy-Class
+		Version number: 1701-E
+		Created on: 2021-01-01T00:00:00.000000Z
+		Author email: Jean-Luc-Picard@federation.org
+		Latest deploy: true
+
+		Version ID: Intrepid-Class
+		Version number: NCC-74656
+		Created on: 2021-02-02T00:00:00.000000Z
+		Author email: Kathryn-Janeway@federation.org
+		Latest deploy: false
+		"
+	`);
+	});
+});

package/src/__tests__/helpers/msw/handlers/deployments.ts

@@ -0,0 +1,80 @@
+import { rest } from "msw";
+
+export type DeploymentListRes = {
+	versions: {
+		version_id: string;
+		version_number: string;
+		metadata: {
+			author_id: string;
+			author_email: string;
+			source: "api" | "dash" | "wrangler" | "terraform" | "other";
+			created_on: string;
+			modified_on: string;
+		};
+		preview: {
+			active: boolean;
+			url: string;
+		};
+		resources: {
+			script: string;
+			bindings: unknown[];
+		};
+	}[];
+};
+
+export const mswSuccessDeployments = [
+	rest.get(
+		"*/accounts/:accountId/workers/versions/by-script/:scriptTag",
+		(_, response, context) =>
+			response.once(
+				context.status(200),
+				context.json({
+					success: true,
+					errors: [],
+					messages: [],
+					result: {
+						versions: [
+							{
+								version_id: "Galaxy-Class",
+								version_number: "1701-E",
+								metadata: {
+									author_id: "Picard-Gamma-6-0-7-3",
+									author_email: "Jean-Luc-Picard@federation.org",
+									source: "wrangler",
+									created_on: "2021-01-01T00:00:00.000000Z",
+									modified_on: "2021-01-01T00:00:00.000000Z",
+								},
+								preview: {
+									active: true,
+									url: "https://example.com",
+								},
+								resources: {
+									script: "script.js",
+									bindings: [],
+								},
+							},
+							{
+								version_id: "Intrepid-Class",
+								version_number: "NCC-74656",
+								metadata: {
+									author_id: "Kathryn-Jane-Gamma-6-0-7-3",
+									author_email: "Kathryn-Janeway@federation.org",
+									source: "wrangler",
+									created_on: "2021-02-02T00:00:00.000000Z",
+									modified_on: "2021-02-02T00:00:00.000000Z",
+								},
+								preview: {
+									active: true,
+									url: "https://example.com",
+								},
+								resources: {
+									script: "script.js",
+									bindings: [],
+								},
+							},
+						],
+					},
+				})
+			)
+	),
+];

package/src/__tests__/helpers/msw/index.ts

@@ -1,8 +1,9 @@
 import { setupServer } from "msw/node";
+import { mswSuccessDeployments } from "./handlers/deployments";
 import { mswSuccessNamespacesHandlers } from "./handlers/namespaces";
 import { mswSuccessOauthHandlers } from "./handlers/oauth";
 import { mswSuccessR2handlers } from "./handlers/r2";
-import { default as mswScriptHandlers } from "./handlers/script";
+import { default as mswSucessScriptHandlers } from "./handlers/script";
 import { mswSuccessUserHandlers } from "./handlers/user";
 export const msw = setupServer();
 
@@ -11,5 +12,6 @@ export {
 	mswSuccessR2handlers,
 	mswSuccessOauthHandlers,
 	mswSuccessNamespacesHandlers,
-	mswScriptHandlers,
+	mswSucessScriptHandlers,
+	mswSuccessDeployments,
 };

package/src/__tests__/index.test.ts

@@ -34,7 +34,7 @@ describe("wrangler", () => {
 			  wrangler init [name]         📥 Create a wrangler.toml configuration file
 			  wrangler dev [script]        👂 Start a local server for developing your worker
 			  wrangler publish [script]    🆙 Publish your Worker to Cloudflare.
-			  wrangler delete [script]     🗑 Delete your Worker from Cloudflare.
+			  wrangler delete [script]     🗑  Delete your Worker from Cloudflare.
 			  wrangler tail [worker]       🦚 Starts a log tailing session for a published Worker.
 			  wrangler secret              🤫 Generate a secret that can be referenced in a Worker
 			  wrangler secret:bulk <json>  🗄️  Bulk upload secrets for a Worker
@@ -49,6 +49,7 @@ describe("wrangler", () => {
 			  wrangler login               🔓 Login to Cloudflare
 			  wrangler logout              🚪 Logout from Cloudflare
 			  wrangler whoami              🕵️  Retrieve your user info and test your auth config
+			  wrangler types               📝 Generate types from bindings & module rules in config
 
 			Flags:
 			  -c, --config   Path to .toml configuration file  [string]
@@ -76,7 +77,7 @@ describe("wrangler", () => {
 			  wrangler init [name]         📥 Create a wrangler.toml configuration file
 			  wrangler dev [script]        👂 Start a local server for developing your worker
 			  wrangler publish [script]    🆙 Publish your Worker to Cloudflare.
-			  wrangler delete [script]     🗑 Delete your Worker from Cloudflare.
+			  wrangler delete [script]     🗑  Delete your Worker from Cloudflare.
 			  wrangler tail [worker]       🦚 Starts a log tailing session for a published Worker.
 			  wrangler secret              🤫 Generate a secret that can be referenced in a Worker
 			  wrangler secret:bulk <json>  🗄️  Bulk upload secrets for a Worker
@@ -91,6 +92,7 @@ describe("wrangler", () => {
 			  wrangler login               🔓 Login to Cloudflare
 			  wrangler logout              🚪 Logout from Cloudflare
 			  wrangler whoami              🕵️  Retrieve your user info and test your auth config
+			  wrangler types               📝 Generate types from bindings & module rules in config
 
 			Flags:
 			  -c, --config   Path to .toml configuration file  [string]

package/src/__tests__/init.test.ts

@@ -1306,7 +1306,7 @@ describe("init", () => {
 						contents: {
 							config: {
 								compilerOptions: expect.objectContaining({
-									types: ["@cloudflare/workers-types"],
+									types: ["@cloudflare/workers-types", "jest"],
 								}),
 							},
 							error: undefined,

package/src/__tests__/pages.test.ts

@@ -1905,6 +1905,116 @@ and that at least one include rule is provided.
 		      `);
 		});
 
+		it("should avoid uploading some files", async () => {
+			mkdirSync("some_dir/node_modules", { recursive: true });
+			mkdirSync("some_dir/functions", { recursive: true });
+
+			writeFileSync("logo.png", "foobar");
+			writeFileSync("some_dir/functions/foo.js", "func");
+			writeFileSync("some_dir/_headers", "headersfile");
+
+			writeFileSync("_headers", "headersfile");
+			writeFileSync("_redirects", "redirectsfile");
+			writeFileSync("_worker.js", "workerfile");
+			writeFileSync("_routes.json", "routesfile");
+			mkdirSync(".git");
+			writeFileSync(".git/foo", "gitfile");
+			writeFileSync("some_dir/node_modules/some_package", "nodefile");
+			mkdirSync("functions");
+			writeFileSync("functions/foo.js", "func");
+
+			setMockResponse(
+				"/pages/assets/check-missing",
+				"POST",
+				async (_, init) => {
+					const body = JSON.parse(init.body as string) as { hashes: string[] };
+					assertLater(() => {
+						expect(init.headers).toMatchObject({
+							Authorization: "Bearer <<funfetti-auth-jwt>>",
+						});
+						expect(body).toMatchObject({
+							hashes: [
+								"2082190357cfd3617ccfe04f340c6247",
+								"95dedb64e6d4940fc2e0f11f711cc2f4",
+								"09a79777abda8ccc8bdd51dd3ff8e9e9",
+							],
+						});
+					});
+					return body.hashes;
+				}
+			);
+
+			// Accumulate multiple requests then assert afterwards
+			const requests: RequestInit[] = [];
+			setMockRawResponse("/pages/assets/upload", "POST", async (_, init) => {
+				requests.push(init);
+
+				return createFetchResult(null, true);
+			});
+
+			assertLater(() => {
+				expect(requests.length).toBe(3);
+
+				expect(requests[0].headers).toMatchObject({
+					Authorization: "Bearer <<funfetti-auth-jwt>>",
+				});
+
+				let body = JSON.parse(
+					requests[0].body as string
+				) as UploadPayloadFile[];
+				expect(body).toMatchObject([
+					{
+						key: "95dedb64e6d4940fc2e0f11f711cc2f4",
+						value: Buffer.from("headersfile").toString("base64"),
+						metadata: {
+							contentType: "application/octet-stream",
+						},
+						base64: true,
+					},
+				]);
+
+				expect(requests[1].headers).toMatchObject({
+					Authorization: "Bearer <<funfetti-auth-jwt>>",
+				});
+
+				body = JSON.parse(requests[1].body as string) as UploadPayloadFile[];
+				expect(body).toMatchObject([
+					{
+						key: "2082190357cfd3617ccfe04f340c6247",
+						value: Buffer.from("foobar").toString("base64"),
+						metadata: {
+							contentType: "image/png",
+						},
+						base64: true,
+					},
+				]);
+
+				expect(requests[2].headers).toMatchObject({
+					Authorization: "Bearer <<funfetti-auth-jwt>>",
+				});
+
+				body = JSON.parse(requests[2].body as string) as UploadPayloadFile[];
+				expect(body).toMatchObject([
+					{
+						key: "09a79777abda8ccc8bdd51dd3ff8e9e9",
+						value: Buffer.from("func").toString("base64"),
+						metadata: {
+							contentType: "application/javascript",
+						},
+						base64: true,
+					},
+				]);
+			});
+
+			await runWrangler("pages project upload .");
+
+			expect(std.out).toMatchInlineSnapshot(`
+			        "✨ Success! Uploaded 3 files (TIMINGS)
+
+			        ✨ Upload complete!"
+		      `);
+		});
+
 		it("should retry uploads", async () => {
 			writeFileSync("logo.txt", "foobar");
 

package/src/__tests__/tail.test.ts

@@ -6,7 +6,7 @@ import { mockConsoleMethods } from "./helpers/mock-console";
 import { mockGetZoneFromHostRequest } from "./helpers/mock-get-zone-from-host";
 import { useMockIsTTY } from "./helpers/mock-istty";
 import { mockCollectKnownRoutesRequest } from "./helpers/mock-known-routes";
-import { msw, mswScriptHandlers } from "./helpers/msw";
+import { msw, mswSucessScriptHandlers } from "./helpers/msw";
 import { runInTempDir } from "./helpers/run-in-tmp";
 import { runWrangler } from "./helpers/run-wrangler";
 import type {
@@ -18,7 +18,7 @@ import type {
 import type { RequestInit } from "undici";
 import type WebSocket from "ws";
 describe("tail", () => {
-	beforeEach(() => msw.use(...mswScriptHandlers));
+	beforeEach(() => msw.use(...mswSucessScriptHandlers));
 	runInTempDir();
 	mockAccountId();
 	mockApiToken();