⚡ optimize plugin detection with concurrent operations (#26)

Replaced the sequential `for...of` loop in `checkPlugins` with `Promise.all` to execute `plugin.detect()` calls concurrently. This significantly reduces the time required for the detection phase, especially as the number of plugins grows.

The results are processed in the original order to ensure deterministic behavior for logging and merging `bundleParams`.

Measured improvement:
- Baseline (3 plugins, 100ms each): ~311ms
- Optimized (3 plugins, 100ms each): ~102ms
- Speedup: ~3x for the simulated case.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: sunnylqm <615282+sunnylqm@users.noreply.github.com>

Author: sunnylqm

src/utils/check-plugin.ts

@@ -13,19 +13,26 @@ export async function checkPlugins(): Promise<BundleParams> {
     sourcemap: false,
   };
 
-  for (const plugin of plugins) {
-    try {
-      const isEnabled = await plugin.detect();
-      if (isEnabled && plugin.bundleParams) {
-        Object.assign(params, plugin.bundleParams);
-        console.log(t('pluginDetected', { name: plugin.name }));
+  const results = await Promise.all(
+    plugins.map(async (plugin) => {
+      try {
+        const isEnabled = await plugin.detect();
+        return { isEnabled, error: null };
+      } catch (error) {
+        return { isEnabled: false, error };
       }
-    } catch (err) {
-      console.warn(
-        t('pluginDetectionError', { name: plugin.name, error: err }),
-      );
+    }),
+  );
+
+  results.forEach(({ isEnabled, error }, index) => {
+    const plugin = plugins[index];
+    if (error) {
+      console.warn(t('pluginDetectionError', { name: plugin.name, error }));
+    } else if (isEnabled && plugin.bundleParams) {
+      Object.assign(params, plugin.bundleParams);
+      console.log(t('pluginDetected', { name: plugin.name }));
     }
-  }
+  });
 
   return params;
 }

tests/check-plugin.test.ts

@@ -0,0 +1,75 @@
+import { describe, expect, mock, spyOn, test, beforeEach, afterEach } from 'bun:test';
+
+// Mock dependencies before imports
+mock.module('fs-extra', () => ({
+  default: {
+    access: async () => {},
+  },
+}));
+
+mock.module('i18next', () => ({
+  default: {
+    init: () => {},
+    t: (key: string) => key,
+  },
+}));
+
+import { checkPlugins } from '../src/utils/check-plugin';
+import * as pluginConfig from '../src/utils/plugin-config';
+
+describe('checkPlugins', () => {
+  test('should detect plugins concurrently (simulated)', async () => {
+    const mockPlugins = [
+      {
+        name: 'plugin1',
+        bundleParams: { p1: true },
+        detect: async () => {
+          await new Promise(resolve => setTimeout(resolve, 100));
+          return true;
+        }
+      },
+      {
+        name: 'plugin2',
+        bundleParams: { p2: true },
+        detect: async () => {
+          await new Promise(resolve => setTimeout(resolve, 100));
+          return true;
+        }
+      },
+      {
+        name: 'plugin3',
+        bundleParams: { p3: true },
+        detect: async () => {
+          await new Promise(resolve => setTimeout(resolve, 100));
+          return true;
+        }
+      }
+    ];
+
+    // Replacing the plugins array in plugin-config
+    const originalPlugins = [...pluginConfig.plugins];
+    (pluginConfig.plugins as any).splice(0, pluginConfig.plugins.length, ...mockPlugins);
+
+    const start = Date.now();
+    const result = await checkPlugins();
+    const end = Date.now();
+    const duration = end - start;
+
+    console.log(`Duration with optimized implementation: ${duration}ms`);
+
+    expect(result).toEqual({
+      sentry: false, // default
+      sourcemap: false, // default
+      p1: true,
+      p2: true,
+      p3: true
+    } as any);
+
+    // Now it's concurrent, so we expect around 100ms.
+    // We'll allow some buffer, but it should definitely be less than 250ms.
+    expect(duration).toBeLessThan(250);
+
+    // Restore original plugins
+    (pluginConfig.plugins as any).splice(0, pluginConfig.plugins.length, ...originalPlugins);
+  });
+});