fix: updated wizard to support Windows environments#11
fix: updated wizard to support Windows environments#11gitoff wants to merge 1 commit intoSteffen025:mainfrom
Conversation
|
Problem: The current installation wizard fails on Windows systems with a "The system cannot find the path specified" error during the Bun runtime check. This is caused by: Unix-style shell redirects (2>/dev/null) in the execSync call. Hardcoded Unix commands (chmod, chown, find) in the fixPermissions function. Changes: Updated checkBun() to use platform-agnostic stdio configuration, allowing it to detect Bun correctly on Windows CMD/PowerShell. Added a platform check to fixPermissions() to skip Unix-specific permission logic when running on Windows (where these steps are typically unnecessary for Bun/OpenCode execution). Added a regex fix for SCRIPT_DIR to handle Windows drive letters (e.g., D:) correctly when parsed from import.meta.url. Testing: Verified on Windows 11 using Bun v1.3.8. The wizard now successfully passes the Bun check and completes the configuration file generation. |
Steffen025
left a comment
There was a problem hiding this comment.
Thanks for the Windows compatibility work, @gitoff! The core fixes are exactly what we need:
✅ Good fixes we want to keep:
SCRIPT_DIRregex for Windows drive lettersstdio: ['ignore', 'pipe', 'ignore']for platform-agnostic Bun check- Platform check in
fixPermissions()to skip chmod/chown on Windows - Windows-specific Bun install command
❌ However, this PR removes too much functionality:
| Removed | Impact |
|---|---|
| Groq, AWS Bedrock, Azure providers | Users lose 3 provider options |
validate() function |
No installation verification |
| Deep Personalization instructions | Users miss TELOS onboarding |
| Full DAIDENTITY.md template | 60 lines → 3 lines |
| Full BASICINFO.md template | 15 lines → 2 lines |
| Voice settings in settings.json | Voice config lost |
Model claude-sonnet-4-5 |
Downgraded to older model |
Request: Could you submit a minimal PR with only the Windows-specific fixes? Something like:
- The
SCRIPT_DIRregex fix - The
execSyncstdio fix forcheckBun() - The platform check in
fixPermissions() - The Windows Bun install message
Keep everything else unchanged. That way we get Windows support without losing features.
I'm happy to help extract just the Windows fixes if you'd like!
Fix: Cross-platform Windows support for Installation Wizard