fix: use absolute path for pip install -r target

try_install_script runs pip with cwd=repo_path, so if requirements_path
were relative it would get re-resolved against that cwd, producing a
doubled path like <rel>/<rel>/requirements.txt.

Wrap the requirements_path with os.path.abspath before handing it to
pip to make the -r target unambiguous regardless of whether the caller
passes an absolute or relative repo_path. Add a regression test that
invokes execute_install_script with a relative repo_path.

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>

Author: bigcat88

comfy_cli/command/custom_nodes/command.py

@@ -171,7 +171,10 @@ def execute_install_script(repo_path):
     if os.path.exists(requirements_path):
         print("Install: pip packages")
         python = resolve_workspace_python(workspace_manager.workspace_path)
-        install_cmd = [python, "-m", "pip", "install", "-r", requirements_path]
+        # Absolute path so pip doesn't re-resolve it against cwd=repo_path
+        # in try_install_script, which would double the path if repo_path
+        # is relative.
+        install_cmd = [python, "-m", "pip", "install", "-r", os.path.abspath(requirements_path)]
         try_install_script(repo_path, install_cmd)
 
     if os.path.exists(install_script_path):

tests/comfy_cli/test_custom_nodes_python_resolution.py

@@ -1,3 +1,4 @@
+import os
 from unittest.mock import patch
 
 from comfy_cli.command.custom_nodes import command
@@ -104,6 +105,30 @@ def test_uses_pip_install_r(self, tmp_path):
         argv = mock_check_call.call_args[0][0]
         assert argv == ["/resolved/python", "-m", "pip", "install", "-r", str(requirements_path)]
 
+    def test_requirements_path_is_absolute_when_repo_path_is_relative(self, tmp_path, monkeypatch):
+        # try_install_script runs pip with cwd=repo_path. If requirements_path
+        # were relative, pip would resolve `-r <rel>/requirements.txt` against
+        # that cwd, producing a doubled path like <rel>/<rel>/requirements.txt.
+        # Guard: the `-r` target must be absolute regardless of the input.
+        (tmp_path / "requirements.txt").write_text("numpy>=1.0\n")
+        monkeypatch.chdir(tmp_path.parent)
+        relative_repo = tmp_path.name  # e.g. "test_requirements_path..." relative to cwd
+
+        with (
+            patch(
+                "comfy_cli.command.custom_nodes.command.resolve_workspace_python",
+                return_value="/resolved/python",
+            ),
+            patch.object(command.workspace_manager, "workspace_path", str(tmp_path)),
+            patch("comfy_cli.command.custom_nodes.command.subprocess.check_call") as mock_check_call,
+        ):
+            command.execute_install_script(relative_repo)
+
+        argv = mock_check_call.call_args[0][0]
+        target = argv[-1]
+        assert os.path.isabs(target), f"-r target is not absolute: {target!r}"
+        assert target == str(tmp_path / "requirements.txt")
+
 
 class TestUpdateNodeIdCache:
     def test_uses_resolved_python(self, tmp_path):