Handle PR feedback

src/NexusMods.Collections/InstallCollectionJob.cs

@@ -214,7 +214,7 @@ await Parallel.ForEachAsync(archive.Children, async (child, token) =>
 
         // If we have any binary patching to do, then we'll do that now.
         if (file.Mod.Patches.Any()) 
-            await CreatePatches(file.Mod, archive, collectionArchive, hashes);
+            await PatchFiles(file.Mod, archive, collectionArchive, hashes);
 
         using var tx = Connection.BeginTransaction();
 
@@ -269,7 +269,7 @@ await Parallel.ForEachAsync(archive.Children, async (child, token) =>
     /// <summary>
     /// This will go through and generate all the patch files for the given archive based on the mod's patches.
     /// </summary>
-    private async Task CreatePatches(Mod modInfo, LibraryArchive.ReadOnly modArchive, LibraryArchive.ReadOnly collectionArchive, 
+    private async Task PatchFiles(Mod modInfo, LibraryArchive.ReadOnly modArchive, LibraryArchive.ReadOnly collectionArchive, 
         ConcurrentDictionary<Md5HashValue, HashMapping> hashes)
     {
         // Index all the files in the collection zip file and the mod archive by their paths so we can find them easily.

src/NexusMods.Collections/Verbs.cs

@@ -20,8 +20,10 @@ internal static class Verbs
 {
     internal static IServiceCollection AddCollectionVerbs(this IServiceCollection collection) =>
         collection
-            .AddVerb(() => InstallCollection)
-            .AddVerb(() => GatherCollectionDefinitions);
+#if DEBUG
+            .AddVerb(() => GatherCollectionDefinitions)
+#endif
+            .AddVerb(() => InstallCollection);
 
 
     [Verb("install-collection", "Installs a collection into the given loadout")]
@@ -49,6 +51,11 @@ private static async Task<int> InstallCollection([Injected] IRenderer renderer,
         return 0;
     }
 
+    /// <summary>
+    /// This verb is only available in DEBUG builds, and is used to get large numbers of collection.json files. The code exists here incase we need
+    /// this behavior again in the future. Disabled in release builds to prevent people from running it without knowing what it does. 
+    /// </summary>
+    /// <returns></returns>
     [Verb("gather-collection-definitions", "Downloads all the collection definitions for a given game, and extracts them to a folder")]
     private static async Task<int> GatherCollectionDefinitions([Injected] IRenderer renderer,
         [Option("g", "game", "Game to gather collection definitions for")] IGame game,