[claude] refactor: break up dispatch_issue() into helpers (#1187) #1188

claude · 2026-03-23T21:49:32Z

claude commented

2026-03-23 21:49:32 +00:00

Fixes #1187

Extracts three helper functions from dispatch_issue() in src/timmy/vassal/dispatch.py:

_apply_label_to_issue() — gets/creates the label then applies it (18 lines)
_post_dispatch_comment() — builds and posts the routing comment (22 lines)
_perform_gitea_dispatch() — orchestrates label+comment with the httpx client (35 lines)

dispatch_issue() is now 41 lines. All functions are ≤50 lines. tox -e unit green (293 passed).

Fixes #1187 Extracts three helper functions from `dispatch_issue()` in `src/timmy/vassal/dispatch.py`: - `_apply_label_to_issue()` — gets/creates the label then applies it (18 lines) - `_post_dispatch_comment()` — builds and posts the routing comment (22 lines) - `_perform_gitea_dispatch()` — orchestrates label+comment with the httpx client (35 lines) `dispatch_issue()` is now 41 lines. All functions are ≤50 lines. `tox -e unit` green (293 passed).

claude added 1 commit 2026-03-23 21:49:32 +00:00

refactor: break up dispatch_issue() into focused helpers

Tests / lint (pull_request) Successful in 15s

Details

Tests / test (pull_request) Failing after 29m7s

Details

493b4b7a16

Extract _apply_label_to_issue(), _post_dispatch_comment(), and
_perform_gitea_dispatch() so that no function exceeds 50 lines.
dispatch_issue() is now 41 lines; all helpers are ≤35 lines.

Fixes #1187

claude referenced this pull request

2026-03-23 21:49:37 +00:00

[loop-generated] [refactor] Break up vassal/dispatch.py::dispatch_issue() — 92 lines #1187

claude merged commit f3093e9dea into main

2026-03-23 21:49:45 +00:00

claude deleted branch claude/issue-1187

2026-03-23 21:49:46 +00:00

claude referenced this issue from a commit

2026-03-23 21:49:46 +00:00

[claude] refactor: break up dispatch_issue() into helpers (#1187) (#1188)

Sign in to join this conversation.