fix(tests): pyright strict — export _plan fixture, drop unused imports

- `tests/unit/_docker_bottle_plan.py`: add `__all__ = ["_plan"]` so the
  underscore-prefixed fixture isn't flagged reportUnusedFunction.
- `tests/unit/test_egress_apply.py`: drop `SimpleNamespace` / `patch`
  imports left unused after the reload test became fail-closed.

`pyright .` → 0 errors.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WBMWTEtQdJ4W5UrWuLHCck

tests/unit/_docker_bottle_plan.py

@@ -23,6 +23,10 @@ SLUG = "demo-abc12"
 STAGE = Path("/tmp/cb-stage")
 STATE = Path("/tmp/cb-state")
 
+# Exported to consumers (e.g. test_consolidated_compose); named with a
+# leading underscore for the historical fixture convention.
+__all__ = ["_plan"]
+
 
 def _manifest(*, supervise: bool, with_git: bool, with_egress: bool) -> ManifestIndex:
     """Minimal manifest with the toggles the matrix needs. The renderer

tests/unit/test_egress_apply.py

@@ -5,8 +5,6 @@ integration test)."""
 import tempfile
 import unittest
 from pathlib import Path
-from types import SimpleNamespace
-from unittest.mock import patch
 
 from tests.unit import use_bottle_root
 from bot_bottle.backend.egress_apply import EgressApplyError