Fix crash when inspect moduel returns invalid Python code

Signed-off-by: Bernát Gábor <gaborjbernat@gmail.com>

Author: gaborbernat

CHANGELOG.md

@@ -1,18 +1,24 @@
 # Changelog
 
+## dev
+
+- Fix crash when the `inspect` module returns an invalid python syntax source
+
 ## 1.14.1
 
-- Fixed `normalize_source_lines()` messing with the indentation of methods with decorators that have parameters starting with `def`.
+- Fixed `normalize_source_lines()` messing with the indentation of methods with decorators that have parameters starting
+  with `def`.
 - Handle `ValueError` or `TypeError` being raised when signature of an object cannot be determined
-- Fix `KeyError` being thrown when argument is not documented (e.g. `cls` argument for class methods, and `self` for methods)
+- Fix `KeyError` being thrown when argument is not documented (e.g. `cls` argument for class methods, and `self` for
+  methods)
 
 ## 1.14.0
 
 - Added `typehints_defaults` config option allowing to automatically annotate parameter defaults.
 
 ## 1.13.1
 
-- Fixed ``NewType`` inserts a reference as first argument instead of a string
+- Fixed `NewType` inserts a reference as first argument instead of a string
 
 ## 1.13.0
 

README.md

@@ -63,7 +63,7 @@ The following configuration options are accepted:
   `:rtype:` directive if no existing `:rtype:` is found.
 - `typehints_defaults` (default: `None`): If `None`, defaults are not added. Otherwise adds a default annotation:
 
-  - `'comma'` adds it after the type, changing Sphinx’ default look to “**param** (*int*, default: `1`) -- text”.
+  - `'comma'` adds it after the type, changing Sphinx’ default look to “**param** (_int_, default: `1`) -- text”.
   - `'braces'` adds `(default: ...)` after the type (useful for numpydoc like styles).
   - `'braces-after'` adds `(default: ...)` at the end of the parameter documentation text instead.
 

src/sphinx_autodoc_typehints/__init__.py

@@ -315,7 +315,7 @@ def _one_child(module: Module) -> stmt | None:
     try:
         code = textwrap.dedent(normalize_source_lines(inspect.getsource(obj)))
         obj_ast = ast.parse(code, **parse_kwargs)  # type: ignore # dynamic kwargs
-    except (OSError, TypeError):
+    except (OSError, TypeError, SyntaxError):
         return {}
 
     obj_ast = _one_child(obj_ast)

tests/test_sphinx_autodoc_typehints.py

@@ -35,6 +35,7 @@
 from sphobjinv import Inventory
 
 from sphinx_autodoc_typehints import (
+    backfill_type_hints,
     format_annotation,
     get_annotation_args,
     get_annotation_class_name,
@@ -751,3 +752,12 @@ def test_bound_class_method(method: FunctionType) -> None:
     )
     app: Sphinx = create_autospec(Sphinx, config=config)
     process_docstring(app, "class", method.__qualname__, method, None, [])
+
+
+def test_syntax_error_backfill() -> None:
+    # Regression test for #188
+    # fmt: off
+    func = (  # Note: line break here is what previously led to SyntaxError in process_docstring
+        lambda x: x)
+    # fmt: on
+    backfill_type_hints(func, "func")