PEP 723 support is faulty: uses wrong TOML field name

[flying-sheep]

Describe the bug
#1100 implemented PEP 723 wrong:

pipx/src/pipx/commands/run.py

Line 351 in 5b18ddf

for requirement in pyproject.get("run", {}).get("requirements", []):

this should be "dependencies", not "requirements".

How to reproduce

Create a PEP 723 compliant script:

#!/usr/bin/env -S pipx run
# /// pyproject
# run.dependencies = ["httpx"]
# ///
import httpx

and run it using pipx run ./script.py. You’ll see

Traceback (most recent call last):
  File "<string>", line 11, in <module>
ModuleNotFoundError: No module named 'requests'

Expected behavior

pipx should run it in a temporary environment.

Also the traceback should contain the file name, not <string>

[dukecat0]

Fixed in #1180.

[flying-sheep]

Ah, indeed. I searched the issues, but apparently not the PRs

[dukecat0]

Also the traceback should contain the file name, not <string>

However, I think this issue still exists.

[maxwell-k]

The example in the issue report is unfortunately also out of date the most recent change to PEP 723 was to change:

-# /// pyproject
+# /// script

I don't want to write an example and risk getting it wrong myself, instead I will point to the example in the PEP
https://peps.python.org/pep-0723/#how-to-teach-this

For the background on this change please see https://discuss.python.org/t/how-would-you-like-to-declare-runtime-dependencies-and-python-requirements-for-pep-723/40418/70

[henryiii]

@maxwell-k this is also fixed in #1180.