chore(headless): fix internal resolution errors in node16

.github/workflows/prbot.yml

@@ -44,6 +44,14 @@ jobs:
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
       - uses: ./.github/actions/setup
       - run: npm test
+  packages-compatibility:
+    name: 'Are packages compatible?'
+    needs: build
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4
+      - uses: ./.github/actions/setup
+      - run: npm run packages:compatibility
   e2e-atomic-csp-test:
     name: 'Run e2e tests on Atomic CSP'
     needs: build

package.json

@@ -11,6 +11,7 @@
     "e2e": "nx run-many --target=e2e",
     "lint:check": "eslint . && cspell **/*.md --no-progress --show-suggestions --show-context && prettier --check '**/*.{css,pcss,html,md,yml,ts,tsx,js,mjs,json}'",
     "lint:fix": "eslint --fix . && prettier --write '**/*.{css,pcss,html,md,yml,ts,tsx,js,mjs,json}'",
+    "packages:compatibility": "node ./scripts/ci/packages-compatibility.mjs",
     "pr:report": "node ./scripts/reports/pr-report.mjs",
     "report:bundle-size:time-series": "node ./scripts/reports/bundle-size/time-series.mjs",
     "commit": "git-cz",
@@ -83,6 +84,7 @@
     "patch-package": "8.0.0",
     "prettier": "3.3.3",
     "prettier-plugin-tailwindcss": "0.6.5",
+    "publint": "0.2.11",
     "react-syntax-highlighter": "15.5.0",
     "rimraf": "5.0.9",
     "semver": "7.6.3",

packages/atomic-react/package.json

@@ -21,7 +21,7 @@
     "promote:npm:latest": "node ../../scripts/deploy/update-npm-tag.mjs latest",
     "build:assets": "ncp ../atomic/dist/atomic/assets dist/assets && ncp ../atomic/dist/atomic/lang dist/lang "
   },
-  "main": "./dist/cjs/atomic-react.js",
+  "main": "./dist/cjs/atomic-react.cjs",
   "module": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
@@ -63,17 +63,17 @@
     ".": {
       "types": "./dist/index.d.ts",
       "import": "./dist/index.js",
-      "require": "./dist/cjs/atomic-react.js"
+      "require": "./dist/cjs/atomic-react.cjs"
     },
     "./commerce": {
       "types": "./dist/commerce.index.d.ts",
       "import": "./dist/commerce.index.js",
-      "require": "./dist/cjs/commerce/atomic-react.js"
+      "require": "./dist/cjs/commerce/atomic-react.cjs"
     },
     "./recommendation": {
       "types": "./dist/recommendation.index.d.ts",
       "import": "./dist/recommendation.index.js",
-      "require": "./dist/cjs/recommendation/atomic-react.js"
+      "require": "./dist/cjs/recommendation/atomic-react.cjs"
     }
   }
 }

packages/atomic-react/rollup.config.mjs

@@ -97,7 +97,7 @@ const commonExternal = [
 
 /** @returns {import('rollup').OutputOptions} */
 const outputCJS = ({useCase}) => ({
-  file: `dist/cjs/${useCase}atomic-react.js`,
+  file: `dist/cjs/${useCase}atomic-react.cjs`,
   format: 'cjs',
 });
 

packages/atomic/package.json

@@ -14,14 +14,14 @@
   "types": "dist/types/index.d.ts",
   "exports": {
     "./loader": {
+      "types": "./loader/index.d.ts",
       "import": "./loader/index.js",
-      "require": "./loader/index.cjs.js",
-      "types": "./loader/index.d.ts"
+      "require": "./loader/index.cjs.js"
     },
     ".": {
+      "types": "./dist/types/index.d.ts",
       "import": "./dist/index.js",
-      "require": "./dist/index.cjs.js",
-      "types": "./dist/types/index.d.ts"
+      "require": "./dist/index.cjs.js"
     },
     "./themes/*": {
       "import": "./dist/atomic/themes/*",
@@ -39,7 +39,8 @@
   "files": [
     "dist/",
     "docs/",
-    "licenses/"
+    "licenses/",
+    "loader/"
   ],
   "scripts": {
     "clean": "rimraf -rf dist/*",

packages/headless/esbuild.mjs

@@ -210,7 +210,7 @@ async function buildBrowserConfig(options, outDir) {
 const nodeCjs = Object.entries(useCaseEntries).map((entry) => {
   const [useCase, entryPoint] = entry;
   const dir = getUseCaseDir('dist/', useCase);
-  const outfile = `${dir}/headless.js`;
+  const outfile = `${dir}/headless.cjs`;
   return buildNodeConfig(
     {
       entryPoints: [entryPoint],

packages/headless/package.json

@@ -6,7 +6,7 @@
     "url": "git+https://github.com/coveo/ui-kit.git",
     "directory": "packages/headless"
   },
-  "main": "./dist/headless.js",
+  "main": "./dist/headless.cjs",
   "module": "./dist/headless.esm.js",
   "type": "module",
   "exports": {
@@ -15,7 +15,7 @@
       "node": {
         "types": "./dist/definitions/index.d.ts",
         "import": "./dist/headless.esm.js",
-        "require": "./dist/headless.js"
+        "require": "./dist/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/index.d.ts",
@@ -30,7 +30,7 @@
       "node": {
         "types": "./dist/definitions/case-assist.index.d.ts",
         "import": "./dist/case-assist/headless.esm.js",
-        "require": "./dist/case-assist/headless.js"
+        "require": "./dist/case-assist/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/case-assist.index.d.ts",
@@ -45,7 +45,7 @@
       "node": {
         "types": "./dist/definitions/commerce.index.d.ts",
         "import": "./dist/commerce/headless.esm.js",
-        "require": "./dist/commerce/headless.js"
+        "require": "./dist/commerce/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/commerce.index.d.ts",
@@ -60,7 +60,7 @@
       "node": {
         "types": "./dist/definitions/insight.index.d.ts",
         "import": "./dist/insight/headless.esm.js",
-        "require": "./dist/insight/headless.js"
+        "require": "./dist/insight/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/insight.index.d.ts",
@@ -75,7 +75,7 @@
       "node": {
         "types": "./dist/definitions/recommendation.index.d.ts",
         "import": "./dist/recommendation/headless.esm.js",
-        "require": "./dist/recommendation/headless.js"
+        "require": "./dist/recommendation/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/recommendation.index.d.ts",
@@ -90,7 +90,7 @@
       "node": {
         "types": "./dist/definitions/ssr.index.d.ts",
         "import": "./dist/ssr/headless.esm.js",
-        "require": "./dist/ssr/headless.js"
+        "require": "./dist/ssr/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/ssr.index.d.ts",
@@ -105,7 +105,7 @@
       "node": {
         "types": "./dist/definitions/ssr-commerce.index.d.ts",
         "import": "./dist/ssr-commerce/headless.esm.js",
-        "require": "./dist/ssr-commerce/headless.js"
+        "require": "./dist/ssr-commerce/headless.cjs"
       },
       "browser": {
         "types": "./dist/definitions/ssr-commerce.index.d.ts",

packages/headless/src/api/analytics/analytics-relay-client.test.ts

@@ -1,8 +1,8 @@
 import {createRelay} from '@coveo/relay';
-import {createMockState} from '../../test/mock-state';
-import {getAnalyticsNextApiBaseUrl} from '../platform-client';
-import {getRelayInstanceFromState} from './analytics-relay-client';
-import {getAnalyticsSource} from './analytics-selectors';
+import {createMockState} from '../../test/mock-state.js';
+import {getAnalyticsNextApiBaseUrl} from '../platform-client.js';
+import {getRelayInstanceFromState} from './analytics-relay-client.js';
+import {getAnalyticsSource} from './analytics-selectors.js';
 
 jest.mock('@coveo/relay');
 jest.mock('./analytics-selectors');

packages/headless/src/api/analytics/analytics-relay-client.ts

@@ -3,9 +3,9 @@ import {createSelector} from '@reduxjs/toolkit';
 import {
   CommerceConfigurationSection,
   ConfigurationSection,
-} from '../../state/state-sections';
-import {getAnalyticsNextApiBaseUrl} from '../platform-client';
-import {getAnalyticsSource} from './analytics-selectors';
+} from '../../state/state-sections.js';
+import {getAnalyticsNextApiBaseUrl} from '../platform-client.js';
+import {getAnalyticsSource} from './analytics-selectors.js';
 
 type StateNeededByRelay = ConfigurationSection | CommerceConfigurationSection;
 

packages/headless/src/api/analytics/analytics-selectors.test.ts

@@ -1,6 +1,6 @@
-import {createMockState} from '../../test/mock-state';
-import {VERSION} from '../../utils/version';
-import {getAnalyticsSource} from './analytics-selectors';
+import {createMockState} from '../../test/mock-state.js';
+import {VERSION} from '../../utils/version.js';
+import {getAnalyticsSource} from './analytics-selectors.js';
 
 describe('#getAnalyticsSources', () => {
   it('without a source, returns an array only with `@coveo/headless`', () => {

packages/headless/src/api/analytics/analytics-selectors.ts

@@ -1,6 +1,6 @@
 import {createSelector} from '@reduxjs/toolkit';
-import {CoreAnalyticsState} from '../../features/configuration/configuration-state';
-import {VERSION} from '../../utils/version';
+import {CoreAnalyticsState} from '../../features/configuration/configuration-state.js';
+import {VERSION} from '../../utils/version.js';
 
 export const getAnalyticsSource = createSelector(
   (state: CoreAnalyticsState) => state.source,

packages/headless/src/api/analytics/base-analytics.test.ts

@@ -1,11 +1,11 @@
-import {SearchEventRequest} from 'coveo.analytics/dist/definitions/events';
-import {getConfigurationInitialState} from '../../features/configuration/configuration-state';
-import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state';
-import {buildMockAnalyticsState} from '../../test/mock-analytics-state';
+import {SearchEventRequest} from 'coveo.analytics/dist/definitions/events.js';
+import {getConfigurationInitialState} from '../../features/configuration/configuration-state.js';
+import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state.js';
+import {buildMockAnalyticsState} from '../../test/mock-analytics-state.js';
 import {
   BaseAnalyticsProvider,
   StateNeededByBaseAnalyticsProvider,
-} from './base-analytics';
+} from './base-analytics.js';
 
 class TestProvider extends BaseAnalyticsProvider<StateNeededByBaseAnalyticsProvider> {
   public getPipeline(): string {

packages/headless/src/api/analytics/base-analytics.ts

@@ -1,14 +1,14 @@
 import {SearchPageClientProvider} from 'coveo.analytics';
-import {SearchEventRequest} from 'coveo.analytics/dist/definitions/events';
-import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state';
+import {SearchEventRequest} from 'coveo.analytics/dist/definitions/events.js';
+import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state.js';
 import {
   ConfigurationSection,
   ContextSection,
   PipelineSection,
   QuerySection,
   SearchHubSection,
-} from '../../state/state-sections';
-import {VERSION} from '../../utils/version';
+} from '../../state/state-sections.js';
+import {VERSION} from '../../utils/version.js';
 
 export const getLanguage = (state: ConfigurationSection) => {
   const langKey = state.configuration.search.locale.split('-')[0];

packages/headless/src/api/analytics/case-assist-analytics.ts

@@ -4,21 +4,21 @@ import {
   CaseAssistClientProvider,
 } from 'coveo.analytics';
 import {Logger} from 'pino';
-import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state';
+import {getSearchHubInitialState} from '../../features/search-hub/search-hub-state.js';
 import {
   CaseAssistConfigurationSection,
   CaseFieldSection,
   CaseInputSection,
   ConfigurationSection,
   DocumentSuggestionSection,
   SearchHubSection,
-} from '../../state/state-sections';
-import {getOrganizationEndpoint} from '../platform-client';
-import {PreprocessRequest} from '../preprocess-request';
+} from '../../state/state-sections.js';
+import {getOrganizationEndpoint} from '../platform-client.js';
+import {PreprocessRequest} from '../preprocess-request.js';
 import {
   wrapAnalyticsClientSendEventHook,
   wrapPreprocessRequest,
-} from './coveo-analytics-utils';
+} from './coveo-analytics-utils.js';
 
 export type StateNeededByCaseAssistAnalytics = ConfigurationSection &
   Partial<CaseAssistConfigurationSection> &

packages/headless/src/api/analytics/coveo-analytics-utils.test.ts

@@ -1,10 +1,13 @@
 import {AnalyticsClientSendEventHook} from 'coveo.analytics';
 import {pino} from 'pino';
-import {PlatformRequestOptions, PreprocessRequest} from '../preprocess-request';
+import {
+  PlatformRequestOptions,
+  PreprocessRequest,
+} from '../preprocess-request.js';
 import {
   wrapAnalyticsClientSendEventHook,
   wrapPreprocessRequest,
-} from './coveo-analytics-utils';
+} from './coveo-analytics-utils.js';
 
 describe('coveo-analytics-utils', () => {
   it('should properly wrap preprocessRequest', () => {

packages/headless/src/api/analytics/coveo-analytics-utils.ts

@@ -5,8 +5,8 @@ import {
   history,
 } from 'coveo.analytics';
 import {Logger} from 'pino';
-import {clone} from '../../utils/utils';
-import {PreprocessRequest} from '../preprocess-request';
+import {clone} from '../../utils/utils.js';
+import {PreprocessRequest} from '../preprocess-request.js';
 
 export const getVisitorID = (options: {
   runtimeEnvironment?: IRuntimeEnvironment;

packages/headless/src/api/analytics/insight-analytics.test.ts

@@ -1,21 +1,21 @@
 import {CoveoAnalyticsClient} from 'coveo.analytics';
-import pino from 'pino';
-import {getConfigurationInitialState} from '../../features/configuration/configuration-state';
-import {getGeneratedAnswerInitialState} from '../../features/generated-answer/generated-answer-state';
-import {buildMockFacetRequest} from '../../test/mock-facet-request';
-import {buildMockFacetResponse} from '../../test/mock-facet-response';
-import {buildMockFacetSlice} from '../../test/mock-facet-slice';
-import {buildMockFacetValue} from '../../test/mock-facet-value';
-import {buildMockFacetValueRequest} from '../../test/mock-facet-value-request';
-import {buildMockInsightState} from '../../test/mock-insight-state';
-import {buildMockQueryState} from '../../test/mock-query-state';
-import {buildMockResult} from '../../test/mock-result';
-import {buildMockSearchState} from '../../test/mock-search-state';
+import {pino} from 'pino';
+import {getConfigurationInitialState} from '../../features/configuration/configuration-state.js';
+import {getGeneratedAnswerInitialState} from '../../features/generated-answer/generated-answer-state.js';
+import {buildMockFacetRequest} from '../../test/mock-facet-request.js';
+import {buildMockFacetResponse} from '../../test/mock-facet-response.js';
+import {buildMockFacetSlice} from '../../test/mock-facet-slice.js';
+import {buildMockFacetValueRequest} from '../../test/mock-facet-value-request.js';
+import {buildMockFacetValue} from '../../test/mock-facet-value.js';
+import {buildMockInsightState} from '../../test/mock-insight-state.js';
+import {buildMockQueryState} from '../../test/mock-query-state.js';
+import {buildMockResult} from '../../test/mock-result.js';
+import {buildMockSearchState} from '../../test/mock-search-state.js';
 import {
   configureInsightAnalytics,
   InsightAnalyticsProvider,
   StateNeededByInsightAnalyticsProvider,
-} from './insight-analytics';
+} from './insight-analytics.js';
 
 describe('insight analytics', () => {
   const logger = pino({level: 'silent'});