Removed repo-scripts changes

Author: maneesht

repo-scripts/api-documenter/src/documenters/IConfigFile.ts

@@ -19,7 +19,7 @@
 // See LICENSE in the project root for license information.
 
 /**
- * TypeScript interface describing the config schema for toc.yml file format.
+ * Typescript interface describing the config schema for toc.yml file format.
  */
 export interface IConfigTableOfContents {
   /**

repo-scripts/changelog-generator/package.json

@@ -20,7 +20,7 @@
     "@changesets/types": "3.3.0",
     "@changesets/get-github-info": "0.5.2",
     "@types/node": "20.8.10",
-    "undici": "6.19.7"
+    "undici": "5.28.4"
   },
   "license": "Apache-2.0",
   "devDependencies": {

repo-scripts/prune-dts/extract-public-api.ts

@@ -38,7 +38,7 @@ const baseApiExtractorConfigFile: string = path.resolve(
 const reportFolder = path.resolve(__dirname, '../../common/api-review');
 const tmpDir = tmp.dirSync().name;
 
-function writeTypeScriptConfig(packageRoot: string): void {
+function writeTypescriptConfig(packageRoot: string): void {
   const tsConfigJson = {
     extends: path.resolve(packageRoot, './tsconfig.json'),
     include: [path.resolve(packageRoot, './src')],
@@ -125,7 +125,7 @@ function loadApiExtractorConfig(
  * @param packageName - The name of the Firebase package (e.g. "database" or
  * "firestore-lite")
  * @param packageRoot - The root path of the package
- * @param typescriptDtsPath - The .d.ts file generated by the TypeScript
+ * @param typescriptDtsPath - The .d.ts file generated by the Typescript
  * compiler as we transpile our sources
  * @param rollupDtsPath - A "bundled" version of our d.ts files that includes
  * all public and private types
@@ -144,7 +144,7 @@ export async function generateApi(
   publicDtsPath: string
 ): Promise<void> {
   console.log(`Configuring API Extractor for ${packageName}`);
-  writeTypeScriptConfig(packageRoot);
+  writeTypescriptConfig(packageRoot);
   writePackageJson(packageName);
 
   let extractorConfig = loadApiExtractorConfig(
@@ -156,9 +156,7 @@ export async function generateApi(
     /* apiReportEnabled= */ false
   );
   Extractor.invoke(extractorConfig, {
-    localBuild: true,
-    showDiagnostics: true,
-    showVerboseMessages: true
+    localBuild: true
   });
 
   console.log('Generated rollup DTS');
@@ -198,7 +196,7 @@ const argv = yargs
     typescriptDts: {
       type: 'string',
       desc:
-        'The .d.ts file generated by the TypeScript compiler as we transpile ' +
+        'The .d.ts file generated by the Typescript compiler as we transpile ' +
         'our sources',
       require: true
     },

repo-scripts/prune-dts/package.json

@@ -13,8 +13,8 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "eslint": "8.56.0",
-    "eslint-plugin-unused-imports": "3.2.0",
+    "eslint": "7.32.0",
+    "eslint-plugin-unused-imports": "2.0.0",
     "prettier": "2.8.7"
   },
   "repository": {

repo-scripts/prune-dts/prune-dts.ts

@@ -108,7 +108,7 @@ export async function removeUnusedImports(
 
 /** Determines whether the provided identifier should be hidden. */
 function hasPrivatePrefix(name: ts.Identifier): boolean {
-  // Identifiers that are prefixed with an underscore are not included in
+  // Identifiers that are prefixed with an underscore are not not included in
   // the public API.
   return !!name.escapedText?.toString().startsWith('_');
 }
@@ -123,11 +123,11 @@ function isExported(
     typeChecker.getSymbolAtLocation(name)?.declarations ?? [];
 
   // Check is this is a public symbol (e.g. part of the DOM library)
-  const isTypeScriptType = declarations.find(
+  const isTypescriptType = declarations.find(
     d => d.getSourceFile().fileName.indexOf('typescript/lib') != -1
   );
   const isImported = declarations.find(d => ts.isImportSpecifier(d));
-  if (isTypeScriptType || isImported) {
+  if (isTypescriptType || isImported) {
     return true;
   }
 
@@ -141,8 +141,8 @@ function isExported(
 /**
  * Replaces an existing constructor implementation if the constructor is marked
  * with the JSDod tag `@hideconstructor`. The replaced constructor can either
- * have `private` visibility` or `protected`. To generate a protected
- * constructor, specify `@hideconstructor protected`.
+ * have `private` visibility` or `proctected`. To generate a protected
+ * constructor, specify `@hideconstructor proctected`.
  *
  * Returns either the modified constructor or the existing constructor if no
  * modification was needed.

repo-scripts/prune-dts/tests/firestore.input.d.ts

@@ -464,7 +464,7 @@ export declare function collection(
 
 /**
  * Gets a `CollectionReference` instance that refers to a subcollection of
- * `reference` at the specified relative path.
+ * `reference` at the the specified relative path.
  *
  * @param reference - A reference to a collection.
  * @param path - A slash-separated path to a collection.
@@ -482,7 +482,7 @@ export declare function collection(
 
 /**
  * Gets a `CollectionReference` instance that refers to a subcollection of
- * `reference` at the specified relative path.
+ * `reference` at the the specified relative path.
  *
  * @param reference - A reference to a Firestore document.
  * @param path - A slash-separated path to a collection.
@@ -794,7 +794,7 @@ export declare function disableNetwork(
 
 /**
  * Gets a `DocumentReference` instance that refers to the document at the
- * specified absolute path.
+ * specified abosulute path.
  *
  * @param firestore - A reference to the root Firestore instance.
  * @param path - A slash-separated path to a document.
@@ -818,7 +818,7 @@ export declare function doc(
  *
  * @param reference - A reference to a collection.
  * @param path - A slash-separated path to a document. Has to be omitted to use
- * auto-generated IDs.
+ * auto-genrated IDs.
  * @param pathSegments - Additional path segments that will be applied relative
  * to the first argument.
  * @throws If the final path has an odd number of segments and does not point to
@@ -1643,14 +1643,14 @@ export declare interface FirestoreDataConverter<T>
 declare interface FirestoreDataConverter_2<T> {
   /**
    * Called by the Firestore SDK to convert a custom model object of type `T`
-   * into a plain JavaScript object (suitable for writing directly to the
+   * into a plain Javascript object (suitable for writing directly to the
    * Firestore database). Used with {@link setData}, {@link WriteBatch#set}
    * and {@link Transaction#set}.
    */
   toFirestore(modelObject: T): DocumentData;
   /**
    * Called by the Firestore SDK to convert a custom model object of type `T`
-   * into a plain JavaScript object (suitable for writing directly to the
+   * into a plain Javascript object (suitable for writing directly to the
    * Firestore database). Used with {@link setData}, {@link WriteBatch#set}
    * and {@link Transaction#set} with `merge:true` or `mergeFields`.
    */

repo-scripts/prune-dts/tests/firestore.output.d.ts

@@ -154,7 +154,7 @@ export declare function collection(
 ): CollectionReference<DocumentData>;
 /**
  * Gets a `CollectionReference` instance that refers to a subcollection of
- * `reference` at the specified relative path.
+ * `reference` at the the specified relative path.
  *
  * @param reference - A reference to a collection.
  * @param path - A slash-separated path to a collection.
@@ -171,7 +171,7 @@ export declare function collection(
 ): CollectionReference<DocumentData>;
 /**
  * Gets a `CollectionReference` instance that refers to a subcollection of
- * `reference` at the specified relative path.
+ * `reference` at the the specified relative path.
  *
  * @param reference - A reference to a Firestore document.
  * @param path - A slash-separated path to a collection.
@@ -262,7 +262,7 @@ export declare function disableNetwork(
 ): Promise<void>;
 /**
  * Gets a `DocumentReference` instance that refers to the document at the
- * specified absolute path.
+ * specified abosulute path.
  *
  * @param firestore - A reference to the root Firestore instance.
  * @param path - A slash-separated path to a document.
@@ -285,7 +285,7 @@ export declare function doc(
  *
  * @param reference - A reference to a collection.
  * @param path - A slash-separated path to a document. Has to be omitted to use
- * auto-generated IDs.
+ * auto-genrated IDs.
  * @param pathSegments - Additional path segments that will be applied relative
  * to the first argument.
  * @throws If the final path has an odd number of segments and does not point to

repo-scripts/prune-dts/tests/resolves-generics-through-inheritence.input.d.ts

@@ -0,0 +1,25 @@
+/**
+ * @license
+ * Copyright 2020 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+declare class A<T> {
+  a: T;
+}
+export class B<T> extends A<string> {
+  b: T;
+}
+export class C<T> extends A<T> {}
+export {};

repo-scripts/size-analysis/README.md

@@ -28,13 +28,13 @@ $firebase-js-sdk/repo-scripts/size-analysis  `ts-node-script cli.ts  -o <path to
 $firebase-js-sdk/repo-scripts/size-analysis  `ts-node-script cli.ts --if <path to dts file> --ib <path to bundle file> -o <path to output FILE>`
 
 
-### Use the Tool Programmatically 
+### Use the Tool Programatically 
 ### `async generateReportForModule(moduleLocation: string): Promise<Report>`
 #### This function generates size analysis report for the given module specified by the `moduleLocation` argument.
 #### `@param moduleLocation: an absolute path to location of a firebase module`
 ```
 try {
-  const moduleLocation: string = "absolute/path/to/firebase/module";
+  const moduleLocation: string = "absoulte/path/to/firebase/module";
   const report: Report = await generateReportForModule(moduleLocation);
   console.log(report);
 

repo-scripts/size-analysis/bundle-analysis.ts

@@ -102,7 +102,7 @@ export async function run({
 function loadBundleDefinitions(path: string): BundleDefinition[] {
   if (!existsSync(path)) {
     throw new Error(
-      `${path} doesn't exist. Please provide a valid path to the bundle definition file.`
+      `${path} doesn't exist. Please provide a valid path to the bundle defintion file.`
     );
   }