Skip to content

windows fixes#374

Merged
thdxr merged 4 commits into
devfrom
windows-fixes
Jun 24, 2025
Merged

windows fixes#374
thdxr merged 4 commits into
devfrom
windows-fixes

Conversation

@thdxr
Copy link
Copy Markdown
Member

@thdxr thdxr commented Jun 24, 2025

  • fix(scripts): add Git hook for Windows
  • fix: don't allow ':' in log file and project dir
  • fix: parse file:// URLs correctly on Windows
  • fix(tui): don't use path/filepath for embed

strager added 4 commits June 24, 2025 17:59
Create an equivalent of scripts/hooks for non-WSL Windows. This fixes
errors in 'bun install'.
Windows does not allow ':' in file or directory names. Make sure the log
file and the project directory do not contain ':' to prevent ENOENT
errors.
On Windows, URL.pathname returns a path with an extra leading '/' (e.g.
'\C:\Users\whatever'). This causes problems when using Bun.spawn(). Fix
these problems by using the right method instead of URL.pathname.
EmbedFS uses '/' always for file paths. It does not support '\' even on
Windows. Make sure we don't give EmbedFS a '\' path on Windows. This
lets the TUI find the theme files, avoiding crashes in various parts of
the code that assume a theme exists.
@thdxr thdxr merged commit ec001ca into dev Jun 24, 2025
moutansos pushed a commit to moutansos/opencode that referenced this pull request Jun 25, 2025
Co-authored-by: Matthew Glazar <strager.nds@gmail.com>
achembarpu pushed a commit to achembarpu/opencode that referenced this pull request Aug 4, 2025
Co-authored-by: Matthew Glazar <strager.nds@gmail.com>
@thdxr thdxr deleted the windows-fixes branch February 1, 2026 03:30
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Co-authored-by: Matthew Glazar <strager.nds@gmail.com>
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.

2 participants