From 94f8b022a9cdc28770c028b532b2eb85199802b7 Mon Sep 17 00:00:00 2001 From: "flamingo[bot]" <277372822+flamingo[bot]@users.noreply.github.com> Date: Wed, 23 Sep 2026 03:21:06 +0000 Subject: [PATCH] fix(adhoc-codewiki-src-be-dependency-analyzer-analysis-cloning-py): Bare except clauses in clone_repository silently swallow git config failures on Windows, hiding real errors --- codewiki/src/be/dependency_analyzer/analysis/cloning.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/codewiki/src/be/dependency_analyzer/analysis/cloning.py b/codewiki/src/be/dependency_analyzer/analysis/cloning.py index df36942f..93d30ebf 100644 --- a/codewiki/src/be/dependency_analyzer/analysis/cloning.py +++ b/codewiki/src/be/dependency_analyzer/analysis/cloning.py @@ -106,8 +106,8 @@ def clone_repository(github_url: str) -> str: capture_output=True, text=True, ) - except: - pass + except (OSError, subprocess.SubprocessError) as e: + logger.debug(f"Failed to set git core.longpaths on Windows: {e}") subprocess.run( [ @@ -158,8 +158,8 @@ def clone_repository(github_url: str) -> str: capture_output=True, text=True, ) - except: - pass + except (OSError, subprocess.SubprocessError) as e: + logger.debug(f"Failed to configure sparse-checkout on Windows: {e}") return temp_dir except subprocess.TimeoutExpired: if os.path.exists(temp_dir):