fixed test errors

Author: zho

.github/workflows/ci.yml

@@ -13,7 +13,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node-version: [18.x, 20.x, 22.x]
+        # Do not add 18.x: Vitest 4 → rolldown imports `styleText` from `node:util` (Node >=20.12 only).
+        # `@vitest/coverage-v8` needs `node:inspector/promises` (Node >=19). Matches `package.json` engines.
+        node-version: [20.x, 22.x]
 
     steps:
       - uses: actions/checkout@v6
@@ -42,15 +44,9 @@ jobs:
       - name: Smoke test CLI
         run: node dist/index.js --help
 
-      # @vitest/coverage-v8 imports node:inspector/promises (Node 19+). Keep 18.x in the matrix for runtime smoke but skip coverage there.
       - name: Run tests with coverage
-        if: matrix.node-version != '18.x'
         run: npm run test:coverage
 
-      - name: Run tests (Node 18, no coverage)
-        if: matrix.node-version == '18.x'
-        run: npm test
-
       - name: Generate CycloneDX SBOM (lockfile-based)
         run: npx --yes @cyclonedx/cyclonedx-npm --package-lock-only --output-file sbom.cdx.json
 

CHANGELOG.md

@@ -17,7 +17,7 @@ Tagged releases are published to npm from GitHub Actions when a **GitHub Release
 - `--version` CLI flag prints the package version and exits.
 - `list_namespaces` response now includes `expires_at_iso` so clients see the cache expiry as an ISO-8601 timestamp without converting `cache_ttl_seconds`.
 - `examples/README.md` describing the library embedding sample.
-- GitHub Actions **CI** on **Ubuntu** with a **Node.js** matrix (**18.x**, **20.x**, **22.x**): typecheck, lint, Prettier, build, tests, **CycloneDX** SBOM artifact upload (per Node version), **Codecov** upload (Node **20.x** only), plus a separate **quality** job (`npm audit`, `npm pack --dry-run`).
+- GitHub Actions **CI** on **Ubuntu** with a **Node.js** matrix (**20.x**, **22.x**): typecheck, lint, Prettier, build, `test:coverage`, **CycloneDX** SBOM artifact upload (per Node version), **Codecov** upload (Node **20.x** only), plus a separate **quality** job (`npm audit`, `npm pack --dry-run`).
 - `npm run test:coverage` with Vitest coverage thresholds (see `vitest.config.ts`).
 - `@vitest/coverage-v8` devDependency for coverage reports (`lcov`, `json-summary`, HTML).
 
@@ -34,7 +34,7 @@ Tagged releases are published to npm from GitHub Actions when a **GitHub Release
 - README Slack URL example now matches the generator output (`https://app.slack.com/client/{team_id}/{channel_id}/p{messageId}`).
 - 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.
 - `npm run ci` now runs `test:coverage` so merges are gated on coverage thresholds.
-- 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**).
+- **Breaking (runtime / tooling):** `engines.node` is now **>=20.12.0**. Vitest **4** (bundled **rolldown**) imports `util.styleText` from `node:util` (added in Node **20.12**), and **`@vitest/coverage-v8`** uses `node:inspector/promises` (Node **≥19**). CI tests only **20.x** and **22.x**.
 - Dependabot groups related **vitest**, **typescript-eslint**, and **eslint/prettier** updates.
 
 ### Removed

README.md

@@ -33,6 +33,8 @@ A Model Context Protocol (MCP) server that provides semantic search over Pinecon
 
 ## Installation
 
+**Node.js [20.12](https://nodejs.org/en/download) or later** is required (`engines` in `package.json`).
+
 ### As a Package
 
 ```bash

package-lock.json

@@ -42,7 +42,7 @@
   },
   "homepage": "https://github.com/CppDigest/pinecone-read-only-mcp-typescript#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=20.12.0"
   },
   "scripts": {
     "clean": "node -e \"const fs=require('fs'); fs.rmSync('dist',{recursive:true,force:true});\"",