fish: always run fish_indent

modules/programs/fish.nix

@@ -146,14 +146,11 @@ let
     (mapAttrsToList (k: v: "alias ${k} ${escapeShellArg v}") cfg.shellAliases);
 
   fishIndent = name: text:
-    if cfg.formatFishScripts then
+    pkgs.runCommand name {
-      pkgs.runCommand name {
+      nativeBuildInputs = [ cfg.package ];
-        nativeBuildInputs = [ cfg.package ];
+      inherit text;
-        inherit text;
+      passAsFile = [ "text" ];
-        passAsFile = [ "text" ];
+    } "fish_indent < $textPath > $out";
-      } "fish_indent < $textPath > $out"
-    else
-      pkgs.writeText name text;
 
 in {
   imports = [
@@ -288,16 +285,6 @@ in {
         <link xlink:href="https://fishshell.com/docs/current/cmds/function.html"/>.
       '';
     };
-
-    programs.fish.formatFishScripts = mkOption {
-      type = types.bool;
-      default = false;
-      description = ''
-        Whether to process fish configuration and scripts with
-        <literal>fish_indent</literal>.
-      '';
-    };
-
   };
 
   config = mkIf cfg.enable (mkMerge [

tests/modules/programs/atuin/fish.nix

@@ -13,7 +13,6 @@
   test.stubs = {
     atuin = { };
     bash-preexec = { };
-    fish = { };
   };
 
   nmt.script = ''

tests/modules/programs/atuin/no-shell.nix

@@ -20,7 +20,6 @@
   test.stubs = {
     atuin = { };
     bash-preexec = { };
-    fish = { };
   };
 
   nmt.script = ''

tests/modules/programs/fish/functions.nix

@@ -6,13 +6,13 @@ let
 
   func = pkgs.writeText "func.fish" ''
     function func
-      echo "Hello"
+        echo Hello
     end
   '';
 
   funcEvent = pkgs.writeText "func-event.fish" ''
     function func-event --on-event="fish_command_not_found"
-      echo "Not found!"
+        echo "Not found!"
     end
   '';
 
@@ -34,8 +34,6 @@ in {
     xdg.dataFile."fish/home-manager_generated_completions".source =
       lib.mkForce (builtins.toFile "empty" "");
 
-    test.stubs.fish = { };
-
     nmt = {
       description =
         "if fish.function is set, check file exists and contents match";

tests/modules/programs/fish/no-functions.nix

@@ -14,8 +14,6 @@ with lib;
     xdg.dataFile."fish/home-manager_generated_completions".source =
       lib.mkForce (builtins.toFile "empty" "");
 
-    test.stubs.fish = { };
-
     nmt = {
       description =
         "if fish.functions is blank, the functions folder should not exist.";

tests/modules/programs/fish/plugins.nix

@@ -12,26 +12,26 @@ let
 
     # Set paths to import plugin components
     if test -d $plugin_dir/functions
-      set fish_function_path $fish_function_path[1] $plugin_dir/functions $fish_function_path[2..-1]
+        set fish_function_path $fish_function_path[1] $plugin_dir/functions $fish_function_path[2..-1]
     end
 
     if test -d $plugin_dir/completions
-      set fish_complete_path $fish_complete_path[1] $plugin_dir/completions $fish_complete_path[2..-1]
+        set fish_complete_path $fish_complete_path[1] $plugin_dir/completions $fish_complete_path[2..-1]
     end
 
     # Source initialization code if it exists.
     if test -d $plugin_dir/conf.d
-      for f in $plugin_dir/conf.d/*.fish
+        for f in $plugin_dir/conf.d/*.fish
-        source $f
+            source $f
-      end
+        end
     end
 
     if test -f $plugin_dir/key_bindings.fish
-      source $plugin_dir/key_bindings.fish
+        source $plugin_dir/key_bindings.fish
     end
 
     if test -f $plugin_dir/init.fish
-      source $plugin_dir/init.fish
+        source $plugin_dir/init.fish
     end
   '';
 
@@ -50,8 +50,6 @@ in {
     xdg.dataFile."fish/home-manager_generated_completions".source =
       lib.mkForce (builtins.toFile "empty" "");
 
-    test.stubs.fish = { };
-
     nmt = {
       description =
         "if fish.plugins set, check conf.d file exists and contents match";

tests/modules/programs/nix-index/assert-on-command-not-found.nix

@@ -12,10 +12,7 @@
     xdg.dataFile."fish/home-manager_generated_completions".source =
       lib.mkForce (builtins.toFile "empty" "");
 
-    test.stubs = {
+    test.stubs.zsh = { };
-      zsh = { };
-      fish = { };
-    };
 
     programs.nix-index.enable = true;
 

tests/modules/programs/nix-index/integrations.nix

@@ -16,10 +16,7 @@ in {
     xdg.dataFile."fish/home-manager_generated_completions".source =
       lib.mkForce (builtins.toFile "empty" "");
 
-    test.stubs = {
+    test.stubs.zsh = { };
-      zsh = { };
-      fish = { };
-    };
 
     programs.nix-index.enable = true;
 

tests/modules/programs/oh-my-posh/fish.nix

@@ -14,10 +14,7 @@
   xdg.dataFile."fish/home-manager_generated_completions".source =
     lib.mkForce (builtins.toFile "empty" "");
 
-  test.stubs = {
+  test.stubs.oh-my-posh = { };
-    oh-my-posh = { };
-    fish = { };
-  };
 
   nmt.script = ''
     assertFileExists home-files/.config/fish/config.fish

tests/modules/programs/pls/fish.nix

@@ -18,10 +18,7 @@ with lib;
     xdg.dataFile."fish/home-manager_generated_completions".source =
       mkForce (builtins.toFile "empty" "");
 
-    test.stubs = {
+    test.stubs.pls = { };
-      pls = { };
-      fish = { };
-    };
 
     nmt.script = ''
       assertFileExists home-files/.config/fish/config.fish

tests/modules/programs/powerline-go/fish.nix

@@ -22,10 +22,7 @@ with lib;
     xdg.dataFile."fish/home-manager_generated_completions".source =
       mkForce (builtins.toFile "empty" "");
 
-    test.stubs = {
+    test.stubs.powerline-go = { };
-      powerline-go = { };
-      fish = { };
-    };
 
     nmt.script = ''
       assertFileExists home-files/.config/fish/config.fish

tests/modules/programs/scmpuff/fish.nix

@@ -8,7 +8,6 @@
   xdg.dataFile."fish/home-manager_generated_completions".source =
     lib.mkForce (builtins.toFile "empty" "");
 
-  test.stubs.fish = { };
   test.stubs.scmpuff = { };
 
   nmt.script = ''

tests/modules/programs/scmpuff/no-fish.nix

@@ -11,7 +11,6 @@
   xdg.dataFile."fish/home-manager_generated_completions".source =
     lib.mkForce (builtins.toFile "empty" "");
 
-  test.stubs.fish = { };
   test.stubs.scmpuff = { };
 
   nmt.script = ''