Skip to content

ensure cancel is idempotent#354

Merged
jackyzha0 merged 1 commit intomainfrom
jackyzha0/cancel-idempotency
Feb 11, 2026
Merged

ensure cancel is idempotent#354
jackyzha0 merged 1 commit intomainfrom
jackyzha0/cancel-idempotency

Conversation

@jackyzha0
Copy link
Member

Why

we weren't sure calling ctx.cancel() twice was safe

What changed

add a test to assert and not regress on this behavior

Versioning

  • Breaking protocol change
  • Breaking ts/js API change

@jackyzha0 jackyzha0 requested a review from a team as a code owner February 11, 2026 19:33
@jackyzha0 jackyzha0 requested review from daweifeng-replit and removed request for a team February 11, 2026 19:33
@jackyzha0 jackyzha0 merged commit 5684334 into main Feb 11, 2026
8 checks passed
@jackyzha0 jackyzha0 deleted the jackyzha0/cancel-idempotency branch February 11, 2026 19:36
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