From be2d97c23153165d61beacb192da2075655c40b2 Mon Sep 17 00:00:00 2001 From: Lourince Daging Date: Mon, 20 Jul 2026 13:24:46 +0200 Subject: [PATCH] docs(ci): fix stale vitest.config.ts comment about lib/cli-error.js coverage The comment above the packages/loopover-mcp/lib/**/*.js coverage.include entry said lib/cli-error.js had no in-process test. That stopped being true when #7409 added test/unit/mcp-cli-error.test.ts, which imports cli-error.js in-process and exercises all three of its exports. Correct the sentence so it reflects that all five of the package's lib/ files now have in-process tests. Comment-only; no glob or behavior change. Closes #7435 --- vitest.config.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/vitest.config.ts b/vitest.config.ts index c43c00e315..2289653013 100644 --- a/vitest.config.ts +++ b/vitest.config.ts @@ -51,10 +51,10 @@ export default defineConfig({ "packages/discovery-index/src/**/*.ts", // packages/loopover-mcp/lib/*.js are plain JS today; issue #7291 migrates them to real TypeScript, // keeping the same .js/.ts/.d.ts triplet shape packages/loopover-miner/lib/** already has (so - // coverage is wired BEFORE that PR lands, not as a follow-up fix). 4 of the 5 files - // (format-table/local-branch/redact-local-path/telemetry) are already imported in-process by - // test/unit/*.test.ts; lib/cli-error.js currently has no in-process test at all, so it will need - // one before a PR touching it can pass codecov/patch -- that's intended enforcement, not a bug. + // coverage is wired BEFORE that PR lands, not as a follow-up fix). All 5 files + // (format-table/local-branch/redact-local-path/telemetry/cli-error) are now imported in-process + // by test/unit/*.test.ts (cli-error's own test/unit/mcp-cli-error.test.ts landed in #7409), so a + // PR touching any of them is covered by codecov/patch -- that's intended enforcement, not a bug. "packages/loopover-mcp/lib/**/*.js", "packages/loopover-mcp/lib/**/*.ts", // packages/loopover-mcp/bin/loopover-mcp.js (~6,600 of ~7,400 lines in the package) is tested