Skip to content

get_app_folder: ignore empty or relative XDG env vars - #101

Merged
thorwhalen merged 2 commits into
masterfrom
fix/app-folder-empty-env-var
Oct 13, 2026
Merged

thorwhalen merged 2 commits into
masterfrom
fix/app-folder-empty-env-var

Conversation

@thorwhalen

@thorwhalen thorwhalen commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Fixes #98. An empty XDG_* variable no longer roots the app folder at the cwd; per the XDG spec an empty or relative value is ignored and the default used. ~ is still expanded (backward compatible). Behavior change: a relative value is now ignored too, as the spec requires.

Test added; full suite 820 passed. Dependents gate (priv test-dependents i2 --transitive): 73 pass / 20 fail, identical set to master (environmental failures). Adversarial Opus review: approve, no blocking findings (15 env values compared old vs new; callers mood/oa/scoopy unaffected); its docstring nit was applied.

🤖 Generated with Claude Code

thorwhalen and others added 2 commits October 13, 2026 07:09
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@thorwhalen
thorwhalen merged commit b5a4f74 into master Oct 13, 2026
12 checks passed
@thorwhalen
thorwhalen deleted the fix/app-folder-empty-env-var branch October 13, 2026 08:24
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.

util: empty env var counts as set in os.getenv(env_var, default) app-folder lookup

1 participant