lower the number of parallel dts bundle generation at prod

Author: ndelangen

.github/workflows/trigger-circle-ci-workflow.yml

@@ -1,68 +1,26 @@
 name: Trigger CircleCI workflow
 
-on:
-  # Use pull_request_target, as we don't need to check out the actual code of the fork in this script.
-  # And this is the only way to trigger the Circle CI API on forks as well.
-  pull_request_target:
-    types: [opened, synchronize, labeled, reopened]
-  push:
-    branches:
-      - next
-      - main
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
+on: push
 jobs:
-  get-branch:
-    if: github.repository_owner == 'storybookjs'
+  test:
     runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: ./scripts
     steps:
-      - id: get-branch
-        env:
-          # Stored as environment variable to prevent script injection
-          REF_NAME: ${{ github.ref_name }}
-          PR_REF_NAME: ${{ github.event.pull_request.head.ref }}
-        run: |
-          if  [ "${{ github.event.pull_request.head.repo.fork }}" = "true" ]; then
-            export BRANCH=pull/${{ github.event.pull_request.number }}/head
-            elif [ "${{ github.event_name }}" = "push" ]; then
-            export BRANCH="$REF_NAME"
-            else
-            export BRANCH="$PR_REF_NAME"
-          fi
-          echo "$BRANCH"
-          echo "branch=$BRANCH" >> $GITHUB_ENV
-    outputs:
-      branch: ${{ env.branch }}
+      - name: Checkout ${{ github.ref_name }}
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 100
 
-  get-parameters:
-    if: github.repository_owner == 'storybookjs'
-    runs-on: ubuntu-latest
-    steps:
-      - if: github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'ci:normal'))
-        run: echo "workflow=normal" >> $GITHUB_ENV
-      - if: github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'ci:docs'))
-        run: echo "workflow=docs" >> $GITHUB_ENV
-      - if: github.event_name == 'push' || contains(github.event.pull_request.labels.*.name, 'ci:merged')
-        run: echo "workflow=merged" >> $GITHUB_ENV
-      - if: github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'ci:daily'))
-        run: echo "workflow=daily" >> $GITHUB_ENV
-    outputs:
-      workflow: ${{ env.workflow }}
-      ghBaseBranch: ${{ github.event.pull_request.base.ref }}
-      ghPrNumber: ${{ github.event.pull_request.number }}
+      - name: Setup Node.js and Install Dependencies
+        uses: ./.github/actions/setup-node-and-install
 
-  trigger-circle-ci-workflow:
-    runs-on: ubuntu-latest
-    needs: [get-branch, get-parameters]
-    if: github.repository_owner == 'storybookjs' && needs.get-parameters.outputs.workflow != ''
-    steps:
-      - name: Trigger Normal tests
-        uses: fjogeleit/http-request-action@v1
-        with:
-          url: 'https://circleci.com/api/v2/project/gh/storybookjs/storybook/pipeline'
-          method: 'POST'
-          customHeaders: '{"Content-Type": "application/json", "Circle-Token": "${{ secrets.CIRCLE_CI_TOKEN }}"}'
-          data: '{ "branch": "${{needs.get-branch.outputs.branch}}", "parameters": ${{toJson(needs.get-parameters.outputs)}} }'
+      - name: run compile
+        run: yarn task --task=compile --start-from=compile --no-link
+      - name: run compile
+        run: yarn task --task=compile --start-from=compile --no-link
+      - name: run compile
+        run: yarn task --task=compile --start-from=compile --no-link
+      - name: run compile
+        run: yarn task --task=compile --start-from=compile --no-link

scripts/build/utils/generate-types.ts

@@ -21,7 +21,7 @@ export async function generateTypesFiles(cwd: string, data: BuildEntries) {
   // ...this way we do not bog down the main process/esbuild and can run them in parallel
   // we limit the number of concurrent processes to 3, because we don't want to overload the host machine
   // by trial and error, 3 seems to be the sweet spot between perf and consistency
-  const limited = limit(10);
+  const limited = limit(5);
   let processes: ReturnType<typeof spawn>[] = [];
 
   await Promise.all(
@@ -83,7 +83,7 @@ export async function generateTypesFiles(cwd: string, data: BuildEntries) {
           processes.forEach((p) => p.kill());
           processes = [];
           process.exit(dtsProcess.exitCode || 1);
-        } else if(!process.env.CI) {
+        } else if (!process.env.CI) {
           console.log('✅ Generated types for', picocolors.cyan(join(DIR_REL, entryPoint)));
         }
       });