Fixed normalize_source_lines messing with the indentation of method (#198)

Author: scganterh

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## dev
+
+- Fixed `normalize_source_lines()` messing with the indentation of methods with decorators that have parameters starting with `def`.
+
 ## 1.14.0
 
 - Added `typehints_defaults` config option allowing to automatically annotate parameter defaults.

src/sphinx_autodoc_typehints/__init__.py

@@ -176,7 +176,7 @@ def remove_prefix(text: str, prefix: str) -> str:
 
     # Find the line and line number containing the function definition
     for i, l in enumerate(lines):
-        if l.lstrip().startswith("def"):
+        if l.lstrip().startswith("def "):
             idx = i
             whitespace_separator = "def"
             break

tests/test_sphinx_autodoc_typehints.py

@@ -679,3 +679,39 @@ def __init__(self): pass
     """
 
     assert normalize_source_lines(dedent(source)) == dedent(expected)
+
+
+def test_normalize_source_lines_def_starting_decorator_parameter() -> None:
+    source = """
+    @_with_parameters(
+        _Parameter("self", _Parameter.POSITIONAL_OR_KEYWORD),
+        *_proxy_instantiation_parameters,
+        _project_id,
+        _Parameter(
+            "node_numbers",
+            _Parameter.POSITIONAL_OR_KEYWORD,
+            default=None,
+            annotation=Optional[Iterable[int]],
+        ),
+    )
+    def __init__(bound_args):  # noqa: N805
+        pass
+    """
+
+    expected = """
+    @_with_parameters(
+        _Parameter("self", _Parameter.POSITIONAL_OR_KEYWORD),
+        *_proxy_instantiation_parameters,
+        _project_id,
+        _Parameter(
+            "node_numbers",
+            _Parameter.POSITIONAL_OR_KEYWORD,
+            default=None,
+            annotation=Optional[Iterable[int]],
+        ),
+    )
+    def __init__(bound_args):  # noqa: N805
+        pass
+    """
+
+    assert normalize_source_lines(dedent(source)) == dedent(expected)