@flowselections/floriday-afhandeling-module 1.0.44 → 1.0.45

package/dist-lib/lib/cartControl.functions.js

@@ -1,6 +1,11 @@
 import { createServerFn } from "@tanstack/react-start";
 import { requireSupabaseAuth } from "../integrations/supabase/auth-middleware";
 import { tryReschedule } from "./cartRescheduling";
+/**
+ * Voorzichtige bovengrens wanneer de klantinstelling ontbreekt of ongeldig is.
+ * Klantonafhankelijk; de werkelijke grens komt uit `afhandeling_settings`.
+ */
+const CONSERVATIVE_BULK_MAX_CARTS = 10;
 function toIso(d) {
     const y = d.getFullYear();
     const m = String(d.getMonth() + 1).padStart(2, "0");
@@ -10,129 +15,130 @@ function toIso(d) {
 async function loadControlPlanningSettings(supabase, userId) {
     const { data: settings } = await supabase
         .from("afhandeling_settings")
-        .select("pick_weekdays, pick_days_before_transport")
+        .select("pick_weekdays, pick_days_before_transport, control_bulk_max_carts")
         .eq("user_id", userId)
         .maybeSingle();
     const pickWeekdays = Array.isArray(settings?.pick_weekdays) && settings.pick_weekdays.length > 0
         ? settings.pick_weekdays
         : [1, 2, 3, 4, 5];
     const minWorkdays = Math.max(0, Number(settings?.pick_days_before_transport ?? 1));
-    return { pickWeekdays, minWorkdays };
+    // Bovengrens per database-aanroep. Zonder geldige instelling geldt de
+    // voorzichtige ondergrens: nooit stil een grote portie proberen.
+    const configured = Number(settings?.control_bulk_max_carts);
+    const maxBulkCarts = Number.isFinite(configured) && configured >= 1 && configured <= 200
+        ? Math.floor(configured)
+        : CONSERVATIVE_BULK_MAX_CARTS;
+    return { pickWeekdays, minWorkdays, maxBulkCarts };
 }
 export function controlCartKey(cart) {
     return cart.cartNumberId ?? `${cart.orderSource}#${cart.orderId}#${cart.cartIndex}`;
 }
-async function loadControlCarts(supabase) {
+async function loadControlCartsDetailed(supabase) {
+    const warnings = [];
     const { data, error } = await supabase.rpc("control_get_todays_carts");
     if (error)
         throw new Error(error.message);
     const allRows = (data ?? []);
     if (allRows.length === 0)
-        return [];
-    // Alleen orders die de Verkoop-module kent mogen zichtbaar zijn.
+        return { carts: [], rpcCount: 0, filteredOut: 0, warnings };
+    // Alleen orders die de Verkoop-module kent mogen zichtbaar zijn. Beide
+    // onafhankelijke zoekpaden lopen tegelijk; dezelfde rijen leveren meteen de
+    // lichte orderverrijking voor de controlelijst.
     const candidateIds = Array.from(new Set(allRows.map((r) => r.order_id).filter(Boolean)));
     const candidateNumbers = Array.from(new Set(allRows.map((r) => r.order_number).filter((v) => !!v)));
     const knownIds = new Set();
     const knownNumbers = new Set();
-    if (candidateIds.length > 0) {
-        const { data: byId } = await supabase
-            .from("orders")
-            .select("id, order_number")
-            .is("deleted_at", null)
-            .in("id", candidateIds);
-        for (const o of (byId ?? [])) {
-            knownIds.add(o.id);
-            if (o.order_number)
-                knownNumbers.add(o.order_number);
-        }
-    }
-    if (candidateNumbers.length > 0) {
-        const { data: byNumber } = await supabase
-            .from("orders")
-            .select("id, order_number")
-            .is("deleted_at", null)
-            .in("order_number", candidateNumbers);
-        for (const o of (byNumber ?? [])) {
-            knownIds.add(o.id);
-            if (o.order_number)
-                knownNumbers.add(o.order_number);
-        }
+    const ORDER_SELECT = "id, order_number, customer_name, delivery_date, custom_fields";
+    const [byIdResult, byNumberResult] = await Promise.all([
+        candidateIds.length > 0
+            ? supabase
+                .from("orders")
+                .select(ORDER_SELECT)
+                .is("deleted_at", null)
+                .in("id", candidateIds)
+            : Promise.resolve({ data: [], error: null }),
+        candidateNumbers.length > 0
+            ? supabase
+                .from("orders")
+                .select(ORDER_SELECT)
+                .is("deleted_at", null)
+                .in("order_number", candidateNumbers)
+            : Promise.resolve({ data: [], error: null }),
+    ]);
+    if (byIdResult.error)
+        warnings.push(`Bestellingen opzoeken op id: ${byIdResult.error.message}`);
+    if (byNumberResult.error)
+        warnings.push(`Bestellingen opzoeken op nummer: ${byNumberResult.error.message}`);
+    const matchedOrders = [
+        ...(byIdResult.data ?? []),
+        ...(byNumberResult.data ?? []),
+    ];
+    for (const o of matchedOrders) {
+        knownIds.add(o.id);
+        if (o.order_number)
+            knownNumbers.add(o.order_number);
     }
     const rows = allRows.filter((r) => knownIds.has(r.order_id) || (r.order_number ? knownNumbers.has(r.order_number) : false));
-    if (rows.length === 0)
-        return [];
-    // --- Verrijking 1: ordergegevens ---
-    const orderIds = Array.from(new Set(rows.map((r) => r.order_id).filter(Boolean)));
-    const orderMap = new Map();
-    if (orderIds.length > 0) {
-        const { data: orderRows } = await supabase
-            .from("orders")
-            .select("id, order_number, customer_name, delivery_date")
-            .in("id", orderIds);
-        for (const o of (orderRows ?? [])) {
-            orderMap.set(o.id, {
-                orderNumber: o.order_number ?? "",
-                customerName: o.customer_name ?? "",
-                deliveryDate: o.delivery_date ?? null,
-            });
-        }
+    if (rows.length === 0) {
+        return { carts: [], rpcCount: allRows.length, filteredOut: allRows.length, warnings };
     }
-    const extras = new Map();
-    try {
-        const { loadTodaysPickstationOrders } = await import("./pickstationOrders.server");
-        const psOrders = await loadTodaysPickstationOrders();
-        for (const o of psOrders) {
-            extras.set(`${o.orderId}#${o.cartIndex - 1}`, {
-                cartNumber: o.cartNumber,
-                carrierName: o.carrierName,
-                pickupDate: o.pickupDate,
-                pickupTime: o.pickupTime,
-                country: o.country,
-                cratesOnCart: o.cratesOnCart,
-                printedCount: o.printedCount,
-                orderNumbers: o.orderNumber ? [o.orderNumber] : [],
-            });
-        }
-    }
-    catch {
-        // Verrijking is optioneel: zonder deze gegevens blijft de kar zichtbaar.
-    }
-    // --- Verrijking 3: totaal aantal karren per order uit de gedeelde karindex ---
-    // De karindex van de Engine is de enige bron voor "hoeveel karren heeft deze
-    // order". Lukt dat niet, dan valt het totaal terug op de karren die we hier
-    // zien (nooit een verzonnen getal).
-    const totalsByOrder = new Map();
-    try {
-        const pickDates = rows.map((r) => r.pick_date).filter(Boolean).sort();
-        if (pickDates.length > 0) {
-            const { data: idx } = await supabase.rpc("logistics_cart_index", {
-                p_from: pickDates[0],
-                p_to: pickDates[pickDates.length - 1],
-            });
-            const perRef = new Map();
-            for (const row of (Array.isArray(idx) ? idx : [])) {
-                if (!row.cart_id)
-                    continue;
-                for (const ref of [row.order_uuid, row.order_number]) {
-                    if (!ref)
-                        continue;
-                    const set = perRef.get(ref) ?? new Set();
-                    set.add(row.cart_id);
-                    perRef.set(ref, set);
-                }
-            }
-            for (const [ref, set] of perRef)
-                totalsByOrder.set(ref, set.size);
-        }
-    }
-    catch {
-        // Zonder karindex vallen we terug op de karren in deze lijst.
+    // Lichte ordergegevens uit dezelfde twee batchqueries.
+    // De Engine bewaart het gekozen ophaalmoment per order als
+    // "datum|van|tot" in custom_fields.pickup_moment (zelfde bron als het
+    // verzet-dialoog). Alleen overnemen; nooit zelf een tijd verzinnen.
+    const parsePickupMoment = (raw) => {
+        const parts = String(raw ?? "").split("|");
+        const date = (parts[0] ?? "").trim();
+        const time = (parts[1] ?? "").trim().slice(0, 5);
+        return { date: date || null, time: time || null };
+    };
+    const orderMap = new Map();
+    for (const o of matchedOrders) {
+        const moment = parsePickupMoment(o.custom_fields?.["pickup_moment"]);
+        const value = {
+            orderNumber: o.order_number ?? "",
+            customerName: o.customer_name ?? "",
+            deliveryDate: o.delivery_date ?? null,
+            pickupDate: moment.date,
+            pickupTime: moment.time,
+        };
+        orderMap.set(o.id, value);
+        if (o.order_number)
+            orderMap.set(o.order_number, value);
     }
+    // De RPC is de Engine-bron voor de zichtbare karren. Het totaal wordt alleen
+    // uit die teruggegeven, stabiele karregels afgeleid; er volgt geen tweede
+    // zware planning- of Pickstation-opbouw voordat de lijst zichtbaar wordt.
     const fallbackTotals = new Map();
     for (const r of rows) {
         const cur = fallbackTotals.get(r.order_id) ?? 0;
         fallbackTotals.set(r.order_id, Math.max(cur, r.cart_index));
     }
+    // Oorspronkelijke plandag: de Engine schuift niet-afgeronde taken elke dag
+    // mee naar vandaag. De vroegst geplande dag per order maakt achterstand
+    // zichtbaar. Mislukt dit, dan blijft de kar gewoon zichtbaar.
+    const originalByRef = new Map();
+    const refs = Array.from(new Set(rows.map((r) => r.order_number).filter((v) => !!v)));
+    if (refs.length > 0) {
+        const { data: taskRows, error: taskError } = await supabase
+            .from("logistics_tasks")
+            .select("order_ref, planned_start")
+            .in("order_ref", refs)
+            .limit(6000);
+        if (taskError) {
+            warnings.push(`Oorspronkelijke plandatum opzoeken: ${taskError.message}`);
+        }
+        else {
+            for (const t of (taskRows ?? [])) {
+                if (!t.order_ref || !t.planned_start)
+                    continue;
+                const day = String(t.planned_start).slice(0, 10);
+                const cur = originalByRef.get(t.order_ref);
+                if (!cur || day < cur)
+                    originalByRef.set(t.order_ref, day);
+            }
+        }
+    }
     const carts = [];
     const seen = new Set();
     for (const r of rows) {
@@ -147,48 +153,56 @@ async function loadControlCarts(supabase) {
         if (seen.has(key))
             continue;
         seen.add(key);
-        const order = orderMap.get(r.order_id);
-        const extra = extras.get(`${r.order_id}#${r.cart_index}`) ??
-            extras.get(`${r.order_id}#${r.cart_index - 1}`);
+        const order = orderMap.get(r.order_id) ?? (r.order_number ? orderMap.get(r.order_number) : undefined);
         const orderNumber = r.order_number ?? order?.orderNumber ?? "";
-        const orderNumbers = Array.from(new Set([orderNumber, ...(extra?.orderNumbers ?? [])].filter(Boolean)));
-        const cartTotal = totalsByOrder.get(r.order_id) ??
-            (orderNumber ? totalsByOrder.get(orderNumber) : undefined) ??
-            fallbackTotals.get(r.order_id) ??
-            null;
+        const orderNumbers = orderNumber ? [orderNumber] : [];
+        // De logistiek-engine levert de karnummering mee zodra die kolommen
+        // bestaan; anders blijft de index de bron.
+        const cartSeq = r.cart_seq ?? r.cart_index;
+        const cartTotal = r.cart_total ?? fallbackTotals.get(r.order_id) ?? null;
+        const originalPickDate = orderNumber ? originalByRef.get(orderNumber) ?? null : null;
         carts.push({
             cartNumberId,
-            cartNumber: r.cart_number ?? extra?.cartNumber ?? null,
+            cartNumber: r.cart_number ?? null,
             orderId: r.order_id,
             orderSource,
             cartIndex: r.cart_index,
-            cartSeq: r.cart_index,
+            cartSeq,
             cartTotal,
+            originalPickDate,
             orderNumber,
             orderNumbers,
             customerName: order?.customerName ?? "",
-            carrierName: extra?.carrierName ?? null,
+            carrierName: null,
             deliveryDate: order?.deliveryDate ?? null,
-            pickupDate: extra?.pickupDate ?? null,
-            pickupTime: extra?.pickupTime ?? null,
-            country: extra?.country ?? null,
-            cratesOnCart: extra?.cratesOnCart ?? null,
+            pickupDate: order?.pickupDate ?? null,
+            pickupTime: order?.pickupTime ?? null,
+            country: null,
+            cratesOnCart: null,
             pickDate: r.pick_date,
-            printedCount: r.printed_today ? Math.max(1, extra?.printedCount ?? 1) : 0,
+            printedCount: r.printed_today ? 1 : 0,
             printedToday: Boolean(r.printed_today),
             printedAt: r.printed_at ?? null,
             status: r.status ?? "pending",
         });
     }
-    return carts;
+    return {
+        carts,
+        rpcCount: allRows.length,
+        filteredOut: allRows.length - carts.length,
+        warnings,
+    };
+}
+/** Alleen de karren, voor paden die geen diagnose nodig hebben. */
+async function loadControlCarts(supabase) {
+    return (await loadControlCartsDetailed(supabase)).carts;
 }
 /** Haal alle karren op die vandaag gecontroleerd moeten worden (RPC is leidend). */
 export const getTodaysControlCarts = createServerFn({ method: "GET" })
     .middleware([requireSupabaseAuth])
     .handler(async ({ context }) => {
     const supabase = context.supabase;
-    const carts = await loadControlCarts(supabase);
-    return { carts };
+    return await loadControlCartsDetailed(supabase);
 });
 export const getControlRescheduleProposals = createServerFn({ method: "POST" })
     .middleware([requireSupabaseAuth])
@@ -252,8 +266,29 @@ export const submitControlSession = createServerFn({ method: "POST" })
     const supabase = context.supabase;
     // 1) Laad exact dezelfde karren als het scherm toont (RPC is leidend)
     const pendingOrders = await loadControlCarts(supabase);
-    const { pickWeekdays, minWorkdays } = await loadControlPlanningSettings(supabase, context.userId);
-    // 2) Maak sessie aan
+    const { pickWeekdays, minWorkdays, maxBulkCarts } = await loadControlPlanningSettings(supabase, context.userId);
+    // 2) Maak sessie aan. Het aanvraag-kenmerk maakt opnieuw indienen veilig:
+    // dezelfde controle levert nooit twee sessies of dubbele regels op.
+    const requestKey = data.requestKey?.trim() || null;
+    if (requestKey) {
+        const { data: existing } = await supabase
+            .from("cart_control_sessions")
+            .select("id, checked_count, rescheduled_count, blocked_count")
+            .eq("user_id", context.userId)
+            .eq("request_key", requestKey)
+            .maybeSingle();
+        if (existing) {
+            return {
+                sessionId: existing.id,
+                checkedCount: existing.checked_count ?? 0,
+                rescheduledCount: existing.rescheduled_count ?? 0,
+                blockedCount: existing.blocked_count ?? 0,
+                rescheduled: [],
+                blocked: [],
+                stillOpen: [],
+            };
+        }
+    }
     const { data: session, error: sessErr } = await supabase
         .from("cart_control_sessions")
         .insert({
@@ -261,6 +296,7 @@ export const submitControlSession = createServerFn({ method: "POST" })
         control_date: toIso(new Date()),
         total_carts: pendingOrders.length,
         notes: data.notes ?? null,
+        request_key: requestKey,
     })
         .select()
         .single();
@@ -270,165 +306,259 @@ export const submitControlSession = createServerFn({ method: "POST" })
     const confirmedRescheduleSet = new Set(data.confirmedRescheduleCartKeys ?? []);
     const items = [];
     const deliveredPayload = [];
+    const overrides = [];
+    const checkedByKey = new Map();
     const rescheduled = [];
     const blocked = [];
+    const stillOpen = [];
     let checkedCount = 0;
-    for (const o of pendingOrders) {
-        const key = controlCartKey(o);
-        const cartNumberId = o.cartNumberId;
-        const baseFields = {
-            session_id: session.id,
-            cart_number_id: cartNumberId,
-            cart_number_label: o.cartNumber,
-            order_id: o.orderId,
-            order_source: o.orderSource,
-            cart_index: o.cartIndex,
-            order_number: o.orderNumber,
-            customer_name: o.customerName,
-            carrier_name: o.carrierName,
-            delivery_date: o.deliveryDate,
-            pickup_date: o.pickupDate,
-        };
-        if (checkedSet.has(key)) {
-            // De controleregel is leidend voor Engine-karren. Werk daarnaast de
-            // verouderde cart_numbers-status bij wanneer die bron dit ondersteunt.
-            deliveredPayload.push({
+    try {
+        for (const o of pendingOrders) {
+            const key = controlCartKey(o);
+            const cartNumberId = o.cartNumberId;
+            const baseFields = {
+                session_id: session.id,
+                cart_number_id: cartNumberId,
+                cart_number_label: o.cartNumber,
                 order_id: o.orderId,
                 order_source: o.orderSource,
                 cart_index: o.cartIndex,
-                cart_number: o.cartNumber,
-                pick_date: o.pickDate || toIso(new Date()),
+                order_number: o.orderNumber,
+                customer_name: o.customerName,
+                carrier_name: o.carrierName,
+                delivery_date: o.deliveryDate,
+                pickup_date: o.pickupDate,
+            };
+            if (checkedSet.has(key)) {
+                // De controleregel is leidend voor Engine-karren. Werk daarnaast de
+                // verouderde cart_numbers-status bij wanneer die bron dit ondersteunt.
+                deliveredPayload.push({
+                    order_id: o.orderId,
+                    order_source: o.orderSource,
+                    cart_index: o.cartIndex,
+                    // Het label uit de Engine is géén karnummer: dat kent
+                    // assign_cart_numbers toe (enige nummeringspad, onder advisory lock).
+                    engine_cart_label: o.cartNumber,
+                    pick_date: o.pickDate || toIso(new Date()),
+                });
+                checkedByKey.set(`${o.orderSource}#${o.orderId}#${o.cartIndex}`, o);
+                items.push({ ...baseFields, action: "checked" });
+                checkedCount++;
+                continue;
+            }
+            // Niet aangevinkt → alleen verplaatsen wanneer de gebruiker dit expliciet
+            // in de bevestigings-pop-up heeft gekozen. Zonder bevestiging blijft de
+            // kar op de huidige pickdatum staan en registreren we uitsluitend een
+            // blokkade/waarschuwing.
+            const result = tryReschedule({
+                currentPickDate: new Date(o.pickDate || toIso(new Date())),
+                deliveryDate: o.deliveryDate ? new Date(o.deliveryDate) : null,
+                pickWeekdays,
+                minWorkdaysBeforeDelivery: minWorkdays,
             });
-            items.push({ ...baseFields, action: "checked" });
-            checkedCount++;
-            continue;
+            if (result.ok && confirmedRescheduleSet.has(key)) {
+                const newIso = toIso(result.newPickDate);
+                overrides.push({
+                    order_id: o.orderId,
+                    order_source: o.orderSource,
+                    cart_index: o.cartIndex,
+                    pick_date: newIso,
+                    moved_from_date: toIso(new Date()),
+                });
+                items.push({
+                    ...baseFields,
+                    action: "rescheduled",
+                    new_pick_date: newIso,
+                });
+                rescheduled.push({
+                    key,
+                    cartNumber: o.cartNumber,
+                    orderNumber: o.orderNumber,
+                    customerName: o.customerName,
+                    newPickDate: newIso,
+                });
+            }
+            else {
+                const blockReason = result.ok
+                    ? `Verplaatsing naar ${toIso(result.newPickDate)} is voorgesteld, maar niet bevestigd. De kar blijft op de huidige pickdatum staan.`
+                    : result.reason;
+                items.push({
+                    ...baseFields,
+                    action: "blocked",
+                    block_reason: blockReason,
+                });
+                blocked.push({
+                    cartNumberId,
+                    cartNumber: o.cartNumber,
+                    orderId: o.orderId,
+                    orderSource: o.orderSource,
+                    cartIndex: o.cartIndex,
+                    orderNumber: o.orderNumber,
+                    customerName: o.customerName,
+                    carrierName: o.carrierName,
+                    deliveryDate: o.deliveryDate,
+                    pickupDate: o.pickupDate,
+                    reason: blockReason,
+                });
+            }
         }
-        // Niet aangevinkt → alleen verplaatsen wanneer de gebruiker dit expliciet
-        // in de bevestigings-pop-up heeft gekozen. Zonder bevestiging blijft de
-        // kar op de huidige pickdatum staan en registreren we uitsluitend een
-        // blokkade/waarschuwing.
-        const result = tryReschedule({
-            currentPickDate: new Date(o.pickDate || toIso(new Date())),
-            deliveryDate: o.deliveryDate ? new Date(o.deliveryDate) : null,
-            pickWeekdays,
-            minWorkdaysBeforeDelivery: minWorkdays,
-        });
-        if (result.ok && confirmedRescheduleSet.has(key)) {
-            const newIso = toIso(result.newPickDate);
-            await supabase
+        // Verplaatsingen van niet-afgevinkte karren in één keer wegschrijven.
+        if (overrides.length > 0) {
+            const { error: overrideError } = await supabase
                 .from("cart_pick_overrides")
+                .upsert(overrides, { onConflict: "order_id,order_source,cart_index" });
+            if (overrideError)
+                throw new Error(overrideError.message);
+        }
+        // Afgevinkte karren krijgen de status 'afgeleverd'. Eén begrensde
+        // bulk-aanroep, daarna één gezamenlijke verificatie en één gerichte naloop
+        // over uitsluitend de karren waarvan geverifieerd is dat ze nog openstaan.
+        // De naloop draait maximaal één keer; wat daarna nog openstaat wordt per kar
+        // gemeld en verdwijnt dus nooit stil.
+        const chunk = (rows, size) => {
+            const out = [];
+            for (let i = 0; i < rows.length; i += size)
+                out.push(rows.slice(i, i + size));
+            return out;
+        };
+        const payloadKey = (cart) => `${String(cart.order_source ?? "")}#${String(cart.order_id ?? "")}#${Number(cart.cart_index ?? 0)}`;
+        const verifyOpen = async (carts) => {
+            const orderIds = Array.from(new Set(carts.map((cart) => String(cart.order_id ?? ""))));
+            const rows = [];
+            for (const idBatch of chunk(orderIds, 200)) {
+                const { data: deliveredRows, error: verifyError } = await supabase
+                    .from("cart_numbers")
+                    .select("order_id, order_source, cart_index, status, delivered_at")
+                    .in("order_id", idBatch);
+                if (verifyError)
+                    throw new Error(`Afleverstatus controleren: ${verifyError.message}`);
+                rows.push(...(deliveredRows ?? []));
+            }
+            const deliveredKeys = new Set(rows
+                .filter((row) => row.status === "delivered" && row.delivered_at)
+                .map((row) => payloadKey(row)));
+            return carts.filter((cart) => !deliveredKeys.has(payloadKey(cart)));
+        };
+        let deliverError = null;
+        const runBulk = async (carts) => {
+            for (const batch of chunk(carts, maxBulkCarts)) {
+                const { error } = await supabase.rpc("control_mark_carts_delivered", { p_carts: batch });
+                if (error && !deliverError)
+                    deliverError = error.message;
+            }
+        };
+        let openCarts = [];
+        if (deliveredPayload.length > 0) {
+            await runBulk(deliveredPayload);
+            openCarts = await verifyOpen(deliveredPayload);
+            if (openCarts.length > 0) {
+                await runBulk(openCarts);
+                openCarts = await verifyOpen(openCarts);
+            }
+        }
+        // Wat nog openstaat blijft zichtbaar: de kar wordt in deze sessie als
+        // 'nog niet afgemeld' vastgelegd en in het scherm benoemd.
+        const openKeys = new Set(openCarts.map(payloadKey));
+        if (openKeys.size > 0) {
+            const openReason = `Deze kar is nog niet afgemeld door de logistiek-engine${deliverError ? ` (${deliverError})` : ""}. De kar blijft in de lijst staan en wordt bij een volgende controle opnieuw aangeboden.`;
+            for (const item of items) {
+                const key = `${String(item.order_source ?? "")}#${String(item.order_id ?? "")}#${Number(item.cart_index ?? 0)}`;
+                if (item.action !== "checked" || !openKeys.has(key))
+                    continue;
+                item.action = "blocked";
+                item.block_reason = openReason;
+                checkedCount--;
+                const cart = checkedByKey.get(key);
+                if (cart) {
+                    const entry = {
+                        cartNumberId: cart.cartNumberId,
+                        cartNumber: cart.cartNumber,
+                        orderId: cart.orderId,
+                        orderSource: cart.orderSource,
+                        cartIndex: cart.cartIndex,
+                        orderNumber: cart.orderNumber,
+                        customerName: cart.customerName,
+                        carrierName: cart.carrierName,
+                        deliveryDate: cart.deliveryDate,
+                        pickupDate: cart.pickupDate,
+                        reason: openReason,
+                    };
+                    blocked.push(entry);
+                    stillOpen.push(entry);
+                }
+            }
+        }
+        const ITEM_BATCH = 200;
+        for (let i = 0; i < items.length; i += ITEM_BATCH) {
+            const { error: itemsError } = await supabase
+                .from("cart_control_items")
+                .insert(items.slice(i, i + ITEM_BATCH));
+            if (itemsError)
+                throw new Error(itemsError.message);
+        }
+        // Update sessie-aantallen
+        const { error: sessionUpdateError } = await supabase
+            .from("cart_control_sessions")
+            .update({
+            checked_count: checkedCount,
+            rescheduled_count: rescheduled.length,
+            blocked_count: blocked.length,
+        })
+            .eq("id", session.id);
+        if (sessionUpdateError)
+            throw new Error(sessionUpdateError.message);
+        // Kritieke melding aanmaken of oplossen
+        const alertKey = `cart_not_rescheduled:${toIso(new Date())}`;
+        if (blocked.length > 0) {
+            const relatedIds = blocked
+                .map((b) => b.cartNumberId)
+                .filter((v) => !!v);
+            await supabase
+                .from("dashboard_critical_alerts")
                 .upsert({
-                order_id: o.orderId,
-                order_source: o.orderSource,
-                cart_index: o.cartIndex,
-                pick_date: newIso,
-                moved_from_date: toIso(new Date()),
-            }, { onConflict: "order_id,order_source,cart_index" });
-            items.push({
-                ...baseFields,
-                action: "rescheduled",
-                new_pick_date: newIso,
-            });
-            rescheduled.push({
-                key,
-                cartNumber: o.cartNumber,
-                orderNumber: o.orderNumber,
-                customerName: o.customerName,
-                newPickDate: newIso,
-            });
+                user_id: context.userId,
+                alert_key: alertKey,
+                alert_type: "cart_not_rescheduled",
+                priority: "critical",
+                title: "Karren kunnen niet doorgeschoven worden",
+                description: `Er zijn ${blocked.length} karren die vandaag niet zijn klaargezet en niet automatisch naar een volgende pickdag kunnen worden verplaatst. Directe actie is vereist.`,
+                related_cart_ids: relatedIds,
+                resolved_at: null,
+                metadata: { blocked: blocked },
+            }, { onConflict: "user_id,alert_key" });
         }
         else {
-            const blockReason = result.ok
-                ? `Verplaatsing naar ${toIso(result.newPickDate)} is voorgesteld, maar niet bevestigd. De kar blijft op de huidige pickdatum staan.`
-                : result.reason;
-            items.push({
-                ...baseFields,
-                action: "blocked",
-                block_reason: blockReason,
-            });
-            blocked.push({
-                cartNumberId,
-                cartNumber: o.cartNumber,
-                orderId: o.orderId,
-                orderSource: o.orderSource,
-                cartIndex: o.cartIndex,
-                orderNumber: o.orderNumber,
-                customerName: o.customerName,
-                carrierName: o.carrierName,
-                deliveryDate: o.deliveryDate,
-                pickupDate: o.pickupDate,
-                reason: blockReason,
-            });
+            // Auto-resolve als er geen blokkades zijn
+            await supabase
+                .from("dashboard_critical_alerts")
+                .update({ resolved_at: new Date().toISOString() })
+                .eq("user_id", context.userId)
+                .eq("alert_key", alertKey)
+                .is("resolved_at", null);
         }
-    }
-    // Afgevinkte karren krijgen de status 'afgeleverd' met het moment van controle,
-    // zodat andere modules alleen de karren van vandaag als afgeleverd tonen.
-    if (deliveredPayload.length > 0) {
-        const { error: deliveredError } = await supabase.rpc("control_mark_carts_delivered", {
-            p_carts: deliveredPayload,
-        });
-        if (deliveredError)
-            throw new Error(deliveredError.message);
-    }
-    if (items.length > 0) {
-        const { error: itemsError } = await supabase.from("cart_control_items").insert(items);
-        if (itemsError)
-            throw new Error(itemsError.message);
-    }
-    // Update sessie-aantallen
-    const { error: sessionUpdateError } = await supabase
-        .from("cart_control_sessions")
-        .update({
-        checked_count: checkedCount,
-        rescheduled_count: rescheduled.length,
-        blocked_count: blocked.length,
-    })
-        .eq("id", session.id);
-    if (sessionUpdateError)
-        throw new Error(sessionUpdateError.message);
-    // Kritieke melding aanmaken of oplossen
-    const alertKey = `cart_not_rescheduled:${toIso(new Date())}`;
-    if (blocked.length > 0) {
-        const relatedIds = blocked
-            .map((b) => b.cartNumberId)
-            .filter((v) => !!v);
+        // Definitieve controle: wis alleen het autosave-concept van deze gebruiker voor vandaag.
         await supabase
-            .from("dashboard_critical_alerts")
-            .upsert({
-            user_id: context.userId,
-            alert_key: alertKey,
-            alert_type: "cart_not_rescheduled",
-            priority: "critical",
-            title: "Karren kunnen niet doorgeschoven worden",
-            description: `Er zijn ${blocked.length} karren die vandaag niet zijn klaargezet en niet automatisch naar een volgende pickdag kunnen worden verplaatst. Directe actie is vereist.`,
-            related_cart_ids: relatedIds,
-            resolved_at: null,
-            metadata: { blocked: blocked },
-        }, { onConflict: "user_id,alert_key" });
-    }
-    else {
-        // Auto-resolve als er geen blokkades zijn
-        await supabase
-            .from("dashboard_critical_alerts")
-            .update({ resolved_at: new Date().toISOString() })
+            .from("cart_control_draft")
+            .delete()
             .eq("user_id", context.userId)
-            .eq("alert_key", alertKey)
-            .is("resolved_at", null);
+            .eq("draft_date", toIso(new Date()));
+        return {
+            sessionId: session.id,
+            checkedCount,
+            rescheduledCount: rescheduled.length,
+            blockedCount: blocked.length,
+            rescheduled,
+            blocked,
+            stillOpen,
+        };
+    }
+    catch (error) {
+        // Een niet-afgeronde poging hoort niet als lege controlesessie in de
+        // historie te blijven staan. De al afgeleverde karren blijven veilig
+        // afgeleverd; een volgende poging behandelt alleen wat nog openstaat.
+        await supabase.from("cart_control_sessions").delete().eq("id", session.id);
+        throw error;
     }
-    // Definitieve controle: wis alleen het autosave-concept van deze gebruiker voor vandaag.
-    await supabase
-        .from("cart_control_draft")
-        .delete()
-        .eq("user_id", context.userId)
-        .eq("draft_date", toIso(new Date()));
-    return {
-        sessionId: session.id,
-        checkedCount,
-        rescheduledCount: rescheduled.length,
-        blockedCount: blocked.length,
-        rescheduled,
-        blocked,
-    };
 });
 export const listControlHistory = createServerFn({ method: "GET" })
     .middleware([requireSupabaseAuth])