fix: CI lint + type-check failures

- ruff: remove unused imports, fix import sorting, line length in test_agent.py
- ruff: SIM105 contextlib.suppress in conftest.py
- mypy: add dict[str, Any] return type in reports.py
- mypy: str() cast for Any return in improve.py
- mypy: add autogen to ignore_missing_imports in pyproject.toml

Co-Authored-By: Oz <oz-agent@warp.dev>

Author: tbitcs

pyproject.toml

@@ -123,6 +123,8 @@ exclude = ["src/specsmith/gui"]
 module = [
     "anthropic",
     "anthropic.*",
+    "autogen",    # AG2 agent shell — optional extra
+    "autogen.*",
     "openai",
     "openai.*",
     "google",

src/specsmith/agents/reports.py

@@ -12,6 +12,7 @@
 from dataclasses import asdict, dataclass, field
 from datetime import datetime, timezone
 from pathlib import Path
+from typing import Any
 
 
 @dataclass
@@ -36,7 +37,7 @@ class ChangeReport:
         default_factory=lambda: datetime.now(tz=timezone.utc).isoformat()
     )
 
-    def to_dict(self) -> dict:
+    def to_dict(self) -> dict[str, Any]:
         """Convert to a JSON-serializable dict (excludes verbose fields)."""
         d = asdict(self)
         # Truncate verbose fields for the summary view

src/specsmith/agents/workflows/improve.py

@@ -120,7 +120,7 @@ def _extract_last_assistant_message(messages: list[dict[str, Any]]) -> str:
     """Get the last assistant message content from a conversation."""
     for msg in reversed(messages):
         if msg.get("role") == "assistant" and msg.get("content"):
-            return msg["content"]
+            return str(msg["content"])
     return ""
 
 

tests/conftest.py

@@ -9,6 +9,7 @@
 
 from __future__ import annotations
 
+import contextlib
 import sys
 
 
@@ -33,10 +34,8 @@ def pytest_configure(config):  # type: ignore[no-untyped-def]  # noqa: ARG001
         _orig = _pytest_pathlib.cleanup_dead_symlinks
 
         def _safe_cleanup(basepath):  # type: ignore[no-untyped-def]
-            try:
+            with contextlib.suppress(OSError):
                 _orig(basepath)
-            except OSError:
-                pass  # Suppress WinError 448
 
         # Patch in both modules — tmpdir imports the function directly
         _pytest_pathlib.cleanup_dead_symlinks = _safe_cleanup

tests/test_agent.py

@@ -11,27 +11,22 @@
 
 from __future__ import annotations
 
-import os
 from pathlib import Path
-from unittest.mock import patch
 
 import pytest
 import yaml
 
 from specsmith.agent.core import (
     CompletionResponse,
     Message,
-    ModelTier,
     Role,
     Tool,
     ToolParam,
-    ToolResult,
 )
 from specsmith.agent.runner import AgentRunner, SessionState, build_system_prompt
 from specsmith.agent.skills import load_skills
 from specsmith.agent.tools import build_tool_registry, get_tool_by_name
 
-
 # ---------------------------------------------------------------------------
 # Fixtures
 # ---------------------------------------------------------------------------
@@ -114,7 +109,11 @@ def test_write_file_handler(self, governed_project: Path) -> None:
         write_file = get_tool_by_name(tools, "write_file")
         assert write_file is not None
         result = write_file.handler(path="test_output.txt", content="hello world")
-        assert "wrote" in result.lower() or "created" in result.lower() or "test_output" in result.lower()
+        assert (
+            "wrote" in result.lower()
+            or "created" in result.lower()
+            or "test_output" in result.lower()
+        )
         assert (governed_project / "test_output.txt").read_text(encoding="utf-8") == "hello world"
 
     def test_run_command_handler(self, governed_project: Path) -> None: