remove i18n-report.json

Author: Brooooooklyn

napi/angular-compiler/.gitignore

@@ -3,4 +3,5 @@
 /test-results
 /playwright-report
 /e2e/compare/compare-report.json
+/e2e/compare/i18n-report.json
 /e2e/compare/dist

napi/angular-compiler/e2e/compare/fixtures/index.ts

@@ -6,8 +6,8 @@
  */
 
 import fg from "fast-glob";
-import { dirname, basename, relative } from "path";
-import { fileURLToPath } from "url";
+import { dirname, relative } from "path";
+import { fileURLToPath, pathToFileURL } from "url";
 import type { Fixture } from "./types.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
@@ -37,7 +37,9 @@ export async function discoverFixtures(
 
   for (const file of files) {
     try {
-      const module = await import(file);
+      // Use pathToFileURL for cross-platform compatibility (fixes Windows paths)
+      const fileUrl = pathToFileURL(file).href;
+      const module = await import(fileUrl);
 
       if (module.fixture && isValidFixture(module.fixture)) {
         // Set default filePath if not provided

napi/angular-compiler/e2e/compare/i18n-report.json

@@ -500,6 +500,24 @@ class AngularJsEmitter {
         const inner = node.expr ? this.emitExpr(node.expr) : "undefined";
         return `(${inner})`;
 
+      case "DynamicImportExpr": {
+        // Dynamic import expression: import('module')
+        // url can be a string or an expression
+        const url =
+          typeof node.url === "string" ? JSON.stringify(node.url) : this.emitExpr(node.url);
+        return `import(${url})`;
+      }
+
+      case "RegularExpressionLiteralExpr": {
+        // Regular expression literal: /pattern/flags
+        const flags = node.flags || "";
+        return `/${node.body}/${flags}`;
+      }
+
+      case "VoidExpr":
+        // Void expression: void (expr)
+        return `void (${this.emitExpr(node.expr)})`;
+
       default:
         // Emit a valid expression with a comment for unknown expression types
         // Using (void 0) produces undefined which is valid in any expression context

napi/angular-compiler/e2e/compare/src/compilers/angular.ts

@@ -1,4 +1,4 @@
-import { compileTemplateSync, type TransformOptions } from "@oxc/vite-plugin-angular";
+import { compileTemplateSync, Severity, type TransformOptions } from "@oxc/vite-plugin-angular";
 import type { CompilerOutput } from "../types.js";
 
 /**
@@ -16,8 +16,8 @@ export function compileWithOxc(
     const result = compileTemplateSync(template, className, filePath, options);
     const compilationTimeMs = performance.now() - startTime;
 
-    // Check for errors - errors array contains all diagnostics
-    const errors = result.errors;
+    // Only fail on actual errors, not warnings or advice
+    const errors = result.errors.filter((e) => e.severity === Severity.Error);
 
     if (errors.length > 0) {
       return {

napi/angular-compiler/e2e/compare/src/compilers/oxc.ts

@@ -8,10 +8,11 @@
 
 import { parseArgs } from "util";
 import { writeFile } from "fs/promises";
-import { resolve } from "path";
+import path, { resolve } from "path";
 import { runComparison } from "./runner.js";
 import { runFixtures, printFixtureSummary } from "../fixtures/runner.js";
 import { listFixtures } from "../fixtures/index.js";
+import { formatPresetList, getPreset, getPresetNames, mergePresetWithCli } from "./presets.js";
 import type { CompilerConfig } from "./types.js";
 
 async function main(): Promise<void> {
@@ -78,6 +79,15 @@ async function main(): Promise<void> {
         type: "boolean",
         description: "List all available fixtures without running",
       },
+      // Preset options
+      preset: {
+        type: "string",
+        description: "Use a predefined configuration preset",
+      },
+      "list-presets": {
+        type: "boolean",
+        description: "List all available presets without running",
+      },
     },
   });
 
@@ -86,13 +96,37 @@ async function main(): Promise<void> {
     process.exit(0);
   }
 
+  // Handle --list-presets
+  if (values["list-presets"]) {
+    console.log(formatPresetList());
+    process.exit(0);
+  }
+
   // Handle --list-fixtures
   if (values["list-fixtures"]) {
     const listing = await listFixtures(values.category);
     console.log(listing);
     process.exit(0);
   }
 
+  // Validate and resolve preset if specified
+  let presetInclude: string[] | undefined;
+  let presetExclude: string[] | undefined;
+  let presetName: string | undefined;
+
+  if (values.preset) {
+    const preset = getPreset(values.preset);
+    if (!preset) {
+      console.error(`Error: Unknown preset "${values.preset}"`);
+      console.error(`Available presets: ${getPresetNames().join(', ')}`);
+      process.exit(1);
+    }
+    presetName = preset.name;
+    const merged = mergePresetWithCli(preset, values.include, values.exclude);
+    presetInclude = merged.include;
+    presetExclude = merged.exclude;
+  }
+
   // Determine run mode
   const runFixturesOnly = values.fixtures && !values.both;
   const runBoth = values.both;
@@ -114,7 +148,7 @@ async function main(): Promise<void> {
 
       // Write fixture report
       const fixtureOutputPath = runBoth
-        ? resolve(values.output!.replace(".json", "-fixtures.json"))
+        ? resolve(values.output!.slice(0, -path.extname(values.output!).length) + "-fixtures.json")
         : resolve(values.output!);
       await writeFile(fixtureOutputPath, JSON.stringify(fixtureReport, null, 2), "utf-8");
       console.log(`\nFixture report written to: ${fixtureOutputPath}`);
@@ -129,10 +163,11 @@ async function main(): Promise<void> {
 
       const config: CompilerConfig = {
         projectRoot,
-        include: values.include,
-        exclude: values.exclude,
+        include: presetInclude ?? values.include,
+        exclude: presetExclude ?? values.exclude,
         parallel: !values["no-parallel"] && values.parallel !== false,
         outputPath: values.output,
+        presetName,
       };
 
       if (runBoth) {
@@ -186,6 +221,10 @@ FIXTURE OPTIONS:
   --category <name>         Filter fixtures by category (can be repeated)
   --list-fixtures           List all available fixtures without running
 
+PRESET OPTIONS:
+  --preset <name>           Use a predefined configuration preset
+  --list-presets            List all available presets without running
+
 EXAMPLES:
   # Compare bitwarden-clients
   pnpm compare -p ../bitwarden-clients
@@ -205,6 +244,15 @@ EXAMPLES:
   # List all fixtures
   pnpm compare --list-fixtures
 
+  # Use a preset for bitwarden
+  pnpm compare -p ../bitwarden-clients --preset bitwarden
+
+  # Use a preset with additional patterns
+  pnpm compare -p ../angular-material --preset material-angular --include "src/cdk-experimental/**/*.ts"
+
+  # List all presets
+  pnpm compare --list-presets
+
 REPORT FORMAT:
   The JSON report includes:
   - summary: Statistics (total, matched, mismatched, errors, pass rate)

napi/angular-compiler/e2e/compare/src/index.ts

@@ -0,0 +1,65 @@
+/**
+ * Preset definitions for the Angular compiler comparison tool.
+ */
+
+export interface Preset {
+  name: string;
+  description: string;
+  include: string[];
+  exclude: string[];
+}
+
+export const PRESETS: Record<string, Preset> = {
+  'bitwarden': {
+    name: 'bitwarden',
+    description: 'Bitwarden clients - .component.ts suffix',
+    include: ['**/*.component.ts'],
+    exclude: ['**/node_modules/**', '**/*.spec.ts', '**/*.test.ts'],
+  },
+  'material-angular': {
+    name: 'material-angular',
+    description: 'Angular Material/CDK - no .component.ts suffix',
+    include: [
+      'src/material/*/*.ts',
+      'src/cdk/*/*.ts',
+      'src/components-examples/**/*-example.ts',
+      'src/dev-app/**/*.ts',
+    ],
+    exclude: [
+      '**/node_modules/**', '**/testing/**', '**/*.spec.ts', '**/*.test.ts',
+      '**/*-module.ts', '**/public-api.ts', '**/index.ts', '**/BUILD.bazel',
+      '**/*.harness.ts', '**/*.e2e.ts', '**/schematics/**',
+    ],
+  },
+};
+
+export function getPreset(name: string): Preset | undefined {
+  return PRESETS[name];
+}
+
+export function getPresetNames(): string[] {
+  return Object.keys(PRESETS);
+}
+
+export function formatPresetList(): string {
+  const lines: string[] = ['Available presets:'];
+  for (const [name, preset] of Object.entries(PRESETS)) {
+    lines.push(`  ${name}`);
+    lines.push(`    ${preset.description}`);
+    lines.push(`    Include: ${preset.include.length} pattern(s)`);
+    lines.push(`    Exclude: ${preset.exclude.length} pattern(s)`);
+    lines.push('');
+  }
+  return lines.join('\n');
+}
+
+export function mergePresetWithCli(
+  preset: Preset,
+  cliInclude?: string[],
+  cliExclude?: string[],
+): { include: string[]; exclude: string[] } {
+  return {
+    include: [...preset.include, ...(cliInclude || [])],
+    exclude: [...preset.exclude, ...(cliExclude || [])],
+  };
+}

napi/angular-compiler/e2e/compare/src/presets.ts

@@ -12,6 +12,8 @@ export interface CompilerConfig {
   parallel?: boolean;
   /** Output file path for JSON report */
   outputPath?: string;
+  /** Name of the preset being used (for reporting purposes) */
+  presetName?: string;
 }
 
 /**