Skip to content

refactor(cli): clean up searching for config files + fix config preference - #371

Open
fischeti wants to merge 2 commits into
masterfrom
cli-config-cleanup
Open

fischeti wants to merge 2 commits into
masterfrom
cli-config-cleanup

Conversation

@fischeti

Copy link
Copy Markdown
Contributor

No description provided.

fischeti and others added 2 commits September 30, 2026 12:19
`PartialConfig::merge` combined the `overrides` and `plugins` maps with
`IndexMap::extend`, which replaces existing keys. As a result, when
several configuration files named the same dependency, the entry from
the lowest-precedence file (e.g. `~/.config/bender.yml` or
`/etc/bender.yml`) won over the one closest to the package.

Keep the higher-precedence entry instead, and lowercase override names
when reading each file rather than after merging, so that names
differing only in case follow the same precedence.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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