fix: Dockerfile + borrow checker errors + SQLX_OFFLINE

- rust:1.81 → rust:1-bookworm
- cargo build --release -p mem-cli
- Fix partial move errors in versioning.rs (ref borrow)
- Set SQLX_OFFLINE=true (needs .sqlx/ cache or query macro conversion)
- Build + push on all CI events

NOTE: Still needs sqlx offline cache (.sqlx/) or convert query_as! to runtime

.gitea/workflows/build.yaml

@@ -2,37 +2,19 @@ name: CI
 
 on:
   push:
-    branches:
+    branches: [main]
-      - main
   pull_request:
-    branches:
+    branches: [main]
-      - main
+  workflow_dispatch:
 
 env:
   REGISTRY: forgejo.riotpiao.com
   IMAGE: forgejo.riotpiao.com/rock/poimen-memory
+  DOCKER_HOST: tcp://localhost:2375
 
 jobs:
-  test:
+  ci:
-    name: Test
+    name: CI
-    runs-on: rust
-    steps:
-      - name: Install Node.js for actions runtime
-        run: apt-get update && apt-get install -y nodejs
-
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      - name: Cargo test
-        run: cargo test -p mem-ingest --lib 2>&1 | tail -50 || true
-
-      - name: Cargo check
-        run: cargo check -p mem-ingest 2>&1 | tail -20 || true
-
-  build-push:
-    name: Build & Push Image
-    needs: test
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
     runs-on: rust
     steps:
       - name: Install Node.js and Docker
@@ -43,11 +25,22 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Cargo test
+        run: cargo test -p mem-ingest --lib 2>&1 | tail -50 || true
+
+      - name: Cargo check
+        run: cargo check -p mem-ingest 2>&1 | tail -20 || true
+
       - name: Get short SHA
         id: sha
+        run: echo "short_sha=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
+
+      - name: Build Docker image
         run: |
-          SHORT_SHA=$(git rev-parse --short HEAD)
+          docker build --no-cache --progress=plain \
-          echo "short_sha=${SHORT_SHA}" >> $GITHUB_OUTPUT
+            -t "${IMAGE}:${{ steps.sha.outputs.short_sha }}" \
+            -t "${IMAGE}:latest" \
+            -f Dockerfile .
 
       - name: Registry login
         run: |
@@ -57,19 +50,11 @@ jobs:
           REGISTRY_USER: ${{ secrets.FORGEJO_REGISTRY_USER }}
           REGISTRY_TOKEN: ${{ secrets.FORGEJO_REGISTRY_TOKEN }}
 
-      - name: Build Docker image
-        run: |
-          docker build --no-cache \
-            -t "${IMAGE}:${{ steps.sha.outputs.short_sha }}" \
-            -t "${IMAGE}:latest" \
-            -f Dockerfile \
-            .
-
       - name: Push Docker image
         run: |
           docker push "${IMAGE}:${{ steps.sha.outputs.short_sha }}"
           docker push "${IMAGE}:latest"
-          echo "✓ Image pushed: ${IMAGE}:${{ steps.sha.outputs.short_sha }}"
+          echo "✓ Pushed: ${IMAGE}:${{ steps.sha.outputs.short_sha }}"
 
       - name: Prune unused images
         run: docker image prune -a --force 2>&1 | tail -3 || true

Dockerfile

@@ -1,15 +1,16 @@
 # Multi-stage build for Poimen Memory Service (Rust)
 
 # Stage 1: Builder
-FROM rust:1.81-bookworm as builder
+FROM rust:1-bookworm as builder
 
 WORKDIR /build
 
 # Copy source
 COPY . .
 
-# Build in release mode
+# Build the mem binary (offline mode - no DATABASE_URL needed)
-RUN cargo build --release
+ENV SQLX_OFFLINE=true
+RUN cargo build --release -p mem-cli
 
 # Stage 2: Runtime
 FROM debian:bookworm-slim

README.md

@@ -99,3 +99,4 @@ See `config/default.toml` for:
 6. Document in API.md
 
 See `CLAUDE.md` for project context and constraints.
+# CI test 1788759975

crates/mem-store/src/versioning.rs

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