Skip to content

Conversation

AkihiroSuda
Copy link
Member

Fix #4070
Fix #4080

Fix issue 4070
Fix issue 4080

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@jandubois jandubois merged commit f47b60e into lima-vm:master Oct 2, 2025
168 of 177 checks passed
@jandubois
Copy link
Member

mcp: use StructuredContent; fix "path is empty" error

structuredContent seems fine.

I don't know what the "path is empty" error is. Can you drop a link, so I can add a BATS test for it?

@AkihiroSuda
Copy link
Member Author

I don't know what the "path is empty" error is.

SearchFileContent was calling RunShellCommand with an empty Directory:

return ts.RunShellCommand(ctx, req, msi.RunShellCommandParams{

guestPath, err := ts.TranslateHostPath(args.Directory)

return "", errors.New("path is empty")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

limactl-mcp info has no outputSchema MCP: switch from JSON TextContent to StructuredContent
2 participants