Skip to content

feat: Cancel workflow by id via the client#310

Merged
jamescmartinez merged 2 commits intoopenworkflowdev:mainfrom
octoper:vaggelis/cancel-workflow-by-id
Feb 15, 2026
Merged

feat: Cancel workflow by id via the client#310
jamescmartinez merged 2 commits intoopenworkflowdev:mainfrom
octoper:vaggelis/cancel-workflow-by-id

Conversation

@octoper
Copy link
Contributor

@octoper octoper commented Feb 15, 2026

Summary

This pull request adds support for canceling workflow runs by their ID in the OpenWorkflow client directly.

Changes

  • Added a cancelWorkflowRun method to the OpenWorkflow class, allowing users to cancel workflow runs by ID.

Copilot AI review requested due to automatic review settings February 15, 2026 00:14
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a convenience method to the OpenWorkflow client that allows users to cancel workflow runs directly by their ID, without needing to have a WorkflowRunHandle instance. This is useful when users have stored a workflow run ID and want to cancel it later without going through the handle.

Changes:

  • Added cancelWorkflowRun(workflowRunId: string) method to the OpenWorkflow class
  • Added tests for the new cancellation method including error handling for non-existent workflow IDs

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/openworkflow/client.ts Added cancelWorkflowRun method to OpenWorkflow class that delegates to the backend's existing cancel functionality
packages/openworkflow/client.test.ts Added two tests: one for successful cancellation via client by ID, and one for error handling when canceling non-existent workflows

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 15, 2026 00:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 15, 2026 05:59
@octoper octoper force-pushed the vaggelis/cancel-workflow-by-id branch from e0282da to 9b48fbc Compare February 15, 2026 05:59
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@octoper octoper force-pushed the vaggelis/cancel-workflow-by-id branch from 504d87a to 5489bf1 Compare February 15, 2026 06:02
Copilot AI review requested due to automatic review settings February 15, 2026 06:03
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@octoper octoper changed the title feat: Cancel workflow via the client with workflow id feat: Cancel workflow by id via the client Feb 15, 2026
@codecov
Copy link

codecov bot commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jamescmartinez
Copy link
Contributor

thanks!

@jamescmartinez jamescmartinez merged commit 8eecadb into openworkflowdev:main Feb 15, 2026
3 checks passed
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

Comments