Fix public typings

Author: andrewbrook

common/api-review/telemetry-react.api.md

@@ -0,0 +1,17 @@
+## API Report File for "@firebase/telemetry-react"
+
+> Do not edit this file. It is a report generated by [API Extractor](https://api-extractor.com/).
+
+```ts
+
+import { FirebaseOptions } from '@firebase/app';
+
+// @public
+export function FirebaseTelemetry({ firebaseOptions }: {
+    firebaseOptions?: FirebaseOptions;
+}): null;
+
+
+// (No @packageDocumentation comment for this package)
+
+```

common/api-review/telemetry.api.md

@@ -6,6 +6,7 @@
 
 import { AnyValueMap } from '@opentelemetry/api-logs';
 import { FirebaseApp } from '@firebase/app';
+import { Instrumentation } from 'next';
 import { LoggerProvider } from '@opentelemetry/sdk-logs';
 
 // @public
@@ -17,13 +18,7 @@ export function flush(telemetry: Telemetry): Promise<void>;
 // @public
 export function getTelemetry(app?: FirebaseApp): Telemetry;
 
-// @public (undocumented)
-export namespace Instrumentation {
-    // Warning: (ae-forgotten-export) The symbol "InstrumentationOnRequestError" needs to be exported by the entry point index.d.ts
-    //
-    // (undocumented)
-    export type onRequestError = InstrumentationOnRequestError;
-}
+export { Instrumentation }
 
 // @public
 export const nextOnRequestError: Instrumentation.onRequestError;

packages/telemetry/api-extractor.json

@@ -1,12 +1,20 @@
 {
   "extends": "../../config/api-extractor.json",
-  // Point it to your entry point d.ts file.
   "mainEntryPointFilePath": "<projectFolder>/dist/index.d.ts",
   "additionalEntryPoints": [
     {
       "modulePath": "react",
       "filePath": "<projectFolder>/dist/react/index.d.ts"
     }
   ],
+  "dtsRollup": {
+    // rollup is not supported when multiple entry points are present.
+    // npm script api-report:* is used to generate dts rollup.
+    "enabled": false
+  },
+  "apiReport": {
+    // apiReport is handled by npm script api-report:*
+    "enabled": false
+  },
   "bundledPackages": ["next"]
 }

packages/telemetry/package.json

@@ -8,7 +8,7 @@
   "browser": "dist/index.esm.js",
   "exports": {
     ".": {
-      "types": "./dist/index.d.ts",
+      "types": "./dist/index-public.d.ts",
       "node": {
         "import": "./dist/node-esm/index.node.esm.js",
         "default": "./dist/index.node.cjs.js"
@@ -20,7 +20,7 @@
       "default": "./dist/index.esm.js"
     },
     "./react": {
-      "types": "./dist/react/index.d.ts",
+      "types": "./dist/react/index-public.d.ts",
       "node": {
         "import": "./dist/react/index.esm.js",
         "default": "./dist/react/index.cjs.js"
@@ -48,8 +48,11 @@
     "test:browser": "karma start",
     "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha **/*.test.ts src/**/*.test.ts --config ../../config/mocharc.node.js",
     "trusted-type-check": "tsec -p tsconfig.json --noEmit",
-    "api-report": "api-extractor run --local --verbose",
-    "typings:public": "node ../../scripts/build/use_typings.js ./dist/index.d.ts"
+    "api-report": "yarn api-report:main && yarn api-report:react && yarn api-report:api-json",
+    "api-report:main": "ts-node-script ../../repo-scripts/prune-dts/extract-public-api.ts --package telemetry --packageRoot . --typescriptDts ./dist/index.d.ts --rollupDts ./dist/private.d.ts --untrimmedRollupDts ./dist/internal.d.ts --publicDts ./dist/index-public.d.ts",
+    "api-report:react": "ts-node-script ../../repo-scripts/prune-dts/extract-public-api.ts --package telemetry-react --packageRoot . --typescriptDts ./dist/react/index.d.ts --rollupDts ./dist/react/private.d.ts --untrimmedRollupDts ./dist/react/internal.d.ts --publicDts ./dist/react/index-public.d.ts",
+    "api-report:api-json": "api-extractor run --local --verbose",
+    "typings:public": "node ../../scripts/build/use_typings.js ./dist/index-public.d.ts"
   },
   "peerDependencies": {
     "@firebase/app": "0.x",