webui: handle playback_pos response with empty contents

More stuff please

flake.nix

@@ -57,14 +57,28 @@
         nativeBuildInputs = [ poetry-core ];
         propagatedBuildInputs = [ setuptools flakes.self.pkgs.remi requests typer rich urllib3 ];
       };
-      default = flakes.self.pkgs.grzegorz-clients;
+      grzegorzctl = pkgs.runCommandNoCCLocal "grzegorzctl" (
+        {
+          nativeBuildInputs = [ pkgs.installShellFiles ];
+        } //
+        { inherit (flakes.self.pkgs.grzegorz-clients) meta; } //
+        { meta.mainProgram = "grzegorzctl"; }
+      )''
+        mkdir -p $out/bin
+        ln -s "${flakes.self.pkgs.grzegorz-clients}/bin/grzegorzctl" $out/bin/grzegorzctl
+        installShellCompletion --cmd grzegorzctl \
+          --bash <($out/bin/grzegorzctl --show-completion bash) \
+          --zsh <($out/bin/grzegorzctl --show-completion zsh) \
+          --fish <($out/bin/grzegorzctl --show-completion fish)
+      '';
+      default = flakes.self.pkgs.grzegorzctl;
     });
 
     apps = forAllSystems ({ system, ...}: rec {
       grzegorz-webui.type = "app";
       grzegorz-webui.program = "${self.packages.${system}.grzegorz-clients}/bin/grzegorz-webui";
       grzegorzctl.type = "app";
-      grzegorzctl.program = "${self.packages.${system}.grzegorz-clients}/bin/grzegorzctl";
+      grzegorzctl.program = "${self.packages.${system}.grzegorzctl}/bin/grzegorzctl";
       default = grzegorzctl;
     });
 

grzegorz_clients/cli.py

@@ -64,7 +64,7 @@ def _add(
         api.playlist_goto(current_index if put_pre else current_index + 1)
         api.set_playing(True)
 
-@cli.command(help="Add one ore more items to the playlist")
+@cli.command(help="Add one or more items to the playlist")
 def play(
     urls: list[str],
     pre: bool = False,
@@ -72,14 +72,14 @@ def play(
 ):
     _add(urls, put_post=not pre, put_pre=pre, play=True, api_base=api_base)
 
-@cli.command(help="Add one ore more items to the playlist")
+@cli.command(help="Add one or more items to the playlist")
 def next(
     urls: list[str],
     api_base: str = DEFAULT_API_BASE,
 ):
     _add(urls, put_post=True, api_base=api_base)
 
-@cli.command(help="Add one ore more items to the playlist")
+@cli.command(help="Add one or more items to the playlist")
 def queue(
     urls: list[str],
     play: bool = True,
@@ -107,6 +107,12 @@ def pause( api_base: str = DEFAULT_API_BASE ):
     api.set_endpoint(api_base)
     rich.print(api.set_playing(False), file=sys.stderr)
 
+@cli.command(help="Toggle playback")
+def toggle(api_base: str = DEFAULT_API_BASE):
+    api.set_endpoint(api_base)
+    playing = api.is_playing()
+    rich.print(api.set_playing(not playing), file=sys.stderr)
+
 @cli.command(help="Goto next item in playlist")
 def skip( api_base: str = DEFAULT_API_BASE ):
     api.set_endpoint(api_base)
@@ -154,10 +160,19 @@ def status(
 
 @cli.command(help="Set the playback volume")
 def set_volume(
-    volume: int,
+    volume: str,
     api_base: str = DEFAULT_API_BASE,
 ):
     api.set_endpoint(api_base)
+
+    volume = volume.removesuffix("%")
+
+    if volume.startswith("+") or volume.startswith("-"):
+        current_volume = api.get_volume()
+        new_volume = max(0, min(100, current_volume + int(volume)))
+    else:
+        new_volume = int(volume)
+
     rich.print(api.set_volume(volume), file=sys.stderr)
 
 

grzegorz_clients/remi_ui.py

@@ -294,7 +294,7 @@ class RemiApp(App):
 			except api.APIError:
 				playback_pos = None
 
-			if playback_pos:
+			if playback_pos and isinstance(playback_pos, dict) and playback_pos["current"] and playback_pos["total"]:
 				slider_pos = playback_pos["current"] / playback_pos["total"] * 100
 				current = seconds_to_timestamp(playback_pos["current"])
 				total = seconds_to_timestamp(playback_pos["total"])