fix logging issues

Author: emily-shen

packages/wrangler/src/dev.ts

@@ -54,7 +54,6 @@ import type {
 } from "./deployment-bundle/worker";
 import type { WorkerRegistry } from "./dev-registry";
 import type { CfAccount } from "./dev/create-worker-preview";
-import type { LoggerLevel } from "./logger";
 import type { EnablePagesAssetsServiceBindingOptions } from "./miniflare-cli/types";
 import type { watch } from "chokidar";
 import type { Json } from "miniflare";
@@ -310,8 +309,6 @@ export const dev = createCommand({
 		"log-level": {
 			choices: ["debug", "info", "log", "warn", "error", "none"] as const,
 			describe: "Specify logging level",
-			// Yargs requires this to type log-level properly
-			default: "log" as LoggerLevel,
 		},
 		"show-interactive-dev-session": {
 			describe:
@@ -666,10 +663,6 @@ export async function startDev(args: StartDevOptions) {
 
 	let unregisterHotKeys: (() => void) | undefined;
 	try {
-		if (args.logLevel) {
-			logger.loggerLevel = args.logLevel;
-		}
-
 		if (args.experimentalLocal) {
 			logger.warn(
 				"--experimental-local is no longer required and will be removed in a future version.\n`wrangler dev` now uses the local Cloudflare Workers runtime by default. 🎉"

packages/wrangler/src/index.ts

@@ -329,11 +329,6 @@ export function createCLIParser(argv: string[]) {
 			alias: ["x-versions", "experimental-gradual-rollouts"],
 		})
 		.check((args) => {
-			// Update logger level, before we do any logging
-			if (Object.keys(LOGGER_LEVELS).includes(args.logLevel as string)) {
-				logger.loggerLevel = args.logLevel as LoggerLevel;
-			}
-
 			// Grab locally specified env params from `.env` file
 			const loaded = loadDotEnv(".env", args.env);
 			for (const [key, value] of Object.entries(loaded?.parsed ?? {})) {
@@ -1056,6 +1051,10 @@ export async function main(argv: string[]): Promise<void> {
 	// Register Yargs middleware to record command as Sentry breadcrumb
 	let recordedCommand = false;
 	const wranglerWithMiddleware = wrangler.middleware((args) => {
+		// Update logger level, before we do any logging
+		if (Object.keys(LOGGER_LEVELS).includes(args.logLevel as string)) {
+			logger.loggerLevel = args.logLevel as LoggerLevel;
+		}
 		// Middleware called for each sub-command, but only want to record once
 		if (recordedCommand) {
 			return;

packages/wrangler/src/metrics/helpers.ts

@@ -1,6 +1,5 @@
 import os from "node:os";
 import { version as wranglerVersion } from "../../package.json";
-import { configFormat } from "../config";
 
 export function getWranglerVersion() {
 	return wranglerVersion;