From 924b122e2b86c33797fc4b85059772adc742c1fc Mon Sep 17 00:00:00 2001 From: claude Date: Fri, 10 Jul 2026 20:36:19 +0000 Subject: [PATCH] fix: remove redundant quoted annotations in BottleImages and abstract methods `from __future__ import annotations` already defers all annotation evaluation, so quoting `str | Path`, `BottleImages` inside the same module was redundant and tripped pyright strict mode. --- bot_bottle/backend/__init__.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/bot_bottle/backend/__init__.py b/bot_bottle/backend/__init__.py index 57679f0..8cb5f6f 100644 --- a/bot_bottle/backend/__init__.py +++ b/bot_bottle/backend/__init__.py @@ -284,8 +284,8 @@ class BottleImages: image refs. For the smolmachines backend they are Path objects pointing to pre-built `.smolmachine` artifacts.""" - agent: "str | Path" - sidecar: "str | Path" + agent: str | Path + sidecar: str | Path class BottleBackend(ABC, Generic[PlanT, CleanupT]): @@ -461,12 +461,12 @@ class BottleBackend(ABC, Generic[PlanT, CleanupT]): yield bottle @abstractmethod - def _build_or_load_images(self, plan: PlanT) -> "BottleImages": + def _build_or_load_images(self, plan: PlanT) -> BottleImages: """Return the agent and sidecar image references (or artifact paths) for this plan, building fresh images when the policy requires it.""" @abstractmethod - def _launch_impl(self, plan: PlanT, images: "BottleImages") -> AbstractContextManager[Bottle]: + def _launch_impl(self, plan: PlanT, images: BottleImages) -> AbstractContextManager[Bottle]: """Bring up the bottle using pre-resolved images; yield a handle; tear down on exit.""" def provision(self, plan: PlanT, bottle: "Bottle") -> str | None: