review feedback

etc/firebase-admin.api.md

@@ -400,7 +400,6 @@ export namespace auth {
         tokensValidAfterTime?: string;
         uid: string;
     }
-    {};
 }
 
 // @public (undocumented)

src/auth/auth.ts

@@ -359,7 +359,8 @@ export class BaseAuth<T extends AbstractAuthRequestHandler> implements BaseAuthI
     // but also provides a providerToLink with a 'phone' provider id. In that
     // case, we'll throw an error.
     properties = deepCopy(properties);
-    if (validator.isNonNullObject(properties) && typeof properties.providerToLink !== 'undefined') {
+
+    if (properties?.providerToLink) {
       if (properties.providerToLink.providerId === 'email') {
         if (typeof properties.email !== 'undefined') {
           throw new FirebaseAuthError(
@@ -380,7 +381,7 @@ export class BaseAuth<T extends AbstractAuthRequestHandler> implements BaseAuthI
         delete properties.providerToLink;
       }
     }
-    if (validator.isNonNullObject(properties) && typeof properties.providersToUnlink !== 'undefined') {
+    if (properties?.providersToUnlink) {
       if (properties.providersToUnlink.indexOf('phone') !== -1) {
         // If we've been told to unlink the phone provider both via setting
         // phoneNumber to null *and* by setting providersToUnlink to include

test/unit/auth/auth.spec.ts

@@ -1674,7 +1674,6 @@ AUTH_CONFIGS.forEach((testConfig) => {
           getAccountInfoByUidStub.restore();
         });
 
-
         it('specifying both email and providerId=email should be rejected', () => {
           expect(() => {
             auth.updateUser(uid, {