@liiift-studio/sales-portal 3.1.1 → 3.1.3

package/api/getSales.js

@@ -2,7 +2,7 @@
 import { authenticateDesigner, processSalesData } from './utils/salesDataProcessor';
 import { sendError, requirePost } from './utils/apiResponse';
 
-export const config = { maxDuration: 300 };
+export const config = { maxDuration: 800 };
 
 export default async function handler(req, res) {
 	if (!requirePost(req, res)) return;

package/api/utils/stripeFetcher.js

@@ -1,9 +1,34 @@
 /**
- * Utility functions for fetching data from Stripe API
+ * Utility functions for fetching data from Stripe API with batched secondary lookups
  */
 
 import { stripe } from './clients';
 
+/** Maximum number of concurrent secondary API calls (disputes, refunds) */
+const CONCURRENCY_LIMIT = 10;
+
+/**
+ * Runs an array of async tasks with a concurrency limit
+ * @param {Array} items - Items to process
+ * @param {Function} fn - Async function to call for each item
+ * @param {number} limit - Max concurrent tasks
+ * @returns {Promise<Array>} Results in original order
+ */
+async function mapWithConcurrency(items, fn, limit = CONCURRENCY_LIMIT) {
+	const results = [];
+	let index = 0;
+
+	async function next() {
+		const i = index++;
+		if (i >= items.length) return;
+		results[i] = await fn(items[i], i);
+		await next();
+	}
+
+	await Promise.all(Array.from({ length: Math.min(limit, items.length) }, () => next()));
+	return results;
+}
+
 /**
  * Fetches dispute details including all balance transactions
  * @param {Object} dispute - Stripe dispute object
@@ -12,28 +37,24 @@ import { stripe } from './clients';
 async function fetchDisputeDetails(dispute) {
 	if (!dispute) return null;
 
-	// Handle dispute as either a string ID or an object with .id
 	const disputeId = typeof dispute === 'string' ? dispute : dispute.id;
 	if (!disputeId) return null;
 
-	// Get dispute with balance transactions
 	const fullDispute = await stripe.disputes.retrieve(disputeId, {
 		expand: ['balance_transactions']
 	});
 
-	// If there's a dispute charge, get its balance transaction
 	if (fullDispute.charge) {
-		const disputeCharge = await stripe.charges.retrieve(fullDispute.charge, {
-			expand: ['balance_transaction']
-		});
-
-		// Get all balance transactions for this dispute
-		const balanceTransactions = await stripe.balanceTransactions.list({
-			type: 'dispute_fee',
-			source: disputeCharge.id
-		});
+		const [disputeCharge, balanceTransactions] = await Promise.all([
+			stripe.charges.retrieve(fullDispute.charge, {
+				expand: ['balance_transaction']
+			}),
+			stripe.balanceTransactions.list({
+				type: 'dispute_fee',
+				source: fullDispute.charge
+			})
+		]);
 
-		// Add balance transactions to the dispute object
 		fullDispute.charge = disputeCharge;
 		fullDispute.balance_transactions = [
 			...(fullDispute.balance_transactions || []),
@@ -44,6 +65,90 @@ async function fetchDisputeDetails(dispute) {
 	return fullDispute;
 }
 
+/**
+ * Enriches a batch of invoices with dispute and refund balance transaction details.
+ * Uses concurrency-limited parallel calls instead of sequential per-invoice awaits.
+ * @param {Array} invoices - Raw invoices from Stripe list
+ * @returns {Promise<Array>} Enriched invoices
+ */
+async function enrichInvoices(invoices) {
+	return mapWithConcurrency(invoices, async (invoice) => {
+		const tasks = [];
+
+		// Dispute details
+		if (invoice.charge?.disputed) {
+			tasks.push(
+				fetchDisputeDetails(invoice.charge.dispute).then(d => {
+					invoice.charge.dispute = d;
+				})
+			);
+		}
+
+		// Refund balance transactions — batch all refunds for this invoice into one Promise.all
+		if (invoice.charge?.refunds?.data?.length) {
+			tasks.push(
+				Promise.all(
+					invoice.charge.refunds.data.map(refund =>
+						stripe.refunds.retrieve(refund.id, { expand: ['balance_transaction'] })
+					)
+				).then(fullRefunds => {
+					invoice.charge.refunds.data = fullRefunds;
+				})
+			);
+		}
+
+		// Only re-fetch payment_intent if Stripe returned a bare string ID (not expanded)
+		if (typeof invoice.payment_intent === 'string') {
+			tasks.push(
+				stripe.paymentIntents.retrieve(invoice.payment_intent, {
+					expand: ['payment_method']
+				}).then(pi => {
+					invoice.payment_intent = pi;
+				}).catch(err => {
+					console.warn(`Could not expand payment intent for invoice ${invoice.id}:`, err.message);
+				})
+			);
+		}
+
+		if (tasks.length) await Promise.all(tasks);
+		return invoice;
+	});
+}
+
+/**
+ * Enriches a batch of payment intents with dispute and refund balance transaction details.
+ * @param {Array} payments - Raw payment intents from Stripe list
+ * @returns {Promise<Array>} Enriched payment intents
+ */
+async function enrichPaymentIntents(payments) {
+	return mapWithConcurrency(payments, async (payment) => {
+		const tasks = [];
+
+		if (payment.latest_charge?.disputed) {
+			tasks.push(
+				fetchDisputeDetails(payment.latest_charge.dispute).then(d => {
+					payment.latest_charge.dispute = d;
+				})
+			);
+		}
+
+		if (payment.latest_charge?.refunds?.data?.length) {
+			tasks.push(
+				Promise.all(
+					payment.latest_charge.refunds.data.map(refund =>
+						stripe.refunds.retrieve(refund.id, { expand: ['balance_transaction'] })
+					)
+				).then(fullRefunds => {
+					payment.latest_charge.refunds.data = fullRefunds;
+				})
+			);
+		}
+
+		if (tasks.length) await Promise.all(tasks);
+		return payment;
+	});
+}
+
 /**
  * Fetches all pages of Stripe invoices for a given date range
  * @param {Object} timeRange - Stripe-compatible date range
@@ -88,45 +193,13 @@ export async function fetchAllInvoices(timeRange, options = {}) {
 					? response.data.filter(invoice => invoice?.metadata?.authors?.includes(options.designerId))
 					: response.data;
 
-				// For each invoice with a dispute, fetch additional details
-				const processedInvoices = await Promise.all(invoices.map(async invoice => {
-					if (invoice.charge?.disputed) {
-						invoice.charge.dispute = await fetchDisputeDetails(invoice.charge.dispute);
-					}
-
-					// For each refund, fetch its balance transaction
-					if (invoice.charge?.refunds?.data?.length) {
-						const refundsWithTransactions = await Promise.all(
-							invoice.charge.refunds.data.map(async refund => {
-								const fullRefund = await stripe.refunds.retrieve(refund.id, {
-									expand: ['balance_transaction']
-								});
-								return fullRefund;
-							})
-						);
-						invoice.charge.refunds.data = refundsWithTransactions;
-					}
-
-					// If payment_intent is a string, try to fetch the full object
-					if (invoice.payment_intent && typeof invoice.payment_intent === 'string') {
-						try {
-							const paymentIntent = await stripe.paymentIntents.retrieve(invoice.payment_intent, {
-								expand: ['payment_method']
-							});
-							invoice.payment_intent = paymentIntent;
-							} catch (error) {
-							console.warn(`Could not expand payment intent for invoice ${invoice.id}:`, error.message);
-						}
-					}
-
-					return invoice;
-				}));
-
-				allInvoices = allInvoices.concat(processedInvoices);
+				// Enrich invoices with dispute/refund details (batched, concurrency-limited)
+				const enriched = await enrichInvoices(invoices);
+				allInvoices = allInvoices.concat(enriched);
+
 				lastId = response.data[response.data.length - 1].id;
 				hasMore = response.has_more;
 				pageCount++;
-
 			} else {
 				hasMore = false;
 			}
@@ -179,36 +252,16 @@ export async function fetchAllPaymentIntents(timeRange, options = {}) {
 				// Filter succeeded payments and by designer if needed
 				const payments = response.data
 					.filter(pi => pi.status === 'succeeded')
-					.filter(pi => !options.filterByDesigner || !options.designerId || 
+					.filter(pi => !options.filterByDesigner || !options.designerId ||
 						pi?.metadata?.authors?.includes(options.designerId));
 
-				// For each payment with a dispute, fetch additional details
-				const processedPayments = await Promise.all(payments.map(async payment => {
-					if (payment.latest_charge?.disputed) {
-						payment.latest_charge.dispute = await fetchDisputeDetails(payment.latest_charge.dispute);
-					}
-
-					// For each refund, fetch its balance transaction
-					if (payment.latest_charge?.refunds?.data?.length) {
-						const refundsWithTransactions = await Promise.all(
-							payment.latest_charge.refunds.data.map(async refund => {
-								const fullRefund = await stripe.refunds.retrieve(refund.id, {
-									expand: ['balance_transaction']
-								});
-								return fullRefund;
-							})
-						);
-						payment.latest_charge.refunds.data = refundsWithTransactions;
-					}
-
-					return payment;
-				}));
-
-				allPayments = allPayments.concat(processedPayments);
+				// Enrich payments with dispute/refund details (batched, concurrency-limited)
+				const enriched = await enrichPaymentIntents(payments);
+				allPayments = allPayments.concat(enriched);
+
 				lastId = response.data[response.data.length - 1].id;
 				hasMore = response.has_more;
 				pageCount++;
-
 			} else {
 				hasMore = false;
 			}

package/components/Insights.js

@@ -259,7 +259,7 @@ export default function Insights({
 			
 			{/* Metric cards */}
 			{metricCards.map((card, index) => (
-				<Box key={`metric-${index}`} sx={{ width: { xs: '100%', sm: 'calc(50% - 8px)', md: 'calc(33.33% - 11px)' } }}>
+				<Box key={`metric-${index}`} sx={{ flex: { xs: '1 1 100%', sm: '1 1 calc(50% - 8px)', md: '1 1 calc(33.33% - 11px)' } }}>
 					<Tooltip 
 						title={card.tooltip} 
 						placement="top" 

package/components/LoginForm.js

@@ -1,6 +1,8 @@
 // Login form component for sales portal authentication
 import React, { useEffect, useState } from 'react';
-import { Typography, Input, Button, Box } from '@mui/material';
+import { Typography, Input, Button, Box, IconButton, InputAdornment } from '@mui/material';
+import VisibilityIcon from '@mui/icons-material/Visibility';
+import VisibilityOffIcon from '@mui/icons-material/VisibilityOff';
 import packageJson from '../package.json';
 
 const { version } = packageJson;
@@ -27,6 +29,7 @@ export default function LoginForm({
 	const [password, setPassword] = useState('');
 	const [message, setMessage] = useState('');
 	const [loading, setLoading] = useState(false);
+	const [showPassword, setShowPassword] = useState(false);
 
 	const {
 		title = 'Sales Portal',
@@ -128,7 +131,7 @@ export default function LoginForm({
 							{title}
 						</Typography>
 
-						<Box sx={{ px: { xs: 4, sm: 6 } }}>
+						<Box sx={{ px: { xs: 0, sm: 6 } }}>
 							<Input
 								placeholder="Email"
 								onChange={(e) => setUser(e.target.value)}
@@ -160,9 +163,21 @@ export default function LoginForm({
 								placeholder="Password"
 								onChange={(e) => setPassword(e.target.value)}
 								value={password}
-								type="password"
+								type={showPassword ? 'text' : 'password'}
 								autoComplete="current-password"
 								onKeyPress={handleKeyPress}
+								endAdornment={
+									<InputAdornment position="end">
+										<IconButton
+											onClick={() => setShowPassword(!showPassword)}
+											edge="end"
+											sx={{ color: 'var(--grey800, #666)', mr: 0.5 }}
+											aria-label={showPassword ? 'Hide password' : 'Show password'}
+										>
+											{showPassword ? <VisibilityOffIcon /> : <VisibilityIcon />}
+										</IconButton>
+									</InputAdornment>
+								}
 								sx={{
 									m: 0,
 									border: 'none',

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@liiift-studio/sales-portal",
-	"version": "3.1.1",
+	"version": "3.1.3",
 	"description": "Centralized sales portal package for Liiift Studio projects",
 	"main": "index.js",
 	"types": "index.d.ts",