Fix optional query param file_id to use *string pointer

Anthrino · gemini-code-assist[bot] · web-flow · commit cfc91314e086 · 2026-02-19T23:25:19.000+05:30
Co-authored-by: gemini-code-assist[bot] &lt;176961590+gemini-code-assist[bot]@users.noreply.github.com&gt;
diff --git a/internal/tools/looker/lookergetlookmltests/lookergetlookmltests.go b/internal/tools/looker/lookergetlookmltests/lookergetlookmltests.go
@@ -125,9 +125,9 @@ func (t Tool) Invoke(ctx context.Context, resourceMgr tools.SourceProvider, para
 		return nil, fmt.Errorf("'project_id' must be a string, got %T", mapParams["project_id"])
 	}
 
-	var fileId string
-	if val, ok := mapParams["file_id"].(string); ok {
-		fileId = val
+	var fileId *string
+	if val, ok := mapParams["file_id"].(string); ok && val != "" {
+		fileId = &val
 	}
 
 	resp, err := sdk.AllLookmlTests(projectId, fileId, source.LookerApiSettings())

Original file line number	Diff line number	Diff line change
`@@ -125,9 +125,9 @@ func (t Tool) Invoke(ctx context.Context, resourceMgr tools.SourceProvider, para`
`125`	`125`	`return nil, fmt.Errorf("'project_id' must be a string, got %T", mapParams["project_id"])`
`126`	`126`	`}`
`127`	`127`
`128`		`- var fileId string`
`129`		`- if val, ok := mapParams["file_id"].(string); ok {`
`130`		`- fileId = val`
	`128`	`+ var fileId *string`
	`129`	`+ if val, ok := mapParams["file_id"].(string); ok && val != "" {`
	`130`	`+ fileId = &val`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`resp, err := sdk.AllLookmlTests(projectId, fileId, source.LookerApiSettings())`