feat(npm/oxc-types): convert to ES modules #13875
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR converts the npm/oxc-types package to ES modules by changing the package.json type field from "commonjs" to "module". The change also affects the npm/runtime package by adding the ES module type configuration.
Key Changes:
- Updated
npm/oxc-types/package.jsonto set"type": "module" - Updated
npm/runtime/package.jsonto add"type": "module"and remove the previous"type": "commonjs"
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| npm/oxc-types/package.json | Changed module type from CommonJS to ES modules |
| npm/runtime/package.json | Added ES module type and removed CommonJS type declaration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
Boshen
force-pushed
the
feat/oxc-types-esm
branch
from
95f4510 to
400c4c7
Compare
Merge activity
|
## Summary - Convert npm/oxc-types package to ES modules by setting `"type": "module"` in package.json ## Test plan - [ ] Verify TypeScript definitions continue to work with both CommonJS and ESM consumers - [ ] Test imports in various module environments - [ ] Run existing test suites 🤖 Generated with [Claude Code](https://claude.ai/code)
graphite-app
bot
force-pushed
the
feat/oxc-types-esm
branch
from
400c4c7 to
89e9f5f
Compare
Summary
"type": "module"in package.jsonTest plan
🤖 Generated with Claude Code