From d517c3dbeaeee561e8dde1aaa4da24406ca20b00 Mon Sep 17 00:00:00 2001 From: kemal Date: Mon, 7 Sep 2026 14:35:50 +0100 Subject: [PATCH] chore: remove unused oauth authorize result types The AuthorizeResult union and its guards were never wired into the approve flow, so knip reported the file as dead. Reintroduce it alongside the error state that uses it. Co-Authored-By: Claude Opus 5 (1M context) --- .../oauth-apps/oauth-apps-authorize.types.ts | 26 ------------------- 1 file changed, 26 deletions(-) delete mode 100644 apps/studio/data/oauth-apps/oauth-apps-authorize.types.ts diff --git a/apps/studio/data/oauth-apps/oauth-apps-authorize.types.ts b/apps/studio/data/oauth-apps/oauth-apps-authorize.types.ts deleted file mode 100644 index 2d2fa1fa417..00000000000 --- a/apps/studio/data/oauth-apps/oauth-apps-authorize.types.ts +++ /dev/null @@ -1,26 +0,0 @@ -import type { OAuthAppsAuthorizeApproveResponse } from './oauth-apps-authorize-approve-mutation' - -export type ProjectRoleFailure = { - projectRef: string - projectName: string - userRole: string - missingScopes: string[] -} - -export type AuthorizeResult = - | { status: 'success'; data: OAuthAppsAuthorizeApproveResponse } - | { status: 'role_validation_failed'; failures: ProjectRoleFailure[] } - | { status: 'error'; code: string; message: string } - -export const isAuthorizeSuccess = ( - result: AuthorizeResult -): result is Extract => result.status === 'success' - -export const isRoleValidationFailure = ( - result: AuthorizeResult -): result is Extract => - result.status === 'role_validation_failed' - -export const isAuthorizeError = ( - result: AuthorizeResult -): result is Extract => result.status === 'error'