build(oxlint): simplify building @oxlint/plugins#18902
build(oxlint): simplify building @oxlint/plugins#18902graphite-app[bot] merged 1 commit intomainfrom
@oxlint/plugins#18902Conversation
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. This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
Pull request overview
This PR refactors the build process for the @oxlint/plugins package to simplify the configuration and build steps. Instead of generating ESM and CommonJS builds into separate subdirectories (esm/ and cjs/) and then copying specific files, both builds now output directly to the same dist-pkg-plugins directory with different file extensions.
Changes:
- Configured TSDown to output both ESM and CommonJS builds to the same directory (
dist-pkg-plugins) - Added manual directory cleanup before TSDown runs, with
clean: falsein the config to prevent TSDown from deleting files between builds - Simplified file copying to use a generic loop over all files instead of an explicit file list
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| apps/oxlint/tsdown.config.ts | Added outDir and clean: false to pluginsPkgConfig; removed separate outDir overrides for ESM and CommonJS builds |
| apps/oxlint/scripts/build.ts | Added manual deletion of dist-pkg-plugins before build; simplified file copying to loop over all files instead of explicit list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Follow-on after #18824. Refactor only. Simplify building `@oxlint/plugins` package. Generate both the ESM and CommonJS builds in the same directory.
5367275 to
e52005e
Compare

Follow-on after #18824. Refactor only.
Simplify building
@oxlint/pluginspackage. Generate both the ESM and CommonJS builds in the same directory.