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

Open

Fromsko wants to merge 15 commits into

trpc-group:mainfrom

Fromsko:fix/lazy-import-docker

Commits on Jul 24, 2026

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