fix(build): convert MSYS paths for Bun in the Windows server-bundle build (#2452)

browse/scripts/build-node-server.sh resolves GSTACK_DIR with pwd, which
under MSYS/Git Bash yields a /c/... style absolute path that Bun cannot
open ('FileNotFound opening root directory') — the Windows Node-server
bundle build died at the first bun build. Convert via cygpath -m on
MINGW/MSYS/CYGWIN before deriving SRC_DIR/DIST_DIR.

Re-derived from PR #2452, taking only the cygpath build half — the PR's
icacls principal-ambiguity half already landed on main
(browse/src/file-permissions.ts's SID-form principal). Verified the build
bug still exists on current code before absorbing (build-node-server.sh:10
had no conversion). Contributed by @chiragborse1.

Co-authored-by: chiragborse1 <chiragborse1@users.noreply.github.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Garry Tan
2026-08-16 10:59:36 -07:00
co-authored by chiragborse1 Claude Fable 5
parent ae47c29b82
commit 504409d75b
+5
View File
@@ -8,6 +8,11 @@
set -e
GSTACK_DIR="$(cd "$(dirname "$0")/../.." && pwd)"
# Windows (MSYS/Git Bash): convert to a Windows-style path — Bun cannot open
# MSYS /c/... absolute paths ("FileNotFound opening root directory").
case "$(uname -s)" in
MINGW*|MSYS*|CYGWIN*) GSTACK_DIR="$(cygpath -m "$GSTACK_DIR")" ;;
esac
SRC_DIR="$GSTACK_DIR/browse/src"
DIST_DIR="$GSTACK_DIR/browse/dist"
mkdir -p "$DIST_DIR"