Remove unnecessary const assignment for directly returned EventBasedError

Co-authored-by: MiYanni <[email protected]>

Author: Copilot

sample/yarn.lock

@@ -406,8 +406,7 @@ To keep your .NET version up to date, please reconnect to the internet at your s
             if (error?.error?.message !== undefined) // DotnetAcquisitionError is a bad but common pattern where the error is included in the thrown object
             {
                 // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-                const convertedError = new EventBasedError(error?.constructor?.name ?? 'DotnetAcquisitionError', error?.error?.message, error?.stack);
-                return convertedError;
+                return new EventBasedError(error?.constructor?.name ?? 'DotnetAcquisitionError', error?.error?.message, error?.stack);
             }
 
             // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access