didericis/bot-bottle
1
0
Fork 0
Code Issues 18 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix: remove redundant quoted annotations in BottleImages and abstract methods

Browse Source 
`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.
This commit is contained in:
didericis (claude)
2026-07-10 20:36:19 +00:00
parent 715e474306
commit 4a4d3b2455
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
bot_bottle/backend/__init__.py
+4 -4
View File
@@ -284,8 +284,8 @@ class BottleImages:
image refs. For the smolmachines backend they are Path objects pointing image refs. For the smolmachines backend they are Path objects pointing
to pre-built `.smolmachine` artifacts.""" to pre-built `.smolmachine` artifacts."""
agent: "str | Path" agent: str | Path
sidecar: "str | Path" sidecar: str | Path
class BottleBackend(ABC, Generic[PlanT, CleanupT]): class BottleBackend(ABC, Generic[PlanT, CleanupT]):
@@ -461,12 +461,12 @@ class BottleBackend(ABC, Generic[PlanT, CleanupT]):
yield bottle yield bottle
@abstractmethod @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) """Return the agent and sidecar image references (or artifact paths)
for this plan, building fresh images when the policy requires it.""" for this plan, building fresh images when the policy requires it."""
@abstractmethod @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.""" """Bring up the bottle using pre-resolved images; yield a handle; tear down on exit."""
def provision(self, plan: PlanT, bottle: "Bottle") -> str | None: def provision(self, plan: PlanT, bottle: "Bottle") -> str | None: