@classytic/mongokit 3.23.0 → 3.24.0

package/dist/{PaginationEngine-BT1ywu9k.mjs → PaginationEngine-CczbvCAq.mjs}

@@ -255,6 +255,59 @@ function resolveCursorFilter(after, sort, cursorVersion, baseFilters = {}, minCu
 //#endregion
 //#region src/pagination/utils/index-hint.ts
 /**
+* The only operators that give the query planner a *bounded equality* it can
+* lead an index with. Everything else (`$ne`, `$nin`, `$gt(e)`, `$lt(e)`,
+* `$all`, `$elemMatch`, `$regex`, `$exists`, …) is a range predicate and, per ESR,
+* must come AFTER the sort keys — never in the leading prefix.
+*
+* `$all` is intentionally excluded: although it combines equality matches,
+* it targets multikey data, whose bounded index prefix cannot generally
+* provide a subsequent index sort. Treating it as residual avoids claiming
+* that a multikey-prefix compound index satisfies the sort.
+*/
+const EQUALITY_OPERATORS = /* @__PURE__ */ new Set(["$eq", "$in"]);
+/**
+* Is a top-level filter value a RANGE predicate (as opposed to equality)?
+*
+* Only a plain operator object — one whose own keys start with `$` — can be a
+* range. Scalars, `Date`, `ObjectId`, arrays, and embedded-doc equality
+* matches (`{ nested: { a: 1 } }`) are all equality for index-leading
+* purposes. An operator object is equality iff every `$`-operator it uses is
+* in {@link EQUALITY_OPERATORS}.
+*/
+function isRangePredicate(value) {
+	if (value === null || typeof value !== "object" || Array.isArray(value)) return false;
+	const operatorKeys = Object.keys(value).filter((k) => k.startsWith("$"));
+	if (operatorKeys.length === 0) return false;
+	return operatorKeys.some((k) => !EQUALITY_OPERATORS.has(k));
+}
+/**
+* Split top-level filter fields into ESR equality vs range predicates.
+*
+* The keyset index check follows MongoDB's ESR rule (Equality → Sort →
+* Range). Only equality predicates belong in an index's leading prefix; a
+* range predicate placed there stops the planner from using any later key for
+* bounds OR for satisfying the sort. So the compatibility check and the
+* recommended-index shape must treat the two classes differently — a field
+* filtered by a range should TRAIL the sort keys, not lead them.
+*
+* `$`-prefixed top-level keys (`$and`/`$or`/…) are ignored (the caller decides
+* how to handle logical combinators).
+*/
+function classifyFilterFields(filters) {
+	const equality = [];
+	const range = [];
+	for (const [key, value] of Object.entries(filters)) {
+		if (key.startsWith("$")) continue;
+		if (isRangePredicate(value)) range.push(key);
+		else equality.push(key);
+	}
+	return {
+		equality,
+		range
+	};
+}
+/**
 * Read a Mongoose schema's declared indexes, defensively.
 * Returns an empty array on any introspection failure.
 */
@@ -626,9 +679,17 @@ var PaginationEngine = class {
 		const sortWithoutIdTail = stripTrailingIdTiebreaker(normalizedSort);
 		const filterKeys = Object.keys(filters).filter((k) => !k.startsWith("$"));
 		const effectiveSortFields = Object.keys(sortWithoutIdTail);
-		if (process.env.NODE_ENV !== "test" && filterKeys.length > 0 && effectiveSortFields.length > 0 && !hasCompatibleKeysetIndex(this._getSchemaIndexes(), filterKeys, sortWithoutIdTail)) {
-			const indexFields = [...filterKeys.map((f) => `${f}: 1`), ...effectiveSortFields.map((f) => `${f}: ${sortWithoutIdTail[f]}`)];
-			warn(`[mongokit] Keyset pagination with filters [${filterKeys.join(", ")}] and sort [${effectiveSortFields.join(", ")}] has no matching schema-declared compound index. For O(1) performance, declare: { ${indexFields.join(", ")} }. (Collection-level indexes created outside the schema are not visible here.)`);
+		if (process.env.NODE_ENV !== "test" && filterKeys.length > 0 && effectiveSortFields.length > 0) {
+			const indexes = this._getSchemaIndexes();
+			const { equality, range } = classifyFilterFields(filters);
+			if (!(hasCompatibleKeysetIndex(indexes, filterKeys, sortWithoutIdTail) || equality.length !== filterKeys.length && hasCompatibleKeysetIndex(indexes, equality, sortWithoutIdTail))) {
+				const indexFields = [
+					...equality.map((f) => `${f}: 1`),
+					...effectiveSortFields.map((f) => `${f}: ${sortWithoutIdTail[f]}`),
+					...range.map((f) => `${f}: 1`)
+				];
+				warn(`[mongokit] Keyset pagination with filters [${filterKeys.join(", ")}] and sort [${effectiveSortFields.join(", ")}] has no matching schema-declared compound index. For O(1) performance, declare: { ${indexFields.join(", ")} }. (Collection-level indexes created outside the schema are not visible here.)`);
+			}
 		}
 		let query = { ...filters };
 		if (after) query = resolveCursorFilter(after, normalizedSort, this.config.cursorVersion, query, this.config.minCursorVersion);

package/dist/{Repository-CLQA3sEz.mjs → Repository-ClOraFjT.mjs}

@@ -2,7 +2,7 @@ import { i as isValidIdForType, t as getSchemaIdType } from "./id-resolution-P2O
 import { a as count, b as LookupBuilder, c as getByQuery, d as deleteById, f as deleteByQuery, h as createMany, l as getOrCreate, m as create, n as update, o as exists, r as updateByQuery, s as getById, t as findOneAndUpdate, x as compileFilterToMongo, y as distinct } from "./update-1Lq9OMD2.mjs";
 import { n as warn } from "./logger-K7dPmu2U.mjs";
 import { n as isDuplicateKeyError, r as parseDuplicateKeyError, t as createError } from "./error-BuByZwKU.mjs";
-import { t as PaginationEngine } from "./PaginationEngine-BT1ywu9k.mjs";
+import { t as PaginationEngine } from "./PaginationEngine-CczbvCAq.mjs";
 import { hasNearOperator, rewriteNearForCount } from "./query/primitives/geo.mjs";
 import { RepositoryBase, runChunkedArchive, runChunkedPurge, throwIfAborted, validatePluginOrder, withRetry } from "@classytic/repo-core/repository";
 import mongoose from "mongoose";
@@ -3778,9 +3778,9 @@ var Repository = class extends RepositoryBase {
 				};
 				if (isKeyset && sort) {
 					const parsedSort = this._parseSort(sort);
-					const { validateKeysetSort } = await import("./PaginationEngine-BT1ywu9k.mjs").then((n) => n.n);
-					const { encodeCursor, resolveCursorFilter } = await import("./PaginationEngine-BT1ywu9k.mjs").then((n) => n.r);
-					const { getPrimaryField } = await import("./PaginationEngine-BT1ywu9k.mjs").then((n) => n.n);
+					const { validateKeysetSort } = await import("./PaginationEngine-CczbvCAq.mjs").then((n) => n.n);
+					const { encodeCursor, resolveCursorFilter } = await import("./PaginationEngine-CczbvCAq.mjs").then((n) => n.r);
+					const { getPrimaryField } = await import("./PaginationEngine-CczbvCAq.mjs").then((n) => n.n);
 					const normalizedSort = validateKeysetSort(parsedSort, this._pagination.config.strictKeysetSortFields);
 					const cursorVersion = this._pagination.config.cursorVersion ?? 1;
 					const minCursorVersion = this._pagination.config.minCursorVersion ?? 1;

package/dist/better-auth/index.mjs

@@ -1,4 +1,4 @@
-import { n as Repository } from "../Repository-CLQA3sEz.mjs";
+import { n as Repository } from "../Repository-ClOraFjT.mjs";
 import { n as createMongooseAdapter } from "../adapter-BWgJ_Jdc.mjs";
 import { pluralizeBetterAuthCollection, resolveBetterAuthCollections } from "@classytic/repo-core/better-auth";
 //#region src/better-auth/index.ts

package/dist/{create-repository-Bx1PpcdF.mjs → create-repository-CVupgeRG.mjs}

@@ -1,5 +1,5 @@
-import { n as Repository } from "./Repository-CLQA3sEz.mjs";
-import { a as methodRegistryPlugin, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, t as timestampPlugin } from "./timestamp.plugin-BnuuIDqJ.mjs";
+import { n as Repository } from "./Repository-ClOraFjT.mjs";
+import { a as methodRegistryPlugin, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, t as timestampPlugin } from "./timestamp.plugin-BhSX0Lu_.mjs";
 //#region src/create-repository.ts
 function isLoggerAudit(value) {
 	const v = value;

package/dist/index.mjs

@@ -1,14 +1,14 @@
 import { n as idVariants } from "./id-resolution-P2OzcyRl.mjs";
-import { a as withTransaction, c as MUTATING_OPERATIONS, d as operationsByPolicyKey, f as MONGOKIT_CAPABILITIES, i as isTransactionUnsupported, l as OP_REGISTRY, n as Repository, o as AggregationBuilder, p as createMongoPurgePort, r as batchTransaction, s as ALL_OPERATIONS, t as HOOK_PRIORITY, u as READ_OPERATIONS } from "./Repository-CLQA3sEz.mjs";
+import { a as withTransaction, c as MUTATING_OPERATIONS, d as operationsByPolicyKey, f as MONGOKIT_CAPABILITIES, i as isTransactionUnsupported, l as OP_REGISTRY, n as Repository, o as AggregationBuilder, p as createMongoPurgePort, r as batchTransaction, s as ALL_OPERATIONS, t as HOOK_PRIORITY, u as READ_OPERATIONS } from "./Repository-ClOraFjT.mjs";
 import { b as LookupBuilder, x as compileFilterToMongo } from "./update-1Lq9OMD2.mjs";
 import { n as warn, t as configureLogger } from "./logger-K7dPmu2U.mjs";
 import { n as isDuplicateKeyError, r as parseDuplicateKeyError, t as createError } from "./error-BuByZwKU.mjs";
 import { t as actions_exports } from "./actions/index.mjs";
-import { a as methodRegistryPlugin, c as getNextSequence, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, l as prefixedId, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, p as AuditTrailQuery, r as adminBypass, s as dateSequentialId, t as timestampPlugin, u as sequentialId } from "./timestamp.plugin-BnuuIDqJ.mjs";
-import { t as PaginationEngine } from "./PaginationEngine-BT1ywu9k.mjs";
+import { a as methodRegistryPlugin, c as getNextSequence, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, l as prefixedId, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, p as AuditTrailQuery, r as adminBypass, s as dateSequentialId, t as timestampPlugin, u as sequentialId } from "./timestamp.plugin-BhSX0Lu_.mjs";
+import { t as PaginationEngine } from "./PaginationEngine-CczbvCAq.mjs";
 import { isGeoOperator, parseGeoFilter } from "./query/primitives/geo.mjs";
-import { t as createRepository } from "./create-repository-Bx1PpcdF.mjs";
-import { _ as appendOnlyPlugin, a as uniqueField, c as subdocumentPlugin, d as mongoOperationsPlugin, f as leasePlugin, g as cascadePlugin, h as elasticSearchPlugin, i as requireField, l as observabilityPlugin, m as fieldFilterPlugin, n as blockIf, o as validationChainPlugin, p as immutableStatesPlugin, r as immutableField, s as createTenantContext, t as autoInject, u as MONGOKIT_PLUGIN_NAMES, v as aggregateHelpersPlugin } from "./validation-chain.plugin-ZFDHxXLn.mjs";
+import { t as createRepository } from "./create-repository-CVupgeRG.mjs";
+import { _ as appendOnlyPlugin, a as uniqueField, c as subdocumentPlugin, d as mongoOperationsPlugin, f as leasePlugin, g as cascadePlugin, h as elasticSearchPlugin, i as requireField, l as observabilityPlugin, m as fieldFilterPlugin, n as blockIf, o as validationChainPlugin, p as immutableStatesPlugin, r as immutableField, s as createTenantContext, t as autoInject, u as MONGOKIT_PLUGIN_NAMES, v as aggregateHelpersPlugin } from "./validation-chain.plugin-Dtt4tO7R.mjs";
 import { i as getMongooseProjection, n as filterResponseData, r as getFieldsForUser, t as createFieldPreset } from "./field-selection-FSQFLpsU.mjs";
 import { buildFieldTypeMap, coerceFieldValue, coerceHeuristic } from "./query/primitives/coercion.mjs";
 import { extractSchemaIndexes } from "./query/primitives/indexes.mjs";

package/dist/pagination/PaginationEngine.mjs

@@ -1,2 +1,2 @@
-import { t as PaginationEngine } from "../PaginationEngine-BT1ywu9k.mjs";
+import { t as PaginationEngine } from "../PaginationEngine-CczbvCAq.mjs";
 export { PaginationEngine };

package/dist/plugins/index.mjs

@@ -1,5 +1,5 @@
-import { a as methodRegistryPlugin, c as getNextSequence, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, l as prefixedId, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, p as AuditTrailQuery, s as dateSequentialId, t as timestampPlugin, u as sequentialId } from "../timestamp.plugin-BnuuIDqJ.mjs";
-import { _ as appendOnlyPlugin, a as uniqueField, c as subdocumentPlugin, d as mongoOperationsPlugin, f as leasePlugin, g as cascadePlugin, h as elasticSearchPlugin, i as requireField, l as observabilityPlugin, m as fieldFilterPlugin, n as blockIf, o as validationChainPlugin, p as immutableStatesPlugin, r as immutableField, s as createTenantContext, t as autoInject, u as MONGOKIT_PLUGIN_NAMES, v as aggregateHelpersPlugin } from "../validation-chain.plugin-ZFDHxXLn.mjs";
+import { a as methodRegistryPlugin, c as getNextSequence, d as cachePlugin, f as batchOperationsPlugin, h as auditLogPlugin, i as multiTenantPlugin, l as prefixedId, m as auditTrailPlugin, n as softDeletePlugin, o as customIdPlugin, p as AuditTrailQuery, s as dateSequentialId, t as timestampPlugin, u as sequentialId } from "../timestamp.plugin-BhSX0Lu_.mjs";
+import { _ as appendOnlyPlugin, a as uniqueField, c as subdocumentPlugin, d as mongoOperationsPlugin, f as leasePlugin, g as cascadePlugin, h as elasticSearchPlugin, i as requireField, l as observabilityPlugin, m as fieldFilterPlugin, n as blockIf, o as validationChainPlugin, p as immutableStatesPlugin, r as immutableField, s as createTenantContext, t as autoInject, u as MONGOKIT_PLUGIN_NAMES, v as aggregateHelpersPlugin } from "../validation-chain.plugin-Dtt4tO7R.mjs";
 //#region src/plugins/change-log.plugin.ts
 const PLUGIN_NAME = "changeLog";
 function isSkipped(context) {

package/dist/testkit/index.mjs

@@ -1,4 +1,4 @@
-import { t as createRepository } from "../create-repository-Bx1PpcdF.mjs";
+import { t as createRepository } from "../create-repository-CVupgeRG.mjs";
 import { createConnection } from "mongoose";
 //#region src/testkit/index.ts
 function idempotentStop(server) {

package/dist/{timestamp.plugin-BnuuIDqJ.mjs → timestamp.plugin-BhSX0Lu_.mjs}

@@ -1,4 +1,4 @@
-import { l as OP_REGISTRY, s as ALL_OPERATIONS, t as HOOK_PRIORITY } from "./Repository-CLQA3sEz.mjs";
+import { l as OP_REGISTRY, s as ALL_OPERATIONS, t as HOOK_PRIORITY } from "./Repository-ClOraFjT.mjs";
 import { n as warn } from "./logger-K7dPmu2U.mjs";
 import { t as createError } from "./error-BuByZwKU.mjs";
 import { throwIfAborted, withRetry } from "@classytic/repo-core/repository";

package/dist/{validation-chain.plugin-ZFDHxXLn.mjs → validation-chain.plugin-Dtt4tO7R.mjs}

@@ -1,4 +1,4 @@
-import { l as OP_REGISTRY, s as ALL_OPERATIONS } from "./Repository-CLQA3sEz.mjs";
+import { l as OP_REGISTRY, s as ALL_OPERATIONS } from "./Repository-ClOraFjT.mjs";
 import { _ as upsert } from "./update-1Lq9OMD2.mjs";
 import { n as warn } from "./logger-K7dPmu2U.mjs";
 import { t as createError } from "./error-BuByZwKU.mjs";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@classytic/mongokit",
-  "version": "3.23.0",
+  "version": "3.24.0",
   "description": "Production-grade MongoDB repositories - smart pagination, events, and plugins. Built on @classytic/repo-core.",
   "type": "module",
   "sideEffects": false,
@@ -140,7 +140,7 @@
     "format": "biome format src/ --write",
     "check": "biome ci src/ --diagnostic-level=error",
     "push": "classytic-push",
-    "prepublishOnly": "npm run check && npm run build && npm run lint:package && npm run typecheck && npm run typecheck:tests && npm test",
+    "prepublishOnly": "npm run check && npm run build && publint && npm run typecheck && npm run typecheck:tests && npm test",
     "publish:dry": "npm publish --dry-run --access public",
     "publish:npm": "npm publish --access public",
     "release:tag": "node -e \"require('child_process').execSync('npm run push -- v'+require('./package.json').version,{stdio:'inherit'})\"",