[CodeQuality] Skip existing imported or short @ see annotation in AddSeeTestAnnotationRector

[samsonasik]

Fixes rectorphp/rector#9875

[samsonasik]

Fixed 🎉 /cc @Brainshaker95

[samsonasik]

@TomasVotruba ready 👍

[TomasVotruba]

Thanks

[Copilot]

🟢 Approval recommended

The change is narrowly scoped to @see resolution, and the added fixtures cover the newly supported cases to prevent regressions.

Pull request overview

Updates AddSeeTestAnnotationRector to correctly detect already-present @see tags when they reference a class using a short name (either imported via use or resolved from the current namespace), preventing duplicate @see additions and aligning behavior with rectorphp/rector#9875.

Changes:

• Inject ClassAnnotationMatcher and use it to resolve @see tag values to fully qualified class names before checking existence via ReflectionProvider.
• Add fixtures covering “short @see in same namespace” and “imported short @see” cases.
• Add a source class used by the imported-@see fixture.

File summaries

File	Description
rules/CodeQuality/Rector/Class_/AddSeeTestAnnotationRector.php	Resolves short @see values to FQCNs before deciding to skip adding an annotation.
rules-tests/CodeQuality/Rector/Class_/AddSeeTestAnnotationRector/Source/SkipExistingImportedSeeTest.php	Adds a resolvable class for the imported-@see fixture scenario.
rules-tests/CodeQuality/Rector/Class_/AddSeeTestAnnotationRector/Fixture/skip_existing_short_see.php.inc	Fixture ensuring a short @see in the same namespace prevents adding another @see.
rules-tests/CodeQuality/Rector/Class_/AddSeeTestAnnotationRector/Fixture/skip_existing_imported_see.php.inc	Fixture ensuring an imported short @see prevents adding another @see.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.