style(backend): drop stale "moved/removed" pointer comments
test / unit (pull_request) Successful in 25s
test / integration (pull_request) Successful in 41s
test / unit (push) Successful in 27s
test / integration (push) Successful in 41s

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:
2026-05-27 19:16:28 -04:00
parent 3b418580a9
commit 5d740a6948
3 changed files with 1 additions and 20 deletions
-5
View File
@@ -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 -8
View File
@@ -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()