Skip to content

[TW-7437] fix(ci): replace removed cloudflare/pages-action with wrangler-action - #489

Merged
benjaminwhtan merged 1 commit into
mainfrom
fix-sdk-reference-cloudflare-action
Sep 30, 2026
Merged

benjaminwhtan merged 1 commit into
mainfrom
fix-sdk-reference-cloudflare-action

Conversation

@benjaminwhtan

Copy link
Copy Markdown
Contributor

Summary

The docs job in sdk-reference.yml fails on every PR and every push to main with Unable to resolve action cloudflare/pages-action, not found. Cloudflare has removed that repo (the GitHub API returns 404) after deprecating it in favour of cloudflare/wrangler-action. The workflow hasn't changed since v6.0.0 and last passed on main on 2026-08-28. This PR switches the deploy step to the supported action.

Changes

  • Replace cloudflare/pages-action@v1 with cloudflare/wrangler-action, pinned to the v4.1.3 commit SHA (953926a…) instead of a moving tag. A moving tag is how this broke without warning.
  • Deploy with pages deploy site --project-name=nylas-python-sdk-reference --branch=${{ env.CLOUDFLARE_BRANCH }}: the same project, directory and branch as before.
  • Change the sdk-reference environment URL from outputs.url to outputs.deployment-url, which is the new action's name for it.
  • Drop wranglerVersion: "3", so the action uses its default Wrangler 4.

CLOUDFLARE_BRANCH can't be set by a PR author: it's the PR number (from refs/pull/<n>/merge) or main. So putting it into command isn't an injection risk.

Testing

  • The workflow YAML parses
  • The docs check on this PR resolves the action, builds and deploys a preview (CI on this PR is the test)
  • After merge, the main push deploys to the production branch

Related

🤖 Generated with Claude Code

…ler-action

- replace cloudflare/pages-action@v1 with cloudflare/wrangler-action pinned to the v4.1.3 commit SHA
- deploy with `wrangler pages deploy site` using the same project name and branch
- point the sdk-reference environment URL at the new deployment-url output

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@benjaminwhtan
benjaminwhtan merged commit 3955490 into main Sep 30, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
sdk-reference — cf19da05 Deployed Sep 30, 2026 by benjaminwhtan via docs #586
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