Skip to content

ci(deploy): let Vercel build main app#3945

Merged
RSO merged 1 commit into
mainfrom
vercel-build-app
Jun 11, 2026
Merged

ci(deploy): let Vercel build main app#3945
RSO merged 1 commit into
mainfrom
vercel-build-app

Conversation

@RSO

@RSO RSO commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Switch the main app production deployment from runner-built prebuilt artifacts to Vercel-side production builds.
  • Apply the same behavior to the manual web redeploy workflow.
  • Keep checkout and .nvmrc Node setup for the Vercel CLI, but remove runner-side dependency install, Next cache, Vercel pull, and local build steps.

Verification

  • Confirmed the workflow diff only changes the deploy-app jobs in the production and manual redeploy workflows.

Visual Changes

N/A

Reviewer Notes

  • This follows the global-app rollout pattern after moving that lower-risk deploy path first.
  • Vercel project build settings for kilocode-app must be correct because GitHub no longer runs vercel build for that project.

@kilo-code-bot

kilo-code-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The PR correctly migrates deploy-app in both deploy-production.yml and redeploy-web.yml from runner-side prebuilt deploys to Vercel-side builds, matching the existing pattern used by deploy-global-app.

Files Reviewed (2 files)
  • .github/workflows/deploy-production.yml
  • .github/workflows/redeploy-web.yml

Reviewed by claude-4.6-sonnet-20260217 · 386,608 tokens

Review guidance: REVIEW.md from base branch main

@RSO RSO merged commit 575de09 into main Jun 11, 2026
16 checks passed
@RSO RSO deleted the vercel-build-app branch June 11, 2026 07:04
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