fix: remove test comments and add logging for JSON parsing errors in mapper

Author: Zalfsten

middleware/sql_to_arc/src/middleware/sql_to_arc/main.py

@@ -563,5 +563,3 @@ async def main() -> None:
 if __name__ == "__main__":
     multiprocessing.freeze_support()
     asyncio.run(main())
-# test unstaged
-# commit test

middleware/sql_to_arc/src/middleware/sql_to_arc/mapper.py

@@ -1,6 +1,7 @@
 """Mapper module to convert database rows to ARCTRL objects."""
 
 import json
+import logging
 from datetime import datetime
 from typing import Any, cast
 
@@ -13,6 +14,8 @@
     Publication,
 )
 
+logger = logging.getLogger(__name__)
+
 
 def map_investigation(row: dict[str, Any]) -> ArcInvestigation:
     """Map a database row to an ArcInvestigation object.
@@ -107,8 +110,13 @@ def map_contact(row: dict[str, Any]) -> Person:
                             tan=r.get("uri"),
                         )
                     )
-        except (json.JSONDecodeError, TypeError):
-            # Fallback for invalid JSON or type mismatch
+        except (json.JSONDecodeError, TypeError) as e:
+            # Fallback for invalid JSON or type mismatch, with logging
+            logger.warning(
+                "Could not parse roles JSON for contact with email '%s'. Error: %s",
+                row.get("email", "N/A"),
+                e,
+            )
             pass
 
     return Person.create(