[dexhunter]

AI usage

• No AI was used in this PR.
• AI was used: Codex + GPT-6 Astra and GPT-5.6 Luna.
  • I have carefully reviewed the AI-generated content myself.

Description

Remove side-effect CommonJS require calls that point to deleted pure CSS chunks so unbundled entries importing CSS can be loaded. Add coverage for minified output, short paths, and unrelated calls that must remain unchanged.

Linked Issues

Fixes #1075, Fixes #912

Additional context

CSS, unbundle, and helper tests: 119 passed, 3 expected failures. Lint and typecheck passed.

Baseline and candidate checks on Weco (retrospective record).

[netlify]

✅ Deploy Preview for tsdown-main ready!

Name	Link
🔨 Latest commit	089824a
🔍 Latest deploy log	https://app.netlify.com/projects/tsdown-main/deploys/6ae34d39e8290b4f3dc6f4a8
😎 Deploy Preview	https://deploy-preview-1077--tsdown-main.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[pkg-pr-new]

Open in StackBlitz

tsdown

pnpm add https://pkg.pr.new/rolldown/tsdown@1077 -D

npm i https://pkg.pr.new/rolldown/tsdown@1077 -D

yarn add https://pkg.pr.new/rolldown/tsdown@1077.tgz -D

create-tsdown

pnpm add https://pkg.pr.new/rolldown/tsdown/create-tsdown@1077 -D

npm i https://pkg.pr.new/rolldown/tsdown/create-tsdown@1077 -D

yarn add https://pkg.pr.new/rolldown/tsdown/create-tsdown@1077.tgz -D

@tsdown/css

pnpm add https://pkg.pr.new/rolldown/tsdown/@tsdown/css@1077 -D

npm i https://pkg.pr.new/rolldown/tsdown/@tsdown/css@1077 -D

yarn add https://pkg.pr.new/rolldown/tsdown/@tsdown/css@1077.tgz -D

@tsdown/exe

pnpm add https://pkg.pr.new/rolldown/tsdown/@tsdown/exe@1077 -D

npm i https://pkg.pr.new/rolldown/tsdown/@tsdown/exe@1077 -D

yarn add https://pkg.pr.new/rolldown/tsdown/@tsdown/exe@1077.tgz -D

tsdown-migrate

pnpm add https://pkg.pr.new/rolldown/tsdown/tsdown-migrate@1077 -D

npm i https://pkg.pr.new/rolldown/tsdown/tsdown-migrate@1077 -D

yarn add https://pkg.pr.new/rolldown/tsdown/tsdown-migrate@1077.tgz -D

commit: 089824a

[Copilot]

🟢 Approval recommended

No unresolved review comments remain, and the changes include targeted regression coverage.

Pull request overview

Fixes dangling CommonJS require calls left by removed pure CSS chunks.

Changes:

• Adds path-aware AST replacement for CSS requires.
• Preserves unrelated requires and source offsets.
• Expands unit and integration coverage, including minified output.

File summaries

File	Description
tests/utils.ts	Adds output inspection and runtime helpers.
tests/css.test.ts	Adds CSS and unbundle integration coverage.
packages/css/src/pure-chunk.ts	Removes dangling CSS imports and requires.
packages/css/src/pure-chunk.test.ts	Tests replacement behavior and edge cases.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[dexhunter]

Merged main at 83b6fb5 in commit 089824a and resolved the test helper conflict. Both the lint directive and CJS helper import remain.

pnpm test run packages/css/src/pure-chunk.test.ts tests/css.test.ts tests/unbundle.test.ts passed: 120 tests, with 3 expected failures. pnpm lint and pnpm typecheck also passed.

Four CSS regression cases fail with upstream's chunk removal code. All six selected cases pass with this fix.

AI-assisted: Codex prepared this merge and ran the checks.

[sxzzbot]

Superseded by #1114, which was merged. It removes the require() with the same regex-based approach Vite uses, instead of AST parsing, to match how the ESM imports are already handled. The Card2 name is fixed separately in Rolldown (rolldown/rolldown#11410).