Extract inline types to an interface

Author: schaable

v-next/hardhat/src/internal/cli/helpers/utils.ts

@@ -6,9 +6,16 @@ import { BUILTIN_OPTIONS } from "../../builtin-options.js";
 
 export const GLOBAL_NAME_PADDING = 6;
 
+interface ArgumentDescriptor {
+  name: string;
+  description: string;
+  type?: ParameterType;
+  isRequired?: boolean;
+}
+
 export function parseGlobalOptions(
   globalOptionsMap: GlobalOptionsMap,
-): Array<{ name: string; description: string }> {
+): ArgumentDescriptor[] {
   const formattedBuiltinOptions = BUILTIN_OPTIONS.map(
     ({ name, description }) => ({
       name: formatOptionName(name),
@@ -27,8 +34,8 @@ export function parseGlobalOptions(
 }
 
 export function parseTasks(taskMap: Map<string, Task>): {
-  tasks: Array<{ name: string; description: string }>;
-  subtasks: Array<{ name: string; description: string }>;
+  tasks: ArgumentDescriptor[];
+  subtasks: ArgumentDescriptor[];
 } {
   const tasks = [];
   const subtasks = [];
@@ -46,9 +53,7 @@ export function parseTasks(taskMap: Map<string, Task>): {
   return { tasks, subtasks };
 }
 
-export function parseSubtasks(
-  task: Task,
-): Array<{ name: string; description: string }> {
+export function parseSubtasks(task: Task): ArgumentDescriptor[] {
   const subtasks = [];
 
   for (const [, subtask] of task.subtasks) {
@@ -62,12 +67,8 @@ export function parseSubtasks(
 }
 
 export function parseOptions(task: Task): {
-  options: Array<{ name: string; description: string; type: ParameterType }>;
-  positionalArguments: Array<{
-    name: string;
-    description: string;
-    isRequired: boolean;
-  }>;
+  options: ArgumentDescriptor[];
+  positionalArguments: ArgumentDescriptor[];
 } {
   const options = [];
   const positionalArguments = [];
@@ -108,7 +109,7 @@ export function getLongestNameLength(tasks: Array<{ name: string }>): number {
 
 export function getSection(
   title: string,
-  items: Array<{ name: string; description: string }>,
+  items: ArgumentDescriptor[],
   namePadding: number,
 ): string {
   return `\n${title}:\n\n${items.map(({ name, description }) => `  ${name.padEnd(namePadding)}${description}`).join("\n")}\n`;
@@ -126,7 +127,7 @@ export function getUsageString(
   }
 
   if (positionalArguments.length > 0) {
-    output += ` [--] ${positionalArguments.map((a) => (a.isRequired ? a.name : `[${a.name}]`)).join(" ")}`;
+    output += ` [--] ${positionalArguments.map((a) => (a.isRequired === true ? a.name : `[${a.name}]`)).join(" ")}`;
   }
 
   return output;