From f5e70950d38f9879720fb27614d3c53136bc9742 Mon Sep 17 00:00:00 2001 From: Benson K B Date: Tue, 1 Sep 2026 16:07:06 +0530 Subject: [PATCH] chore(ci): point the circular-deps gate at dpdm's real JS entrypoint (#11615) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `node_modules/.bin/dpdm` is an npm shell wrapper, so `node ` crashed with `SyntaxError: missing ) after argument list` and the advisory circular-deps gate in ci.yml (job quality-extended) reported an error instead of a result on every run. Pointing DPDM_BIN at `node_modules/dpdm/lib/bin/dpdm.js` restores it: the gate now completes and reports circularDeps=154 (exit 0). Scope reduced during merge — the branch was 522 commits behind and carried three base-drift files that were reconciled back to the release tip: open-sse/services/combo.ts (reverted routing code + a @/lib/localDb barrel import, Hard Rule #2), config/quality/eslint-suppressions.json (dropped ~45% of the frozen suppressions), and tests/unit/cli-env-inline-comment-10100.test.ts (replaced a working module import with new Function() source scraping, Hard Rule #3). Rationale documented in the PR discussion. Verified: check:circular-deps crashes on the pure tip and completes on the merged branch; tests/unit/cli-env-inline-comment-10100.test.ts 5/5 green against the restored version. Thanks @benzntech for catching the dpdm breakage. --- scripts/check/check-circular-deps.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/check/check-circular-deps.mjs b/scripts/check/check-circular-deps.mjs index 3d922e1b2d..e2ce45ef4b 100644 --- a/scripts/check/check-circular-deps.mjs +++ b/scripts/check/check-circular-deps.mjs @@ -40,7 +40,7 @@ const ENTRYPOINTS = [ "src/lib/db/core.ts", ]; -const DPDM_BIN = resolve(projectRoot, "node_modules/.bin/dpdm"); +const DPDM_BIN = resolve(projectRoot, "node_modules/dpdm/lib/bin/dpdm.js"); const TSCONFIG = resolve(projectRoot, "tsconfig.json"); /**