From 50404f62d9a7ea76b4dd34029d58e72085a92499 Mon Sep 17 00:00:00 2001
From: Russell Ballestrini <russell@unturf.com>
Date: Sun, 7 Dec 2025 15:34:35 -0500
Subject: [PATCH] Fix F824 lint error and add lint instructions to CLAUDE.md

---
 CLAUDE.md | 10 ++++++++++
 app.py    |  2 +-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index b5642de..d7da992 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -17,6 +17,15 @@
 - Write tests for new functionality when appropriate
 - Verify changes work as expected
 
+## Linting
+- **ALWAYS run lint before committing**: `make lint` or `flake8 app.py activity.py --select=E9,F63,F7,F82`
+- Fix all lint errors before pushing - GitHub CI will fail on lint errors
+- Key error codes checked:
+  - E9: Runtime errors (syntax errors, IO errors)
+  - F63: Invalid print syntax
+  - F7: Syntax errors in type comments
+  - F82: Undefined names, unused globals (F824)
+
 ## Documentation
 - Update relevant documentation when making significant changes
 - Keep README files current with new features or setup changes
@@ -41,6 +50,7 @@
 - `make venv` - Create virtual environment and install dependencies
 - `make init-db` - Initialize database tables
 - `make test` - Run all tests
+- `make lint` - Run code linting (black, isort, flake8)
 - `make dev-setup` - Install development dependencies
 
 ### Network Infrastructure
diff --git a/app.py b/app.py
index 5ccadb4..9fb3915 100644
--- a/app.py
+++ b/app.py
@@ -104,7 +104,7 @@ def is_vision_model(model_name: str) -> bool:
 
 
 def initialize_model_map():
-    global SYSTEM_USERS, VISION_MODELS
+    global SYSTEM_USERS
     MODEL_CLIENT_MAP.clear()
     VISION_MODELS.clear()
     for ep_config in ENDPOINTS: