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": "5.28.4"
+    "undici": "6.19.7"
   },
   "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(
@@ -196,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": "7.32.0",
-    "eslint-plugin-unused-imports": "2.0.0",
+    "eslint": "8.56.0",
+    "eslint-plugin-unused-imports": "3.2.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 not included in
+  // Identifiers that are prefixed with an underscore are 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 `proctected`. To generate a protected
- * constructor, specify `@hideconstructor proctected`.
+ * have `private` visibility` or `protected`. To generate a protected
+ * constructor, specify `@hideconstructor protected`.
  *
  * 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 the specified relative path.
+ * `reference` at 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 the specified relative path.
+ * `reference` at 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 abosulute path.
+ * specified absolute 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-genrated IDs.
+ * auto-generated 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 the specified relative path.
+ * `reference` at 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 the specified relative path.
+ * `reference` at 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 abosulute path.
+ * specified absolute 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-genrated IDs.
+ * auto-generated 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/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 Programatically 
+### Use the Tool Programmatically 
 ### `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 = "absoulte/path/to/firebase/module";
+  const moduleLocation: string = "absolute/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 defintion file.`
+      `${path} doesn't exist. Please provide a valid path to the bundle definition file.`
     );
   }
 

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

@@ -31,7 +31,7 @@ const projectRoot = dirname(resolve(__dirname, '../../package.json'));
 /**
  * Support Command Line Options
  * -- inputModule (optional) : can be left unspecified which results in running analysis on all exp modules.
- *            can specify one to many module names seperated by space.
+ *            can specify one to many module names separated by space.
  *            eg: --inputModule "@firebase/functions-exp" "firebase/auth-exp"
  *
  * -- inputDtsFile (optional) : adhoc support. Specify a path to dts file. Must enable -- inputBundleFile if this flag is specified.

repo-scripts/size-analysis/package.json

@@ -35,14 +35,14 @@
     "typescript": "4.7.4",
     "terser": "5.16.1",
     "yargs": "17.7.2",
-    "@firebase/util": "1.9.6",
+    "@firebase/util": "1.9.7",
     "gzip-size": "6.0.0",
     "glob": "7.2.3"
   },
   "license": "Apache-2.0",
   "devDependencies": {
     "@firebase/logger": "0.4.2",
-    "@firebase/app": "0.10.3"
+    "@firebase/app": "0.10.10"
   },
   "repository": {
     "directory": "repo-scripts/size-analysis",

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

@@ -66,21 +66,21 @@ describe('extractExports', () => {
     expect(extractedDeclarations.variables).to.include.members([
       'basicVarDeclarationExport',
       'basicVarStatementExport',
-      'reExportVarStatmentExport'
+      'reExportVarStatementExport'
     ]);
   });
   it('test re-exported variable extractions from same module - named re-exports', () => {
     expect(extractedDeclarations.variables).to.include.members([
       'basicVarDeclarationExportFar',
       'basicVarStatementExportFar',
-      'reExportVarStatmentExportFar'
+      'reExportVarStatementExportFar'
     ]);
   });
   it('test re-exported variable extractions from same module - * re-exports', () => {
     expect(extractedDeclarations.variables).to.include.members([
       'basicVarDeclarationExportBar',
       'basicVarStatementExportBar',
-      'reExportVarStatmentExportBar'
+      'reExportVarStatementExportBar'
     ]);
   });
 
@@ -296,7 +296,7 @@ describe('test dedup helper function', () => {
 });
 
 describe('test replaceAll helper function', () => {
-  it('test replaceAll with multiple occurences of an element', () => {
+  it('test replaceAll with multiple occurrences of an element', () => {
     const memberList: MemberList = {
       functions: ['aFunc', 'aFunc', 'bFunc', 'cFunc'],
       classes: ['aClass', 'bClass', 'aClass', 'cClass'],
@@ -320,7 +320,7 @@ describe('test replaceAll helper function', () => {
     ).to.equal(2);
   });
 
-  it('test replaceAll with single occurence of an element', () => {
+  it('test replaceAll with single occurrence of an element', () => {
     const memberList: MemberList = {
       functions: ['aFunc', 'aFunc', 'bFunc', 'cFunc'],
       classes: ['aClass', 'bClass', 'aClass', 'cClass'],
@@ -344,7 +344,7 @@ describe('test replaceAll helper function', () => {
     ).to.equal(1);
   });
 
-  it('test replaceAll with zero occurence of an element', () => {
+  it('test replaceAll with zero occurrence of an element', () => {
     const memberList: MemberList = {
       functions: ['aFunc', 'aFunc', 'bFunc', 'cFunc'],
       classes: ['aClass', 'bClass', 'aClass', 'cClass'],

repo-scripts/size-analysis/test/test-inputs/bar.ts

@@ -18,7 +18,7 @@ import { LogLevel } from '@firebase/logger';
 
 export let basicVarDeclarationExportBar: string;
 export const basicVarStatementExportBar = 'basicVarStatementExportBar';
-export const reExportVarStatmentExportBar = LogLevel;
+export const reExportVarStatementExportBar = LogLevel;
 
 export enum BasicEnumExportBar {
   DEBUG = 0,

repo-scripts/size-analysis/test/test-inputs/far.ts

@@ -18,7 +18,7 @@ import { LogLevel } from '@firebase/logger';
 
 export let basicVarDeclarationExportFar: string;
 export const basicVarStatementExportFar = 'basicVarStatementExportFar';
-export const reExportVarStatmentExportFar = name;
+export const reExportVarStatementExportFar = name;
 
 export enum BasicEnumExportFar {
   DEBUG = 0,

repo-scripts/size-analysis/test/test-inputs/index.ts

@@ -33,11 +33,11 @@ export {
   basicUniqueFuncFar,
   basicVarDeclarationExportFar,
   basicVarStatementExportFar,
-  reExportVarStatmentExportFar
+  reExportVarStatementExportFar
 } from './far';
 export let basicVarDeclarationExport: string;
 export const basicVarStatementExport = 'basicVarStatementExport';
-export const reExportVarStatmentExport = LogLevel1;
+export const reExportVarStatementExport = LogLevel1;
 
 export enum BasicEnumExport {
   DEBUG = 0,