fix(SDK-6463): robust TS config compile in NX monorepos + tolerate a11y afterEach scan timeouts

[Bhargavi-BS]

SDK-6463 — Cypress test-count / status inconsistencies (customer: Sapiens, NX monorepo)

Fixes two client-side browserstack-cypress-cli defects surfaced by SDK-6463. The customer's reported dashboard count discrepancy is largely not a CLI bug (see RCA below); these two fixes address the genuine CLI defects: silently broken parallelization in NX monorepos, and accessibility-hook timeouts skipping tests.

RCA — the "inconsistency" is three layers

• A. Test Observability rendering (grep-filtered tests shown as "Pending", timeout placeholders, new-dashboard counting) — faithful; owned by the TO/dashboard team, not the CLI.
• B. Client-side test selection (@cypress/grep tags + browserstack.json exclude globs) — expected behavior.
• C. Two real CLI defects — fixed here.

C.1 — TS cypress-config compile fails silently in NX monorepos → parallelization collapses

readCypressConfigUtil.convertTsConfig compiles the TS config to read it. In an NX/monorepo:

• the extends temp tsconfig inherited base options (noEmit / emitDeclarationOnly / composite / noEmitOnError) that suppress or redirect the compiled JS, and
• any tsc type error made tsc exit non-zero, which short-circuited the &&-chained tsc-alias, leaving path aliases (e.g. @org/lib) un-rewritten so the compiled config couldn't be require()d.

The read error was then silently swallowed (readCypressConfigFile catch) → cypressConfigFile = {} → getNumberOfSpecFiles fell back to the default cypress/e2e/** glob → 0 specs → setParallels forced parallels to browserCombinations.length. This is the customer's Parallels limit specified: 1 (despite parallels: 3) and the repeated Cypress config file not found at: …tmpBstackCompiledJs/… errors.

Fix (bin/helpers/readCypressConfigUtil.js):

• Force emit-friendly overrides in the extends temp tsconfig: noEmit:false, emitDeclarationOnly:false, composite:false, declaration:false, declarationMap:false, noEmitOnError:false, incremental:false. These only affect the throwaway temp tsconfig used to read the config — not the user's real build.
• Run tsc-alias unconditionally (& on Windows / ; on Unix instead of &&) so a tsc type error doesn't leave aliases un-rewritten. convertTsConfig already tolerates tsc errors by parsing the emitted-files output.

C.2 — accessibility afterEach scan/save timeout skips the rest of the spec

bin/accessibility-automation/cypress/index.js calls cy.wrap(performScan(win), {timeout:30000}) and cy.wrap(saveTestResults(win, …), {timeout:30000}) in the global afterEach with no rejection handling. When the in-page scanner doesn't echo A11Y_SCAN_FINISHED / A11Y_RESULTS_SAVED, the wrap times out, the afterEach hook fails, and Cypress skips all remaining tests in the spec — the ticket's "cy.wrap() timed out … Because this error occurred during a after each hook we are skipping all of the remaining tests" and "should have failed but got skipped".

Fix: add .catch to both wrap chains so a hung scan/save is logged, not cascaded into skipped tests.

Validation

C.1 — real BrowserStack infra (same project & config, only the CLI differs):

	Unfixed CLI	Fixed CLI
Cypress config file not found errors	2	0
Parallels limit specified (config has parallels: 3)	1	3
Build	7493f8339065f0570cd0867d7bec599487bb6dd9	bba6009df5e214058e1567f17ac268ea312675e0

C.1 — local NX tsconfig matrix: noEmit / emitDeclarationOnly / composite / noEmitOnError+type-error all fail to read the config before the fix and read OK after it.

C.2 — local simulation: drives the real plugin afterEach with a hung scanner; the new regression test fails without the fix and passes with it across hung-scan, hung-save, and happy-path scenarios.

Unit suite: 685 passing (was 678 → +7 new tests), 0 new failures (13 failures pre-exist on a clean tree and are unrelated).

Files

• bin/helpers/readCypressConfigUtil.js — C.1 fix
• bin/accessibility-automation/cypress/index.js — C.2 fix
• test/unit/bin/helpers/readCypressConfigUtil.js — +3 C.1 regression tests
• test/unit/bin/accessibility-automation/cypress/index.js — new, +4 C.2 regression tests

Notes / follow-ups

• The dashboard count discrepancy (Layer A) should be routed to the Test Observability/dashboard team — it is not addressed here.
• Latent bug spotted while investigating C.2: index.js:292 references an undefined includeTags (should be includeTagArray), reachable only when accessibility include/exclude tags are set. Not fixed here; worth a separate ticket.

🤖 Generated with Claude Code