overhaul tsc checking mechanism, nx is not behaving https://app.circleci.com/pipelines/github/storybookjs/storybook/113873/workflows/cac49e34-2f09-410b-9075-39d3bf5da951/jobs/1058783

Author: ndelangen

scripts/check/check-package.ts

@@ -1,3 +1,4 @@
+import { existsSync } from 'node:fs';
 import { isAbsolute } from 'node:path';
 import { parseArgs } from 'node:util';
 
@@ -19,15 +20,17 @@ const normalizedCwd = cwd ? (isAbsolute(cwd) ? cwd : join(ROOT_DIRECTORY, cwd))
 
 const tsconfigPath = join(normalizedCwd, 'tsconfig.json');
 
-const { options, fileNames } = getTSFilesAndConfig(tsconfigPath, normalizedCwd);
-const { program, host } = getTSProgramAndHost(fileNames, options);
-
-const tsDiagnostics = getTSDiagnostics(program, normalizedCwd, host);
-if (tsDiagnostics.length > 0) {
-  console.log(tsDiagnostics);
-  process.exit(1);
-} else if (!process.env.CI) {
-  console.log('✅ No type errors');
+if (existsSync(tsconfigPath)) {
+  const { options, fileNames } = getTSFilesAndConfig(tsconfigPath, normalizedCwd);
+  const { program, host } = getTSProgramAndHost(fileNames, options);
+
+  const tsDiagnostics = getTSDiagnostics(program, normalizedCwd, host);
+  if (tsDiagnostics.length > 0) {
+    console.log(tsDiagnostics);
+    process.exit(1);
+  } else if (!process.env.CI) {
+    console.log('✅ No type errors');
+  }
 }
 
 // TODO, add more package checks here, like:

scripts/tasks/check.ts

@@ -1,31 +1,62 @@
-import type { Task } from '../task';
-import { ROOT_DIRECTORY } from '../utils/constants';
-import { exec } from '../utils/exec';
-import { maxConcurrentTasks } from '../utils/maxConcurrentTasks';
-
-// The amount of VCPUs for the check task on CI is 4 (large resource)
+import { join } from 'node:path';
 
-const parallel = process.env.CI ? '' : `--parallel=${maxConcurrentTasks}`;
+// eslint-disable-next-line depend/ban-dependencies
+import { execaCommand } from 'execa';
 
-const linkCommand = `yarn nx run-many -t check ${parallel}`;
-const nolinkCommand = `yarn nx run-many -t check -c production ${parallel}`;
+import type { Task } from '../task';
+import { CODE_DIRECTORY, ROOT_DIRECTORY } from '../utils/constants';
+import { getCodeWorkspaces } from '../utils/workspace';
 
 export const check: Task = {
   description: 'Typecheck the source code of the monorepo',
   async ready() {
     return false;
   },
-  async run(_, { dryRun, debug, link, skipCache }) {
-    const command = link ? linkCommand : nolinkCommand;
-    return exec(
-      `${command} ${skipCache || process.env.CI ? '--skip-nx-cache' : ''}`,
-      { cwd: ROOT_DIRECTORY },
-      {
-        startMessage: '🥾 Checking for TS errors',
-        errorMessage: '❌ TS errors detected',
-        dryRun,
-        debug,
+  async run(_, {}) {
+    const failed: string[] = [];
+    // const command = link ? linkCommand : nolinkCommand;
+    const workspaces = await getCodeWorkspaces();
+
+    for (const workspace of workspaces) {
+      if (workspace.location === '.') {
+        continue; // skip root directory
+      }
+      const cwd = join(CODE_DIRECTORY, workspace.location);
+      console.log('');
+      console.log('Checking ' + workspace.name + ' at ' + cwd);
+
+      let command = '';
+      if (workspace.name === '@storybook/vue3') {
+        command = `npx vue-tsc --noEmit --project ${join(cwd, 'tsconfig.json')}`;
+      } else if (workspace.name === '@storybook/svelte') {
+        command = `npx svelte-check`;
+      } else {
+        const script = join(ROOT_DIRECTORY, 'scripts', 'check', 'check-package.ts');
+        command = `yarn exec jiti ${script} --cwd ${cwd}`;
+        // command = `npx tsc --noEmit --project ${join(cwd, 'tsconfig.json')}`;
       }
-    );
+
+      const sub = execaCommand(`${command}`, {
+        cwd,
+        env: {
+          NODE_ENV: 'production',
+        },
+      });
+
+      sub.stdout?.on('data', (data) => {
+        process.stdout.write(data);
+      });
+      sub.stderr?.on('data', (data) => {
+        process.stderr.write(data);
+      });
+
+      await sub.catch((error) => {
+        failed.push(workspace.name);
+      });
+    }
+
+    if (failed.length > 0) {
+      throw new Error(`Failed to check ${failed.join(', ')}`);
+    }
   },
 };