fixing linting

Author: aronchick

.pre-commit-config.yaml

@@ -15,7 +15,7 @@ repos:
     hooks:
       - id: ruff
         name: Ruff auto-fix
-        args: [--fix, --select, "F,E,W,I", --ignore, "E501,W293"]  # Only critical issues
+        args: [--fix, --select, "F,E,W,I,TRY", --ignore, "E501,W293,TRY401"]  # Include TRY rules, ignore redundant exception in log
       - id: ruff-format
         name: Ruff formatter
 

CLAUDE.md

@@ -88,10 +88,10 @@ The system uses a nested identity format:
 ## Development Guidelines
 - **CRITICAL: Pre-commit hooks are configured for practical development**
   - Setup: `uv run scripts/setup.py --dev`
-  - On commit: Auto-fixes formatting, checks syntax, removes trailing whitespace, runs type checking
+  - On commit: Auto-fixes formatting, checks syntax, removes trailing whitespace, runs type checking, checks exception handling (TRY rules)
   - On push: Runs tests and security scans
   - Manual full check: `uv run scripts/check.py --fix`
-- Always use ruff for python lint checking
+- Always use ruff for python lint checking (includes TRY rules for proper exception handling)
 - Always use uv, instead of python, when writing python scripts
 - **Type checking with mypy**:
   - Runs automatically on every commit via pre-commit hooks

src/database.py

@@ -228,8 +228,8 @@ def commit_batch(self):
             self.batch_buffer = []
             self.last_commit_time = time.time()
 
-        except sqlite3.Error as e:
-            self.logger.exception(f"Failed to commit batch: {e}")
+        except sqlite3.Error:
+            self.logger.exception("Failed to commit batch")
             raise
         else:
             # Return the count of committed records
@@ -367,8 +367,8 @@ def get_readings(self, limit: int | None = None) -> list:
             columns = [desc[0] for desc in cursor.description]
             rows = cursor.fetchall()
             return [dict(zip(columns, row, strict=False)) for row in rows]
-        except Exception as e:
-            self.logger.exception(f"Failed to get readings: {e}")
+        except Exception:
+            self.logger.exception("Failed to get readings")
             return []
 
     def get_reading_stats(self) -> dict:
@@ -422,8 +422,8 @@ def get_database_stats(self) -> dict:
                     "avg_insert_time_ms": 0.0,  # Not tracking this
                 },
             }
-        except Exception as e:
-            self.logger.exception(f"Failed to get database stats: {e}")
+        except Exception:
+            self.logger.exception("Failed to get database stats")
             return {
                 "total_readings": 0,
                 "anomaly_count": 0,