fix: the file download error is fixed

bot_telegram.py

@@ -95,7 +95,8 @@ class TelegramBot:
                 "You can:\n"
                 "1. Send me any question as text\n"
                 "2. Send me an Excel file with questions (must have a 'question' column in 'rfp' sheet)\n\n"
-                "Note: Excel files must contain no more than 50 questions."
+                "Note: Excel files must contain no more than 50 questions.\n\n"
+                "type '/status' to check the status of the request"
             )
         else:
             AWAITING_PASSWORD.add(user_id)
@@ -210,7 +211,7 @@ class TelegramBot:
         asyncio.create_task(self._process_excel_file(update, context, message_id))
 
     async def _process_excel_file(self, update, context, message_id):
-        """Process Excel file with progress updates but no timeout"""
+        """Process Excel file with progress updates"""
         try:
             document = update.message.document
             
@@ -218,53 +219,65 @@ class TelegramBot:
             processing_msg = await update.message.reply_text(
                 f"📊 Starting Excel processing...\n"
                 f"File: {document.file_name}\n"
-                f"Request ID: #{message_id}\n"
-                f"Processing time may vary depending on file size and complexity. type /status to check the status of the request."
+                f"Request ID: #{message_id}"
             )
 
+            # Step 1: Download file - ADD THIS PART
+            try:
+                file = await context.bot.get_file(document.file_id)
+                file_bytes = await file.download_as_bytearray()  # Define file_bytes here
+            except Exception as e:
+                logging.error(f"Error downloading file: {e}")
+                await processing_msg.edit_text(
+                    f"❌ Error downloading file: {document.file_name}\n"
+                    f"Please try again or contact support."
+                )
+                return
+
+            # Step 2: Update status to processing
             self.active_excel_files[message_id] = {
                 'filename': document.file_name,
                 'status': 'processing',
                 'start_time': time.time()
             }
+            await processing_msg.edit_text(
+                f"⚙️ Processing Excel file...\n"
+                f"File: {document.file_name}\n"
+                f"Request ID: #{message_id}"
+            )
 
-            # Process in thread pool without timeout
-            try:
-                # Start a background task to update progress message
-                asyncio.create_task(self._update_progress(processing_msg, message_id, document.file_name))
-
-                result = await asyncio.get_event_loop().run_in_executor(
-                    self.excel_executor,
-                    self._process_excel_sync,
-                    file_bytes,
-                    document.file_name
-                )
-
-                if result is None:
-                    await processing_msg.edit_text(
-                        f"❌ Failed to process file\n"
-                        f"File: {document.file_name}\n"
-                        f"Please check the file format and try again."
-                    )
-                    return
-
-                # Send processed file
-                await context.bot.send_document(
-                    chat_id=update.effective_chat.id,
-                    document=io.BytesIO(result),
-                    filename=f'processed_{document.file_name}',
-                    caption=f"✅ Excel processing completed!\nRequest ID: #{message_id}"
-                )
-                await processing_msg.delete()
+            # Step 3: Process in thread pool
+            result = await asyncio.get_event_loop().run_in_executor(
+                self.excel_executor,
+                self._process_excel_sync,
+                file_bytes,  # Now file_bytes is defined
+                document.file_name
+            )
 
-            except Exception as e:
-                logging.error(f"Error processing file: {e}")
+            if result is None:
                 await processing_msg.edit_text(
-                    f"❌ Error processing file\n"
+                    f"❌ Failed to process file\n"
                     f"File: {document.file_name}\n"
-                    f"Error: {str(e)}"
+                    f"Please check the file format and try again."
                 )
+                return
+
+            # Send processed file
+            await context.bot.send_document(
+                chat_id=update.effective_chat.id,
+                document=io.BytesIO(result),
+                filename=f'processed_{document.file_name}',
+                caption=f"✅ Excel processing completed!\nRequest ID: #{message_id}"
+            )
+            await processing_msg.delete()
 
+        except Exception as e:
+            logging.error(f"Error processing file: {e}")
+            await processing_msg.edit_text(
+                f"❌ Error processing file\n"
+                f"File: {document.file_name}\n"
+                f"Error: {str(e)}"
+            )
         finally:
             if message_id in self.active_excel_files:
                 self.active_excel_files[message_id]['status'] = 'completed'