Clean up compat docgen script

Author: hsubox76

package.json

@@ -39,9 +39,7 @@
     "test:setup": "node tools/config.js",
     "test:saucelabs": "node scripts/run_saucelabs.js",
     "docgen": "ts-node-script scripts/docgen/docgen.ts",
-    "docgen:compat": "yarn docgen:compat:js; yarn docgen:compat:node",
-    "docgen:compat:js": "node scripts/docgen-compat/generate-docs.js --api js",
-    "docgen:compat:node": "node scripts/docgen-compat/generate-docs.js --api node",
+    "docgen:compat": "node scripts/docgen-compat/generate-docs.js --api js",
     "lint": "lerna run --scope @firebase/* lint",
     "lint:fix": "lerna run --scope @firebase/* lint:fix",
     "size-report": "ts-node-script scripts/size_report/report_binary_size.ts",

packages/app-check/src/index.ts

@@ -1,6 +1,6 @@
 /**
  * The Firebase App Check Web SDK.
- * 
+ *
  * @remarks
  * Firebase App Check does not work in Node.js using `ReCaptchaV3Provider` or
  * `ReCaptchaEnterpriseProvider`, but can be used in Node.js if you use

packages/auth/src/model/public_types.ts

@@ -193,7 +193,7 @@ export interface Auth {
    * This makes it easy for a user signing in to specify whether their session should be
    * remembered or not. It also makes it easier to never persist the Auth state for applications
    * that are shared by other users or have sensitive data.
-   * 
+   *
    * This method has no effect in Node.js.
    *
    * @example

packages/auth/src/platform_browser/mfa/assertions/phone.ts

@@ -83,7 +83,7 @@ export class PhoneMultiFactorGenerator {
 
   /**
    * Provides a {@link PhoneMultiFactorAssertion} to confirm ownership of the phone second factor.
-   * 
+   *
    * @remarks
    * This method is not available in Node.js.
    *

packages/auth/src/platform_browser/popup_redirect.ts

@@ -189,7 +189,7 @@ class BrowserPopupRedirectResolver implements PopupRedirectResolverInternal {
 /**
  * An implementation of {@link PopupRedirectResolver} suitable for browser
  * based applications.
- * 
+ *
  * @remarks
  * This method is not available in Node.js.
  *

packages/auth/src/platform_browser/providers/phone.ts

@@ -50,7 +50,7 @@ import { ProviderId, SignInMethod } from '../../model/enums';
  * const phoneCredential = PhoneAuthProvider.credential(verificationId, verificationCode);
  * const userCredential = await signInWithCredential(auth, phoneCredential);
  * ```
- * 
+ *
  * @public
  */
 export class PhoneAuthProvider {

packages/auth/src/platform_browser/recaptcha/recaptcha_verifier.ts

@@ -43,7 +43,7 @@ type TokenCallback = (token: string) => void;
 
 /**
  * An {@link https://www.google.com/recaptcha/ | reCAPTCHA}-based application verifier.
- * 
+ *
  * @remarks
  * `RecaptchaVerifier` is not available in Node.js.
  *

packages/auth/src/platform_browser/strategies/phone.ts

@@ -81,7 +81,7 @@ class ConfirmationResultImpl implements ConfirmationResult {
  * This function can work on other platforms that do not support the
  * {@link RecaptchaVerifier} (like React Native), but you need to use a
  * third-party {@link ApplicationVerifier} implementation.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example
@@ -117,7 +117,7 @@ export async function signInWithPhoneNumber(
 
 /**
  * Links the user account with the given phone number.
- * 
+ *
  * @remarks
  * This method is not available in Node.js.
  *
@@ -149,7 +149,7 @@ export async function linkWithPhoneNumber(
  *
  * @remarks
  * Use before operations such as {@link updatePassword} that require tokens from recent sign-in attempts.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @param user - The user.
@@ -257,7 +257,7 @@ export async function _verifyPhoneNumber(
 
 /**
  * Updates the user's phone number.
- * 
+ *
  * @remarks
  * This method is not available in Node.js.
  *

packages/auth/src/platform_browser/strategies/popup.ts

@@ -62,7 +62,7 @@ export const _POLL_WINDOW_CLOSE_TIMEOUT = new Delay(2000, 10000);
  * @remarks
  * If succeeds, returns the signed in user along with the provider's credential. If sign in was
  * unsuccessful, returns an error object containing additional information about the error.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example
@@ -110,7 +110,7 @@ export async function signInWithPopup(
  * @remarks
  * If the reauthentication is successful, the returned result will contain the user and the
  * provider's credential.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example
@@ -153,7 +153,7 @@ export async function reauthenticateWithPopup(
  *
  * @remarks
  * If the linking is successful, the returned result will contain the user and the provider's credential.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example

packages/auth/src/platform_browser/strategies/redirect.ts

@@ -44,7 +44,7 @@ import { getModularInstance } from '@firebase/util';
  * To handle the results and errors for this operation, refer to {@link getRedirectResult}.
  * Follow the {@link https://firebase.google.com/docs/auth/web/redirect-best-practices
  * | best practices} when using {@link signInWithRedirect}.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example
@@ -115,7 +115,7 @@ export async function _signInWithRedirect(
  * To handle the results and errors for this operation, refer to {@link getRedirectResult}.
  * Follow the {@link https://firebase.google.com/docs/auth/web/redirect-best-practices
  * | best practices} when using {@link reauthenticateWithRedirect}.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example
@@ -246,7 +246,7 @@ export async function _linkWithRedirect(
  * @remarks
  * If sign-in succeeded, returns the signed in user. If sign-in was unsuccessful, fails with an
  * error. If no redirect operation was called, returns `null`.
- * 
+ *
  * This method is not available in Node.js.
  *
  * @example

packages/firebase/compat/index.d.ts

@@ -8047,30 +8047,6 @@ declare namespace firebase.storage {
     UNSUPPORTED_ENVIRONMENT = 'unsupported-environment'
   }
 
-  /**
-   * An error returned by the Firebase Storage SDK.
-   */
-  export interface FirebaseStorageError extends FirebaseError {
-    /**
-     * Stores custom error data unique to the `StorageError`.
-     */
-    customData: {
-      serverResponse: string | null;
-    };
-
-    get status(): number;
-    set status(status: number);
-    /**
-     * Compares a `StorageErrorCode` against this error's code, filtering out the prefix.
-     */
-    _codeEquals(code: StorageErrorCode): boolean;
-    /**
-     * Optional response message that was added by the server.
-     */
-    get serverResponse(): null | string;
-    set serverResponse(serverResponse: string | null);
-  }
-
   interface StorageObserver<T> {
     next?: NextFn<T> | null;
     error?: (error: FirebaseStorageError) => void | null;
@@ -8268,6 +8244,30 @@ declare namespace firebase.storage {
      */
     totalBytes: number;
   }
+
+  /**
+   * An error returned by the Firebase Storage SDK.
+   */
+  export interface FirebaseStorageError extends FirebaseError {
+    /**
+     * Stores custom error data unique to the `StorageError`.
+     */
+    customData: {
+      serverResponse: string | null;
+    };
+
+    get status(): number;
+    set status(status: number);
+    /**
+     * Compares a `StorageErrorCode` against this error's code, filtering out the prefix.
+     */
+    _codeEquals(code: StorageErrorCode): boolean;
+    /**
+     * Optional response message that was added by the server.
+     */
+    get serverResponse(): null | string;
+    set serverResponse(serverResponse: string | null);
+  }
 }
 
 declare namespace firebase.firestore {

packages/firestore/src/api/settings.ts

@@ -81,7 +81,7 @@ export interface FirestoreSettings extends LiteSettings {
    * may be removed in a future release. If you find yourself using it to
    * work around a specific network reliability issue, please tell us about
    * it in https://github.com/firebase/firebase-js-sdk/issues/1674.
-   * 
+   *
    * This setting cannot be used in a Node.js environment.
    */
   experimentalForceLongPolling?: boolean;
@@ -93,7 +93,7 @@ export interface FirestoreSettings extends LiteSettings {
    *
    * This setting will likely be enabled by default in future releases and
    * cannot be combined with `experimentalForceLongPolling`.
-   * 
+   *
    * This setting cannot be used in a Node.js environment.
    */
   experimentalAutoDetectLongPolling?: boolean;

scripts/docgen-compat/content-sources/js/toc.yaml

@@ -282,6 +282,8 @@ toc:
     path: /docs/reference/js/v8/firebase.storage.SettableMetadata
   - title: "Storage"
     path: /docs/reference/js/v8/firebase.storage.Storage
+  - title: "StorageErrorCode"
+    path: /docs/reference/js/v8/firebase.storage.StorageErrorCode
   - title: "StorageObserver"
     path: /docs/reference/js/v8/firebase.storage.StorageObserver
   - title: "UploadMetadata"