code_executors: defer docker and magic imports to avoid hang on Windows - #231

Merged

raychen911 merged 15 commits into

trpc-group:mainfrom

Fromsko:fix/lazy-import-docker

Aug 10, 2026

Commits on Jul 24, 2026

code_executors: defer docker and magic imports to avoid hang on Windows without Docker/libmagic
Fromsko
committed
bb27456
tests: skip magic test on Windows; keep HAS_MAGIC for backward compat
Fromsko
committed
33aedbf
code_executors: fix HAS_MAGIC dead code and test mock (address review)
Fromsko
committed
7f60d93
address review: TYPE_CHECKING guard, import short-circuit, yapf format
Fromsko
committed
fffa3a5
fix: flake8 F821 + missing logger import for CI lint
Fromsko
committed
7f2e29a
tests: fix container_cli mock compatibility with lazy docker import
Fromsko
committed
f2ff424
address final review: restore HAS_MAGIC, add threading lock, cleanup
Fromsko
committed
2314680
refactor: use explicit module-level placeholders instead of globals().update
Fromsko
committed
2e13abb
fix: widen magic import exception handling and restore HAS_MAGIC in tests
Fromsko
committed
1e6e190
fix: guard docker=None before except block to prevent AttributeError
Fromsko
committed
0949e38
fix: make docker=None guard reachable and restore HAS_MAGIC import-time semantics
Fromsko
committed
ba298ca
fix: move _import_docker to fixture, add magic thread lock
Fromsko
committed
d72f875
refactor: use TYPE_CHECKING + private _docker_* names, no public shadowing
Fromsko
committed
f493e18
fix: fully lazy magic import, remove dead code, widen docker except
Fromsko
committed
8e676d2
polish: add logger.debug for docker import failure, fix import ordering
Fromsko
committed
85b8def