huggingface.js icon indicating copy to clipboard operation
huggingface.js copied to clipboard

fix(mcp): use a more generic folder to put mcp filesystem root

Open mfuntowicz opened this issue 7 months ago • 3 comments

Currently the MCP server attempts to access a folder under ${HOMEDIR}/Desktop which is not portable accross different OSes.

This PR proposes to change the Desktop part to .mcp which is OS agnostic and ensure the folder exists when spawning.

mfuntowicz avatar Apr 30 '25 20:04 mfuntowicz

i can finish your PR if you want @mfuntowicz? let me know

julien-c avatar May 02 '25 08:05 julien-c

I've put ~/.mcp to have something which can be easily implemented without branching on plateform. @julien-c if we are fine with the branching, I totally think we can do something closer to the original for all platform.

Happy to take a snab at it 👍🏻

mfuntowicz avatar May 02 '25 10:05 mfuntowicz

yes, branching depending on OS is completely fine

julien-c avatar May 02 '25 11:05 julien-c