chore: revert some changes

Author: logaretm

dev-packages/rollup-utils/bundleHelpers.mjs

@@ -142,35 +142,25 @@ export function makeBundleConfigVariants(baseConfig, options = {}) {
   const terserPlugin = makeTerserPlugin();
   const setSdkSourcePlugin = makeSetSDKSourcePlugin('cdn');
 
-  const baseOutput = baseConfig.output;
-  if (!baseOutput || Array.isArray(baseOutput)) {
-    throw new Error('Base config must have a single output object');
-  }
-
-  const baseOutputEntryFileNames = baseOutput.entryFileNames;
-  if (typeof baseOutputEntryFileNames !== 'function') {
-    throw new Error('Base config must have a function for entryFileNames');
-  }
-
   // The additional options to use for each variant we're going to create.
   const variantSpecificConfigMap = {
     '.js': {
       output: {
-        entryFileNames: chunkInfo => `${baseOutputEntryFileNames(chunkInfo)}.js`,
+        entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.js`,
       },
       plugins: [includeDebuggingPlugin, setSdkSourcePlugin],
     },
 
     '.min.js': {
       output: {
-        entryFileNames: chunkInfo => `${baseOutputEntryFileNames(chunkInfo)}.min.js`,
+        entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.min.js`,
       },
       plugins: [stripDebuggingPlugin, setSdkSourcePlugin, terserPlugin],
     },
 
     '.debug.min.js': {
       output: {
-        entryFileNames: chunkInfo => `${baseOutputEntryFileNames(chunkInfo)}.debug.min.js`,
+        entryFileNames: chunkInfo => `${baseConfig.output.entryFileNames(chunkInfo)}.debug.min.js`,
       },
       plugins: [includeDebuggingPlugin, setSdkSourcePlugin, terserPlugin],
     },

dev-packages/rollup-utils/npmHelpers.mjs

@@ -97,27 +97,18 @@ export function makeBaseNPMConfig(options = {}) {
 
 export function makeNPMConfigVariants(baseConfig, options = {}) {
   const { emitEsm = true, emitCjs = true, splitDevProd = false } = options;
-  const baseOutput = baseConfig.output;
-  if (!baseOutput || Array.isArray(baseOutput)) {
-    throw new Error('Base config must have a single output object');
-  }
-
-  const baseOutputDir = baseOutput.dir;
-  if (typeof baseOutputDir !== 'string') {
-    throw new Error('Base config must have a string for dir');
-  }
 
   const variantSpecificConfigs = [];
 
   if (emitCjs) {
     if (splitDevProd) {
-      variantSpecificConfigs.push({ output: { format: 'cjs', dir: path.join(baseOutputDir, 'cjs/dev') } });
+      variantSpecificConfigs.push({ output: { format: 'cjs', dir: path.join(baseConfig.output.dir, 'cjs/dev') } });
       variantSpecificConfigs.push({
-        output: { format: 'cjs', dir: path.join(baseOutputDir, 'cjs/prod') },
+        output: { format: 'cjs', dir: path.join(baseConfig.output.dir, 'cjs/prod') },
         plugins: [makeProductionReplacePlugin()],
       });
     } else {
-      variantSpecificConfigs.push({ output: { format: 'cjs', dir: path.join(baseOutputDir, 'cjs') } });
+      variantSpecificConfigs.push({ output: { format: 'cjs', dir: path.join(baseConfig.output.dir, 'cjs') } });
     }
   }
 
@@ -126,22 +117,22 @@ export function makeNPMConfigVariants(baseConfig, options = {}) {
       variantSpecificConfigs.push({
         output: {
           format: 'esm',
-          dir: path.join(baseOutputDir, 'esm/dev'),
+          dir: path.join(baseConfig.output.dir, 'esm/dev'),
           plugins: [makePackageNodeEsm()],
         },
       });
       variantSpecificConfigs.push({
         output: {
           format: 'esm',
-          dir: path.join(baseOutputDir, 'esm/prod'),
+          dir: path.join(baseConfig.output.dir, 'esm/prod'),
           plugins: [makeProductionReplacePlugin(), makePackageNodeEsm()],
         },
       });
     } else {
       variantSpecificConfigs.push({
         output: {
           format: 'esm',
-          dir: path.join(baseOutputDir, 'esm'),
+          dir: path.join(baseConfig.output.dir, 'esm'),
           plugins: [makePackageNodeEsm()],
         },
       });