informatics-isi-edu / deriva-mcp-ui / 24903691322

90%

Build Type

push

github

Committed by mikedarcy

Commit Message

Fix session expiry UX, audit event cleanup, and MCP 401 handling

  Session expiry detection: expose session_id in /session-info; client
  echoes it on every chat POST; server returns 401 session_expired when
  the ID mismatches (stale session silently replaced by anonymous one).
  visibilitychange listener detects expiry when user returns to a
  backgrounded tab. Authenticated users now get a "session expired / log
  in / refresh" banner instead of showLoggedOutState() on any 401; input
  area is never hidden.

  MCP 401 via ExceptionGroup: streamablehttp_client uses asyncio.TaskGroup
  so HTTP 401s surface as BaseExceptionGroup wrapping HTTPStatusError
  rather than bare MCPAuthError. Fix in both _connect() (mcp_client.py)
  and _event_stream (server.py) to unwrap and classify correctly.

  Audit event cleanup: drop chat_request and chat_complete (redundant with
  chat_turn); rename llm_usage -> llm_api_call; add session_id and turn
  to llm_api_call for correlation; add prompt/completion/cache token
  deltas to chat_turn; add session_id and turn to chat_cancelled.

  Access logging: remove rotating file handler; fall back to stderr (Docker
  / AWS log drivers) when syslog is unavailable; remove access_logfile_path
  config var.

Coverage Stats

9 of 28 new or added lines in 2 files covered. (32.14%)

1 existing line in 1 file now uncovered.

1487 of 1654 relevant lines covered (89.9%)

0.9 hits per line