Skip to content

windows fixes#1

Merged
jdx merged 6 commits intomainfrom
convert-to-vfox-plugin
Aug 11, 2025
Merged

windows fixes#1
jdx merged 6 commits intomainfrom
convert-to-vfox-plugin

Conversation

@jdx
Copy link
Member

@jdx jdx commented Aug 11, 2025

No description provided.

jdx and others added 6 commits August 11, 2025 15:30
- Rewrite available.lua to use pure Lua parsing instead of shell pipes
- Remove dependency on sed, grep, and sort commands that fail on Windows
- Add mise ls-remote validation step to GitHub Actions to verify versions are found
- Implement proper version sorting in Lua for cross-platform compatibility

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

Co-Authored-By: Claude <noreply@anthropic.com>
- Use NUL instead of /dev/null for stderr redirection on Windows
- Detect Windows platform using package.config
- This fixes "The system cannot find the path specified" error on Windows

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

Co-Authored-By: Claude <noreply@anthropic.com>
- Fix stderr redirection to use NUL on Windows instead of /dev/null
- Use platform-specific mkdir commands (mkdir vs mkdir -p)
- Skip chmod on Windows (not needed for JS files)
- Skip GPG check on Windows entirely
- Prefer curl over wget for better Windows Git Bash compatibility

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

Co-Authored-By: Claude <noreply@anthropic.com>
- Change author from vfox-yarn contributors to mise-plugins
- Update homepage to mise-plugins/mise-yarn repository
- Update description to reference mise instead of vfox

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

Co-Authored-By: Claude <noreply@anthropic.com>
- Download yarn.js instead of yarn for v2+
- Create yarn.cmd wrapper for Windows CMD
- Create shell wrapper script for Unix and Git Bash
- Wrapper scripts properly invoke Node.js to run yarn.js

This fixes the issue where Yarn v2+ wouldn't run on Windows.

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

Co-Authored-By: Claude <noreply@anthropic.com>
@jdx jdx merged commit 7952317 into main Aug 11, 2025
6 checks passed
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.

1 participant

Comments