refactor: simplify user update logic in Clerk webhook handler to reuse findOrCreateUser

app/routes/api/webhooks/clerk.ts

@@ -1,6 +1,6 @@
 import { Webhook } from "svix";
 import type { Route } from "./+types/clerk";
-import { findOrCreateUser, updateUserByClerkId } from "~/models/user";
+import { findOrCreateUser } from "~/models/user";
 
 export async function action({ request }: Route.ActionArgs) {
   // Get the webhook secret from environment
@@ -65,29 +65,16 @@ export async function action({ request }: Route.ActionArgs) {
           `User created in database: ${user.id} (${user.username || user.displayName})`
         );
       } else {
-        // Update existing user
+        // Update existing user (or create if doesn't exist)
-        const email = email_addresses?.[0]?.email_address;
+        const user = await findOrCreateUser({
-        if (!email) {
+          id,
-          throw new Error("User must have an email address");
+          emailAddresses:
-        }
+            email_addresses?.map((e: any) => ({
-
+              emailAddress: e.email_address,
-        // Generate display name
+            })) || [],
-        let displayName = "";
-        if (first_name || last_name) {
-          displayName = [first_name, last_name]
-            .filter(Boolean)
-            .join(" ")
-            .trim();
-        } else {
-          displayName = email.split("@")[0];
-        }
-
-        const user = await updateUserByClerkId(id, {
-          email,
           username,
-          displayName,
+          firstName: first_name,
-          firstName: first_name || undefined,
+          lastName: last_name,
-          lastName: last_name || undefined,
           imageUrl: image_url,
         });
         console.log(