Skip to content

[wrangler] Remove pubsub subcommands (no longer available)#12211

Merged
petebacondarwin merged 4 commits intomainfrom
remove-pubsub
Jan 29, 2026
Merged

[wrangler] Remove pubsub subcommands (no longer available)#12211
petebacondarwin merged 4 commits intomainfrom
remove-pubsub

Conversation

@elithrar
Copy link
Copy Markdown
Contributor

@elithrar elithrar commented Jan 28, 2026

The Pub/Sub product was never made publicly available and has been discontinued. This PR removes the wrangler pubsub subcommand and all related functionality.

  • Removes pubsub CLI commands (pubsub namespace, pubsub broker, etc.)
  • Removes associated test files
  • Removes pubsub-related metrics event types
  • Updates help output snapshots

  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: removal of unused code
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: feature was never publicly documented

Open with Devin

@elithrar elithrar requested a review from a team as a code owner January 28, 2026 16:37
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 28, 2026

🦋 Changeset detected

Latest commit: ba30187

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 28, 2026

Claude finished @elithrar's task —— View job


Changeset Review

Reviewing changeset file...

Todo:

  • Read changeset guidelines
  • Read changeset file
  • Validate changeset
  • Post results

All changesets look good

The changeset .changeset/remove-pubsub.md is valid:

  • Version type: patch is appropriate for removing unused/deprecated functionality
  • Description: Clear and well-formatted with a title and explanatory body
  • Markdown headers: None present (✓)
  • Analytics: Not applicable for this removal
  • Package coverage: Correctly lists wrangler as the affected package

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

View issue and 3 additional flags in Devin Review.

Open in Devin Review

Comment thread packages/wrangler/src/index.ts
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 28, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12211

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12211

miniflare

npm i https://pkg.pr.new/miniflare@12211

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12211

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12211

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12211

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12211

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12211

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@12211

wrangler

npm i https://pkg.pr.new/wrangler@12211

commit: ba30187

@elithrar elithrar enabled auto-merge January 28, 2026 16:44
/**
* Registers a category for a legacy command that doesn't use the CommandRegistry.
* This is used for commands like `containers`, `pubsub`, etc, that use the old yargs pattern.
* This is used for commands like `containers`, etc, that use the old yargs pattern.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO for @petebacondarwin - remove this in a follow up as the containers commands no longer use it either.

@elithrar elithrar added this pull request to the merge queue Jan 29, 2026
@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jan 29, 2026
@penalosa penalosa removed this pull request from the merge queue due to a manual request Jan 29, 2026
@petebacondarwin petebacondarwin added this pull request to the merge queue Jan 29, 2026
Merged via the queue into main with commit a5fca2c Jan 29, 2026
37 checks passed
@petebacondarwin petebacondarwin deleted the remove-pubsub branch January 29, 2026 12:41
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants