feat(web): distinguish the staging environment#4075
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryThe new commit correctly adds the Files Reviewed (4 files)
Previous Review Summaries (2 snapshots, latest commit 44fdde3)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 44fdde3)Status: No Issues Found | Recommendation: Merge Executive SummaryAdds an in-memory dismiss button (×) to the staging environment banner — clean client-side state, no hooks rule violations, proper a11y attributes. Files Reviewed (4 files)
Previous review (commit a1fb25e)Status: No Issues Found | Recommendation: Merge Executive SummaryStraightforward staging environment differentiation using Files Reviewed (4 files)
Reviewed by deepseek-v4-pro-20260423 · 136,289 tokens Review guidance: REVIEW.md from base branch |
|
@kilocode-bot please add an x to the banner, which dismisses the banner temporarily (whatever is easiest, say for 'this session, or 'this load'), so we can temporarily hide it when we want to give a demo or make a screenshot. i DO NOT want this persisted in ANY way |
|
To use Kilo from GitHub you first need to link your GitHub account to Kilo. Link your Kilo account to continue. After linking, mention me again in this issue or pull request. |
|
@kilocode-bot please add an x to the banner, which dismisses the banner temporarily (whatever is easiest, say for 'this session, or 'this load'), so we can temporarily hide it when we want to give a demo or make a screenshot. i DO NOT want this persisted in ANY way |
|
Done! The dismiss button has been added to the staging banner in PR #4075. Here's what was implemented:
|
|
@kilocode-bot please fix the failing build |
|
The build fix has been pushed! ✅ Root cause: Fix applied: Added |
Summary
VERCEL_TARGET_ENVwithout changing production or development branding.Verification
VERCEL_TARGET_ENV=stagingand confirmed the banner appears across the full viewport./kilo-v1-STAGING.svg.Visual Changes
Reviewer Notes
Staging detection relies on Vercel injecting
VERCEL_TARGET_ENV=stagingfor the custom deployment target.Built for markijbema by Kilo