alacritty: cleanup after 0.13 merge in nixpkgs

modules/programs/alacritty.nix

@@ -4,9 +4,7 @@ with lib;
 
 let
   cfg = config.programs.alacritty;
-  useToml = lib.versionAtLeast cfg.package.version "0.13";
   tomlFormat = pkgs.formats.toml { };
-  configFileName = "alacritty.${if useToml then "toml" else "yml"}";
 in {
   options = {
     programs.alacritty = {
@@ -52,24 +50,15 @@ in {
   config = mkIf cfg.enable {
     home.packages = [ cfg.package ];
 
-    xdg.configFile."alacritty/${configFileName}" =
+    xdg.configFile."alacritty/alacritty.toml" = lib.mkIf (cfg.settings != { }) {
-      lib.mkIf (cfg.settings != { }) (lib.mkMerge [
+      source = (tomlFormat.generate "alacritty.toml" cfg.settings).overrideAttrs
-        (lib.mkIf useToml {
+        (finalAttrs: prevAttrs: {
-          source =
+          buildCommand = lib.concatStringsSep "\n" [
-            (tomlFormat.generate configFileName cfg.settings).overrideAttrs
+            prevAttrs.buildCommand
-            (finalAttrs: prevAttrs: {
+            # TODO: why is this needed? Is there a better way to retain escape sequences?
-              buildCommand = lib.concatStringsSep "\n" [
+            "substituteInPlace $out --replace '\\\\' '\\'"
-                prevAttrs.buildCommand
+          ];
-                # TODO: why is this needed? Is there a better way to retain escape sequences?
+        });
-                "substituteInPlace $out --replace '\\\\' '\\'"
+    };
-              ];
-            });
-        })
-        # TODO remove this once we don't need to support Alacritty < 0.12 anymore
-        (lib.mkIf (!useToml) {
-          text =
-            replaceStrings [ "\\\\" ] [ "\\" ] (builtins.toJSON cfg.settings);
-        })
-      ]);
   };
 }

tests/modules/programs/alacritty/empty-settings.nix

@@ -1,7 +1,3 @@
-{ config, lib, pkgs, ... }:
-
-with lib;
-
 {
   config = {
     programs.alacritty.enable = true;

tests/modules/programs/alacritty/example-settings-expected.toml

@@ -1,5 +1,5 @@
 [[keyboard.bindings]]
-chars = "\x0c"
+chars = "\u000c"
 key = "K"
 mods = "Control"
 

tests/modules/programs/alacritty/example-settings.nix

@@ -1,12 +1,10 @@
-{ config, lib, pkgs, ... }:
+{ config, ... }:
-
-with lib;
 
 {
   config = {
     programs.alacritty = {
       enable = true;
-      package = config.lib.test.mkStubPackage { version = "0.13.0"; };
+      package = config.lib.test.mkStubPackage { };
 
       settings = {
         window.dimensions = {
@@ -17,7 +15,7 @@ with lib;
         keyboard.bindings = [{
           key = "K";
           mods = "Control";
-          chars = "\\x0c";
+          chars = "\\u000c";
         }];
       };
     };

tests/modules/programs/alacritty/settings-merging-expected.yml

@@ -1 +0,0 @@
-{"font":{"bold":{"family":"SFMono"},"normal":{"family":"SFMono"}},"key_bindings":[{"chars":"\x0c","key":"K","mods":"Control"}],"window":{"dimensions":{"columns":200,"lines":3}}}

tests/modules/programs/alacritty/settings-merging.nix

@@ -1,12 +1,10 @@
-{ config, lib, pkgs, ... }:
+{ config, lib, ... }:
-
-with lib;
 
 {
   config = {
     programs.alacritty = {
       enable = true;
-      package = config.lib.test.mkStubPackage { version = "0.12.3"; };
+      package = config.lib.test.mkStubPackage { };
 
       settings = {
         window.dimensions = {
@@ -14,10 +12,10 @@ with lib;
           columns = 200;
         };
 
-        key_bindings = [{
+        keyboard.bindings = [{
           key = "K";
           mods = "Control";
-          chars = "\\x0c";
+          chars = "\\u000c";
         }];
 
         font = let
@@ -32,8 +30,8 @@ with lib;
 
     nmt.script = ''
       assertFileContent \
-        home-files/.config/alacritty/alacritty.yml \
+        home-files/.config/alacritty/alacritty.toml \
-        ${./settings-merging-expected.yml}
+        ${./settings-toml-expected.toml}
     '';
   };
 }

tests/modules/programs/alacritty/toml_config.nix

@@ -4,7 +4,7 @@
   config = {
     programs.alacritty = {
       enable = true;
-      package = config.lib.test.mkStubPackage { version = "0.13.0"; };
+      package = config.lib.test.mkStubPackage { };
 
       settings = {
         window.dimensions = {