chore(headless)!: remove headless fetch & crypto polyfills

packages/atomic-react/package.json

@@ -32,11 +32,13 @@
     "@coveo/atomic": "2.78.0"
   },
   "devDependencies": {
-    "@coveo/release": "1.0.0",
     "@coveo/headless": "2.80.0",
+    "@coveo/release": "1.0.0",
     "@rollup/plugin-commonjs": "^25.0.0",
+    "@rollup/plugin-json": "6.1.0",
     "@rollup/plugin-node-resolve": "^15.0.0",
     "@rollup/plugin-replace": "^5.0.0",
+    "@rollup/plugin-terser": "0.4.4",
     "@rollup/plugin-typescript": "^11.0.0",
     "@types/node": "20.14.12",
     "@types/react": "18.3.3",
@@ -45,8 +47,7 @@
     "react": "18.3.1",
     "react-dom": "18.3.1",
     "rollup": "3.29.4",
-    "rollup-plugin-polyfill-node": "^0.13.0",
-    "@rollup/plugin-terser": "0.4.4"
+    "rollup-plugin-polyfill-node": "^0.13.0"
   },
   "peerDependencies": {
     "@coveo/headless": "2.80.0",

packages/atomic-react/rollup.config.mjs

@@ -1,4 +1,5 @@
 import commonjs from '@rollup/plugin-commonjs';
+import json from '@rollup/plugin-json';
 import {nodeResolve} from '@rollup/plugin-node-resolve';
 import replace from '@rollup/plugin-replace';
 import terser from '@rollup/plugin-terser';
@@ -54,6 +55,7 @@ const outputIIFECommerce = ({minify}) => ({
 });
 
 const plugins = [
+  json(),
   nodePolyfills(),
   typescript({tsconfig: 'tsconfig.iife.json'}),
   commonjs(),

packages/headless-react/package.json

@@ -51,8 +51,7 @@
     "publint": "0.2.9",
     "rimraf": "5.0.9",
     "ts-jest": "29.2.3",
-    "typescript": "5.4.5",
-    "undici": "5.28.4"
+    "typescript": "5.4.5"
   },
   "peerDependencies": {
     "react": "^18",

packages/headless/esbuild.mjs

@@ -174,6 +174,15 @@ function resolveEsm(moduleName) {
   );
 }
 
+function resolveBrowser(moduleName) {
+  const packageJsonPath = require.resolve(`${moduleName}/package.json`);
+  const packageJson = require(packageJsonPath);
+  return resolve(
+    dirname(packageJsonPath),
+    packageJson['browser'] || packageJson['main']
+  );
+}
+
 /**
  * @param {import('esbuild').BuildOptions} options
  * @returns {Promise<import('esbuild').BuildResult>}
@@ -190,10 +199,7 @@ async function buildBrowserConfig(options, outDir) {
     plugins: [
       alias({
         'coveo.analytics': resolveEsm('coveo.analytics'),
-        '@coveo/please-give-me-fetch': resolve(
-          './ponyfills',
-          'fetch-ponyfill-browser.js'
-        ),
+        pino: resolveBrowser('pino'),
         '@coveo/pendragon': resolve('./ponyfills', 'magic-cookie-browser.js'),
       }),
       ...(options.plugins || []),
@@ -227,6 +233,7 @@ const nodeEsm = Object.entries(useCaseEntries).map((entry) => {
       entryPoints: [entryPoint],
       outfile,
       format: 'esm',
+      external: ['pino'],
     },
     dir
   );
@@ -244,11 +251,7 @@ async function buildNodeConfig(options, outDir) {
     treeShaking: true,
     plugins: [
       alias({
-        'coveo.analytics': require.resolve('coveo.analytics'),
-        '@coveo/please-give-me-fetch': resolve(
-          './ponyfills',
-          'fetch-ponyfill-node.js'
-        ),
+        'coveo.analytics': resolveEsm('coveo.analytics'),
         '@coveo/pendragon': resolve('./ponyfills', 'magic-cookie-node.js'),
       }),
     ],

packages/headless/jest.config.js

@@ -3,10 +3,7 @@ process.env.TZ = 'Australia/Eucla';
 module.exports = {
   preset: 'ts-jest',
   silent: true,
-  setupFiles: ['./jest.setup.js'],
   moduleNameMapper: {
-    '@coveo/please-give-me-fetch':
-      '<rootDir>/ponyfills/fetch-ponyfill-browser.js',
     '@coveo/pendragon': '<rootDir>/ponyfills/magic-cookie-node.js',
   },
 };

packages/headless/package.json

@@ -63,8 +63,7 @@
     "node-abort-controller": "^3.0.0",
     "pino": "8.21.0",
     "redux-thunk": "3.1.0",
-    "ts-debounce": "4.0.0",
-    "undici": "5.28.4"
+    "ts-debounce": "4.0.0"
   },
   "devDependencies": {
     "@coveo/release": "1.0.0",

packages/headless/src/api/platform-client.test.ts

@@ -1,6 +1,3 @@
-//TODO V3: remove this import, global fetch is a requirement now.
-//@ts-expect-error package is just an alias.
-import fetch from '@coveo/please-give-me-fetch';
 import * as BackOff from 'exponential-backoff';
 import pino from 'pino';
 import {allValidPlatformCombination} from '../test/platform-url';
@@ -19,10 +16,9 @@ import {
   PreprocessRequest,
 } from './preprocess-request';
 
-jest.mock('@coveo/please-give-me-fetch');
-
 const {Response} = jest.requireActual('node-fetch');
-const mockFetch = fetch as jest.Mock;
+global.fetch = jest.fn();
+const mockFetch = global.fetch as jest.Mock;
 
 describe('url helper', () => {
   it('return the correct #platformUrl()', () => {

packages/headless/src/api/platform-client.ts

@@ -1,6 +1,3 @@
-//TODO V3: remove this import, global fetch is a requirement now.
-//@ts-expect-error package is just an alias.
-import fetch from '@coveo/please-give-me-fetch';
 import {backOff} from 'exponential-backoff';
 import {Logger} from 'pino';
 import {DisconnectedError, ExpiredTokenError} from '../utils/errors';

packages/headless/src/case-assist.index.ts

@@ -1,6 +1,3 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
-
-polyfillCryptoNode();
 // 3rd Party Libraries
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';

packages/headless/src/commerce.index.ts

@@ -1,10 +1,8 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
 import * as Selectors from './selectors/commerce-selectors.index';
 import * as HighlightUtils from './utils/highlight';
 
 export type {HighlightKeyword} from './utils/highlight';
 
-polyfillCryptoNode();
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';
 

packages/headless/src/index.ts

@@ -1,8 +1,5 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
 import * as HighlightUtils from './utils/highlight';
 
-polyfillCryptoNode();
-
 // 3rd Party Libraries
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {AnalyticsClientSendEventHook} from 'coveo.analytics';

packages/headless/src/insight.index.ts

@@ -1,7 +1,5 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
 import * as HighlightUtils from './utils/highlight';
 
-polyfillCryptoNode();
 // 3rd Party Libraries
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';

packages/headless/src/product-listing.index.ts

@@ -1,6 +1,3 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
-
-polyfillCryptoNode();
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';
 

packages/headless/src/product-recommendation.index.ts

@@ -1,7 +1,3 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
-
-polyfillCryptoNode();
-
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';
 

packages/headless/src/recommendation.index.ts

@@ -1,9 +1,7 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
 import * as HighlightUtils from './utils/highlight';
 
 export {HighlightUtils};
 
-polyfillCryptoNode();
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {Relay} from '@coveo/relay';
 

packages/headless/src/ssr.index.ts

@@ -1,7 +1,3 @@
-import {polyfillCryptoNode} from './api/analytics/analytics-crypto-polyfill';
-
-polyfillCryptoNode();
-
 // 3rd Party Libraries
 export type {Unsubscribe, Middleware} from '@reduxjs/toolkit';
 export type {AnalyticsClientSendEventHook} from 'coveo.analytics';

patches/coveo.analytics+2.30.26.patch

@@ -0,0 +1,31 @@
+diff --git a/node_modules/coveo.analytics/dist/browser.mjs b/node_modules/coveo.analytics/dist/browser.mjs
+index 3de0ed7..b7d2c0e 100644
+--- a/node_modules/coveo.analytics/dist/browser.mjs
++++ b/node_modules/coveo.analytics/dist/browser.mjs
+@@ -1093,8 +1093,6 @@ class NoopAnalyticsClient {
+     }
+ }
+
+-const fetch$1 = window.fetch;
+-
+ class AnalyticsFetchClient {
+     constructor(opts) {
+         this.opts = opts;
+@@ -1114,7 +1112,7 @@ class AnalyticsFetchClient {
+             const _a = Object.assign(Object.assign({}, defaultOptions), (preprocessRequest ? yield preprocessRequest(defaultOptions, 'analyticsFetch') : {})), { url } = _a, fetchData = __rest(_a, ["url"]);
+             let response;
+             try {
+-                response = yield fetch$1(url, fetchData);
++                response = yield fetch(url, fetchData);
+             }
+             catch (error) {
+                 console.error('An error has occured when sending the event.', error);
+@@ -1142,7 +1140,7 @@ class AnalyticsFetchClient {
+         return __awaiter(this, void 0, void 0, function* () {
+             const { baseUrl } = this.opts;
+             const url = `${baseUrl}/analytics/visit`;
+-            yield fetch$1(url, { headers: this.getHeaders(), method: 'DELETE' });
++            yield fetch(url, { headers: this.getHeaders(), method: 'DELETE' });
+         });
+     }
+     shouldAppendVisitorId(eventType) {