fix(parser): handle conda git-describe versions as unconstrained in dependency resolution

Author: trgiangv

docs/architecture/Execution/code-execution.md

@@ -46,7 +46,7 @@ flowchart LR
 
 - `PythonInitializer` chooses Pixi first, then pip-backed pyRevit CPython if Pixi cannot run.
 - `PythonEmbedded` extracts `Parser.py`, `ToolParser.py`, `PytestRunner.py`, setup scripts, and `pixi.toml`.
-- `PythonDepsManager` parses PEP 723 dependencies through `Parser.py`.
+- `PythonDepsManager` parses PEP 723 dependencies through `Parser.py`. Installed-state JSON may include conda git-describe versions; Parser treats those as unconstrained instead of failing the resolve.
 - Pixi uses conda-forge first and PyPI fallback.
 - Pip fallback depends on `pyrevit.exe attached` to locate `bin/cengines/CPY*/python.exe`.
 

source/DevTools.Execution/Resources/scripts/Parser.py

@@ -27,12 +27,12 @@
 import re
 import sys
 from pathlib import Path
-import tomllib
 
-from packaging.requirements import Requirement, InvalidRequirement
-from packaging.utils import canonicalize_name
+import tomllib
+from packaging.requirements import InvalidRequirement, Requirement
 from packaging.specifiers import SpecifierSet
-from packaging.version import Version, InvalidVersion
+from packaging.utils import canonicalize_name
+from packaging.version import InvalidVersion, Version
 
 _BLOCK_RE = re.compile(
     r"^#\s*///\s*script\s*\n"
@@ -63,7 +63,10 @@ def _parse_script(script_path: Path) -> tuple[str, list[Requirement]]:
         try:
             reqs.append(Requirement(raw.strip()))
         except InvalidRequirement as e:
-            print(json.dumps({"error": f"Invalid dependency '{raw}': {e}"}), file=sys.stderr)
+            print(
+                json.dumps({"error": f"Invalid dependency '{raw}': {e}"}),
+                file=sys.stderr,
+            )
             sys.exit(1)
 
     return metadata.get("requires-python", ""), reqs
@@ -87,7 +90,7 @@ def _parse_specifier(val: object) -> SpecifierSet:
 
     try:
         return SpecifierSet(spec_str)
-    except Exception:
+    except Exception:  # noqa: BLE001
         return SpecifierSet()
 
 
@@ -117,6 +120,10 @@ def _parse_pip_json(text: str) -> dict[str, SpecifierSet]:
     pip list returns ``[{"name": "foo", "version": "1.2.3"}, ...]``.
     We build an exact ``==version`` specifier so _needs_install can do
     proper version comparison.
+
+    conda-forge git-describe versions (e.g. libwinpthread
+    ``12.0.0.r4.gg4f2fc60ca``) are valid in ``pixi list --json`` but not
+    PEP 440. Treat those as unconstrained rather than failing the resolve.
     """
     try:
         entries = json.loads(text)
@@ -129,7 +136,7 @@ def _parse_pip_json(text: str) -> dict[str, SpecifierSet]:
         version = entry.get("version", "")
         if not name:
             continue
-        spec = SpecifierSet(f"=={version}") if version else SpecifierSet()
+        spec = _parse_specifier(f"=={version}") if version else SpecifierSet()
         managed[canonicalize_name(name)] = spec
     return managed
 
@@ -147,8 +154,7 @@ def _parse_pixi_toml(text: str) -> dict[str, SpecifierSet]:
         for name, val in data.get(section, {}).items()
     }
 
-    if "python" in managed:
-        del managed["python"]
+    managed.pop("python", None)
     return managed
 
 
@@ -220,6 +226,6 @@ def main(script_path: Path, stdin_content: str) -> None:
 
     try:
         main(path, stdin_content)
-    except Exception as e:
+    except Exception as e:  # noqa: BLE001
         print(json.dumps({"error": str(e)}), file=sys.stderr)
         sys.exit(1)

tests/DevTools.Execution.Tests/PipEnvironmentTests.cs

@@ -128,6 +128,31 @@ public async Task Parser_WithPixiListJson_SkipsAlreadyInstalled()
         Assert.Contains(packages, p => p.StartsWith("requests", StringComparison.OrdinalIgnoreCase));
     }
 
+    [Fact]
+    public async Task Parser_WithCondaGitDescribeVersion_DoesNotFail()
+    {
+        // conda-forge win-64 libwinpthread uses git-describe versions that are
+        // not PEP 440; pixi list --json still emits them. Parser must skip the
+        // invalid specifier and keep resolving PEP 723 deps.
+        var listJson = """
+            [
+              {"name": "libwinpthread", "version": "12.0.0.r4.gg4f2fc60ca", "kind": "conda"},
+              {"name": "packaging", "version": "26.0", "kind": "conda"}
+            ]
+            """;
+
+        var result = await RunParserAsync(Path.Combine(FixturesPath, "pep723_sample.py"), listJson);
+        Assert.Equal(0, result.ExitCode);
+        Assert.DoesNotContain("Invalid specifier", result.Stderr, StringComparison.Ordinal);
+
+        var packages = JsonDocument.Parse(result.Stdout).RootElement
+            .GetProperty("to_install").EnumerateArray()
+            .Select(e => e.GetString()!).ToList();
+
+        Assert.DoesNotContain(packages, p => p.Equals("packaging", StringComparison.OrdinalIgnoreCase));
+        Assert.Contains(packages, p => p.StartsWith("requests", StringComparison.OrdinalIgnoreCase));
+    }
+
     [Fact]
     public async Task Parser_WithEmptyPixiListJson_InstallsAllPep723Deps()
     {