refactor: use uuid v7 instead of v4

Author: wangeguo

Cargo.lock

@@ -8,7 +8,7 @@ resolver = "2"
 [package]
 name = "backend"
 description = "Backend API and services for StackClass"
-version = "0.39.0"
+version = "0.40.0"
 edition = "2024"
 
 default-run = "stackclass-server"
@@ -66,4 +66,4 @@ tracing-subscriber = { version = "0.3.19", features = ["env-filter"] }
 url = "2.5.4"
 utoipa = { version = "5.4.0", features = ["axum_extras", "uuid", "chrono", "macros"] }
 utoipa-swagger-ui = { version = "9.0.2", features = ["axum", "reqwest"] }
-uuid = { version = "1.18.0", features = ["serde", "v4", "fast-rng", "macro-diagnostics"] }
+uuid = { version = "1.18.0", features = ["v7", "serde"] }

Cargo.toml

@@ -0,0 +1,12 @@
+-- Migration to remove UUID default values since IDs are generated in application code
+-- This allows using UUID v7 from the application layer
+
+-- Remove default values from all tables
+ALTER TABLE courses ALTER COLUMN id DROP DEFAULT;
+ALTER TABLE extensions ALTER COLUMN id DROP DEFAULT;
+ALTER TABLE stages ALTER COLUMN id DROP DEFAULT;
+ALTER TABLE user_courses ALTER COLUMN id DROP DEFAULT;
+ALTER TABLE user_stages ALTER COLUMN id DROP DEFAULT;
+
+-- Drop the uuid-ossp extension since it's no longer needed
+DROP EXTENSION IF EXISTS "uuid-ossp";

migrations/0011_remove_uuid_defaults.sql

@@ -6,7 +6,7 @@
     "license": {
       "name": ""
     },
-    "version": "0.39.0"
+    "version": "0.40.0"
   },
   "paths": {
     "/v1/courses": {

openapi.json

@@ -75,7 +75,7 @@ impl CourseModel {
 impl From<&Course> for CourseModel {
     fn from(course: &Course) -> Self {
         Self {
-            id: Uuid::new_v4(),
+            id: Uuid::now_v7(),
             slug: course.slug.clone(),
             name: course.name.clone(),
             short_name: course.short_name.clone(),
@@ -134,9 +134,9 @@ pub struct UserCourseModel {
 impl Default for UserCourseModel {
     fn default() -> Self {
         Self {
-            id: Uuid::new_v4(),
+            id: Uuid::now_v7(),
             user_id: String::new(),
-            course_id: Uuid::new_v4(),
+            course_id: Uuid::now_v7(),
             course_slug: String::new(),
             started_at: Utc::now(),
             current_stage_id: None,

src/model/course.rs

@@ -72,9 +72,9 @@ impl ExtensionModel {
 impl From<Extension> for ExtensionModel {
     fn from(ext: Extension) -> Self {
         Self {
-            id: Uuid::new_v4(),
+            id: Uuid::now_v7(),
             // Will be replaced by actual course_id
-            course_id: Uuid::new_v4(),
+            course_id: Uuid::now_v7(),
             slug: ext.slug,
             name: ext.name,
             description: ext.description,

src/model/extension.rs

@@ -85,9 +85,9 @@ impl StageModel {
 impl From<Stage> for StageModel {
     fn from(stage: Stage) -> Self {
         Self {
-            id: Uuid::new_v4(),
+            id: Uuid::now_v7(),
             // Will be replaced by actual course_id
-            course_id: Uuid::new_v4(),
+            course_id: Uuid::now_v7(),
             // Will be replaced by actual extension_id
             extension_id: None,
             extension_slug: None,
@@ -139,7 +139,7 @@ impl UserStageModel {
     /// Creates a new instance with default values
     pub fn new(user_course_id: Uuid, stage_id: Uuid) -> Self {
         Self {
-            id: Uuid::new_v4(),
+            id: Uuid::now_v7(),
             user_course_id,
             course_slug: String::new(),
             stage_id,

src/model/stage.rs

@@ -111,7 +111,7 @@ impl PipelineService {
 
     /// Generates a PipelineRun resource for the given repository.
     async fn generate(&self, repo: &str, course: &str, stage: &str) -> Result<DynamicObject> {
-        let name = Uuid::new_v4().to_string();
+        let name = Uuid::now_v7().to_string();
 
         // Define labels for identification
         let labels = vec![