fix: Use ts-node rather than bash for scripting integration bundling (#5789)

Author: timfish

packages/integrations/.eslintrc.js

@@ -1,3 +1,11 @@
 module.exports = {
   extends: ['../../.eslintrc.js'],
+  overrides: [
+    {
+      files: ['scripts/**/*.ts'],
+      parserOptions: {
+        project: ['../../tsconfig.dev.json'],
+      },
+    },
+  ],
 };

packages/integrations/package.json

@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "run-p build:rollup build:types build:bundle",
-    "build:bundle": "yarn ts-node ../../scripts/ensure-bundle-deps.ts && bash scripts/buildBundles.sh",
+    "build:bundle": "ts-node ../../scripts/ensure-bundle-deps.ts && ts-node scripts/buildBundles.ts --parallel",
     "build:dev": "run-p build:rollup build:types",
     "build:rollup": "rollup -c rollup.npm.config.js",
     "build:types": "tsc -p tsconfig.types.json",

packages/integrations/scripts/buildBundles.sh

@@ -0,0 +1,58 @@
+import { spawn } from 'child_process';
+import { readdirSync } from 'fs';
+import { join } from 'path';
+
+const runParallel = process.argv.includes('--parallel');
+
+/** Gets a list of src filenames, one for each integration and excludes the index.ts */
+function getIntegrations(): string[] {
+  const srcDir = join(__dirname, '..', 'src');
+  const srcFiles = readdirSync(srcDir);
+  // The index file is only there for the purposes of npm builds
+  // (for the CDN we create a separate bundle for each integration)
+  return srcFiles.filter(file => file !== 'index.ts');
+}
+
+/** Builds a bundle for a specific integration and JavaScript ES version */
+async function buildBundle(integration: string, jsVersion: string): Promise<void> {
+  return new Promise((resolve, reject) => {
+    const child = spawn('yarn', ['--silent', 'rollup', '--config', 'rollup.bundle.config.js'], {
+      env: { ...process.env, INTEGRATION_FILE: integration, JS_VERSION: jsVersion },
+    });
+
+    child.on('exit', exitcode => {
+      if (exitcode !== 0) {
+        reject(new Error(`Failed to build bundle for integration "${integration}" with exit code: ${exitcode}`));
+      } else {
+        resolve();
+      }
+    });
+  });
+}
+
+if (runParallel) {
+  // We're building a bundle for each integration and each JavaScript version.
+  const tasks = getIntegrations().reduce(
+    (tasks, integration) => [...tasks, buildBundle(integration, 'ES5'), buildBundle(integration, 'ES6')],
+    [] as Promise<void>[],
+  );
+
+  Promise.all(tasks)
+    // eslint-disable-next-line no-console
+    .then(_ => console.log('\nIntegration bundles built successfully'))
+    .catch(error => {
+      // eslint-disable-next-line no-console
+      console.error(error);
+      // Important to exit with a non-zero exit code, so that the build fails.
+      process.exit(1);
+    });
+} else {
+  void (async () => {
+    for (const integration of getIntegrations()) {
+      await buildBundle(integration, 'ES5');
+      await buildBundle(integration, 'ES6');
+    }
+    // eslint-disable-next-line no-console
+    console.log('\nIntegration bundles built successfully');
+  })();
+}