Skip to content

Fix contexts for user aspects to avoid duplicates.#153

Merged
vic merged 1 commit intomainfrom
dups
Feb 11, 2026
Merged

Fix contexts for user aspects to avoid duplicates.#153
vic merged 1 commit intomainfrom
dups

Conversation

@vic
Copy link
Owner

@vic vic commented Feb 11, 2026

Prevously we had a { OS, HM, host, user} context applied to both, the host aspect and the user aspect, that was causing duplicate definitions since the context was the same and both host and user aspects include den.default.

To fix this properly we now have separate aspects.

The host aspect is applied to { OS, host, user } for each home-managed user.
The user aspect is applied to { HM, host, user } for each home-managed user.

![NOTICE] Update your context-aware aspects if they use { OS, HM, host, user}
As mentioned above this context is now split in two according if it applies to a host or to a user aspect.
I've also updated the documentation: https://den.oeiuwq.com/os-hm-dependencies.html

Also refactored the code to make it more idiomatic and easy to read. Added tests based on the bogus repos from #147 (reply in thread) and #150.

Closes #150

@vic vic changed the title fix(unfree,inputs'): Fix context for user aspects to avoid duplicates. Fix contexts for user aspects to avoid duplicates. Feb 11, 2026
@vic vic merged commit 79dfb50 into main Feb 11, 2026
11 checks passed
@vic vic deleted the dups branch February 11, 2026 18:52
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.

BUG: unfree battery broken for Home Manager packages

1 participant