tool description最適化

6 files changed, 85 insertions, 19 deletions

diff --git a/src/mcp/tools/delete_message.c b/src/mcp/tools/delete_message.c
index 8b6e8f9..f90bbe3 100644
--- a/src/mcp/tools/delete_message.c
+++ b/src/mcp/tools/delete_message.c
@@ -43,8 +43,22 @@ call_delete_message(const mcpc_tool_t *tool, mcpc_ucbr_t *result)
 bool
 tiny_mcp_register_delete_message(mcpc_server_t *server)
 {
-  mcpc_tool_t *tool = tiny_mcp_tool_new("delete_message", "Delete a message", call_delete_message);
-  return tool && tiny_mcp_tool_add_property(tool, "message_id", "Message ULID", MCPC_U8STR, true)
-         && tiny_mcp_tool_add_property(tool, "session_id", "Optional session ID", MCPC_U8STR, false)
+  mcpc_tool_t *tool = tiny_mcp_tool_new(
+      "delete_message",
+      "Delete a persisted message so it is immediately excluded from reads and "
+      "searches. Physical removal occurs after rebuild_index compacts the store.",
+      call_delete_message);
+  return tool
+         && tiny_mcp_tool_add_property(tool,
+                                       "message_id",
+                                       "Canonical 26-character ULID of the message to delete.",
+                                       MCPC_U8STR,
+                                       true)
+         && tiny_mcp_tool_add_property(tool,
+                                       "session_id",
+                                       "Optional session guard; when provided, delete only if the "
+                                       "message belongs to this session.",
+                                       MCPC_U8STR,
+                                       false)
          && mcpc_server_add_tool(server, tool) == MCPC_EC_0;
 }
diff --git a/src/mcp/tools/get_system_health.c b/src/mcp/tools/get_system_health.c
index eb02090..5fd3fc8 100644
--- a/src/mcp/tools/get_system_health.c
+++ b/src/mcp/tools/get_system_health.c
@@ -43,7 +43,10 @@ call_get_system_health(const mcpc_tool_t *tool, mcpc_ucbr_t *result)
 bool
 tiny_mcp_register_get_system_health(mcpc_server_t *server)
 {
-  mcpc_tool_t *tool
-      = tiny_mcp_tool_new("get_system_health", "Report storage health", call_get_system_health);
+  mcpc_tool_t *tool = tiny_mcp_tool_new(
+      "get_system_health",
+      "Inspect storage health, recovery state, message counts, and file sizes. Use "
+      "it to diagnose persistence or recovery problems.",
+      call_get_system_health);
   return tool && mcpc_server_add_tool(server, tool) == MCPC_EC_0;
 }
diff --git a/src/mcp/tools/read_messages.c b/src/mcp/tools/read_messages.c
index 1581534..44a026f 100644
--- a/src/mcp/tools/read_messages.c
+++ b/src/mcp/tools/read_messages.c
@@ -318,11 +318,29 @@ call_read_messages(const mcpc_tool_t *tool, mcpc_ucbr_t *result)
 bool
 tiny_mcp_register_read_messages(mcpc_server_t *server)
 {
-  mcpc_tool_t *tool = tiny_mcp_tool_new("read_messages", "Read a session page", call_read_messages);
-  return tool && tiny_mcp_tool_add_property(tool, "session_id", "Session ID", MCPC_U8STR, true)
+  mcpc_tool_t *tool
+      = tiny_mcp_tool_new("read_messages",
+                          "Read persisted messages from one known session in chronological order. "
+                          "Returns a paginated history for reconstructing that conversation.",
+                          call_read_messages);
+  return tool
+         && tiny_mcp_tool_add_property(tool,
+                                       "session_id",
+                                       "Conversation session whose chronological history to read.",
+                                       MCPC_U8STR,
+                                       true)
          && tiny_mcp_tool_add_property(
-             tool, "limit", "Optional page limit from 1 to 1000; defaults to 100", MCPC_I32, false)
+             tool,
+             "limit",
+             "Maximum messages to return in this page, from 1 to 1000; defaults to 100.",
+             MCPC_I32,
+             false)
          && tiny_mcp_tool_add_property(
-             tool, "cursor", "Optional opaque page cursor", MCPC_U8STR, false)
+             tool,
+             "cursor",
+             "Opaque nextCursor returned by a previous read_messages call; "
+             "omit for the first page.",
+             MCPC_U8STR,
+             false)
          && mcpc_server_add_tool(server, tool) == MCPC_EC_0;
 }
diff --git a/src/mcp/tools/rebuild_index.c b/src/mcp/tools/rebuild_index.c
index 7208d77..e6dbf69 100644
--- a/src/mcp/tools/rebuild_index.c
+++ b/src/mcp/tools/rebuild_index.c
@@ -59,6 +59,11 @@ call_rebuild_index(const mcpc_tool_t *tool, mcpc_ucbr_t *result)
 bool
 tiny_mcp_register_rebuild_index(mcpc_server_t *server)
 {
-  mcpc_tool_t *tool = tiny_mcp_tool_new("rebuild_index", "Compact and rebuild", call_rebuild_index);
+  mcpc_tool_t *tool = tiny_mcp_tool_new(
+      "rebuild_index",
+      "Compact the store and rebuild its snapshot and in-memory indexes. Use it "
+      "for maintenance or to physically remove logically deleted records; it may "
+      "block other requests while running.",
+      call_rebuild_index);
   return tool && mcpc_server_add_tool(server, tool) == MCPC_EC_0;
 }
diff --git a/src/mcp/tools/search_messages.c b/src/mcp/tools/search_messages.c
index d20d7b2..44db1ef 100644
--- a/src/mcp/tools/search_messages.c
+++ b/src/mcp/tools/search_messages.c
@@ -127,11 +127,29 @@ call_search_messages(const mcpc_tool_t *tool, mcpc_ucbr_t *result)
 bool
 tiny_mcp_register_search_messages(mcpc_server_t *server)
 {
-  mcpc_tool_t *tool = tiny_mcp_tool_new("search_messages", "Search messages", call_search_messages);
-  return tool && tiny_mcp_tool_add_property(tool, "query", "Search text", MCPC_U8STR, true)
+  mcpc_tool_t *tool
+      = tiny_mcp_tool_new("search_messages",
+                          "Search persisted message content and tags by text relevance. Use it to "
+                          "recall matching information globally or within one session.",
+                          call_search_messages);
+  return tool
          && tiny_mcp_tool_add_property(
-             tool, "session_id", "Optional session ID; omitted searches all", MCPC_U8STR, false)
+             tool,
+             "query",
+             "UTF-8 text to match against persisted message content and tags.",
+             MCPC_U8STR,
+             true)
          && tiny_mcp_tool_add_property(
-             tool, "limit", "Optional result limit from 1 to 100; defaults to 20", MCPC_I32, false)
+             tool,
+             "session_id",
+             "Conversation session to search; omit to search across all sessions.",
+             MCPC_U8STR,
+             false)
+         && tiny_mcp_tool_add_property(
+             tool,
+             "limit",
+             "Maximum ranked matches to return, from 1 to 100; defaults to 20.",
+             MCPC_I32,
+             false)
          && mcpc_server_add_tool(server, tool) == MCPC_EC_0;
 }
diff --git a/src/mcp/tools/write_message.c b/src/mcp/tools/write_message.c
index c4b18cc..7df0058 100644
--- a/src/mcp/tools/write_message.c
+++ b/src/mcp/tools/write_message.c
@@ -211,13 +211,21 @@ bool
 tiny_mcp_register_write_message(mcpc_server_t *server)
 {
   mcpc_tool_t *tool
-      = tiny_mcp_tool_new("write_message", "Append a conversation message", call_write_message);
-  return tool && tiny_mcp_tool_add_property(tool, "session_id", "Session ID", MCPC_U8STR, true)
-         && tiny_mcp_tool_add_property(tool, "role", "user, assistant, or system", MCPC_U8STR, true)
-         && tiny_mcp_tool_add_property(tool, "content", "Message text", MCPC_U8STR, true)
+      = tiny_mcp_tool_new("write_message",
+                          "Append and durably persist a conversation message. Use it to store "
+                          "information that should remain available after server restarts.",
+                          call_write_message);
+  return tool
+         && tiny_mcp_tool_add_property(
+             tool, "session_id", "Conversation session that owns the message.", MCPC_U8STR, true)
+         && tiny_mcp_tool_add_property(
+             tool, "role", "Message author role: user, assistant, or system.", MCPC_U8STR, true)
+         && tiny_mcp_tool_add_property(
+             tool, "content", "UTF-8 message text to persist.", MCPC_U8STR, true)
          && tiny_mcp_tool_add_property(tool,
                                        "metadata_json",
-                                       "Optional metadata object JSON; defaults to {}",
+                                       "Optional JSON object string containing tool_call_id, tags, "
+                                       "and/or memory_tier; defaults to {}.",
                                        MCPC_U8STR,
                                        false)
          && mcpc_server_add_tool(server, tool) == MCPC_EC_0;