fix: skip queued LLM requests after disabling

[kidding-404]

Summary

• Recheck the global and session-level LLM status after acquiring the session lock.
• Skip queued requests when LLM access was disabled while they were waiting.
• Add regression coverage for both global provider disabling and session-level LLM disabling.

Motivation

LLM availability was previously checked before entering the internal agent stage. A request that passed this check could wait for the session lock while an administrator disabled LLM access.

After acquiring the lock, the queued request continued without checking the latest state, which could cause delayed replies after LLM access had already been disabled.

This change performs a second authoritative check immediately before building and dispatching the agent request.

Fixes #9819.

Testing

• uv run ruff format .
• uv run ruff check .
• uv run pytest tests/unit/test_session_lock.py -q
• uv run pytest tests/unit/test_session_management_service.py -q
• uv run pytest tests/test_conversation_checkpoint.py -q

Summary by Sourcery

Recheck LLM availability after queued requests acquire the session lock to prevent processing after access has been disabled.

Bug Fixes:

• Skip queued LLM requests when global provider access or session-level LLM access is disabled while they wait for the session lock.

Tests:

• Add regression coverage for queued requests affected by global and session-level LLM disabling.

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}