Skip to content

ci(workers): add manual deploy choices#3435

Closed
jrf0110 wants to merge 1 commit into
mainfrom
manual-deploy-workers
Closed

ci(workers): add manual deploy choices#3435
jrf0110 wants to merge 1 commit into
mainfrom
manual-deploy-workers

Conversation

@jrf0110
Copy link
Copy Markdown
Contributor

@jrf0110 jrf0110 commented May 22, 2026

Summary

  • Replace the freeform worker path input in the Cloudflare Workers deploy workflow with a fixed dropdown of standard Wrangler-deployable workers.
  • Preserve the existing push-to-main behavior that deploys only changed workers.

Verification

N/A

Visual Changes

N/A

Reviewer Notes

  • Manual dispatch now only allows workers listed in the workflow choices; adding a new Wrangler worker will require adding it to this list if manual deploy should be available.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 22, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

This CI workflow change is clean — replacing the freeform worker path input with a validated dropdown of 27 Wrangler-deployable workers. The choices align correctly with the workers that have wrangler.jsonc files, and the intentional exclusions (services/kiloclaw for Docker deploy, services/deploy-infra/builder-docker-container/container-files as a build template) remain consistent between the manual-dispatch list and the push-to-main detection script.

Files Reviewed (1 file)
  • .github/workflows/deploy-workers.yml

Reviewed by claude-sonnet-4.6 · 217,673 tokens

Review guidance: REVIEW.md from base branch main

@jrf0110 jrf0110 closed this May 22, 2026
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.

1 participant