[irfn]

…uild

Related PR

hashicorp/terraform-cdk#2133

Description

This PR optimizes CI for the @cdktn/hcl2cdk package with three changes that together save ~16 min of aggregate CI compute per run:

1. Jest globalSetup / globalTeardown — New files (globalSetup.ts, globalTeardown.ts) that run once before any test worker starts. The setup pre-generates all 12 provider bindings, initializes base projects for required languages, and pre-caches provider schemas via CDKTF_EXPERIMENTAL_PROVIDER_SCHEMA_CACHE_PATH. This eliminates all terraform init / terraform providers schema calls from test workers, which was the root cause of cache collisions under parallelism.

2. Parallel test execution — Removed --runInBand from hcl2cdk test scripts and refactored the test helper (convert.ts) to read from a pre-generated fixtures manifest instead of running expensive cdktn init/cdktn get per-worker. This is now safe because all shared resources are generated once in globalSetup. The hcl2cdk test step drops from ~11m27s (serial) to ~7 min (parallel).

3. Scoped builds in CI — Replaced the full monorepo yarn build in .github/workflows/unit.yml with per-package lerna run --scope builds. Nx's build.dependsOn: ["^build"] automatically resolves transitive dependencies. A conditional step adds cdktn-cli build + yarn package only for the hcl2cdk job that needs the CLI binary. This saves ~35s per job across the 14 non-hcl2cdk jobs (~8 min aggregate).

The design choice to use a shared globalSetup with a JSON manifest (rather than trying to fix per-worker locking) was driven by the history here: parallel workers independently running cdktf get/cdktf init caused Terraform plugin cache collisions, and previous attempts to fix the parallelism (41a2e7f, f2e06fd) failed, leading to the --runInBand workaround (5ba985c). Pre-generating everything once sidesteps the problem entirely.

Verified locally: 27/28 suites passed (1 skipped), 279/280 tests passed (1 skipped), 202 snapshots matched, 0 Terraform cache collision failures.

Checklist

• I have updated the PR title to match [CDKTN's style guide](https://github.com/open-constructs/cdk-terrain/blob/main/CONTRIBUTING.md#pull-requests-1)
• I have run the linter on my code locally
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the [documentation](https://github.com/open-constructs/cdk-terrain/tree/main/website) if applicable
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works if applicable
• New and existing unit tests pass locally with my changes

[so0k]

LGTM - the CI is running and only some small concerns.

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.