Skip to content

chore: follow up workflow fixes with docs, comments, and regression tests#139

Merged
adrunkhuman merged 4 commits intomasterfrom
docs-tests-followup
Mar 22, 2026
Merged

chore: follow up workflow fixes with docs, comments, and regression tests#139
adrunkhuman merged 4 commits intomasterfrom
docs-tests-followup

Conversation

@adrunkhuman
Copy link
Owner

@adrunkhuman adrunkhuman commented Mar 22, 2026

Summary

Why

  • The recent sequence of changes fixed real race conditions and untangled a lot of stateful behavior, but it also left some contracts implicit and a few important edge cases under-tested
  • This follow-up makes the current behavior easier to understand and harder to regress without trying to change product scope again

Notes

Verification

  • uv run ruff check scripts/restore_db.py tests/test_admin_commands.py tests/test_bot.py tests/test_config.py tests/test_database.py tests/test_results_handler.py tests/test_restore_db.py tests/test_thread_prediction_handler.py tests/test_timezone.py typer_bot/bot.py typer_bot/commands/admin_panel/base.py typer_bot/commands/admin_panel/modals.py typer_bot/database/connection.py typer_bot/handlers/dm_prediction_handler.py typer_bot/handlers/thread_prediction_handler.py typer_bot/services/workflow_state.py typer_bot/utils/config.py typer_bot/utils/prediction_parser.py
  • uv run pytest

Remove comment narration that was restating code so the touched files stay easier to scan. Keep the comments that still carry workflow or safety context.
Document the real deployment defaults, restore flow, and message-routing behavior so operators and contributors stop guessing from stale assumptions. Add docstrings where the recent workflow and database changes made public contracts harder to infer from code alone.
Add coverage for the concurrency, routing, restore, timezone, and admin command paths that changed across the recent fixes and refactors. This locks in the behavior the docs now describe and reduces the risk of regressions in the stateful workflows.
@railway-app railway-app bot temporarily deployed to patient-quietude / matchday-typer-pr-139 March 22, 2026 00:27 Destroyed
@railway-app
Copy link

railway-app bot commented Mar 22, 2026

🚅 Deployed to the matchday-typer-pr-139 environment in patient-quietude

Service Status Web Updated (UTC)
matchday-typer 🕒 Building (View Logs) Mar 22, 2026 at 12:31 am

@adrunkhuman adrunkhuman changed the title docs: follow up workflow fixes with docs and regression coverage chore: follow up workflow fixes with docs, comments, and regression tests Mar 22, 2026
Update the repo-local agent instructions so they match the current data path defaults, timezone default, split database layout, backup timing, and restore safety behavior. This keeps future follow-up work from being driven by stale project context.
@railway-app railway-app bot temporarily deployed to patient-quietude / matchday-typer-pr-139 March 22, 2026 00:30 Destroyed
@adrunkhuman adrunkhuman merged commit 8df65ab into master Mar 22, 2026
1 of 2 checks passed
@adrunkhuman adrunkhuman deleted the docs-tests-followup branch March 22, 2026 00:31
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