chore: Remove undici, set minimum node version to 18.17

.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,typescript,@builder.io/*}' --binaries --browsers`
+ description: Output of `npx envinfo --system --npmPackages '{vite,typescript,@builder.io/*}' --binaries --browsers`
  render: shell
  placeholder: System, Binaries, Browsers
  validations:

package.json

@@ -29,15 +29,6 @@
  ],
  "pinVersion": "1.40.0"
  }
- ],
- "semverGroups": [
- {
- "label": "Undici should always be * until we remove it",
- "range": "*",
- "dependencies": [
- "undici"
- ]
- }
  ]
  }
  },
@@ -123,7 +114,6 @@
  "terser": "^5.31.0",
  "tsm": "^2.3.0",
  "typescript": "5.4.5",
- "undici": "*",
  "vfile": "^6.0.1",
  "vite": "^5.2.11",
  "vite-imagetools": "^6.2.9",
@@ -135,7 +125,7 @@
  "zod": "^3.23.8"
  },
  "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": ">=9.0.5"

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';

packages/docs/package.json

@@ -43,14 +43,13 @@
  "terser": "^5.31.0",
  "tsm": "^2.3.0",
  "typescript": "5.4.5",
- "undici": "*",
  "valibot": "^0.29.0",
  "vite": "^5.2.11",
  "vite-plugin-inspect": "^0.8.4",
  "wrangler": "^3.53.1"
  },
  "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"

packages/docs/scripts/showcase.js

@@ -134,7 +134,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`;

packages/eslint-plugin-qwik/package.json

@@ -17,7 +17,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/QwikDev/qwik#readme",
  "keywords": [

packages/insights/package.json

@@ -35,13 +35,12 @@
  "prettier-plugin-tailwindcss": "^0.5.14",
  "tailwindcss": "^3.4.3",
  "typescript": "5.4.5",
- "undici": "*",
  "vite": "^5.2.11",
  "vite-tsconfig-paths": "^4.3.2",
  "vitest": "^1.6.0"
  },
  "engines": {
- "node": ">=16.8.0 <18.0.0 || >=18.11"
+ "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
  },
  "private": true,
  "scripts": {

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": {
  ".": {

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.__qwikCityNew = true;
 
  const api: QwikCityPluginApi = {

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,

packages/qwik-city/package.json

@@ -8,7 +8,6 @@
  "@types/mdx": "^2.0.13",
  "source-map": "0.7.4",
  "svgo": "^3.2.0",
- "undici": "*",
  "vfile": "^6.0.1",
  "vite": "^5.2.11",
  "vite-imagetools": "^6.2.9",
@@ -43,7 +42,7 @@
  "yaml": "^2.4.2"
  },
  "engines": {
- "node": ">=16.8.0 <18.0.0 || >=18.11"
+ "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
  },
  "exports": {
  ".": {

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';
@@ -75,9 +74,9 @@ test('useCache', () => {
 
 export function mockRequest(url?: string): Request {
  url = url || 'https://qwik.dev/';
- 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);
 }

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);

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',

packages/qwik-labs/package.json

@@ -13,11 +13,10 @@
  "np": "^10.0.5",
  "prettier": "^3.2.5",
  "typescript": "5.4.5",
- "undici": "*",
  "vite": "^5.2.11"
  },
  "engines": {
- "node": ">=16.8.0 <18.0.0 || >=18.11"
+ "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
  },
  "exports": {
  ".": {

packages/qwik-react/package.json

@@ -13,7 +13,7 @@
  "vite": "^5.2.10"
  },
  "engines": {
- "node": ">=16.8.0 <18.0.0 || >=18.11"
+ "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
  },
  "exports": {
  ".": {

packages/qwik-worker/package.json

@@ -9,7 +9,7 @@
  "vite-plugin-static-copy": "^1.0.4"
  },
  "engines": {
- "node": ">=16.8.0 <18.0.0 || >=18.11"
+ "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
  },
  "exports": {
  ".": {

packages/qwik/package.json

@@ -33,7 +33,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": {
  ".": {

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: '^18.17.0 || ^20.3.0 || >=21.0.0' },
+ engines: { node: srcPkg.engines?.node || '^18.17.0 || ^20.3.0 || >=21.0.0' },
  };
 
  Object.keys(cleanedPkg).forEach((prop) => {

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

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);

scripts/package-json.ts

@@ -21,9 +21,6 @@ export async function generatePackageJson(config: BuildConfig) {
  qwik: './qwik-cli.cjs',
  },
  type: 'module',
- peerDependencies: {
- undici: '*',
- },
  dependencies: rootPkg.dependencies,
  exports: recursiveChangePrefix(rootPkg.exports!, './dist/', './'),
  files: Array.from(new Set(rootPkg.files)).sort((a, b) => {