chore: replace node-fetch with native fetch

package.json

@@ -4,7 +4,7 @@
   "description": "",
   "private": true,
   "engines": {
-    "node": ">=15.0.0",
+    "node": ">=18.17.0",
     "npm": ">=7.0.0"
   },
   "engineStrict": true,

packages/cli/package.json

@@ -8,7 +8,7 @@
     "redocly": "bin/cli.js"
   },
   "engines": {
-    "node": ">=14.19.0",
+    "node": ">=18.17.0",
     "npm": ">=7.0.0"
   },
   "engineStrict": true,
@@ -46,7 +46,6 @@
     "glob": "^7.1.6",
     "handlebars": "^4.7.6",
     "mobx": "^6.0.4",
-    "node-fetch": "^2.6.1",
     "pluralize": "^8.0.0",
     "react": "^17.0.0 || ^18.2.0",
     "react-dom": "^17.0.0 || ^18.2.0",

packages/cli/src/__tests__/commands/push-region.test.ts

@@ -4,12 +4,6 @@ import { promptClientToken } from '../../commands/login';
 import { ConfigFixture } from '../fixtures/config';
 
 jest.mock('fs');
-jest.mock('node-fetch', () => ({
-  default: jest.fn(() => ({
-    ok: true,
-    json: jest.fn().mockResolvedValue({}),
-  })),
-}));
 jest.mock('@redocly/openapi-core');
 jest.mock('../../commands/login');
 jest.mock('../../utils/miscellaneous');
@@ -22,8 +16,18 @@ describe('push-with-region', () => {
   const redoclyClient = require('@redocly/openapi-core').__redoclyClient;
   redoclyClient.isAuthorizedWithRedoclyByRegion = jest.fn().mockResolvedValue(false);
 
+  afterAll(() => {
+    jest.restoreAllMocks(); // Restore original fetch after tests
+  });
+
   beforeAll(() => {
     jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    global.fetch = jest.fn(() =>
+      Promise.resolve({
+        ok: true,
+        json: jest.fn().mockResolvedValue({}), // Mocked JSON response
+      } as unknown as Response)
+    );
   });
 
   it('should call login with default domain when region is US', async () => {

packages/cli/src/__tests__/commands/push.test.ts

@@ -6,12 +6,6 @@ import { ConfigFixture } from '../fixtures/config';
 import { yellow } from 'colorette';
 
 jest.mock('fs');
-jest.mock('node-fetch', () => ({
-  default: jest.fn(() => ({
-    ok: true,
-    json: jest.fn().mockResolvedValue({}),
-  })),
-}));
 jest.mock('@redocly/openapi-core');
 jest.mock('../../utils/miscellaneous');
 
@@ -22,6 +16,16 @@ describe('push', () => {
 
   beforeEach(() => {
     jest.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    global.fetch = jest.fn(() =>
+      Promise.resolve({
+        ok: true,
+        json: jest.fn().mockResolvedValue({}), // Mocked JSON response
+      } as unknown as Response)
+    );
+  });
+
+  afterAll(() => {
+    jest.restoreAllMocks(); // Restore original fetch after tests
   });
 
   it('pushes definition', async () => {

packages/cli/src/__tests__/fetch-with-timeout.test.ts

@@ -1,10 +1,8 @@
 import AbortController from 'abort-controller';
 import fetchWithTimeout from '../utils/fetch-with-timeout';
-import nodeFetch from 'node-fetch';
 import { getProxyAgent } from '@redocly/openapi-core';
 import { HttpsProxyAgent } from 'https-proxy-agent';
 
-jest.mock('node-fetch');
 jest.mock('@redocly/openapi-core');
 
 describe('fetchWithTimeout', () => {
@@ -16,6 +14,12 @@ describe('fetchWithTimeout', () => {
 
   beforeEach(() => {
     (getProxyAgent as jest.Mock).mockReturnValueOnce(undefined);
+    global.fetch = jest.fn(() =>
+      Promise.resolve({
+        ok: true,
+        json: jest.fn().mockResolvedValue({}), // Mocked JSON response
+      } as unknown as Response)
+    );
   });
 
   afterEach(() => {
@@ -26,7 +30,7 @@ describe('fetchWithTimeout', () => {
     await fetchWithTimeout('url', { timeout: 1000 });
 
     expect(global.setTimeout).toHaveBeenCalledTimes(1);
-    expect(nodeFetch).toHaveBeenCalledWith('url', {
+    expect(fetch).toHaveBeenCalledWith('url', {
       signal: new AbortController().signal,
       agent: undefined,
     });
@@ -40,14 +44,14 @@ describe('fetchWithTimeout', () => {
 
     await fetchWithTimeout('url');
 
-    expect(nodeFetch).toHaveBeenCalledWith('url', { agent: proxyAgent });
+    expect(fetch).toHaveBeenCalledWith('url', { agent: proxyAgent });
   });
 
   it('should call node-fetch without signal when timeout is not passed', async () => {
     await fetchWithTimeout('url');
 
     expect(global.setTimeout).not.toHaveBeenCalled();
-    expect(nodeFetch).toHaveBeenCalledWith('url', { agent: undefined });
+    expect(fetch).toHaveBeenCalledWith('url', { agent: undefined });
     expect(global.clearTimeout).not.toHaveBeenCalled();
   });
 });