feat(@angular/cli): support Oxfmt for automatic file formatting

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

@@ -19,8 +19,8 @@ import { isPackageNameSafeForAnalytics } from '../analytics/analytics';
 import { EventCustomDimension } from '../analytics/analytics-parameters';
 import { getProjectByCwd, getSchematicDefaults } from '../utilities/config';
 import { assertIsError } from '../utilities/error';
+import { formatFiles } from '../utilities/formatting';
 import { memoize } from '../utilities/memoize';
-import { formatFiles } from '../utilities/prettier';
 import { isTTY } from '../utilities/tty';
 import {
   CommandModule,

packages/angular/cli/src/commands/update/utilities/migration.ts

@@ -19,8 +19,8 @@ import * as semver from 'semver';
 import { subscribeToWorkflow } from '../../../command-builder/utilities/schematic-workflow';
 import { colors } from '../../../utilities/color';
 import { assertIsError } from '../../../utilities/error';
+import { formatFiles } from '../../../utilities/formatting';
 import { writeErrorToLogFile } from '../../../utilities/log-file';
-import { formatFiles } from '../../../utilities/prettier';
 import { askChoices } from '../../../utilities/prompt';
 import { isTTY } from '../../../utilities/tty';
 import { coerceVersionNumber } from './cli-version';

packages/angular/cli/src/utilities/prettier.ts → packages/angular/cli/src/utilities/formatting.ts

@@ -13,7 +13,6 @@ import { dirname, join } from 'node:path';
 import { promisify } from 'node:util';
 
 const execFileAsync = promisify(execFile);
-let prettierCliPath: string | null | undefined;
 
 /**
  * Conservative upper bound for the length of a single spawned command line.
@@ -60,7 +59,7 @@ export function batchFilesByArgumentLength(
 }
 
 /**
- * Formats files using Prettier.
+ * Formats files using the workspace's Oxfmt or Prettier installation.
  * @param cwd The current working directory.
  * @param files The files to format.
  */
@@ -69,40 +68,44 @@ export async function formatFiles(cwd: string, files: Set<string>): Promise<void
     return;
   }
 
-  if (prettierCliPath === undefined) {
+  const workspaceRequire = createRequire(cwd + '/');
+  let formatterCliPath: string | undefined;
+  let formatter: string | undefined;
+  for (const name of ['oxfmt', 'prettier']) {
+    let packageJsonPath: string;
     try {
-      const prettierPath = createRequire(cwd + '/').resolve('prettier/package.json');
-      const prettierPackageJson = JSON.parse(await readFile(prettierPath, 'utf-8')) as {
-        bin: string;
-      };
-      prettierCliPath = join(dirname(prettierPath), prettierPackageJson.bin);
+      packageJsonPath = workspaceRequire.resolve(`${name}/package.json`);
     } catch {
-      // Prettier is not installed.
-      prettierCliPath = null;
+      continue;
     }
+
+    const { bin } = JSON.parse(await readFile(packageJsonPath, 'utf-8')) as {
+      bin: string | Record<string, string>;
+    };
+    formatterCliPath = join(dirname(packageJsonPath), typeof bin === 'string' ? bin : bin[name]);
+    formatter = name;
+    break;
   }
 
-  if (!prettierCliPath) {
+  if (!formatterCliPath) {
     return;
   }
 
-  const baseArgs = [
-    prettierCliPath,
-    '--write',
-    '--no-error-on-unmatched-pattern',
-    '--ignore-unknown',
-  ];
+  const baseArgs = [formatterCliPath, '--write', '--no-error-on-unmatched-pattern'];
+  if (formatter === 'prettier') {
+    baseArgs.push('--ignore-unknown');
+  }
   // `process.execPath` is spawned as the first argument and also counts toward the
-  // command-line limit; it and `prettierCliPath` are absolute paths that may contain
+  // command-line limit; it and `formatterCliPath` are absolute paths that may contain
   // spaces and therefore be quoted.
   const baseLength = [process.execPath, ...baseArgs].reduce(
     (total, arg) => total + arg.length + (arg.includes(' ') ? 3 : 1),
     0,
   );
 
-  // Spawn Prettier once per batch so repositories with many changed files do not
+  // Spawn the formatter once per batch so repositories with many changed files do not
   // overflow the OS command-line length limit. A failure in one batch (e.g. a file
-  // Prettier cannot parse) must not stop the remaining batches, matching the previous
+  // the formatter cannot parse) must not stop the remaining batches, matching the previous
   // single-invocation behavior, so errors are collected and reported together.
   const errors: string[] = [];
   for (const batch of batchFilesByArgumentLength(files, baseLength, MAX_COMMAND_LINE_LENGTH)) {

packages/angular/cli/src/utilities/prettier_spec.ts → packages/angular/cli/src/utilities/formatting_spec.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { batchFilesByArgumentLength } from './prettier';
+import { batchFilesByArgumentLength } from './formatting';
 
 describe('batchFilesByArgumentLength', () => {
   it('returns no batches when there are no files', () => {

tests/e2e/tests/generate/formatting.ts

@@ -0,0 +1,32 @@
+import assert from 'node:assert/strict';
+import { expectFileToMatch, readFile, writeFile } from '../../utils/fs';
+import { installPackage, uninstallPackage } from '../../utils/packages';
+import { ng } from '../../utils/process';
+
+export default async function () {
+  await uninstallPackage('prettier');
+  await installPackage('oxfmt');
+
+  const untouched = 'export const untouched={value:"unchanged"};';
+  await writeFile('src/untouched.ts', untouched);
+  await writeFile('.oxfmtrc.json', JSON.stringify({ singleQuote: false, semi: false }));
+
+  await ng('generate', 'service', 'oxfmt-only', '--skip-tests');
+  await expectFileToMatch('src/app/oxfmt-only.ts', 'from "@angular/core"\n');
+
+  // Distinct configurations make the selected formatter observable when both are installed.
+  await installPackage('prettier');
+  await writeFile('.prettierrc', JSON.stringify({ singleQuote: true, semi: false }));
+  await ng('generate', 'service', 'both-formatters', '--skip-tests');
+  await expectFileToMatch('src/app/both-formatters.ts', 'from "@angular/core"\n');
+
+  await uninstallPackage('oxfmt');
+  await ng('generate', 'service', 'prettier-only', '--skip-tests');
+  await expectFileToMatch('src/app/prettier-only.ts', "from '@angular/core'\n");
+
+  await uninstallPackage('prettier');
+  await ng('generate', 'service', 'no-formatter', '--skip-tests');
+  await expectFileToMatch('src/app/no-formatter.ts', "from '@angular/core';\n");
+
+  assert.equal(await readFile('src/untouched.ts'), untouched);
+}