Merge pull request #4373 from pragneshbagary/glob_issue

Fix #4173: glob returns an error instead of an empty list

Author: AkihiroSuda

hack/bats/tests/mcp.bats

@@ -296,8 +296,7 @@ tools_call() {
 }
 
 @test 'glob returns an empty list when the pattern does not match' {
-    skip "see https://github.com/lima-vm/lima/issues/4173"
-
+   
     tools_call glob '{"pattern":"nothing.to.see"}'
 
     run_yq '.structuredContent.matches[]' <<<"$output"

pkg/mcp/toolset/filesystem.go

@@ -130,6 +130,9 @@ func (ts *ToolSet) Glob(_ context.Context,
 	}
 	pattern := path.Join(guestPath, args.Pattern)
 	matches, err := ts.sftp.Glob(pattern)
+	if matches == nil {
+		matches = []string{}
+	}
 	if err != nil {
 		return nil, nil, err
 	}