Skip to content

feat: Add path command to show absolute path of installed SDKs#2

Closed
yeshan333 wants to merge 1 commit intomainfrom
claude/issue-1-20250727-1537
Closed

feat: Add path command to show absolute path of installed SDKs#2
yeshan333 wants to merge 1 commit intomainfrom
claude/issue-1-20250727-1537

Conversation

@yeshan333
Copy link
Copy Markdown
Member

This PR adds a new path command that returns the absolute path of an installed SDK.
If the SDK is not installed, it returns notfound.

Example usage:
vfox path nodejs@24

Returns: /home/user/.version-fox/cache/nodejs/v-24.4.1/nodejs-24.4.1 (if installed)

Returns: notfound (if not installed)

Fixes version-fox#497

Generated with Claude Code

This commit adds a new 'path' command that returns the absolute path of an installed SDK.
If the SDK is not installed, it returns 'notfound'.

Example usage:
  vfox path nodejs@24
  # Returns: /home/user/.version-fox/cache/nodejs/v-24.4.1/nodejs-24.4.1 (if installed)
  # Returns: notfound (if not installed)

Fixes version-fox#497

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: yeshan333 <yeshan333@users.noreply.github.com>
@yeshan333 yeshan333 marked this pull request as draft July 27, 2025 15:47
@yeshan333
Copy link
Copy Markdown
Member Author

@claude Tests should cover both normal and exceptional paths. Please install a plugin in practice to test the completeness of the implementation, such as

First, run vfox add python, then run vfox install python@3.13.5. After installing the version, vfox path python@3.13.5 can normally find the path.

But vfox path python@3.13.4 cannot be found.

@claude
Copy link
Copy Markdown

claude bot commented Jul 27, 2025

Claude encountered an error —— View job


Todo List

  • Analyze the path command implementation
  • Read existing test file to understand current coverage
  • Add tests for normal path (SDK installed)
  • Add tests for exceptional path (SDK not installed)
  • Add specific test case for python@3.13.5 vs python@3.13.4 scenario
  • Verify all tests pass
  • Commit and push changes

Adding comprehensive tests for the path command to cover both normal and exceptional paths...

@yeshan333 yeshan333 closed this Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: 增加命令可以返回sdk对应的绝对路径

1 participant