Remove external const enums

Author: sam-gc

common/api-review/auth-exp.api.md

@@ -20,18 +20,18 @@ export interface ActionCodeInfo {
         multiFactorInfo?: MultiFactorInfo | null;
         previousEmail?: string | null;
     };
-    operation: ActionCodeOperation;
+    operation: string;
 }
 
 // @public
-export const enum ActionCodeOperation {
-    EMAIL_SIGNIN = "EMAIL_SIGNIN",
-    PASSWORD_RESET = "PASSWORD_RESET",
-    RECOVER_EMAIL = "RECOVER_EMAIL",
-    REVERT_SECOND_FACTOR_ADDITION = "REVERT_SECOND_FACTOR_ADDITION",
-    VERIFY_AND_CHANGE_EMAIL = "VERIFY_AND_CHANGE_EMAIL",
-    VERIFY_EMAIL = "VERIFY_EMAIL"
-}
+export const ActionCodeOperation: {
+    EMAIL_SIGNIN: string;
+    PASSWORD_RESET: string;
+    RECOVER_EMAIL: string;
+    REVERT_SECOND_FACTOR_ADDITION: string;
+    VERIFY_AND_CHANGE_EMAIL: string;
+    VERIFY_EMAIL: string;
+};
 
 // @public
 export interface ActionCodeSettings {
@@ -56,7 +56,7 @@ export class ActionCodeURL {
     readonly code: string;
     readonly continueUrl: string | null;
     readonly languageCode: string | null;
-    readonly operation: ActionCodeOperation;
+    readonly operation: string;
     static parseLink(link: string): ActionCodeURL | null;
     readonly tenantId: string | null;
 }
@@ -239,9 +239,9 @@ export class FacebookAuthProvider extends BaseOAuthProvider {
 }
 
 // @public
-export const enum FactorId {
-    PHONE = "phone"
-}
+export const FactorId: {
+    PHONE: string;
+};
 
 // @public
 export function fetchSignInMethodsForEmail(auth: Auth, email: string): Promise<string[]>;
@@ -324,19 +324,19 @@ export function multiFactor(user: User): MultiFactorUser;
 
 // @public
 export interface MultiFactorAssertion {
-    readonly factorId: FactorId;
+    readonly factorId: string;
 }
 
 // @public
 export interface MultiFactorError extends AuthError {
-    readonly operationType: OperationType;
+    readonly operationType: string;
 }
 
 // @public
 export interface MultiFactorInfo {
     readonly displayName?: string | null;
     readonly enrollmentTime: string;
-    readonly factorId: FactorId;
+    readonly factorId: string;
     readonly uid: string;
 }
 
@@ -405,11 +405,11 @@ export function onAuthStateChanged(auth: Auth, nextOrObserver: NextOrObserver<Us
 export function onIdTokenChanged(auth: Auth, nextOrObserver: NextOrObserver<User>, error?: ErrorFn, completed?: CompleteFn): Unsubscribe;
 
 // @public
-export const enum OperationType {
-    LINK = "link",
-    REAUTHENTICATE = "reauthenticate",
-    SIGN_IN = "signIn"
-}
+export const OperationType: {
+    LINK: string;
+    REAUTHENTICATE: string;
+    SIGN_IN: string;
+};
 
 // @public
 export function parseActionCodeURL(link: string): ActionCodeURL | null;
@@ -502,20 +502,17 @@ export interface PopupRedirectResolver {
 export const prodErrorMap: AuthErrorMap;
 
 // @public
-export const enum ProviderId {
-    // @internal (undocumented)
-    ANONYMOUS = "anonymous",
-    // @internal (undocumented)
-    CUSTOM = "custom",
-    FACEBOOK = "facebook.com",
-    // @internal (undocumented)
-    FIREBASE = "firebase",
-    GITHUB = "github.com",
-    GOOGLE = "google.com",
-    PASSWORD = "password",
-    PHONE = "phone",
-    TWITTER = "twitter.com"
-}
+export const ProviderId: {
+    ANONYMOUS: string;
+    CUSTOM: string;
+    FACEBOOK: string;
+    FIREBASE: string;
+    GITHUB: string;
+    GOOGLE: string;
+    PASSWORD: string;
+    PHONE: string;
+    TWITTER: string;
+};
 
 // @public
 export interface ReactNativeAsyncStorage {
@@ -583,17 +580,16 @@ export function setPersistence(auth: Auth, persistence: Persistence): Promise<vo
 export function signInAnonymously(auth: Auth): Promise<UserCredential>;
 
 // @public
-export const enum SignInMethod {
-    // @internal (undocumented)
-    ANONYMOUS = "anonymous",
-    EMAIL_LINK = "emailLink",
-    EMAIL_PASSWORD = "password",
-    FACEBOOK = "facebook.com",
-    GITHUB = "github.com",
-    GOOGLE = "google.com",
-    PHONE = "phone",
-    TWITTER = "twitter.com"
-}
+export const SignInMethod: {
+    ANONYMOUS: string;
+    EMAIL_LINK: string;
+    EMAIL_PASSWORD: string;
+    FACEBOOK: string;
+    GITHUB: string;
+    GOOGLE: string;
+    PHONE: string;
+    TWITTER: string;
+};
 
 // @public
 export function signInWithCredential(auth: Auth, credential: AuthCredential): Promise<UserCredential>;
@@ -630,7 +626,7 @@ export class TwitterAuthProvider extends BaseOAuthProvider {
 }
 
 // @public
-export function unlink(user: User, providerId: ProviderId): Promise<User>;
+export function unlink(user: User, providerId: string): Promise<User>;
 
 export { Unsubscribe }
 
@@ -677,7 +673,7 @@ export interface User extends UserInfo {
 
 // @public
 export interface UserCredential {
-    operationType: OperationType;
+    operationType: string;
     providerId: string | null;
     user: User;
 }

packages-exp/auth-exp/index.node.ts

@@ -41,6 +41,7 @@ FetchProvider.initialize(
 
 // Core functionality shared by all clients
 export * from './src';
+export { FactorId, ProviderId, SignInMethod, OperationType, ActionCodeOperation } from './src/model/enum_maps';
 
 export function getAuth(app: FirebaseApp): Auth {
   const provider = _getProvider(app, 'auth-exp');

packages-exp/auth-exp/index.ts

@@ -33,12 +33,6 @@ import { Auth } from './src/model/public_types';
 
 // Public types
 export {
-  // Enums
-  ActionCodeOperation,
-  FactorId,
-  OperationType,
-  ProviderId,
-  SignInMethod,
   // Interfaces
   ActionCodeInfo,
   ActionCodeSettings,
@@ -79,6 +73,9 @@ export {
   Unsubscribe
 } from './src/model/public_types';
 
+// Helper maps (not used internally)
+export { FactorId, ProviderId, SignInMethod, OperationType, ActionCodeOperation } from './src/model/enum_maps';
+
 // Core functionality shared by all clients
 export * from './src';
 

packages-exp/auth-exp/src/core/action_code_url.ts

@@ -107,7 +107,7 @@ export class ActionCodeURL {
    * The action performed by the email action link. It returns from one of the types from
    * {@link ActionCodeInfo}
    */
-  readonly operation: ActionCodeOperation;
+  readonly operation: string;
   /**
    * The tenant ID of the email action link. Null if the email action is from the parent project.
    */

packages-exp/auth-exp/src/core/user/link_unlink.ts

@@ -38,7 +38,7 @@ import { getModularInstance } from '@firebase/util';
  */
 export async function unlink(
   user: User,
-  providerId: ProviderId
+  providerId: string
 ): Promise<User> {
   const userInternal = getModularInstance(user) as UserInternal;
   await _assertLinkedStatus(true, userInternal, providerId);

packages-exp/auth-exp/src/model/enum_maps.ts

@@ -0,0 +1,95 @@
+/**
+ * An enum of factors that may be used for multifactor authentication.
+ *
+ * @public
+ */
+ export const FactorId = {
+  /** Phone as second factor */
+  PHONE: 'phone'
+};
+
+/**
+ * Enumeration of supported providers.
+ *
+ * @public
+ */
+ export const ProviderId = {
+  /** @internal */
+  ANONYMOUS: 'anonymous',
+  /** @internal */
+  CUSTOM: 'custom',
+  /** Facebook provider ID */
+  FACEBOOK: 'facebook.com',
+  /** @internal */
+  FIREBASE: 'firebase',
+  /** GitHub provider ID */
+  GITHUB: 'github.com',
+  /** Google provider ID */
+  GOOGLE: 'google.com',
+  /** Password provider */
+  PASSWORD: 'password',
+  /** Phone provider */
+  PHONE: 'phone',
+  /** Twitter provider ID */
+  TWITTER: 'twitter.com'
+};
+
+
+/**
+ * Enumeration of supported sign-in methods.
+ *
+ * @public
+ */
+ export const SignInMethod = {
+  /** @internal */
+  ANONYMOUS: 'anonymous',
+  /** Email link sign in method */
+  EMAIL_LINK: 'emailLink',
+  /** Email/password sign in method */
+  EMAIL_PASSWORD: 'password',
+  /** Facebook sign in method */
+  FACEBOOK: 'facebook.com',
+  /** GitHub sign in method */
+  GITHUB: 'github.com',
+  /** Google sign in method */
+  GOOGLE: 'google.com',
+  /** Phone sign in method */
+  PHONE: 'phone',
+  /** Twitter sign in method */
+  TWITTER: 'twitter.com'
+};
+
+/**
+ * Enumeration of supported operation types.
+ *
+ * @public
+ */
+ export const OperationType = {
+  /** Operation involving linking an additional provider to an already signed-in user. */
+  LINK: 'link',
+  /** Operation involving using a provider to reauthenticate an already signed-in user. */
+  REAUTHENTICATE: 'reauthenticate',
+  /** Operation involving signing in a user. */
+  SIGN_IN: 'signIn'
+};
+
+
+/**
+ * An enumeration of the possible email action types.
+ *
+ * @public
+ */
+ export const ActionCodeOperation = {
+  /** The email link sign-in action. */
+  EMAIL_SIGNIN: 'EMAIL_SIGNIN',
+  /** The password reset action. */
+  PASSWORD_RESET: 'PASSWORD_RESET',
+  /** The email revocation action. */
+  RECOVER_EMAIL: 'RECOVER_EMAIL',
+  /** The revert second factor addition email action. */
+  REVERT_SECOND_FACTOR_ADDITION: 'REVERT_SECOND_FACTOR_ADDITION',
+  /** The revert second factor addition email action. */
+  VERIFY_AND_CHANGE_EMAIL: 'VERIFY_AND_CHANGE_EMAIL',
+  /** The email verification action. */
+  VERIFY_EMAIL: 'VERIFY_EMAIL'
+};