refactor(dataclass): remove pyright ignores by using parameterized generic aliases as field factories (#5)

* Initial plan

* refactor(kagent): replace unnecessary lambda wrappers in dataclass field factories

Co-authored-by: shenxiangzhuang <17157965+shenxiangzhuang@users.noreply.github.com>

* refactor: replace bare list/dict default_factory with parameterized generics, removing pyright ignores

Co-authored-by: shenxiangzhuang <17157965+shenxiangzhuang@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shenxiangzhuang <17157965+shenxiangzhuang@users.noreply.github.com>

Author: Copilot

packages/kagent/src/kagent/state.py

@@ -38,7 +38,7 @@ class AgentState:
     trace: Trace = field(default_factory=Trace)
     """Append-only execution trace.  The single source of truth."""
 
-    tools: list[Tool] = field(default_factory=lambda: list[Tool]())
+    tools: list[Tool] = field(default_factory=list[Tool])
     """Available tools for the agent."""
 
     @property

packages/kagent/src/kagent/trace/entry.py

@@ -105,7 +105,7 @@ class TraceEntry:
     message: Message | None = None
     """The ``kai.Message`` for message-bearing entries."""
 
-    data: dict[str, Any] = field(default_factory=lambda: {})
+    data: dict[str, Any] = field(default_factory=dict[str, Any])
     """Kind-specific structured data (e.g. anchor name, event payload)."""
 
     meta: TraceMeta = field(default_factory=TraceMeta)

packages/kcastle/src/kcastle/config.py

@@ -49,7 +49,7 @@ class ChannelConfig:
     """Configuration for a single channel."""
 
     enabled: bool = True
-    options: dict[str, object] = field(default_factory=dict)  # pyright: ignore[reportUnknownVariableType]
+    options: dict[str, object] = field(default_factory=dict[str, object])
 
 
 @dataclass(frozen=True, slots=True)
@@ -65,7 +65,7 @@ class CastleConfig:
     skills_dir: Path
     """User-level skills directory."""
 
-    providers: dict[str, ProviderEntry] = field(default_factory=dict)  # pyright: ignore[reportUnknownVariableType]
+    providers: dict[str, ProviderEntry] = field(default_factory=dict[str, ProviderEntry])
     """Configured providers keyed by name."""
 
     default_provider: str = ""

packages/kcastle/src/kcastle/providers/config.py

@@ -22,7 +22,7 @@ class ProviderConfig:
     api_key: str | None = None
     base_url: str | None = None
     extra_body: dict[str, object] | None = None
-    options: dict[str, object] = field(default_factory=dict)  # pyright: ignore[reportUnknownVariableType]
+    options: dict[str, object] = field(default_factory=dict[str, object])
 
     @property
     def name(self) -> str:
@@ -40,7 +40,7 @@ class ModelConfig:
     active: bool = True
     """Whether this model is available for use."""
 
-    options: dict[str, object] = field(default_factory=dict)  # pyright: ignore[reportUnknownVariableType]
+    options: dict[str, object] = field(default_factory=dict[str, object])
     """Provider-specific model options (``max_tokens``, ``reasoning``, etc.)."""
 
 
@@ -55,7 +55,7 @@ class ProviderEntry:
     config: ProviderConfig
     """Runtime provider config."""
 
-    models: list[ModelConfig] = field(default_factory=list)  # pyright: ignore[reportUnknownVariableType]
+    models: list[ModelConfig] = field(default_factory=list[ModelConfig])
     """Available models for this provider."""
 
     @property

packages/kcastle/src/kcastle/skills/skill.py

@@ -40,7 +40,7 @@ class Skill:
     path: Path
     """Absolute path to the skill directory."""
 
-    tags: list[str] = field(default_factory=list)  # pyright: ignore[reportUnknownVariableType]
+    tags: list[str] = field(default_factory=list[str])
     """Optional searchable tags."""
 
     instructions: str = ""