wrangler 0.0.0-e6733a3 → 0.0.0-edc4b53

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wrangler",
-  "version": "0.0.0-e6733a3",
+  "version": "0.0.0-edc4b53",
   "author": "wrangler@cloudflare.com",
   "description": "Command-line interface for all things Cloudflare Workers",
   "bin": {

package/src/__tests__/kv.test.ts

@@ -335,8 +335,16 @@ describe("wrangler", () => {
             expect(namespaceId).toEqual(expectedNamespaceId);
             expect(key).toEqual(expectedKey);
             expect(body).toEqual(expectedValue);
-            expect(query.get("expiration")).toEqual(`${expiration}`);
-            expect(query.get("expiration_ttl")).toEqual(`${expirationTtl}`);
+            if (expiration !== undefined) {
+              expect(query.get("expiration")).toEqual(`${expiration}`);
+            } else {
+              expect(query.has("expiration")).toBe(false);
+            }
+            if (expirationTtl) {
+              expect(query.get("expiration_ttl")).toEqual(`${expirationTtl}`);
+            } else {
+              expect(query.has("expiration_ttl")).toBe(false);
+            }
             return null;
           }
         );

package/src/api/form_data.ts

@@ -23,7 +23,7 @@ export function toMimeType(type: CfModuleType): string {
   }
 }
 
-function toModule(module: CfModule, entryType?: CfModuleType): Blob {
+function toModule(module: CfModule, entryType: CfModuleType): Blob {
   const { type: moduleType, content } = module;
   const type = toMimeType(moduleType ?? entryType);
 

package/src/cfetch/internal.ts

@@ -1,5 +1,5 @@
-import fetch from "node-fetch";
-import type { RequestInit, HeadersInit } from "node-fetch";
+import fetch, { Headers } from "node-fetch";
+import type { RequestInit } from "node-fetch";
 import { getAPIToken, loginOrRefreshIfRequired } from "../user";
 
 export const CF_API_BASE_URL =
@@ -21,7 +21,7 @@ export async function fetchInternal<ResponseType>(
 ): Promise<ResponseType> {
   await requireLoggedIn();
   const apiToken = requireApiToken();
-  const headers = cloneHeaders(init.headers);
+  const headers = new Headers(init.headers);
   addAuthorizationHeader(headers, apiToken);
 
   const queryString = queryParams ? `?${queryParams.toString()}` : "";
@@ -55,11 +55,7 @@ function requireApiToken(): string {
   return apiToken;
 }
 
-function cloneHeaders(headers: HeadersInit): HeadersInit {
-  return { ...headers };
-}
-
-function addAuthorizationHeader(headers: HeadersInit, apiToken: string): void {
+function addAuthorizationHeader(headers: Headers, apiToken: string): void {
   if (headers["Authorization"]) {
     throw new Error(
       "The request already specifies an authorisation header - cannot add a new one."

package/src/kv.tsx

@@ -82,7 +82,7 @@ export interface NamespaceKeyInfo {
 export async function listNamespaceKeys(
   accountId: string,
   namespaceId: string,
-  prefix?: string
+  prefix = ""
 ) {
   return await fetchListResult<NamespaceKeyInfo>(
     `/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/keys`,
@@ -98,15 +98,20 @@ export async function putKeyValue(
   value: string,
   args?: { expiration?: number; expiration_ttl?: number }
 ) {
+  let searchParams: URLSearchParams | undefined;
+  if (args) {
+    searchParams = new URLSearchParams();
+    if (args.expiration) {
+      searchParams.set("expiration", `${args.expiration}`);
+    }
+    if (args.expiration_ttl) {
+      searchParams.set("expiration_ttl", `${args.expiration_ttl}`);
+    }
+  }
   return await fetchResult(
     `/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/values/${key}`,
     { method: "PUT", body: value },
-    args
-      ? new URLSearchParams({
-          expiration: args.expiration?.toString(),
-          expiration_ttl: args.expiration_ttl?.toString(),
-        })
-      : undefined
+    searchParams
   );
 }
 
@@ -262,6 +267,10 @@ export function getNamespaceId({
 /**
  * KV namespace binding names must be valid JS identifiers.
  */
-export function isValidNamespaceBinding(binding: string): boolean {
-  return /^[a-zA-Z_][a-zA-Z0-9_]*$/.test(binding);
+export function isValidNamespaceBinding(
+  binding: string | undefined
+): binding is string {
+  return (
+    typeof binding === "string" && /^[a-zA-Z_][a-zA-Z0-9_]*$/.test(binding)
+  );
 }

package/src/sites.tsx

@@ -89,7 +89,11 @@ export async function syncAssets(
   );
 
   const manifest = {};
-  const upload = [];
+  const upload: {
+    key: string;
+    value: string;
+    base64: boolean;
+  }[] = [];
   // TODO: this can be more efficient by parallelising
   for await (const file of getFilesInFolder(dirPath)) {
     // TODO: "exclude:" config

package/src/user.tsx

@@ -298,9 +298,9 @@ let initialised = false;
 
 // we do this because we have some async stuff
 // TODO: this should just happen in the top level
-// abd we should fiure out how to do top level await
+// and we should figure out how to do top level await
 export async function initialise(): Promise<void> {
-  // get refreshtoken/accesstoken from fs if exists
+  // get refreshToken/accessToken from fs if exists
   try {
     // if CF_API_TOKEN available, use that
     if (process.env.CF_API_TOKEN) {
@@ -350,7 +350,9 @@ export function getAPIToken(): string {
   }
 
   throwIfNotInitialised();
-  return LocalState.accessToken?.value;
+  // `throwIfNotInitialised()` ensures that the accessToken is guaranteed to be defined.
+  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+  return LocalState.accessToken!.value;
 }
 
 interface AccessContext {
@@ -971,14 +973,14 @@ export async function getAccountId() {
     });
   } catch (err) {
     // probably offline
+    return;
   }
-  if (!response) return;
-  let accountId: string;
-  // @ts-expect-error need to type this response
-  const responseJSON: {
+
+  let accountId: string | undefined;
+  const responseJSON = (await response.json()) as {
     success: boolean;
     result: { id: string; account: { id: string; name: string } }[];
-  } = await response.json();
+  };
 
   if (responseJSON.success === true) {
     if (responseJSON.result.length === 1) {

package/wrangler-dist/cli.js

@@ -106633,7 +106633,7 @@ var import_http = __toModule(require("http"));
 var import_react = __toModule(require_react());
 
 // package.json
-var version = "0.0.0-e6733a3";
+var version = "0.0.0-edc4b53";
 
 // src/inspect.ts
 function useInspector(props) {
@@ -108103,7 +108103,7 @@ function getAPIToken() {
     return import_node_process.default.env.CF_API_TOKEN;
   }
   throwIfNotInitialised();
-  return LocalState.accessToken?.value;
+  return LocalState.accessToken.value;
 }
 var ErrorOAuth2 = class extends Error {
   toString() {
@@ -108522,9 +108522,8 @@ async function getAccountId() {
       }
     });
   } catch (err2) {
-  }
-  if (!response)
     return;
+  }
   let accountId;
   const responseJSON = await response.json();
   if (responseJSON.success === true) {
@@ -108563,7 +108562,7 @@ var CF_API_BASE_URL = process.env.CF_API_BASE_URL || "https://api.cloudflare.com
 async function fetchInternal(resource, init = {}, queryParams) {
   await requireLoggedIn();
   const apiToken = requireApiToken();
-  const headers = cloneHeaders(init.headers);
+  const headers = new Headers(init.headers);
   addAuthorizationHeader(headers, apiToken);
   const queryString = queryParams ? `?${queryParams.toString()}` : "";
   const response = await fetch(`${CF_API_BASE_URL}${resource}${queryString}`, {
@@ -108590,9 +108589,6 @@ function requireApiToken() {
   }
   return apiToken;
 }
-function cloneHeaders(headers) {
-  return { ...headers };
-}
 function addAuthorizationHeader(headers, apiToken) {
   if (headers["Authorization"]) {
     throw new Error("The request already specifies an authorisation header - cannot add a new one.");
@@ -111093,14 +111089,21 @@ async function listNamespaces(accountId) {
   }
   return results;
 }
-async function listNamespaceKeys(accountId, namespaceId, prefix2) {
+async function listNamespaceKeys(accountId, namespaceId, prefix2 = "") {
   return await fetchListResult(`/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/keys`, {}, new import_node_url3.URLSearchParams({ prefix: prefix2 }));
 }
 async function putKeyValue(accountId, namespaceId, key2, value, args2) {
-  return await fetchResult(`/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/values/${key2}`, { method: "PUT", body: value }, args2 ? new import_node_url3.URLSearchParams({
-    expiration: args2.expiration?.toString(),
-    expiration_ttl: args2.expiration_ttl?.toString()
-  }) : void 0);
+  let searchParams;
+  if (args2) {
+    searchParams = new import_node_url3.URLSearchParams();
+    if (args2.expiration) {
+      searchParams.set("expiration", `${args2.expiration}`);
+    }
+    if (args2.expiration_ttl) {
+      searchParams.set("expiration_ttl", `${args2.expiration_ttl}`);
+    }
+  }
+  return await fetchResult(`/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/values/${key2}`, { method: "PUT", body: value }, searchParams);
 }
 async function putBulkKeyValue(accountId, namespaceId, keyvalueStr) {
   return await fetchResult(`/accounts/${accountId}/storage/kv/namespaces/${namespaceId}/bulk`, {
@@ -111182,7 +111185,7 @@ function getNamespaceId({
   return namespaceId;
 }
 function isValidNamespaceBinding(binding) {
-  return /^[a-zA-Z_][a-zA-Z0-9_]*$/.test(binding);
+  return typeof binding === "string" && /^[a-zA-Z_][a-zA-Z0-9_]*$/.test(binding);
 }
 
 // src/sites.tsx