[loop-generated] [bug] 256 test errors from missing swarm module — need stub or optional import guard #657
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
tox -e unitproduces 256 ERRORs becauseswarmis not installed. The swarm package is imported in:src/timmy/thinking.py(lines 816, 1222)src/timmy/briefing.py(line 170)src/dashboard/app.py(line 439)src/infrastructure/error_capture.py(lines 133, 183, 210)src/spark/memory.py(line 168)All imports are already inside try/except or conditional blocks, but something is still causing 256 test collection errors.
Root cause
Likely a top-level import or conftest fixture that tries to import swarm unconditionally. The
conftest.pyreferencesswarm.dbmodules but wraps in try/except — need to trace the actual failure path.Fix
Either:
swarmstub package to the test dependenciesswarmto optional dependencies in pyproject.tomlScope
1-3 files. The 256 errors should collapse to 0 with a proper fix.
Verification
Errors should drop from 256 to 0.
@kimi — Investigate why
tox -e unitproduces 256 errors fromModuleNotFoundError: No module named swarm.Steps:
tox -e unit 2>&1 | grep ModuleNotFoundError | head -5to confirma. Better import guards (try/except at the point of failure)
b. A stub
swarmpackage in tests/c. Adding swarm as optional dep
Files likely involved:
tests/conftest.pysrc/dashboard/app.pysrc/infrastructure/error_capture.pyVerify:
Errors should drop from 256 to 0 (1889 passed should stay passed).
Kimi Task — Fix #657: Missing swarm module causes 256 test errors
The
swarmpackage is not installed, causing 256 test collection errors intox -e unit.What to do:
tox -e unitand capture the actual error traceback to find the real import path causing failuressrc/timmy/thinking.py,src/timmy/briefing.py, etc. are already guarded — find where the guard is leakingswarmstub to make tests collectibletox -e unit— errors should drop from 256 to 0Files to check:
src/timmy/thinking.py(lines 816, 1222)src/timmy/briefing.py(line 170)src/dashboard/app.py(line 439)tests/conftest.py— likely the real culpritsrc/infrastructure/error_capture.pyBranch:
fix/swarm-import-guardInvestigated this issue on branch
kimi/issue-657. Runningtox -e unitproduces 2205 passed, 1 skipped, 0 errors — the 256 test collection errors are no longer present.The
swarmmodule is already properly stubbed intests/conftest.py(lines 34-35 viasys.modules.setdefault()), and all swarm DB paths are redirected to temp directories. Lint also passes clean.This issue appears to have been resolved by a prior commit. No code changes needed — closing as already fixed.