fix(vector-code-nvim): install or upgrade vectorcode-cli based on cli existence#1713
fix(vector-code-nvim): install or upgrade vectorcode-cli based on cli existence#1713cristobalgvera wants to merge 1 commit into
vectorcode-cli based on cli existence#1713Conversation
Review ChecklistDoes this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist: Proper conventional commit scoping:
|
ac7d3f0 to
0303e45
Compare
Uzaaft
left a comment
There was a problem hiding this comment.
Sorry for the delay in reviewing this.
| if vim.fn.executable "uv" == 0 then error "The VectorCode pack requires `uv` installed" end | ||
|
|
||
| local action = vim.fn.executable "vectorcode" == 0 and "install" or "upgrade" | ||
| vim.system { "uv", "tool", action, "vectorcode" } |
There was a problem hiding this comment.
we should support uv and pipx.
If uv is installed, use uv no matter what. fallback to pipx, and none of them are there, notify the end-user. Also update the docs to reflect this.
📑 Description
This pull request updates the build logic for the
VectorCodeplugin to improve how dependencies are checked and managed. The main change is that it now conditionally installs or upgrades thevectorcodetool based on its presence, and usesvim.systemfor better process handling.Dependency management improvements:
uvis installed by verifying thatvim.fn.executable "uv"returns 0, and throws an error if not.vectorcodeby checking if it is already executable, and then callsuv tool install vectorcodeoruv tool upgrade vectorcodeaccordingly usingvim.systemfor process execution.