fix(pack): inlineCss with autoCssModules cause panic#2814
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces performance optimizations for one-shot build sessions by adding an is_short_session hint to the turbo-tasks engine. This allows the system to use ReadWriteOnShutdown storage mode and conditionally disable dependency tracking when persistent caching is not required, reducing bookkeeping overhead. Additionally, the changes include a fix for CSS module facade analysis to prevent class extraction failures when Analyze references are inlined into JavaScript. New snapshot tests for Less CSS modules have also been added to ensure correct behavior. I have no feedback to provide as there were no review comments to evaluate.
866027b to
3f7aa9b
Compare
📊 Performance Benchmark Report (with-antd)Utoopack Performance ReportReport ID: Executive Summary
Build Phase TimelineShows when each build phase is active and how much CPU it consumes.
Workload Distribution by Diagnostic Tier
Top 20 Tasks by Self-TimeSelf-time is the exclusive duration: time spent in the task itself, not in sub-tasks.
Critical Path AnalysisThe longest sequential dependency chains that determine wall-clock time.
Batching CandidatesHigh-volume tasks dominated by a single parent. If the parent can batch them,
Duration Distribution
Action Items
Report generated by Utoopack Performance Analysis Agent |
Summary
styles.inlineCss+ autoCssModules will cause panic, a case from smallfish widget build.Test Plan
Update test case for both library and client