@react-querybuilder/core 8.9.2 → 8.11.0

package/dist/formatQuery.mjs

@@ -1,9 +1,9 @@
-import { defaultPlaceholderFieldName, defaultPlaceholderOperatorName, joinWith, splitBy, toArray, trimIfString } from "./arrayUtils-BF1P8iHS.mjs";
-import { isPojo, isRuleGroup, isRuleGroupType, isRuleGroupTypeIC, lc, nullOrUndefinedOrEmpty, numericRegex } from "./isRuleGroup-CnhYpLOM.mjs";
-import { parseNumber } from "./parseNumber-BtGKa58z.mjs";
-import { transformQuery } from "./transformQuery-DdMvmrCh.mjs";
-import { getOption, toFlatOptionArray, toFullOptionList } from "./optGroupUtils-Duv-M8rf.mjs";
-import { convertFromIC } from "./convertQuery-H7RhQiIc.mjs";
+import { d as defaultPlaceholderOperatorName, i as toFlatOptionArray, o as toFullOptionList, t as getOption, u as defaultPlaceholderFieldName } from "./optGroupUtils-VeZ3k7-1.mjs";
+import { i as trimIfString, n as splitBy, r as toArray, t as joinWith } from "./arrayUtils-A_OXU9W1.mjs";
+import { a as lc, i as isPojo, n as isRuleGroupType, o as nullOrUndefinedOrEmpty, r as isRuleGroupTypeIC, s as numericRegex, t as isRuleGroup } from "./isRuleGroup-CYcfPgbg.mjs";
+import { t as parseNumber } from "./parseNumber-CXdMVNFx.mjs";
+import { t as transformQuery } from "./transformQuery-DCCpjtyq.mjs";
+import { t as convertFromIC } from "./convertQuery-CqX3rPvj.mjs";
 import { produce } from "immer";
 
 //#region src/utils/isRuleOrGroupValid.ts
@@ -13,9 +13,11 @@ import { produce } from "immer";
 const isValidationResult = (vr) => isPojo(vr) && typeof vr.valid === "boolean";
 /**
 * Determines if a rule or group is valid based on a validation result (if defined)
-* or a validator function. Returns `true` if neither are defined.
+* or a validator function. Returns `true` if neither are defined and the `muted`
+* property is not `true`.
 */
 const isRuleOrGroupValid = (rg, validationResult, validator) => {
+	if (rg.muted) return false;
 	if (typeof validationResult === "boolean") return validationResult;
 	if (isValidationResult(validationResult)) return validationResult.valid;
 	if (typeof validator === "function" && !isRuleGroup(rg)) {
@@ -289,19 +291,45 @@ const defaultRuleGroupProcessorCEL = (ruleGroup, options) => {
 	const { fields, fallbackExpression, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
 	const processRuleGroup = (rg, outermost) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermost ? fallbackExpression : "";
-		const expression = rg.rules.map((rule) => {
-			if (typeof rule === "string") return celCombinatorMap[rule];
-			if (isRuleGroup(rule)) return processRuleGroup(rule);
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = celCombinatorMap[rule];
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule);
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
 			const [validationResult, fieldValidator] = validateRule(rule);
-			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) return "";
+			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) continue;
 			const fieldData = getOption(fields, rule.field);
-			return ruleProcessor(rule, {
+			const processedRule = ruleProcessor(rule, {
 				...options,
 				parseNumbers: getParseNumberBoolean(fieldData?.inputType),
 				escapeQuotes: (rule.valueSource ?? "value") === "value",
 				fieldData
 			});
-		}).filter(Boolean).join(isRuleGroupType(rg) ? ` ${celCombinatorMap[rg.combinator]} ` : " ");
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
+		const expression = processedRules.join(isRuleGroupType(rg) ? ` ${celCombinatorMap[rg.combinator]} ` : " ");
 		const [prefix, suffix] = rg.not || !outermost ? [`${rg.not ? "!" : ""}(`, ")"] : ["", ""];
 		return expression ? `${prefix}${expression}${suffix}` : fallbackExpression;
 	};
@@ -328,11 +356,10 @@ const defaultRuleProcessorCEL = (rule, opts = {}) => {
 	else if (matchEval) {
 		const { mode, threshold } = matchEval;
 		const arrayElementAlias = "elem_alias";
-		const celQuery = transformQuery(rule.value, { ruleProcessor: (r) => ({
+		const nestedArrayFilter = defaultRuleGroupProcessorCEL(transformQuery(rule.value, { ruleProcessor: (r) => ({
 			...r,
 			field: `${arrayElementAlias}${r.field ? `.${r.field}` : ""}`
-		}) });
-		const nestedArrayFilter = defaultRuleGroupProcessorCEL(celQuery, opts);
+		}) }), opts);
 		switch (mode) {
 			case "all": return `${field}.all(${arrayElementAlias}, ${nestedArrayFilter})`;
 			case "none":
@@ -366,8 +393,7 @@ const defaultRuleProcessorCEL = (rule, opts = {}) => {
 		case "in":
 		case "notin": {
 			const [prefix, suffix] = shouldNegate$2(operatorTL) ? ["!(", ")"] : ["", ""];
-			const valueAsArray = toArray(value);
-			return `${prefix}${field} in [${valueAsArray.map((val) => valueIsField || shouldRenderAsNumber(val, parseNumbers) ? `${trimIfString(val)}` : `"${escapeDoubleQuotes(val, escapeQuotes)}"`).join(", ")}]${suffix}`;
+			return `${prefix}${field} in [${toArray(value).map((val) => valueIsField || shouldRenderAsNumber(val, parseNumbers) ? `${trimIfString(val)}` : `"${escapeDoubleQuotes(val, escapeQuotes)}"`).join(", ")}]${suffix}`;
 		}
 		case "between":
 		case "notbetween": {
@@ -587,19 +613,45 @@ const defaultRuleGroupProcessorSpEL = (ruleGroup, options) => {
 	const { fields, fallbackExpression, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
 	const processRuleGroup = (rg, outermost) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermost ? fallbackExpression : "";
-		const expression = rg.rules.map((rule) => {
-			if (typeof rule === "string") return rule;
-			if (isRuleGroup(rule)) return processRuleGroup(rule);
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = rule;
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule);
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
 			const [validationResult, fieldValidator] = validateRule(rule);
-			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) return "";
+			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) continue;
 			const fieldData = getOption(fields, rule.field);
-			return ruleProcessor(rule, {
+			const processedRule = ruleProcessor(rule, {
 				...options,
 				parseNumbers: getParseNumberBoolean(fieldData?.inputType),
 				escapeQuotes: (rule.valueSource ?? "value") === "value",
 				fieldData
 			});
-		}).filter(Boolean).join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ");
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
+		const expression = processedRules.join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ");
 		const [prefix, suffix] = rg.not || !outermost ? [`${rg.not ? "!" : ""}(`, ")"] : ["", ""];
 		return expression ? `${prefix}${expression}${suffix}` : fallbackExpression;
 	};
@@ -655,15 +707,9 @@ const defaultRuleProcessorSpEL = (rule, opts = {}) => {
 		case "contains":
 		case "doesnotcontain": return wrapInNegation(`${field} matches ${valueIsField || useBareValue ? trimIfString(value) : `'${escapeSingleQuotes(value, escapeQuotes)}'`}`, shouldNegate$1(operatorTL));
 		case "beginswith":
-		case "doesnotbeginwith": {
-			const valueTL = valueIsField ? `'^'.concat(${trimIfString(value)})` : `'${typeof value === "string" && !value.startsWith("^") || useBareValue ? "^" : ""}${escapeSingleQuotes(value, escapeQuotes)}'`;
-			return wrapInNegation(`${field} matches ${valueTL}`, shouldNegate$1(operatorTL));
-		}
+		case "doesnotbeginwith": return wrapInNegation(`${field} matches ${valueIsField ? `'^'.concat(${trimIfString(value)})` : `'${typeof value === "string" && !value.startsWith("^") || useBareValue ? "^" : ""}${escapeSingleQuotes(value, escapeQuotes)}'`}`, shouldNegate$1(operatorTL));
 		case "endswith":
-		case "doesnotendwith": {
-			const valueTL = valueIsField ? `${trimIfString(value)}.concat('$')` : `'${escapeSingleQuotes(value, escapeQuotes)}${typeof value === "string" && !value.endsWith("$") || useBareValue ? "$" : ""}'`;
-			return wrapInNegation(`${field} matches ${valueTL}`, shouldNegate$1(operatorTL));
-		}
+		case "doesnotendwith": return wrapInNegation(`${field} matches ${valueIsField ? `${trimIfString(value)}.concat('$')` : `'${escapeSingleQuotes(value, escapeQuotes)}${typeof value === "string" && !value.endsWith("$") || useBareValue ? "$" : ""}'`}`, shouldNegate$1(operatorTL));
 		case "null": return `${field} == null`;
 		case "notnull": return `${field} != null`;
 		case "in":
@@ -776,11 +822,10 @@ const defaultRuleProcessorDrizzle = (rule, _options) => {
 		if (opts.preset !== "postgresql") return;
 		const { mode, threshold } = matchEval;
 		const arrayElementAlias = "elem_alias";
-		const sqlQuery = transformQuery(rule.value, { ruleProcessor: (r) => ({
+		const nestedArrayFilter = defaultRuleGroupProcessorDrizzle(transformQuery(rule.value, { ruleProcessor: (r) => ({
 			...r,
 			field: arrayElementAlias
-		}) });
-		const nestedArrayFilter = defaultRuleGroupProcessorDrizzle(sqlQuery, {
+		}) }), {
 			...opts,
 			context: {
 				...opts.context,
@@ -866,7 +911,6 @@ const defaultRuleGroupProcessorDrizzle = (ruleGroup, options, _meta) => (columns
 	const { fields, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, validateRule, validationMap } = options;
 	if (!columns || !drizzleOperators) return;
 	const { and, not, or } = drizzleOperators;
-	const query = isRuleGroupType(ruleGroup) ? ruleGroup : convertFromIC(ruleGroup);
 	const ruleProcessor = defaultRuleProcessorDrizzle;
 	const processRuleGroup = (rg, _outermost) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return;
@@ -890,7 +934,7 @@ const defaultRuleGroupProcessorDrizzle = (ruleGroup, options, _meta) => (columns
 		const ruleGroupSQL = rg.combinator === "or" ? or(...processedRules) : and(...processedRules);
 		return rg.not ? not(ruleGroupSQL) : ruleGroupSQL;
 	};
-	return processRuleGroup(query, true);
+	return processRuleGroup(convertFromIC(ruleGroup), true);
 };
 
 //#endregion
@@ -902,7 +946,6 @@ const defaultRuleGroupProcessorDrizzle = (ruleGroup, options, _meta) => (columns
 */
 const defaultRuleGroupProcessorElasticSearch = (ruleGroup, options) => {
 	const { fields, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
-	const query = convertFromIC(ruleGroup);
 	const processRuleGroup = (rg) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return false;
 		const processedRules = rg.rules.map((rule) => {
@@ -919,7 +962,7 @@ const defaultRuleGroupProcessorElasticSearch = (ruleGroup, options) => {
 		if (processedRules.length === 0) return false;
 		return { bool: rg.not ? { must_not: /^or$/i.test(rg.combinator) ? { bool: { should: processedRules } } : processedRules } : { [/^or$/i.test(rg.combinator) ? "should" : "must"]: processedRules } };
 	};
-	const processedRuleGroup = processRuleGroup(query);
+	const processedRuleGroup = processRuleGroup(convertFromIC(ruleGroup));
 	return processedRuleGroup === false ? {} : processedRuleGroup;
 };
 
@@ -934,19 +977,45 @@ const defaultRuleGroupProcessorJSONata = (ruleGroup, options) => {
 	const { fields, fallbackExpression, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
 	const processRuleGroup = (rg, outermost) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermost ? fallbackExpression : "";
-		const expression = rg.rules.map((rule) => {
-			if (typeof rule === "string") return rule;
-			if (isRuleGroup(rule)) return processRuleGroup(rule);
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = rule;
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule);
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
 			const [validationResult, fieldValidator] = validateRule(rule);
-			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) return "";
+			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) continue;
 			const fieldData = getOption(fields, rule.field);
-			return ruleProcessor(rule, {
+			const processedRule = ruleProcessor(rule, {
 				...options,
 				parseNumbers: getParseNumberBoolean(fieldData?.inputType),
 				escapeQuotes: (rule.valueSource ?? "value") === "value",
 				fieldData
 			});
-		}).filter(Boolean).join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ");
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
+		const expression = processedRules.join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ");
 		const [prefix, suffix] = rg.not || !outermost ? [`${rg.not ? "$not" : ""}(`, ")"] : ["", ""];
 		return expression ? `${prefix}${expression}${suffix}` : fallbackExpression;
 	};
@@ -992,7 +1061,6 @@ const defaultRuleGroupProcessorJsonLogic = (ruleGroup, options) => {
 */
 const defaultRuleGroupProcessorLDAP = (ruleGroup, options) => {
 	const { fields, fallbackExpression, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
-	const query = convertFromIC(ruleGroup);
 	const processRuleGroup = (rg, outermost) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermost ? fallbackExpression : "";
 		const rules = rg.rules.map((rule) => {
@@ -1012,7 +1080,7 @@ const defaultRuleGroupProcessorLDAP = (ruleGroup, options) => {
 		const [prefix, suffix] = rules.length > 1 ? [`${notPrefix}(${rg.combinator === "or" ? "|" : "&"}`, `)${notSuffix}`] : [notPrefix, notSuffix];
 		return expression ? `${prefix}${expression}${suffix}` : fallbackExpression;
 	};
-	return processRuleGroup(query, true);
+	return processRuleGroup(convertFromIC(ruleGroup), true);
 };
 
 //#endregion
@@ -1067,20 +1135,46 @@ const defaultRuleGroupProcessorNL = (ruleGroup, options) => {
 	const processRuleGroup = (rg, outermostOrLonelyInGroup) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermostOrLonelyInGroup ? fallbackExpression : "";
 		const rg2 = isRuleGroupTypeIC(rg) && rg.rules.some((r) => typeof r === "string" && lc(r) === "xor") ? convertFromIC(rg) : rg;
-		const processedRules = rg2.rules.map((rule) => {
-			if (typeof rule === "string") return `, ${translations[rule] ?? rule} `;
-			if (isRuleGroup(rule)) return processRuleGroup(rule, rg2.rules.length === 1 && !(rg2.not || /^xor$/i.test(rg2.combinator ?? "")));
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg2.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = `, ${translations[rule] ?? rule} `;
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule, rg2.rules.length === 1 && !(rg2.not || /^xor$/i.test(rg2.combinator ?? "")));
+				// istanbul ignore else
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
 			const [validationResult, fieldValidator] = validateRule(rule);
-			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) return "";
+			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) continue;
 			const escapeQuotes = (rule.valueSource ?? "value") === "value";
 			const fieldData = getOption(fields, rule.field);
-			return ruleProcessor(rule, {
+			const processedRule = ruleProcessor(rule, {
 				...options,
 				parseNumbers: getParseNumberBoolean(fieldData?.inputType),
 				escapeQuotes,
 				fieldData
 			});
-		});
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
 		if (processedRules.length === 0) return fallbackExpression;
 		const isXOR = lc(rg2.combinator ?? "") === "xor";
 		const combinator = isXOR ? rg2.combinator.slice(1) : rg2.combinator;
@@ -1088,7 +1182,7 @@ const defaultRuleGroupProcessorNL = (ruleGroup, options) => {
 		const [prefixTL, suffixTL] = ["groupPrefix", "groupSuffix"].map((key) => rg2.not ? isXOR ? getNLTranslataion(key, translations, ["not", "xor"]) : getNLTranslataion(key, translations, ["not"]) : isXOR ? getNLTranslataion(key, translations, ["xor"]) : getNLTranslataion(key, translations));
 		const prefix = mustWrap ? `${prefixTL} (`.trim() : "";
 		const suffix = mustWrap ? `) ${suffixTL}`.trim() : "";
-		return `${prefix}${processedRules.filter(Boolean).join(isRuleGroupType(rg2) ? `, ${translations[combinator] ?? combinator} ` : "")}${suffix}`;
+		return `${prefix}${processedRules.join(isRuleGroupType(rg2) ? `, ${translations[combinator] ?? combinator} ` : "")}${suffix}`;
 	};
 	return processRuleGroup(ruleGroup, true);
 };
@@ -1138,11 +1232,37 @@ const defaultRuleGroupProcessorParameterized = (ruleGroup, options) => {
 	};
 	const processRuleGroup = (rg, outermostOrLonelyInGroup) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermostOrLonelyInGroup ? fallbackExpression : "";
-		const processedRules = rg.rules.map((rule) => {
-			if (typeof rule === "string") return rule;
-			if (isRuleGroup(rule)) return processRuleGroup(rule, rg.rules.length === 1);
-			return processRule(rule);
-		}).filter(Boolean);
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = rule;
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule, rg.rules.length === 1);
+				// istanbul ignore else
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
+			const processedRule = processRule(rule);
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
 		if (processedRules.length === 0) return fallbackExpression;
 		return `${rg.not ? "NOT " : ""}(${processedRules.join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ")})`;
 	};
@@ -1251,20 +1371,46 @@ const defaultRuleGroupProcessorSQL = (ruleGroup, options) => {
 	const { fields, fallbackExpression, getParseNumberBoolean, placeholderFieldName, placeholderOperatorName, placeholderValueName, ruleProcessor, validateRule, validationMap } = options;
 	const processRuleGroup = (rg, outermostOrLonelyInGroup) => {
 		if (!isRuleOrGroupValid(rg, validationMap[rg.id ?? ""])) return outermostOrLonelyInGroup ? fallbackExpression : "";
-		const processedRules = rg.rules.map((rule) => {
-			if (typeof rule === "string") return rule;
-			if (isRuleGroup(rule)) return processRuleGroup(rule, rg.rules.length === 1);
+		const processedRules = [];
+		let precedingCombinator = "";
+		let firstRule = true;
+		for (const rule of rg.rules) {
+			if (typeof rule === "string") {
+				precedingCombinator = rule;
+				continue;
+			}
+			if (isRuleGroup(rule)) {
+				const processedGroup = processRuleGroup(rule, rg.rules.length === 1);
+				// istanbul ignore else
+				if (processedGroup) {
+					if (!firstRule && precedingCombinator) {
+						processedRules.push(precedingCombinator);
+						precedingCombinator = "";
+					}
+					firstRule = false;
+					processedRules.push(processedGroup);
+				}
+				continue;
+			}
 			const [validationResult, fieldValidator] = validateRule(rule);
-			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) return "";
+			if (!isRuleOrGroupValid(rule, validationResult, fieldValidator) || rule.field === placeholderFieldName || rule.operator === placeholderOperatorName || placeholderValueName !== void 0 && rule.value === placeholderValueName) continue;
 			const escapeQuotes = (rule.valueSource ?? "value") === "value";
 			const fieldData = getOption(fields, rule.field);
-			return ruleProcessor(rule, {
+			const processedRule = ruleProcessor(rule, {
 				...options,
 				parseNumbers: getParseNumberBoolean(fieldData?.inputType),
 				escapeQuotes,
 				fieldData
 			});
-		}).filter(Boolean);
+			if (processedRule) {
+				if (!firstRule && precedingCombinator) {
+					processedRules.push(precedingCombinator);
+					precedingCombinator = "";
+				}
+				firstRule = false;
+				processedRules.push(processedRule);
+			}
+		}
 		if (processedRules.length === 0) return fallbackExpression;
 		return `${rg.not ? "NOT " : ""}(${processedRules.join(isRuleGroupType(rg) ? ` ${rg.combinator} ` : " ")})`;
 	};
@@ -1542,10 +1688,7 @@ const defaultRuleProcessorJsonLogic = (rule, options = {}) => {
 		case "null":
 		case "notnull": return { [`${operatorLC === "notnull" ? "!" : "="}=`]: [fieldObject, null] };
 		case "in":
-		case "notin": {
-			const valueAsArray = toArray(value).map((v) => fieldOrNumberRenderer(v));
-			return negateIfNotOp(operatorLC, { in: [fieldObject, valueAsArray] });
-		}
+		case "notin": return negateIfNotOp(operatorLC, { in: [fieldObject, toArray(value).map((v) => fieldOrNumberRenderer(v))] });
 		case "between":
 		case "notbetween": {
 			const valueAsArray = toArray(value);
@@ -1576,20 +1719,11 @@ const defaultRuleProcessorJsonLogic = (rule, options = {}) => {
 			return false;
 		}
 		case "contains":
-		case "doesnotcontain": {
-			const jsonRule = { in: [fieldOrNumberRenderer(value), fieldObject] };
-			return negateIfNotOp(operatorLC, jsonRule);
-		}
+		case "doesnotcontain": return negateIfNotOp(operatorLC, { in: [fieldOrNumberRenderer(value), fieldObject] });
 		case "beginswith":
-		case "doesnotbeginwith": {
-			const jsonRule = { startsWith: [fieldObject, fieldOrNumberRenderer(value)] };
-			return negateIfNotOp(operatorLC, jsonRule);
-		}
+		case "doesnotbeginwith": return negateIfNotOp(operatorLC, { startsWith: [fieldObject, fieldOrNumberRenderer(value)] });
 		case "endswith":
-		case "doesnotendwith": {
-			const jsonRule = { endsWith: [fieldObject, fieldOrNumberRenderer(value)] };
-			return negateIfNotOp(operatorLC, jsonRule);
-		}
+		case "doesnotendwith": return negateIfNotOp(operatorLC, { endsWith: [fieldObject, fieldOrNumberRenderer(value)] });
 	}
 	return false;
 };
@@ -1624,10 +1758,7 @@ const defaultRuleProcessorLDAP = (rule, options = {}) => {
 		case "null":
 		case "notnull": return negateIf(`(${field}=*)`, operatorLC === "notnull");
 		case "in":
-		case "notin": {
-			const valueAsArray = toArray(value);
-			return negateIf(`(|${valueAsArray.map((val) => `(${field}=${ldapEscape(val)})`).join("")})`, operatorLC === "notin");
-		}
+		case "notin": return negateIf(`(|${toArray(value).map((val) => `(${field}=${ldapEscape(val)})`).join("")})`, operatorLC === "notin");
 		case "between":
 		case "notbetween": {
 			const valueAsArray = toArray(value);
@@ -1795,10 +1926,9 @@ const defaultRuleProcessorNL = (rule, opts) => {
 	});
 	const operatorLC = lc(operator);
 	if ((operatorLC === "in" || operatorLC === "notin" || operatorLC === "between" || operatorLC === "notbetween") && !value) return "";
-	const processedOperator = operatorProcessor(rule, opts);
 	const wordOrderMap = {
 		S: processedField,
-		V: processedOperator,
+		V: operatorProcessor(rule, opts),
 		O: value
 	};
 	return normalizeConstituentWordOrder(wordOrder).map((term) => `${wordOrderMap[term]}`).join(" ").trim();