Merge pull request #328 from xsteadfastx/sql-formatter-fix

sql-formatter: fix multi file input

Author: zimbatm

examples/formatter-sql-formatter.toml

@@ -1,6 +1,6 @@
 # Example generated by ../examples.sh
 [formatter.sql-formatter]
-command = "sql-formatter"
+command = "sql-formatter-fix"
 excludes = []
 includes = ["*.sql"]
-options = ["--fix"]
+options = []

programs/sql-formatter.nix

@@ -1,4 +1,5 @@
 {
+  pkgs,
   lib,
   config,
   mkFormatterModule,
@@ -34,9 +35,6 @@ in
     (mkFormatterModule {
       name = "sql-formatter";
       package = "sql-formatter";
-      args = [
-        "--fix"
-      ];
       includes = [ "*.sql" ];
     })
   ];
@@ -52,10 +50,15 @@ in
 
   config = lib.mkIf cfg.enable {
     settings.formatter.sql-formatter = {
-      command = "${cfg.package}/bin/sql-formatter";
-      options = lib.optionals (cfg.dialect != null) [
-        "-l=${cfg.dialect}"
-      ];
+      # sql-formatter doesn't support multiple file targets
+      # see https://github.com/sql-formatter-org/sql-formatter/issues/552
+      command = pkgs.writeShellScriptBin "sql-formatter-fix" ''
+        for file in "$@"; do
+          ${cfg.package}/bin/sql-formatter --fix ${
+            lib.optionalString (cfg.dialect != null) "-l ${cfg.dialect}"
+          } $file
+        done
+      '';
     };
   };
 }