Ensure initialization of GitRepositoryManager before doing plugin operation

Thanks for your detailed explanation.

In my usecase I just need a list of all available Git repos, I do not need to react on changes at runtime.

Is there an event when the full initialization is done (probably also a more global one)? A very ugly workaround would also be to delay accessing GitRepoManager by a hardcoded number of seconds...

And when/where do you need this list? Is it a toolwindow that need to show some information about them, or something like that?

Because if it is (just for example) something like AnAction which shows a dialog when called, then you don't need the list of repositories on startup, and can get the list when the action is called.

At any case, maybe it will be enough for you to get the list of Git roots (as VirtualFiles), and not GitRepository objects? Then you can subscribe to directoryMappingsChanged() and update the list of every mapping change. But, as you saw above, it still is not available on the startup immediately.

I need it for loading a list in a tool window (I load all open reviews from Gerrit for repositories which are available in current project).

And plain roots as VirtualFiles are not enough, because you need to access GitRemotes, right?

I think the right thing to do is to subscribe to GitRepository.GIT_REPO_CHANGE events. After all, the toolwindow should be updated, if the user adds a new Git root to the project, or removes one, shouldn't it? 

(It should also update if a user removes a Git root from the project, and GIT_REPO_CHANGE won't be fired in this case: you'll need directoryMappingsChange() to handle removal then).

I have implemented something which basically does the work based on your input. 

List<GitRepository> repositories = GitUtil.getRepositoryManager(project).getRepositories();
if (!repositories.isEmpty()) {
    reloadChanges(project, false); // calls async method
}
GitRepositoryChangeListener gitRepositoryChangeListener = new GitRepositoryChangeListener() {
    @Override
    public void repositoryChanged(@NotNull GitRepository gitRepository) {
        reloadChanges(project, false); // calls async method
    }
};
project.getMessageBus().connect().subscribe(GitRepository.GIT_REPO_CHANGE, gitRepositoryChangeListener);

The 'if' is required because it is possible that the event got fired before my listener is subscribed.

Is there an easy way to debounce the 'reloadChanges' call in your API? It is an expensive REST call which should not be executed many times when many repos are in one project.

What do you mean by "debounce"? The call will happen on almost every change in .git in all repositories. If you need to call it only for new repositories, just store a list of GitRepositories and check if gitRepository in the method parameter is new.

With "debounce" I mean something like that: https://lodash.com/docs/#debounce

I.e. when a project contains 4 Git repos, my listener is called 4 times (which is fine of course). But I do not want to execute 4 REST HTTP requests in row. I would do it after the last of these 4 events in row has been fired. If there is no easy way to do that with your API, I could try to do something like that: https://stackoverflow.com/a/20978973

Then maybe you should try the second approach that I've mentioned above: listen to directoryMappingsChanged(), transfer execution to another thread and call getRepositoryForRoot() for every repository, which will force their initialization right away.

This debouncing looks like a hack for this task, since it relies only on time, and not on the number of roots in the project.

Thank you very much! This is the solution I've implemented based on your proposals: https://github.com/uwolfer/gerrit-intellij-plugin/commit/ef256e28aea0412cb7bc8f42965e09549ec60bcd

Hi! 

I think it's better to use VCS_REPOSITORY_MAPPING_UPDATED - a special topic for VcsRepositoryMappingListener.

Subscription to this event means that Repositories has been changed (mappings changes and Repository instances already created if needed).

Thanks Nadya, that works fine.