chore: use defineCommand in wrangler docs (#7246)

* use defineCommand

* feedback

Author: emily-shen

packages/wrangler/src/docs/index.ts

@@ -1,55 +1,53 @@
-import { printWranglerBanner } from "..";
-import { readConfig } from "../config";
+import { defineCommand } from "../core";
 import { logger } from "../logger";
 import * as metrics from "../metrics";
 import openInBrowser from "../open-in-browser";
 import { runSearch } from "./helpers";
-import type {
-	CommonYargsArgv,
-	StrictYargsOptionsToInterface,
-} from "../yargs-types";
 
-export function docsOptions(yargs: CommonYargsArgv) {
-	return yargs
-		.positional("search", {
+defineCommand({
+	command: "wrangler docs",
+	metadata: {
+		description: "📚 Open Wrangler's command documentation in your browser\n",
+		owner: "Workers: Authoring and Testing",
+		status: "stable",
+	},
+	args: {
+		search: {
 			describe:
 				"Enter search terms (e.g. the wrangler command) you want to know more about",
 			type: "string",
 			array: true,
-		})
-		.option("yes", {
+		},
+		yes: {
 			alias: "y",
 			type: "boolean",
-			description: "Takes you to the docs, even if search fails",
-		});
-}
-
-export async function docsHandler(
-	args: StrictYargsOptionsToInterface<typeof docsOptions>
-) {
-	//if no command is provided, open the docs homepage
-	//or, if a command IS provided, but we can't find anything, open the docs homepage
-	let urlToOpen =
-		args.yes || !args.search || args.search.length === 0
-			? "https://developers.cloudflare.com/workers/wrangler/commands/"
-			: "";
+			describe: "Takes you to the docs, even if search fails",
+		},
+	},
+	positionalArgs: ["search"],
+	async handler(args, { config }) {
+		//if no command is provided, open the docs homepage
+		//or, if a command IS provided, but we can't find anything, open the docs homepage
+		let urlToOpen =
+			args.yes || !args.search || args.search.length === 0
+				? "https://developers.cloudflare.com/workers/wrangler/commands/"
+				: "";
 
-	if (args.search && args.search.length > 0) {
-		const searchTerm = args.search.join(" ");
-		const searchResult = await runSearch(searchTerm);
+		if (args.search && args.search.length > 0) {
+			const searchTerm = args.search.join(" ");
+			const searchResult = await runSearch(searchTerm);
 
-		urlToOpen = searchResult ?? urlToOpen;
-	}
+			urlToOpen = searchResult ?? urlToOpen;
+		}
 
-	if (!urlToOpen) {
-		return;
-	}
-	await printWranglerBanner();
+		if (!urlToOpen) {
+			return;
+		}
 
-	logger.log(`Opening a link in your default browser: ${urlToOpen}`);
-	await openInBrowser(urlToOpen);
-	const config = readConfig(undefined, args);
-	await metrics.sendMetricsEvent("view docs", {
-		sendMetrics: config.send_metrics,
-	});
-}
+		logger.log(`Opening a link in your default browser: ${urlToOpen}`);
+		await openInBrowser(urlToOpen);
+		await metrics.sendMetricsEvent("view docs", {
+			sendMetrics: config.send_metrics,
+		});
+	},
+});

packages/wrangler/src/index.ts

@@ -37,7 +37,6 @@ import {
 } from "./deprecated";
 import { devHandler, devOptions } from "./dev";
 import { workerNamespaceCommands } from "./dispatch-namespace";
-import { docsHandler, docsOptions } from "./docs";
 import {
 	CommandLineArgsError,
 	JsonFriendlyFatalError,
@@ -46,6 +45,7 @@ import {
 import { generateHandler, generateOptions } from "./generate";
 import { hyperdrive } from "./hyperdrive/index";
 import { initHandler, initOptions } from "./init";
+import "./docs";
 import "./kv";
 import "./workflows";
 import "./user/commands";
@@ -317,12 +317,7 @@ export function createCLIParser(argv: string[]) {
 	/*                 WRANGLER COMMANDS                  */
 	/******************************************************/
 	// docs
-	wrangler.command(
-		"docs [search..]",
-		"📚 Open Wrangler's command documentation in your browser\n",
-		docsOptions,
-		docsHandler
-	);
+	register.registerNamespace("docs");
 
 	/******************** CMD GROUP ***********************/
 	// init