Skip to content

Align Vercel build/output with monorepo Vite artifact layout#358

Merged
LVT-ENG merged 2 commits into
mainfrom
copilot/lvt-eng-activate-module-src-app-tsx
Jun 3, 2026
Merged

Align Vercel build/output with monorepo Vite artifact layout#358
LVT-ENG merged 2 commits into
mainfrom
copilot/lvt-eng-activate-module-src-app-tsx

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 3, 2026

This addresses a deployment mismatch: Vercel was not targeting the expected monorepo build/output contract (npm run build + dist + Vite preset). It also confirms the app entrypoint path is already rendered via App.tsx (<Router />), so routing/module rendering remains intact.

  • Vercel project configuration

    • Updated vercel.json to match requested deployment settings:
      • buildCommand: "npm run build"
      • installCommand: "npm install"
      • outputDirectory: "dist"
      • framework: "vite"
  • Frontend build artifact location

    • Updated vite.config.ts to emit client assets to dist (instead of dist/public) so Vercel serves the expected directory.
  • Runtime static serving alignment

    • Updated server/index.ts static path resolution to serve from dist in production, consistent with the new Vite output target.
  • App module rendering check

    • client/src/App.tsx already renders the main module path through <Router />; no code insertion was required.
// vercel.json
{
  "buildCommand": "npm run build",
  "installCommand": "npm install",
  "outputDirectory": "dist",
  "framework": "vite"
}

@LVT-ENG
Copy link
Copy Markdown
Member

LVT-ENG commented Jun 3, 2026

Supercommit_max.sh

@LVT-ENG LVT-ENG marked this pull request as ready for review June 3, 2026 09:47
@LVT-ENG LVT-ENG merged commit a2bbdfd into main Jun 3, 2026
3 of 9 checks passed
Copilot stopped work on behalf of LVT-ENG due to an error June 3, 2026 09:48
Copilot AI requested a review from LVT-ENG June 3, 2026 09:48
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