style(backend): drop stale "moved/removed" pointer comments
PR #78 review comments 580, 582, 584. Each was a comment describing what the previous refactor removed or relocated — information that's in git history, not load-bearing for a reader of the file as-is. - claude_bottle/backend/docker/cleanup.py: drop trailing "enumerate_active moved to enumerate.py" note. - tests/unit/test_dashboard_active_agents.py: drop module docstring paragraph about which tests moved where. - tests/unit/test_docker_enumerate_active.py: drop "noop-when-docker-missing lives at the cross-backend gate now" trailing comment. 607 tests still pass. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit was merged in pull request #78.
This commit is contained in:
@@ -159,8 +159,3 @@ def cleanup(plan: DockerBottleCleanupPlan) -> None:
|
|||||||
shutil.rmtree(path, ignore_errors=True)
|
shutil.rmtree(path, ignore_errors=True)
|
||||||
except OSError as e:
|
except OSError as e:
|
||||||
warn(f"failed to remove {path}: {e}")
|
warn(f"failed to remove {path}: {e}")
|
||||||
|
|
||||||
|
|
||||||
# `enumerate_active` moved to `backend/docker/enumerate.py` to
|
|
||||||
# mirror the smolmachines layout. Cleanup keeps the orphan
|
|
||||||
# enumeration; enumeration of live agents is its own concern.
|
|
||||||
|
|||||||
@@ -1,11 +1,4 @@
|
|||||||
"""Unit: dashboard's row-formatting + selection helpers (PRD 0019).
|
"""Unit: dashboard's row-formatting + selection helpers (PRD 0019)."""
|
||||||
|
|
||||||
The active-bottle enumeration tests moved to
|
|
||||||
`test_docker_enumerate_active.py` (issue #77) — dashboard now
|
|
||||||
delegates listing to `enumerate_active_agents` so the parser
|
|
||||||
and assembly tests live next to the docker backend's
|
|
||||||
implementation.
|
|
||||||
"""
|
|
||||||
|
|
||||||
from __future__ import annotations
|
from __future__ import annotations
|
||||||
|
|
||||||
|
|||||||
@@ -173,13 +173,6 @@ class TestEnumerateActive(_FakeHomeMixin, unittest.TestCase):
|
|||||||
[a.slug for a in active],
|
[a.slug for a in active],
|
||||||
)
|
)
|
||||||
|
|
||||||
# `noop when docker missing` lives at the cross-backend gate
|
|
||||||
# now (`enumerate_active_agents` skips backends whose
|
|
||||||
# `is_available()` reports False — see
|
|
||||||
# `test_backend_selection.TestEnumerateActiveAgents`). This
|
|
||||||
# module assumes docker is available when called, matching the
|
|
||||||
# smolmachines/enumerate.py contract.
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
unittest.main()
|
unittest.main()
|
||||||
|
|||||||
Reference in New Issue
Block a user