refactor: fix unit tests

Author: John Doe

packages/cli/package.json

@@ -39,7 +39,7 @@
   },
   "type": "module",
   "bin": {
-    "code-pushup": "./src/index.js"
+    "code-pushup": "./src/index.mjs"
   },
   "engines": {
     "node": ">=20"

packages/create-cli/package.json

@@ -2,7 +2,7 @@
   "name": "@code-pushup/create-cli",
   "version": "0.80.2",
   "license": "MIT",
-  "bin": "index.js",
+  "bin": "src/index.mjs",
   "homepage": "https://github.com/code-pushup/cli/tree/main/packages/create-cli#readme",
   "bugs": {
     "url": "https://github.com/code-pushup/cli/issues?q=is%3Aissue%20state%3Aopen%20type%3ABug%20label%3A\"🧩%20create-cli\""

packages/plugin-lighthouse/src/lib/runner/constants.ts

@@ -13,60 +13,44 @@ const { audits, categories } = defaultConfig;
 
 export const PLUGIN_SLUG = 'lighthouse';
 
-// Initialize with empty arrays - will be populated by initializeLighthouseConstants()
-export let LIGHTHOUSE_NAVIGATION_AUDITS: Audit[] = [];
-export let LIGHTHOUSE_GROUPS: Group[] = [];
+const allRawLighthouseAudits = await Promise.all(
+  (audits ?? []).map(loadLighthouseAudit),
+);
 
-// Lazy initialization flag
-let initialized = false;
+export const LIGHTHOUSE_NAVIGATION_AUDITS: Audit[] = allRawLighthouseAudits
+  // This plugin only supports the "navigation" mode of Lighthouse in the current implementation
+  // If we don't exclude other audits we throw in the plugin output validation as some of the provided audits are not included in `lighthouse-report.json`
+  .filter(
+    audit =>
+      audit.meta.supportedModes == null ||
+      (Array.isArray(audit.meta.supportedModes) &&
+        audit.meta.supportedModes.includes('navigation')),
+  )
+  .map(audit => ({
+    slug: audit.meta.id,
+    title: getMetaString(audit.meta.title),
+    description: getMetaString(audit.meta.description),
+  }));
 
-// Call this once before using the constants
-export async function initializeLighthouseConstants(): Promise<void> {
-  if (initialized) {
-    return;
-  }
-
-  const allRawLighthouseAudits = await Promise.all(
-    (audits ?? []).map(loadLighthouseAudit),
-  );
-
-  LIGHTHOUSE_NAVIGATION_AUDITS = allRawLighthouseAudits
-    // This plugin only supports the "navigation" mode of Lighthouse in the current implementation
-    // If we don't exclude other audits we throw in the plugin output validation as some of the provided audits are not included in `lighthouse-report.json`
-    .filter(
-      audit =>
-        audit.meta.supportedModes == null ||
-        (Array.isArray(audit.meta.supportedModes) &&
-          audit.meta.supportedModes.includes('navigation')),
-    )
-    .map(audit => ({
-      slug: audit.meta.id,
-      title: getMetaString(audit.meta.title),
-      description: getMetaString(audit.meta.description),
-    }));
+const navigationAuditSlugs = new Set(
+  LIGHTHOUSE_NAVIGATION_AUDITS.map(({ slug }) => slug),
+);
 
-  const navigationAuditSlugs = new Set(
-    LIGHTHOUSE_NAVIGATION_AUDITS.map(({ slug }) => slug),
-  );
-
-  LIGHTHOUSE_GROUPS = Object.entries(categories ?? {}).map(
-    ([id, category]) => ({
-      slug: id,
-      title: getMetaString(category.title),
-      ...(category.description && {
-        description: getMetaString(category.description),
-      }),
-      refs: category.auditRefs
-        .filter(({ id: auditSlug }) => navigationAuditSlugs.has(auditSlug))
-        .map(ref => ({
-          slug: ref.id,
-          weight: ref.weight,
-        })),
+export const LIGHTHOUSE_GROUPS: Group[] = Object.entries(categories ?? {}).map(
+  ([id, category]) => ({
+    slug: id,
+    title: getMetaString(category.title),
+    ...(category.description && {
+      description: getMetaString(category.description),
     }),
-  );
-
-  initialized = true;
-}
+    refs: category.auditRefs
+      .filter(({ id: auditSlug }) => navigationAuditSlugs.has(auditSlug))
+      .map(ref => ({
+        slug: ref.id,
+        weight: ref.weight,
+      })),
+  }),
+);
 
 function getMetaString(value: string | IcuMessage): string {
   if (typeof value === 'string') {

packages/plugin-lighthouse/tsdown.config.ts

@@ -5,5 +5,7 @@ const __dirname = import.meta.dirname;
 
 export default defineConfig({
   ...baseConfig({ projectRoot: __dirname }),
+  // Override format to ESM only - this package uses top-level await in constants.ts
+  format: ['esm'],
   external: await getExternalDependencies(__dirname),
 });