Update app.py

app.py

@@ -40,6 +40,10 @@ SF_INSTANCE_URL = os.getenv("SF_INSTANCE_URL")
 # Validate Salesforce environment variables
 if not all([SF_USERNAME, SF_PASSWORD, SF_SECURITY_TOKEN, SF_INSTANCE_URL]):
     logging.error("One or more Salesforce environment variables are missing. Salesforce functionality will be disabled.")
+    logging.error(f"SF_USERNAME: {SF_USERNAME}")
+    logging.error(f"SF_PASSWORD: {'Set' if SF_PASSWORD else 'Not Set'}")
+    logging.error(f"SF_SECURITY_TOKEN: {'Set' if SF_SECURITY_TOKEN else 'Not Set'}")
+    logging.error(f"SF_INSTANCE_URL: {SF_INSTANCE_URL}")
     SALESFORCE_AVAILABLE = False
 
 # Hugging Face configuration
@@ -143,7 +147,7 @@ def create_salesforce_record(score, checklist_summary, missing_summary, status,
             "QA_Report__c": qa_report
         }
 
-        logging.debug(f"Attempting to create Salesforce record: {record}")
+        logging.debug(f"Attempting to create Salesforce record with data: {record}")
         result = sf.Project_Closure_Handover__c.create(record)
         logging.info(f"Successfully created Salesforce record: {result}")
         return "Record created successfully"
@@ -249,19 +253,11 @@ def generate_pdf(score, checklist_summary, missing_summary, status, logs, qa_rep
         logging.info("Sanitizing inputs for PDF generation")
         score = str(float(score)) if score is not None else "0"
         checklist_summary = str(checklist_summary) if checklist_summary is not None else ""
-        missing_summary = str(missing_summary) if missing_summary is not None else ""
         status = str(status) if status is not None else ""
-        logs = str(logs) if logs else ""
-        qa_report = str(qa_report) if qa_report else ""
-        punch_list_text = str(punch_list_text) if punch_list_text else ""
 
         # Remove non-ASCII characters to prevent rendering issues
         checklist_summary = checklist_summary.encode('ascii', 'ignore').decode('ascii')
-        missing_summary = missing_summary.encode('ascii', 'ignore').decode('ascii')
         status = status.encode('ascii', 'ignore').decode('ascii')
-        logs = logs.encode('ascii', 'ignore').decode('ascii')
-        qa_report = qa_report.encode('ascii', 'ignore').decode('ascii')
-        punch_list_text = punch_list_text.encode('ascii', 'ignore').decode('ascii')
 
         # Define the temporary file path
         pdf_path = "readiness_report.pdf"
@@ -295,77 +291,20 @@ def generate_pdf(score, checklist_summary, missing_summary, status, logs, qa_rep
             c.drawString(50, y, line)
             y -= 20
 
-        # Draw remaining fields with Y-position checks
-        logging.info("Drawing remaining fields")
-        if y < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 20, "Missing Items:")
-        
+        # Draw signature slots
         if y - 40 < BOTTOM_MARGIN:
             logging.info("Y position below margin, creating new page")
             c.showPage()
             c.setFont("Times-Roman", 12)
             y = 750
-        c.drawString(50, y - 40, missing_summary)
+        c.drawString(50, y - 40, "Stakeholder Signature: ____________________")
 
         if y - 60 < BOTTOM_MARGIN:
             logging.info("Y position below margin, creating new page")
             c.showPage()
             c.setFont("Times-Roman", 12)
             y = 750
-        c.drawString(50, y - 60, "Logs:")
-        
-        if y - 80 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 80, logs[:100] + "..." if len(logs) > 100 else logs)
-
-        if y - 100 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 100, "QA Report:")
-        
-        if y - 120 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 120, qa_report[:100] + "..." if len(qa_report) > 100 else qa_report)
-
-        if y - 140 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 140, "Punch List:")
-        
-        if y - 160 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 160, punch_list_text[:100] + "..." if len(punch_list_text) > 100 else punch_list_text)
-
-        if y - 180 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 180, "Stakeholder Signature: ____________________")
-
-        if y - 200 < BOTTOM_MARGIN:
-            logging.info("Y position below margin, creating new page")
-            c.showPage()
-            c.setFont("Times-Roman", 12)
-            y = 750
-        c.drawString(50, y - 200, "Date: ____________________")
+        c.drawString(50, y - 60, "Date: ____________________")
 
         # Finalize the PDF
         logging.info("Finalizing PDF")
@@ -377,7 +316,7 @@ def generate_pdf(score, checklist_summary, missing_summary, status, logs, qa_rep
             raise FileNotFoundError(f"PDF file {pdf_path} was not created")
 
         logging.info(f"PDF generated successfully at {pdf_path}")
-        return pdf_path, "PDF generation triggered successfully"
+        return pdf_path, "PDF generation completed. Click the link to download."
     except Exception as e:
         logging.error(f"Error in generate_pdf: {str(e)}")
         raise
@@ -401,7 +340,7 @@ with gr.Blocks(css="""
             logs_input = gr.Textbox(label="Project Logs", lines=5, placeholder="Enter project logs (e.g., 'Project complete, handover done')")
             qa_input = gr.Textbox(label="QA Report", placeholder="Enter QA status (e.g., 'Approved')")
             punch_input = gr.Textbox(label="Punch List (Plain Text)", placeholder="Enter punch list status (e.g., 'None' or 'Resolved')")
-            submit_btn = gr.Button("Evaluate and Download PDF")
+            submit_btn = gr.Button("Evaluate and Generate PDF")
         with gr.Column(scale=3):
             score_output = gr.Number(label="Readiness Score (%)")
             progress_output = gr.HTML(label="Alert Indicator: Progress")
@@ -411,7 +350,7 @@ with gr.Blocks(css="""
                 gr.Markdown("### Handover Summary")
                 checklist_output = gr.Textbox(label="Checklist Summary")
                 missing_output = gr.Textbox(label="Missing Items")
-            pdf_output = gr.File(label="PDF Report (Auto-Download)", type="filepath", interactive=False)  # Output only, no upload
+            pdf_output = gr.File(label="Download PDF Report", type="filepath", interactive=False)  # Output only, no auto-download
             pdf_debug = gr.Textbox(label="PDF Debug Output")  # Temporary debug output
 
     # Chain the evaluation, PDF generation, and Salesforce record creation