Fix a typo in docs

Simplify add_reachability_report function
Remove type hints for symbols.py

Signed-off-by: ziad hany <ziadhany2016@gmail.com>

Author: ziadhany

scanpipe/pipelines/analyze_symbols_reachability.py

@@ -49,9 +49,12 @@ class SymbolReachability(Pipeline):
 
     The analysis checks if vulnerable symbols are defined, imported, called, or
     exactly match a fingerprint within the project files. The results, including
-    evidence and a reachability status (REACHABLE, POTENTIALLY_REACHABLE, or
+    evidence and a reachability status (REACHABLE, UNKNOWN, or
     NOT_REACHABLE), are stored in the `extra_data` of the matching resources
     under the `symbols_reachability` key.
+
+    Finally, a summary report is generated for each vulnerability
+    advisory and saved as a JSON output file.
     """
 
     download_inputs = False

scanpipe/pipes/reachability.py

@@ -538,9 +538,7 @@ def add_reachability_report(resource, commit_hash, vcs_url, new_report):
 
     old_reports = resource.extra_data.get("symbols_reachability", [])
     for old_report in old_reports:
-        actual_report = old_report.get("symbols_reachability", old_report)
-
-        patch_info = actual_report.get("patch", {})
+        patch_info = old_report.get("patch", {})
         old_commit_hash = patch_info.get("commit_hash")
         old_vcs_url = patch_info.get("vcs_url")
 
@@ -550,7 +548,7 @@ def add_reachability_report(resource, commit_hash, vcs_url, new_report):
                 replaced = True
             # Skip old duplicate
         else:
-            cleaned_reports.append(actual_report)
+            cleaned_reports.append(old_report)
 
     if not replaced:
         cleaned_reports.append(new_report)