wrangler 4.8.0 → 4.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wrangler",
-  "version": "4.8.0",
+  "version": "4.9.0",
   "description": "Command-line interface for all things Cloudflare Workers",
   "keywords": [
     "wrangler",
@@ -57,7 +57,7 @@
     "unenv": "2.0.0-rc.15",
     "workerd": "1.20250405.0",
     "@cloudflare/kv-asset-handler": "0.4.0",
-    "miniflare": "4.20250405.0"
+    "miniflare": "4.20250405.1"
   },
   "devDependencies": {
     "@aws-sdk/client-s3": "^3.721.0",
@@ -137,7 +137,7 @@
     "yargs": "^17.7.2",
     "@cloudflare/cli": "1.1.1",
     "@cloudflare/eslint-config-worker": "1.1.0",
-    "@cloudflare/pages-shared": "^0.13.23",
+    "@cloudflare/pages-shared": "^0.13.24",
     "@cloudflare/workers-shared": "0.17.1",
     "@cloudflare/workers-tsconfig": "0.0.0"
   },

package/wrangler-dist/cli.d.ts

@@ -708,11 +708,13 @@ declare interface CfSecretsStoreSecrets {
 /**
  * A binding to send email.
  */
-declare interface CfSendEmailBindings {
+declare type CfSendEmailBindings = {
     name: string;
+} & ({
     destination_address?: string;
+} | {
     allowed_destination_addresses?: string[];
-}
+});
 
 declare interface CfService {
     binding: string;

package/wrangler-dist/cli.js

@@ -81185,7 +81185,7 @@ var import_undici3 = __toESM(require_undici());
 
 // package.json
 var name = "wrangler";
-var version = "4.8.0";
+var version = "4.9.0";
 
 // src/environment-variables/misc-variables.ts
 init_import_meta_url();
@@ -83958,16 +83958,17 @@ function printBindings(bindings, context2 = {}) {
   if (send_email !== void 0 && send_email.length > 0) {
     output.push({
       name: friendlyBindingNames.send_email,
-      entries: send_email.map(
-        ({ name: name2, destination_address, allowed_destination_addresses }) => {
-          return {
-            key: name2,
-            value: addSuffix(
-              destination_address || allowed_destination_addresses?.join(", ") || "unrestricted"
-            )
-          };
-        }
-      )
+      entries: send_email.map((emailBinding) => {
+        const destination_address = "destination_address" in emailBinding ? emailBinding.destination_address : void 0;
+        const allowed_destination_addresses = "allowed_destination_addresses" in emailBinding ? emailBinding.allowed_destination_addresses : void 0;
+        return {
+          key: emailBinding.name,
+          value: addSuffix(
+            destination_address || allowed_destination_addresses?.join(", ") || "unrestricted",
+            { isSimulatedLocally: true }
+          )
+        };
+      })
     });
   }
   if (queues2 !== void 0 && queues2.length > 0) {
@@ -84772,7 +84773,14 @@ function normalizeAndValidateConfig(rawConfig, configPath, userConfigPath, args)
   const definedEnvironments = Object.keys(rawConfig.env ?? {});
   if (isRedirectedConfig && definedEnvironments.length > 0) {
     diagnostics.errors.push(
-      `Redirected configurations cannot include environments but the following have been found: ${definedEnvironments.map((env6) => JSON.stringify(env6)).join(", ")}`
+      dedent`
+				Redirected configurations cannot include environments but the following have been found:\n${definedEnvironments.map((env6) => `	- ${env6}`).join("\n")}
+
+
+				Such configurations are generated by tools, meaning that one of the tools
+				your application is using is generating the incorrect configuration.
+				Report this issue to the tool's author so that this can be fixed there.
+				`
     );
   }
   const envName = args.env;
@@ -92596,11 +92604,10 @@ function castLogLevel(level) {
 }
 __name(castLogLevel, "castLogLevel");
 function buildLog() {
-  let level = castLogLevel(logger.loggerLevel);
-  if (level <= import_miniflare6.LogLevel.DEBUG) {
-    level = Math.min(level, import_miniflare6.LogLevel.WARN);
-  }
-  return new WranglerLog(level, { prefix: "wrangler-UserWorker" });
+  const level = castLogLevel(logger.loggerLevel);
+  return new WranglerLog(level, {
+    prefix: level === import_miniflare6.LogLevel.DEBUG ? "wrangler-UserWorker" : "wrangler"
+  });
 }
 __name(buildLog, "buildLog");
 async function buildSourceOptions(config) {
@@ -92938,6 +92945,9 @@ function buildMiniflareBindingOptions(config) {
         binding
       ]) ?? []
     ),
+    email: {
+      send_email: bindings.send_email
+    },
     durableObjects: Object.fromEntries([
       ...internalObjects.map(({ name: name2, class_name }) => {
         const useSQLite = classNameToUseSQLite.get(class_name);
@@ -93157,6 +93167,14 @@ async function buildMiniflareOptions(log2, config, proxyToUserWorkerAuthenticati
     liveReload: config.liveReload,
     upstream,
     unsafeProxySharedSecret: proxyToUserWorkerAuthenticationSecret,
+    unsafeTriggerHandlers: true,
+    // The way we run Miniflare instances with wrangler dev is that there are two:
+    //  - one holding the proxy worker,
+    //  - and one holding the user worker.
+    // The issue with that setup is that end users would see two sets of request logs from Miniflare!
+    // Instead of hiding all logs from this Miniflare instance, we specifically hide the request logs,
+    // allowing other logs to be shown to the user (such as details about emails being triggered)
+    logRequests: false,
     log: log2,
     verbose: logger.loggerLevel === "debug",
     handleRuntimeStdio,
@@ -103929,16 +103947,16 @@ function createWorkerUploadForm(worker) {
       });
     }
   });
-  bindings.send_email?.forEach(
-    ({ name: name2, destination_address, allowed_destination_addresses }) => {
-      metadataBindings.push({
-        name: name2,
-        type: "send_email",
-        destination_address,
-        allowed_destination_addresses
-      });
-    }
-  );
+  bindings.send_email?.forEach((emailBinding) => {
+    const destination_address = "destination_address" in emailBinding ? emailBinding.destination_address : void 0;
+    const allowed_destination_addresses = "allowed_destination_addresses" in emailBinding ? emailBinding.allowed_destination_addresses : void 0;
+    metadataBindings.push({
+      name: emailBinding.name,
+      type: "send_email",
+      destination_address,
+      allowed_destination_addresses
+    });
+  });
   bindings.durable_objects?.bindings.forEach(
     ({ name: name2, class_name, script_name, environment }) => {
       metadataBindings.push({
@@ -129045,6 +129063,14 @@ function validateCorsOrigins(values) {
   if (!values || !values.length) {
     return values;
   }
+  if (values.includes("none")) {
+    if (values.length > 1) {
+      throw new UserError(
+        "When specifying 'none', only one value is permitted."
+      );
+    }
+    return [];
+  }
   if (values.includes("*")) {
     if (values.length > 1) {
       throw new UserError("When specifying '*', only one value is permitted.");
@@ -129160,7 +129186,7 @@ function addCreateOptions(yargs) {
     demandOption: false
   }).group(["shard-count"], `${source_default.bold("Pipeline settings")}`).option("shard-count", {
     type: "number",
-    describe: "Number of pipeline shards. More shards handle higher request volume; fewer shards produce larger output files. Defaults to 2 if unset. Minimum: 1, Maximum: 15",
+    describe: "Number of shards for the pipeline. More shards handle higher request volume; fewer shards produce larger output files. Defaults to 2 if unset. Minimum: 1, Maximum: 15",
     demandOption: false
   });
 }
@@ -129254,14 +129280,14 @@ async function createPipelineHandler(args) {
   if (args.shardCount) {
     pipelineConfig.metadata.shards = args.shardCount;
   }
-  logger.log(`\u{1F300} Creating Pipeline named "${name2}"`);
+  logger.log(`\u{1F300} Creating pipeline named "${name2}"`);
   const pipeline = await createPipeline(accountId, pipelineConfig);
   logger.log(
-    `\u2705 Successfully created Pipeline "${pipeline.name}" with ID ${pipeline.id}
+    `\u2705 Successfully created pipeline "${pipeline.name}" with ID ${pipeline.id}
 `
   );
   logger.log(formatPipelinePretty(pipeline));
-  logger.log("\u{1F389} You can now send data to your Pipeline!");
+  logger.log("\u{1F389} You can now send data to your pipeline!");
   if (args.source.includes("worker")) {
     logger.log(
       `
@@ -129284,7 +129310,7 @@ To send data to your pipeline from a Worker, add the following to your wrangler
   }
   if (args.source.includes("http")) {
     logger.log(`
-Send data to your Pipeline's HTTP endpoint:
+Send data to your pipeline's HTTP endpoint:
 `);
     logger.log(`curl "${pipeline.endpoint}" -d '[{"foo": "bar"}]'
 `);
@@ -129297,7 +129323,7 @@ init_import_meta_url();
 function addDeleteOptions(yargs) {
   return yargs.positional("pipeline", {
     type: "string",
-    describe: "The name of the Pipeline to show",
+    describe: "The name of the pipeline to delete",
     demandOption: true
   });
 }
@@ -129308,9 +129334,9 @@ async function deletePipelineHandler(args) {
   const accountId = await requireAuth(config);
   const name2 = args.pipeline;
   validateName("pipeline name", name2);
-  logger.log(`Deleting Pipeline ${name2}.`);
+  logger.log(`Deleting pipeline ${name2}.`);
   await deletePipeline(accountId, name2);
-  logger.log(`Deleted Pipeline ${name2}.`);
+  logger.log(`Deleted pipeline ${name2}.`);
 }
 __name(deletePipelineHandler, "deletePipelineHandler");
 
@@ -129319,7 +129345,7 @@ init_import_meta_url();
 function addGetOptions(yargs) {
   return yargs.positional("pipeline", {
     type: "string",
-    describe: "The name of the Pipeline to show",
+    describe: "The name of the pipeline to inspect",
     demandOption: true
   }).option("format", {
     type: "string",
@@ -129375,7 +129401,7 @@ __name(listPipelinesHandler, "listPipelinesHandler");
 init_import_meta_url();
 function addUpdateOptions(yargs) {
   return yargs.positional("pipeline", {
-    describe: "The name of the Pipeline to update",
+    describe: "The name of the pipeline to update",
     type: "string",
     demandOption: true
   }).option("r2-bucket", {
@@ -129388,7 +129414,7 @@ function addUpdateOptions(yargs) {
     `${source_default.bold("Source settings")}`
   ).option("source", {
     type: "array",
-    describe: "Space separated list of allowed sources. Options are 'http' or 'worker'. Setting this will remove all other existing sources.",
+    describe: "Space separated list of allowed sources. Options are 'http' or 'worker'",
     demandOption: false
   }).option("require-http-auth", {
     type: "boolean",
@@ -129458,7 +129484,7 @@ function addUpdateOptions(yargs) {
     demandOption: false
   }).group(["shard-count"], `${source_default.bold("Pipeline settings")}`).option("shard-count", {
     type: "number",
-    describe: "Number of pipeline shards. More shards handle higher request volume; fewer shards produce larger output files",
+    describe: "Number of shards for the pipeline. More shards handle higher request volume; fewer shards produce larger output files",
     demandOption: false
   });
 }
@@ -129516,7 +129542,7 @@ async function updatePipelineHandler(args) {
     const sourceHandlers = {
       http: /* @__PURE__ */ __name(() => {
         const existing = existingSources.find((s5) => s5.type === "http");
-        const http5 = {
+        return {
           ...existing,
           // Copy over existing properties for forwards compatibility
           type: "http",
@@ -129524,10 +129550,6 @@ async function updatePipelineHandler(args) {
           ...args.requireHttpAuth && { authentication: args.requireHttpAuth }
           // Include only if defined
         };
-        if (args.corsOrigins && args.corsOrigins.length > 0) {
-          http5.cors = { origins: args.corsOrigins };
-        }
-        return http5;
       }, "http"),
       worker: /* @__PURE__ */ __name(() => {
         const existing = existingSources.find(
@@ -129573,14 +129595,14 @@ async function updatePipelineHandler(args) {
     if (args.requireHttpAuth) {
       httpSource.authentication = args.requireHttpAuth;
     }
-    if (args.corsOrigins && args.corsOrigins.length > 0) {
+    if (args.corsOrigins) {
       httpSource.cors = { origins: args.corsOrigins };
     }
   }
-  logger.log(`\u{1F300} Updating Pipeline "${name2}"`);
+  logger.log(`\u{1F300} Updating pipeline "${name2}"`);
   const pipeline = await updatePipeline(accountId, name2, pipelineConfig);
   logger.log(
-    `\u2705 Successfully updated Pipeline "${pipeline.name}" with ID ${pipeline.id}
+    `\u2705 Successfully updated pipeline "${pipeline.name}" with ID ${pipeline.id}
 `
   );
 }
@@ -129670,27 +129692,27 @@ __name(parseTransform, "parseTransform");
 function pipelines(pipelineYargs) {
   return pipelineYargs.command(
     "create <pipeline>",
-    "Create a new Pipeline",
+    "Create a new pipeline",
     addCreateOptions,
     createPipelineHandler
   ).command(
     "list",
-    "List current Pipelines",
+    "List all pipelines",
     (yargs) => yargs,
     listPipelinesHandler
   ).command(
     "get <pipeline>",
-    "Get a Pipeline configuration",
+    "Get a pipeline's configuration",
     addGetOptions,
     getPipelineHandler
   ).command(
     "update <pipeline>",
-    "Update a Pipeline",
+    "Update a pipeline",
     addUpdateOptions,
     updatePipelineHandler
   ).command(
     "delete <pipeline>",
-    "Delete a Pipeline",
+    "Delete a pipeline",
     addDeleteOptions,
     deletePipelineHandler
   );
@@ -150345,7 +150367,7 @@ function createCLIParser(argv) {
   registry.registerNamespace("workflows");
   wrangler.command(
     "pipelines",
-    `\u{1F6B0} Manage Worker Pipelines ${source_default.hex(betaCmdColor)("[open beta]")}`,
+    `\u{1F6B0} Manage Cloudflare Pipelines ${source_default.hex(betaCmdColor)("[open beta]")}`,
     (pipelinesYargs) => {
       return pipelines(pipelinesYargs.command(subHelp));
     }