Skip to content

Commit 7b16f12

Browse files
devallibusclaude
andcommitted
fix(ci): split deploy steps and add wrangler devDependency
Split deploy-mcp into separate conditional steps for production and development to avoid passing --env="" on master (which targets an empty-named environment instead of production). Add wrangler as a devDependency so local scripts work from a clean checkout. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 4c2baea commit 7b16f12

3 files changed

Lines changed: 130 additions & 3 deletions

File tree

.github/workflows/validate.yml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,21 @@ jobs:
113113
- name: Install Dependencies
114114
run: bun install --frozen-lockfile
115115

116-
- name: Deploy MCP Worker
116+
- name: Deploy MCP Worker (production)
117+
if: github.ref == 'refs/heads/master'
117118
uses: cloudflare/wrangler-action@v3
118119
with:
119120
apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
120121
workingDirectory: packages/mcp
121-
command: deploy ${{ github.ref == 'refs/heads/development' && '--env development' || '--env=""' }}
122+
command: deploy
123+
124+
- name: Deploy MCP Worker (development)
125+
if: github.ref == 'refs/heads/development'
126+
uses: cloudflare/wrangler-action@v3
127+
with:
128+
apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
129+
workingDirectory: packages/mcp
130+
command: deploy --env development
122131

123132
publish-cli:
124133
needs: check

0 commit comments

Comments
 (0)