fix: cover module interop and reuse the preview build

Author: fireairforce

.github/workflows/surge-preview.yml

@@ -46,6 +46,7 @@ jobs:
         with:
           surge_token: ${{ env.SURGE_TOKEN }}
           github_token: ${{ secrets.GITHUB_TOKEN }}
+          build: test -d docs-dist
           dist: docs-dist
           failOnError: false
           setCommitStatus: false

scripts/update-browser-field.js

@@ -13,9 +13,10 @@ const addEntry = (from, to) => {
   browserEntries.push([from, to]);
 };
 
-// Preserve previous behavior of preferring the ES build for the main entry.
+// Redirect the resolved main file for legacy bundlers that do not use exports.
 addEntry('./lib/index.js', './es/index.mjs');
 
+/** Keep legacy browser request names while pointing them at the emitted ESM files. */
 const addDirMappings = (dirPath, browserPrefix, targetPrefix) => {
   if (!fs.existsSync(dirPath)) {
     return;

src/PickerPanel/index.tsx

@@ -64,6 +64,7 @@ export interface BasePickerPanelProps<DateType extends object = any>
       | 'superPrevIcon'
       | 'superNextIcon'
     >,
+    // Panel values may be null or multiple; the time default only accepts one date.
     Omit<SharedTimeProps<DateType>, 'defaultValue'>,
     Pick<React.HTMLAttributes<HTMLDivElement>, 'tabIndex'> {
   // Style

tests/util.spec.tsx

@@ -3,10 +3,32 @@
 import dayjs from 'dayjs';
 import dayGenerate from '../src/generate/dayjs';
 import { isInRange } from '../src/utils/dateUtil';
+import getDefaultExport from '../src/utils/getDefaultExport';
 
 global.error = console.error;
 
 describe('Picker.Util', () => {
+  describe('getDefaultExport', () => {
+    it('unwraps a transpiled CommonJS namespace loaded by Node ESM', () => {
+      const component = () => null;
+      const namespace = { default: component };
+      Object.defineProperty(namespace, '__esModule', { value: true });
+      expect(getDefaultExport(namespace)).toBe(component);
+    });
+
+    it.each([
+      null,
+      undefined,
+      () => null,
+      { $$typeof: Symbol.for('react.forward_ref'), render: () => null },
+      { default: 'ordinary property' },
+      { __esModule: false, default: 'ordinary property' },
+      { __esModule: true },
+    ])('preserves an already usable export: %p', (value) => {
+      expect(getDefaultExport(value)).toBe(value);
+    });
+  });
+
   describe('isInRange', () => {
     it('not break with null', () => {
       expect(isInRange(dayGenerate, null, null, dayjs())).toBeFalsy();