refactor(@angular/cli): remove circular dependency in command module

This commit extracts shared definitions to `definitions.ts` to resolve a circular dependency between `analytics.ts` and `command-module.ts`.

Author: clydin

goldens/circular-deps/packages.json

@@ -30,9 +30,5 @@
   [
     "packages/angular/build/src/tools/esbuild/utils.ts",
     "packages/angular/build/src/utils/server-rendering/manifest.ts"
-  ],
-  [
-    "packages/angular/cli/src/analytics/analytics.ts",
-    "packages/angular/cli/src/command-builder/command-module.ts"
   ]
 ]

packages/angular/cli/src/analytics/analytics.ts

@@ -8,7 +8,7 @@
 
 import { json, tags } from '@angular-devkit/core';
 import { randomUUID } from 'node:crypto';
-import type { CommandContext } from '../command-builder/command-module';
+import type { CommandContext } from '../command-builder/definitions';
 import { colors } from '../utilities/color';
 import { getWorkspace } from '../utilities/config';
 import { analyticsDisabled } from '../utilities/environment-options';

packages/angular/cli/src/command-builder/command-module.ts

@@ -9,12 +9,7 @@
 import { logging, schema } from '@angular-devkit/core';
 import { readFileSync } from 'node:fs';
 import * as path from 'node:path';
-import type {
-  ArgumentsCamelCase,
-  Argv,
-  CamelCaseKey,
-  CommandModule as YargsCommandModule,
-} from 'yargs';
+import type { ArgumentsCamelCase, Argv, CommandModule as YargsCommandModule } from 'yargs';
 import { Parser as yargsParser } from 'yargs/helpers';
 import { getAnalyticsUserId } from '../analytics/analytics';
 import { AnalyticsCollector } from '../analytics/analytics-collector';
@@ -23,42 +18,11 @@ import { considerSettingUpAutocompletion } from '../utilities/completion';
 import { AngularWorkspace } from '../utilities/config';
 import { memoize } from '../utilities/memoize';
 import { PackageManagerUtils } from '../utilities/package-manager';
+import { CommandContext, CommandScope, Options, OtherOptions } from './definitions';
 import { Option, addSchemaOptionsToCommand } from './utilities/json-schema';
 
-export type Options<T> = { [key in keyof T as CamelCaseKey<key>]: T[key] };
-
-export enum CommandScope {
-  /** Command can only run inside an Angular workspace. */
-  In,
-
-  /** Command can only run outside an Angular workspace. */
-  Out,
-
-  /** Command can run inside and outside an Angular workspace. */
-  Both,
-}
-
-export interface CommandContext {
-  currentDirectory: string;
-  root: string;
-  workspace?: AngularWorkspace;
-  globalConfiguration: AngularWorkspace;
-  logger: logging.Logger;
-  packageManager: PackageManagerUtils;
-  yargsInstance: Argv<{}>;
-
-  /** Arguments parsed in free-from without parser configuration. */
-  args: {
-    positional: string[];
-    options: {
-      help: boolean;
-      jsonHelp: boolean;
-      getYargsCompletions: boolean;
-    } & Record<string, unknown>;
-  };
-}
-
-export type OtherOptions = Record<string, unknown>;
+export { CommandScope };
+export type { CommandContext, Options, OtherOptions };
 
 export interface CommandModuleImplementation<T extends {} = {}> extends Omit<
   YargsCommandModule<{}, T>,

packages/angular/cli/src/command-builder/definitions.ts

@@ -0,0 +1,47 @@
+/**
+ * @license
+ * Copyright Google LLC All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.dev/license
+ */
+
+import { logging } from '@angular-devkit/core';
+import type { Argv, CamelCaseKey } from 'yargs';
+import { AngularWorkspace } from '../utilities/config';
+import { PackageManagerUtils } from '../utilities/package-manager';
+
+export enum CommandScope {
+  /** Command can only run inside an Angular workspace. */
+  In,
+
+  /** Command can only run outside an Angular workspace. */
+  Out,
+
+  /** Command can run inside and outside an Angular workspace. */
+  Both,
+}
+
+export interface CommandContext {
+  currentDirectory: string;
+  root: string;
+  workspace?: AngularWorkspace;
+  globalConfiguration: AngularWorkspace;
+  logger: logging.Logger;
+  packageManager: PackageManagerUtils;
+  yargsInstance: Argv<{}>;
+
+  /** Arguments parsed in free-from without parser configuration. */
+  args: {
+    positional: string[];
+    options: {
+      help: boolean;
+      jsonHelp: boolean;
+      getYargsCompletions: boolean;
+    } & Record<string, unknown>;
+  };
+}
+
+export type Options<T> = { [key in keyof T as CamelCaseKey<key>]: T[key] };
+
+export type OtherOptions = Record<string, unknown>;