gotify-server.ui: modernize

- Use finalAttrs.
- Use yarnConfigHook and yarnBuildHook.
- nix-update specify ui subpackage.

Author: Moraxyc

pkgs/by-name/go/gotify-server/package.nix

@@ -28,20 +28,20 @@ buildGoModule (finalAttrs: {
     sqlite
   ];
 
-  ui = callPackage ./ui.nix { };
+  ui = callPackage ./ui.nix { inherit (finalAttrs) src version; };
 
-  preBuild = ''
-    if [ -n "$ui" ] # to make the preBuild a no-op inside the goModules fixed-output derivation, where it would fail
-    then
-      cp -r $ui ui/build
-    fi
+  # Use preConfigure instead of preBuild to keep goModules independent from ui
+  preConfigure = ''
+    cp -r ${finalAttrs.ui} ui/build
   '';
 
   passthru = {
-    # For nix-update to detect the location of this attribute from this
-    # derivation.
-    inherit (finalAttrs.ui) offlineCache;
-    updateScript = nix-update-script { };
+    updateScript = nix-update-script {
+      extraArgs = [
+        "--subpackage"
+        "ui"
+      ];
+    };
     tests = {
       nixos = nixosTests.gotify-server;
     };

pkgs/by-name/go/gotify-server/ui.nix

@@ -1,45 +1,31 @@
 {
+  src,
+  version,
   stdenv,
-  yarn,
-  fixup-yarn-lock,
+  yarnConfigHook,
+  yarnBuildHook,
   nodejs-slim,
   fetchYarnDeps,
-  gotify-server,
 }:
 
-stdenv.mkDerivation rec {
+stdenv.mkDerivation (finalAttrs: {
   pname = "gotify-ui";
-  inherit (gotify-server) version;
+  inherit version;
 
-  src = gotify-server.src + "/ui";
+  src = src + "/ui";
 
-  offlineCache = fetchYarnDeps {
-    yarnLock = "${src}/yarn.lock";
+  yarnOfflineCache = fetchYarnDeps {
+    yarnLock = "${finalAttrs.src}/yarn.lock";
     hash = "sha256-ejHzo6NHCMlNiYePWvfMY9Blb58pj3UQ5PFI0V84flI=";
   };
 
   nativeBuildInputs = [
-    yarn
-    fixup-yarn-lock
+    yarnConfigHook
+    yarnBuildHook
     nodejs-slim
   ];
 
-  postPatch = ''
-    export HOME=$NIX_BUILD_TOP/fake_home
-    yarn config --offline set yarn-offline-mirror $offlineCache
-    fixup-yarn-lock yarn.lock
-    yarn install --offline --frozen-lockfile --ignore-scripts --no-progress --non-interactive
-    patchShebangs node_modules/
-  '';
-
-  buildPhase = ''
-    runHook preBuild
-
-    export NODE_OPTIONS=--openssl-legacy-provider
-    yarn --offline build
-
-    runHook postBuild
-  '';
+  env.NODE_OPTIONS = "--openssl-legacy-provider";
 
   installPhase = ''
     runHook preInstall
@@ -48,4 +34,4 @@ stdenv.mkDerivation rec {
 
     runHook postInstall
   '';
-}
+})