fix: replace bare except clauses with except Exception (#344)

Bare `except:` catches BaseException including KeyboardInterrupt and
SystemExit. This replaces 9 bare except clauses with
`except Exception:` to only catch application-level exceptions.

Co-authored-by: haosenwang1018 <haosenwang1018@users.noreply.github.com>

Author: haosenwang1018

opencontext/context_consumption/generation/smart_todo_manager.py

@@ -92,7 +92,7 @@ def generate_todo_tasks(self, start_time: int, end_time: int) -> Optional[str]:
                             deadline = datetime.datetime.strptime(deadline_str, "%Y-%m-%d %H:%M")
                         else:
                             deadline = datetime.datetime.strptime(task["due_date"], "%Y-%m-%d")
-                    except:
+                    except Exception:
                         pass
 
                 todo_id = get_storage().insert_todo(

opencontext/llm/llm_client.py

@@ -439,7 +439,7 @@ def _extract_error_summary(error: Any) -> str:
                                         if ". Request id:" in actual_msg:
                                             actual_msg = actual_msg.split(". Request id:")[0]
                                         return actual_msg
-                            except:
+                            except Exception:
                                 pass
                         return f"Error {code}"
 

opencontext/monitoring/metrics_collector.py

@@ -48,7 +48,7 @@ def wrapper(*args, **kwargs):
                     if hasattr(result, "__len__") and not isinstance(result, str):
                         try:
                             context_count = len(result)
-                        except:
+                        except Exception:
                             context_count = 1
 
                     monitor.record_processing_metrics(
@@ -94,7 +94,7 @@ def wrapper(*args, **kwargs):
                     elif hasattr(result, "__len__") and not isinstance(result, str):
                         try:
                             snippets_count = len(result)
-                        except:
+                        except Exception:
                             snippets_count = 0
 
                     # 尝试从参数中获取query

opencontext/storage/backends/sqlite_backend.py

@@ -951,7 +951,7 @@ def save_monitoring_token_usage(
             logger.error(f"Failed to save token usage: {e}")
             try:
                 self.connection.rollback()
-            except:
+            except Exception:
                 pass
             return False
 
@@ -1047,7 +1047,7 @@ def save_monitoring_stage_timing(
             logger.error(f"Failed to save stage timing: {e}")
             try:
                 self.connection.rollback()
-            except:
+            except Exception:
                 pass
             return False
 
@@ -1087,7 +1087,7 @@ def save_monitoring_data_stats(
             logger.error(f"Failed to save data stats: {e}")
             try:
                 self.connection.rollback()
-            except:
+            except Exception:
                 pass
             return False
 
@@ -1313,7 +1313,7 @@ def cleanup_old_monitoring_data(self, days: int = 7) -> bool:
             logger.error(f"Failed to cleanup old monitoring data: {e}")
             try:
                 self.connection.rollback()
-            except:
+            except Exception:
                 pass
             return False
 

opencontext/utils/json_parser.py

@@ -101,5 +101,5 @@ def fix_quotes_in_match(match):
     try:
         fixed = re.sub(pattern, fix_quotes_in_match, json_str)
         return fixed
-    except:
+    except Exception:
         return json_str