{"record":{"id":"cf839e4793ef577f","repo":"googleapis/mcp-toolbox","slug":"bucket-cannot-be-empty-for-tool-q-cf839e","errorCode":null,"errorMessage":"bucket cannot be empty for tool %q","messagePattern":"bucket cannot be empty for tool %q","errorType":"validation","errorClass":null,"httpStatus":null,"severity":"error","filePath":"internal/tools/cloudstorage/cloudstoragegetbucketmetadata/cloudstoragegetbucketmetadata.go","lineNumber":72,"sourceCode":"\ttools.ConfigBase `yaml:\",inline\"`\n\tType             string                 `yaml:\"type\" validate:\"required\"`\n\tSource           string                 `yaml:\"source\" validate:\"required\"`\n\tAnnotations      *tools.ToolAnnotations `yaml:\"annotations,omitempty\"`\n\tBucket           *string                `yaml:\"bucket,omitempty\"`\n}\n\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\tif cfg.Bucket != nil && *cfg.Bucket == \"\" {\n\t\treturn nil, fmt.Errorf(\"bucket cannot be empty for tool %q\", cfg.Name)\n\t}\n\n\tallParameters := parameters.Parameters{}\n\tif cfg.Bucket == nil {\n\t\tallParameters = append(allParameters, parameters.NewStringParameter(bucketKey, \"Name of the Cloud Storage bucket to inspect.\"))\n\t}\n\n\treturn Tool{\n\t\tBaseTool: tools.NewBaseTool(\n\t\t\tcfg,\n\t\t\ttools.GetAnnotationsOrDefault(cfg.Annotations, tools.NewReadOnlyAnnotations),\n\t\t\ttools.Manifest{Description: cfg.Description, Parameters: allParameters.Manifest(), AuthRequired: cfg.AuthRequired},\n\t\t\tallParameters,\n\t\t),\n\t}, nil\n}\n\nvar _ tools.Tool = Tool{}","sourceCodeStart":54,"sourceCodeEnd":90,"githubUrl":"https://github.com/googleapis/mcp-toolbox/blob/8cc6e09de2ad7b8bffc77751799585a1401a48eb/internal/tools/cloudstorage/cloudstoragegetbucketmetadata/cloudstoragegetbucketmetadata.go#L54-L90","documentation":"Same validation as the get-bucket-iam-policy tool: cloudstorage-get-bucket-metadata's Initialize rejects a bucket field that is present (non-nil pointer) but an empty string. Either supply a real fixed bucket or omit the field so it becomes a tool parameter.","triggerScenarios":"cfg.Bucket != nil and *cfg.Bucket == \"\" at Initialize — YAML `bucket: \"\"` or programmatic tools.Ptr(\"\").","commonSituations":"Empty env-var substitution into the bucket field; YAML key present with no value; generated configs with unset placeholders.","solutions":["Set a concrete bucket name, e.g. `bucket: my-bucket`.","Remove the `bucket` key so the bucket becomes a required invocation-time parameter.","Pre-validate config strings for emptiness after environment expansion."],"exampleFix":"# before\nbucket: \"\"\n# after\nbucket: my-gcs-bucket","handlingStrategy":"validation","validationCode":"if cfg.Bucket != nil && strings.TrimSpace(*cfg.Bucket) == \"\" {\n    return errors.New(\"bucket cannot be empty; set a name or omit the field\")\n}","typeGuard":"func bucketProvided(cfg Config) bool { return cfg.Bucket != nil && *cfg.Bucket != \"\" }","tryCatchPattern":null,"preventionTips":["Prefer omitting `bucket` over `bucket: \"\"`","Validate substituted template values before server start","Unit-test tool Initialize with representative configs"],"tags":["go","cloudstorage","config-validation","tool-initialization"],"backgroundTag":"tool-config-validation","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"}