refactor(firestore/pipelines): drop RN-only Type export

Author: mikehardy

.github/scripts/compare-types/configs/firestore-pipelines.ts

@@ -22,13 +22,7 @@ import type { PackageConfig } from '../src/types';
 const config: PackageConfig = {
   nameMapping: {},
   missingInRN: [],
-  extraInRN: [
-    {
-      name: 'Type',
-      reason:
-        'RN Firebase exposes a local type discriminator alias for pipeline expression helpers.',
-    },
-  ],
+  extraInRN: [],
   differentShape: [],
 };
 

docs/firestore/pipelines/sdk-compatibility.mdx

@@ -16,7 +16,6 @@ CI enforces parity through `yarn compare:types` and the `firestore-pipelines` al
 | Core pipeline builder (`pipeline()`, stages, `execute()`)                   | **Supported** on Android, iOS, and macOS                              |
 | Expression helpers (aggregates, strings, arrays, maps, timestamps, vectors) | **Mostly supported** — see gaps below                                 |
 | Newer firebase-js-sdk helpers (2025–2026)                                   | **Partial** — tracked in "Not yet available"                          |
-| firebase-js-sdk `Type` export                                               | **React Native Firebase only** — use `isType()` with our `Type` union |
 
 # Not yet available in React Native Firebase
 
@@ -35,16 +34,10 @@ These exports exist in both SDKs but differ in typing or optional parameters:
 | Export            | Difference                                                            |
 | ----------------- | --------------------------------------------------------------------- |
 | `timestampDiff`   | Declaration overload ordering differs; all four SDK shapes supported  |
-| `isType`          | RNFB accepts the local `Type` alias; JS SDK accepts a broader string  |
+| `isType`          | Accepts the type name as a `string`, matching the JS SDK              |
 | `StageOptions`    | Equivalent runtime shape; declaration formatting differs              |
 | `TimeGranularity` | RNFB uses `isoWeek` / `isoYear`; JS SDK also lists lowercase variants |
 
-# React Native Firebase only
-
-| Export | Notes                                                                   |
-| ------ | ----------------------------------------------------------------------- |
-| `Type` | Local type discriminator union used with `isType()` and related helpers |
-
 # Platform execution matrix
 
 | Platform | Execution backend               | Pipeline database  |

okf-bundle/testing/compare-types-work-queue.md

@@ -84,7 +84,7 @@ Before any item's `implementation`:
 | **A10** | cross-cutting | `fix(types): align compare-types modular API with firebase-js-sdk` | closed | closed | closed | — | `area-focused` | Review green — app structural + callbacks |
 | **B0** | Phase B scope | `refactor!(types): align modular APIs with firebase-js-sdk sync signatures` | closed | closed | closed | — | `none` | Gap-analysis + grilling decisions 2026-07-03 — see [Phase B Notes](#phase-b-notes) |
 | **B1** | firestore-pipelines | `refactor(firestore/pipelines): add constant preferIntegers option` | closed | closed | closed | — | `area-focused` | Re-review green 2026-07-04: parser integerLiteral iOS/Android, preferIntegers e2e, sdk-compat; macOS 147 / iOS 152 / Android 152 |
-| **B2** | firestore-pipelines | `feat(firestore/pipelines): expose search stage and pipeline expressions` | closed | closed | closed | — | `area-focused` | 6× `missingInRN` cleared; macOS 150/150; iOS/Android 155/155; firebase-tools 15.22.4; search index verify cycle |
+| **B2** | firestore-pipelines | `feat(firestore/pipelines): expose search stage and pipeline expressions` | closed | closed | closed | — | `area-focused` | 6× `missingInRN` cleared; macOS 150/150; iOS/Android 155/155; firebase-tools 15.22.4; search index verify cycle. Follow-up: dropped RN-only `Type` export → pipelines 0 documented diffs |
 | **B3** | storage | — | open | open | open | — | `area-focused` | **Blocked** — Phase S native sync |
 | **B4** | analytics | `refactor!(types): align modular APIs with firebase-js-sdk sync signatures` | closed | closed | closed | — | `unit-focused` | logEvent sync void |
 | **B5** | app-check | `refactor!(types): align modular APIs with firebase-js-sdk sync signatures` | closed | closed | closed | — | `area-focused` | initializeAppCheck sync AppCheck |

packages/firestore/consumer-type-test.ts

@@ -258,7 +258,6 @@ import type {
   FunctionExpression,
   AggregateFunction,
   ExpressionType,
-  Type as PipelineValueType,
   TimeGranularity,
   TimePart,
   TimeUnit,
@@ -902,7 +901,6 @@ type _AllPipelineTypes = [
   FunctionExpression,
   AggregateFunction,
   ExpressionType,
-  PipelineValueType,
   TimeGranularity,
   TimePart,
   TimeUnit,
@@ -1009,7 +1007,7 @@ void isAbsent(field('optionalField'));
 void isAbsent('optionalField');
 // isError
 void isError(field('computedField'));
-// isType: (string, Type) | (Expression, Type)
+// isType: (string, string) | (Expression, string)
 void isType(field('value'), 'string');
 void isType('value', 'number');
 // ifAbsent: (Expression, Expression) | (Expression, unknown) | (string, Expression)

packages/firestore/lib/pipelines/expressions.ts

@@ -34,32 +34,6 @@ export type ExpressionType =
   | 'Variable'
   | 'PipelineValue';
 
-/**
- * @beta
- * Firestore value type for isType() checks.
- */
-export type Type =
-  | 'null'
-  | 'array'
-  | 'boolean'
-  | 'bytes'
-  | 'timestamp'
-  | 'geo_point'
-  | 'number'
-  | 'int32'
-  | 'int64'
-  | 'float64'
-  | 'decimal128'
-  | 'map'
-  | 'reference'
-  | 'string'
-  | 'vector'
-  | 'max_key'
-  | 'min_key'
-  | 'object_id'
-  | 'regex'
-  | 'request_timestamp';
-
 /**
  * @beta
  * Time unit for timestamp arithmetic expressions.

packages/firestore/lib/pipelines/index.ts

@@ -35,7 +35,6 @@ export type {
   FunctionExpression,
   AggregateFunction,
   ExpressionType,
-  Type,
   TimeGranularity,
   TimePart,
   TimeUnit,

packages/firestore/type-test.ts

@@ -244,7 +244,6 @@ import type {
   FunctionExpression,
   AggregateFunction,
   ExpressionType,
-  Type as PipelineValueType,
   TimeGranularity,
   TimePart,
   TimeUnit,
@@ -874,7 +873,6 @@ type _AllPipelineTypes = [
   FunctionExpression,
   AggregateFunction,
   ExpressionType,
-  PipelineValueType,
   TimeGranularity,
   TimePart,
   TimeUnit,
@@ -980,7 +978,7 @@ void isAbsent(field('optionalField'));
 void isAbsent('optionalField');
 // isError
 void isError(field('computedField'));
-// isType: (string, Type) | (Expression, Type)
+// isType: (string, string) | (Expression, string)
 void isType(field('value'), 'string');
 void isType('value', 'number');
 // ifAbsent: (Expression, Expression) | (Expression, unknown) | (string, Expression)