@tech-leads-club/harness-toolkit 0.15.2 → 0.15.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/bin/tlc-cli.ts CHANGED
@@ -1342,7 +1342,15 @@ export function buildTestSteps(): TestStep[] {
1342
1342
  // `error` in biome.json was measured instead and rejected: it enables each group's non-recommended rules too,
1343
1343
  // which produced 3763 findings and included `noBarrelFile` and `noReExportAll` — the two rules that forbid the
1344
1344
  // core facade this architecture is built on ([/decisions/ad-004.md](/decisions/ad-004.md)) — and `noNodejsModules` in a Node CLI.
1345
- { label: "biome check", bin: "npx", args: ["biome", "check", "--error-on-warnings"] },
1345
+ // why: `--max-diagnostics=none`. Biome defaults to showing 20, and an `info`-level rule with a standing count
1346
+ // above that (`noExcessiveCognitiveComplexity`, at `info` so it does not block on pre-existing debt — see
1347
+ // `check-complexity.ts`) can fill the whole budget, leaving a real `error` finding unprinted though it still
1348
+ // fails the step ([/decisions/ad-139.md](/decisions/ad-139.md)).
1349
+ {
1350
+ label: "biome check",
1351
+ bin: "npx",
1352
+ args: ["biome", "check", "--error-on-warnings", "--max-diagnostics=none"],
1353
+ },
1346
1354
  { label: "tsc --noEmit", bin: "npx", args: ["tsc", "--noEmit"] },
1347
1355
  { label: "src suite", bin: "node", args: [...TEST_ENV_IMPORT, "--test", "src/**/__test__/*.test.ts"] },
1348
1356
  { label: "tools suite", bin: "node", args: [...TEST_ENV_IMPORT, "--test", "tools/__test__/*.test.ts"] },
@@ -1363,6 +1371,10 @@ export function buildTestSteps(): TestStep[] {
1363
1371
  args: ["knip", "--exports", "--max-issues", String(KNIP_EXPORTS_CEILING)],
1364
1372
  },
1365
1373
  { label: "check-boundaries", bin: "node", args: ["tools/dev/check-boundaries.ts"] },
1374
+ // why: `noExcessiveCognitiveComplexity` runs at `info` in biome.json, so step 1 above never fails on it —
1375
+ // this is the ceiling that catches growth instead, the same shape as the knip step above
1376
+ // ([/decisions/ad-139.md](/decisions/ad-139.md)).
1377
+ { label: "check-complexity", bin: "node", args: ["tools/dev/check-complexity.ts"] },
1366
1378
  // why: `--error-on-warnings` above cannot see a rule that was suppressed rather than fixed, and biome accepts
1367
1379
  // any text after the colon. This is what makes the reason a reason ([/decisions/ad-051.md](/decisions/ad-051.md)).
1368
1380
  { label: "check-suppressions", bin: "node", args: ["tools/dev/check-suppressions.ts"] },