Skip to content

rebase transport: preserve #79250 Windows warning fix - #99

Merged
andrexibiza merged 1 commit into
tmp/rebase-79250-current-20260821from
tmp/79250-original
Aug 21, 2026
Merged

rebase transport: preserve #79250 Windows warning fix#99
andrexibiza merged 1 commit into
tmp/rebase-79250-current-20260821from
tmp/79250-original

Conversation

@andrexibiza

Copy link
Copy Markdown
Owner

Temporary proof-preserving rebase transport for upstream NousResearch#79250. Replays @hernanda-git's original commit onto exact upstream main at ac8dff4fbcf47a392a3cddcbec068aa05930ab47; not a public delivery artifact.

gitFor() opts into simple-git's unsafe.allowUnsafeCustomBinary when the
resolved git binary path contains a space (the default Windows install,
C:\Program Files\Git\cmd\git.exe). That opt-in trades simple-git's throw
for a console.warn emitted on EVERY client construction, and gitFor() is
not memoized — the coding rail rebuilds a client on each status poll, so
the desktop log filled with hundreds of identical lines that buried real
errors.

Suppress only that known warning, only for the duration of the
construction call, and restore console.warn in a finally so unrelated
warnings still surface.

Fixes NousResearch#79245
@andrexibiza
andrexibiza merged commit e90dc2a into tmp/rebase-79250-current-20260821 Aug 21, 2026
@andrexibiza
andrexibiza deleted the tmp/79250-original branch August 26, 2026 19:28
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