fix: Dockerfile build mem-cli binary + CI builds and pushes on all events

- rust:1.81 → rust:1-bookworm (track latest stable)
- cargo build --release → cargo build --release -p mem-cli
- Build + push on all events (verify before merge)
- --progress=plain for visible build logs

Dockerfile

@@ -8,8 +8,7 @@ WORKDIR /build
 # Copy source
 COPY . .
 
-# Build the mem binary (offline mode - no DATABASE_URL needed)
-ENV SQLX_OFFLINE=true
+# Build the mem binary
 RUN cargo build --release -p mem-cli
 
 # Stage 2: Runtime

crates/mem-store/src/versioning.rs

@@ -111,28 +111,28 @@ impl EntityVersioningService {
         let mut modified = Vec::new();
         
         // Check removed and modified
-        if let Some(ref from) = from_obj {
+        if let Some(from) = from_obj {
             for (key, from_val) in from {
                 if let Some(to) = &to_obj {
-                    if let Some(to_val) = to.get(key) {
-                        if from_val != to_val {
+                    if let Some(to_val) = to.get(&key) {
+                        if from_val != *to_val {
                             modified.push(DiffField {
-                                name: key.clone(),
-                                from_value: Some(from_val.clone()),
+                                name: key,
+                                from_value: Some(from_val),
                                 to_value: Some(to_val.clone()),
                             });
                         }
                     } else {
                         removed.push(DiffField {
-                            name: key.clone(),
-                            from_value: Some(from_val.clone()),
+                            name: key,
+                            from_value: Some(from_val),
                             to_value: None,
                         });
                     }
                 } else {
                     removed.push(DiffField {
-                        name: key.clone(),
-                        from_value: Some(from_val.clone()),
+                        name: key,
+                        from_value: Some(from_val),
                         to_value: None,
                     });
                 }
@@ -301,28 +301,28 @@ fn compute_diff(
     let mut removed = Vec::new();
     let mut modified = Vec::new();
     
-    if let Some(ref from) = from_obj {
+    if let Some(from) = from_obj {
         for (key, from_val) in from {
             if let Some(to) = &to_obj {
-                if let Some(to_val) = to.get(key) {
-                    if from_val != to_val {
+                if let Some(to_val) = to.get(&key) {
+                    if from_val != *to_val {
                         modified.push(DiffField {
-                            name: key.clone(),
-                            from_value: Some(from_val.clone()),
+                            name: key,
+                            from_value: Some(from_val),
                             to_value: Some(to_val.clone()),
                         });
                     }
                 } else {
                     removed.push(DiffField {
-                        name: key.clone(),
-                        from_value: Some(from_val.clone()),
+                        name: key,
+                        from_value: Some(from_val),
                         to_value: None,
                     });
                 }
             } else {
                 removed.push(DiffField {
-                    name: key.clone(),
-                    from_value: Some(from_val.clone()),
+                    name: key,
+                    from_value: Some(from_val),
                     to_value: None,
                 });
             }