review comments

Author: ioanarm

tools/sift.go

@@ -233,11 +233,10 @@ var ListSiftInvestigations = mcpgrafana.MustTool(
 
 // FindErrorPatternLogsParams defines the parameters for running an ErrorPatternLogs check
 type FindErrorPatternLogsParams struct {
-	Name     string            `json:"name" jsonschema:"required,description=The name of the investigation"`
-	Labels   map[string]string `json:"labels" jsonschema:"required,description=Labels to scope the analysis"`
-	Start    time.Time         `json:"start,omitempty" jsonschema:"description=Start time for the investigation. Defaults to 30 minutes ago if not specified."`
-	End      time.Time         `json:"end,omitempty" jsonschema:"description=End time for the investigation. Defaults to now if not specified."`
-	QueryURL string            `json:"queryUrl,omitempty" jsonschema:"description=Optional query URL for the investigation"`
+	Name   string            `json:"name" jsonschema:"required,description=The name of the investigation"`
+	Labels map[string]string `json:"labels" jsonschema:"required,description=Labels to scope the analysis"`
+	Start  time.Time         `json:"start,omitempty" jsonschema:"description=Start time for the investigation. Defaults to 30 minutes ago if not specified."`
+	End    time.Time         `json:"end,omitempty" jsonschema:"description=End time for the investigation. Defaults to now if not specified."`
 }
 
 // findErrorPatternLogs creates an investigation with ErrorPatternLogs check, waits for it to complete, and returns the analysis
@@ -299,11 +298,10 @@ var FindErrorPatternLogs = mcpgrafana.MustTool(
 
 // FindSlowRequestsParams defines the parameters for running an SlowRequests check
 type FindSlowRequestsParams struct {
-	Name     string            `json:"name" jsonschema:"required,description=The name of the investigation"`
-	Labels   map[string]string `json:"labels" jsonschema:"required,description=Labels to scope the analysis"`
-	Start    time.Time         `json:"start,omitempty" jsonschema:"description=Start time for the investigation. Defaults to 30 minutes ago if not specified."`
-	End      time.Time         `json:"end,omitempty" jsonschema:"description=End time for the investigation. Defaults to now if not specified."`
-	QueryURL string            `json:"queryUrl,omitempty" jsonschema:"description=Optional query URL for the investigation"`
+	Name   string            `json:"name" jsonschema:"required,description=The name of the investigation"`
+	Labels map[string]string `json:"labels" jsonschema:"required,description=Labels to scope the analysis"`
+	Start  time.Time         `json:"start,omitempty" jsonschema:"description=Start time for the investigation. Defaults to 30 minutes ago if not specified."`
+	End    time.Time         `json:"end,omitempty" jsonschema:"description=End time for the investigation. Defaults to now if not specified."`
 }
 
 // findSlowRequests creates an investigation with SlowRequests check, waits for it to complete, and returns the analysis