Skip to content

Deduplicate the Windows Git runtime#659

Merged
luokerenx4 merged 2 commits into
devfrom
codex/windows-dugite-wrapper
Jul 17, 2026
Merged

Deduplicate the Windows Git runtime#659
luokerenx4 merged 2 commits into
devfrom
codex/windows-dugite-wrapper

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

What changed

  • keep dugite's JavaScript wrapper in Windows desktop packages while excluding its embedded Git payload
  • route packaged dugite calls through the pinned PortableGit runtime already shipped for git.exe and bash.exe
  • assert that Windows packages contain the wrapper and PortableGit but no second Git tree
  • exercise a real init / add / commit / status cycle through the packaged wrapper
  • preserve the existing macOS dugite Git path and document the platform boundary

Why

The Windows installer was shipping PortableGit and dugite's embedded Git simultaneously. PortableGit already supplies both the managed Git executable and the Bash runtime required by Windows Workspaces, so the second binary payload was redundant.

Validation

  • npx tsc --noEmit
  • pnpm test — 314 files passed, 3162 tests passed
  • focused desktop package tests — 11 tests passed
  • generated Windows dugite probe syntax check
  • pnpm electron:smoke:workspace — packaged macOS Workspace acceptance passed

@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 17, 2026 3:50am

Request Review

@luokerenx4
luokerenx4 marked this pull request as ready for review July 17, 2026 03:38
@luokerenx4
luokerenx4 merged commit 3252d01 into dev Jul 17, 2026
16 checks passed
@luokerenx4
luokerenx4 deleted the codex/windows-dugite-wrapper branch July 17, 2026 03:55
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