From f597ab2810d4d7e023e43091d5692a9e32108742 Mon Sep 17 00:00:00 2001 From: anti Date: Fri, 1 May 2026 01:26:24 -0400 Subject: [PATCH] =?UTF-8?q?fix(types):=20T1=20=E2=80=94=20remove=2015=20st?= =?UTF-8?q?ale=20type:=20ignore=20comments=20confirmed=20unused=20by=20myp?= =?UTF-8?q?y?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- decnet/bus/factory.py | 2 +- decnet/canary/dns_server.py | 6 +++--- decnet/canary/generators/honeydoc_pdf.py | 2 +- decnet/canary/instrumenters/image.py | 2 +- decnet/canary/instrumenters/pdf.py | 2 +- decnet/canary/worker.py | 2 +- decnet/collector/worker.py | 4 ++-- decnet/engine/services_live.py | 2 +- decnet/topology/validate.py | 2 +- decnet/vectorstore/sqlite_vec.py | 2 +- decnet/web/router/deckies/api_tarpit.py | 2 +- decnet/web/worker_registry.py | 2 +- 12 files changed, 15 insertions(+), 15 deletions(-) diff --git a/decnet/bus/factory.py b/decnet/bus/factory.py index f7a935ac..6743da1c 100644 --- a/decnet/bus/factory.py +++ b/decnet/bus/factory.py @@ -76,7 +76,7 @@ def _maybe_wrap_telemetry(bus: BaseBus) -> BaseBus: up at all we no-op. """ try: - from decnet.telemetry import wrap_repository # type: ignore[attr-defined] + from decnet.telemetry import wrap_repository except ImportError: return bus try: diff --git a/decnet/canary/dns_server.py b/decnet/canary/dns_server.py index 65cc6f60..b8d25756 100644 --- a/decnet/canary/dns_server.py +++ b/decnet/canary/dns_server.py @@ -169,10 +169,10 @@ class CanaryDNSProtocol(asyncio.DatagramProtocol): self._answer_ip = answer_ip self._transport: Optional[asyncio.DatagramTransport] = None - def connection_made(self, transport) -> None: # type: ignore[override] - self._transport = transport # type: ignore[assignment] + def connection_made(self, transport) -> None: + self._transport = transport - def datagram_received( # type: ignore[override] + def datagram_received( self, data: bytes, addr: Tuple[str, int], ) -> None: try: diff --git a/decnet/canary/generators/honeydoc_pdf.py b/decnet/canary/generators/honeydoc_pdf.py index 400271ff..681e134e 100644 --- a/decnet/canary/generators/honeydoc_pdf.py +++ b/decnet/canary/generators/honeydoc_pdf.py @@ -43,7 +43,7 @@ class HoneydocPdfGenerator(CanaryGenerator): def generate(self, ctx: CanaryContext) -> CanaryArtifact: try: - from pikepdf import Pdf, Name, Dictionary, String # type: ignore[import-not-found] + from pikepdf import Pdf, Name, Dictionary, String except ImportError as e: raise InstrumenterRejectedError( "honeydoc_pdf requires pikepdf; install it (`pip install " diff --git a/decnet/canary/instrumenters/image.py b/decnet/canary/instrumenters/image.py index 69e31ff4..99052a83 100644 --- a/decnet/canary/instrumenters/image.py +++ b/decnet/canary/instrumenters/image.py @@ -32,7 +32,7 @@ class ImageInstrumenter(CanaryInstrumenter): self, blob: bytes, ctx: CanaryContext, *, target_path: str, ) -> CanaryArtifact: try: - from PIL import Image, PngImagePlugin # type: ignore[import-not-found] + from PIL import Image, PngImagePlugin except ImportError as e: raise InstrumenterRejectedError( "image instrumenter requires Pillow; install it (`pip " diff --git a/decnet/canary/instrumenters/pdf.py b/decnet/canary/instrumenters/pdf.py index 516b6999..e868f8b0 100644 --- a/decnet/canary/instrumenters/pdf.py +++ b/decnet/canary/instrumenters/pdf.py @@ -34,7 +34,7 @@ class PdfInstrumenter(CanaryInstrumenter): self, blob: bytes, ctx: CanaryContext, *, target_path: str, ) -> CanaryArtifact: try: - import pikepdf # type: ignore[import-not-found] + import pikepdf except ImportError as e: raise InstrumenterRejectedError( "PDF instrumenter requires pikepdf; install it (`pip " diff --git a/decnet/canary/worker.py b/decnet/canary/worker.py index 9ce390ca..3eea2573 100644 --- a/decnet/canary/worker.py +++ b/decnet/canary/worker.py @@ -380,7 +380,7 @@ async def _start_dns_server( local_addr=(_dns_bind(), _dns_port()), ) log.info("canary.dns listening zone=%s port=%d", zone, _dns_port()) - return transport # type: ignore[return-value] + return transport # ---------------------------- entry point -------------------------------- diff --git a/decnet/collector/worker.py b/decnet/collector/worker.py index 885afe85..d744cec1 100644 --- a/decnet/collector/worker.py +++ b/decnet/collector/worker.py @@ -368,7 +368,7 @@ def _stream_container( publish_fn: CollectorPublishFn | None = None, ) -> None: """Stream logs from one container and append to the host log files.""" - import docker # type: ignore[import] + import docker lf: Optional[Any] = None jf: Optional[Any] = None @@ -472,7 +472,7 @@ async def log_collector_worker(log_file: str) -> None: Watches Docker events to pick up containers started after initial scan. """ - import docker # type: ignore[import] + import docker log_path = Path(log_file) json_path = log_path.with_suffix(".json") diff --git a/decnet/engine/services_live.py b/decnet/engine/services_live.py index ca81a30c..cb5dc266 100644 --- a/decnet/engine/services_live.py +++ b/decnet/engine/services_live.py @@ -338,7 +338,7 @@ async def _persist_fleet_change( repo: BaseRepository, decky: Any, services: list[str], compose_path: Path, ) -> None: """Persist the mutation to JSON state, compose file, and the DB row.""" - config, _ = _load_state() # type: ignore[misc] — checked earlier + config, _ = _load_state() target = _fleet_find_decky(config, decky.name) target.services = services _save_state(config, compose_path) diff --git a/decnet/topology/validate.py b/decnet/topology/validate.py index 5e71cbec..1cce0b8c 100644 --- a/decnet/topology/validate.py +++ b/decnet/topology/validate.py @@ -374,7 +374,7 @@ def check_no_host_port_collision(h: dict[str, Any]) -> list[ValidationIssue]: return [] try: - import psutil # type: ignore + import psutil bound = { c.laddr.port for c in psutil.net_connections(kind="inet") diff --git a/decnet/vectorstore/sqlite_vec.py b/decnet/vectorstore/sqlite_vec.py index b38c414d..46b46ca1 100644 --- a/decnet/vectorstore/sqlite_vec.py +++ b/decnet/vectorstore/sqlite_vec.py @@ -45,7 +45,7 @@ class SqliteVecUnavailable(RuntimeError): def _load_sqlite_vec(conn: sqlite3.Connection) -> None: try: - import sqlite_vec # type: ignore[import-untyped] + import sqlite_vec except ImportError as e: raise SqliteVecUnavailable("sqlite_vec package not installed") from e try: diff --git a/decnet/web/router/deckies/api_tarpit.py b/decnet/web/router/deckies/api_tarpit.py index e8c1896a..73f44496 100644 --- a/decnet/web/router/deckies/api_tarpit.py +++ b/decnet/web/router/deckies/api_tarpit.py @@ -32,7 +32,7 @@ router = APIRouter(prefix="/deckies/{decky_name}/tarpit", tags=["Deckies"]) _DECKY_RE = r"^[a-z0-9\-]{1,64}$" -def _tc(*args: str) -> subprocess.CompletedProcess: # type: ignore[type-arg] +def _tc(*args: str) -> subprocess.CompletedProcess[str]: cmd = ["tc", *args] return subprocess.run(cmd, capture_output=True, text=True) # nosec B603 B404 diff --git a/decnet/web/worker_registry.py b/decnet/web/worker_registry.py index 2db56d5d..c36e076a 100644 --- a/decnet/web/worker_registry.py +++ b/decnet/web/worker_registry.py @@ -99,7 +99,7 @@ class WorkerRegistry: payload.pop("ts", None) out.append(WorkerStatus( name=name, - status=status, # type: ignore[arg-type] + status=status, last_heartbeat_ts=ts, seconds_since=seconds_since, extra=payload,