Skip to content

Conversation

@pombosilva
Copy link
Contributor

@pombosilva pombosilva commented Dec 10, 2025

Fixes WOR-1057.

This adds a new command to wrangler wrangler workflows instances send-event <name> <id> with options --type and --payload to allow users to just call this command to send an event to a deployed running workflow instance rather than having to implement it themselves or having to use the API.


  • Tests
    • Tests included/updated
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: it will be automatic
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: Workflows were not GA in v3

A picture of a cute animal (not mandatory, but encouraged)
Okay, if you really want... here is the cutest dog (@sidharthachatterjee) Now try not to fall in love
image

@pombosilva pombosilva requested a review from a team as a code owner December 10, 2025 15:35
@changeset-bot
Copy link

changeset-bot bot commented Dec 10, 2025

🦋 Changeset detected

Latest commit: d18edef

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

@github-actions
Copy link
Contributor

Failed to automatically backport this PR's changes to Wrangler v3. Please manually create a PR targeting the v3-maintenance branch with your changes. Thank you for helping us keep Wrangler v3 supported!

Depending on your changes, running git rebase --onto v3-maintenance main osilva/add-send-event-wrangler-command might be a good starting point.

Notes:

  • your PR branch should be named v3-backport-11590
  • add the skip-v3-pr label to the current PR to stop this workflow from failing

@pombosilva pombosilva force-pushed the osilva/add-send-event-wrangler-command branch 2 times, most recently from ba59a07 to 4dec5aa Compare December 10, 2025 16:05
@Caio-Nogueira
Copy link
Contributor

LGTM from the workflows team

Copy link
Contributor

@vicb vicb left a comment

Choose a reason for hiding this comment

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

Could you please get an approval from within your team first ?
(That's the usual process from contributions originating from outside the ANT team).
Thanks

@pombosilva
Copy link
Contributor Author

Could you please get an approval from within your team first ? (That's the usual process from contributions originating from outside the ANT team). Thanks

I have an approval from my team. Caio left a comment above

@pombosilva pombosilva requested a review from vicb December 11, 2025 11:43
@pombosilva pombosilva force-pushed the osilva/add-send-event-wrangler-command branch from 0954bb1 to c2890de Compare December 12, 2025 12:23
@pombosilva pombosilva force-pushed the osilva/add-send-event-wrangler-command branch from c2890de to 0dea233 Compare December 15, 2025 11:29
Copy link
Contributor

@vicb vicb left a comment

Choose a reason for hiding this comment

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

Thanks!

And sorry for the localeCompare thingy, now I understand that the reason for having this is that there is no non-locale aware variant of it

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Dec 15, 2025
@pombosilva pombosilva force-pushed the osilva/add-send-event-wrangler-command branch from 0dea233 to d18edef Compare December 15, 2025 14:07
@pombosilva pombosilva merged commit 7d8d4a6 into main Dec 15, 2025
34 checks passed
@pombosilva pombosilva deleted the osilva/add-send-event-wrangler-command branch December 15, 2025 14:56
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Dec 15, 2025
AmirSa12 pushed a commit to AmirSa12/workers-sdk that referenced this pull request Jan 5, 2026
* [Wrangler] Add Workflows sendEvent wrangler command

* Apply comment suggestions

* Apply review suggestions

* Refactor some repeated code and remove localeCompare

* Revert removal of localeCompare
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.

4 participants