fix: cancelled threads show distinct "cancelled" status #525

Merged
xiaomo merged 1 commits from fix/522-cancelled-thread-status into main 2026-05-25 15:51:29 +00:00
Owner

What

Cancelled threads now show status: "cancelled" instead of "completed".

Why

uwf thread cancel moved threads to history but they appeared as completed. No way to distinguish cancelled from completed threads. Fixes #522.

Changes

  • store.tsThreadHistoryLine gains reason field
  • commands/thread.tsThreadStatus adds "cancelled", cmdThreadCancel writes reason: "cancelled", collectCompletedThreads reads it
  • cli.ts"cancelled" added to valid status filter values
  • New test file: thread-cancel-status.test.ts with 4 tests
  • Updated existing tests for new reason field

Ref

Fixes #522

## What Cancelled threads now show `status: "cancelled"` instead of `"completed"`. ## Why `uwf thread cancel` moved threads to history but they appeared as completed. No way to distinguish cancelled from completed threads. Fixes #522. ## Changes - `store.ts` — `ThreadHistoryLine` gains `reason` field - `commands/thread.ts` — `ThreadStatus` adds `"cancelled"`, `cmdThreadCancel` writes `reason: "cancelled"`, `collectCompletedThreads` reads it - `cli.ts` — `"cancelled"` added to valid status filter values - New test file: `thread-cancel-status.test.ts` with 4 tests - Updated existing tests for new `reason` field ## Ref Fixes #522
xiaonuo added 1 commit 2026-05-25 15:40:50 +00:00
xiaomo approved these changes 2026-05-25 15:51:19 +00:00
xiaomo left a comment
Owner

xiaomo merged commit 0097633a3b into main 2026-05-25 15:51:29 +00:00
This repo is archived. You cannot comment on pull requests.
No Reviewers
No Label
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/workflow#525