docs(ci): fix stale vitest.config.ts comment about lib/cli-error.js coverage - #7472
Conversation
…overage 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 JSONbored#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 JSONbored#7435
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7472 +/- ##
=======================================
Coverage 91.36% 91.36%
=======================================
Files 717 717
Lines 73016 73016
Branches 21628 21631 +3
=======================================
Hits 66713 66713
Misses 5265 5265
Partials 1038 1038
Flags with carried forward coverage won't be shown. Click here to find out more. |
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-20 11:34:41 UTC
Review summary Nits — 3 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Linked issue satisfactionAddressed Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
Closes #7435
What
vitest.config.ts's comment above thepackages/loopover-mcp/lib/**/*.jscoverage.includeentry claimedlib/cli-error.jshad no in-process test. That was true when written (80b59c64), but #7409 (merged 2026-07-20) addedtest/unit/mcp-cli-error.test.ts, which importscli-error.jsin-process and exercises all three of its exports. The comment was never updated, so it now tells a contributor something false about the package's test-coverage state.This corrects the sentence to reflect that all five of the package's
lib/files (format-table/local-branch/redact-local-path/telemetry/cli-error) now have in-process tests.Scope
coverage.include/coverage.excludeglob or any other behavior changed (verified: the diff is 4 comment lines).