From 3d8ba756f853e30b6f5c2776f0d9a4cb4c7e3bdc Mon Sep 17 00:00:00 2001 From: "flamingo[bot]" <277372822+flamingo[bot]@users.noreply.github.com> Date: Mon, 14 Sep 2026 05:09:33 +0000 Subject: [PATCH 1/4] fix(adhoc-sweep-fixes): 4 review findings across 4 files --- .../analysis/call_graph_analyzer.py | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/codewiki/src/be/dependency_analyzer/analysis/call_graph_analyzer.py b/codewiki/src/be/dependency_analyzer/analysis/call_graph_analyzer.py index db524edf..b6d720e3 100644 --- a/codewiki/src/be/dependency_analyzer/analysis/call_graph_analyzer.py +++ b/codewiki/src/be/dependency_analyzer/analysis/call_graph_analyzer.py @@ -356,16 +356,11 @@ def _resolve_call_relationships(self): relationship.is_resolved = True resolved_count += 1 elif "." in callee_name: - if callee_name in func_lookup: - relationship.callee = func_lookup[callee_name] + method_name = callee_name.split(".")[-1] + if method_name in func_lookup: + relationship.callee = func_lookup[method_name] relationship.is_resolved = True resolved_count += 1 - else: - method_name = callee_name.split(".")[-1] - if method_name in func_lookup: - relationship.callee = func_lookup[method_name] - relationship.is_resolved = True - resolved_count += 1 def _deduplicate_relationships(self): """ @@ -539,3 +534,4 @@ def _select_most_connected_nodes(self, target_count: int): if rel.caller in selected_func_ids and rel.callee in selected_func_ids ] + From 0c111791a8422ba77212ec3f47ef654466ff75a6 Mon Sep 17 00:00:00 2001 From: "flamingo[bot]" <277372822+flamingo[bot]@users.noreply.github.com> Date: Mon, 14 Sep 2026 05:09:34 +0000 Subject: [PATCH 2/4] fix(adhoc-sweep-fixes): 4 review findings across 4 files --- codewiki/src/fe/github_processor.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/codewiki/src/fe/github_processor.py b/codewiki/src/fe/github_processor.py index a31ce6b8..53fb33e6 100644 --- a/codewiki/src/fe/github_processor.py +++ b/codewiki/src/fe/github_processor.py @@ -71,6 +71,8 @@ def clone_repository(clone_url: str, target_dir: str, commit_id: str = None) -> if result.returncode != 0: logger.error(f"Error cloning repository: {result.stderr}") + if os.path.isdir(target_dir): + shutil.rmtree(target_dir, ignore_errors=True) return False # Checkout specific commit @@ -91,6 +93,8 @@ def clone_repository(clone_url: str, target_dir: str, commit_id: str = None) -> if result.returncode != 0: logger.error(f"Error cloning repository: {result.stderr}") + if os.path.isdir(target_dir): + shutil.rmtree(target_dir, ignore_errors=True) return False return True @@ -99,3 +103,4 @@ def clone_repository(clone_url: str, target_dir: str, commit_id: str = None) -> if os.path.isdir(target_dir): shutil.rmtree(target_dir, ignore_errors=True) return False + From a9911ce7b6b033d973aac5a1d35473389d75c3b8 Mon Sep 17 00:00:00 2001 From: "flamingo[bot]" <277372822+flamingo[bot]@users.noreply.github.com> Date: Mon, 14 Sep 2026 05:09:36 +0000 Subject: [PATCH 3/4] fix(adhoc-sweep-fixes): 4 review findings across 4 files --- codewiki/src/be/documentation_generator.py | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/codewiki/src/be/documentation_generator.py b/codewiki/src/be/documentation_generator.py index ae3c7634..f0ca485a 100644 --- a/codewiki/src/be/documentation_generator.py +++ b/codewiki/src/be/documentation_generator.py @@ -201,6 +201,7 @@ async def generate_module_documentation(self, components: Dict[str, Any], leaf_n # Process modules in dependency order final_module_tree = module_tree processed_modules = set() + failed_modules = [] if len(module_tree) > 0: logger.info(f"├─ Processing {len(processing_order)} modules...") @@ -284,9 +285,20 @@ async def generate_module_documentation(self, components: Dict[str, Any], leaf_n logger.error(f"│ ├─ [{idx}/{len(processing_order)}] ❌ Failed: {module_key}") logger.error(f"│ │ └─ Error: {str(e)}") logger.error(f"│ │ └─ Traceback:\n{traceback.format_exc()}") - # Continue processing other modules (graceful degradation) + # Record the failure so the overall run can be reported/failed + # instead of silently degrading to a stale/incomplete tree. + failed_modules.append((module_key, str(e))) continue + if failed_modules: + failure_summary = "; ".join(f"{key}: {err}" for key, err in failed_modules) + logger.error( + f"├─ ❌ {len(failed_modules)}/{len(processing_order)} module(s) failed to generate: {failure_summary}" + ) + raise RuntimeError( + f"Module documentation generation failed for {len(failed_modules)} module(s): {failure_summary}" + ) + # Generate repo overview logger.info(f"├─ 📚 Generating repository overview...") import time @@ -492,4 +504,4 @@ async def run(self) -> None: except Exception as e: logger.error(f"Documentation generation failed: {str(e)}") logger.error(f"Traceback: {traceback.format_exc()}") - raise \ No newline at end of file + raise From 722f5f8e733cb95f1b3f1c54d5398425817b37bf Mon Sep 17 00:00:00 2001 From: "flamingo[bot]" <277372822+flamingo[bot]@users.noreply.github.com> Date: Mon, 14 Sep 2026 05:09:37 +0000 Subject: [PATCH 4/4] fix(adhoc-sweep-fixes): 4 review findings across 4 files --- codewiki/src/fe/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codewiki/src/fe/config.py b/codewiki/src/fe/config.py index cb041f29..8637aac9 100644 --- a/codewiki/src/fe/config.py +++ b/codewiki/src/fe/config.py @@ -22,7 +22,7 @@ class WebAppConfig: CACHE_EXPIRY_DAYS = 365 # Job cleanup settings - JOB_CLEANUP_HOURS = 24000 + JOB_CLEANUP_HOURS = 24 RETRY_COOLDOWN_MINUTES = 3 # Server settings