Repository navigation
fix(webpack): keep packed CommonJS loaders on the installed WASM - #789
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>
Refs #683 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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, Refs #683
다른 기능 PR에 쌓지 않는
main기반 수정입니다. 사용자 병합이 완료된 #788/#768의 main347baaf9를 실제 두 부모 merge로 통합했습니다.요약
--define import.meta.url=module.filename을 적용하여 실행 중인 설치 패키지 기준으로 loader를 찾습니다. 제품 TS/Rust, ESM/loader 빌드, 테마/reset, exports, 의존성, CSS 생성과 짧은 클래스 카운터는 변경하지 않습니다.Patch입니다.동작
수정 전 동일 빌드 추적에서 소비자 WASM은 상속 테마를 등록하여 올바른 173바이트 base CSS를 생성했습니다. 그러나 packed CJS의 절대 소스 URL이 작업 디렉터리의 JS/CSS loader를 선택했고, 그 별도 WASM에는 테마가 없어 117바이트 CSS로 덮어썼습니다. loader 반환, CSS module, 실제 HTML-linked asset까지 누락이 이어졌으며 마지막
done의 디스크 복원은 이미 발행된 CSS를 복구하지 못했습니다.CJS의
createRequire기준을module.filename으로 유지하여 플러그인과 두 loader가 동일한 설치 패키지/WASM을 사용하게 합니다. 최신 Bun이 소스 경로 변수를 만드는__filename치환은 실제 회귀를 해결하지 못해 사용하지 않았습니다. 임의 URL shim이나 테마 fallback을 추가하지 않습니다.실제 제공 CSS의 선언과 계산 결과는
--primary:#17395B,--gutter:11px, 테마rgb(23, 57, 91)/11px이며 별도 imported control은rgb(19, 87, 155)/9px입니다.새로 생기는 오류
제품 빌드 오류나 클라이언트 스타일링 런타임은 추가하지 않습니다. 회귀 CI는 실제 상속 선언/계산값 또는 설치 loader/WASM 동일성이 어긋나면 실패합니다.
set -euo pipefail과 증거 파일 필수 업로드로 테스트 실패를 숨기지 않습니다.남는 한계
초기 production 상속 테마 누락을 standalone CJS, Next
--webpack, standalone direct ESM의 두 CSS 모드에서 검증합니다. 이 변경은 해당 CJS origin의 두 loader 해석에 한정하며 다른 URL 처리로 일반화하지 않습니다.검증
e462e771685b70d60603db75c94c7c3414c3c709의 Linux CI 39194897926 SUCCESS: publish, benchmark, webpack-extends-theme, landing-next-e2e, react-18, vinext-rsc-css-e2e 모두 성공했습니다. 실제 Linux Rust 13,964/13,964=100.00%, Bun 6,341/0 및 함수/라인100%, Bun preload race 9/0입니다. release/deploy는 PR에서 의도대로 skip되었습니다.runner.temp표현식 거부로 job 실행 전에 실패했습니다. 경로를 runner step에서$RUNNER_TEMP/$GITHUB_ENV로 구성하도록 수정하고 실제 Bash(공백 경로 포함), 정상 hook, 새로운 matching-head CI로 해결했습니다. 최신 CJS SHA-2560eb45e14caaafe85795ddb2c879b5c33d40a2973ed2bf0cb4ab2d7f069052eaa가 빌드/tarball/여덟 설치 사본에서 동일합니다.