Skip to content

Commit c2bee1f

Browse files
author
zho
committed
fixed test errors
1 parent aaf1fdf commit c2bee1f

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,13 @@ jobs:
4242
- name: Smoke test CLI
4343
run: node dist/index.js --help
4444

45+
# @vitest/coverage-v8 imports node:inspector/promises (Node 19+). Keep 18.x in the matrix for runtime smoke but skip coverage there.
4546
- name: Run tests with coverage
47+
if: matrix.node-version != '18.x'
4648
run: npm run test:coverage
4749

48-
- name: Run tests (no coverage)
50+
- name: Run tests (Node 18, no coverage)
51+
if: matrix.node-version == '18.x'
4952
run: npm test
5053

5154
- name: Generate CycloneDX SBOM (lockfile-based)

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ Future releases are managed automatically by [release-please](https://github.com
3636
- README Slack URL example now matches the generator output (`https://app.slack.com/client/{team_id}/{channel_id}/p{messageId}`).
3737
- README "Comparison with Python Version" no longer claims an identical API interface; the new TypeScript-only tools (`guided_query`, `query_documents`, `keyword_search`, `namespace_router`, `suggest_query_params`, `count`, `generate_urls`) are listed explicitly.
3838
- `npm run ci` now runs `test:coverage` so merges are gated on coverage thresholds.
39+
- CI: on Node **18.x**, run `npm test` only; **`test:coverage`** runs on **20.x** / **22.x** because Vitest V8 coverage uses `node:inspector/promises` (Node **≥19**).
3940
- Dependabot groups related **vitest**, **typescript-eslint**, and **eslint/prettier** updates.
4041

4142

0 commit comments

Comments
 (0)