build: add prebuild command support

Author: michaelfaith

fixtures/eslint-v6/build.mjs

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 
-import {exec} from 'node:child_process';
+import {execSync} from 'node:child_process';
 import {dirname, resolve} from 'node:path';
 import {fileURLToPath} from 'node:url';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
-exec('yarn build -r stable eslint-plugin-react-hooks', {
+execSync('yarn build -r stable eslint-plugin-react-hooks', {
   cwd: resolve(__dirname, '..', '..'),
+  stdio: 'inherit',
 });

fixtures/eslint-v7/build.mjs

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 
-import {exec} from 'node:child_process';
+import {execSync} from 'node:child_process';
 import {dirname, resolve} from 'node:path';
 import {fileURLToPath} from 'node:url';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
-exec('yarn build -r stable eslint-plugin-react-hooks', {
+execSync('yarn build -r stable eslint-plugin-react-hooks', {
   cwd: resolve(__dirname, '..', '..'),
+  stdio: 'inherit',
 });

fixtures/eslint-v8/build.mjs

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 
-import {exec} from 'node:child_process';
+import {execSync} from 'node:child_process';
 import {dirname, resolve} from 'node:path';
 import {fileURLToPath} from 'node:url';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
-exec('yarn build -r stable eslint-plugin-react-hooks', {
+execSync('yarn build -r stable eslint-plugin-react-hooks', {
   cwd: resolve(__dirname, '..', '..'),
+  stdio: 'inherit',
 });

fixtures/eslint-v9/build.mjs

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 
-import {exec} from 'node:child_process';
+import {execSync} from 'node:child_process';
 import {dirname, resolve} from 'node:path';
 import {fileURLToPath} from 'node:url';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
 
-exec('yarn build -r stable eslint-plugin-react-hooks', {
+execSync('yarn build -r stable eslint-plugin-react-hooks', {
   cwd: resolve(__dirname, '..', '..'),
+  stdio: 'inherit',
 });

packages/eslint-plugin-react-hooks/package.json

@@ -21,7 +21,6 @@
     "react"
   ],
   "scripts": {
-    "dependencies": "cd ../../compiler/packages/babel-plugin-react-compiler && yarn && yarn tsc -b",
     "test": "jest",
     "typecheck": "tsc --noEmit"
   },

scripts/rollup/build.js

@@ -12,6 +12,7 @@ const stripBanner = require('rollup-plugin-strip-banner');
 const chalk = require('chalk');
 const resolve = require('@rollup/plugin-node-resolve').nodeResolve;
 const fs = require('fs');
+const childProcess = require('child_process');
 const argv = require('minimist')(process.argv.slice(2));
 const Modules = require('./modules');
 const Bundles = require('./bundles');
@@ -829,6 +830,11 @@ function handleRollupError(error) {
   }
 }
 
+function runShellCommand(command) {
+  console.log(chalk.dim('Running: ') + chalk.cyan(command));
+  childProcess.execSync(command, {stdio: 'inherit', shell: true});
+}
+
 async function buildEverything() {
   if (!argv['unsafe-partial']) {
     await asyncRimRaf('build');
@@ -876,6 +882,9 @@ async function buildEverything() {
 
   // eslint-disable-next-line no-for-of-loops/no-for-of-loops
   for (const [bundle, bundleType] of bundles) {
+    if (bundle.prebuild) {
+      runShellCommand(bundle.prebuild);
+    }
     await createBundle(bundle, bundleType);
   }
 

scripts/rollup/bundles.js

@@ -1197,6 +1197,8 @@ const bundles = [
     wrapWithModuleBoundaries: false,
     externals: [],
     tsconfig: './packages/eslint-plugin-react-hooks/tsconfig.json',
+    prebuild:
+      'cd compiler/packages/babel-plugin-react-compiler && yarn && yarn tsc -b',
   },
 
   /******* React Fresh *******/