Skip to content

Features/require review#103

Merged
aliroberts merged 10 commits intodevfrom
features/require-review
Jan 22, 2026
Merged

Features/require review#103
aliroberts merged 10 commits intodevfrom
features/require-review

Conversation

@aliroberts
Copy link
Contributor

PR Title

Switch weco-cli to new execution task API + UX upgrades

Summary

This PR migrates weco-cli to the new execution task API and removes the legacy flow, while improving the interactive UX and run lifecycle handling.

Key changes

  • API migration
    • Switch to the new execution task API
    • Use updated task API responses that include run data
    • Remove the legacy flow
  • CLI / UX improvements
    • Add rich UI for a better interactive experience
    • Ensure the best metric value is shown in the resume display
    • Add automatic browser opening where appropriate
    • Add filename validation and improve interrupt handling (cleaner shutdown)
    • Stop the heartbeat when a run completes
  • New commands / flags
    • Add login command
    • Add run --require-review option

Why

  • Align weco-cli with the new task execution model and data shape.
  • Improve reliability and polish around run lifecycle, shutdown, and display correctness.

Notes / follow-ups

  • If the API migration changes expected output or flags, consider a short docs update or changelog entry.

@aliroberts aliroberts force-pushed the features/require-review branch from d6a0cd1 to 48740bb Compare January 22, 2026 16:01
@aliroberts aliroberts merged commit 3d65557 into dev Jan 22, 2026
1 check passed
@aliroberts aliroberts deleted the features/require-review branch January 22, 2026 16:09
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.

1 participant