feature: Stack Trace Updates (#43)

* Added functionality for Exceptions to Rethrow, such that their stack traces do not get eaten by Unwrap().

* CSharpier

ChainSharp.Tests.Integration/IntegrationTests/WorkflowTests.cs

@@ -1,3 +1,4 @@
+using ChainSharp.Exceptions;
 using ChainSharp.Step;
 using ChainSharp.Tests.Examples.Brewery;
 using ChainSharp.Tests.Examples.Brewery.Steps.Bottle;
@@ -319,6 +320,24 @@ public async Task TestChainWithShortCircuitStaysLeft()
         result.Should().NotBeNull();
     }
 
+    [Theory]
+    public async Task TestWithException()
+    {
+        // Arrange
+        var workflow = new ChainTestWithException();
+
+        // Act
+        // Assert
+
+        Assert.ThrowsAsync<WorkflowException>(async () => await workflow.Run(Unit.Default));
+    }
+
+    private class ThrowsStep : Step<Unit, Unit>
+    {
+        public override Task<Unit> Run(Unit input) =>
+            throw new WorkflowException("This is a workflow exception.");
+    }
+
     private class OuterProperty
     {
         public string OuterString { get; set; }
@@ -616,4 +635,10 @@ Ingredients input
                 .Resolve();
         }
     }
+
+    private class ChainTestWithException : Workflow<Unit, Unit>
+    {
+        protected override async Task<Either<Exception, Unit>> RunInternal(Unit input) =>
+            Activate(input).Chain<ThrowsStep>().Resolve();
+    }
 }

ChainSharp/Step/Step.cs

@@ -1,3 +1,4 @@
+using System.Reflection;
 using LanguageExt;
 using LanguageExt.UnsafeValueAccess;
 
@@ -20,8 +21,19 @@ public async Task<Either<Exception, TOut>> RailwayStep(Either<Exception, TIn> pr
         }
         catch (Exception e)
         {
-            Console.WriteLine($"Found Exception ({e.InnerException ?? e})");
-            return e.InnerException ?? e;
+            var messageField = typeof(Exception).GetField(
+                "_message",
+                BindingFlags.Instance | BindingFlags.NonPublic
+            );
+
+            if (messageField != null)
+                messageField.SetValue(
+                    e,
+                    $"{{ \"Step\": \"{GetType().Name}\", \"Type\": \"{e.GetType().Name}\", \"Message\": \"{e.Message}\" }}"
+                );
+
+            Console.WriteLine($"Step: ({GetType().Name}) failed with Exception: ({e.Message})");
+            return e;
         }
     }
 }

ChainSharp/Workflow/Workflow.cs

@@ -1,5 +1,6 @@
 using ChainSharp.Extensions;
 using LanguageExt;
+using LanguageExt.UnsafeValueAccess;
 
 namespace ChainSharp.Workflow;
 
@@ -12,7 +13,15 @@ public abstract partial class Workflow<TInput, TReturn> : IWorkflow<TInput, TRet
     private TReturn ShortCircuitValue { get; set; } = default!;
     private bool ShortCircuitValueSet { get; set; } = false;
 
-    public async Task<TReturn> Run(TInput input) => await RunEither(input).Unwrap();
+    public async Task<TReturn> Run(TInput input)
+    {
+        var resultEither = await RunEither(input);
+
+        if (resultEither.IsLeft)
+            resultEither.Swap().ValueUnsafe().Rethrow();
+
+        return resultEither.Unwrap();
+    }
 
     public Task<Either<Exception, TReturn>> RunEither(TInput input)
     {

Directory.Build.props

@@ -1,5 +1,5 @@
 <Project>
     <PropertyGroup>
-        <Version>2.1.5</Version>
+        <Version>2.2.0</Version>
     </PropertyGroup>
 </Project>