refactor: centralize unfree package allowlists

Move all allowUnfreePredicate declarations to global configs to prevent
the "last definition wins" merging issue. Unfree packages are now managed
in two central locations:
- NixOS system packages: nixos/common/global/default.nix
- Home Manager packages: home-manager/sajenim/global/default.nix

home-manager/sajenim/features/desktop/default.nix

@@ -5,6 +5,7 @@
 }: {
   imports = [
     ./cava
+    ./discord
     ./dunst
     ./mpv
     ./obs

home-manager/sajenim/features/desktop/discord/default.nix

@@ -0,0 +1,15 @@
+{
+  pkgs,
+  ...
+}: {
+  nixpkgs.overlays = [
+    (final: prev: {
+      discord = prev.discord.override {withOpenASAR = true;};
+    })
+  ];
+
+  home.packages = with pkgs; [
+    discord
+    betterdiscordctl
+  ];
+}

home-manager/sajenim/features/editors/default.nix

@@ -37,14 +37,6 @@
       ])
     ]); # https://github.com/theCapypara/nix-jetbrains-plugins
 
-  # Allow unfree packages for proprietary software
-  nixpkgs.config.allowUnfreePredicate = pkg:
-    builtins.elem (lib.getName pkg) [
-      "claude-code"
-      "idea-ultimate"
-      "idea-ultimate-with-plugins" 
-    ];
-
   # Copy our configuration files to home directory
   home.file = {
     ".ideavimrc".source = ./ideavimrc;

home-manager/sajenim/global/default.nix

@@ -1,4 +1,4 @@
-{outputs, ...}: {
+{outputs, lib, ...}: {
   imports = [
     ./zsh.nix
   ];
@@ -11,6 +11,19 @@
     ];
     config = {
       allowUnfree = false;
+      # Centralized unfree package allowlist.
+      # Note: nixpkgs.config.allowUnfreePredicate doesn't merge across modules - only the
+      # last definition wins. To maintain explicit control over unfree packages, we list
+      # all allowed packages here rather than scattering predicates across feature modules.
+      allowUnfreePredicate = pkg:
+        builtins.elem (lib.getName pkg) [
+          # Editors
+          "claude-code"
+          "idea-ultimate"
+          "idea-ultimate-with-plugins"
+          # Desktop
+          "discord"
+        ];
     };
   };
 

nixos/common/global/default.nix

@@ -1,4 +1,4 @@
-{outputs, ...}: {
+{outputs, lib, ...}: {
   imports = [
     ./age.nix
     ./env.nix
@@ -15,6 +15,18 @@
     ];
     config = {
       allowUnfree = false;
+      # Centralized unfree package allowlist for NixOS system configuration.
+      # Note: nixpkgs.config.allowUnfreePredicate doesn't merge across modules - only the
+      # last definition wins. To maintain explicit control over unfree packages, we list
+      # all allowed packages here rather than scattering predicates across system modules.
+      allowUnfreePredicate = pkg:
+        builtins.elem (lib.getName pkg) [
+          # Gaming
+          "steam"
+          "steam-unwrapped"
+          # Services
+          "minecraft-server"
+        ];
     };
   };
 

nixos/common/users/sajenim/steam/default.nix

@@ -27,10 +27,4 @@
     "d /home/sajenim/.local 0755 sajenim users -"
     "d /home/sajenim/.local/share 0755 sajenim users -"
   ];
-
-  nixpkgs.config.allowUnfreePredicate = pkg:
-    builtins.elem (lib.getName pkg) [
-      "steam"
-      "steam-unwrapped"
-    ];
 }

nixos/viridian/services/minecraft/default.nix

@@ -116,9 +116,4 @@ in {
       }
     ];
   };
-
-  # Allow unfree packages.
-  nixpkgs.config.allowUnfreePredicate = pkg: builtins.elem (lib.getName pkg) [
-    "minecraft-server"
-  ];
 }