chore: remove undici, set node to >=18.17

The node engines "^18.17.0 || ^20.3.0 || >=21.0.0" are taken from the `sharp` dependency, which probably has good reason for them. All of these include global fetch, so undici is not needed. Bun and deno also include fetch.
QwikDev · Feb 4, 2024 · 344a776 · 344a776
1 parent c55c1eb
commit 344a776
Show file tree

Hide file tree

Showing 32 changed files with 23 additions and 159 deletions.
diff --git a/.github/ISSUE_TEMPLATE/bug.yaml b/.github/ISSUE_TEMPLATE/bug.yaml
@@ -53,7 +53,7 @@ body:
     id: system-info
     attributes:
       label: System Info
-      description: Output of `npx envinfo --system --npmPackages '{vite,undici,@builder.io/*}' --binaries --browsers`
+      description: Output of `npx envinfo --system --npmPackages '{vite,@builder.io/*}' --binaries --browsers`
       render: shell
       placeholder: System, Binaries, Browsers
     validations:

diff --git a/package.json b/package.json
@@ -91,7 +91,6 @@
     "terser": "^5.27.0",
     "tsm": "^2.3.0",
     "typescript": "5.3.3",
-    "undici": "*",
     "vfile": "^6.0.1",
     "vite": "^5.0.12",
     "vite-imagetools": "^6.2.9",
@@ -102,7 +101,7 @@
     "zod": "^3.22.4"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11",
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0",
     "npm": "please-use-pnpm",
     "yarn": "please-use-pnpm",
     "pnpm": ">=8.6.12"

diff --git a/packages/create-qwik/package.json b/packages/create-qwik/package.json
@@ -12,8 +12,7 @@
     "yargs": "17.7.2"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11",
-    "npm": ">=6.0.0"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "files": [
     "create-qwik.cjs",

diff --git a/packages/docs/contributors.ts b/packages/docs/contributors.ts
@@ -1,5 +1,4 @@
 /* eslint-disable no-console */
-import { fetch } from 'undici';
 import fs from 'node:fs';
 import path from 'node:path';
 import url from 'node:url';

diff --git a/packages/docs/package.json b/packages/docs/package.json
@@ -46,14 +46,13 @@
     "tailwindcss": "^3.4.1",
     "tsm": "^2.3.0",
     "typescript": "5.3.3",
-    "undici": "*",
     "valibot": "^0.25.0",
     "vite": "^5.0.12",
     "vite-plugin-inspect": "^0.8.1",
     "wrangler": "^3.22.4"
   },
   "engines": {
-    "node": ">=18.11",
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0",
     "npm": "please-use-pnpm",
     "yarn": "please-use-pnpm",
     "pnpm": ">=8.6.12"

diff --git a/packages/docs/scripts/showcase.js b/packages/docs/scripts/showcase.js
@@ -133,7 +133,6 @@ async function captureMultipleScreenshots() {
 }
 
 async function getPagespeedData(url) {
-  const { fetch } = await import('undici');
   const requestURL = `https://www.googleapis.com/pagespeedonline/v5/runPagespeed?url=${encodeURIComponent(
     url
   )}&key=AIzaSyApBC9gblaCzWrtEBgHnZkd_B37OF49BfM&category=PERFORMANCE&strategy=MOBILE`;

diff --git a/packages/eslint-plugin-qwik/package.json b/packages/eslint-plugin-qwik/package.json
@@ -16,7 +16,7 @@
     "redent": "^4.0.0"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "homepage": "https://github.com/BuilderIO/qwik#readme",
   "keywords": [

diff --git a/packages/insights/package.json b/packages/insights/package.json
@@ -34,13 +34,12 @@
     "prettier-plugin-tailwindcss": "^0.5.11",
     "tailwindcss": "^3.4.1",
     "typescript": "5.3.3",
-    "undici": "*",
     "vite": "^5.0.12",
     "vite-tsconfig-paths": "^4.3.1",
     "vitest": "^1.2.1"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "private": true,
   "scripts": {

diff --git a/packages/qwik-auth/package.json b/packages/qwik-auth/package.json
@@ -14,7 +14,7 @@
     "set-cookie-parser": "^2.6.0"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik-city/buildtime/vite/plugin.ts b/packages/qwik-city/buildtime/vite/plugin.ts
@@ -13,7 +13,6 @@ import { build } from '../build';
 import { ssrDevMiddleware, staticDistMiddleware } from './dev-server';
 import { transformMenu } from '../markdown/menu';
 import { generateQwikCityEntries } from '../runtime-generation/generate-entries';
-import { patchGlobalThis } from '../../middleware/node/node-fetch';
 import type { QwikVitePlugin } from '@builder.io/qwik/optimizer';
 import fs from 'node:fs';
 import {
@@ -42,9 +41,6 @@ function qwikCityPlugin(userOpts?: QwikCityVitePluginOptions): any {
   let ssrFormat: 'esm' | 'cjs' = 'esm';
   let outDir: string | null = null;
 
-  // Patch Stream APIs
-  patchGlobalThis();
-
   (globalThis as any).__qwikCityNew = true;
 
   const api: QwikCityPluginApi = {

diff --git a/packages/qwik-city/middleware/node/index.ts b/packages/qwik-city/middleware/node/index.ts
@@ -12,17 +12,13 @@ import { extname, join, basename } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { computeOrigin, fromNodeHttp, getUrl } from './http';
 import { MIME_TYPES } from '../request-handler/mime-types';
-import { patchGlobalThis } from './node-fetch';
 import { _deserializeData, _serializeData, _verifySerializable } from '@builder.io/qwik';
 import type { Http2ServerRequest } from 'node:http2';
 
 // @builder.io/qwik-city/middleware/node
 
 /** @public */
 export function createQwikCity(opts: QwikCityNodeRequestOptions) {
-  // Patch Stream APIs
-  patchGlobalThis();
-
   const qwikSerializer = {
     _deserializeData,
     _serializeData,

diff --git a/packages/qwik-city/middleware/node/node-fetch.ts b/packages/qwik-city/middleware/node/node-fetch.ts
diff --git a/packages/qwik-city/package.json b/packages/qwik-city/package.json
@@ -8,7 +8,6 @@
     "@types/mdx": "^2.0.8",
     "source-map": "0.7.4",
     "svgo": "^3.2.0",
-    "undici": "*",
     "vfile": "^6.0.1",
     "vite": "^5.0.12",
     "vite-imagetools": "^6.2.9",
@@ -44,7 +43,7 @@
     "yaml": "^2.3.2"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik-city/runtime/src/service-worker/utils.unit.ts b/packages/qwik-city/runtime/src/service-worker/utils.unit.ts
@@ -1,4 +1,3 @@
-import { Request as NodeRequest, Response as NodeResponse } from 'undici';
 import type { AppBundle } from './types';
 import { getCacheToDelete, isAppBundleRequest, useCache } from './utils';
 import { assert, test } from 'vitest';
@@ -78,9 +77,9 @@ test('useCache', () => {
 
 export function mockRequest(url?: string): Request {
   url = url || 'https://qwik.builder.io/';
-  return new NodeRequest(url) as any;
+  return new Request(url);
 }
 
 export function mockResponse(body?: any): Response {
-  return new NodeResponse(body) as any;
+  return new Response(body);
 }
diff --git a/packages/qwik-city/static/node/index.ts b/packages/qwik-city/static/node/index.ts
@@ -3,7 +3,6 @@ import { createSystem } from './node-system';
 import { isMainThread, workerData } from 'node:worker_threads';
 import { mainThread } from '../main-thread';
 import { workerThread } from '../worker-thread';
-import { patchGlobalThis } from 'packages/qwik-city/middleware/node/node-fetch';
 
 export async function generate(opts: StaticGenerateOptions) {
   if (isMainThread) {
@@ -17,8 +16,6 @@ export async function generate(opts: StaticGenerateOptions) {
 
 if (!isMainThread && workerData) {
   (async () => {
-    patchGlobalThis();
-
     // self initializing worker thread with workerData
     const sys = await createSystem(workerData);
     await workerThread(sys);

diff --git a/packages/qwik-city/static/node/node-system.ts b/packages/qwik-city/static/node/node-system.ts
@@ -2,15 +2,12 @@
 import type { StaticGenerateOptions, System } from '../types';
 import fs from 'node:fs';
 import { dirname, join } from 'node:path';
-import { patchGlobalThis } from '../../middleware/node/node-fetch';
 import { createNodeMainProcess } from './node-main';
 import { createNodeWorkerProcess } from './node-worker';
 import { normalizePath } from '../../utils/fs';
 
 /** @public */
 export async function createSystem(opts: StaticGenerateOptions) {
-  patchGlobalThis();
-
   const createWriteStream = (filePath: string) => {
     return fs.createWriteStream(filePath, {
       flags: 'w',

diff --git a/packages/qwik-labs/package.json b/packages/qwik-labs/package.json
@@ -13,11 +13,10 @@
     "np": "^8.0.4",
     "prettier": "^3.2.4",
     "typescript": "5.3.3",
-    "undici": "*",
     "vite": "^5.0.12"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik-react/package.json b/packages/qwik-react/package.json
@@ -13,7 +13,7 @@
     "vite": "^5.0.12"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik-worker/package.json b/packages/qwik-worker/package.json
@@ -9,7 +9,7 @@
     "vite-plugin-static-copy": "^1.0.0"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik/package.json b/packages/qwik/package.json
@@ -31,7 +31,7 @@
     "kleur": "4.1.5"
   },
   "engines": {
-    "node": ">=16.8.0 <18.0.0 || >=18.11"
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "exports": {
     ".": {

diff --git a/packages/qwik/src/cli/utils/utils.ts b/packages/qwik/src/cli/utils/utils.ts
@@ -81,7 +81,7 @@ export function cleanPackageJson(srcPkg: IntegrationPackageJson) {
     types: srcPkg.types,
     exports: srcPkg.exports,
     files: srcPkg.files,
-    engines: { node: '>=15.0.0' },
+    engines: { node: srcPkg.engines?.node || '^18.17.0 || ^20.3.0 || >=21.0.0' },
   };
 
   Object.keys(cleanedPkg).forEach((prop) => {

diff --git a/packages/qwik/src/optimizer/src/plugins/vite-server.ts b/packages/qwik/src/optimizer/src/plugins/vite-server.ts
@@ -35,22 +35,9 @@ export async function configureDevServer(
   isClientDevOnly: boolean,
   clientDevInput: string | undefined
 ) {
-  if (typeof fetch !== 'function' && sys.env === 'node') {
-    // polyfill fetch() when not available in Node.js
-
-    try {
-      if (!globalThis.fetch) {
-        const undici = await sys.strictDynamicImport('undici');
-        globalThis.fetch = undici.fetch;
-        globalThis.Headers = undici.Headers;
-        globalThis.Request = undici.Request;
-        globalThis.Response = undici.Response;
-        globalThis.FormData = undici.FormData;
-      }
-    } catch {
-      console.warn('Global fetch() was not installed');
-      // Nothing
-    }
+  if (typeof fetch !== 'function') {
+    console.error('Global fetch() is missing');
+    process.exit(1);
   }
 
   // qwik middleware injected BEFORE vite internal middlewares

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/scripts/binding-platform.ts b/scripts/binding-platform.ts
@@ -1,7 +1,6 @@
 import { type BuildConfig, ensureDir } from './util';
 import spawn from 'cross-spawn';
 import { join } from 'node:path';
-import { fetch } from 'undici';
 import semver from 'semver';
 import { existsSync } from 'node:fs';
 import { copyFile, readdir, writeFile } from 'fs/promises';

diff --git a/scripts/create-qwik-cli.ts b/scripts/create-qwik-cli.ts
@@ -217,7 +217,6 @@ async function updatePackageJson(config: BuildConfig, destDir: string) {
   setVersionFromRoot('prettier');
   setVersionFromRoot('typescript');
   setVersionFromRoot('node-fetch');
-  setVersionFromRoot('undici');
   setVersionFromRoot('vite');
 
   await writePackageJson(destDir, pkgJson);