WIP

Author: schmidt-sebastian

common/api-review/firestore-lite.api.md

@@ -4,10 +4,16 @@
 
 ```ts
 
-// Warning: (ae-forgotten-export) The symbol "A1" needs to be exported by the entry point index.d.ts
+// Warning: (ae-forgotten-export) The symbol "Foo" needs to be exported by the entry point index.d.ts
 //
 // @public (undocumented)
-export class B1 extends A1<string> {
+export class BarPublic extends Foo<number> {
+}
+
+// @public (undocumented)
+export class FooPublic<T> extends Foo<string> {
+    // (undocumented)
+    randomData?: T;
 }
 
 

packages/firestore/lite-types/index.d.ts

@@ -15,9 +15,11 @@
  * limitations under the License.
  */
 
-export declare class B1 {
-  data?: string;
-  foo(data: string): string;
-  foo(data: string): string | void;
+export declare class BarPublic {
+  stringOrNumberData?: number;
+}
+export declare class FooPublic<T> {
+  randomData?: T;
+  stringOrNumberData?: string;
 }
 export {};

packages/firestore/lite/index.ts

@@ -15,12 +15,22 @@
  * limitations under the License.
  */
 
-class A1<T> {
-  // Non-exported class I want to hide
-  data?: T;
-  foo(data: T | null): T | null;
-  foo(data: T): T | void;
-  foo(data: T | null): T | null | void {}
+// class A1<T> {
+//   // Non-exported class I want to hide
+//   data?: T;
+//   foo(data: T | null): T | null;
+//   foo(data: T): T | void;
+//   foo(data: T | null): T | null | void {}
+// }
+//
+// export class B1 extends A1<string> {}
+
+class Foo<T> {
+  stringOrNumberData?: T;
+}
+
+export class FooPublic<T> extends Foo<string> {
+  randomData?: T;
 }
 
-export class B1 extends A1<string> {}
+export class BarPublic extends Foo<number> {}

packages/firestore/scripts/prune-dts.ts

@@ -151,7 +151,7 @@ function prunePrivateImports<
           sourceFile,
           type.expression
         );
-        if (publicSymbol && publicSymbol.name !== currentName) {
+        if (publicSymbol && publicSymbol.name !== currentName && !!type.typeArguments?.length) {
           // If there is a public type that we can refer to, update the import
           // statement to refer to the public type.
           exportedTypes.push(