From 3d3a712ed2c7c1149fd49567a70ab12de478292d Mon Sep 17 00:00:00 2001
From: Matt <matt@letsbe.solutions>
Date: Tue, 10 Jun 2025 18:09:53 +0200
Subject: [PATCH] Fix email loading issues and improve performance

- Removed limit on cached emails - now loads ALL cached emails from MinIO
- Kept IMAP limit at 50 for reasonable performance
- Added loading indicator showing when emails are being fetched
- Fixed ES module import issue in email-utils.ts
- Improved user experience with visual feedback during loading

This ensures all email threads load completely while maintaining reasonable performance
---
 components/ClientEmailSection.vue |  8 +++++++-
 server/api/email/fetch-thread.ts  | 13 +++++--------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/components/ClientEmailSection.vue b/components/ClientEmailSection.vue
index e635f02..6001974 100644
--- a/components/ClientEmailSection.vue
+++ b/components/ClientEmailSection.vue
@@ -75,8 +75,14 @@
         </div>
       </div>
 
+      <!-- Loading State -->
+      <div v-if="isRefreshing && emailThreads.length === 0 && threads.length === 0" class="text-center py-8">
+        <v-progress-circular indeterminate color="primary" :size="48" />
+        <div class="text-body-2 mt-3 text-grey">Loading email history...</div>
+      </div>
+      
       <!-- Email Thread List -->
-      <div v-if="emailThreads.length > 0 || threads.length > 0" class="email-threads">
+      <div v-else-if="emailThreads.length > 0 || threads.length > 0" class="email-threads">
         <div class="text-subtitle-1 mb-3 d-flex align-center">
           Email History
           <v-spacer />
diff --git a/server/api/email/fetch-thread.ts b/server/api/email/fetch-thread.ts
index 7f80edc..af97bda 100644
--- a/server/api/email/fetch-thread.ts
+++ b/server/api/email/fetch-thread.ts
@@ -24,8 +24,8 @@ export default defineEventHandler(async (event) => {
 
   try {
     const body = await readBody(event);
-    // Limit emails to improve performance
-    const { clientEmail, interestId, sessionId, limit = 20 } = body;
+    // Limit IMAP emails but load all cached emails
+    const { clientEmail, interestId, sessionId, limit = 50 } = body;
 
     if (!clientEmail || !sessionId) {
       throw createError({ 
@@ -84,13 +84,10 @@ export default defineEventHandler(async (event) => {
         
         console.log('Found cached email files:', files.length);
         
-        // Limit cached emails to most recent ones
-        const sortedFiles = files
-          .filter(f => f.name.endsWith('.json'))
-          .sort((a, b) => b.name.localeCompare(a.name)) // Sort by filename (newer files have higher timestamps)
-          .slice(0, limit); // Only load up to the limit
+        // Load ALL cached emails (no limit) - just filter JSON files
+        const jsonFiles = files.filter(f => f.name.endsWith('.json'));
         
-        for (const file of sortedFiles) {
+        for (const file of jsonFiles) {
           try {
             // Read file directly on server using MinIO client (works with private buckets)
             const client = getMinioClient();