Skip to content

Conversation

a-usr
Copy link
Member

@a-usr a-usr commented Oct 13, 2025

Implement functionality that lazily loads modules (by ab)using the freeformType option"

Blockers

  • This PR is currently in a non-working state. Merging this would be self-sabotage

Meta

Related Issue(s): <None>

AI used to generate code included in this PR?: Minimally

One section of code intruduced is based off of code from my personal nixos configuration, which in turn was generated with the help of Claude Sonnet 4.5

All Submissions:

  • Formatted commit message in accordance with CONTRIBUTING.md guidelines
  • Filled in all meta items
  • Mentioned any blockers before the PR can merge
  • Verified there are no conflicting PRs open

New Module Submissions:

  • Followed the general API laid out in CONTRIBUTING.md
  • Wrote tests (or expressed a need for help on tests)

Possibly related:

#114

Implement functionality that lazily loads modules (by ab)using the
`freeformType` option"
@a-usr a-usr requested a review from Lunarnovaa October 13, 2025 19:08
@a-usr
Copy link
Member Author

a-usr commented Oct 13, 2025

@Lunarnovaa Putting this here since it seemed from your discord messages that you'd be interested in taking a look

Comment on lines +75 to +79
/**
pop the last element from the list
pop :: [ T ] -> [ T ]
*/
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function vanished, will clean up later

Comment on lines +48 to +50
config = {
files = usedConfig;
};
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is intended to be config = usedConfig;, however during debugging I suspected that to cause the infrec, and I haven't changed it back yet

Comment on lines +18 to +25
{
config,
options,
...
}: {
options = options.rum;
config.rum = builtins.removeAttrs config ["rum"];
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This exists as a workaround to the fact that we are basically declaring options at user.<user>.rum.rum atm.

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