fix: send_message tool name collision, thinking leak, team error reporting

1. send_message tool: renamed 'type' param to 'message_type' to avoid
   JSON Schema keyword collision that caused LLMs to silently drop it.
   Made message_type optional with default 'task'. Only 'to' and
   'content' are now required.

2. Thinking events: mesh forwarder now maps executor.EventThinking to
   "thinking" type. Team event converter suppresses thinking events
   from team output (model reasoning is noise in team context).

3. Team error reporting: completion now emits error events for each
   agent failure before the completion event, so users see what went
   wrong instead of just "completed".

Note: Qwen3 thinking content still leaks as regular text when Ollama's
think mode is not explicitly enabled. This is an Ollama/model behavior
issue — without think:true in config, reasoning is mixed into the
content stream with no way to separate it at the Genkit level.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: intel352

internal/daemon/teams.go

@@ -635,6 +635,11 @@ func (tm *TeamManager) StartMeshTeam(
 						},
 					},
 				}
+			case "thinking":
+				// Suppress thinking events from team output — they contain
+				// the model's internal reasoning which is noise in team context.
+				// If we want to expose this later, TeamEvent needs a Thinking variant.
+				continue
 			case "error":
 				pbEv = &pb.TeamEvent{
 					Event: &pb.TeamEvent_Error{
@@ -668,10 +673,26 @@ func (tm *TeamManager) StartMeshTeam(
 		// doneCh, so Result() is safe to call without waiting again).
 		result := handle.Result()
 
-		summary := fmt.Sprintf("Team completed task: %s (status: %s)", task, result.Status)
+		// Emit error events for any agent failures before the completion event.
 		if len(result.Errors) > 0 {
-			summary += fmt.Sprintf(" [%d errors]", len(result.Errors))
+			for _, err := range result.Errors {
+				select {
+				case eventCh <- &pb.TeamEvent{
+					Event: &pb.TeamEvent_Error{
+						Error: &pb.ErrorEvent{Message: err.Error()},
+					},
+				}:
+				case <-ctx.Done():
+					return
+				}
+			}
+		}
+
+		status := result.Status
+		if status == "" {
+			status = "completed"
 		}
+		summary := fmt.Sprintf("Team %s task: %s", status, task)
 
 		select {
 		case eventCh <- &pb.TeamEvent{
@@ -684,7 +705,7 @@ func (tm *TeamManager) StartMeshTeam(
 		case <-ctx.Done():
 		}
 
-		tm.markDone(ti, result.Status)
+		tm.markDone(ti, status)
 	}()
 
 	return teamID, eventCh

internal/mesh/mesh.go

@@ -364,6 +364,8 @@ func makeEventForwarder(ch chan<- Event, agentID string) func(executor.Event) {
 			evType = "tool_call"
 		case executor.EventText:
 			evType = "text"
+		case executor.EventThinking:
+			evType = "thinking"
 		case executor.EventCompleted:
 			evType = "complete"
 		case executor.EventFailed:

internal/mesh/tools.go

@@ -190,27 +190,31 @@ func (t *SendMessageTool) Definition() provider.ToolDef {
 					"type":        "string",
 					"description": "Target agent name (e.g., 'coder'), node ID, or '*' for broadcast",
 				},
-				"type": map[string]any{
+				"message_type": map[string]any{
 					"type":        "string",
 					"description": "Message type: task, result, feedback, or request",
+					"default":     "task",
 				},
 				"content": map[string]any{
 					"type":        "string",
 					"description": "Message body",
 				},
 			},
-			"required": []string{"to", "type", "content"},
+			"required": []string{"to", "content"},
 		},
 	}
 }
 
 func (t *SendMessageTool) Execute(_ context.Context, args map[string]any) (any, error) {
 	to, _ := args["to"].(string)
-	msgType, _ := args["type"].(string)
+	msgType, _ := args["message_type"].(string)
 	content, _ := args["content"].(string)
 
-	if to == "" || msgType == "" {
-		return nil, fmt.Errorf("to and type are required")
+	if to == "" {
+		return nil, fmt.Errorf("'to' is required")
+	}
+	if msgType == "" {
+		msgType = "task" // default when omitted
 	}
 
 	msg := Message{