From 25d8f549ecc8a8aea3739b2229bf206ad03accda Mon Sep 17 00:00:00 2001
From: Dominic Ferrando <dominicf2001@gmail.com>
Date: Fri, 17 Jul 2026 15:38:28 -0400
Subject: [PATCH] Make products table more consistent with other tables

---
 apps/api/src/server.ts                        |  30 ++-
 apps/api/src/services/commerce.ts             |   9 +
 apps/portal/src/lib/printful.ts               |   7 -
 apps/portal/src/lib/webflow.ts                |  12 -
 .../routes/(app)/apparel/orders/+page.svelte  |   2 +-
 .../(app)/apparel/products/+page.server.ts    |  12 -
 .../(app)/apparel/products/+page.svelte       | 214 ++++++++++++------
 7 files changed, 171 insertions(+), 115 deletions(-)
 delete mode 100644 apps/portal/src/lib/printful.ts
 delete mode 100644 apps/portal/src/lib/webflow.ts
 delete mode 100644 apps/portal/src/routes/(app)/apparel/products/+page.server.ts

diff --git a/apps/api/src/server.ts b/apps/api/src/server.ts
index 55104a4..34d7a46 100644
--- a/apps/api/src/server.ts
+++ b/apps/api/src/server.ts
@@ -13,7 +13,7 @@ import {
 import { DEFAULT_NAME, env, log } from "./util";
 import serverTiming from "@elysia/server-timing";
 import jwt from "@elysia/jwt";
-import { CommerceService } from "./services/commerce";
+import { CommerceService, WOrderStatusSchema } from "./services/commerce";
 import cluster from "node:cluster";
 import { randomUUIDv7, sleep } from "bun";
 import { CalculatorService, CalculatorUnavailableError } from "./services/calculator";
@@ -201,6 +201,25 @@ export const app = new Elysia()
     // COMMERCE
     .group("/commerce", { auth: true }, (app) => app
         .group("/products", (app) => app
+            .get("/", async ({ query }) => {
+                const pProducts = await s.Commerce.Printful.Products.list({
+                    limit: query.limit,
+                    offset: query.offset,
+                });
+                const isSyncedFlags = await Promise.all(
+                    pProducts.map(async (pProduct) => {
+                        const wProductId = pProduct.external_id.split("-")[0];
+                        if (!wProductId) return false;
+                        return (await s.Commerce.Webflow.Products.get(wProductId)) !== undefined;
+                    })
+                );
+                return pProducts.map((pProduct, i) => ({ pProduct, isSynced: isSyncedFlags[i] }));
+            }, {
+                query: t.Object({
+                    limit: t.Optional(t.Numeric()),
+                    offset: t.Optional(t.Numeric()),
+                })
+            })
             .group("/sync", (app) => app
                 // Sync status
                 .get("/", async ({ sessionId }) => {
@@ -251,14 +270,7 @@ export const app = new Elysia()
                 return wOrders.map((wOrder, i) => ({ wOrder, isSynced: isSyncedFlags[i] }));
             }, {
                 query: t.Object({
-                    status: t.Optional(t.Union([
-                        t.Literal("pending"),
-                        t.Literal("unfulfilled"),
-                        t.Literal("fulfilled"),
-                        t.Literal("disputed"),
-                        t.Literal("dispute-lost"),
-                        t.Literal("refunded"),
-                    ])),
+                    status: t.Optional(WOrderStatusSchema),
                     limit: t.Optional(t.Numeric()),
                     offset: t.Optional(t.Numeric()),
                 })
diff --git a/apps/api/src/services/commerce.ts b/apps/api/src/services/commerce.ts
index 8327fb7..27f39b0 100644
--- a/apps/api/src/services/commerce.ts
+++ b/apps/api/src/services/commerce.ts
@@ -9,6 +9,15 @@ import zipcodesUs from "zipcodes-us";
 import { secToMs } from "@blade-and-brawn/domain";
 import { EventsService } from "./events";
 
+export const WOrderStatusSchema = t.Union([
+    t.Literal("pending"),
+    t.Literal("unfulfilled"),
+    t.Literal("fulfilled"),
+    t.Literal("disputed"),
+    t.Literal("dispute-lost"),
+    t.Literal("refunded"),
+]);
+
 export class CommerceService {
     readonly Printful = new PrintfulClient({
         token: env.PRINTFUL_AUTH,
diff --git a/apps/portal/src/lib/printful.ts b/apps/portal/src/lib/printful.ts
deleted file mode 100644
index 783df2f..0000000
--- a/apps/portal/src/lib/printful.ts
+++ /dev/null
@@ -1,7 +0,0 @@
-import { PrintfulClient } from "@blade-and-brawn/commerce";
-import { env } from "$env/dynamic/private";
-
-export const printful = new PrintfulClient({
-    token: env.PRINTFUL_AUTH,
-    storeId: env.PRINTFUL_STORE_ID
-});
diff --git a/apps/portal/src/lib/webflow.ts b/apps/portal/src/lib/webflow.ts
deleted file mode 100644
index 245866b..0000000
--- a/apps/portal/src/lib/webflow.ts
+++ /dev/null
@@ -1,12 +0,0 @@
-import { WebflowClient } from "@blade-and-brawn/commerce";
-import { env } from "$env/dynamic/private";
-
-export const webflow = new WebflowClient({
-    siteId: env.WEBFLOW_SITE_ID,
-    collectionIds: {
-        products: env.WEBFLOW_COLLECTION_PRODUCTS_ID,
-        skus: env.WEBFLOW_COLLECTION_SKUS_ID,
-    },
-    token: env.WEBFLOW_AUTH,
-    webhookSecret: env.WEBFLOW_WEBHOOK_SECRET,
-});
diff --git a/apps/portal/src/routes/(app)/apparel/orders/+page.svelte b/apps/portal/src/routes/(app)/apparel/orders/+page.svelte
index a43bc33..ac766ac 100644
--- a/apps/portal/src/routes/(app)/apparel/orders/+page.svelte
+++ b/apps/portal/src/routes/(app)/apparel/orders/+page.svelte
@@ -6,7 +6,7 @@
         Awaited<ReturnType<typeof api.commerce.orders.get>>["data"]
     >[number];
 
-    const LIMIT = 25;
+    const LIMIT = 5;
 
     let orders = $state<OrderRow[]>([]);
     let loading = $state(true);
diff --git a/apps/portal/src/routes/(app)/apparel/products/+page.server.ts b/apps/portal/src/routes/(app)/apparel/products/+page.server.ts
deleted file mode 100644
index 1276095..0000000
--- a/apps/portal/src/routes/(app)/apparel/products/+page.server.ts
+++ /dev/null
@@ -1,12 +0,0 @@
-import type { PageServerLoad } from "./$types";
-import { printful } from "$lib/printful";
-import { webflow } from "$lib/webflow";
-
-export const load = async ({ }: Parameters<PageServerLoad>[0]) => {
-    return {
-        products: {
-            printful: printful.Products.list({ forceAll: true }),
-            webflow: webflow.Products.list({ forceAll: true }),
-        },
-    };
-};
diff --git a/apps/portal/src/routes/(app)/apparel/products/+page.svelte b/apps/portal/src/routes/(app)/apparel/products/+page.svelte
index e05af92..72f3ca9 100644
--- a/apps/portal/src/routes/(app)/apparel/products/+page.svelte
+++ b/apps/portal/src/routes/(app)/apparel/products/+page.svelte
@@ -1,13 +1,58 @@
 <script lang="ts">
-    import type { Printful } from "@blade-and-brawn/commerce";
     import { onMount } from "svelte";
     import { api } from "$lib/api.js";
     import type { EventStatus } from "@blade-and-brawn/api";
 
-    const { data } = $props();
+    type ProductRow = NonNullable<
+        Awaited<ReturnType<typeof api.commerce.products.get>>["data"]
+    >[number];
 
     type SyncStatus = EventStatus | "none";
 
+    const LIMIT = 25;
+
+    let products = $state<ProductRow[]>([]);
+    let loading = $state(true);
+    let loadError = $state<string | null>(null);
+    let offset = $state(0);
+
+    function errorMessage(err: unknown): string {
+        const value = (err as { value?: { error?: string } })?.value;
+        return (
+            value?.error ??
+            (err instanceof Error ? err.message : "Unknown error")
+        );
+    }
+
+    async function refresh() {
+        loading = true;
+        loadError = null;
+        try {
+            const res = await api.commerce.products.get({
+                query: {
+                    limit: LIMIT,
+                    offset,
+                },
+            });
+            if (res.error) throw res.error;
+            products = res.data as ProductRow[];
+        } catch (err) {
+            loadError = errorMessage(err);
+        } finally {
+            loading = false;
+        }
+    }
+
+    function prevPage() {
+        offset = Math.max(0, offset - LIMIT);
+        refresh();
+    }
+
+    function nextPage() {
+        offset += LIMIT;
+        refresh();
+    }
+
     const synchronizer = $state({
         status: "none" as SyncStatus,
         syncingPProductIds: [] as number[],
@@ -30,8 +75,12 @@
         },
         startPolling: () => {
             const intervalId = setInterval(async () => {
+                const wasInProgress = synchronizer.isInProgress;
                 await synchronizer.poll();
-                if (!synchronizer.isInProgress) clearInterval(intervalId);
+                if (!synchronizer.isInProgress) {
+                    clearInterval(intervalId);
+                    if (wasInProgress) refresh();
+                }
             }, 100);
         },
         syncAll: async () => {
@@ -49,23 +98,15 @@
         if (synchronizer.isInProgress) {
             synchronizer.startPolling();
         }
+        await refresh();
     });
-
-    const isProductSynced = async (pProduct: Printful.Products.SyncProduct) => {
-        const wProducts = await data.products.webflow;
-        return wProducts.some(
-            (p) => p.product.id === pProduct.external_id.split("-")[0],
-        );
-    };
 </script>
 
-{#await data.products.printful}
-    <p>Loading...</p>
-{:then pProducts}
+<div class="w-full flex items-center gap-2 flex-wrap mt-10 mb-5">
     <button
         disabled={synchronizer.isInProgress}
         onclick={() => synchronizer.syncAll()}
-        class="btn btn-xl mb-5 mt-10"
+        class="btn btn-xl"
     >
         {#if synchronizer.isInProgress}
             Syncing...
@@ -73,64 +114,89 @@
             Sync all
         {/if}
     </button>
-    <div class="w-full overflow-x-auto">
-        <table class="table">
-            <thead>
-                <tr>
-                    <th>Name</th>
-                    <th>Status</th>
-                    <th>Sync</th>
-                </tr>
-            </thead>
-            <tbody>
-                {#each pProducts as pProduct}
-                    <tr class="hover:bg-base-300">
-                        <td
-                            ><button
-                                onclick={async () => {
-                                    const res = await api.commerce
-                                        .products({
-                                            pProductId: pProduct.id,
-                                        })
-                                        .get();
-                                    console.log(res.data);
-                                }}
-                                class="cursor-pointer underline"
-                            >
-                                {pProduct.name}
-                            </button></td
-                        >
-                        <td>
-                            {#await isProductSynced(pProduct) then isSynced}
-                                {#if synchronizer.syncingPProductIds.includes(pProduct.id)}
-                                    <div class="badge badge-warning">
-                                        Syncing...
-                                    </div>
-                                {:else if isSynced}
-                                    <div class="badge badge-success">
-                                        Synced
-                                    </div>
-                                {:else}
-                                    <div class="badge badge-error">
-                                        Desynced
-                                    </div>
-                                {/if}
-                            {/await}
-                        </td>
-                        <td>
-                            <button
-                                disabled={synchronizer.status === "processing"}
-                                onclick={() => synchronizer.sync(pProduct.id)}
-                                class="btn"
-                            >
-                                Sync
-                            </button>
-                        </td>
-                    </tr>
-                {/each}
-            </tbody>
-        </table>
+
+    <button class="btn btn-sm" onclick={refresh} disabled={loading}>
+        {loading ? "Refreshing..." : "Refresh"}
+    </button>
+
+    <div class="flex-1"></div>
+
+    <button
+        class="btn btn-sm"
+        onclick={prevPage}
+        disabled={offset === 0 || loading}
+    >
+        Prev
+    </button>
+    <button
+        class="btn btn-sm"
+        onclick={nextPage}
+        disabled={products.length < LIMIT || loading}
+    >
+        Next
+    </button>
+</div>
+
+{#if loadError}
+    <div role="alert" class="alert alert-error w-full mb-4">
+        <span>{loadError}</span>
     </div>
-{:catch error}
-    <p>Something went wrong: {error.message}</p>
-{/await}
+{/if}
+
+<div class="w-full overflow-x-auto">
+    <table class="table">
+        <thead>
+            <tr>
+                <th>Name</th>
+                <th>Status</th>
+                <th>Sync</th>
+            </tr>
+        </thead>
+        <tbody>
+            {#each products as product (product.pProduct.id)}
+                <tr class="hover:bg-base-300">
+                    <td
+                        ><button
+                            onclick={async () => {
+                                const res = await api.commerce
+                                    .products({
+                                        pProductId: product.pProduct.id,
+                                    })
+                                    .get();
+                                console.log(res.data);
+                            }}
+                            class="cursor-pointer underline"
+                        >
+                            {product.pProduct.name}
+                        </button></td
+                    >
+                    <td>
+                        {#if synchronizer.syncingPProductIds.includes(product.pProduct.id)}
+                            <div class="badge badge-warning">Syncing...</div>
+                        {:else if product.isSynced}
+                            <div class="badge badge-success">Synced</div>
+                        {:else}
+                            <div class="badge badge-error">Desynced</div>
+                        {/if}
+                    </td>
+                    <td>
+                        <button
+                            disabled={synchronizer.status === "processing"}
+                            onclick={() =>
+                                synchronizer.sync(product.pProduct.id)}
+                            class="btn"
+                        >
+                            Sync
+                        </button>
+                    </td>
+                </tr>
+            {:else}
+                <tr>
+                    <td colspan="3" class="text-center opacity-60">
+                        {loading ? "Loading..." : "No products found"}
+                    </td>
+                </tr>
+            {/each}
+        </tbody>
+    </table>
+</div>