From 9cc9de8cdc9d3e5de419ed0f16521941aff2dc58 Mon Sep 17 00:00:00 2001 From: Ahmed Allam Date: Fri, 4 Sep 2026 21:43:41 +0000 Subject: [PATCH] fix(warmup): drop docker from WARMUP_MODULES The Docker checks import the Docker SDK on the main thread before the warm-up join, so warming it saves nothing and leaves one module shared between the two threads during the startup window. --- strix/llm/warmup.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/strix/llm/warmup.py b/strix/llm/warmup.py index e4c7357b..c46d0b1a 100644 --- a/strix/llm/warmup.py +++ b/strix/llm/warmup.py @@ -1,10 +1,11 @@ """Background pre-import of the heavy scan dependencies. The scan engine's import graph (the agents SDK, OpenAI client, LiteLLM, the -Caido SDK, the Docker SDK) costs seconds to import cold, but none of it is -needed until a scan actually starts. Importing it on a daemon thread at CLI -entry overlaps that cost with the I/O-bound startup work that always precedes -a scan (argument parsing, Docker checks, image pull, TUI setup). +Caido SDK) costs seconds to import cold, but none of it is needed until a scan +actually starts. Importing it on a daemon thread at CLI entry overlaps that +cost with the I/O-bound startup work that always precedes a scan (argument +parsing, Docker checks, image pull, TUI setup). The Docker SDK is not on the +list: the Docker checks import it on the main thread during that same window. The main thread must call :func:`wait_for_import_warmup` before its first import from that graph. Two threads that enter the same package graph from @@ -25,7 +26,6 @@ WARMUP_MODULES = ( "strix.core.runner", "litellm", "caido_sdk_client", - "docker", ) _thread: threading.Thread | None = None