refactor: use result type 📦

packages/app/src/hooks/endpoint.ts

@@ -554,10 +554,16 @@ export const useEndpoint = (): UseEndpointReturn => {
               error: await getHTTPError(response),
             };
           }
-          globalThis.location.href = extractAuthorizationUrl(
+          const authorizationUrlResult = extractAuthorizationUrl(
             request.operation.responses,
             await response.json()
           );
+          if (authorizationUrlResult.isFailure()) {
+            return {
+              error: authorizationUrlResult.value,
+            };
+          }
+          globalThis.location.href = authorizationUrlResult.value;
         }
       } catch (e: unknown) {
         remove(KEY.OAUTH_ENDPOINT_ID);

packages/app/src/utils/oas/oauth.ts

@@ -1,20 +1,32 @@
+import {
+  Success,
+  Failure,
+  EndpointOAuthIllegalResponseError,
+  Result,
+} from '~/errors';
 import { Responses } from '~/types/oas';
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export const extractAuthorizationUrl = (responses: Responses, json: any) => {
+export const extractAuthorizationUrl = (
+  responses: Responses,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  json: any
+): Result<string, EndpointOAuthIllegalResponseError> => {
   const response = responses['200'];
   if (!response || !response.content?.['application/json']) {
-    return null;
+    return new Failure(new EndpointOAuthIllegalResponseError());
   }
   const schema = response.content['application/json'].schema;
   if (!schema.properties) {
-    return null;
+    return new Failure(new EndpointOAuthIllegalResponseError());
   }
+
   const uriKey = Object.keys(schema.properties).find((key) => {
     const property = schema.properties?.[key];
     return property?.type === 'string' && property.format === 'uri';
   });
-  if (uriKey) {
-    return json[uriKey];
+  if (!uriKey) {
+    return new Failure(new EndpointOAuthIllegalResponseError());
   }
+
+  return new Success(json[uriKey]);
 };