refactor: remove MarkCompleted calls - output is flushed immediately

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: thomhurst

src/ModularPipelines/Console/ProgressSession.cs

@@ -174,9 +174,6 @@ public void OnModuleCompleted(ModuleState state, bool isSuccessful)
                 var increment = 100.0 / _totalModuleCount;
                 _totalTask?.Increment(increment);
 
-                // Mark buffer as completed for ordering
-                _coordinator.GetModuleBuffer(state.ModuleType).MarkCompleted();
-
                 // Check if all done
                 if (_completedModuleCount >= _totalModuleCount)
                 {
@@ -216,8 +213,6 @@ public void OnModuleSkipped(ModuleState state)
             var increment = 100.0 / _totalModuleCount;
             _totalTask?.Increment(increment);
 
-            _coordinator.GetModuleBuffer(state.ModuleType).MarkCompleted();
-
             if (_completedModuleCount >= _totalModuleCount)
             {
                 _totalTask?.StopTask();