Simplify codebase per audit of 8b89f14

internal/agents/summary.go

@@ -37,17 +37,12 @@ func (m *Manager) setTaskSummaryTool(l *live) mcp.Tool {
 			},
 			"required": ["summary"]
 		}`),
-		Handler: func(ctx context.Context, raw json.RawMessage) (string, error) {
-			args := setTaskSummaryArgs{}
-			err := json.Unmarshal(raw, &args)
-			if err != nil {
-				return "", fmt.Errorf("malformed set_task_summary call: %w", err)
-			}
+		Handler: mcp.Typed(func(ctx context.Context, args setTaskSummaryArgs) (string, error) {
 			summary := strings.TrimSpace(args.Summary)
 			if summary == "" {
 				return "", fmt.Errorf("%w: summary must not be empty", ErrInvalid)
 			}
-			err = m.store.SetTaskSummary(ctx, l.taskID, summary, agentActor)
+			err := m.store.SetTaskSummary(ctx, l.taskID, summary, agentActor)
 			if errors.Is(err, store.ErrNotFound) {
 				return "", fmt.Errorf("%w: this task is closed", ErrConflict)
 			}
@@ -55,7 +50,7 @@ func (m *Manager) setTaskSummaryTool(l *live) mcp.Tool {
 				return "", err
 			}
 			return "Task summary saved.", nil
-		},
+		}),
 	}
 }
 
@@ -78,14 +73,9 @@ func (m *Manager) searchClosedTool(l *live) mcp.Tool {
 		Name:        "search_closed_tasks",
 		Description: searchClosedDescription + " Searches this task's project.",
 		InputSchema: json.RawMessage(`{"type": "object", "properties": {` + searchClosedProperties + `}}`),
-		Handler: func(ctx context.Context, raw json.RawMessage) (string, error) {
-			args := searchClosedArgs{}
-			err := json.Unmarshal(raw, &args)
-			if err != nil {
-				return "", fmt.Errorf("malformed search_closed_tasks call: %w", err)
-			}
+		Handler: mcp.Typed(func(ctx context.Context, args searchClosedArgs) (string, error) {
 			return m.searchClosedText(ctx, l.projectID, args.Query, args.Limit)
-		},
+		}),
 	}
 }