Repository navigation
fix(wasm): skip source-map generation in the no-map export - #771
Merged
Merged
Conversation
Refs #686 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Refs #686 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Contributor
Changepacks@devup-ui/wasm@1.0.83 → 1.0.84 - bindings/devup-ui-wasm/package.jsonPatch
@devup-ui/bun-plugin@1.0.22 → 1.0.23 - packages/bun-plugin/package.jsonPatch
@devup-ui/components@0.1.60 → 0.1.61 - packages/components/package.jsonPatch
@devup-ui/eslint-plugin@1.0.22 → 1.0.23 - packages/eslint-plugin/package.jsonPatch
@devup-ui/next-plugin@1.0.90 → 1.0.91 - packages/next-plugin/package.jsonPatch
@devup-ui/plugin-utils@1.0.17 → 1.0.18 - packages/plugin-utils/package.jsonPatch
@devup-ui/rsbuild-plugin@1.0.67 → 1.0.68 - packages/rsbuild-plugin/package.jsonPatch
@devup-ui/vite-plugin@1.0.73 → 1.0.74 - packages/vite-plugin/package.jsonPatch
@devup-ui/webpack-plugin@1.0.71 → 1.0.72 - packages/webpack-plugin/package.jsonPatch
|
Codecov Report✅ All modified and coverable lines are covered by tests.
🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #686, #682 (W48).
origin/main의11790bef에서 시작한 독립 PR이며 먼저 병합해야 하는 다른 PR은 없습니다.요약
codeExtractWithoutSourceMap의 실제 WASM export가 이름과 달리SourceMapMode::Generate를 전달하여 소스맵을 생성하던 버그를 수정합니다. native 테스트용 내부 함수는 이미Skip을 사용하므로 기존 Rust 테스트만으로는 이 export 버그를 찾을 수 없었습니다.프로덕션 변경은 export의
Generate를Skip으로 바꾸는 한 줄입니다. WASM 및 회귀 테스트가 들어간 Next 패키지에Patchchangepack을 추가합니다.동작
map은undefined이고 Oxc의 소스맵 생성 경로를 실행하지 않습니다. 모듈 resolver가 있는 경우와 없는 경우 모두 검증합니다.codeExtract는 원본 소스가 들어 있는 v3 map을 그대로 생성합니다.codeExtract를 사용합니다. Next는sourceMap:false일 때 no-map export를 선택하지만 optional map을 복사/전달하므로 사전 확인을 거쳐 진행했습니다. 실제 coordinator HTTP 및 loader에서 prewarm 적중/미적중 모두 map 비활성화는null, 활성화는 기존 map을 그대로 전달함을 검증했습니다. 플러그인 프로덕션 정책은 바꾸지 않습니다.수정 전 main을 새로 WASM 빌드한 뒤 다음 입력을 실제 export에 전달하면
map이 v3 문자열이어서 map 부재 회귀 검증이 실패합니다. 수정 후 같은 호출은 다음 코드/CSS를 그대로 반환하면서 map을 반환하지 않습니다.간단한 실제 앱 측정
기존
apps/next의 Next 16.3.6/Turbopack 프로덕션 빌드를 전후 각각 한 번 실행했습니다. 기존vite-lib앱 의존성을 먼저 빌드했으며 앱 소스는 수정하지 않았습니다.공유 머신과 서로 다른 incremental 캐시 상태에서 얻은 단일 표본입니다. 전체 빌드가 빨라졌다는 주장이나 재현 가능한 성능 개선 수치로 해석하지 않습니다. 불필요한 map이 실제로 생성되지 않는 점은 시간 대신 회귀 테스트로 검증합니다.
새로 생기는 오류
없습니다. 기존 오류 및 source-map 활성화 동작은 유지됩니다.
남는 한계
다른 플러그인의 기존 source-map 선택 정책은 변경하지 않습니다. 로컬 normal-format Rust tarpaulin에는 기존 여러 줄 구문 계측 공백이 있으므로 전체 Rust 100% 판정은 CI의 기존 wide-format 설정과
--fail-under 100결과를 기준으로 합니다. LSP는 TypeScript/Biome 미설치 및 Rust daemon timeout으로 이용할 수 없었으며 통과로 기록하지 않습니다.검증
cargo fmt --all, Rust 1.99clippy --workspace --all-targets -- -D warnings통과.INSTA_FORCE_PASS를 해제한cargo test --workspace: 2,395개 통과, 0 실패. 기존 스냅샷 및.snap.new변경 없음.verify:dist, 실제 Next 앱 빌드 통과.bun lint: 0 오류, 기존 경고 2개만 존재.Linux CI 37909481974: success, head
1a606ef3f5e75a842df19950b76fb42e60acf670.publish,benchmark,landing-next-e2e,vinext-rsc-css-e2e모두 success.