Skip to content

Commit 5cd9096

Browse files
tbjersclaude
andcommitted
Fix deploy: remove wrangler from dashboard deps, install v4 in CI
@sveltejs/adapter-cloudflare@4.x declares a peer dep on wrangler@^3; having wrangler@^4 in devDependencies caused an ERESOLVE conflict on npm ci. Remove it from package.json and install wrangler@^4 globally as a dedicated CI step before the deploy, keeping peer deps clean. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 1dd4c8a commit 5cd9096

3 files changed

Lines changed: 745 additions & 567 deletions

File tree

.github/workflows/deploy-dashboard.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,9 @@ jobs:
3333
working-directory: dashboard
3434
run: npm ci
3535

36+
- name: Install wrangler
37+
run: npm install -g wrangler@^4
38+
3639
- name: Build
3740
working-directory: dashboard
3841
run: npm run build
@@ -41,7 +44,7 @@ jobs:
4144
id: deploy
4245
working-directory: dashboard
4346
run: |
44-
OUTPUT=$(node_modules/.bin/wrangler pages deploy \
47+
OUTPUT=$(wrangler pages deploy \
4548
${{ github.ref != 'refs/heads/main' && format('--branch {0}', github.head_ref || github.ref_name) || '' }})
4649
echo "$OUTPUT"
4750
URL=$(echo "$OUTPUT" | grep -oP 'https://[^\s]+\.pages\.dev[^\s]*' | tail -1)

0 commit comments

Comments
 (0)