Skip to content

Fix: Vercel build error by adding root vercel.json for monorepo#10

Open
raziquehasan wants to merge 2 commits into
ASHISHVERMASH:masterfrom
raziquehasan:master
Open

Fix: Vercel build error by adding root vercel.json for monorepo#10
raziquehasan wants to merge 2 commits into
ASHISHVERMASH:masterfrom
raziquehasan:master

Conversation

@raziquehasan

Copy link
Copy Markdown
Contributor

thik kar diye

Copilot AI review requested due to automatic review settings March 12, 2026 17:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a root-level Vercel configuration so Vercel can build/deploy the Frontend Vite app in this monorepo layout (with Backend living separately).

Changes:

  • Add root vercel.json to run the Vite build inside Frontend and publish Frontend/dist.
  • Remove an unused root package-lock.json (there’s no root package.json; the real lockfile is under Frontend/).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
vercel.json Configures Vercel build/install commands and output directory for the Frontend Vite app.
package-lock.json Removes a root lockfile that doesn’t correspond to any root Node project.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread vercel.json
@@ -0,0 +1,6 @@
{
"buildCommand": "cd Frontend && npm install && npm run build",

Copilot AI Mar 12, 2026

Copy link

Choose a reason for hiding this comment

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

buildCommand runs npm install even though installCommand already does. On Vercel this typically means dependencies are installed twice, which slows builds and can undermine caching. Consider removing the install step from buildCommand (or removing installCommand and keeping install only once) so install happens a single time.

Suggested change
"buildCommand": "cd Frontend && npm install && npm run build",
"buildCommand": "cd Frontend && npm run build",

Copilot uses AI. Check for mistakes.
@vercel

vercel Bot commented Mar 12, 2026

Copy link
Copy Markdown

@raziquehasan is attempting to deploy a commit to the ashishvermash's projects Team on Vercel.

A member of the Team first needs to authorize it.

@AarishNasim AarishNasim left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good

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.

3 participants