Fix Antigravity support#32
Conversation
Update the default install path for Antigravity from: ~/.gemini/antigravity/global_skills to: ~/.gemini/antigravity/skills This aligns with the official Antigravity documentation: https://antigravity.google/docs/skills
Some AI tools don't support symbolic links for skills. This adds a --copy flag that creates full copies instead of symlinks. Changes: - Add --copy/-c flag to install command - Add copyItem to FileSystem protocol - Implement copyItem in InMemoryFileSystem for tests - Add test for copy installation - Document when to use --copy in README Usage: pfw install --tool <tool> --copy Note: Using --copy duplicates skills for each tool and requires re-running install after updates.
The copyItem method had code handling files and symlinks as source, but in practice we always copy directories from ~/.pfw/skills/. Removed unreachable code paths for clarity.
41a956b to
e4b20a1
Compare
The source directory (~/.pfw/skills/) never contains symlinks, so there's no need to copy them.
|
should be rebased on top #31 after merging it |
mbrandonw
left a comment
There was a problem hiding this comment.
Hey @ibrahimkteish, thanks for doing this! We decided to tone it down a bit by just special casing cursor for now to do the copying rather than have a --copy flag.
|
Hey @mbrandonw, whatever you see fit, but there is also a report on Slack that symlinks don't work with Antigravity. But also, I understand if you want to just keep cursor for now. |
|
@ibrahimkteish We're down to maintain a whitelist for those that don't work, and maybe link to any open issues. Do you know if there's an Antigravity issue we can track? |
|
@stephencelis I am not aware of any link for the antigravity issue. It was reported in the Slack channel only by one of the users of the tool |
|
From : 2.5 got released and for me symlinks seem to now work. So the Cursor whitelist might not be needed anymore. |
Adds a
--copyflag to create full copies instead of symlinks for AI tools that don't support symbolic links.Usage:
Changes:
--copy/-cflag to install commandcopyItemto FileSystem protocolAll 58 tests pass.