Skip to content

fix(ui): add vitest.config.ts to fix TypeScript 7 test failure in CI#1309

Open
Hunter275 with Copilot wants to merge 2 commits into
mainfrom
copilot/revert-1307-changes
Open

fix(ui): add vitest.config.ts to fix TypeScript 7 test failure in CI#1309
Hunter275 with Copilot wants to merge 2 commits into
mainfrom
copilot/revert-1307-changes

Conversation

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

packages/ui had no vitest.config.ts, so vitest fell back to loading its vite.config.ts. That build config uses vite-plugin-dtsunplugin-dts, which requires the TypeScript 6 Compiler API — removed in TypeScript 7 (now a pure Go binary). This caused a startup crash across all PRs:

Error: [unplugin-dts] The installed "typescript" package does not provide the JavaScript Compiler API (this happens with TypeScript 7+)

Related Issues

Relates to #1307

Changes Made

  • Added packages/ui/vitest.config.ts with defineProject and include: [] — vitest uses this instead of falling back to vite.config.ts, correctly treating packages/ui as a test-free project

Testing Done

CI on this PR should pass the test step that was failing on #1307.

Screenshots (if applicable)

N/A

Checklist

  • Code follows project style guidelines
  • Documentation has been updated or added
  • Tests have been added or updated
  • All i18n translation labels have been added (read
    CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)

@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
web-test Ready Ready Preview, Comment Jul 19, 2026 3:59am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants