fix: resolve CI lint errors and add build step for prettier version tests

Fix lint issues in build-externals test (import style, naming conventions,
encoding identifier) and add missing `pnpm build` step to test-prettier-versions
CI job so the dist file exists when the build-externals test runs. Also make
assertions resilient to both CJS and ESM output formats.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: lifeart

.github/workflows/ci.yml

@@ -77,4 +77,6 @@ jobs:
 
       - run: pnpm install --frozen-lockfile
 
+      - run: pnpm build
+
       - run: pnpm test

tests/unit-tests/build-externals.test.ts

@@ -1,27 +1,32 @@
 import { readFileSync } from 'node:fs';
-import { resolve } from 'node:path';
+import path from 'node:path';
+
 import { describe, expect, test } from 'vitest';
 
 describe('build externals', () => {
-  const distPath = resolve(
+  const distributionPath = path.resolve(
     import.meta.dirname,
     '../../dist/prettier-plugin-ember-template-tag.js',
   );
 
   test('prettier/plugins/estree is externalized, not bundled', () => {
-    const dist = readFileSync(distPath, 'utf-8');
+    const distribution = readFileSync(distributionPath, 'utf8');
 
-    // The dist should require estree as an external dependency
-    expect(dist).toContain('require("prettier/plugins/estree")');
+    // The dist should reference estree as an external dependency (require or import)
+    expect(distribution).toMatch(
+      /require\(["']prettier\/plugins\/estree["']\)|from\s+["']prettier\/plugins\/estree["']/,
+    );
 
     // The dist should NOT contain estree printer internals (canAttachComment is
     // a function from the estree printer that would be present if bundled)
-    expect(dist).not.toMatch(/function\s+\w*canAttachComment/);
+    expect(distribution).not.toMatch(/function\s+\w*canAttachComment/);
   });
 
   test('prettier/plugins/babel is externalized, not bundled', () => {
-    const dist = readFileSync(distPath, 'utf-8');
+    const distribution = readFileSync(distributionPath, 'utf8');
 
-    expect(dist).toContain('require("prettier/plugins/babel")');
+    expect(distribution).toMatch(
+      /require\(["']prettier\/plugins\/babel["']\)|from\s+["']prettier\/plugins\/babel["']/,
+    );
   });
 });