Skip to content

Resolve #25 -- Add simple task/queue actions to the inspector - #27

Merged
codingjoe merged 1 commit into
mainfrom
actions
Jul 7, 2026
Merged

Resolve #25 -- Add simple task/queue actions to the inspector#27
codingjoe merged 1 commit into
mainfrom
actions

Conversation

@codingjoe

Copy link
Copy Markdown
Owner

No description provided.

@codingjoecodingjoe self-assigned this Jul 7, 2026
@codingjoe
codingjoeforce-pushed the actions branch 4 times, most recently from 5008ce4 to 4133851CompareJuly 7, 2026 18:26
@codingjoe
codingjoe marked this pull request as ready for review July 7, 2026 18:39
@codingjoe
codingjoe requested a review from CopilotJuly 7, 2026 18:39

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds interactive “mutating” queue/task actions to the Textual-based inspector (confirm/retype modals + keyboard bindings), backed by new backend APIs and Redis implementations, with accompanying tests and README updates.

Changes:

  • Add confirmation modal screens (ConfirmScreen, PurgeScreen) and dialog styling for inspector actions.
  • Add inspector keybindings/actions to requeue failed tasks, dequeue tasks, and purge queues (with confirmation flows).
  • Extend backend API with dequeue() / purge() and implement them (plus extra Redis cleanup in requeue()), with expanded test coverage and docs.

Reviewed changes

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

Show a summary per file
FileDescription
threadmill/inspector/screens.pyAdds modal confirmation screens for destructive/confirmable inspector actions.
threadmill/inspector/inspector.scssAdds dialog styling and global screen alignment (needs scoping adjustment).
threadmill/inspector/app.pyAdds keybindings and action handlers for refresh/requeue/dequeue/purge in the inspector.
threadmill/backends/redis.pyAdds Redis implementations for dequeue/purge and improves requeue cleanup.
threadmill/backends/base.pyExtends backend interface with dequeue/purge APIs and expands requeue docstring.
tests/test_inspector.pyAdds extensive inspector action and modal interaction tests.
tests/backends/test_redis.pyAdds Redis backend tests for requeue cleanup, dequeue, and purge behavior.
tests/backends/test_base.pyAdds base backend NotImplementedError tests for new APIs.
README.mdDocuments inspector queue actions and shortcuts (currently mismatched for dequeue).

Comment threadREADME.md Outdated
Comment threadthreadmill/inspector/inspector.scss
Comment threadthreadmill/inspector/app.py Outdated
Comment threadthreadmill/backends/redis.py
Comment threadthreadmill/backends/base.py Outdated
@codingjoe
codingjoe merged commit 190acbe into mainJul 7, 2026
4 checks passed
@codingjoe
codingjoe deleted the actions branch July 7, 2026 21:51
Sign up for freeto 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

@codingjoe