fix: Do not always add docstrings to attributes

Author: pawamoy

src/griffe_typingdoc/_dynamic.py

@@ -29,8 +29,8 @@ def _doc(name: str, hints: dict[str, Any]) -> str | None:
         return None
 
 
-def _attribute_docs(node: ObjectNode, attr: Attribute) -> str | None:
-    return _doc(attr.name, _hints(node))
+def _attribute_docs(node: ObjectNode, attr: Attribute) -> str:
+    return _doc(attr.name, _hints(node)) or ""
 
 
 def _parameters_docs(node: ObjectNode, func: Function) -> DocstringSectionParameters | None:

src/griffe_typingdoc/_extension.py

@@ -33,8 +33,18 @@ def on_attribute_instance(
         """Post-process Griffe attributes to create their docstring."""
         module = _dynamic if isinstance(node, ObjectNode) else _static
 
+        new_sections = (
+            docstring := module._attribute_docs(node, attr),
+            deprecated_section := module._deprecated_docs(node, attr),
+            raises_section := module._raises_docs(node, attr),
+            warns_section := module._warns_docs(node, attr),
+        )
+
+        if not any(new_sections):
+            return
+
         if not attr.docstring:
-            attr.docstring = Docstring(module._attribute_docs(node, attr), parent=attr)
+            attr.docstring = Docstring(docstring, parent=attr)
 
         sections = attr.docstring.parsed
 

src/griffe_typingdoc/_static.py

@@ -83,7 +83,7 @@ def _set_metadata(metadata: dict[str, Any], data: ExprCall) -> None:
 
 def _metadata(annotation: str | Expr | None) -> dict[str, Any]:
     metadata: dict[str, Any] = {"raises": [], "warns": []}
-    if isinstance(annotation, ExprSubscript) and annotation.left.canonical_path in {
+    if isinstance(annotation, ExprSubscript) and annotation.canonical_path in {
         "typing.Annotated",
         "typing_extensions.Annotated",
     }: