fix(agent): resolve oauth cache home cross-platform - #5151
Merged
kalvinnchau merged 1 commit intoAug 7, 2026
Conversation
Co-authored-by: Kalvin C <kalvinnchau@users.noreply.github.com> Signed-off-by: Kalvin C <kalvinnchau@users.noreply.github.com>
kalvinnchau
force-pushed
the
windows-xp/fix-windows-oauth-home
branch
from
August 7, 2026 05:22
836d820 to
b3b1528
Compare
wpfleger96
approved these changes
Aug 7, 2026
41fred
pushed a commit
to 41fred/buzz
that referenced
this pull request
Aug 9, 2026
## Summary - resolve the OAuth cache home directory with the repository's platform-aware `dirs` convention - preserve the existing `.config/buzz-agent/oauth` cache layout on macOS and Linux - make the cache-path regression assertion portable across path separators ## Problem `buzz-agent` read only `$HOME` when constructing the OAuth token cache path. Packaged Windows processes do not guarantee that Unix variable, so OAuth source construction failed with `oauth cache: $HOME not set` even though Windows had a valid user profile. ## Validation Independent reviewers validated exact commit `836d820483b141b7291170cb33535ac7cb49b2eb` on Windows/MSVC with `HOME` unset and `USERPROFILE` present: - `cargo +1.94.1 clippy -p buzz-agent --all-targets --locked -- -D warnings` - `cargo +1.94.1 fmt --all -- --check` - `git diff --check f53bbd1..836d820` - `auth::` tests: 11/11 passed with `HOME` unset - full package lib target: 396 passed / 2 failed; identical-base controls classified both as pre-existing Windows failures The changed regression fails on the base with `$HOME not set` and passes on this branch. `dirs 6.0.0` was already locked by other workspace crates; the lockfile change adds only the `buzz-agent` dependency edge. Signed-off-by: Kalvin C <kalvinnchau@users.noreply.github.com>
wpfleger96
pushed a commit
that referenced
this pull request
Aug 10, 2026
…gaps * origin/main: (26 commits) fix(sdk): preserve self-mention p tags in message and forum event builders (#4975) bump @tauri-apps/cli to ~2.11.4 to fix linux app icon issue (#4858) feat(desktop): adding rich link previews to messages (#3818) fix(buzz-agent): Responses reasoning summary, Anthropic display:summarized, ACP v2 messageId (#5195) fix(desktop): retain distinct agent instances in autocomplete (#5202) fix(desktop): defer channel visibility change to Save (#5203) feat(desktop): Projects follow-ups — access restrictions, fast loading, activity feed polish (#5073) refactor(cli): replace probe/decider/detail split with single typed extractor (#5191) fix(desktop): drop unhandled rejection from throwing window.Notification (#5143) fix(desktop): fence localStorage SecurityError from killing the React tree (#5142) fix(desktop): make terminal output selectable (#4980) fix(desktop): use WEBKIT_DMABUF_RENDERER_FORCE_SHM for NVIDIA/AppImage (#3654) (#4505) Make public starter channels best effort (#5192) Mobile: add anchored reaction popover (#5025) feat(mobile): add bee pull-to-refresh (#5059) Remove agent creation success modal (#5063) fix(buzz-agent): escalate LLM timeouts per retry and log per-call latency (#5130) fix(agent): resolve oauth cache home cross-platform (#5151) Improve video review readiness and controls (#5161) Polish advanced agent setup and Welcome composer (#4926) ... Signed-off-by: Hayt <9e1c23a3fd83f61da34420e4e88ff1b16e45cafcc0cd9019eb07d4ecfa8ca9b0@buzz.block.builderlab.xyz>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
dirsconvention.config/buzz-agent/oauthcache layout on macOS and LinuxProblem
buzz-agentread only$HOMEwhen constructing the OAuth token cache path. Packaged Windows processes do not guarantee that Unix variable, so OAuth source construction failed withoauth cache: $HOME not seteven though Windows had a valid user profile.Validation
Independent reviewers validated exact commit
836d820483b141b7291170cb33535ac7cb49b2ebon Windows/MSVC withHOMEunset andUSERPROFILEpresent:cargo +1.94.1 clippy -p buzz-agent --all-targets --locked -- -D warningscargo +1.94.1 fmt --all -- --checkgit diff --check f53bbd1..836d820auth::tests: 11/11 passed withHOMEunsetThe changed regression fails on the base with
$HOME not setand passes on this branch.dirs 6.0.0was already locked by other workspace crates; the lockfile change adds only thebuzz-agentdependency edge.