Fix size testing for pipelines bundles

MarkDuckworth · MarkDuckworth · commit 8590c329d1e3 · 2025-10-09T10:47:16.000-06:00
diff --git a/packages/firebase/firestore/lite/pipelines/index.ts b/packages/firebase/firestore/lite/pipelines/index.ts
@@ -0,0 +1,18 @@
+/**
+ * @license
+ * Copyright 2024 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+export * from '@firebase/firestore/lite/pipelines';
diff --git a/packages/firebase/firestore/lite/pipelines/package.json b/packages/firebase/firestore/lite/pipelines/package.json
@@ -0,0 +1,7 @@
+{
+  "name": "firebase/firestore/lite/pipelines",
+  "main": "dist/pipelines.cjs.js",
+  "browser": "dist/esm/pipelines.esm.js",
+  "module": "dist/esm/pipelines.esm.js",
+  "typings": "dist/firestore/lite/pipelines/index.d.ts"
+}
diff --git a/packages/firebase/firestore/package.json b/packages/firebase/firestore/package.json
@@ -4,4 +4,4 @@
   "browser": "dist/esm/index.esm.js",
   "module": "dist/esm/index.esm.js",
   "typings": "dist/firestore/index.d.ts"
-}
+}
diff --git a/packages/firebase/firestore/pipelines/package.json b/packages/firebase/firestore/pipelines/package.json
@@ -3,5 +3,5 @@
   "main": "dist/pipelines.cjs.js",
   "browser": "dist/esm/pipelines.esm.js",
   "module": "dist/esm/pipelines.esm.js",
-  "typings": "dist/firestore/lite/pipelines.d.ts"
+  "typings": "dist/firestore/pipelines/index.d.ts"
 }
diff --git a/packages/firebase/package.json b/packages/firebase/package.json
@@ -132,16 +132,16 @@
       "default": "./firestore/dist/esm/index.esm.js"
     },
     "./firestore/pipelines": {
-      "types": "./firestore/dist/firestore/pipelines.d.ts",
+      "types": "./firestore/pipelines/dist/firestore/pipelines/index.d.ts",
       "node": {
-        "require": "./firestore/dist/pipelines.cjs.js",
-        "import": "./firestore/dist/pipelines.mjs"
+        "require": "./firestore/pipelines/dist/pipelines.cjs.js",
+        "import": "./firestore/pipelines/dist/pipelines.mjs"
       },
       "browser": {
-        "require": "./firestore/dist/pipelines.cjs.js",
-        "import": "./firestore/dist/esm/pipelines.esm.js"
+        "require": "./firestore/pipelines/dist/pipelines.cjs.js",
+        "import": "./firestore/pipelines/dist/esm/pipelines.esm.js"
       },
-      "default": "./firestore/dist/esm/pipelines.esm.js"
+      "default": "./firestore/pipelines/dist/esm/pipelines.esm.js"
     },
     "./firestore/lite": {
       "types": "./firestore/lite/dist/firestore/lite/index.d.ts",
@@ -155,6 +155,18 @@
       },
       "default": "./firestore/lite/dist/esm/index.esm.js"
     },
+    "./firestore/lite/pipelines": {
+      "types": "./firestore/lite/pipelines/dist/firestore/lite/pipelines/index.d.ts",
+      "node": {
+        "require": "./firestore/lite/pipelines/dist/pipelines.cjs.js",
+        "import": "./firestore/lite/pipelines/dist/pipelines.mjs"
+      },
+      "browser": {
+        "require": "./firestore/lite/pipelines/dist/pipelines.cjs.js",
+        "import": "./firestore/lite/pipelines/dist/esm/pipelines.esm.js"
+      },
+      "default": "./firestore/lite/pipelines/dist/esm/pipelines.esm.js"
+    },
     "./functions": {
       "types": "./functions/dist/functions/index.d.ts",
       "node": {
@@ -464,7 +476,9 @@
     "auth/web-extension",
     "functions",
     "firestore",
+    "firestore/pipelines",
     "firestore/lite",
+    "firestore/lite/pipelines",
     "installations",
     "storage",
     "performance",
diff --git a/packages/firestore/lite/pipelines/package.json b/packages/firestore/lite/pipelines/package.json
@@ -3,12 +3,12 @@
   "description": "Pipelines for the lite Firestore SDK",
   "main": "../../dist/lite/pipelines.node.cjs.js",
   "main-esm": "../../dist/lite/pipelines.node.mjs",
-  "module": "../../dist/lite/pipelines.browser.esm2017.js",
-  "browser": "../../dist/lite/pipelines.browser.esm2017.js",
-  "react-native": "../../dist/lite/pipelines.rn.esm2017.js",
+  "module": "../../dist/lite/pipelines.browser.esm.js",
+  "browser": "../../dist/lite/pipelines.browser.esm.js",
+  "react-native": "../../dist/lite/pipelines.rn.esm.js",
   "typings": "./pipelines.d.ts",
   "private": true,
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=20.0.0"
   }
 }
diff --git a/packages/firestore/pipelines/package.json b/packages/firestore/pipelines/package.json
@@ -1,14 +1,14 @@
 {
-  "name": "@firebase/firestore/pipelines",
+  "name": "@firebase/firestore-pipelines",
   "description": "pipelines",
   "main": "../dist/pipelines.node.cjs.js",
   "main-esm": "../dist/pipelines.node.mjs",
-  "module": "../dist/pipelines.browser.esm2017.js",
-  "browser": "../dist/pipelines.browser.esm2017.js",
-  "react-native": "../dist/pipelines.rn.esm2017.js",
+  "module": "../dist/pipelines.browser.esm.js",
+  "browser": "../dist/pipelines.browser.esm.js",
+  "react-native": "../dist/pipelines.rn.esm.js",
   "typings": "./pipelines.d.ts",
   "private": true,
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=20.0.0"
   }
 }
diff --git a/packages/firestore/test/lite/pipeline.test.ts b/packages/firestore/test/lite/pipeline.test.ts
@@ -136,6 +136,7 @@ use(chaiAsPromised);
 const testUnsupportedFeatures = false;
 const timestampDeltaMS = 1000;
 
+// eslint-disable-next-line no-restricted-properties
 describe.skip('Firestore Pipelines', () => {
   addEqualityMatcher();
 
diff --git a/repo-scripts/size-analysis/bundle-definitions/firestore.json b/repo-scripts/size-analysis/bundle-definitions/firestore.json
@@ -191,7 +191,7 @@
         "versionOrTag": "latest",
         "imports": [
           {
-            "path": "firestore/pipelines",
+            "path": "firestore",
             "imports": [
               "getFirestore"
             ]
@@ -235,11 +235,21 @@
           {
             "path": "firestore",
             "imports": [
-              "getFirestore",
+              "getFirestore"
+            ]
+          }
+        ]
+      },
+      {
+        "packageName": "firebase",
+        "versionOrTag": "latest",
+        "imports": [
+          {
+            "path": "firestore/pipelines",
+            "imports": [
               "lessThan",
               "Field",
-              "useFirestorePipelines",
-              "andFunction"
+              "and"
             ]
           }
         ]

Original file line number	Diff line number	Diff line change
`@@ -3,5 +3,5 @@`
`3`	`3`	`"main": "dist/pipelines.cjs.js",`
`4`	`4`	`"browser": "dist/esm/pipelines.esm.js",`
`5`	`5`	`"module": "dist/esm/pipelines.esm.js",`
`6`		`- "typings": "dist/firestore/lite/pipelines.d.ts"`
	`6`	`+ "typings": "dist/firestore/pipelines/index.d.ts"`
`7`	`7`	`}`
Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@`
`191`	`191`	`"versionOrTag": "latest",`
`192`	`192`	`"imports": [`
`193`	`193`	`{`
`194`		`- "path": "firestore/pipelines",`
	`194`	`+ "path": "firestore",`
`195`	`195`	`"imports": [`
`196`	`196`	`"getFirestore"`
`197`	`197`	`]`
`@@ -235,11 +235,21 @@`
`235`	`235`	`{`
`236`	`236`	`"path": "firestore",`
`237`	`237`	`"imports": [`
`238`		`- "getFirestore",`
	`238`	`+ "getFirestore"`
	`239`	`+ ]`
	`240`	`+ }`
	`241`	`+ ]`
	`242`	`+ },`
	`243`	`+ {`
	`244`	`+ "packageName": "firebase",`
	`245`	`+ "versionOrTag": "latest",`
	`246`	`+ "imports": [`
	`247`	`+ {`
	`248`	`+ "path": "firestore/pipelines",`
	`249`	`+ "imports": [`
`239`	`250`	`"lessThan",`
`240`	`251`	`"Field",`
`241`		`- "useFirestorePipelines",`
`242`		`- "andFunction"`
	`252`	`+ "and"`
`243`	`253`	`]`
`244`	`254`	`}`
`245`	`255`	`]`