Checking if all files are being committed

Author: StanleyGoldman

src/GitHub.Api/Git/GitClient.cs

@@ -62,6 +62,9 @@ ITask<string> Commit(string message, string body,
         ITask<string> Add(IList<string> files,
             IOutputProcessor<string> processor = null);
 
+        ITask<string> Add(GitAddTask.AddFileOption addFileOption,
+            IOutputProcessor<string> processor = null);
+
         ITask<string> Remove(IList<string> files,
             IOutputProcessor<string> processor = null);
 
@@ -262,11 +265,31 @@ public ITask<string> Commit(string message, string body,
                 .Configure(processManager);
         }
 
+        public ITask<string> Add(GitAddTask.AddFileOption addFileOption, IOutputProcessor<string> processor = null)
+        {
+            return new GitAddTask(addFileOption, cancellationToken, processor)
+                .Configure(processManager);
+        }
+
         public ITask<string> Add(IList<string> files,
             IOutputProcessor<string> processor = null)
         {
-            return new GitAddTask(files, cancellationToken, processor)
-                .Configure(processManager);
+            GitAddTask last = null;
+            foreach (var batch in files.Spool(5000))
+            {
+                var current = new GitAddTask(batch, cancellationToken, processor).Configure(processManager);
+                if (last == null)
+                {
+                    last = current;
+                }
+                else
+                {
+                    last.Then(current);
+                    last = current;
+                }
+            }
+
+            return last;
         }
 
         public ITask<string> Remove(IList<string> files,

src/GitHub.Api/Git/Tasks/GitAddTask.cs

@@ -1,15 +1,21 @@
+using System;
 using System.Collections.Generic;
 using System.Threading;
 
 namespace GitHub.Unity
 {
     class GitAddTask : ProcessTask<string>
     {
+        public enum AddFileOption
+        {
+            All,
+            CurrentDirectory
+        }
+
         private readonly string arguments;
 
-        public GitAddTask(IEnumerable<string> files,
-            CancellationToken token, IOutputProcessor<string> processor = null)
-            : base(token, processor ?? new SimpleOutputProcessor())
+        public GitAddTask(IEnumerable<string> files, CancellationToken token, 
+            IOutputProcessor<string> processor = null) : base(token, processor ?? new SimpleOutputProcessor())
         {
             Guard.ArgumentNotNull(files, "files");
 
@@ -22,6 +28,25 @@ public GitAddTask(IEnumerable<string> files,
             }
         }
 
+        public GitAddTask(AddFileOption addFileOption, CancellationToken token,
+            IOutputProcessor<string> processor = null) : base(token, processor ?? new SimpleOutputProcessor())
+        {
+            arguments = "add ";
+
+            switch (addFileOption)
+            {
+                case AddFileOption.All:
+                    arguments += "-A";
+                    break;
+
+                case AddFileOption.CurrentDirectory:
+                    arguments += ".";
+                    break;
+
+                default: throw new ArgumentOutOfRangeException(nameof(addFileOption), addFileOption, null);
+            }
+        }
+
         public override string Name { get { return "git add"; } }
         public override string ProcessArguments { get { return arguments; } }
         public override TaskAffinity Affinity { get { return TaskAffinity.Exclusive; } }

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/ChangesView.cs

@@ -189,9 +189,23 @@ private void Commit()
             // Do not allow new commits before we have received one successful update
             busy = true;
 
-            var files = Enumerable.Range(0, tree.Entries.Count).Where(i => tree.CommitTargets[i].All).Select(i => tree.Entries[i].Path).ToArray();
+            var files = Enumerable.Range(0, tree.Entries.Count)
+                .Where(i => tree.CommitTargets[i].All)
+                .Select(i => tree.Entries[i].Path)
+                .ToArray();
 
-            GitClient.Add(files)
+            ITask<string> addTask;
+
+            if (files.Length == tree.Entries.Count)
+            {
+                addTask = GitClient.Add(GitAddTask.AddFileOption.All);
+            }
+            else
+            {
+                addTask = GitClient.Add(files);
+            }
+
+            addTask
                 .Then(GitClient.Commit(commitMessage, commitBody))
                 .Then(GitClient.Status())
                 .FinallyInUI((b, exception) =>