fix(run): a blank script crashes pdm run --list (#3842)

* fix(run): fix crash when listing blank scripts

* chore(news): rename fragment to the PR number

* fix(run): mark a blank script as <BLANK_SCRIPT> per review

---------

Co-authored-by: VXNCXNX <VXNCXNX@users.noreply.github.com>

Author: VXNCXNX

news/3842.bugfix.md

@@ -0,0 +1 @@
+Don't crash with an `IndexError` when listing scripts and one of them is blank. A blank script is now listed as `<BLANK_SCRIPT>`.

src/pdm/cli/commands/run.py

@@ -151,7 +151,11 @@ def short_description(self) -> str:
             fallback = f" {termui.Emoji.ARROW_SEPARATOR} ".join(self.args)
         else:
             lines = [line.strip() for line in str(self.args).splitlines() if line.strip()]
-            fallback = f"{lines[0]}{termui.Emoji.ELLIPSIS}" if len(lines) > 1 else lines[0]
+            if not lines:
+                # a blank script leaves no lines at all; mark it rather than showing an empty cell
+                fallback = "<BLANK_SCRIPT>"
+            else:
+                fallback = f"{lines[0]}{termui.Emoji.ELLIPSIS}" if len(lines) > 1 else lines[0]
         return self.options.get("help", fallback)
 
 

tests/cli/test_run.py

@@ -630,6 +630,18 @@ def test_run_show_list_of_scripts(project, pdm):
     assert result_lines[4][1:-1].strip() == "test_shell     │ shell     │ shell command"
 
 
+def test_run_show_list_of_scripts_with_blank_script(project, pdm):
+    project.pyproject.settings["scripts"] = {
+        "test_blank": "   ",
+        "test_cmd": "flask db upgrade",
+    }
+    project.pyproject.write()
+    result = pdm(["run", "--list"], obj=project)
+    result_lines = result.output.splitlines()[3:]
+    assert result_lines[0][1:-1].strip() == "test_blank │ cmd  │ <BLANK_SCRIPT>"
+    assert result_lines[1][1:-1].strip() == "test_cmd   │ cmd  │ flask db upgrade"
+
+
 def test_run_show_list_of_scripts_hide_internals(project, pdm):
     project.pyproject.settings["scripts"] = {
         "public": "true",