fix(compat): set class .name on OAuth subclass factory to match v1 named declarations

Author: felixweinberger

packages/core/src/errors/oauthErrorsCompat.ts

@@ -16,13 +16,15 @@ type OAuthErrorSubclass = {
 };
 
 function sub(code: OAuthErrorCode, name: string): OAuthErrorSubclass {
-    return class extends OAuthError {
+    const Sub = class extends OAuthError {
         static errorCode = code as string;
         constructor(message: string, errorUri?: string) {
             super(code, message, errorUri);
             this.name = name;
         }
     };
+    Object.defineProperty(Sub, 'name', { value: name, configurable: true });
+    return Sub;
 }
 
 /* eslint-disable @typescript-eslint/naming-convention */

packages/core/test/errors/compat.test.ts

@@ -43,6 +43,13 @@ describe('v1-compat error aliases', () => {
         spy.mockRestore();
     });
 
+    it('OAuth subclass class .name matches v1 named-declaration behavior', () => {
+        expect(InvalidTokenError.name).toBe('InvalidTokenError');
+        const e = new InvalidTokenError('expired');
+        expect(e.name).toBe('InvalidTokenError');
+        expect(e.constructor.name).toBe('InvalidTokenError');
+    });
+
     it('subclass static errorCode and toResponseObject() match v1 wire format', () => {
         expect(ServerError.errorCode).toBe('server_error');
         const e = new ServerError('boom');