@flowselections/floriday-verkoop-module 1.0.60 → 1.0.62

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

@@ -44,7 +44,9 @@ function mapSalesOrderRow(record, connectionId) {
         data: record,
     };
 }
-async function runSalesOrdersSyncInternal() {
+async function runSalesOrdersSyncInternal(budget) {
+    const budgetMs = budget?.budgetMs ?? 25000;
+    const mirrorLimit = budget?.mirrorLimit ?? 500;
     const { supabaseAdmin } = await import("../integrations/supabase/client.server");
     const connections = await loadActiveConnections(supabaseAdmin);
     const results = [];
@@ -58,7 +60,7 @@ async function runSalesOrdersSyncInternal() {
                 cacheTable: CACHE_TABLE,
                 conflictTarget: "connection_id,floriday_id",
                 mapRow: mapSalesOrderRow,
-                budgetMs: 25000,
+                budgetMs,
             });
             results.push(r);
         }
@@ -79,13 +81,16 @@ async function runSalesOrdersSyncInternal() {
             });
         }
     }
-    // Spiegel recent gewijzigde FO's naar de lokale orders-tabel zodat
-    // buffergroepering en gedeelde karren werken.
+    // Nawerk (spiegelen naar lokale orders) hoort bij de cron-ronde, niet bij de
+    // knop. Karplanning doet de Logistiek Engine zelf in haar minuut-cron.
+    if (budget?.fetchOnly) {
+        return { ok: results.every((r) => r.ok), results, mirrorOk: true, mirrored: 0 };
+    }
     let mirrorOk = true;
     let mirrored = 0;
     try {
         const { mirrorFloridayOrdersFromCron } = await import("./floriday-mirror.functions");
-        const m = (await mirrorFloridayOrdersFromCron(500));
+        const m = (await mirrorFloridayOrdersFromCron(mirrorLimit));
         mirrorOk = m?.ok !== false;
         mirrored = Number(m?.created ?? 0) + Number(m?.updated ?? 0) || Number(m?.processed ?? 0) || 0;
     }
@@ -93,28 +98,18 @@ async function runSalesOrdersSyncInternal() {
         mirrorOk = false;
         console.error("[floriday-mirror] failed", mirrorErr);
     }
-    // Direct doorpakken: karplanning voor de zojuist binnengekomen orders,
-    // zodat de gebruiker niet op de volgende klokslag hoeft te wachten.
-    let replanOk = true;
-    let replanned = 0;
-    try {
-        const { drainCartReplanQueue } = await import("./cart-replan.functions");
-        const r = await drainCartReplanQueue(100);
-        replanOk = r.ok;
-        replanned = r.processed;
-    }
-    catch (replanErr) {
-        replanOk = false;
-        console.error("[cart-replan] failed", replanErr);
-    }
-    return { ok: results.every((r) => r.ok), results, mirrorOk, mirrored, replanOk, replanned };
+    return { ok: results.every((r) => r.ok), results, mirrorOk, mirrored };
 }
 export async function runSalesOrdersSyncFromCron() {
     return runSalesOrdersSyncInternal();
 }
+// Handmatig ophalen via de knop doet één ding: verse orders bij Floriday
+// ophalen, met een hard tijdsbudget van 8 seconden.
+const syncInput = z.object({ quick: z.boolean().optional() });
 export const runSalesOrdersSync = createServerFn({ method: "POST" })
     .middleware([requireSupabaseAuth])
-    .handler(async () => runSalesOrdersSyncInternal());
+    .inputValidator((input) => syncInput.parse(input ?? {}))
+    .handler(async ({ data }) => runSalesOrdersSyncInternal(data?.quick ? { budgetMs: 8000, fetchOnly: true } : undefined));
 // ─── Write actions ────────────────────────────────────────────────────────
 const CreateOrderSchema = z.object({
     customerOrganizationId: z.string().min(1),
@@ -236,6 +231,7 @@ export const getSalesOrderDetail = createServerFn({ method: "POST" })
     .inputValidator((d) => DetailInputSchema.parse(d))
     .handler(async ({ data, context }) => {
     const supabase = context.supabase;
+    const { supabaseAdmin } = await import("../integrations/supabase/client.server");
     // 1) Actieve environment + connectie.
     const envRes = await supabase
         .from("floriday_settings")
@@ -244,7 +240,11 @@ export const getSalesOrderDetail = createServerFn({ method: "POST" })
         .maybeSingle();
     const rawEnv = String(envRes.data?.active_environment ?? "staging");
     const environment = rawEnv === "production" || rawEnv === "prod" || rawEnv === "live" ? "live" : "staging";
-    const connRes = await supabase
+    // De connectietabel bevat ook geheime configuratie en heeft daarom geen
+    // directe leespolicy voor gebruikers. Lees uitsluitend de veilige velden
+    // server-side; alle order- en klantdata hieronder blijft via de ingelogde
+    // gebruiker en dus onder RLS lopen.
+    const connRes = await supabaseAdmin
         .from("floriday_connections")
         .select("id, organization_name, label")
         .eq("is_active", true)