Skip to content
This repository was archived by the owner on Feb 25, 2026. It is now read-only.

fix: set worker format to es in kilo-ui storybook vite config#449

Merged
catrielmuller merged 1 commit into
devfrom
fix/kilo-ui-storybook-worker-format
Feb 18, 2026
Merged

fix: set worker format to es in kilo-ui storybook vite config#449
catrielmuller merged 1 commit into
devfrom
fix/kilo-ui-storybook-worker-format

Conversation

@catrielmuller

Copy link
Copy Markdown
Contributor

Summary

  • Fixes Storybook build failure in packages/kilo-ui/ caused by @pierre/diffs worker being loaded via ?worker&url
  • Vite defaults worker format to iife, which is incompatible with code-splitting builds, resulting in: "Invalid value 'iife' for option 'worker.format' - UMD and IIFE output formats are not supported for code-splitting builds"
  • Adds worker: { format: "es" } to the Vite config in .storybook/main.ts to resolve this

@vercel

vercel Bot commented Feb 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
kilo-ui-storybook Ready Ready Preview, Comment Feb 18, 2026 9:44pm

Request Review

@kilo-code-bot

kilo-code-bot Bot commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The change adds worker: { format: "es" } to the Vite config in the Storybook viteFinal hook. This correctly configures Vite to emit web workers as ES modules rather than the default IIFE format, which is appropriate for modern browser environments and consistent with the ES module setup already in place.

Files Reviewed (1 files)
  • packages/kilo-ui/.storybook/main.ts - 0 issues

@catrielmuller catrielmuller merged commit d900b2f into dev Feb 18, 2026
12 checks passed
@catrielmuller catrielmuller deleted the fix/kilo-ui-storybook-worker-format branch February 18, 2026 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants