Update services/auth.py

services/auth.py

@@ -19,6 +19,7 @@ def get_user(db: Session, username: str) -> Optional[User]:
     user = db.exec(statement).first()
     return user
 
+# In services/auth.py
 def create_user_in_db(user_data: UserCreate) -> Optional[User]:
     hashed_password = get_password_hash(user_data.password)
     db_user = User(
@@ -26,28 +27,20 @@ def create_user_in_db(user_data: UserCreate) -> Optional[User]:
         hashed_password=hashed_password,
         email=user_data.email,
         full_name=user_data.full_name
+        # The 'chat_sessions' attribute is an empty list by default here, not yet loaded from DB
     )
     try:
-        with get_session_context() as db:
-            # Check if user already exists
+        with get_session_context() as db: # Session starts here
             existing_user = get_user(db, user_data.username)
             if existing_user:
                 app_logger.warning(f"User {user_data.username} already exists.")
-                return None # Or raise an exception
+                return None
             
             db.add(db_user)
-            db.commit() # Commit is handled by context manager, but explicit commit for return
-            db.refresh(db_user)
-        return db_user
+            # db.commit() # Commit is handled by context manager
+            db.refresh(db_user) # This makes sure all attributes are up-to-date from DB *within this session*
+        # Session closes here. db_user is now "detached".
+        return db_user # Returning the detached db_user object
     except Exception as e:
         app_logger.error(f"Error creating user {user_data.username}: {e}")
-        return None
-
-def authenticate_user(username: str, password: str) -> Optional[User]:
-    with get_session_context() as db:
-        user = get_user(db, username)
-        if not user:
-            return None
-        if not verify_password(password, user.hashed_password):
-            return None
-        return user
+        return None