flake.nix (1)

4-6: 💤 Low value

Consider using flake-utils for cleaner system iteration.

The manual forAllSystems implementation works correctly, but the flake-utils library provides a standard eachDefaultSystem helper that reduces boilerplate and is widely adopted in the Nix community.

♻️ Optional refactor using flake-utils

Add flake-utils to inputs:

  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixos-unstable";
+   flake-utils.url = "github:numtide/flake-utils";
  };

Then simplify the outputs:

  outputs =
-   { self, nixpkgs }:
+   { self, nixpkgs, flake-utils }:
-   let
-     systems = [
-       "aarch64-darwin"
-       "aarch64-linux"
-       "x86_64-darwin"
-       "x86_64-linux"
-     ];
-
-     forAllSystems = nixpkgs.lib.genAttrs systems;
-
-     pkgsFor =
-       system:
-       import nixpkgs {
-         inherit system;
-       };
-   in
+   flake-utils.lib.eachDefaultSystem (system:
+     let
+       pkgs = import nixpkgs { inherit system; };
+     in
    {
-     packages = forAllSystems (
-       system:
-       let
-         pkgs = pkgsFor system;
-       in
-       {
+     packages = {
         default = pkgs.callPackage ./nix/package.nix { };
         why-cli = self.packages.${system}.default;
-       }
-     );
+     };
      # ... rest of outputs
-   };
+   }) // {
+     overlays.default = final: _prev: {
+       why-cli = final.callPackage ./nix/package.nix { };
+     };
+     homeManagerModules.default = ...;
+   };

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@flake.nix` around lines 4 - 6, Add flake-utils to the inputs and replace your
custom system iteration with its helper: add an input named flake-utils (e.g.,
flake-utils.url) and in outputs swap the manual forAllSystems logic for
flake-utils.eachDefaultSystem, calling eachDefaultSystem(fn: system: { ... })
instead of your forAllSystems implementation; update any references to the
iterated `system` value inside outputs to match the eachDefaultSystem callback
signature and remove the old forAllSystems helper.