Update admin_routes.py

admin_routes.py

@@ -73,7 +73,7 @@ class ProjectUpdate(BaseModel):
     
 class VersionCreate(BaseModel):
     caption: str
-    source_version_id: int | None = None
+    source_version_no: int | None = None
 
 class IntentModel(BaseModel):
     name: str
@@ -531,10 +531,10 @@ async def create_version(
     log_info(f"✅ Version created for project {project_id} by {username}")
     return new_version.model_dump()
 
-@router.put("/projects/{project_id}/versions/{version_id}")
+@router.put("/projects/{project_id}/versions/{version_no}")
 async def update_version(
     project_id: int,
-    version_id: int,
+    version_no: int,
     update: VersionUpdate,
     force: bool = Query(default=False, description="Force update despite conflicts"),
     username: str = Depends(verify_token)
@@ -543,17 +543,17 @@ async def update_version(
     try:
         # Force parametresi kontrolü
         if force:
-            log_warning(f"⚠️ Force update requested for version {version_id} by {username}")
+            log_warning(f"⚠️ Force update requested for version {version_no} by {username}")
         
         result = ConfigProvider.update_version(
             project_id,
-            version_id,
+            version_no,
             update.model_dump(),
             username,
             expected_last_update=update.last_update_date if not force else None
         )
         
-        log_info(f"✅ Version {version_id} updated by {username}")
+        log_info(f"✅ Version {version_no} updated by {username}")
         return result
         
     except RaceConditionError as e:
@@ -561,14 +561,14 @@ async def update_version(
             # Force modunda race condition'ı yoksay
             result = ConfigProvider.update_version(
                 project_id,
-                version_id,
+                version_no,
                 update.model_dump(),
                 username,
                 expected_last_update=None
             )
             return result
         else:
-            log_warning(f"⚠️ Race condition detected for version {version_id}")
+            log_warning(f"⚠️ Race condition detected for version {version_no}")
             raise HTTPException(
                 status_code=409,
                 detail={
@@ -581,19 +581,19 @@ async def update_version(
     except ResourceNotFoundError:
         raise HTTPException(status_code=404, detail="Version not found")
     except Exception as e:
-        log_error(f"❌ Error updating version {version_id}", e)
+        log_error(f"❌ Error updating version {version_no}", e)
         raise HTTPException(status_code=500, detail=str(e))
         
-@router.post("/projects/{project_id}/versions/{version_id}/publish")
+@router.post("/projects/{project_id}/versions/{version_no}/publish")
 async def publish_version(
     project_id: int,
-    version_id: int,
+    version_no: int,
     username: str = Depends(verify_token)
 ):
     """Publish version"""
-    project, version = ConfigProvider.publish_version(project_id, version_id, username)
+    project, version = ConfigProvider.publish_version(project_id, version_no, username)
     
-    log_info(f"✅ Version {version_id} published for project '{project.name}' by {username}")
+    log_info(f"✅ Version {version_no} published for project '{project.name}' by {username}")
 
     # Notify LLM provider if project is enabled and provider requires repo info
     cfg = ConfigProvider.get()
@@ -608,16 +608,16 @@ async def publish_version(
 
     return {"success": True}
 
-@router.delete("/projects/{project_id}/versions/{version_id}")
+@router.delete("/projects/{project_id}/versions/{version_no}")
 async def delete_version(
     project_id: int,
-    version_id: int,
+    version_no: int,
     username: str = Depends(verify_token)
 ):
     """Delete version (soft delete)"""
-    ConfigProvider.delete_version(project_id, version_id, username)
+    ConfigProvider.delete_version(project_id, version_no, username)
     
-    log_info(f"✅ Version {version_id} deleted for project {project_id} by {username}")
+    log_info(f"✅ Version {version_no} deleted for project {project_id} by {username}")
     return {"success": True}
 
 @router.get("/projects/{project_name}/versions")
@@ -638,7 +638,7 @@ async def get_project_versions(
     for v in project.versions:
         if not getattr(v, 'deleted', False):
             versions.append({
-                "version_number": v.id,
+                "version_number": v.no,
                 "caption": v.caption,
                 "published": v.published,
                 "description": getattr(v, 'description', ''),
@@ -656,8 +656,8 @@ async def get_project_versions(
 @router.get("/projects/{project_id}/versions/{version1_id}/compare/{version2_id}")
 async def compare_versions(
     project_id: int,
-    version1_id: int,
-    version2_id: int,
+    version1_no: int,
+    version2_no: int,
     username: str = Depends(verify_token)
 ):
     """Compare two versions and return differences"""
@@ -665,8 +665,8 @@ async def compare_versions(
     if not project:
         raise HTTPException(status_code=404, detail="Project not found")
     
-    v1 = next((v for v in project.versions if v.id == version1_id), None)
-    v2 = next((v for v in project.versions if v.id == version2_id), None)
+    v1 = next((v for v in project.versions if v.no == version1_no), None)
+    v2 = next((v for v in project.versions if v.no == version2_no), None)
     
     if not v1 or not v2:
         raise HTTPException(status_code=404, detail="Version not found")
@@ -958,7 +958,7 @@ async def notify_llm_startup(project, version):
         # Build project config for startup
         project_config = {
             "name": project.name,
-            "version_id": version.id,
+            "version_no": version.no,
             "repo_id": version.llm.repo_id,
             "generation_config": version.llm.generation_config,
             "use_fine_tune": version.llm.use_fine_tune,