[TomasVotruba]

Fixes rectorphp/rector#9959

An invalid docblock like @return array' (trailing quote) parses to an InvalidTagValueNode, so getReturnTagValue() returns null. The rule then falls into the else branch of changeReturnTypeNode(), which keeps adding a fresh @return array<...> tag on every pass while the invalid tag stays in place. The file never reaches a fixed point, so Rector loops until timeout.

Bail out early when the @return tag is invalid, mirroring the existing hasInvalidTag('@return') guard already used in PhpDocTypeChanger::changeReturnType().

Reproducer:

/**
 * @return array'
 */
function test(): array
{
    return ['value'];
}

Added skip_invalid_return_tag.php.inc to cover it.