Add logging to Store calls (#915)

Improve traceability of the sector data store by adding logs for each `Transform` and `Picker`.
dalyIsaac · Jun 27, 2024 · 1ac3bab · 1ac3bab
1 parent b06e7c7
commit 1ac3bab
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/Whim/Context/CoreSavedStateManager.cs b/src/Whim/Context/CoreSavedStateManager.cs
@@ -56,7 +56,11 @@ protected virtual void Dispose(bool disposing)
 			if (disposing)
 			{
 				// dispose managed state (managed objects)
-				SaveState();
+				try
+				{
+					SaveState();
+				}
+				catch (Exception) { }
 			}
 
 			_disposedValue = true;

diff --git a/src/Whim/Store/Store.cs b/src/Whim/Store/Store.cs
@@ -67,6 +67,8 @@ public Result<TResult> Dispatch<TResult>(Transform<TResult> transform)
 		{
 			return Task.Run(() =>
 			{
+				Logger.Debug($"Entering task, executing transform {transform}");
+
 				try
 				{
 					_lock.EnterWriteLock();
@@ -80,6 +82,7 @@ public Result<TResult> Dispatch<TResult>(Transform<TResult> transform)
 			}).Result;
 		}
 
+		Logger.Debug($"Executing transform {transform}");
 		return DispatchFn(transform);
 	}
 
@@ -93,6 +96,8 @@ public TResult Pick<TResult>(Picker<TResult> picker)
 		{
 			return Task.Run(() =>
 			{
+				Logger.Debug($"Entering task, executing picker {picker}");
+
 				try
 				{
 					_lock.EnterReadLock();
@@ -105,6 +110,7 @@ public TResult Pick<TResult>(Picker<TResult> picker)
 			}).Result;
 		}
 
+		Logger.Debug($"Executing picker {picker}");
 		return PickFn(picker);
 	}
 
@@ -117,6 +123,7 @@ public TResult Pick<TResult>(PurePicker<TResult> picker)
 		{
 			return Task.Run(() =>
 			{
+				Logger.Debug($"Entering task, executing picker {picker}");
 				try
 				{
 					_lock.EnterReadLock();
@@ -129,6 +136,7 @@ public TResult Pick<TResult>(PurePicker<TResult> picker)
 			}).Result;
 		}
 
+		Logger.Debug($"Executing picker {picker}");
 		return PurePickFn(picker);
 	}