Fix Windows path assertion + correct stale Tavily/demo-tools comments

- test_default_servers_curated_set_and_filesystem_root hardcoded a forward-slash
  path, which fails on Windows where str(Path(...)) uses backslashes; compare
  against str(root) instead. The production code (a native path for the
  filesystem MCP server) was already correct.
- Update three docstrings/comments in the live agent that still referenced
  TAVILY_API_KEY / --demo-tools after the switch to Firecrawl and default-loaded
  tools (flagged by review).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S2tt5oaMXD2RCVoAqGbqyR

Author: claude

aai_cli/agent_cascade/brain.py

@@ -60,8 +60,8 @@ def _tool_capabilities(tools: Sequence[BaseTool]) -> list[str]:
     """The spoken-capability phrases backed by an actually-present tool.
 
     Derived from the resolved tool names so the prompt never advertises a capability the
-    agent can't perform: web search is present only with a ``TAVILY_API_KEY``, and the docs
-    tools are best-effort (absent when the docs host is unreachable).
+    agent can't perform: web search is present only with a ``FIRECRAWL_API_KEY``, and the
+    docs tools are best-effort (absent when the docs host is unreachable).
     """
     names = {tool.name for tool in tools}
     capabilities: list[str] = []
@@ -94,7 +94,7 @@ def build_system_prompt(
 
     The guidance is tailored to the bound tools so the model is only told about
     capabilities it actually has — advertising a missing tool (web search without a
-    ``TAVILY_API_KEY``) made the agent announce an action it then couldn't take, leaving
+    ``FIRECRAWL_API_KEY``) made the agent announce an action it then couldn't take, leaving
     the turn hanging with no answer. ``tools`` are the built-in legs (web search, URL
     fetch, AssemblyAI docs); ``extra_tools`` are user-configured MCP tools, advertised
     generically by name. With no tools at all the model answers from its own knowledge.

aai_cli/agent_cascade/config.py

@@ -44,7 +44,8 @@ class CascadeConfig:
     # Extra streaming-TTS query params (the --tts-config escape hatch).
     tts_extra: Mapping[str, str] = field(default_factory=dict[str, str])
     # MCP servers (name -> launch spec) whose tools the deepagents brain can call. Empty
-    # by default; populated from --mcp-config files and/or the --demo-tools curated set.
+    # here by default; the live command populates it with the curated default set plus any
+    # --mcp-config files.
     mcp_servers: Mapping[str, Mapping[str, object]] = field(
         default_factory=dict[str, Mapping[str, object]]
     )

tests/test_agent_cascade_mcp.py

@@ -20,16 +20,18 @@
 
 
 def test_default_servers_curated_set_and_filesystem_root():
-    servers = mcp_tools.default_servers(Path("/notes/dir"))
+    root = Path("/notes/dir")
+    servers = mcp_tools.default_servers(root)
     # The five curated, no-auth servers, each with a real launch command.
     assert set(servers) == {"time", "fetch", "memory", "filesystem", "weather"}
     assert servers["time"] == {"command": "uvx", "args": ["mcp-server-time"]}
     assert servers["memory"]["args"] == ["-y", "@modelcontextprotocol/server-memory"]
-    # The filesystem server is scoped to the passed-in root directory.
+    # The filesystem server is scoped to the passed-in root directory. Compare against
+    # str(root), not a hardcoded "/notes/dir", so it holds on Windows (backslash paths).
     assert servers["filesystem"]["args"] == [
         "-y",
         "@modelcontextprotocol/server-filesystem",
-        "/notes/dir",
+        str(root),
     ]