Fix @throws line number to use actual tag line

The line number assigned to @throws dependencies was incorrectly set to
the function's start line. This fix computes the actual file line of each
@throws tag using PHPStan's startLine attribute (line relative to the
docblock) combined with the docblock's file start line.

Added assertions to verify correct line numbers for all @throws test cases.

https://claude.ai/code/session_01MV9occCkrjhX1JmKUsaYZA

Author: claude

src/Analyzer/Docblock.php

@@ -55,6 +55,30 @@ public function getThrowTagsTypes(): array
         return array_filter($throwTypes);
     }
 
+    /**
+     * @return array<array{type: string, line: int}>
+     */
+    public function getThrowTagsTypesWithLines(): array
+    {
+        $result = [];
+
+        foreach ($this->phpDocNode->getTags() as $tag) {
+            if (!($tag->value instanceof ThrowsTagValueNode)) {
+                continue;
+            }
+
+            $type = $this->getType($tag->value->type);
+
+            if (null === $type) {
+                continue;
+            }
+
+            $result[] = ['type' => $type, 'line' => $tag->getAttribute('startLine') ?? 1];
+        }
+
+        return $result;
+    }
+
     public function getVarTagTypes(): array
     {
         $varTypes = array_map(

src/Analyzer/DocblockTypesResolver.php

@@ -181,23 +181,27 @@ private function resolveReturnValueType(Node $node, Docblock $docblock): void
      */
     private function resolveThrowsValueType(Node $node, Docblock $docblock): void
     {
-        // extract throw types from throw tag
-        $throwValues = $docblock->getThrowTagsTypes();
+        // extract throw types from throw tag, with their docblock-relative line numbers
+        $throwValues = $docblock->getThrowTagsTypesWithLines();
 
         if (empty($throwValues)) {
             return;
         }
 
+        $docComment = $node->getDocComment();
+        $docblockStartLine = null !== $docComment ? $docComment->getStartLine() : $node->getStartLine();
+
         $throwsTypesResolved = [];
 
-        foreach ($throwValues as $throwValue) {
+        foreach ($throwValues as ['type' => $throwValue, 'line' => $tagDocblockLine]) {
             if (str_starts_with($throwValue, '\\')) {
                 $name = new FullyQualified(substr($throwValue, 1));
             } else {
                 $name = $this->resolveName(new Name($throwValue), Stmt\Use_::TYPE_NORMAL);
             }
 
-            $name->setAttribute('startLine', $node->getStartLine());
+            // compute the actual file line: docblock start + tag's line within docblock - 1
+            $name->setAttribute('startLine', $docblockStartLine + $tagDocblockLine - 1);
 
             $throwsTypesResolved[] = $name;
         }

tests/Unit/Analyzer/DocblockTypesResolverTest.php

@@ -90,8 +90,11 @@ public function myMethod2(array $aParam, array $users): array
         self::assertEquals('Application\Model\Product', $dep[4]->getFQCN()->toString());
         self::assertEquals('Domain\Foo\MyOtherClass', $dep[5]->getFQCN()->toString());
         self::assertEquals('Exception', $dep[6]->getFQCN()->toString());
+        self::assertEquals(42, $dep[6]->getLine());
         self::assertEquals('Domain\Foo\FooException', $dep[7]->getFQCN()->toString());
+        self::assertEquals(43, $dep[7]->getLine());
         self::assertEquals('Domain\Foo\BarException', $dep[8]->getFQCN()->toString());
+        self::assertEquals(44, $dep[8]->getLine());
 
         self::assertEquals('Application\Model\User', $dep[9]->getFQCN()->toString());
         self::assertEquals(46, $dep[9]->getLine());

tests/Unit/Analyzer/FileParser/CanParseDocblocksTest.php

@@ -536,10 +536,12 @@ public function method2()
         // Should have 3 dependencies from @throws: FooException, BarException, Exception
         self::assertCount(3, $dependencies);
 
-        $fqcns = array_map(static fn ($dep) => $dep->getFQCN()->toString(), $dependencies);
-        self::assertContains('Domain\FooException', $fqcns);
-        self::assertContains('Domain\BarException', $fqcns);
-        self::assertContains('Exception', $fqcns);
+        self::assertEquals('Domain\FooException', $dependencies[0]->getFQCN()->toString());
+        self::assertEquals(11, $dependencies[0]->getLine());
+        self::assertEquals('Domain\BarException', $dependencies[1]->getFQCN()->toString());
+        self::assertEquals(12, $dependencies[1]->getLine());
+        self::assertEquals('Exception', $dependencies[2]->getFQCN()->toString());
+        self::assertEquals(19, $dependencies[2]->getLine());
     }
 
     public function test_it_collects_throws_tag_with_fully_qualified_names(): void
@@ -573,9 +575,11 @@ public function doSomething()
         // Should have 3 dependencies from @throws
         self::assertCount(3, $dependencies);
 
-        $fqcns = array_map(static fn ($dep) => $dep->getFQCN()->toString(), $dependencies);
-        self::assertContains('Exception', $fqcns);
-        self::assertContains('Domain\FooException', $fqcns);
-        self::assertContains('App\Services\BarException', $fqcns);
+        self::assertEquals('Exception', $dependencies[0]->getFQCN()->toString());
+        self::assertEquals(8, $dependencies[0]->getLine());
+        self::assertEquals('Domain\FooException', $dependencies[1]->getFQCN()->toString());
+        self::assertEquals(9, $dependencies[1]->getLine());
+        self::assertEquals('App\Services\BarException', $dependencies[2]->getFQCN()->toString());
+        self::assertEquals(10, $dependencies[2]->getLine());
     }
 }