{"record":{"id":"2c18ee68f4eb1902","repo":"googleapis/mcp-toolbox","slug":"description-is-required-for-tool-q-2c18ee","errorCode":null,"errorMessage":"description is required for tool %q","messagePattern":"description is required for tool %q","errorType":"validation","errorClass":null,"httpStatus":null,"severity":"error","filePath":"internal/tools/trino/trinosql/trinosql.go","lineNumber":70,"sourceCode":"\ttools.ConfigBase   `yaml:\",inline\"`\n\tType               string                 `yaml:\"type\" validate:\"required\"`\n\tSource             string                 `yaml:\"source\" validate:\"required\"`\n\tStatement          string                 `yaml:\"statement\" validate:\"required\"`\n\tParameters         parameters.Parameters  `yaml:\"parameters\"`\n\tTemplateParameters parameters.Parameters  `yaml:\"templateParameters\"`\n\tAnnotations        *tools.ToolAnnotations `yaml:\"annotations,omitempty\"`\n}\n\n// validate interface\nvar _ tools.ToolConfig = Config{}\n\nfunc (cfg Config) ToolConfigType() string {\n\treturn resourceType\n}\n\nfunc (cfg Config) Initialize(context.Context) (tools.Tool, error) {\n\tif cfg.Description == \"\" {\n\t\treturn nil, fmt.Errorf(\"description is required for tool %q\", cfg.Name)\n\t}\n\n\tallParameters, paramManifest, err := parameters.ProcessParameters(cfg.TemplateParameters, cfg.Parameters)\n\tif err != nil {\n\t\treturn nil, fmt.Errorf(\"unable to process parameters: %w\", err)\n\t}\n\n\treturn Tool{\n\t\tBaseTool: tools.NewBaseTool(\n\t\t\tcfg,\n\t\t\ttools.GetAnnotationsOrDefault(cfg.Annotations, tools.NewDestructiveAnnotations),\n\t\t\ttools.Manifest{Description: cfg.Description, Parameters: paramManifest, AuthRequired: cfg.AuthRequired},\n\t\t\tallParameters,\n\t\t),\n\t}, nil\n}\n\n// validate interface","sourceCodeStart":52,"sourceCodeEnd":88,"githubUrl":"https://github.com/googleapis/mcp-toolbox/blob/8cc6e09de2ad7b8bffc77751799585a1401a48eb/internal/tools/trino/trinosql/trinosql.go#L52-L88","documentation":"The trino-sql tool requires a non-empty `description` in its config; Config.Initialize returns this error when cfg.Description is empty. Like all tools, the description is published in the manifest for LLM clients.","triggerScenarios":"Defining a trino-sql tool in the YAML (or building its Config directly) without a `description` field or with an empty string, then initializing it.","commonSituations":"Omitting description in tools.yaml; empty string value; YAML nesting mistake placing description outside the tool block; templated configs missing the field.","solutions":["Add a `description` field to the trino-sql tool config with a meaningful non-empty string","Fix YAML indentation so `description` belongs to the tool entry","Replace any empty-string description with real text"],"exampleFix":"// before (tools.yaml)\ntrino_query:\n  kind: trino-sql\n  source: my-trino\n// after\ntrino_query:\n  kind: trino-sql\n  source: my-trino\n  description: Executes a parameterized SQL statement against Trino.","handlingStrategy":"validation","validationCode":"if cfg[\"description\"] == \"\" || cfg[\"description\"] == nil {\n    return errors.New(\"trino-sql tool: description is required\")\n}","typeGuard":null,"tryCatchPattern":"tool, err := cfg.Initialize(ctx)\nif err != nil {\n    if strings.Contains(err.Error(), \"description is required\") {\n        return fmt.Errorf(\"add description for tool %q\", cfg.Name)\n    }\n    return err\n}","preventionTips":["Include a description for every tool defined in tools.yaml","Review configs for empty-string fields before merge","Use a shared config template that includes description"],"tags":["go","validation","trino","config"],"backgroundTag":"missing-required-argument","analyzedSha":"8cc6e09de2ad7b8bffc77751799585a1401a48eb","analyzedAt":"2026-09-05T01:10:36.887Z","contentChangedAt":"2026-09-05T01:10:36.887Z","schemaVersion":2},"datasetVersion":"2026-09-08T10:18:20.063Z"}