Skip to content

Commit be2d97c

Browse files
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
1 parent fbd1cba commit be2d97c

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

vitest.config.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,10 @@ export default defineConfig({
5151
"packages/discovery-index/src/**/*.ts",
5252
// packages/loopover-mcp/lib/*.js are plain JS today; issue #7291 migrates them to real TypeScript,
5353
// keeping the same .js/.ts/.d.ts triplet shape packages/loopover-miner/lib/** already has (so
54-
// coverage is wired BEFORE that PR lands, not as a follow-up fix). 4 of the 5 files
55-
// (format-table/local-branch/redact-local-path/telemetry) are already imported in-process by
56-
// test/unit/*.test.ts; lib/cli-error.js currently has no in-process test at all, so it will need
57-
// one before a PR touching it can pass codecov/patch -- that's intended enforcement, not a bug.
54+
// coverage is wired BEFORE that PR lands, not as a follow-up fix). All 5 files
55+
// (format-table/local-branch/redact-local-path/telemetry/cli-error) are now imported in-process
56+
// by test/unit/*.test.ts (cli-error's own test/unit/mcp-cli-error.test.ts landed in #7409), so a
57+
// PR touching any of them is covered by codecov/patch -- that's intended enforcement, not a bug.
5858
"packages/loopover-mcp/lib/**/*.js",
5959
"packages/loopover-mcp/lib/**/*.ts",
6060
// packages/loopover-mcp/bin/loopover-mcp.js (~6,600 of ~7,400 lines in the package) is tested

0 commit comments

Comments
 (0)