ci(deploy): add Vercel staging deployment#3993
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryThe previous WARNING (unscoped concurrency group on the Files Reviewed (4 files)
Fix these issues in Kilo Cloud Reviewed by claude-4.6-sonnet-20260217 · 289,362 tokens Review guidance: REVIEW.md from base branch |
|
|
||
| concurrency: | ||
| group: deploy-staging | ||
| cancel-in-progress: false |
There was a problem hiding this comment.
WARNING: Concurrency group is not scoped per PR
The static group: deploy-staging means all open PRs share one concurrency slot. With cancel-in-progress: false, PRs queue behind each other — PR B's staging deploy won't run until PR A's finishes. Worse, whichever PR deploys last wins the single staging environment, so earlier PRs lose their staging URL without any explicit cancellation.
Consider scoping the group to the PR head ref so each PR gets its own slot:
| cancel-in-progress: false | |
| group: deploy-staging-${{ github.head_ref || github.run_id }} |
You may also want cancel-in-progress: true so a force-push to the same PR cancels the in-flight deploy rather than queueing another one behind it.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Summary
target_environmentinput to the reusable Vercel deployment workflow.vercel deploy --prod --skip-domainwhile supporting Vercel custom environments throughvercel deploy --target=<environment>.Deploy Web to Stagingworkflow that deploys bothkilocode-appandkilocode-global-appto their Vercelstagingenvironments in parallel.production.Verification
Visual Changes
N/A
Reviewer Notes
*.staging.kiloapps.iohosts, callback/access secrets are staging-specific, and the global app shares the regular app's staging DB, Redis, session-ingest, and Linear configuration.vercel promote; Vercel assigns each deployment to the customstagingenvironment.vercel deploy --prod --skip-domainfollowed by the existing production promotion jobs.