Skip to content

Close Every Code threads during bridge shutdown#93

Merged
shiny-code-bot merged 2 commits into
mainfrom
fix/every-code-shutdown-thread-cleanup
Jun 16, 2026
Merged

Close Every Code threads during bridge shutdown#93
shiny-code-bot merged 2 commits into
mainfrom
fix/every-code-shutdown-thread-cleanup

Conversation

@cbusillo

Copy link
Copy Markdown
Owner

Summary

  • close active Every Code Discord threads during bridge shutdown instead of only closing websockets
  • protect shutdown cleanup with the existing session attach lock so partially-attached sessions cannot leave orphaned threads
  • add regression coverage for bound sessions, already-closed websockets, and attach-time shutdown races

Validation

  • uv run python -m unittest discover -s tests -q
  • uv run ruff check --diff discord_blue/doodads/every_code/bridge.py tests/test_every_code.py tests/fakes_every_code.py
  • uv run ruff check discord_blue/doodads/every_code/bridge.py tests/test_every_code.py tests/fakes_every_code.py
  • uv run mypy .

@shiny-code-bot shiny-code-bot merged commit eadfedb into main Jun 16, 2026
6 checks passed
@shiny-code-bot shiny-code-bot deleted the fix/every-code-shutdown-thread-cleanup branch June 16, 2026 00:54
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.

2 participants