fix: Vercel production readiness for React/Vite SPA by Copilot · Pull Request #61 · Krosebrook/interact

Copilot · 2026-02-21T02:25:24Z

Several vercel.json misconfigurations and missing Node version signals would cause broken deployments and silent runtime failures. This PR fixes the blockers and adds deployment documentation.

`vercel.json` fixes

Wrong env var name — VITE_BASE44_PROJECT_ID → VITE_BASE44_APP_ID + VITE_BASE44_BACKEND_URL (matches actual app config; old name would silently be undefined at runtime)
Missing SPA fallback — added /(.*) → /index.html rewrite; without it, React Router hard-navigations and page refreshes 404
No-op rewrite removed — /api/:path* → /api/:path* did nothing
Invalid Node version env var removed — env.NODE_VERSION in vercel.json is not how Vercel reads the Node version; replaced with proper signals below

Node version

Added .nvmrc → 20
Added engines.node: ">=20" to package.json

CI

actions/upload-artifact and actions/download-artifact: v3 → v4 (v3 is EOL)

Docs

README: added Deploy to Vercel section — one-click deploy badge, build settings table, required/optional env vars
docs/operations/VERCEL_READINESS_CHECKLIST.md: reusable 12-point audit template for other Krosebrook repos

Original prompt

Assess repository Krosebrook/interact (ref main) for Vercel production readiness and implement fixes to remove blockers.

Acceptance criteria:

Detect app type (Vercel Web App vs Vercel Functions/API vs N/A) and document it.

Ensure it builds and deploys on Vercel with minimal configuration.

Verify package manager/lockfile; add missing lockfile guidance.

Verify package.json scripts include build and start (or appropriate for framework).

Ensure Node version compatibility (add .nvmrc / engines field if helpful).

Add vercel.json only if necessary (monorepo, custom output, rewrites, functions config).

Add README section: Deploy to Vercel, build settings, output dir, required env vars.

Add .env.example capturing required env vars (no secrets).

Add/update GitHub Actions CI to run tests/build (optional but recommended).

Produce a short report in PR description: readiness status, detected framework, blockers fixed, remaining blockers.

Also create a high-level checklist template that can be reused to audit other Krosebrook repos for Vercel readiness (but only change files within this repo).

This pull request was created from Copilot chat.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Summary by cubic

Prepares this repo for Vercel by fixing vercel.json (env vars and SPA routing), adding Node 20 config, improving deploy docs, and upgrading CI. Aligns with the Linear task to assess Vercel readiness and remove blockers.

New Features
- README: One‑click deploy, build settings (Vite, npm run build, dist), env vars table, SPA routing note.
- vercel.json: correct VITE vars, remove NODE_VERSION, add catch‑all rewrite to /index.html, use npm ci.
- Node 20 set via .nvmrc and engines.node.
- CI: artifact actions upgraded to v4.
- Reusable Vercel readiness checklist added.
Migration
- App type: Vite SPA; build command and output dir configured.
- Remaining: verify lockfile/install alignment (npm ci + package-lock.json) and set env vars in Vercel project.

^{Written for commit 04e7eae. Summary will update on new commits.}

…es, deploy docs, CI upgrades Co-authored-by: Krosebrook <214532761+Krosebrook@users.noreply.github.com>

Copilot

Pull request overview

This PR fixes critical Vercel deployment blockers and adds comprehensive deployment documentation for the Interact platform. The changes align environment variables with actual code usage, enable proper React Router SPA fallback routing, establish Node.js version compatibility, upgrade deprecated CI actions, and provide detailed deployment guidance.

Changes:

Corrected environment variable names in vercel.json from incorrect VITE_BASE44_PROJECT_ID to actual VITE_BASE44_APP_ID and VITE_BASE44_BACKEND_URL used in the codebase
Added critical SPA fallback rewrite /(.*) → /index.html to prevent React Router 404s on hard navigation
Replaced invalid env.NODE_VERSION in vercel.json with proper Node 20 signals via .nvmrc and package.json engines field
Upgraded GitHub Actions artifact actions from EOL v3 to v4
Added comprehensive Vercel deployment documentation to README and created reusable readiness checklist

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
`vercel.json`	Fixed env var names (APP_ID + BACKEND_URL), removed invalid NODE_VERSION, added SPA catch-all rewrite, removed no-op API rewrite
`package.json`	Added Node.js engines constraint (>=20) for Vercel detection
`.nvmrc`	Added Node 20 version pin for local development and CI consistency
`.github/workflows/ci.yml`	Upgraded upload-artifact and download-artifact from v3 to v4 (v3 EOL)
`README.md`	Added Deploy to Vercel section with one-click deploy badge, build settings table, required/optional env vars, and SPA routing explanation
`docs/operations/VERCEL_READINESS_CHECKLIST.md`	Created comprehensive 12-point audit template for Vercel deployments across Krosebrook repos

cubic-dev-ai

3 issues found across 6 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="README.md">

<violation number="1" location="README.md:170">
P3: The new Node.js requirement in the Vercel build settings conflicts with the existing Quick Start prerequisite (`Node.js 18+`), leaving contradictory setup guidance in the same README.</violation>
</file>

<file name="vercel.json">

<violation number="1" location="vercel.json:11">
P1: Documentation files reference obsolete `VITE_BASE44_API_KEY` and `VITE_API_URL` environment variables not used by actual application code</violation>

<violation number="2" location="vercel.json:44">
P1: The new SPA rewrite is too broad and rewrites `/api/*` requests to `index.html`, breaking same-origin API calls (for example `/api/logError`). Exclude API routes from the catch-all rewrite.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

cubic-dev-ai · 2026-03-05T04:19:03Z

vercel.json

    {
-      "source": "/api/:path*",
-      "destination": "/api/:path*"
+      "source": "/(.*)",


P1: The new SPA rewrite is too broad and rewrites /api/* requests to index.html, breaking same-origin API calls (for example /api/logError). Exclude API routes from the catch-all rewrite.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At vercel.json, line 44: <comment>The new SPA rewrite is too broad and rewrites `/api/*` requests to `index.html`, breaking same-origin API calls (for example `/api/logError`). Exclude API routes from the catch-all rewrite.</comment> <file context> @@ -44,8 +41,8 @@ { - "source": "/api/:path*", - "destination": "/api/:path*" + "source": "/(.*)", + "destination": "/index.html" } </file context>

Suggested change

"source": "/(.*)",

"source": "/((?!api/).*)",

cubic-dev-ai · 2026-03-05T04:19:03Z

vercel.json

@@ -5,13 +5,10 @@
  "outputDirectory": "dist",


P1: Documentation files reference obsolete VITE_BASE44_API_KEY and VITE_API_URL environment variables not used by actual application code

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At vercel.json, line 11: <comment>Documentation files reference obsolete `VITE_BASE44_API_KEY` and `VITE_API_URL` environment variables not used by actual application code</comment> <file context> @@ -5,13 +5,10 @@ - "VITE_API_URL": "@api_url", - "VITE_BASE44_PROJECT_ID": "@base44_project_id" + "VITE_BASE44_APP_ID": "@vite_base44_app_id", + "VITE_BASE44_BACKEND_URL": "@vite_base44_backend_url" } }, </file context>

cubic-dev-ai · 2026-03-05T04:19:03Z

README.md

+| **Build Command** | `npm run build` |
+| **Output Directory** | `dist` |
+| **Install Command** | `npm ci` |
+| **Node.js Version** | 20.x |


P3: The new Node.js requirement in the Vercel build settings conflicts with the existing Quick Start prerequisite (Node.js 18+), leaving contradictory setup guidance in the same README.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At README.md, line 170: <comment>The new Node.js requirement in the Vercel build settings conflicts with the existing Quick Start prerequisite (`Node.js 18+`), leaving contradictory setup guidance in the same README.</comment> <file context> @@ -152,7 +153,49 @@ node scripts/generate-prd.js --file ideas/feature.txt +| **Build Command** | `npm run build` | +| **Output Directory** | `dist` | +| **Install Command** | `npm ci` | +| **Node.js Version** | 20.x | + +### Required Environment Variables </file context>

Initial plan

3e99f4d

Copilot AI assigned Copilot and Krosebrook Feb 21, 2026

Copilot started work on behalf of Krosebrook February 21, 2026 02:25 View session

fix: Vercel production readiness - fix vercel.json, add .nvmrc, engin…

04e7eae

…es, deploy docs, CI upgrades Co-authored-by: Krosebrook <214532761+Krosebrook@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Assess repository for Vercel production readiness and implement fixes~~ fix: Vercel production readiness for React/Vite SPA Feb 21, 2026

Copilot AI requested a review from Krosebrook February 21, 2026 02:32

Copilot finished work on behalf of Krosebrook February 21, 2026 02:32

Krosebrook requested a review from Copilot February 21, 2026 22:11

Copilot started reviewing on behalf of Krosebrook February 21, 2026 22:12 View session

Copilot AI reviewed Feb 21, 2026

View reviewed changes

Krosebrook approved these changes Mar 5, 2026

View reviewed changes

Krosebrook marked this pull request as ready for review March 5, 2026 04:14

cubic-dev-ai bot reviewed Mar 5, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: Vercel production readiness for React/Vite SPA#61

fix: Vercel production readiness for React/Vite SPA#61
Copilot wants to merge 2 commits intomainfrom
copilot/assess-vercel-production-readiness

Copilot AI commented Feb 21, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

cubic-dev-ai bot left a comment

Uh oh!

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading

Uh oh!

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading

Uh oh!

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

vercel.json fixes

Node version

CI

Docs

Summary by cubic

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai bot Mar 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Feb 21, 2026 •

edited

Loading

`vercel.json` fixes

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading

cubic-dev-ai bot Mar 5, 2026 •

edited

Loading