fix(common): Improve constant SEP from "-" to "__" for namespaced names (#298)

* fix(common): Improve constant SEP from "-" to "__" for namespaced names

- Modify test cases to include prompt testing

* fix(#298): Improve some hard-coded SEP value in tests

* fix(#298): Change 'check_weather_tool' back to 'check_weather' to avoid affect other tests

* update

* filtering improvements

* improve filtering

* break down filtering tests

---------

Co-authored-by: evalstate <[email protected]>

Author: codeboyzhou

examples/a2a/agent_executor.py

@@ -18,7 +18,7 @@
 DEFAULT_AGENT_NAME = "helper"
 
 fast = FastAgent(
-    "A2A FastAgent Demo",
+    "A2A fast-agent Demo",
     parse_cli_args=False,
     quiet=True,
 )

src/fast_agent/agents/agent_types.py

@@ -5,7 +5,6 @@
 from dataclasses import dataclass, field
 from enum import StrEnum, auto
 from pathlib import Path
-from typing import Dict, List, Optional
 
 from mcp.client.session import ElicitationFnT
 
@@ -35,12 +34,12 @@ class AgentConfig:
 
     name: str
     instruction: str = "You are a helpful agent."
-    servers: List[str] = field(default_factory=list)
-    tools: Optional[Dict[str, List[str]]] = None
-    resources: Optional[Dict[str, List[str]]] = None
-    prompts: Optional[Dict[str, List[str]]] = None
+    servers: list[str] = field(default_factory=list)
+    tools: dict[str, list[str]] = field(default_factory=dict)  # filters for tools
+    resources: dict[str, list[str]] = field(default_factory=dict)  # filters for resources
+    prompts: dict[str, list[str]] = field(default_factory=dict)  # filters for prompts
     skills: SkillManifest | SkillRegistry | Path | str | None = None
-    skill_manifests: List[SkillManifest] = field(default_factory=list, repr=False)
+    skill_manifests: list[SkillManifest] = field(default_factory=list, repr=False)
     model: str | None = None
     use_history: bool = True
     default_request_params: RequestParams | None = None

src/fast_agent/agents/llm_agent.py

@@ -10,18 +10,7 @@
 
 from typing import Callable, List, Optional, Tuple
 
-try:
-    from a2a.types import AgentCapabilities  # type: ignore
-except Exception:  # pragma: no cover - optional dependency fallback
-    from dataclasses import dataclass
-
-    @dataclass
-    class AgentCapabilities:  # minimal fallback
-        streaming: bool = False
-        push_notifications: bool = False
-        state_transition_history: bool = False
-
-
+from a2a.types import AgentCapabilities
 from mcp import Tool
 from rich.text import Text