Add comment, fix repo storage retrieval

Author: Arlodotexe

src/Commands/Repo/WacsdkRepoGetCommand.cs

@@ -34,14 +34,18 @@ public WacsdkRepoGetCommand(WacsdkCommandConfig config, Option<string> idOption)
         /// <returns>A <see cref="Task"/> representing the asynchronous operation.</returns>
         public async Task InvokeAsync(string id)
         {
-            var thisRepoStorage = (IModifiableFolder)await Config.RepositoryStorage.CreateFolderAsync(id, overwrite: false);
+            var thisRepoStorage = (IModifiableFolder)await Config.RepositoryStorage.GetFirstByNameAsync(id);
 
             Logger.LogInformation($"Getting repo store with ID {id} at {thisRepoStorage.GetType().Name} {thisRepoStorage.Id}");
             var repoSettings = new WacsdkNomadSettings(thisRepoStorage);
             await repoSettings.LoadAsync();
 
             var repositoryContainer = new RepositoryContainer(Config.KuboOptions, Config.Client, repoSettings.ManagedKeys, repoSettings.ManagedUserConfigs, repoSettings.ManagedProjectConfigs, repoSettings.ManagedPublisherConfigs);
 
+            // Why are we doing this here?
+            // - Isn't this a "Get" command? Why the extra logic? What's the intent?
+            // - Does it belong here? Or does it belong in a reusable helper? Does it even need to be reusable, or just isolated for clarity?
+            // - TODO: Assess and analyze intent and options
             var savedMissingFromRepoConfigs = repositoryContainer.PublisherRepository.ManagedConfigs.Where(x => repoSettings.ManagedPublisherConfigs.Any(y => x.RoamingId != y.RoamingId)).ToList();
             var repoMissingFromSavedConfigs = repoSettings.ManagedPublisherConfigs.Where(x => repositoryContainer.PublisherRepository.ManagedConfigs.Any(y => x.RoamingId != y.RoamingId)).ToList();