mcp: remove search and over tools

This commit is contained in:
Adrià Arrufat
2026-03-03 22:50:06 +09:00
parent 48df38cbfe
commit 7bddc0a89c

View File

@@ -23,19 +23,6 @@ pub const tool_list = [_]protocol.Tool{
\\} \\}
), ),
}, },
.{
.name = "search",
.description = "Use a search engine to look for specific words, terms, sentences. The search page will then be loaded in memory.",
.inputSchema = protocol.minify(
\\{
\\ "type": "object",
\\ "properties": {
\\ "text": { "type": "string", "description": "The text to search for, must be a valid search query." }
\\ },
\\ "required": ["text"]
\\}
),
},
.{ .{
.name = "markdown", .name = "markdown",
.description = "Get the page content in markdown format. If a url is provided, it navigates to that url first.", .description = "Get the page content in markdown format. If a url is provided, it navigates to that url first.",
@@ -74,19 +61,6 @@ pub const tool_list = [_]protocol.Tool{
\\} \\}
), ),
}, },
.{
.name = "over",
.description = "Used to indicate that the task is over and give the final answer if there is any. This is the last tool to be called in a task.",
.inputSchema = protocol.minify(
\\{
\\ "type": "object",
\\ "properties": {
\\ "result": { "type": "string", "description": "The final result of the task." }
\\ },
\\ "required": ["result"]
\\}
),
},
}; };
pub fn handleList(server: *Server, arena: std.mem.Allocator, req: protocol.Request) !void { pub fn handleList(server: *Server, arena: std.mem.Allocator, req: protocol.Request) !void {
@@ -98,19 +72,11 @@ const GotoParams = struct {
url: [:0]const u8, url: [:0]const u8,
}; };
const SearchParams = struct {
text: [:0]const u8,
};
const EvaluateParams = struct { const EvaluateParams = struct {
script: [:0]const u8, script: [:0]const u8,
url: ?[:0]const u8 = null, url: ?[:0]const u8 = null,
}; };
const OverParams = struct {
result: [:0]const u8,
};
const ToolStreamingText = struct { const ToolStreamingText = struct {
server: *Server, server: *Server,
action: enum { markdown, links }, action: enum { markdown, links },
@@ -155,21 +121,17 @@ const ToolStreamingText = struct {
const ToolAction = enum { const ToolAction = enum {
goto, goto,
navigate, navigate,
search,
markdown, markdown,
links, links,
evaluate, evaluate,
over,
}; };
const tool_map = std.StaticStringMap(ToolAction).initComptime(.{ const tool_map = std.StaticStringMap(ToolAction).initComptime(.{
.{ "goto", .goto }, .{ "goto", .goto },
.{ "navigate", .navigate }, .{ "navigate", .navigate },
.{ "search", .search },
.{ "markdown", .markdown }, .{ "markdown", .markdown },
.{ "links", .links }, .{ "links", .links },
.{ "evaluate", .evaluate }, .{ "evaluate", .evaluate },
.{ "over", .over },
}); });
pub fn handleCall(server: *Server, arena: std.mem.Allocator, req: protocol.Request) !void { pub fn handleCall(server: *Server, arena: std.mem.Allocator, req: protocol.Request) !void {
@@ -192,11 +154,9 @@ pub fn handleCall(server: *Server, arena: std.mem.Allocator, req: protocol.Reque
switch (action) { switch (action) {
.goto, .navigate => try handleGoto(server, arena, req.id.?, call_params.arguments), .goto, .navigate => try handleGoto(server, arena, req.id.?, call_params.arguments),
.search => try handleSearch(server, arena, req.id.?, call_params.arguments),
.markdown => try handleMarkdown(server, arena, req.id.?, call_params.arguments), .markdown => try handleMarkdown(server, arena, req.id.?, call_params.arguments),
.links => try handleLinks(server, arena, req.id.?, call_params.arguments), .links => try handleLinks(server, arena, req.id.?, call_params.arguments),
.evaluate => try handleEvaluate(server, arena, req.id.?, call_params.arguments), .evaluate => try handleEvaluate(server, arena, req.id.?, call_params.arguments),
.over => try handleOver(server, arena, req.id.?, call_params.arguments),
} }
} }
@@ -208,24 +168,6 @@ fn handleGoto(server: *Server, arena: std.mem.Allocator, id: std.json.Value, arg
try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content }); try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content });
} }
fn handleSearch(server: *Server, arena: std.mem.Allocator, id: std.json.Value, arguments: ?std.json.Value) !void {
const args = try parseArguments(SearchParams, arena, arguments, server, id, "search");
const component: std.Uri.Component = .{ .raw = args.text };
var url_aw = std.Io.Writer.Allocating.init(arena);
component.formatQuery(&url_aw.writer) catch {
return server.sendError(id, .InternalError, "Internal error formatting query");
};
const url = std.fmt.allocPrintSentinel(arena, "https://duckduckgo.com/?q={s}", .{url_aw.written()}, 0) catch {
return server.sendError(id, .InternalError, "Internal error formatting URL");
};
try performGoto(server, url, id);
const content = [_]protocol.TextContent([]const u8){.{ .text = "Search performed successfully." }};
try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content });
}
fn handleMarkdown(server: *Server, arena: std.mem.Allocator, id: std.json.Value, arguments: ?std.json.Value) !void { fn handleMarkdown(server: *Server, arena: std.mem.Allocator, id: std.json.Value, arguments: ?std.json.Value) !void {
const MarkdownParams = struct { const MarkdownParams = struct {
url: ?[:0]const u8 = null, url: ?[:0]const u8 = null,
@@ -292,13 +234,6 @@ fn handleEvaluate(server: *Server, arena: std.mem.Allocator, id: std.json.Value,
try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content }); try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content });
} }
fn handleOver(server: *Server, arena: std.mem.Allocator, id: std.json.Value, arguments: ?std.json.Value) !void {
const args = try parseArguments(OverParams, arena, arguments, server, id, "over");
const content = [_]protocol.TextContent([]const u8){.{ .text = args.result }};
try server.sendResult(id, protocol.CallToolResult([]const u8){ .content = &content });
}
fn parseArguments(comptime T: type, arena: std.mem.Allocator, arguments: ?std.json.Value, server: *Server, id: std.json.Value, tool_name: []const u8) !T { fn parseArguments(comptime T: type, arena: std.mem.Allocator, arguments: ?std.json.Value, server: *Server, id: std.json.Value, tool_name: []const u8) !T {
if (arguments == null) { if (arguments == null) {
try server.sendError(id, .InvalidParams, "Missing arguments"); try server.sendError(id, .InvalidParams, "Missing arguments");