From 544b0c5cd004c786ac1df4783b0f99f98a7f3b94 Mon Sep 17 00:00:00 2001
From: Timothy Baldridge <tbaldridge@gmail.com>
Date: Wed, 2 Aug 2023 04:53:16 -0600
Subject: [PATCH] Update comments to handle feedback

---
 src/NexusMods.DataModel/Loadouts/LoadoutSyncronizer.cs | 3 ++-
 src/NexusMods.DataModel/ToolManager.cs                 | 4 ++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/NexusMods.DataModel/Loadouts/LoadoutSyncronizer.cs b/src/NexusMods.DataModel/Loadouts/LoadoutSyncronizer.cs
index d77218a1ac..a25ced921b 100644
--- a/src/NexusMods.DataModel/Loadouts/LoadoutSyncronizer.cs
+++ b/src/NexusMods.DataModel/Loadouts/LoadoutSyncronizer.cs
@@ -322,7 +322,8 @@ public async ValueTask<IngestPlan> MakeIngestPlan(Loadout loadout, Func<Absolute
                 {
                     await EmitIngestReplacePlan(plan, planFile, existing);
                 }
-                // TODO: Fix this
+                // TODO: Fix this, it doesn't contain support for IGeneratedFile, 
+                // once we re-design apply/ingest this should be implemented
                 continue;
             }
 
diff --git a/src/NexusMods.DataModel/ToolManager.cs b/src/NexusMods.DataModel/ToolManager.cs
index 6bf5e02d6a..334ee97bac 100644
--- a/src/NexusMods.DataModel/ToolManager.cs
+++ b/src/NexusMods.DataModel/ToolManager.cs
@@ -69,6 +69,10 @@ public async Task<Loadout> RunTool(ITool tool, Loadout loadout, ModId? generated
             generatedFilesMod = mod.Id;
         }
 
+        // We don't yet properly support ingesting data. The issue is if a bad apply occurs, the ingest can 
+        // delete files we don't yet have a way of recreating. Also we have no way to create branches, roll back the 
+        // ingest, etc. in the loadout. So for now we just don't ingest.
+        
         //var ingestPlan = await _loadoutSynchronizer.MakeIngestPlan(loadout, _ => generatedFilesMod.Value, token);
         //return await _loadoutSynchronizer.Ingest(ingestPlan, $"Updating {tool.Name} Generated Files");
         return loadout;