chore: improve query readability and deprecate unused fields

Author: mbani01

services/apps/git_integration/src/crowdgit/database/crud.py

@@ -23,8 +23,6 @@
     r."segmentId",
     r."gitIntegrationId",
     r."forkedFrom",
-    r."createdAt",
-    r."updatedAt",
     rp.state,
     rp.priority,
     rp."lockedAt",
@@ -74,21 +72,22 @@ async def acquire_onboarding_repo() -> Repository | None:
     selected_repo AS (
         SELECT r.id
         FROM public.repositories r
-        JOIN git."repositoryProcessing" rp ON rp."repositoryId" = r.id,
-             current_onboarding_count c
+        JOIN git."repositoryProcessing" rp ON rp."repositoryId" = r.id
+        CROSS JOIN current_onboarding_count c
         WHERE rp.state = $2
             AND rp."lockedAt" IS NULL
             AND r."deletedAt" IS NULL
             AND c.count < $3
-        ORDER BY rp.priority ASC, r."createdAt" ASC
+        ORDER BY rp.priority ASC, rp."createdAt" ASC
         LIMIT 1
         FOR UPDATE OF rp SKIP LOCKED
     )
     UPDATE git."repositoryProcessing" rp
     SET "lockedAt" = NOW(),
         state = $1,
         "updatedAt" = NOW()
-    FROM public.repositories r, selected_repo
+    FROM public.repositories r
+    CROSS JOIN selected_repo
     WHERE rp."repositoryId" = r.id
         AND rp."repositoryId" = selected_repo.id
     RETURNING {REPO_SELECT_COLUMNS}
@@ -142,7 +141,8 @@ async def acquire_recurrent_repo() -> Repository | None:
     SET "lockedAt" = NOW(),
         state = $1,
         "updatedAt" = NOW()
-    FROM public.repositories r, selected_repo
+    FROM public.repositories r
+    CROSS JOIN selected_repo
     WHERE rp."repositoryId" = r.id
         AND rp."repositoryId" = selected_repo.id
     RETURNING {REPO_SELECT_COLUMNS}

services/apps/git_integration/src/crowdgit/models/repository.py

@@ -46,11 +46,9 @@ class Repository(BaseModel):
         description="Indicates if the stuck repository is resolved by a re-onboarding",
     )
     re_onboarding_count: int = Field(
-        ...,
+        default=0,
         description="Tracks the number of times this repository has been re-onboarded. Used to identify unreachable commits via activity.attributes.cycle matching pattern onboarding-{reOnboardingCount}",
     )
-    created_at: datetime = Field(..., description="Creation timestamp")
-    updated_at: datetime = Field(..., description="Last update timestamp")
 
     @classmethod
     def from_db(cls, db_data: dict[str, Any]) -> Repository:
@@ -65,8 +63,6 @@ def from_db(cls, db_data: dict[str, Any]) -> Repository:
 
         # Map database field names to model field names
         field_mapping = {
-            "createdAt": "created_at",
-            "updatedAt": "updated_at",
             "lastProcessedAt": "last_processed_at",
             "lastProcessedCommit": "last_processed_commit",
             "lockedAt": "locked_at",