Skip to content

chore(app): upgrade to Vite v8#1962

Open
JeanMeijer wants to merge 1 commit into
devfrom
chore/upgrade-to-vite-v8
Open

chore(app): upgrade to Vite v8#1962
JeanMeijer wants to merge 1 commit into
devfrom
chore/upgrade-to-vite-v8

Conversation

@JeanMeijer
Copy link
Copy Markdown
Collaborator

Summary

  • Upgrades the app/tooling stack to Vite 8.
  • Updates related Vite plugin/package versions across app, UI demo, Den web, and landing packages.
  • Adjusts PostCSS/Tailwind imports/config for the upgraded toolchain.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 26, 2026

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

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment May 26, 2026 1:45pm
openwork-den Ready Ready Preview, Comment May 26, 2026 1:45pm
openwork-den-worker-proxy Ready Ready Preview, Comment May 26, 2026 1:45pm
openwork-landing Ready Ready Preview, Comment, Open in v0 May 26, 2026 1:45pm
openwork-share Error Error May 26, 2026 1:45pm

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 10 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="ee/apps/den-web/package.json">

<violation number="1" location="ee/apps/den-web/package.json:25">
P2: `autoprefixer` is redundant with Tailwind CSS v4. The `@tailwindcss/postcss` plugin handles vendor prefixing internally via Lightning CSS, so keeping autoprefixer can produce duplicated prefixes in the output CSS. Remove `autoprefixer` from both `package.json` and `postcss.config.js`.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

"react-dom": "catalog:"
},
"devDependencies": {
"@tailwindcss/postcss": "4.3.0",
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot May 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2: autoprefixer is redundant with Tailwind CSS v4. The @tailwindcss/postcss plugin handles vendor prefixing internally via Lightning CSS, so keeping autoprefixer can produce duplicated prefixes in the output CSS. Remove autoprefixer from both package.json and postcss.config.js.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At ee/apps/den-web/package.json, line 25:

<comment>`autoprefixer` is redundant with Tailwind CSS v4. The `@tailwindcss/postcss` plugin handles vendor prefixing internally via Lightning CSS, so keeping autoprefixer can produce duplicated prefixes in the output CSS. Remove `autoprefixer` from both `package.json` and `postcss.config.js`.</comment>

<file context>
@@ -22,12 +22,13 @@
     "react-dom": "catalog:"
   },
   "devDependencies": {
+    "@tailwindcss/postcss": "4.3.0",
     "@types/node": "20.12.12",
     "@types/react": "19.2.14",
</file context>
Fix with Cubic

@JeanMeijer JeanMeijer marked this pull request as draft May 26, 2026 14:00
Copy link
Copy Markdown
Member

@benjaminshafii benjaminshafii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved via automated Daytona Electron smoke test (agent). Tested: Electron boot, app render (Vite v8 dev server), settings navigation, back-to-app. All passed, no regressions from the Vite upgrade.

@benjaminshafii benjaminshafii marked this pull request as ready for review May 26, 2026 20:53
@benjaminshafii
Copy link
Copy Markdown
Member

This PR has merge conflicts after the other PRs in this batch were merged (#1964, #1960, #1946, #1944). Needs a rebase to resolve. (Agent note: Daytona Electron smoke test passed before the conflicts arose -- the branch itself is good, just needs conflict resolution.)

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 10 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="ee/apps/landing/package.json">

<violation number="1" location="ee/apps/landing/package.json:23">
P2: `autoprefixer` is redundant with Tailwind CSS v4 and should be removed. According to the Tailwind v4 upgrade guide, `@tailwindcss/postcss` handles vendor prefixing automatically via Lightning CSS. Keeping both may cause double-prefixing and adds an unnecessary dependency. Remove `autoprefixer` from `package.json` and from `postcss.config.js`.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

"react-dom": "catalog:react18"
},
"devDependencies": {
"@tailwindcss/postcss": "4.3.0",
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot May 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2: autoprefixer is redundant with Tailwind CSS v4 and should be removed. According to the Tailwind v4 upgrade guide, @tailwindcss/postcss handles vendor prefixing automatically via Lightning CSS. Keeping both may cause double-prefixing and adds an unnecessary dependency. Remove autoprefixer from package.json and from postcss.config.js.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At ee/apps/landing/package.json, line 23:

<comment>`autoprefixer` is redundant with Tailwind CSS v4 and should be removed. According to the Tailwind v4 upgrade guide, `@tailwindcss/postcss` handles vendor prefixing automatically via Lightning CSS. Keeping both may cause double-prefixing and adds an unnecessary dependency. Remove `autoprefixer` from `package.json` and from `postcss.config.js`.</comment>

<file context>
@@ -20,12 +20,13 @@
     "react-dom": "catalog:react18"
   },
   "devDependencies": {
+    "@tailwindcss/postcss": "4.3.0",
     "@types/node": "20.12.12",
     "@types/react": "18.2.79",
</file context>
Fix with Cubic

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