Skip to content

Avoid any in Cloudflare tunnel error handling#8089

Draft
alfonso-noriega wants to merge 1 commit into
mainfrom
maintenance-cloudflare-unknown-errors
Draft

Avoid any in Cloudflare tunnel error handling#8089
alfonso-noriega wants to merge 1 commit into
mainfrom
maintenance-cloudflare-unknown-errors

Conversation

@alfonso-noriega

@alfonso-noriega alfonso-noriega commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

The Cloudflare tunnel implementation used any for caught and callback errors, requiring lint suppressions and weakening type safety in production code.

WHAT is this pull request doing?

Types caught and callback errors as unknown, adds a small helper to extract an error message, removes explicit-any suppressions in the tunnel implementation, and fixes a typo in an error message.

How to test your changes?

pnpm nx run cli-kit:build --skip-nx-cache
pnpm --dir packages/plugin-cloudflare type-check
pnpm --dir packages/plugin-cloudflare lint
pnpm --dir packages/plugin-cloudflare vitest src/tunnel.test.ts

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • This change is not user-facing; no changeset is needed.

Assisted-By: devx/d9c9b8a1-2521-4fdd-af59-d77381c6f0ee

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant