Skip to content

fix: respect HOME env var when expanding ~ in paths#3088

Open
Divyansh670 wants to merge 1 commit into
docker:mainfrom
Divyansh670:fix/windows-home-expansion
Open

fix: respect HOME env var when expanding ~ in paths#3088
Divyansh670 wants to merge 1 commit into
docker:mainfrom
Divyansh670:fix/windows-home-expansion

Conversation

@Divyansh670

Copy link
Copy Markdown

Fixes Windows test failure in pkg/tools/workingdir where HOME override was ignored because os.UserHomeDir() reads USERPROFILE instead.

@Divyansh670 Divyansh670 requested a review from a team as a code owner June 12, 2026 12:07
@aheritier aheritier added area/agent For work that has to do with the general agent loop/agentic features of the app kind/fix PR fixes a bug (maps to fix:). Use on PRs only. area/cli CLI commands, flags, output formatting area/config For configuration parsing, YAML, environment variables area/tools For features/issues/fixes related to the usage of built-in and MCP tools and removed area/agent For work that has to do with the general agent loop/agentic features of the app area/config For configuration parsing, YAML, environment variables area/cli CLI commands, flags, output formatting labels Jun 12, 2026
@aheritier aheritier added the area/cli CLI commands, flags, output formatting label Jun 22, 2026
@Sayt-0

Sayt-0 commented Jun 26, 2026

Copy link
Copy Markdown
Member

Looks good. Thanks for the contribution!

@Sayt-0

Sayt-0 commented Jun 26, 2026

Copy link
Copy Markdown
Member

One small thing: the commit (c114ed67) is unsigned. This repo requires commits to be signed with a GPG or SSH key. Could you re-sign and force-push? Thanks!

@Divyansh670 Divyansh670 force-pushed the fix/windows-home-expansion branch from c114ed6 to 25eb122 Compare June 26, 2026 15:39
@Divyansh670

Divyansh670 commented Jun 26, 2026 via email

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli CLI commands, flags, output formatting area/tools For features/issues/fixes related to the usage of built-in and MCP tools kind/fix PR fixes a bug (maps to fix:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants