rmfakecloud: build the webui together with the app

This avoids maintaining an artificial derivation for the web UI,
and hopefully will allow the auto-update bot to make PRs for further
updates

Author: martinetd

pkgs/by-name/rm/rmfakecloud/package.nix

@@ -2,8 +2,9 @@
   lib,
   fetchFromGitHub,
   buildGoModule,
-  callPackage,
   enableWebui ? true,
+  pnpm_9,
+  nodejs,
   nixosTests,
 }:
 
@@ -20,17 +21,32 @@ buildGoModule rec {
 
   vendorHash = "sha256-9tfxE03brUvCYusmewiqNpCkKyIS9qePqylrzDWrJLY=";
 
-  ui = callPackage ./webui.nix { inherit version src; };
-
-  postPatch =
-    if enableWebui then
-      ''
-        cp -a ${ui} ui/dist
-      ''
-    else
-      ''
-        sed -i '/go:/d' ui/assets.go
-      '';
+  # if using webUI build it
+  pnpmRoot = "ui";
+  pnpmDeps = pnpm_9.fetchDeps {
+    inherit pname version src;
+    sourceRoot = "${src.name}/ui";
+    pnpmLock = "${src}/ui/pnpm-lock.yaml";
+    hash = "sha256-VNmCT4um2W2ii8jAm+KjQSjixYEKoZkw7CeRwErff/o=";
+  };
+  preBuild = lib.optionals enableWebui ''
+    # using sass-embedded fails at executing node_modules/sass-embedded-linux-x64/dart-sass/src/dart
+    rm -r ui/node_modules/sass-embedded ui/node_modules/.pnpm/sass-embedded*
+
+    # avoid re-running pnpm i...
+    touch ui/pnpm-lock.yaml
+
+    make ui/dist
+  '';
+  nativeBuildInputs = lib.optionals enableWebui [
+    nodejs
+    pnpm_9.configHook
+  ];
+
+  # ... or don't embed it in the server
+  postPatch = lib.optionals (!enableWebui) ''
+    sed -i '/go:/d' ui/assets.go
+  '';
 
   ldflags = [
     "-s"