From 6bd631c061d0a8ffad990fffe37f82145a318735 Mon Sep 17 00:00:00 2001
From: Reuben Bond <reuben.bond@gmail.com>
Date: Fri, 3 Nov 2023 16:56:45 -0700
Subject: [PATCH] Avoid hiding the stack of exceptions thrown from user code

---
 .../Runtime/GrainReference.cs                     | 15 ---------------
 src/Orleans.Runtime/Services/GrainService.cs      |  1 -
 test/Orleans.Serialization.UnitTests/Request.cs   | 14 --------------
 3 files changed, 30 deletions(-)

diff --git a/src/Orleans.Core.Abstractions/Runtime/GrainReference.cs b/src/Orleans.Core.Abstractions/Runtime/GrainReference.cs
index 4c2a19a031..93826dcd71 100644
--- a/src/Orleans.Core.Abstractions/Runtime/GrainReference.cs
+++ b/src/Orleans.Core.Abstractions/Runtime/GrainReference.cs
@@ -540,7 +540,6 @@ public void AddInvokeMethodOptions(InvokeMethodOptions options)
         }
 
         /// <inheritdoc/>
-        [DebuggerHidden]
         public abstract ValueTask<Response> Invoke();
 
         /// <inheritdoc/>
@@ -586,7 +585,6 @@ public void AddInvokeMethodOptions(InvokeMethodOptions options)
     [SerializerTransparent]
     public abstract class Request : RequestBase 
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -606,7 +604,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(ValueTask resultTask)
         {
             try
@@ -621,7 +618,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask resultTas
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract ValueTask InvokeInner();
     }
 
@@ -635,7 +631,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask resultTas
     public abstract class Request<TResult> : RequestBase
     {
         /// <inheritdoc/>
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -654,7 +649,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(ValueTask<TResult> resultTask)
         {
             try
@@ -672,7 +666,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask<TResult>
         /// Invokes the request against the target.
         /// </summary>
         /// <returns>The invocation result.</returns>
-        [DebuggerHidden]
         protected abstract ValueTask<TResult> InvokeInner();
     }
 
@@ -686,7 +679,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask<TResult>
     public abstract class TaskRequest<TResult> : RequestBase
     {
         /// <inheritdoc/>
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -706,7 +698,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(Task<TResult> resultTask)
         {
             try
@@ -724,7 +715,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(Task<TResult> resul
         /// Invokes the request against the target.
         /// </summary>
         /// <returns>The invocation result.</returns>
-        [DebuggerHidden]
         protected abstract Task<TResult> InvokeInner();
     }
 
@@ -735,7 +725,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(Task<TResult> resul
     public abstract class TaskRequest : RequestBase
     {
         /// <inheritdoc/>
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -756,7 +745,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(Task resultTask)
         {
             try
@@ -774,7 +762,6 @@ private static async ValueTask<Response> CompleteInvokeAsync(Task resultTask)
         /// Invokes the request against the target.
         /// </summary>
         /// <returns>The invocation result.</returns>
-        [DebuggerHidden]
         protected abstract Task InvokeInner();
     }
 
@@ -791,7 +778,6 @@ protected VoidRequest()
         }
 
         /// <inheritdoc/>
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -808,7 +794,6 @@ public sealed override ValueTask<Response> Invoke()
         /// <summary>
         /// Invokes the request against the target.
         /// </summary>
-        [DebuggerHidden]
         protected abstract void InvokeInner();
     }
 }
diff --git a/src/Orleans.Runtime/Services/GrainService.cs b/src/Orleans.Runtime/Services/GrainService.cs
index 6aa00091af..bb0ca6fe4f 100644
--- a/src/Orleans.Runtime/Services/GrainService.cs
+++ b/src/Orleans.Runtime/Services/GrainService.cs
@@ -39,7 +39,6 @@ protected GrainServiceStatus Status
 
         /// <summary>Only to make Reflection happy. Do not use it in your implementation</summary>
         [System.ComponentModel.EditorBrowsable(System.ComponentModel.EditorBrowsableState.Never)]
-        [System.Diagnostics.DebuggerHidden]
         protected GrainService() : base()
         {
             throw new Exception("This should not be constructed by client code.");
diff --git a/test/Orleans.Serialization.UnitTests/Request.cs b/test/Orleans.Serialization.UnitTests/Request.cs
index 9623ea003a..49114f1b36 100644
--- a/test/Orleans.Serialization.UnitTests/Request.cs
+++ b/test/Orleans.Serialization.UnitTests/Request.cs
@@ -28,7 +28,6 @@ public abstract class UnitTestRequestBase : IInvokable
     [GenerateSerializer]
     public abstract class UnitTestRequest : UnitTestRequestBase
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -48,7 +47,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(ValueTask resultTask)
         {
             try
@@ -63,14 +61,12 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask resultTas
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract ValueTask InvokeInner();
     }
 
     [GenerateSerializer]
     public abstract class UnitTestRequest<TResult> : UnitTestRequestBase
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -89,7 +85,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(ValueTask<TResult> resultTask)
         {
             try
@@ -104,14 +99,12 @@ private static async ValueTask<Response> CompleteInvokeAsync(ValueTask<TResult>
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract ValueTask<TResult> InvokeInner();
     }
 
     [GenerateSerializer]
     public abstract class UnitTestTaskRequest<TResult> : UnitTestRequestBase
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -131,7 +124,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(Task<TResult> resultTask)
         {
             try
@@ -146,14 +138,12 @@ private static async ValueTask<Response> CompleteInvokeAsync(Task<TResult> resul
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract Task<TResult> InvokeInner();
     }
 
     [GenerateSerializer]
     public abstract class UnitTestTaskRequest : UnitTestRequestBase
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -174,7 +164,6 @@ public sealed override ValueTask<Response> Invoke()
             }
         }
 
-        [DebuggerHidden]
         private static async ValueTask<Response> CompleteInvokeAsync(Task resultTask)
         {
             try
@@ -189,14 +178,12 @@ private static async ValueTask<Response> CompleteInvokeAsync(Task resultTask)
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract Task InvokeInner();
     }
 
     [GenerateSerializer]
     public abstract class UnitTestVoidRequest : UnitTestRequestBase
     {
-        [DebuggerHidden]
         public sealed override ValueTask<Response> Invoke()
         {
             try
@@ -211,7 +198,6 @@ public sealed override ValueTask<Response> Invoke()
         }
 
         // Generated
-        [DebuggerHidden]
         protected abstract void InvokeInner();
     }
 }
\ No newline at end of file