yt-dlp: add settings option

.github/CODEOWNERS

@@ -314,6 +314,7 @@ Makefile                                              @thiagokokada
 /tests/modules/programs/xmobar                        @t4ccer
 
 /modules/programs/yt-dlp.nix                          @marsam
+/tests/modules/programs/yt-dlp                        @marsam
 
 /modules/programs/z-lua.nix                           @marsam
 

modules/programs/yt-dlp.nix

@@ -6,6 +6,12 @@ let
 
   cfg = config.programs.yt-dlp;
 
+  renderSettings = mapAttrsToList (name: value:
+    if isBool value then
+      if value then "--${name}" else "--no-${name}"
+    else
+      "--${name} ${toString value}");
+
 in {
   meta.maintainers = [ maintainers.marsam ];
 
@@ -19,29 +25,48 @@ in {
       description = "Package providing the <command>yt-dlp</command> tool.";
     };
 
-    extraConfig = mkOption {
+    settings = mkOption {
-      type = types.lines;
+      type = with types; attrsOf (oneOf [ bool int str ]);
-      default = "";
+      default = { };
       example = literalExpression ''
-        --embed-thumbnail
+        embed-thumbnail = true;
-        --embed-subs
+        embed-subs = true;
-        --sub-langs all
+        sub-langs = "all";
-        --downloader aria2c
+        downloader = "aria2c";
-        --downloader-args aria2c:'-c -x8 -s8 -k1M'
+        downloader-args = "aria2c:'-c -x8 -s8 -k1M'";
       '';
       description = ''
         Configuration written to
-        <filename>$XDG_CONFIG_HOME/yt-dlp/config</filename>. See
+        <filename>$XDG_CONFIG_HOME/yt-dlp/config</filename>.
-        <link xlink:href="https://github.com/yt-dlp/yt-dlp#configuration" />
+        </para><para>
+        Options must be specified in their <quote>long form</quote>, for
+        example, <code>update = true;</code> instead of <code>U = true;</code>.
+        Short options can be specified in the <code>extraConfig</code> option.
+        See <link xlink:href="https://github.com/yt-dlp/yt-dlp#configuration"/>
         for explanation about possible values.
       '';
     };
+
+    extraConfig = mkOption {
+      type = types.lines;
+      default = "";
+      example = ''
+        --update
+        -F
+      '';
+      description = ''
+        Extra configuration to add to
+        <filename>$XDG_CONFIG_HOME/yt-dlp/config</filename>.
+      '';
+    };
   };
 
   config = mkIf cfg.enable {
     home.packages = [ cfg.package ];
 
-    xdg.configFile."yt-dlp/config" =
+    xdg.configFile."yt-dlp/config" = mkIf (cfg.settings != { }) {
-      mkIf (cfg.extraConfig != "") { text = cfg.extraConfig; };
+      text = concatStringsSep "\n"
+        (remove "" (renderSettings cfg.settings ++ [ cfg.extraConfig ])) + "\n";
+    };
   };
 }

tests/default.nix

@@ -155,6 +155,7 @@ import nmt {
     ./modules/programs/terminator
     ./modules/programs/waybar
     ./modules/programs/xmobar
+    ./modules/programs/yt-dlp
     ./modules/services/barrier
     ./modules/services/devilspie2
     ./modules/services/dropbox

tests/modules/programs/yt-dlp/default.nix

@@ -0,0 +1 @@
+{ yt-dlp-simple-config = ./yt-dlp-simple-config.nix; }

tests/modules/programs/yt-dlp/yt-dlp-simple-config-expected

@@ -0,0 +1,8 @@
+--downloader aria2c
+--downloader-args aria2c:'-c -x8 -s8 -k1M'
+--no-embed-subs
+--embed-thumbnail
+--sub-langs all
+--trim-filenames 30
+--config-locations /home/user/.yt-dlp.conf
+

tests/modules/programs/yt-dlp/yt-dlp-simple-config.nix

@@ -0,0 +1,27 @@
+{ ... }:
+
+{
+  programs.yt-dlp = {
+    enable = true;
+    settings = {
+      embed-thumbnail = true;
+      embed-subs = false;
+      sub-langs = "all";
+      downloader = "aria2c";
+      downloader-args = "aria2c:'-c -x8 -s8 -k1M'";
+      trim-filenames = 30;
+    };
+    extraConfig = ''
+      --config-locations /home/user/.yt-dlp.conf
+    '';
+  };
+
+  test.stubs.yt-dlp = { };
+
+  nmt.script = ''
+    assertFileExists home-files/.config/yt-dlp/config
+    assertFileContent home-files/.config/yt-dlp/config ${
+      ./yt-dlp-simple-config-expected
+    }
+  '';
+}