feat: use jiti for plugin loading

package.json

@@ -64,6 +64,7 @@
     "@nx/react": "22.3.3",
     "@nx/vite": "22.3.3",
     "@nx/workspace": "22.3.3",
+    "@push-based/jiti-tsc": "^0.0.2",
     "@push-based/nx-verdaccio": "0.0.7",
     "@swc-node/register": "1.9.2",
     "@swc/cli": "0.6.0",
@@ -102,7 +103,7 @@
     "husky": "^8.0.0",
     "inquirer": "^9.3.7",
     "jest-extended": "^6.0.0",
-    "jiti": "2.4.2",
+    "jiti": "^2.6.1",
     "jsdom": "~24.0.0",
     "jsonc-eslint-parser": "^2.4.0",
     "knip": "^5.33.3",

packages/cli/mocks/fixtures/configs/code-pushup.config.js

@@ -0,0 +1,43 @@
+export default {
+  upload: {
+    organization: 'code-pushup',
+    project: 'cli-js',
+    apiKey: 'e2e-api-key',
+    server: 'https://e2e.com/api',
+  },
+  categories: [
+    {
+      slug: 'category-1',
+      title: 'Category 1',
+      refs: [
+        {
+          type: 'audit',
+          plugin: 'node',
+          slug: 'node-version',
+          weight: 1,
+        },
+      ],
+    },
+  ],
+  plugins: [
+    {
+      audits: [
+        {
+          slug: 'node-version',
+          title: 'Node version',
+          description: 'prints node version to file',
+          docsUrl: 'https://nodejs.org/',
+        },
+      ],
+      runner: {
+        command: 'node',
+        args: ['-v'],
+        outputFile: 'output.json',
+      },
+      groups: [],
+      slug: 'node',
+      title: 'Node.js',
+      icon: 'javascript',
+    },
+  ],
+};

packages/cli/mocks/fixtures/configs/code-pushup.config.mjs

@@ -0,0 +1,43 @@
+export default {
+  upload: {
+    organization: 'code-pushup',
+    project: 'cli-mjs',
+    apiKey: 'e2e-api-key',
+    server: 'https://e2e.com/api',
+  },
+  categories: [
+    {
+      slug: 'category-1',
+      title: 'Category 1',
+      refs: [
+        {
+          type: 'audit',
+          plugin: 'node',
+          slug: 'node-version',
+          weight: 1,
+        },
+      ],
+    },
+  ],
+  plugins: [
+    {
+      audits: [
+        {
+          slug: 'node-version',
+          title: 'Node version',
+          description: 'prints node version to file',
+          docsUrl: 'https://nodejs.org/',
+        },
+      ],
+      runner: {
+        command: 'node',
+        args: ['-v'],
+        outputFile: 'output.json',
+      },
+      groups: [],
+      slug: 'node',
+      title: 'Node.js',
+      icon: 'javascript',
+    },
+  ],
+};

packages/cli/mocks/fixtures/configs/code-pushup.config.ts

@@ -0,0 +1,45 @@
+import { type CoreConfig } from '@code-pushup/models';
+
+export default {
+  upload: {
+    organization: 'code-pushup',
+    project: 'cli-ts',
+    apiKey: 'e2e-api-key',
+    server: 'https://e2e.com/api',
+  },
+  categories: [
+    {
+      slug: 'category-1',
+      title: 'Category 1',
+      refs: [
+        {
+          type: 'audit',
+          plugin: 'node',
+          slug: 'node-version',
+          weight: 1,
+        },
+      ],
+    },
+  ],
+  plugins: [
+    {
+      audits: [
+        {
+          slug: 'node-version',
+          title: 'Node version',
+          description: 'prints node version to file',
+          docsUrl: 'https://nodejs.org/',
+        },
+      ],
+      runner: {
+        command: 'node',
+        args: ['-v'],
+        outputFile: 'output.json',
+      },
+      groups: [],
+      slug: 'node',
+      title: 'Node.js',
+      icon: 'javascript',
+    },
+  ],
+} satisfies CoreConfig;

packages/cli/mocks/fixtures/configs/code-pushup.needs-tsconfig-fail.config.ts

@@ -0,0 +1,10 @@
+// the point is to test runtime import which relies on alias defined in tsconfig.json "paths"
+// non-type import from '@example/custom-plugin' wouldn't work without --tsconfig
+// eslint-disable-next-line import/no-unresolved
+import customPlugin from '@definitely-non-existent-package/custom-plugin';
+
+const config = {
+  plugins: [customPlugin],
+};
+
+export default config;

packages/cli/mocks/fixtures/configs/code-pushup.needs-tsconfig.config.ts

@@ -0,0 +1,10 @@
+// the point is to test runtime import which relies on alias defined in tsconfig.json "paths"
+// non-type import from '@example/custom-plugin' wouldn't work without --tsconfig
+// eslint-disable-next-line import/no-unresolved
+import customPlugin from '@definitely-non-existent-package/custom-plugin';
+
+const config = {
+  plugins: [customPlugin],
+};
+
+export default config;

packages/cli/mocks/fixtures/configs/custom-plugin.ts

@@ -0,0 +1,24 @@
+const customPluginConfig = {
+  slug: 'good-feels',
+  title: 'Good feels',
+  icon: 'javascript',
+  audits: [
+    {
+      slug: 'always-perfect',
+      title: 'Always perfect',
+    },
+  ],
+  runner: () => [
+    {
+      slug: 'always-perfect',
+      score: 1,
+      value: 100,
+      displayValue: '✅ Perfect! 👌',
+    },
+  ],
+};
+
+export function customPlugin() {
+  return customPluginConfig;
+}
+export default customPluginConfig;

packages/cli/mocks/fixtures/configs/tsconfig.alias.json

@@ -0,0 +1,7 @@
+{
+  "compilerOptions": {
+    "paths": {
+      "@definitely-non-existent-package/custom-plugin": ["./custom-plugin.ts"]
+    }
+  }
+}

packages/cli/mocks/fixtures/configs/tsconfig.json

@@ -0,0 +1,5 @@
+{
+  "compilerOptions": {
+    "paths": {}
+  }
+}

packages/cli/src/lib/implementation/core-config.middleware.int.test.ts

@@ -18,6 +18,16 @@ const configDirPath = path.join(
   'configs',
 );
 
+const localMocks = path.join(
+  fileURLToPath(path.dirname(import.meta.url)),
+  '..',
+  '..',
+  '..',
+  'mocks',
+  'fixtures',
+  'configs',
+);
+
 describe('coreConfigMiddleware', () => {
   const CLI_DEFAULTS = {
     plugins: [],
@@ -29,7 +39,7 @@ describe('coreConfigMiddleware', () => {
     'should load a valid .%s config',
     async extension => {
       const config = await coreConfigMiddleware({
-        config: path.join(configDirPath, `code-pushup.config.${extension}`),
+        config: path.join(localMocks, `code-pushup.config.${extension}`),
         ...CLI_DEFAULTS,
       });
       expect(config.config).toContain(`code-pushup.config.${extension}`);
@@ -46,11 +56,8 @@ describe('coreConfigMiddleware', () => {
   it('should load config which relies on provided --tsconfig', async () => {
     await expect(
       coreConfigMiddleware({
-        config: path.join(
-          configDirPath,
-          'code-pushup.needs-tsconfig.config.ts',
-        ),
-        tsconfig: path.join(configDirPath, 'tsconfig.json'),
+        config: path.join(localMocks, 'code-pushup.needs-tsconfig.config.ts'),
+        tsconfig: path.join(localMocks, 'tsconfig.alias.json'),
         ...CLI_DEFAULTS,
       }),
     ).resolves.toBeTruthy();
@@ -60,11 +67,13 @@ describe('coreConfigMiddleware', () => {
     await expect(
       coreConfigMiddleware({
         config: path.join(
-          configDirPath,
-          'code-pushup.needs-tsconfig.config.ts',
+          localMocks,
+          'code-pushup.needs-tsconfig-fail.config.ts',
         ),
         ...CLI_DEFAULTS,
       }),
-    ).rejects.toThrow("Cannot find package '@example/custom-plugin'");
+    ).rejects.toThrow(
+      "Cannot find module '@definitely-non-existent-package/custom-plugin'",
+    );
   });
 });

packages/core/src/lib/implementation/read-rc-file.unit.test.ts

@@ -4,32 +4,39 @@ import { CONFIG_FILE_NAME, type CoreConfig } from '@code-pushup/models';
 import { MEMFS_VOLUME } from '@code-pushup/test-utils';
 import { autoloadRc } from './read-rc-file.js';
 
-// mock bundleRequire inside importEsmModule used for fetching config
-vi.mock('bundle-require', async () => {
+// mock jiti used for fetching config
+vi.mock('jiti', async () => {
   const { CORE_CONFIG_MOCK }: Record<string, CoreConfig> =
     await vi.importActual('@code-pushup/test-fixtures');
 
+  const actualJiti = await vi.importActual('jiti');
+
   return {
-    bundleRequire: vi
-      .fn()
-      .mockImplementation((options: { filepath: string }) => {
-        const extension = options.filepath.split('.').at(-1);
-        return {
-          mod: {
-            default: {
+    ...actualJiti,
+    createJiti: vi.fn().mockImplementation(() => ({
+      import: vi
+        .fn()
+        .mockImplementation(
+          (filepath: string, options: { default?: boolean }) => {
+            const extension = filepath.split('.').at(-1);
+            const config = {
               ...CORE_CONFIG_MOCK,
               upload: {
                 ...CORE_CONFIG_MOCK?.upload,
                 project: extension, // returns loaded file extension to check format precedence
               },
-            },
+            };
+
+            // When default: true is passed, return the config directly
+            // Otherwise return { default: config }
+            return options?.default ? config : { default: config };
           },
-        };
-      }),
+        ),
+    })),
   };
 });
 
-// Note: memfs files are only listed to satisfy a system check, value is used from bundle-require mock
+// Note: memfs files are only listed to satisfy a system check, value is used from jiti mock
 describe('autoloadRc', () => {
   it('prioritise a .ts configuration file', async () => {
     vol.fromJSON(