fix: json should support Path (#1026)

henryiii · web-flow · commit b17113dc9f10 · 2025-10-30T14:28:24.000-04:00
Signed-off-by: Henry Schreiner &lt;henryfs@princeton.edu&gt;
diff --git a/nox/tasks.py b/nox/tasks.py
@@ -314,7 +314,7 @@ def _produce_json_listing(manifest: Manifest, global_config: Namespace) -> None:
                     "call_spec": getattr(session.func, "call_spec", {}),
                 }
             )
-    print(json.dumps(report))
+    print(json.dumps(report, default=str))
 
 
 def honor_list_request(manifest: Manifest, global_config: Namespace) -> Manifest | int:
diff --git a/tests/test_tasks.py b/tests/test_tasks.py
@@ -21,6 +21,7 @@
 import os
 import platform
 import typing
+from pathlib import Path
 from textwrap import dedent
 from unittest import mock
 
@@ -33,7 +34,6 @@
 
 if typing.TYPE_CHECKING:
     from collections.abc import Callable, Generator
-    from pathlib import Path
     from types import ModuleType
 
 RESOURCES = os.path.join(os.path.dirname(__file__), "resources")
@@ -531,7 +531,7 @@ def test_honor_list_json_request(capsys: pytest.CaptureFixture[builtins.str]) ->
                 name="bar",
                 friendly_name="foo",
                 description="simple",
-                func=argparse.Namespace(python="123"),
+                func=argparse.Namespace(python=Path("123")),
                 tags=[],
             ),
             True,

Original file line number	Diff line number	Diff line change
`@@ -314,7 +314,7 @@ def _produce_json_listing(manifest: Manifest, global_config: Namespace) -> None:`
`314`	`314`	`"call_spec": getattr(session.func, "call_spec", {}),`
`315`	`315`	`}`
`316`	`316`	`)`
`317`		`- print(json.dumps(report))`
	`317`	`+ print(json.dumps(report, default=str))`
`318`	`318`
`319`	`319`
`320`	`320`	`def honor_list_request(manifest: Manifest, global_config: Namespace) -> Manifest \| int:`