Skip to content

feat(web-env): support scoped rotations and redeploys - #5334

Merged
RSO merged 2 commits into
mainfrom
iterate-on-webenv-command-2
Aug 18, 2026
Merged

feat(web-env): support scoped rotations and redeploys#5334
RSO merged 2 commits into
mainfrom
iterate-on-webenv-command-2

Conversation

@RSO

@RSO RSO commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add --only development|staging|production to rotate one Vercel environment across both web projects
  • prompt to redeploy affected Staging or Production deployments after changing environment values
  • redeploy both Vercel projects concurrently from their latest ready deployment
  • document scoped rotation behavior and expand focused parser/redeploy coverage

Verification

  • pnpm exec tsc -p scripts/web-env/tsconfig.json
  • pnpm exec oxlint scripts/web-env/*.ts
  • pnpm run test:web-env
  • git diff --check origin/main...HEAD

Comment thread scripts/web-env/index.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
scripts/web-env/index.ts 153 Plan output over-promises a redeploy prompt for --only development runs
Files Reviewed (7 files)
  • DEVELOPMENT.md
  • package.json
  • scripts/web-env/index.ts - 1 issue
  • scripts/web-env/options.ts
  • scripts/web-env/options.test.ts
  • scripts/web-env/shared.ts
  • scripts/web-env/shared.test.ts

The scoped-rotation option parsing, vault gating, async Vercel redeploy helper, and tests are sound; the only finding is a minor plan-message inconsistency for development-only rotations.

Fix these issues in Kilo Cloud

Previous Review Summary (commit 8fabd4f)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 8fabd4f)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
scripts/web-env/index.ts 153 Plan output over-promises a redeploy prompt for --only development runs
Files Reviewed (7 files)
  • DEVELOPMENT.md
  • package.json
  • scripts/web-env/index.ts - 1 issue
  • scripts/web-env/options.ts
  • scripts/web-env/options.test.ts
  • scripts/web-env/shared.ts
  • scripts/web-env/shared.test.ts

The scoped-rotation option parsing, vault gating, async Vercel redeploy helper, and tests are sound; the only finding is a minor plan-message inconsistency for development-only rotations.

Fix these issues in Kilo Cloud


Reviewed by kimi-k3 · Input: 11.1K · Output: 67 · Cached: 7.7K

Review guidance: REVIEW.md from base branch main

@RSO
RSO merged commit 15a5b87 into main Aug 18, 2026
14 checks passed
@RSO
RSO deleted the iterate-on-webenv-command-2 branch August 18, 2026 11:43
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