fix(bundler): axios/toolkit 스모크 테스트 — 5가지 버그 수정#296
Merged
Conversation
1. exports fallback array: package.json exports 값이 배열일 때 순서대로 시도
(has-tostringtag/shams → [{"default":"./shams.js"}, "./shams.js"])
2. 서브패스 resolve: splitBareSpecifier의 subpath가 "/shams" 형태(leading /)로
std.fs.path.resolve가 절대 경로로 취급하는 문제 수정
3. CJS require tree-shaking: sideEffects:false 패키지를 CJS require()로
참조할 때 tree-shaker가 제외하는 버그 — require 타겟은 항상 포함
4. 외부 모듈 ESM import: scope hoisting이 import 문을 제거하지만
require() preamble을 생성하지 않아 변수가 미정의되는 버그 수정
5. 글로벌 shadowing deconflict: const TextEncoder 등 모듈 지역 변수가
well-known global을 shadowing할 때 TDZ 방지를 위해 리네임
스모크 테스트 5/5 → 7/7 (axios, @reduxjs/toolkit 추가)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Benchmark Results (CI)macos-latesttranspile — small (100 lines)
transpile — medium (1K lines)
transpile — large (5K lines)
bundle — small (10 modules)
bundle — medium (50 modules)
bundle — large (200 modules)
ubuntu-latesttranspile — small (100 lines)
transpile — medium (1K lines)
transpile — large (5K lines)
bundle — small (10 modules)
bundle — medium (50 modules)
bundle — large (200 modules)
windows-latesttranspile — small (100 lines)
transpile — medium (1K lines)
transpile — large (5K lines)
bundle — small (10 modules)
bundle — medium (50 modules)
bundle — large (200 modules)
|
…fier 정규화 1. isReservedName: O(n) 선형 탐색 → comptime StaticStringMap O(1) 조회 2. splitBareSpecifier: subpath를 "/sub" → "sub"로 반환하여 caller의 leading / 제거 workaround 및 "." concat 할당 제거 3. external preamble: namespace/default vs named 분기 통합 (중복 제거) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
/simplify에서 subpath 반환값을 변경했으나 기존 계약 유지가 맞음. caller(resolvePackage)에서 leading / 처리. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
has-tostringtag등)/처리 수정 (has-symbols/shams)TextEncoder,URL등)Test plan
zig build test통과🤖 Generated with Claude Code