feat(git-provider): Load previous registrations, add debug hook

- add way to redirect debug output
- remove unique constraint on Location

src/ModuleCore/Git/GitManager.cs

@@ -12,18 +12,48 @@ public class GitManager
 	private static readonly Lazy<GitManager> GitManagerInstance = new(() => new GitManager());
 	private readonly ConcurrentDictionary<string, InternalGitRegistration> _registrations;
 	private readonly DatabaseManager _db;
+	private static Action<string>? _debugWriterDelegate;
 
 	private GitManager()
 	{
-		Debug.WriteLine($"{nameof(GitManager)} init");
-
 		_registrations = new ConcurrentDictionary<string, InternalGitRegistration>();
 		_db = new DatabaseManager("git.db");
 
+		InitialiseRegistrations();
+	}
+
+	/// <summary>
+	/// Creates up any database tables and loads all previously saved git registrations.
+	/// </summary>
+	private void InitialiseRegistrations()
+	{
+		_debugWriterDelegate?.Invoke("Initialising GitManager from first run - this should only happen once.");
+
 		_db.InConnection(conn =>
 		{
-			conn.CreateTable<InternalGitRegistration>();
+			var createTableResult = conn.CreateTable<InternalGitRegistration>();
+
+			if (createTableResult == CreateTableResult.Created)
+			{
+				_debugWriterDelegate?.Invoke($"Created table {InternalGitRegistration.TableName}.");
+			}
 		});
+
+		_debugWriterDelegate?.Invoke("Loading previous registrations from database.");
+
+		var registrations = _db.InConnection<List<InternalGitRegistration>>(conn =>
+			conn.Table<InternalGitRegistration>()
+				.ToList()
+		);
+
+		foreach (var internalGitRegistration in registrations)
+		{
+			_debugWriterDelegate?.Invoke($"Loading {internalGitRegistration.Name} ({internalGitRegistration.Id}) from database...");
+			if (!_registrations.TryAdd(internalGitRegistration.Name, internalGitRegistration))
+			{
+				_debugWriterDelegate?.Invoke("...failed to restore - potential duplicate name.");
+			}
+		}
 	}
 
 	public static GitManager Instance => GitManagerInstance.Value;
@@ -55,23 +85,22 @@ public class GitManager
 
 		return _db.InConnection<string>(conn =>
 		{
-			// Query if we already have a registration either by name or location.
+			// Query if we already have a registration either by name. Previously we also checked by location, but I
+			// decided to stick with constraining to the name only, same as the key used for the dictionary.
 			// tbh this is a bit of a janky way to do exists when I have to pass the query in anyway, but I just didn't
 			// want to do null checks and a truthy check so I wrap it in a barely-valuable method.
 			var registrationExists = _db.Exists(
 				$"""
 				 SELECT 1 
 				 FROM {InternalGitRegistration.TableName}
-				 WHERE Name = ? OR
-				       Location = ?
+				 WHERE Name = ?
 				 """,
-				gitRegistration.Name,
-				gitRegistration.Location
+				gitRegistration.Name
 			);
 
 			if (registrationExists)
 			{
-				throw new Exception($"A Git repo is already registered with the name {registrationName} or location {absoluteRepositoryLocation}");
+				throw new Exception($"A Git repo is already registered with the name {registrationName}.");
 			}
 
 			// Insert the new record
@@ -116,6 +145,24 @@ public class GitManager
 		throw new Exception($"No git repo has been registered with the name {registeredName}");
 	}
 
+	/// <summary>
+	/// Registers an output for debug output. <see cref="ClearDebugWriter"/> should be called as soon as the need for output
+	/// is no longer needed.
+	/// </summary>
+	/// <param name="commandRuntime"></param>
+	public static void SetDebugWriter(Action<string> commandRuntime)
+	{
+		_debugWriterDelegate = commandRuntime;
+	}
+
+	/// <summary>
+	/// Clears any output previously registered with <see cref="SetDebugWriter"/>
+	/// </summary>
+	public static void ClearDebugWriter()
+	{
+		_debugWriterDelegate = null;
+	}
+
 	/// <summary>
 	/// Used for internal git registration and handles getting the current branch
 	/// </summary>
@@ -132,7 +179,6 @@ public class GitManager
 		[Indexed(Unique = true)]
 		public string Name { get; set; } = null!;
 
-		[Indexed(Unique = true)]
 		public string Location { get; set; } = null!;
 
 		public string CurrentBranch => GetCurrentBranch();