Skip to content

refactor(handlers): centralize conversation helpers - #21

Merged
scottishwidow merged 1 commit into
mainfrom
refactor/handler-conversation-helpers
Apr 24, 2026
Merged

refactor(handlers): centralize conversation helpers#21
scottishwidow merged 1 commit into
mainfrom
refactor/handler-conversation-helpers

Conversation

@scottishwidow

Copy link
Copy Markdown
Owner

Summary

  • Add handlers.conversation for shared user state access, song ID/callback parsing, cancel filters/fallbacks, and home/remove reply markup helpers.
  • Refactor repertoire, chart upload, backup import, and navigation handlers to use the shared helper module without changing callback payloads or user-facing copy.
  • Add focused helper coverage and update the button-navigation feature note.

Verification

  • uv run pytest tests/test_handlers.py tests/test_navigation.py tests/test_handler_conversation_helpers.py (59 passed, 7 existing PTB warnings)
  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy
  • pre-commit during git commit passed ruff, ruff-format, large-file, EOF, whitespace, and mypy hooks

Note: pre-existing untracked docs/plans/ work was left untouched.

@scottishwidow
scottishwidow merged commit ec65871 into main Apr 24, 2026
4 checks passed
@scottishwidow
scottishwidow deleted the refactor/handler-conversation-helpers branch April 24, 2026 18:47
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