just remembered functions do exist.

[vic]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors repeated attribute-update patterns in default.nix by introducing two helper functions and then using them to simplify configuration updates.

• Introduce accAttr and mergeAttrs helpers to DRY up repeated self (c: …) calls.
• Refactor accumulating (filter, map, addPath, etc.) and non-accumulating (withLib, pipeTo, leafs) methods to use these helpers.

Comments suppressed due to low confidence (4)

default.nix:102

• [nitpick] The helper name accAttr is quite terse and may not clearly convey its purpose; consider a more descriptive name like updateAttribute or accumulateAttribute.

            accAttr = attrName: acc: self (c: mapAttr (f c) attrName acc);

default.nix:103

• [nitpick] The helper name mergeAttrs could be more explicit—names such as mergeWithAttrs or overrideAttributes might better reflect its behavior.

            mergeAttrs = attrs: self (c: (f c) // attrs);

default.nix:102

• [nitpick] Consider adding a brief comment above accAttr to explain its purpose and usage, which will help future maintainers understand the abstraction.

            accAttr = attrName: acc: self (c: mapAttr (f c) attrName acc);

default.nix:103

• [nitpick] Adding a short comment for mergeAttrs describing how it merges existing configuration with new attributes would improve readability.

            mergeAttrs = attrs: self (c: (f c) // attrs);