refactor: profile upsert

[RaoHai]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
petercat	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 4, 2025 3:16am

[petercat-assistant]

Walkthrough

This pull request refactors the profile upsert functionality by replacing direct database interaction with a DAO (Data Access Object) pattern. The changes involve using the ProfilesDAO class to handle user profile upserts instead of directly using the supabase client.

Changes

File	Summary
server/auth/clients/base.py, server/auth/clients/local.py	Replaced direct supabase client usage with ProfilesDAO for upserting user profiles.
server/core/dao/profilesDAO.py	Added upsert_user method to handle user profile upserts through the DAO pattern.

[petercat-assistant]

Ensure that the ProfilesDAO instance is properly initialized and that the upsert_user method handles all necessary exceptions that might occur during the database operation. This is crucial to prevent any runtime errors that could disrupt the user login process.

[codecov]

Codecov Report

Attention: Patch coverage is 42.85714% with 8 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
server/auth/clients/local.py	57.14%	3 Missing ⚠️
server/core/dao/profilesDAO.py	25.00%	3 Missing ⚠️
server/auth/clients/base.py	33.33%	2 Missing ⚠️

Files with missing lines	Coverage Δ
server/auth/clients/base.py	52.77% <33.33%> (ø)
server/auth/clients/local.py	53.33% <57.14%> (ø)
server/core/dao/profilesDAO.py	37.50% <25.00%> (ø)

[xingwanying]

LG