I'm only wondering about that part inObjectRepositoryTypeProvider :

public PhpType getType(PsiElement e) {// ...        if(!(e instanceof MethodReference)                /*|| !PhpElementsUtil.isMethodWithFirstStringOrFieldReference(e, "getRepository") */        ) {            return null;        }// ...}

Maybe removing that commented line as I did is too permissive and can cause performance issues? If so, we could narrow the matching methods to "get*" for example...

@theredled yes this mentioned method check is for performance and also to not overflow the phpstorm signatures for every method. By removing this simply we are saying we are responsible for all methods.

We need some filtering here. But for now i just see to support it via a configuration.

Another way would be by using the "php7 returns" and index every file to get "getRepo" automaticcally. But as the index is build inside "getType" we dont have access to it at this point. So this must then be cached. So would be a bigger task.