That's the config of the packages submodule from flake-parts. And we don't need builtins.currentSystem -- just pass system from the forAllSystems into that call, such as here:…
We may have talked about it in a side channel, but this is how one can do it: https://github.com/fricklerhandwerk/flake-inputs (real-world example: https://github.com/ngi-nix/ngipkgs/blob/bbf2cdaa6…
We can just re-export that from default.nix no? Why keep the split?
Is that imported from anywhere else except the top level? Because if not, we could just leave it with { pkgs }: for now.
++ [ pkgs.npins ] (we had a schema divergence multiple times now, please always ship npins with the project)
We currently have no need to emphasize composition, and if we need to we can still follow through with the module system idea. Right now it's just a monorepo with stuff in it.
Looks good generally (we can keep trimming it down later), but I don't understand why we would need multiple shell.nix or .envrc
here it's simply not setting those fields because they're optional
this was required to somehow propagate that fixed attrs are required fields