Please file an issue/bug to track this. We only have the one provider at the moment, so this shouldn't block some testing/selfhosting. But let's not forget to come back to this and fix it.

Adding this as a potential fix. Using an MSBuild task to iterate through all the extensions and populate XAML before it is built. It would be difficult though. Just an idea.

PropName can be used here instead of re-typing the string.

I believe the null checks are used because of iterating through the properties and this needs the latest commit to be not null.

I don't like the duplication of the check.

If LatestCommit returns null for a property, would a second call to LatestCommit return a non-null value? If this is the case the call to LatestCommit can be moved outside the foreach loop.

The null check for LatestCommit still needs to be in each case "If not null, add the property"

Another way to do this is,

1. Put LatestCommit outside of the for loop.
2. For every property use result.Add(propName, latestCommit?.[PropertyName] ?? string.empty)
3. Before returning result remove all entries that contain string.empty.

Why is this in its own scope?

I believe this can be simplified to commits.FirstOrDefault()?.Tree?[relativePath] == null

NIT: commit.Select(x => x.Tree[relativePath]).Where(x +> x is not null)

NIT: currentCommit.Parents.Count() == 0

NIT: repo?.Dispose()

NIT: You can use Microsoft.Windows.System.EnvironmentManager and the EV will be removed when the app is uninstalled.

Why use PInvoke to get the ILocalRepositoryProvider? I believe DevHome has interfaces to get the providers already.

old name? Can the name be changed to something more descriptive?

Considering on line 72 the case is ignored.

I understand. Tracked repositories is a map from a root path to the extension that is tracking the repository. I understand now. Okay.

NIT: change the name to TrackedRepositoriesToExtension, or something similar.

That is why so many other files use pUnk to determing the extension type.

return value.

NIT for the file: Make CLSID the second argument. That is what it is. All other methods need either

1. nothing
2. RootPath
3. RootPath and CLSID.

This keeps the arguments in a consistent order.

you already retrieved TrackedRepositories[rootPath] in TryGetValue. existingExtensionCLSID can be used in this scope.

NVM. This is used to check the existence of the key. In the case. can you use .ContainsKey instead?

NIT: use UTcNow. An edge case can occur if using .Now. For example, Daylight savings time occurred. Or the user moved between times zones.

Is this TODO still relevant?

NIT: repoCollection.Select(x => new RepositoryInformation(x.Key, x.Value))

NIT: GetLocalProvider(rootFolderPath).GetRepository(rootFolderPath)

Is this TODO still relevant?