Files
cleveragents-core/features
hamza.khyari cf6e33f2bb
CI / push-validation (pull_request) Successful in 33s
CI / helm (pull_request) Successful in 36s
CI / build (pull_request) Successful in 1m5s
CI / lint (pull_request) Successful in 1m17s
CI / quality (pull_request) Successful in 1m39s
CI / security (pull_request) Successful in 1m46s
CI / typecheck (pull_request) Successful in 1m55s
CI / integration_tests (pull_request) Successful in 3m31s
CI / unit_tests (pull_request) Successful in 5m35s
CI / docker (pull_request) Successful in 1m53s
CI / coverage (pull_request) Successful in 11m5s
CI / status-check (pull_request) Successful in 4s
fix(tui): address all CoreRasurae review findings — safety, markup, logging
Fix 1 (HIGH): exclusive=False -> exclusive=True, name="llm-dispatch"
  Serialises concurrent dispatches on the same session, preventing
  interleaved append_message() calls, widget thrashing, and DB contention.

Fix 2 (MEDIUM): Rich markup injection — apply rich.markup.escape()
  to all conversation.update() calls so user/LLM text renders literally.

Fix 3 (MEDIUM): Narrow contextlib.suppress(Exception) in _build_tui_facade
  and _create_tui_session to specific exception types so KeyboardInterrupt,
  SystemExit, MemoryError etc. propagate correctly.

Fix 4 (MEDIUM): Add structlog logging to all _run_llm_dispatch except
  branches for operational observability.

Fix 5 (MEDIUM): Empty assistant_message falls back to "(no response)"
  instead of rendering blank "Assistant: " text.

Fix 6 (LOW): _format_worker_outcome re-raises BaseException subclasses
  (KeyboardInterrupt, SystemExit) instead of stringifying them.

Fix 7 (LOW): Extract _FALLBACK_SESSION_ID and _PRE_SESSION_PERSONA_KEY
  constants — "default" no longer serves two distinct purposes.

Fix 8 (LOW): Add 3 new BDD scenarios: empty response fallback, Rich
  markup escaping verification, KeyboardInterrupt re-raise (15 total).

Refs: #11230
2026-05-19 10:40:57 +00:00
..