Merge branch 'prepskills' of https://github.com/pamelafox/azure-search-openai-demo into prepskills

Author: pamelafox

app/backend/prepdocslib/page.py

@@ -64,8 +64,10 @@ def from_skill_payload(cls, data: dict[str, Any]) -> tuple["ImageOnPage", str]:
         filename = data.get("filename")
         figure_id = data.get("figure_id")
         placeholder = data.get("placeholder")
-        assert filename is not None, "filename is required"
-        assert figure_id is not None, "figure_id is required"
+        if filename is None:
+            raise ValueError("filename is required")
+        if figure_id is None:
+            raise ValueError("figure_id is required")
 
         # Generate placeholder if not provided
         if placeholder is None:

app/functions/document_extractor/function_app.py

@@ -154,7 +154,7 @@ async def extract_document(req: func.HttpRequest) -> func.HttpResponse:
                 }
             ]
         except Exception as e:
-            logger.error(f"Error processing record {record_id}: {str(e)}", exc_info=True)
+            logger.error("Error processing record %s: %s", record_id, str(e), exc_info=True)
             output_values = [
                 {
                     "recordId": record_id,
@@ -167,7 +167,7 @@ async def extract_document(req: func.HttpRequest) -> func.HttpResponse:
         return func.HttpResponse(json.dumps({"values": output_values}), mimetype="application/json", status_code=200)
 
     except Exception as e:
-        logger.error(f"Fatal error in extract_document: {str(e)}", exc_info=True)
+        logger.error("Fatal error in extract_document: %s", str(e), exc_info=True)
         return func.HttpResponse(json.dumps({"error": str(e)}), mimetype="application/json", status_code=500)