nit: change discover_profiles tool name to "list_profiles"

Author: patricka3125

docs/skills.md

@@ -100,7 +100,7 @@ CAO ships with two default skills:
 
 ## How Agents Discover Skills
 
-All installed skills are available to all CAO agents — there is no per-profile skill declaration. When an agent is launched, CAO appends a catalog block to the prompt listing every installed skill's name and description, along with instructions to use the `get_skill` MCP tool to retrieve full content. The agent then decides when and whether to load each skill based on the task at hand.
+All installed skills are available to all CAO agents — there is no per-profile skill declaration. When an agent is launched, CAO appends a catalog block to the prompt listing every installed skill's name and description, along with instructions to use the `load_skill` MCP tool to retrieve full content. The agent then decides when and whether to load each skill based on the task at hand.
 
 You can explicitly instruct the agent to load specific skills eagerly in the agent profile body:
 
@@ -114,19 +114,19 @@ Skills are delivered to agents differently depending on the provider. The table
 
 | Provider | Injection Method | When Catalog Updates | Skill Retrieval |
 |----------|-----------------|---------------------|-----------------|
-| Claude Code | Runtime prompt | Every terminal creation | `get_skill` MCP tool |
-| Codex | Runtime prompt | Every terminal creation | `get_skill` MCP tool |
-| Gemini CLI | Runtime prompt | Every terminal creation | `get_skill` MCP tool |
-| Kimi CLI | Runtime prompt | Every terminal creation | `get_skill` MCP tool |
+| Claude Code | Runtime prompt | Every terminal creation | `load_skill` MCP tool |
+| Codex | Runtime prompt | Every terminal creation | `load_skill` MCP tool |
+| Gemini CLI | Runtime prompt | Every terminal creation | `load_skill` MCP tool |
+| Kimi CLI | Runtime prompt | Every terminal creation | `load_skill` MCP tool |
 | Kiro CLI | Native `skill://` resources | Every terminal creation | Kiro progressive loading |
-| Q CLI | Baked into agent JSON at install | On `cao skills add/remove` | `get_skill` MCP tool |
-| Copilot CLI | Baked into `.agent.md` at install | On `cao skills add/remove` | `get_skill` MCP tool |
+| Q CLI | Baked into agent JSON at install | On `cao skills add/remove` | `load_skill` MCP tool |
+| Copilot CLI | Baked into `.agent.md` at install | On `cao skills add/remove` | `load_skill` MCP tool |
 
 ### Runtime Prompt Providers (Claude Code, Codex, Gemini CLI, Kimi CLI)
 
 For these providers, the skill catalog is built fresh each time a terminal is created. The catalog — a list of skill names and descriptions — is appended to the system prompt via the provider's native CLI flags.
 
-The agent retrieves full skill content at runtime by calling the `get_skill` MCP tool, which fetches the skill body from the CAO server.
+The agent retrieves full skill content at runtime by calling the `load_skill` MCP tool, which fetches the skill body from the CAO server.
 
 No action is needed after `cao skills add` or `cao skills remove` — the next terminal created will automatically reflect the current set of installed skills.
 

src/cli_agent_orchestrator/mcp_server/server.py

@@ -39,7 +39,7 @@
     """,
 )
 
-GET_SKILL_TOOL_DESCRIPTION = """Retrieve the full Markdown body of an available skill from cao-server.
+LOAD_SKILL_TOOL_DESCRIPTION = """Retrieve the full Markdown body of an available skill from cao-server.
 
 Use this tool when your prompt lists a CAO skill and you need its full instructions at runtime.
 
@@ -266,7 +266,7 @@ def _extract_error_detail(response: requests.Response, fallback: str) -> str:
     return fallback
 
 
-def _get_skill_impl(name: str) -> Union[str, Dict[str, Any]]:
+def _load_skill_impl(name: str) -> Union[str, Dict[str, Any]]:
     """Fetch a skill body from cao-server and return content or a structured error."""
     try:
         response = requests.get(f"{API_BASE_URL}/skills/{name}")
@@ -608,12 +608,12 @@ async def send_message(
     return _send_message_impl(receiver_id, message)
 
 
-@mcp.tool(description=GET_SKILL_TOOL_DESCRIPTION)
-async def get_skill(
+@mcp.tool(description=LOAD_SKILL_TOOL_DESCRIPTION)
+async def load_skill(
     name: str = Field(description="Name of the skill to retrieve"),
 ) -> Any:
     """Retrieve skill content from cao-server."""
-    return _get_skill_impl(name)
+    return _load_skill_impl(name)
 
 
 def main():

src/cli_agent_orchestrator/utils/skills.py

@@ -14,7 +14,7 @@
 
 SKILL_CATALOG_INSTRUCTION = (
     "The following skills are available exclusively in this CAO orchestration context. "
-    "To load a skill's full content, use the `get_skill` MCP tool provided by the CAO MCP server. "
+    "To load a skill's full content, use the `load_skill` MCP tool provided by the CAO MCP server. "
     "These skills are not accessible through provider-native skill commands or directories."
 )
 

test/e2e/test_skills.py

@@ -3,7 +3,7 @@
 Tests the full skill pipeline — validates that:
 1. The skill catalog text is injected into the provider CLI command
    (verified by inspecting the tmux scrollback, not LLM output)
-2. The get_skill API endpoint returns installed skill content
+2. The load_skill MCP tool / API endpoint returns installed skill content
 
 Requires: running CAO server, authenticated CLI tools, tmux, seeded skills.
 

test/mcp_server/test_get_skill.py

@@ -163,7 +163,7 @@ def test_create_terminal_appends_skill_catalog(
         mock_build_skill_catalog.return_value = (
             "## Available Skills\n\n"
             "The following skills are available exclusively in this CAO orchestration context. "
-            "To load a skill's full content, use the `get_skill` MCP tool provided by the "
+            "To load a skill's full content, use the `load_skill` MCP tool provided by the "
             "CAO MCP server. These skills are not accessible through provider-native skill "
             "commands or directories.\n\n"
             "- **cao-worker-protocols**: Worker communication\n"
@@ -180,7 +180,7 @@ def test_create_terminal_appends_skill_catalog(
         assert skill_prompt == (
             "## Available Skills\n\n"
             "The following skills are available exclusively in this CAO orchestration context. "
-            "To load a skill's full content, use the `get_skill` MCP tool provided by the "
+            "To load a skill's full content, use the `load_skill` MCP tool provided by the "
             "CAO MCP server. These skills are not accessible through provider-native skill "
             "commands or directories.\n\n"
             "- **cao-worker-protocols**: Worker communication\n"
@@ -266,7 +266,7 @@ def test_create_terminal_does_not_pass_skill_prompt_to_non_runtime_provider(
         mock_build_skill_catalog.return_value = (
             "## Available Skills\n\n"
             "The following skills are available exclusively in this CAO orchestration context. "
-            "To load a skill's full content, use the `get_skill` MCP tool provided by the "
+            "To load a skill's full content, use the `load_skill` MCP tool provided by the "
             "CAO MCP server. These skills are not accessible through provider-native skill "
             "commands or directories.\n\n"
             "- **python-testing**: Pytest conventions"

test/services/test_terminal_service_full.py

@@ -268,7 +268,7 @@ def test_renders_all_installed_skills(self, mock_list_skills):
         assert build_skill_catalog() == (
             "## Available Skills\n\n"
             "The following skills are available exclusively in this CAO orchestration context. "
-            "To load a skill's full content, use the `get_skill` MCP tool provided by the "
+            "To load a skill's full content, use the `load_skill` MCP tool provided by the "
             "CAO MCP server. These skills are not accessible through provider-native skill "
             "commands or directories.\n\n"
             "- **cao-worker-protocols**: Worker communication\n"