Use logger.exception for cleaner exception logging

Drops redundant type(e).__name__/str(e) args; traceback now preserved.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: J03D03

vfc_datasets/transformations/enrichment/add_no_comment.py

@@ -279,8 +279,8 @@ def _process_batch(args: tuple[str, list[str], int, bool]) -> dict[str, str]:
                 )
                 if diff is not None:
                     results[commit_id] = diff
-    except Exception as e:
-        logger.error("Batch error for %s: %s: %s", repo_path, type(e).__name__, e)
+    except Exception:
+        logger.exception("Batch error for %s", repo_path)
 
     return results
 

vfc_datasets/utils/git/repository.py

@@ -306,8 +306,8 @@ def _strategy_for(url: str) -> CloneStrategy:
                         failed_urls.append(url)
                         pbar.set_postfix_str(f"Failed: {len(failed_urls)}")
 
-                except Exception as e:
-                    logger.exception("Exception cloning %s: %s: %s", url, type(e).__name__, e)
+                except Exception:
+                    logger.exception("Exception cloning %s", url)
                     results[url] = None
                     failed_urls.append(url)
                     pbar.set_postfix_str(f"Failed: {len(failed_urls)}")