@rebasepro/server-mongo 0.14.1-canary.g7e666eb → 0.14.1

package/dist/db/MongoConditionBuilder.d.ts

@@ -79,6 +79,17 @@ export declare class MongoConditionBuilder {
         searchString?: string;
         properties?: CollectionConfig["properties"];
     }): Filter<Document>;
+    /**
+     * The primary key's name in a stored document.
+     *
+     * Rows leave this driver with `_id` renamed to `id` — see
+     * `MongoDataService.documentToRow` — so `id` is the only name a caller ever
+     * sees, and the one the SDK's own examples use for a tie-breaker. A sort
+     * document naming `id` names a field no document has: Mongo does not
+     * complain, it just returns them in natural order, so `.orderBy("id")` read
+     * as "no sort at all" with a 200 to go with it.
+     */
+    private static readonly ID_FIELD;
     /**
      * Build MongoDB sort options from Rebase options
      *
@@ -86,6 +97,15 @@ export declare class MongoConditionBuilder {
      * a multi-key sort is expressed directly: the keys are applied in insertion
      * order, each breaking ties on the one before it.
      *
+     * The id closes every sort, descending, for the same reason the Postgres
+     * driver appends `id DESC`: it is what makes the ordering *total*. Without
+     * it, two rows sharing a sort value are returned in whatever order the
+     * engine pleases, and that order is free to differ between two executions of
+     * the same query — so paging by `offset` over a non-unique sort column
+     * repeats some rows and skips others. A single-column sort has always had
+     * this exposure here; a multi-key sort merely made it easier to reach, since
+     * the whole point of the later keys is that the earlier ones tie.
+     *
      * @param orderBy - Field to order by, or the `[field, direction]` list
      * @param order - Sort direction, for the single-field spelling
      * @returns MongoDB sort object

package/dist/db/MongoConditionBuilder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"MongoConditionBuilder.d.ts","sourceRoot":"","sources":["../../src/db/MongoConditionBuilder.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,gBAAgB,EAAmB,YAAY,EAAE,gBAAgB,EAAE,YAAY,EAAiB,MAAM,kBAAkB,CAAC;AAElI,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAkD3C;;;;;GAKG;AACH,qBAAa,qBAAqB;IAC9B;;;;;OAKG;IACH,MAAM,CAAC,qBAAqB,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EACtD,MAAM,EAAE,YAAY,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC,CAAC,GAC/C,MAAM,CAAC,QAAQ,CAAC,EAAE;IAqBrB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;IAwC7B;;;;;;OAMG;IACH,MAAM,CAAC,sBAAsB,CAAC,OAAO,EAAE,gBAAgB,GAAG,SAAS,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAuBlG;;;;;;OAMG;IACH,MAAM,CAAC,qBAAqB,CACxB,YAAY,EAAE,MAAM,EAKpB,UAAU,EAAE,gBAAgB,CAAC,YAAY,CAAC,GAC3C,MAAM,CAAC,QAAQ,CAAC,EAAE;IAkCrB;;;;;OAKG;IACH,MAAM,CAAC,wBAAwB,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,EAAE,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAM7F;;;;;OAKG;IACH,MAAM,CAAC,uBAAuB,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,EAAE,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAM5F;;;;;OAKG;IACH,MAAM,CAAC,UAAU,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,OAAO,EAAE;QACtD,MAAM,CAAC,EAAE,YAAY,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC,CAAC,CAAC;QAChD;;;;;WAKG;QACH,OAAO,CAAC,EAAE,gBAAgB,CAAC;QAC3B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,UAAU,CAAC,EAAE,gBAAgB,CAAC,YAAY,CAAC,CAAC;KAC/C,GAAG,MAAM,CAAC,QAAQ,CAAC;IA8BpB;;;;;;;;;;OAUG;IACH,MAAM,CAAC,SAAS,CACZ,OAAO,CAAC,EAAE,MAAM,GAAG,YAAY,EAAE,EACjC,KAAK,CAAC,EAAE,KAAK,GAAG,MAAM,GACvB,MAAM,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,SAAS;CAYxC"}
+{"version":3,"file":"MongoConditionBuilder.d.ts","sourceRoot":"","sources":["../../src/db/MongoConditionBuilder.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,gBAAgB,EAAmB,YAAY,EAAE,gBAAgB,EAAE,YAAY,EAAiB,MAAM,kBAAkB,CAAC;AAElI,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAkD3C;;;;;GAKG;AACH,qBAAa,qBAAqB;IAC9B;;;;;OAKG;IACH,MAAM,CAAC,qBAAqB,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EACtD,MAAM,EAAE,YAAY,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC,CAAC,GAC/C,MAAM,CAAC,QAAQ,CAAC,EAAE;IAqBrB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;IAwC7B;;;;;;OAMG;IACH,MAAM,CAAC,sBAAsB,CAAC,OAAO,EAAE,gBAAgB,GAAG,SAAS,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAuBlG;;;;;;OAMG;IACH,MAAM,CAAC,qBAAqB,CACxB,YAAY,EAAE,MAAM,EAKpB,UAAU,EAAE,gBAAgB,CAAC,YAAY,CAAC,GAC3C,MAAM,CAAC,QAAQ,CAAC,EAAE;IAkCrB;;;;;OAKG;IACH,MAAM,CAAC,wBAAwB,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,EAAE,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAM7F;;;;;OAKG;IACH,MAAM,CAAC,uBAAuB,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,EAAE,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,SAAS;IAM5F;;;;;OAKG;IACH,MAAM,CAAC,UAAU,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,OAAO,EAAE;QACtD,MAAM,CAAC,EAAE,YAAY,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC,CAAC,CAAC;QAChD;;;;;WAKG;QACH,OAAO,CAAC,EAAE,gBAAgB,CAAC;QAC3B,YAAY,CAAC,EAAE,MAAM,CAAC;QACtB,UAAU,CAAC,EAAE,gBAAgB,CAAC,YAAY,CAAC,CAAC;KAC/C,GAAG,MAAM,CAAC,QAAQ,CAAC;IA8BpB;;;;;;;;;OASG;IACH,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,QAAQ,CAAS;IAEzC;;;;;;;;;;;;;;;;;;;OAmBG;IACH,MAAM,CAAC,SAAS,CACZ,OAAO,CAAC,EAAE,MAAM,GAAG,YAAY,EAAE,EACjC,KAAK,CAAC,EAAE,KAAK,GAAG,MAAM,GACvB,MAAM,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,SAAS;CAkBxC"}

package/dist/index.es.js

@@ -108,7 +108,7 @@ function likePatternToRegExp(pattern, caseInsensitive) {
 * Provides static methods to translate Rebase filter conditions
 * to MongoDB query filters.
 */
-var MongoConditionBuilder = class {
+var MongoConditionBuilder = class MongoConditionBuilder {
 	/**
 	* Build MongoDB filter conditions from Rebase FilterValues
 	*
@@ -238,12 +238,32 @@ var MongoConditionBuilder = class {
 		return this.combineConditionsWithAnd(conditions) ?? {};
 	}
 	/**
+	* The primary key's name in a stored document.
+	*
+	* Rows leave this driver with `_id` renamed to `id` — see
+	* `MongoDataService.documentToRow` — so `id` is the only name a caller ever
+	* sees, and the one the SDK's own examples use for a tie-breaker. A sort
+	* document naming `id` names a field no document has: Mongo does not
+	* complain, it just returns them in natural order, so `.orderBy("id")` read
+	* as "no sort at all" with a 200 to go with it.
+	*/
+	static ID_FIELD = "_id";
+	/**
 	* Build MongoDB sort options from Rebase options
 	*
 	* A Mongo sort document is already an ordered map of field to direction, so
 	* a multi-key sort is expressed directly: the keys are applied in insertion
 	* order, each breaking ties on the one before it.
 	*
+	* The id closes every sort, descending, for the same reason the Postgres
+	* driver appends `id DESC`: it is what makes the ordering *total*. Without
+	* it, two rows sharing a sort value are returned in whatever order the
+	* engine pleases, and that order is free to differ between two executions of
+	* the same query — so paging by `offset` over a non-unique sort column
+	* repeats some rows and skips others. A single-column sort has always had
+	* this exposure here; a multi-key sort merely made it easier to reach, since
+	* the whole point of the later keys is that the earlier ones tie.
+	*
 	* @param orderBy - Field to order by, or the `[field, direction]` list
 	* @param order - Sort direction, for the single-field spelling
 	* @returns MongoDB sort object
@@ -252,7 +272,11 @@ var MongoConditionBuilder = class {
 		const keys = normalizeDriverOrderBy(orderBy, order);
 		if (!keys) return void 0;
 		const sort = {};
-		for (const [field, direction] of keys) if (!(field in sort)) sort[field] = direction === "desc" ? -1 : 1;
+		for (const [field, direction] of keys) {
+			const key = field === "id" ? MongoConditionBuilder.ID_FIELD : field;
+			if (!(key in sort)) sort[key] = direction === "desc" ? -1 : 1;
+		}
+		if (!(MongoConditionBuilder.ID_FIELD in sort)) sort[MongoConditionBuilder.ID_FIELD] = -1;
 		return sort;
 	}
 };