Skip to content

fix: emit Node-compatible ESM output - #27

Open
zombieJ wants to merge 1 commit into
mainfrom
fix/native-esm-output
Open

fix: emit Node-compatible ESM output#27
zombieJ wants to merge 1 commit into
mainfrom
fix/native-esm-output

Conversation

@zombieJ

@zombieJ zombieJ commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • emit fully specified relative JavaScript imports and exports for native ESM output
  • complete legacy package subpaths such as dayjs/plugin/weekday while preserving packages that define exports
  • align declaration specifiers and add an ESM package marker to the output directory
  • activate only when exports.import targets the Father ESM output, leaving legacy main/module packages unchanged

Context

This addresses the shared build behavior behind react-component/picker#1002 and react-component/picker#1004 in the Father plugin instead of requiring each package to maintain a post-build script.

Validation

  • ut lint:es
  • ut build
  • ut test (9 tests)
  • rebuilt @rc-component/picker from master and verified native Node ESM imports, CommonJS require, and a TypeScript NodeNext consumer; the lib output remained identical
  • ran antd master with its picker aliases pointed at the rebuilt es output; Dumi/Webpack compiled 16,828 modules and both the DatePicker page and /umi.js returned HTTP 200

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 3f89746d-bcd2-4f0b-b63e-71b71b800211


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant