Skip to content

Replace with direct run step#52

Open
mendral-app[bot] wants to merge 1 commit into
v2from
mendral/replace-wrangler-action-with-run-step
Open

Replace with direct run step#52
mendral-app[bot] wants to merge 1 commit into
v2from
mendral/replace-wrangler-action-with-run-step

Conversation

@mendral-app
Copy link
Copy Markdown
Contributor

@mendral-app mendral-app Bot commented Apr 15, 2026

Summary

  • Replace third-party cloudflare/wrangler-action with a direct npx wrangler pages deploy run step, eliminating supply chain risk and the Node.js 20 deprecation warning
  • Remove unused deployments: write permission (was only needed by the action's GitHub deployment status feature)

Changes

.github/workflows/deploy.yaml

Before:

- uses: "cloudflare/wrangler-action@da0e0dfe58b7a431659754fdf3f186c529afbe65" # v3
  with:
    apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
    accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
    command: pages deploy dist --project-name=www-getprobo-com
    gitHubToken: ${{ secrets.GITHUB_TOKEN }}

After:

- run: npx wrangler pages deploy dist --project-name=www-getprobo-com
  env:
    CLOUDFLARE_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
    CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}

Wrangler is already installed as a project dependency via npm ci, so no additional setup is needed. The CLOUDFLARE_API_TOKEN and CLOUDFLARE_ACCOUNT_ID environment variables are read natively by wrangler.

Notes

  • The gitHubToken input was used by the action to create GitHub deployment status entries. This is no longer set. If deployment status tracking is desired, it can be added separately.
  • The deployments: write permission was only needed for that feature and has been removed.

Related insight: Deploy workflow uses third-party cloudflare/wrangler-action


Note

Created by Mendral. Tag @mendral-app with feedback or questions.

Remove third-party cloudflare/wrangler-action and replace with a direct
npx wrangler command. Wrangler is already installed via npm ci as a
project dependency.

This eliminates supply chain risk from the third-party action and
resolves the Node.js 20 deprecation warning it was emitting.

Also removes the now-unused deployments:write permission which was only
needed by the action's GitHub deployment status feature.
@mendral-app mendral-app Bot requested a review from gearnode April 15, 2026 12:09
@mendral-app mendral-app Bot marked this pull request as ready for review April 15, 2026 12:19
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.

0 participants