fix(firestore): remove unknown from modular types

Author: JiriHoffmann

packages/firestore/lib/modular/index.d.ts

@@ -827,12 +827,6 @@ export declare function doc<AppModelType, DbModelType extends DocumentData>(
   ...pathSegments: string[]
 ): DocumentReference<DocumentData, DocumentData>;
 
-export function doc<T>(
-  parent: Firestore | CollectionReference<T> | DocumentReference<unknown>,
-  path?: string,
-  ...pathSegments: string[]
-): DocumentReference;
-
 /**
  * Gets a `CollectionReference` instance that refers to the collection at
  * the specified absolute path.
@@ -905,12 +899,6 @@ export function collection(
   ...pathSegments: string[]
 ): CollectionReference<DocumentData>;
 
-export function collection(
-  parent: Firestore | DocumentReference<unknown> | CollectionReference<unknown>,
-  path: string,
-  ...pathSegments: string[]
-): CollectionReference<DocumentData>;
-
 /**
  *Returns true if the provided references are equal.
  *

packages/firestore/lib/modular/index.js

@@ -45,7 +45,7 @@ export function getFirestore(app, databaseId) {
 }
 
 /**
- * @param {Firestore | CollectionReference | DocumentReference<unknown>} parent
+ * @param {Firestore | CollectionReference | DocumentReference} parent
  * @param {string?} path
  * @param {string?} pathSegments
  * @returns {DocumentReference}
@@ -59,7 +59,7 @@ export function doc(parent, path, ...pathSegments) {
 }
 
 /**
- * @param {Firestore | DocumentReference<unknown> | CollectionReference<unknown>} parent
+ * @param {Firestore | DocumentReference | CollectionReference} parent
  * @param {string} path
  * @param {string?} pathSegments
  * @returns {CollectionReference<DocumentData>}

packages/firestore/lib/modular/query.d.ts

@@ -183,7 +183,7 @@ export function orderBy(
  * @param snapshot - The snapshot of the document to start at.
  * @returns A {@link QueryStartAtConstraint} to pass to `query()`.
  */
-export function startAt(snapshot: DocumentSnapshot<unknown>): QueryStartAtConstraint;
+export function startAt(snapshot: DocumentSnapshot): QueryStartAtConstraint;
 /**
  *
  * Creates a {@link QueryStartAtConstraint} that modifies the result set to
@@ -196,9 +196,7 @@ export function startAt(snapshot: DocumentSnapshot<unknown>): QueryStartAtConstr
  */
 export function startAt(...fieldValues: unknown[]): QueryStartAtConstraint;
 
-export function startAt(
-  ...docOrFields: Array<unknown | DocumentSnapshot<unknown>>
-): QueryStartAtConstraint;
+export function startAt(...docOrFields: Array<unknown | DocumentSnapshot>): QueryStartAtConstraint;
 
 /**
  * Creates a {@link QueryStartAtConstraint} that modifies the result set to