fix(mcp): ensure required field if nil (#278)

gedzeppelin · web-flow · commit 1f6eef460ec9 · 2025-07-01T11:50:43.000+02:00
diff --git a/internal/llm/agent/mcp-tools.go b/internal/llm/agent/mcp-tools.go
@@ -33,11 +33,15 @@ type MCPClient interface {
 }
 
 func (b *mcpTool) Info() tools.ToolInfo {
+	required := b.tool.InputSchema.Required
+	if required == nil {
+		required = make([]string, 0)
+	}
 	return tools.ToolInfo{
 		Name:        fmt.Sprintf("%s_%s", b.mcpName, b.tool.Name),
 		Description: b.tool.Description,
 		Parameters:  b.tool.InputSchema.Properties,
-		Required:    b.tool.InputSchema.Required,
+		Required:    required,
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -33,11 +33,15 @@ type MCPClient interface {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`func (b *mcpTool) Info() tools.ToolInfo {`
	`36`	`+ required := b.tool.InputSchema.Required`
	`37`	`+ if required == nil {`
	`38`	`+ required = make([]string, 0)`
	`39`	`+ }`
`36`	`40`	`return tools.ToolInfo{`
`37`	`41`	`Name: fmt.Sprintf("%s_%s", b.mcpName, b.tool.Name),`
`38`	`42`	`Description: b.tool.Description,`
`39`	`43`	`Parameters: b.tool.InputSchema.Properties,`
`40`		`- Required: b.tool.InputSchema.Required,`
	`44`	`+ Required: required,`
`41`	`45`	`}`
`42`	`46`	`}`
`43`	`47`