From cc8f014414d200cef9ff5aa549f0b21e218f5bbe Mon Sep 17 00:00:00 2001
From: adisbladis <adisbladis@gmail.com>
Date: Wed, 4 Sep 2024 12:13:00 +1200
Subject: [PATCH] renderers: Add mkPythonEditablePackage

---
 lib/renderers.nix      | 72 +++++++++++++++++++++++++++++++++++++++++-
 lib/test_project.nix   |  1 +
 lib/test_renderers.nix | 27 +++++++++++++++-
 3 files changed, 98 insertions(+), 2 deletions(-)

diff --git a/lib/renderers.nix b/lib/renderers.nix
index 4ac7758..07b398b 100644
--- a/lib/renderers.nix
+++ b/lib/renderers.nix
@@ -3,6 +3,7 @@
   pep508,
   pep440,
   pep621,
+  renderers,
   ...
 }:
 let
@@ -19,6 +20,8 @@ let
     mapAttrs
     filterAttrs
     concatMap
+    nameValuePair
+    isString
     ;
 
   # Group licenses by their SPDX IDs for easy lookup
@@ -103,7 +106,7 @@ in
       project,
       # Python derivation
       python,
-      # Python extras (markers) to enable.
+      # Python extras (optional-dependencies) to enable.
       extras ? [ ],
       # Map a Python extras group name to a Nix attribute set like:
       # { dev = "checkInputs"; }
@@ -201,4 +204,71 @@ in
         }
       )
       (attrNames filteredDeps.extras);
+
+  /*
+    Renders a project as an argument that can be passed to mkPythonEditablePackage.
+
+    Evaluates PEP-508 environment markers to select correct dependencies for the platform but does not validate version constraints.
+    For validation see `lib.validators`.
+
+    Note for Nix Flake users:
+    Flakes are copied to the store when using pure evaluation, meaning that the project root will point to a store directory.
+    Either set root manually to a string using the returned attribute set, or evaluate using `--impure`.
+
+    Type: mkPythonEditablePackage :: AttrSet -> AttrSet
+
+    Example:
+      # mkPythonEditablePackage { project = lib.project.loadPyproject ...; python = pkgs.python3;  }
+        { pname = "blinker"; version = "1.3.3.7"; dependencies = [ ]; }
+  */
+  mkPythonEditablePackage =
+    let
+      cleanArgs = lib.flip removeAttrs [ "root" ];
+    in
+    {
+      # Project metadata as returned by `lib.project.loadPyproject`
+      project,
+      # Editable root directory as a string
+      root ? toString project.projectRoot,
+      # Unknown args passed on verbatim to renderers.buildPythonPackage
+      ...
+    }@args:
+    let
+      # Render using buildPythonPackage
+      attrs = renderers.buildPythonPackage (cleanArgs args);
+
+      project' = project.pyproject.project;
+    in
+    # Reshape into mkPythonEditablePackage
+    assert isString root && root != "";
+    {
+      inherit (attrs)
+        dependencies
+        optional-dependencies
+        build-system
+        meta
+        ;
+      inherit root;
+    }
+    // optionalAttrs (project' ? scripts) {
+      inherit (project') scripts;
+    }
+    // optionalAttrs (project' ? gui-scripts) {
+      inherit (project') gui-scripts;
+    }
+    // optionalAttrs (project' ? entry-points) {
+      inherit (project') entry-points;
+    }
+    // optionalAttrs (attrs ? pname) {
+      inherit (attrs) pname;
+    }
+    // optionalAttrs (attrs ? version) {
+      inherit (attrs) version;
+    }
+    // optionalAttrs (args ? extrasAttrMappings && args.extrasAttrMappings != { }) {
+      # Inject derivationArgs for additional functionality
+      derivationArgs = lib.listToAttrs (
+        map (attr: nameValuePair attr attrs.${attr}) (attrValues args.extrasAttrMappings)
+      );
+    };
 }
diff --git a/lib/test_project.nix b/lib/test_project.nix
index 30aba26..fbe0af4 100644
--- a/lib/test_project.nix
+++ b/lib/test_project.nix
@@ -391,6 +391,7 @@ lib.fix (self: {
       expected = {
         renderers = [
           "buildPythonPackage"
+          "mkPythonEditablePackage"
           "withPackages"
         ];
         requires-python = null;
diff --git a/lib/test_renderers.nix b/lib/test_renderers.nix
index 5712fd7..e2a072c 100644
--- a/lib/test_renderers.nix
+++ b/lib/test_renderers.nix
@@ -33,7 +33,7 @@ let
     // optionalAttrs (attrs ? checkInputs) { checkInputs = map (drv: drv.pname) attrs.checkInputs; };
 
 in
-{
+rec {
   # withPackages can't be easily tested by mocks, dummy out.
   # Tested by integration tests.
   withPackages = {
@@ -43,6 +43,31 @@ in
     };
   };
 
+  mkPythonEditablePackage = {
+    testPdm = {
+      expr = clearDrvInputs (
+        renderers.mkPythonEditablePackage {
+          root = "/path/to/my_root";
+          project = projects.pdm;
+          python = mocks.cpythonLinux38;
+        }
+      );
+      expected = {
+        inherit (buildPythonPackage.testPdm.expected)
+          dependencies
+          optional-dependencies
+          build-system
+          meta
+          pname
+          ;
+        root = "/path/to/my_root";
+        scripts = {
+          pdm = "pdm.core:main";
+        };
+      };
+    };
+  };
+
   buildPythonPackage = {
     testPdm = {
       expr = clearDrvInputs (