From 5d285b9ffc1c4b0f573f6506d71bb2843e0e56e5 Mon Sep 17 00:00:00 2001 From: Codeflash Bot Date: Fri, 14 Nov 2025 17:22:48 -0800 Subject: [PATCH 1/2] submodule bugs Signed-off-by: Codeflash Bot --- codeflash/discovery/functions_to_optimize.py | 5 ++++- codeflash/tracing/tracing_utils.py | 5 ++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/codeflash/discovery/functions_to_optimize.py b/codeflash/discovery/functions_to_optimize.py index 48a453479..fed48199b 100644 --- a/codeflash/discovery/functions_to_optimize.py +++ b/codeflash/discovery/functions_to_optimize.py @@ -405,7 +405,10 @@ def is_git_repo(file_path: str) -> bool: def ignored_submodule_paths(module_root: str) -> list[str]: if is_git_repo(module_root): git_repo = git.Repo(module_root, search_parent_directories=True) - return [Path(git_repo.working_tree_dir, submodule.path).resolve() for submodule in git_repo.submodules] + try: + return [Path(git_repo.working_tree_dir, submodule.path).resolve() for submodule in git_repo.submodules] + except Exception as e: + logger.warning(f"Error getting submodule paths: {e}") return [] diff --git a/codeflash/tracing/tracing_utils.py b/codeflash/tracing/tracing_utils.py index 0114ea01c..b14eb2581 100644 --- a/codeflash/tracing/tracing_utils.py +++ b/codeflash/tracing/tracing_utils.py @@ -38,7 +38,10 @@ def ignored_submodule_paths(module_root: str) -> list[Path]: if is_git_repo(module_root): git_repo = git.Repo(module_root, search_parent_directories=True) working_tree_dir = cast("Path", git_repo.working_tree_dir) - return [Path(working_tree_dir, submodule.path).resolve() for submodule in git_repo.submodules] + try: + return [Path(working_tree_dir, submodule.path).resolve() for submodule in git_repo.submodules] + except Exception as e: + print(f"Failed to get submodule paths {str(e)}") # no logger since used in the tracer return [] From 036d6f6d29e273445376067378fd573580b4c619 Mon Sep 17 00:00:00 2001 From: Codeflash Bot Date: Fri, 14 Nov 2025 17:28:15 -0800 Subject: [PATCH 2/2] submodule bugs Signed-off-by: Codeflash Bot --- codeflash/tracing/tracing_utils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codeflash/tracing/tracing_utils.py b/codeflash/tracing/tracing_utils.py index b14eb2581..ac8b3c888 100644 --- a/codeflash/tracing/tracing_utils.py +++ b/codeflash/tracing/tracing_utils.py @@ -41,7 +41,7 @@ def ignored_submodule_paths(module_root: str) -> list[Path]: try: return [Path(working_tree_dir, submodule.path).resolve() for submodule in git_repo.submodules] except Exception as e: - print(f"Failed to get submodule paths {str(e)}") # no logger since used in the tracer + print(f"Failed to get submodule paths {e!s}") # no logger since used in the tracer return []