Skip to content

Commit f01a04b

Browse files
willwashburnclaude
andauthored
ci(observer): fix ERESOLVE in dashboard deploy from wrangler peer bump (#245)
The deploy step runs `npx @cloudflare/next-on-pages@1`, whose ephemeral install resolves the latest wrangler (4.108.0). That wrangler now peers on @cloudflare/workers-types@^5, while next-on-pages peers on ^4, so npm aborts the tree with ERESOLVE and the deploy fails. Pin next-on-pages to 1.13.16 and set NPM_CONFIG_LEGACY_PEER_DEPS so the optional-peer mismatch no longer blocks the install. Claude-Session: https://claude.ai/code/session_01DatJJXxGLYDdgTWzT9zncq Co-authored-by: Claude <noreply@anthropic.com>
1 parent 4f8d2eb commit f01a04b

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

  • .github/actions/deploy-observer-dashboard

.github/actions/deploy-observer-dashboard/action.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,12 @@ runs:
5858
run: |
5959
set -euo pipefail
6060
cd packages/observer-dashboard
61-
npx @cloudflare/next-on-pages@1
61+
npx @cloudflare/next-on-pages@1.13.16
6262
npx wrangler pages deploy .vercel/output/static --project-name="${{ inputs.project-name }}" --branch="${{ inputs.branch }}"
6363
env:
6464
CLOUDFLARE_API_TOKEN: ${{ inputs.cloudflare-api-token }}
6565
CLOUDFLARE_ACCOUNT_ID: ${{ inputs.cloudflare-account-id }}
66+
# next-on-pages peers on @cloudflare/workers-types@^4, but the wrangler it
67+
# transitively resolves now peers on ^5; tolerate that optional-peer mismatch
68+
# so the ephemeral npx install can resolve. See CI ERESOLVE on wrangler 4.108.0.
69+
NPM_CONFIG_LEGACY_PEER_DEPS: "true"

0 commit comments

Comments
 (0)