@atlaspack/core 2.26.1-dev-68c10d2af.0 → 2.26.2

package/src/requests/AtlaspackConfigRequest.ts

@@ -12,6 +12,7 @@ import type {
   PureAtlaspackConfigPipeline,
   AtlaspackOptions,
   ProcessedAtlaspackConfig,
+  AtlaspackPluginNode,
 } from '../types';
 
 import {createBuildCache} from '@atlaspack/build-cache';
@@ -197,13 +198,29 @@ export async function resolveAtlaspackConfig(
     await parseAndProcessConfig(configPath, contents, options);
 
   if (options.additionalReporters.length > 0) {
-    config.reporters = [
-      ...options.additionalReporters.map(({packageName, resolveFrom}) => ({
-        packageName,
-        resolveFrom,
-      })),
-      ...(config.reporters ?? []),
-    ];
+    if (options.featureFlags.deduplicateReporters) {
+      const reporterMap = new Map<PackageName, AtlaspackPluginNode>();
+
+      options.additionalReporters.forEach(({packageName, resolveFrom}) => {
+        reporterMap.set(packageName, {packageName, resolveFrom});
+      });
+
+      config.reporters?.forEach((reporter) => {
+        if (!reporterMap.has(reporter.packageName)) {
+          reporterMap.set(reporter.packageName, reporter);
+        }
+      });
+
+      config.reporters = Array.from(reporterMap.values());
+    } else {
+      config.reporters = [
+        ...options.additionalReporters.map(({packageName, resolveFrom}) => ({
+          packageName,
+          resolveFrom,
+        })),
+        ...(config.reporters ?? []),
+      ];
+    }
   }
 
   return {config, extendedFiles, usedDefault};

package/test/AtlaspackConfigRequest.test.ts

@@ -801,6 +801,78 @@ describe('AtlaspackConfigRequest', () => {
     });
   });
 
+  describe('additionalReporters', () => {
+    it('should deduplicate reporters by packageName', async () => {
+      const configPath = path.join(
+        __dirname,
+        'fixtures',
+        'config-with-reporters',
+        '.parcelrc',
+      );
+
+      const options = {
+        ...DEFAULT_OPTIONS,
+        projectRoot: path.join(__dirname, 'fixtures', 'config-with-reporters'),
+        config: configPath,
+        additionalReporters: [
+          {
+            packageName: '@atlaspack/reporter-cli',
+            resolveFrom: '/custom/path',
+          },
+          {
+            packageName: '@atlaspack/reporter-dev-server',
+            resolveFrom: '/another/path',
+          },
+        ],
+      };
+
+      const result = await resolveAtlaspackConfig(options);
+      assert(result !== null && result !== undefined);
+
+      const {config} = result;
+
+      // With feature flag enabled, should have 3 reporters (with deduplication)
+      assert(
+        config.reporters && config.reporters.length === 3,
+        'Should have exactly 3 reporters with deduplication',
+      );
+
+      const cliReporter = config.reporters?.find(
+        (r) => r.packageName === '@atlaspack/reporter-cli',
+      );
+      const bundleAnalyzerReporter = config.reporters?.find(
+        (r) => r.packageName === '@atlaspack/reporter-bundle-analyzer',
+      );
+      const devServerReporter = config.reporters?.find(
+        (r) => r.packageName === '@atlaspack/reporter-dev-server',
+      );
+
+      assert(cliReporter, 'CLI reporter should exist');
+      assert.equal(
+        cliReporter.resolveFrom,
+        '/custom/path',
+        'CLI reporter should use additional reporter resolveFrom',
+      );
+
+      assert(
+        bundleAnalyzerReporter,
+        'Bundle analyzer reporter should exist from original config',
+      );
+
+      assert(devServerReporter, 'Dev server reporter should exist');
+      assert.equal(devServerReporter.resolveFrom, '/another/path');
+
+      const cliReporters = config.reporters?.filter(
+        (r) => r.packageName === '@atlaspack/reporter-cli',
+      );
+      assert.equal(
+        cliReporters?.length,
+        1,
+        'Should have exactly one CLI reporter after deduplication',
+      );
+    });
+  });
+
   describe('resolve', () => {
     it('should return null if there is no .parcelrc file found', async () => {
       let resolved = await resolveAtlaspackConfig(DEFAULT_OPTIONS);

package/test/fixtures/config-with-reporters/.parcelrc

@@ -0,0 +1,7 @@
+{
+  "extends": "@atlaspack/config-default",
+  "reporters": [
+    "@atlaspack/reporter-cli",
+    "@atlaspack/reporter-bundle-analyzer"
+  ]
+}