@happychef/algorithm 1.2.10 → 1.2.12

package/getAvailableTimeblocks.js

@@ -1,158 +1,158 @@
-// getAvailableTimeblocks.js
-
-const { timeblocksAvailable } = require('./processing/timeblocksAvailable');
-const { parseTime, getMealTypeByTime } = require('./tableHelpers');
-
-/**
- * Parses a time string in "HH:MM" format into a Date object on a specific date.
- * @param {string} dateStr - The date string in "YYYY-MM-DD" format.
- * @param {string} timeStr - Time string in "HH:MM" format.
- * @param {string} timeZone - The IANA time zone identifier (not used, kept for compatibility).
- * @returns {Date} Date object representing the time on the specified date.
- */
-function parseDateTimeInTimeZone(dateStr, timeStr, timeZone) {
-  const [year, month, day] = dateStr.split('-').map(Number);
-  const [hours, minutes] = timeStr.split(':').map(Number);
-
-  // Create a simple date object for the given date and time
-  // This represents the local time on that date
-  return new Date(year, month - 1, day, hours, minutes);
-}
-
-/**
- * Gets the available time blocks or shifts for a reservation, considering 'uurOpVoorhand' and 'dagenInToekomst'.
- * @param {Object} data - The main data object containing settings and meal information.
- * @param {string} dateStr - The date string in "YYYY-MM-DD" format.
- * @param {Array} reservations - An array of reservation objects.
- * @param {number} guests - The number of guests for the reservation.
- * @param {Array} blockedSlots - Optional array of blocked time slots to exclude.
- * @param {string|null} giftcard - Optional giftcard to filter times by meal.
- * @param {boolean} isAdmin - Optional flag to bypass time restrictions for admin users.
- * @returns {Object} - Returns a pruned object of available time blocks or shifts, or an empty object if out of range.
- */
-function getAvailableTimeblocks(data, dateStr, reservations, guests, blockedSlots = [], giftcard = null, isAdmin = false) {
-  // Get 'uurOpVoorhand' from general settings
-  let uurOpVoorhand = 0;
-  if (
-    data['general-settings'] &&
-    data['general-settings'].uurOpVoorhand &&
-    parseInt(data['general-settings'].uurOpVoorhand, 10) >= 0
-  ) {
-    uurOpVoorhand = parseInt(data['general-settings'].uurOpVoorhand, 10);
-  }
-
-  // Get 'dagenInToekomst' from general settings
-  let dagenInToekomst = 90; // Default if not defined
-  if (
-    data['general-settings'] &&
-    data['general-settings'].dagenInToekomst &&
-    parseInt(data['general-settings'].dagenInToekomst, 10) > 0
-  ) {
-    dagenInToekomst = parseInt(data['general-settings'].dagenInToekomst, 10);
-  }
-
-  // Time zone for CEST/CET (Europe/Amsterdam)
-  const timeZone = 'Europe/Amsterdam';
-
-  // Current date/time in local timezone (system time)
-  // Note: We assume the server is running in the correct timezone
-  const now = new Date();
-  const currentTimeInTimeZone = now;
-
-  // Calculate the maximum allowed date
-  const maxAllowedDate = new Date(currentTimeInTimeZone.getTime());
-  maxAllowedDate.setDate(maxAllowedDate.getDate() + dagenInToekomst);
-  maxAllowedDate.setHours(23, 59, 59, 999);
-
-  // Parse the target date (just the date part, no time)
-  const [year, month, day] = dateStr.split('-').map(Number);
-  const targetDateInTimeZone = new Date(year, month - 1, day);
-
-  // Check if targetDateInTimeZone is within dagenInToekomst (skip for admin)
-  if (!isAdmin && targetDateInTimeZone > maxAllowedDate) {
-    // Out of allowed range, return empty object
-    return {};
-  }
-
-  // Check if the target date is today in the specified time zone
-  const isToday =
-    currentTimeInTimeZone.toDateString() === targetDateInTimeZone.toDateString();
-
-  // Get available time blocks or shifts
-  const availableTimeblocks = timeblocksAvailable(data, dateStr, reservations, guests, blockedSlots, giftcard, isAdmin);
-
-  // If the date is today and uurOpVoorhand is greater than zero, prune time blocks (skip for admin)
-  if (!isAdmin && isToday && uurOpVoorhand >= 0) {
-    const cutoffTime = new Date(currentTimeInTimeZone.getTime());
-    cutoffTime.setHours(cutoffTime.getHours() + uurOpVoorhand);
-
-    for (const [key, value] of Object.entries(availableTimeblocks)) {
-      let timeStr = key;
-
-      const timeBlockDateTime = parseDateTimeInTimeZone(dateStr, timeStr, timeZone);
-
-      if (timeBlockDateTime < cutoffTime) {
-        delete availableTimeblocks[key];
-      }
-    }
-  }
-
-  // Apply last booking time filter (lunchStop, dinerStop, ontbijtStop) - skip for admin
-  // IMPORTANT: This filter should only apply to TODAY, just like uurOpVoorhand
-  // The filter compares CURRENT TIME in Brussels against the stop time,
-  // not the reservation time slot against the stop time
-  if (!isAdmin && isToday) {
-    const settings = data?.['general-settings'] || {};
-    const breakfastStop = settings.ontbijtStop || null;
-    const lunchStop = settings.lunchStop || null;
-    const dinnerStop = settings.dinerStop || null;
-
-    // Only apply if at least one stop time is configured
-    if (breakfastStop || lunchStop || dinnerStop) {
-      // Get current time in Brussels timezone (in minutes since midnight)
-      // Using Intl.DateTimeFormat for reliable timezone conversion
-      const formatter = new Intl.DateTimeFormat('en-US', {
-        timeZone: 'Europe/Brussels',
-        hour: '2-digit',
-        minute: '2-digit',
-        hour12: false,
-      });
-      const parts = formatter.formatToParts(now);
-      const timeParts = Object.fromEntries(parts.map(p => [p.type, p.value]));
-      const currentTimeMinutes = parseInt(timeParts.hour, 10) * 60 + parseInt(timeParts.minute, 10);
-
-      // Check if current time has passed any stop times
-      const breakfastStopMinutes = breakfastStop ? parseTime(breakfastStop) : null;
-      const lunchStopMinutes = lunchStop ? parseTime(lunchStop) : null;
-      const dinnerStopMinutes = dinnerStop ? parseTime(dinnerStop) : null;
-
-      const breakfastStopped = breakfastStopMinutes !== null && currentTimeMinutes >= breakfastStopMinutes;
-      const lunchStopped = lunchStopMinutes !== null && currentTimeMinutes >= lunchStopMinutes;
-      const dinnerStopped = dinnerStopMinutes !== null && currentTimeMinutes >= dinnerStopMinutes;
-
-      // Remove ALL slots of a meal type if current time >= stop time for that meal
-      for (const time in availableTimeblocks) {
-        const mealType = getMealTypeByTime(time);
-        let shouldRemove = false;
-
-        if (mealType === 'breakfast' && breakfastStopped) {
-          shouldRemove = true;
-        } else if (mealType === 'lunch' && lunchStopped) {
-          shouldRemove = true;
-        } else if (mealType === 'dinner' && dinnerStopped) {
-          shouldRemove = true;
-        }
-
-        if (shouldRemove) {
-          delete availableTimeblocks[time];
-        }
-      }
-    }
-  }
-
-  return availableTimeblocks;
-}
-
-module.exports = {
-  getAvailableTimeblocks,
-};
+// getAvailableTimeblocks.js
+
+const { timeblocksAvailable } = require('./processing/timeblocksAvailable');
+const { parseTime, getMealTypeByTime } = require('./tableHelpers');
+
+/**
+ * Parses a time string in "HH:MM" format into a Date object on a specific date.
+ * @param {string} dateStr - The date string in "YYYY-MM-DD" format.
+ * @param {string} timeStr - Time string in "HH:MM" format.
+ * @param {string} timeZone - The IANA time zone identifier (not used, kept for compatibility).
+ * @returns {Date} Date object representing the time on the specified date.
+ */
+function parseDateTimeInTimeZone(dateStr, timeStr, timeZone) {
+  const [year, month, day] = dateStr.split('-').map(Number);
+  const [hours, minutes] = timeStr.split(':').map(Number);
+
+  // Create a simple date object for the given date and time
+  // This represents the local time on that date
+  return new Date(year, month - 1, day, hours, minutes);
+}
+
+/**
+ * Gets the available time blocks or shifts for a reservation, considering 'uurOpVoorhand' and 'dagenInToekomst'.
+ * @param {Object} data - The main data object containing settings and meal information.
+ * @param {string} dateStr - The date string in "YYYY-MM-DD" format.
+ * @param {Array} reservations - An array of reservation objects.
+ * @param {number} guests - The number of guests for the reservation.
+ * @param {Array} blockedSlots - Optional array of blocked time slots to exclude.
+ * @param {string|null} giftcard - Optional giftcard to filter times by meal.
+ * @param {boolean} isAdmin - Optional flag to bypass time restrictions for admin users.
+ * @returns {Object} - Returns a pruned object of available time blocks or shifts, or an empty object if out of range.
+ */
+function getAvailableTimeblocks(data, dateStr, reservations, guests, blockedSlots = [], giftcard = null, isAdmin = false) {
+  // Get 'uurOpVoorhand' from general settings
+  let uurOpVoorhand = 0;
+  if (
+    data['general-settings'] &&
+    data['general-settings'].uurOpVoorhand &&
+    parseInt(data['general-settings'].uurOpVoorhand, 10) >= 0
+  ) {
+    uurOpVoorhand = parseInt(data['general-settings'].uurOpVoorhand, 10);
+  }
+
+  // Get 'dagenInToekomst' from general settings
+  let dagenInToekomst = 90; // Default if not defined
+  if (
+    data['general-settings'] &&
+    data['general-settings'].dagenInToekomst &&
+    parseInt(data['general-settings'].dagenInToekomst, 10) > 0
+  ) {
+    dagenInToekomst = parseInt(data['general-settings'].dagenInToekomst, 10);
+  }
+
+  // Time zone for CEST/CET (Europe/Amsterdam)
+  const timeZone = 'Europe/Amsterdam';
+
+  // Current date/time in local timezone (system time)
+  // Note: We assume the server is running in the correct timezone
+  const now = new Date();
+  const currentTimeInTimeZone = now;
+
+  // Calculate the maximum allowed date
+  const maxAllowedDate = new Date(currentTimeInTimeZone.getTime());
+  maxAllowedDate.setDate(maxAllowedDate.getDate() + dagenInToekomst);
+  maxAllowedDate.setHours(23, 59, 59, 999);
+
+  // Parse the target date (just the date part, no time)
+  const [year, month, day] = dateStr.split('-').map(Number);
+  const targetDateInTimeZone = new Date(year, month - 1, day);
+
+  // Check if targetDateInTimeZone is within dagenInToekomst (skip for admin)
+  if (!isAdmin && targetDateInTimeZone > maxAllowedDate) {
+    // Out of allowed range, return empty object
+    return {};
+  }
+
+  // Check if the target date is today in the specified time zone
+  const isToday =
+    currentTimeInTimeZone.toDateString() === targetDateInTimeZone.toDateString();
+
+  // Get available time blocks or shifts
+  const availableTimeblocks = timeblocksAvailable(data, dateStr, reservations, guests, blockedSlots, giftcard, isAdmin);
+
+  // If the date is today and uurOpVoorhand is greater than zero, prune time blocks (skip for admin)
+  if (!isAdmin && isToday && uurOpVoorhand >= 0) {
+    const cutoffTime = new Date(currentTimeInTimeZone.getTime());
+    cutoffTime.setHours(cutoffTime.getHours() + uurOpVoorhand);
+
+    for (const [key, value] of Object.entries(availableTimeblocks)) {
+      let timeStr = key;
+
+      const timeBlockDateTime = parseDateTimeInTimeZone(dateStr, timeStr, timeZone);
+
+      if (timeBlockDateTime < cutoffTime) {
+        delete availableTimeblocks[key];
+      }
+    }
+  }
+
+  // Apply last booking time filter (lunchStop, dinerStop, ontbijtStop) - skip for admin
+  // IMPORTANT: This filter should only apply to TODAY, just like uurOpVoorhand
+  // The filter compares CURRENT TIME in Brussels against the stop time,
+  // not the reservation time slot against the stop time
+  if (!isAdmin && isToday) {
+    const settings = data?.['general-settings'] || {};
+    const breakfastStop = settings.ontbijtStop || null;
+    const lunchStop = settings.lunchStop || null;
+    const dinnerStop = settings.dinerStop || null;
+
+    // Only apply if at least one stop time is configured
+    if (breakfastStop || lunchStop || dinnerStop) {
+      // Get current time in Brussels timezone (in minutes since midnight)
+      // Using Intl.DateTimeFormat for reliable timezone conversion
+      const formatter = new Intl.DateTimeFormat('en-US', {
+        timeZone: 'Europe/Brussels',
+        hour: '2-digit',
+        minute: '2-digit',
+        hour12: false,
+      });
+      const parts = formatter.formatToParts(now);
+      const timeParts = Object.fromEntries(parts.map(p => [p.type, p.value]));
+      const currentTimeMinutes = parseInt(timeParts.hour, 10) * 60 + parseInt(timeParts.minute, 10);
+
+      // Check if current time has passed any stop times
+      const breakfastStopMinutes = breakfastStop ? parseTime(breakfastStop) : null;
+      const lunchStopMinutes = lunchStop ? parseTime(lunchStop) : null;
+      const dinnerStopMinutes = dinnerStop ? parseTime(dinnerStop) : null;
+
+      const breakfastStopped = breakfastStopMinutes !== null && currentTimeMinutes >= breakfastStopMinutes;
+      const lunchStopped = lunchStopMinutes !== null && currentTimeMinutes >= lunchStopMinutes;
+      const dinnerStopped = dinnerStopMinutes !== null && currentTimeMinutes >= dinnerStopMinutes;
+
+      // Remove ALL slots of a meal type if current time >= stop time for that meal
+      for (const time in availableTimeblocks) {
+        const mealType = getMealTypeByTime(time);
+        let shouldRemove = false;
+
+        if (mealType === 'breakfast' && breakfastStopped) {
+          shouldRemove = true;
+        } else if (mealType === 'lunch' && lunchStopped) {
+          shouldRemove = true;
+        } else if (mealType === 'dinner' && dinnerStopped) {
+          shouldRemove = true;
+        }
+
+        if (shouldRemove) {
+          delete availableTimeblocks[time];
+        }
+      }
+    }
+  }
+
+  return availableTimeblocks;
+}
+
+module.exports = {
+  getAvailableTimeblocks,
+};

package/grouping.js

@@ -1,162 +1,162 @@
-// grouping.js
-/**
- * Try to allocate a reservation via pinned groups.
- * Returns { tables: number[], tableIds: string[], viaGroup: string } on success,
- * null if grouping is enabled but no match (and strict=false),
- * and throws if grouping is strict and no group matches.
- *
- * Dependencies (passed in): isTemporaryTableValid, isTableFreeForAllSlots
- */
-function tryGroupTables({
-  restaurantSettings,
-  allTables,
-  guests,
-  date,
-  time,
-  requiredSlots,
-  tableOccupiedSlots,
-  isTemporaryTableValid,
-  isTableFreeForAllSlots,
-}) {
-  const gset = restaurantSettings["grouping-settings"] || {};
-  const groupingEnabled = gset.enable === true;
-  const groupingStrict = gset.strict === true;
-  const maxSubtablesPerGroup = Number.isFinite(gset.maxSubtables)
-    ? gset.maxSubtables
-    : undefined;
-  const orderStrategy = gset.orderStrategy || "smallest-first"; // 'smallest-first' | 'capacity-asc' | 'closest-fit'
-  const seed = gset.seed ?? 0;
-
-  const pinnedGroups = Array.isArray(restaurantSettings.pinnedGroups)
-    ? restaurantSettings.pinnedGroups
-    : [];
-
-  if (!groupingEnabled || pinnedGroups.length === 0) {
-    return null;
-  }
-
-  // Build lookups from allTables
-  const byId = new Map(allTables.map((t) => [String(t.tableId), t]));
-  const byNumber = new Map(
-    allTables
-      .filter((t) => typeof t.tableNumber === "number")
-      .map((t) => [t.tableNumber, t])
-  );
-
-  // Quick check: is there already a single-table fit?
-  const hasSingleFit = allTables.some(
-    (t) =>
-      t.minCapacity <= guests &&
-      guests <= t.maxCapacity &&
-      isTemporaryTableValid(t, date, time) &&
-      isTableFreeForAllSlots(t.tableNumber, requiredSlots, tableOccupiedSlots)
-  );
-  if (hasSingleFit) {
-    // Let caller handle single-table path; grouping is only used when no single table fits
-    return null;
-  }
-
-  // Stable group ordering
-  const stableGroups = [...pinnedGroups].sort((a, b) => {
-    const aid = String(a.id || a.name || "");
-    const bid = String(b.id || b.name || "");
-    return aid.localeCompare(bid);
-  });
-
-  // Deterministic RNG (kept for parity; not used by default)
-  const rnd = (() => {
-    let x = Math.imul(0x9e3779b1, (seed | 0) + 0x7f4a7c15) | 0;
-    return () => (
-      (x = (x ^ (x >>> 15)) * (x | 1)),
-      (x ^= x + Math.imul(x ^ (x >>> 7), x | 61)),
-      ((x ^ (x >>> 14)) >>> 0) / 4294967296
-    );
-  })();
-
-  // Subtable ordering strategy
-  const subSort = (a, b) => {
-    if (orderStrategy === "closest-fit") {
-      const slA = (a.maxCapacity || 0) - (a.minCapacity || 0);
-      const slB = (b.maxCapacity || 0) - (b.minCapacity || 0);
-      if (slA !== slB) return slA - slB;
-    }
-    if (a.maxCapacity !== b.maxCapacity) return a.maxCapacity - b.maxCapacity;
-    if (a.tableNumber !== b.tableNumber) return a.tableNumber - b.tableNumber;
-    const ida = String(a.tableId),
-      idb = String(b.tableId);
-    return ida.localeCompare(idb);
-  };
-
-  // Try each pinned group
-  for (const g of stableGroups) {
-    const rawIds = Array.isArray(g.tableIds) ? g.tableIds.map(String) : [];
-    const rawNums = Array.isArray(g.tableNumbers)
-      ? g.tableNumbers.filter((n) => typeof n === "number")
-      : [];
-
-    let subs = [];
-    if (rawIds.length) subs = rawIds.map((id) => byId.get(id)).filter(Boolean);
-    else if (rawNums.length)
-      subs = rawNums.map((n) => byNumber.get(n)).filter(Boolean);
-
-    if (maxSubtablesPerGroup && subs.length > maxSubtablesPerGroup) {
-      subs = subs.slice(0, maxSubtablesPerGroup);
-    }
-
-    if (subs.length < 2) continue;
-
-    // All subtables must be valid & free for all required slots
-    if (
-      !subs.every(
-        (t) =>
-          isTemporaryTableValid(t, date, time) &&
-          isTableFreeForAllSlots(
-            t.tableNumber,
-            requiredSlots,
-            tableOccupiedSlots
-          )
-      )
-    ) {
-      continue;
-    }
-
-    // Capacity checks
-    const totalMax = subs.reduce((s, t) => s + (t.maxCapacity || 0), 0);
-    const totalMin = subs.reduce((s, t) => s + (t.minCapacity || 0), 0);
-    if (guests < totalMin) continue;
-    if (guests > totalMax) continue;
-
-    // Deterministic allocation:
-    const ordered = [...subs].sort(subSort);
-    const alloc = new Map(
-      ordered.map((t) => [t.tableNumber, Math.max(0, t.minCapacity || 0)])
-    );
-    let remaining = guests - [...alloc.values()].reduce((s, v) => s + v, 0);
-    if (remaining < 0) continue;
-
-    for (const t of ordered) {
-      if (remaining <= 0) break;
-      const cur = alloc.get(t.tableNumber);
-      const addable = Math.max(0, (t.maxCapacity || 0) - cur);
-      const give = Math.min(addable, remaining);
-      alloc.set(t.tableNumber, cur + give);
-      remaining -= give;
-    }
-    if (remaining !== 0) continue; // couldn’t fit exactly
-
-    // Success
-    return {
-      tables: ordered.map((t) => t.tableNumber),
-      tableIds: ordered.map((t) => t.tableId),
-      viaGroup: String(g.id || g.name || "pinned-group"),
-    };
-  }
-
-  // No match
-  if (groupingStrict) {
-    throw new Error("No matching pinned group found (strict grouping)");
-  }
-  return null;
-}
-
-module.exports = { tryGroupTables };
+// grouping.js
+/**
+ * Try to allocate a reservation via pinned groups.
+ * Returns { tables: number[], tableIds: string[], viaGroup: string } on success,
+ * null if grouping is enabled but no match (and strict=false),
+ * and throws if grouping is strict and no group matches.
+ *
+ * Dependencies (passed in): isTemporaryTableValid, isTableFreeForAllSlots
+ */
+function tryGroupTables({
+  restaurantSettings,
+  allTables,
+  guests,
+  date,
+  time,
+  requiredSlots,
+  tableOccupiedSlots,
+  isTemporaryTableValid,
+  isTableFreeForAllSlots,
+}) {
+  const gset = restaurantSettings["grouping-settings"] || {};
+  const groupingEnabled = gset.enable === true;
+  const groupingStrict = gset.strict === true;
+  const maxSubtablesPerGroup = Number.isFinite(gset.maxSubtables)
+    ? gset.maxSubtables
+    : undefined;
+  const orderStrategy = gset.orderStrategy || "smallest-first"; // 'smallest-first' | 'capacity-asc' | 'closest-fit'
+  const seed = gset.seed ?? 0;
+
+  const pinnedGroups = Array.isArray(restaurantSettings.pinnedGroups)
+    ? restaurantSettings.pinnedGroups
+    : [];
+
+  if (!groupingEnabled || pinnedGroups.length === 0) {
+    return null;
+  }
+
+  // Build lookups from allTables
+  const byId = new Map(allTables.map((t) => [String(t.tableId), t]));
+  const byNumber = new Map(
+    allTables
+      .filter((t) => typeof t.tableNumber === "number")
+      .map((t) => [t.tableNumber, t])
+  );
+
+  // Quick check: is there already a single-table fit?
+  const hasSingleFit = allTables.some(
+    (t) =>
+      t.minCapacity <= guests &&
+      guests <= t.maxCapacity &&
+      isTemporaryTableValid(t, date, time) &&
+      isTableFreeForAllSlots(t.tableNumber, requiredSlots, tableOccupiedSlots)
+  );
+  if (hasSingleFit) {
+    // Let caller handle single-table path; grouping is only used when no single table fits
+    return null;
+  }
+
+  // Stable group ordering
+  const stableGroups = [...pinnedGroups].sort((a, b) => {
+    const aid = String(a.id || a.name || "");
+    const bid = String(b.id || b.name || "");
+    return aid.localeCompare(bid);
+  });
+
+  // Deterministic RNG (kept for parity; not used by default)
+  const rnd = (() => {
+    let x = Math.imul(0x9e3779b1, (seed | 0) + 0x7f4a7c15) | 0;
+    return () => (
+      (x = (x ^ (x >>> 15)) * (x | 1)),
+      (x ^= x + Math.imul(x ^ (x >>> 7), x | 61)),
+      ((x ^ (x >>> 14)) >>> 0) / 4294967296
+    );
+  })();
+
+  // Subtable ordering strategy
+  const subSort = (a, b) => {
+    if (orderStrategy === "closest-fit") {
+      const slA = (a.maxCapacity || 0) - (a.minCapacity || 0);
+      const slB = (b.maxCapacity || 0) - (b.minCapacity || 0);
+      if (slA !== slB) return slA - slB;
+    }
+    if (a.maxCapacity !== b.maxCapacity) return a.maxCapacity - b.maxCapacity;
+    if (a.tableNumber !== b.tableNumber) return a.tableNumber - b.tableNumber;
+    const ida = String(a.tableId),
+      idb = String(b.tableId);
+    return ida.localeCompare(idb);
+  };
+
+  // Try each pinned group
+  for (const g of stableGroups) {
+    const rawIds = Array.isArray(g.tableIds) ? g.tableIds.map(String) : [];
+    const rawNums = Array.isArray(g.tableNumbers)
+      ? g.tableNumbers.filter((n) => typeof n === "number")
+      : [];
+
+    let subs = [];
+    if (rawIds.length) subs = rawIds.map((id) => byId.get(id)).filter(Boolean);
+    else if (rawNums.length)
+      subs = rawNums.map((n) => byNumber.get(n)).filter(Boolean);
+
+    if (maxSubtablesPerGroup && subs.length > maxSubtablesPerGroup) {
+      subs = subs.slice(0, maxSubtablesPerGroup);
+    }
+
+    if (subs.length < 2) continue;
+
+    // All subtables must be valid & free for all required slots
+    if (
+      !subs.every(
+        (t) =>
+          isTemporaryTableValid(t, date, time) &&
+          isTableFreeForAllSlots(
+            t.tableNumber,
+            requiredSlots,
+            tableOccupiedSlots
+          )
+      )
+    ) {
+      continue;
+    }
+
+    // Capacity checks
+    const totalMax = subs.reduce((s, t) => s + (t.maxCapacity || 0), 0);
+    const totalMin = subs.reduce((s, t) => s + (t.minCapacity || 0), 0);
+    if (guests < totalMin) continue;
+    if (guests > totalMax) continue;
+
+    // Deterministic allocation:
+    const ordered = [...subs].sort(subSort);
+    const alloc = new Map(
+      ordered.map((t) => [t.tableNumber, Math.max(0, t.minCapacity || 0)])
+    );
+    let remaining = guests - [...alloc.values()].reduce((s, v) => s + v, 0);
+    if (remaining < 0) continue;
+
+    for (const t of ordered) {
+      if (remaining <= 0) break;
+      const cur = alloc.get(t.tableNumber);
+      const addable = Math.max(0, (t.maxCapacity || 0) - cur);
+      const give = Math.min(addable, remaining);
+      alloc.set(t.tableNumber, cur + give);
+      remaining -= give;
+    }
+    if (remaining !== 0) continue; // couldn’t fit exactly
+
+    // Success
+    return {
+      tables: ordered.map((t) => t.tableNumber),
+      tableIds: ordered.map((t) => t.tableId),
+      viaGroup: String(g.id || g.name || "pinned-group"),
+    };
+  }
+
+  // No match
+  if (groupingStrict) {
+    throw new Error("No matching pinned group found (strict grouping)");
+  }
+  return null;
+}
+
+module.exports = { tryGroupTables };