Skip to content

fix(rolldown_plugin_transform): don't load tsconfig for files in node_modules#5074

Merged
shulaoda merged 1 commit intomainfrom
06-26-fix_rolldown_plugin_transform_don_t_load_tsconfig_for_files_in_node_modules
Jun 26, 2025
Merged

fix(rolldown_plugin_transform): don't load tsconfig for files in node_modules#5074
shulaoda merged 1 commit intomainfrom
06-26-fix_rolldown_plugin_transform_don_t_load_tsconfig_for_files_in_node_modules

Conversation

@sapphi-red
Copy link
Member

@sapphi-red sapphi-red commented Jun 26, 2025 •
edited
Loading

@sapphi-red Graphite App
Copy link
Member Author

sapphi-red commented Jun 26, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sapphi-red sapphi-red marked this pull request as ready for review June 26, 2025 04:10
@netlify
Copy link

netlify bot commented Jun 26, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 1159ca9
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/685cc7f7265f2a00086734ad

@sapphi-red sapphi-red requested a review from shulaoda June 26, 2025 04:10
@github-actions
Copy link
Contributor

github-actions bot commented Jun 26, 2025

Benchmarks Rust

  • target: main(a0ec4ad)
  • pr: 06-26-fix_rolldown_plugin_transform_don_t_load_tsconfig_for_files_in_node_modules(1159ca9)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     75.7±2.70ms        ? ?/sec    1.00     75.9±2.34ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     86.4±1.57ms        ? ?/sec    1.00     86.4±1.94ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    112.4±1.47ms        ? ?/sec    1.02    114.3±5.90ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.00    134.2±2.04ms        ? ?/sec    1.00    134.0±1.61ms        ? ?/sec
bundle/bundle@threejs                                        1.03     42.2±2.52ms        ? ?/sec    1.00     41.1±1.20ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     50.8±2.13ms        ? ?/sec    1.00     51.0±0.64ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    437.3±4.51ms        ? ?/sec    1.00    436.6±6.13ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    510.9±3.49ms        ? ?/sec    1.01    514.3±5.19ms        ? ?/sec
scan/scan@rome_ts                                            1.00     89.0±1.51ms        ? ?/sec    1.01     89.8±1.37ms        ? ?/sec
scan/scan@threejs                                            1.00     30.9±0.78ms        ? ?/sec    1.00     30.9±0.48ms        ? ?/sec
scan/scan@threejs10x                                         1.00    323.0±5.18ms        ? ?/sec    1.00    322.7±5.21ms        ? ?/sec

@shulaoda shulaoda added this pull request to the merge queue Jun 26, 2025
Merged via the queue into main with commit 64cd069 Jun 26, 2025
25 checks passed
@shulaoda shulaoda deleted the 06-26-fix_rolldown_plugin_transform_don_t_load_tsconfig_for_files_in_node_modules branch June 26, 2025 06:45
@github-actions github-actions bot mentioned this pull request Jun 27, 2025
Merged
hyf0 pushed a commit that referenced this pull request Jun 27, 2025
@github-actions @sapphi-red
release: v1.0.0-beta.21 (#5084)
c747854 
## [1.0.0-beta.21] - 2025-06-27

### 🚀 Features

- rolldown: oxc_resolver v11.3.0 (#5078) by @Boshen
- debug: support to emit `meta.json` (#5075) by @hyf0
- plugin/vite-resolve: support tsconfig paths (#5004) by @sapphi-red

### 🐛 Bug Fixes

- plugin/vite-resolve: resolve glob side effects field correctly (#5079)
by @sapphi-red
- rolldown_plugin_transform: don't load tsconfig for files in
node_modules (#5074) by @sapphi-red
- preserveModules: correctly generate absolute chunk file names (#5072)
by @shulaoda
- vite-tests: fix `pnpm install` failure (#5070) by @shulaoda

### 📚 Documentation

- contrib-guide: add memory profile steps (#5053) by @sapphi-red

### ⚙️ Miscellaneous Tasks

- remove `just lint` in `packages/debug/package.json` (#5080) by @hyf0
- deps: update dependency rolldown-plugin-dts to v0.13.12 (#5077) by
@renovate[bot]
- repo: make `pnpm lint-knip` part of `just lint-node` (#5076) by @hyf0
- infra: move `insta.yaml` to the workspace root (#5073) by @shulaoda

Co-authored-by: sapphi-red <49056869+sapphi-red@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shulaoda shulaoda shulaoda approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sapphi-red @shulaoda