fix(codex): inherit method docblocks from ancestor classes through intermediate classes

closes #766

Signed-off-by: azjezz <[email protected]>

Author: azjezz

crates/analyzer/tests/cases/issue_766.php

@@ -0,0 +1,74 @@
+<?php
+
+declare(strict_types=1);
+
+class Entity
+{
+    public function __construct(
+        public int $id,
+    ) {}
+}
+
+/**
+ * @template TEntity of object
+ */
+abstract class AbstractVoter
+{
+    /** @param TEntity $object */
+    abstract protected function voteOnEntity(object $object): bool;
+}
+
+/**
+ * @extends AbstractVoter<Entity>
+ */
+abstract class AbstractEntityVoter extends AbstractVoter
+{
+}
+
+final class Voter1 extends AbstractEntityVoter
+{
+    #[Override]
+    protected function voteOnEntity(object $object): bool
+    {
+        return $object->id === 1;
+    }
+}
+
+final class Voter1Inherit extends AbstractEntityVoter
+{
+    /**
+     * @inheritDoc
+     */
+    #[Override]
+    protected function voteOnEntity(object $object): bool
+    {
+        return $object->id === 1;
+    }
+}
+
+/**
+ * @extends AbstractVoter<Entity>
+ */
+final class Voter2 extends AbstractVoter
+{
+    #[Override]
+    protected function voteOnEntity(object $object): bool
+    {
+        return $object->id === 2;
+    }
+}
+
+/**
+ * @extends AbstractVoter<Entity>
+ */
+final class Voter2Inherit extends AbstractVoter
+{
+    /**
+     * @inheritDoc
+     */
+    #[Override]
+    protected function voteOnEntity(object $object): bool
+    {
+        return $object->id === 2;
+    }
+}

crates/analyzer/tests/mod.rs

@@ -471,6 +471,7 @@ test_case!(issue_755);
 test_case!(issue_756);
 test_case!(issue_764);
 test_case!(issue_765);
+test_case!(issue_766);
 
 #[test]
 fn test_all_test_cases_are_ran() {

crates/codex/src/populator/docblock.rs

@@ -191,6 +191,12 @@ pub fn inherit_method_docblocks(codebase: &mut CodebaseMetadata) {
                 }
             }
 
+            if parent_method_id.is_none() {
+                if let Some((declaring_class, method_id)) = method_ids.iter().next() {
+                    parent_method_id = Some((*declaring_class, *method_id.get_method_name()));
+                }
+            }
+
             if let Some((parent_class, parent_method)) = parent_method_id {
                 inheritance_work.push((*class_name, *method_name, parent_class, parent_method));
             }