From 5d740a6948426f11ed9e4020abab5df2f15a00cf Mon Sep 17 00:00:00 2001 From: claude Date: Wed, 27 May 2026 19:16:28 -0400 Subject: [PATCH] style(backend): drop stale "moved/removed" pointer comments MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- claude_bottle/backend/docker/cleanup.py | 5 ----- tests/unit/test_dashboard_active_agents.py | 9 +-------- tests/unit/test_docker_enumerate_active.py | 7 ------- 3 files changed, 1 insertion(+), 20 deletions(-) diff --git a/claude_bottle/backend/docker/cleanup.py b/claude_bottle/backend/docker/cleanup.py index 0487e11..0af9eed 100644 --- a/claude_bottle/backend/docker/cleanup.py +++ b/claude_bottle/backend/docker/cleanup.py @@ -159,8 +159,3 @@ def cleanup(plan: DockerBottleCleanupPlan) -> None: shutil.rmtree(path, ignore_errors=True) except OSError as 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. diff --git a/tests/unit/test_dashboard_active_agents.py b/tests/unit/test_dashboard_active_agents.py index 3bf2b3e..55299d7 100644 --- a/tests/unit/test_dashboard_active_agents.py +++ b/tests/unit/test_dashboard_active_agents.py @@ -1,11 +1,4 @@ -"""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. -""" +"""Unit: dashboard's row-formatting + selection helpers (PRD 0019).""" from __future__ import annotations diff --git a/tests/unit/test_docker_enumerate_active.py b/tests/unit/test_docker_enumerate_active.py index 6ba1c9f..48d97b4 100644 --- a/tests/unit/test_docker_enumerate_active.py +++ b/tests/unit/test_docker_enumerate_active.py @@ -173,13 +173,6 @@ class TestEnumerateActive(_FakeHomeMixin, unittest.TestCase): [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__": unittest.main()