fix: use modern PEP 723 syntax, fix mistake in name

CHANGELOG.md

@@ -1,7 +1,7 @@
 ## dev
 
 - Allow skipping maintenance tasks during list command
- 
+
 ## 1.4.1
 
 - Set default logging level to WARNING, so debug log messages won't be shown without passing additional flags such as `--verbose`
@@ -15,6 +15,7 @@
 - Add `--quiet` and `--verbose` options for the `pipx` subcommands
 - [docs] Add Scoop installation instructions
 - Add ability to install multiple packages at once
+- Update `pipx run` on scripts using `/// script` and no `run` table following PEP 723 (#1180)
 
 ## 1.3.3
 

src/pipx/commands/run.py

@@ -328,14 +328,28 @@ def _get_requirements_from_script(content: str) -> Optional[List[str]]:
     Supports inline script metadata.
     """
 
-    name = "pyproject"
+    name = "script"
 
     # Windows is currently getting un-normalized line endings, so normalize
     content = content.replace("\r\n", "\n")
 
     matches = [m for m in INLINE_SCRIPT_METADATA.finditer(content) if m.group("type") == name]
 
     if not matches:
+        pyproject_matches = [m for m in INLINE_SCRIPT_METADATA.finditer(content) if m.group("type") == "pyproject"]
+        if pyproject_matches:
+            logger.error(
+                pipx_wrap(
+                    f"""
+                    {hazard}  Using old form of requirements table. Use updated PEP
+                    723 syntax by replacing `# /// pyproject` with `# /// script`
+                    and `run.dependencies` (or `run.requirements`) with
+                    `dependencies`.
+                    """,
+                    subsequent_indent=" " * 4,
+                )
+            )
+            raise ValueError("Old 'pyproject' table found")
         return None
 
     if len(matches) > 1:
@@ -348,7 +362,7 @@ def _get_requirements_from_script(content: str) -> Optional[List[str]]:
     pyproject = tomllib.loads(content)
 
     requirements = []
-    for requirement in pyproject.get("run", {}).get("requirements", []):
+    for requirement in pyproject.get("dependencies", []):
         # Validate the requirement
         try:
             req = Requirement(requirement)

tests/test_run.py

@@ -198,8 +198,8 @@ def test_run_with_requirements(caplog, pipx_temp_env, tmp_path):
     script.write_text(
         textwrap.dedent(
             f"""
-                # /// pyproject
-                # run.requirements = ["requests==2.31.0"]
+                # /// script
+                # dependencies = ["requests==2.31.0"]
                 # ///
 
                 # Check requests can be imported
@@ -217,6 +217,32 @@ def test_run_with_requirements(caplog, pipx_temp_env, tmp_path):
     assert out.read_text() == "2.31.0"
 
 
+@mock.patch("os.execvpe", new=execvpe_mock)
+def test_run_with_requirements_old(caplog, pipx_temp_env, tmp_path):
+    script = tmp_path / "test.py"
+    out = tmp_path / "output.txt"
+    script.write_text(
+        textwrap.dedent(
+            f"""
+                # /// pyproject
+                # run.requirements = ["requests==2.31.0"]
+                # ///
+
+                # Check requests can be imported
+                import requests
+                # Check dependencies of requests can be imported
+                import certifi
+                # Check the installed version
+                from pathlib import Path
+                Path({repr(str(out))}).write_text(requests.__version__)
+            """
+        ).strip(),
+        encoding="utf-8",
+    )
+    with pytest.raises(ValueError):
+        run_pipx_cli_exit(["run", script.as_uri()])
+
+
 @mock.patch("os.execvpe", new=execvpe_mock)
 def test_run_with_args(caplog, pipx_temp_env, tmp_path):
     script = tmp_path / "test.py"
@@ -241,8 +267,8 @@ def test_run_with_requirements_and_args(caplog, pipx_temp_env, tmp_path):
     script.write_text(
         textwrap.dedent(
             f"""
-                # /// pyproject
-                # run.requirements = ["packaging"]
+                # /// script
+                # dependencies = ["packaging"]
                 # ///
                 import packaging
                 import sys
@@ -261,8 +287,8 @@ def test_run_with_invalid_requirement(capsys, pipx_temp_env, tmp_path):
     script.write_text(
         textwrap.dedent(
             """
-                # /// pyproject
-                # run.requirements = ["this is an invalid requirement"]
+                # /// script
+                # dependencies = ["this is an invalid requirement"]
                 # ///
                 print()
             """