Fix Python failure handling in Pythonwin/pywin/framework/scriptutils.py

CHANGES.txt

@@ -13,6 +13,7 @@ https://mhammond.github.io/pywin32_installers.html .
 
 Coming in build 311, as yet unreleased
 --------------------------------------
+* Fixed Python failure handling in Pythonwin and improve error message (#2319, @Avasam)
 * Fixed a regression that broke special __dunder__ methods with CoClass. (#1870, #2493, @Avasam, @geppi)
 
 Build 310, released 2025/03/16

Pythonwin/pywin/framework/scriptutils.py

@@ -2,6 +2,8 @@
 Various utilities for running/importing a script
 """
 
+from __future__ import annotations
+
 import bdb
 import linecache
 import os
@@ -547,14 +549,13 @@ def RunTabNanny(filename):
     data = newout.getvalue()
     if data:
         try:
-            lineno = data.split()[1]
-            lineno = int(lineno)
+            lineno = int(data.split()[1])
             _JumpToPosition(filename, lineno)
             try:  # Try and display whitespace
                 GetActiveEditControl().SCISetViewWS(1)
             except:
                 pass
-            win32ui.SetStatusText("The TabNanny found trouble at line %d" % lineno)
+            win32ui.SetStatusText(f"The TabNanny found trouble at line {lineno}")
         except (IndexError, TypeError, ValueError):
             print("The tab nanny complained, but I can't see where!")
             print(data)
@@ -602,20 +603,19 @@ def JumpToDocument(fileName, lineno=0, col=1, nChars=0, bScrollToTop=0):
     return view
 
 
-def _HandlePythonFailure(what, syntaxErrorPathName=None):
+def _HandlePythonFailure(what: str, syntaxErrorPathName: str | None = None):
     typ, details, tb = sys.exc_info()
     if isinstance(details, SyntaxError):
+        filename = details.filename
+        if (not filename or filename == "<string>") and syntaxErrorPathName:
+            filename = syntaxErrorPathName
         try:
-            msg, (fileName, line, col, text) = details
-            if (not fileName or fileName == "<string>") and syntaxErrorPathName:
-                fileName = syntaxErrorPathName
-            _JumpToPosition(fileName, line, col)
+            _JumpToPosition(filename, details.lineno, details.offset)
         except (TypeError, ValueError):
-            msg = str(details)
-        win32ui.SetStatusText(f"Failed to {what} - syntax error - {msg}")
+            pass
     else:
         traceback.print_exc()
-        win32ui.SetStatusText(f"Failed to {what} - {details}")
+    win32ui.SetStatusText(f"Failed to {what} - {type(details)} - {details}")
     tb = None  # Clean up a cycle.