Skip to content

docs(config): note that XDG_* is ignored on macOS/Windows and must stay that way - #120

Merged
ErikBjare merged 1 commit into
masterfrom
docs/platformdirs-xdg-comment
Sep 25, 2026
Merged

ErikBjare merged 1 commit into
masterfrom
docs/platformdirs-xdg-comment

Conversation

@ErikBjare

Copy link
Copy Markdown
Member

Comment-only. Directory resolution must match the server that writes the files aw-client reads (e.g. the local API key). aw-server-rust (dirs crate) and aw-core/aw-server (platformdirs) ignore XDG_* on macOS/Windows, so the client must not honor it either. Context: #119.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@ErikBjare
ErikBjare merged commit ed64fb2 into master Sep 25, 2026
7 checks passed
@ErikBjare
ErikBjare deleted the docs/platformdirs-xdg-comment branch September 25, 2026 10:10
@greptile-apps

greptile-apps Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low risk] Adds explanatory comment to config code.

The comment-only change appears safe to merge.

Summary

Adds guidance that client directory resolution should remain aligned with the server when locating local files such as its API key. No executable behavior changes.

Reviews (1) · Last reviewed commit: "docs(dirs): note that XDG_* is ignored o..."

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