Fix CI failures: update Node.js matrix and test configuration

- Drop Node.js 14.x and 16.x from CI matrix (pnpm 10 requires >= 18.12,
  vitest 4 / vite 7 requires >= 20)
- Test on Node.js 20.x and 22.x instead
- Move e2e test exclusion from global vitest config to per-package test
  scripts so that test-e2e commands still work
- Fix get-uri file descriptor GC error by passing the FileHandle directly
  to createReadStream instead of extracting the raw fd

Author: TooTallNate

.github/workflows/test.yml

@@ -15,11 +15,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        node-version: [14.x, 16.x, 18.x, 20.x]
-        exclude:
-          # Node 14 is not supported on macOS arm64
-          - os: macos-latest
-            node-version: 14.x
+        node-version: [20.x, 22.x]
       fail-fast: false
 
     runs-on: ${{ matrix.os }}

packages/get-uri/src/file.ts

@@ -42,8 +42,6 @@ export const file: GetUriProtocol<FileOptions> = async (
 		// `open()` first to get a file descriptor and ensure that the file
 		// exists.
 		const fdHandle = await fsPromises.open(filepath, flags, mode);
-		// extract the numeric file descriptor
-		const fd = fdHandle.fd;
 
 		// store the stat object for the cache.
 		const stat = await fdHandle.stat();
@@ -54,12 +52,12 @@ export const file: GetUriProtocol<FileOptions> = async (
 			throw new NotModifiedError();
 		}
 
-		// `fs.ReadStream` takes care of calling `fs.close()` on the
-		// fd after it's done reading
+		// `fs.ReadStream` takes care of closing the file handle
+		// after it's done reading
 		const rs = createReadStream(filepath, {
 			autoClose: true,
 			...opts,
-			fd,
+			fd: fdHandle,
 		}) as FileReadable;
 		rs.stat = stat;
 		return rs;

packages/https-proxy-agent/package.json

@@ -14,7 +14,7 @@
   ],
   "scripts": {
     "build": "tsc",
-    "test": "vitest",
+    "test": "vitest --exclude test/e2e.test.ts",
     "test-e2e": "vitest run test/e2e.test.ts",
     "lint": "eslint --ext .ts",
     "pack": "node ../../scripts/pack.mjs"

packages/socks-proxy-agent/package.json

@@ -135,7 +135,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "vitest",
+    "test": "vitest --exclude test/e2e.test.ts",
     "test-e2e": "vitest run test/e2e.test.ts",
     "lint": "eslint . --ext .ts",
     "pack": "node ../../scripts/pack.mjs"

vitest.config.ts

@@ -5,6 +5,5 @@ export default defineConfig({
 		globals: true,
 		testTimeout: 10000,
 		include: ['**/*.{test,spec}.?(c|m)[jt]s?(x)', '**/test/test.[jt]s?(x)'],
-		exclude: ['**/node_modules/**', '**/.git/**', '**/e2e.test.*'],
 	},
 });