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

@@ -1,24 +1,26 @@
-name: CI & Build & Push
+name: CI
 
 on:
   push:
-    branches:
-      - main
+    branches: [main]
   pull_request:
-    branches:
-      - main
+    branches: [main]
+  workflow_dispatch:
 
 env:
   REGISTRY: forgejo.riotpiao.com
-  REGISTRY_USER: rock
+  IMAGE: forgejo.riotpiao.com/rock/poimen-memory
+  DOCKER_HOST: tcp://localhost:2375
 
 jobs:
-  test:
-    name: Test & Lint
+  ci:
+    name: CI
     runs-on: rust
     steps:
-      - name: Install Node.js for actions runtime
-        run: apt-get update && apt-get install -y nodejs
+      - name: Install Node.js and Docker
+        run: |
+          apt-get update
+          apt-get install -y nodejs docker.io
 
       - name: Checkout code
         uses: actions/checkout@v4
@@ -29,49 +31,30 @@ jobs:
       - name: Cargo check
         run: cargo check -p mem-ingest 2>&1 | tail -20 || true
 
-  build-and-push:
-    name: Build & Push Image
-    runs-on: rust
-    needs: test
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    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: Get commit SHA
+      - name: Get short SHA
         id: sha
-        run: |
-          SHORT_SHA=$(git rev-parse --short HEAD)
-          echo "short_sha=$SHORT_SHA" >> $GITHUB_OUTPUT
-          echo "Building image tag: ${{ env.REGISTRY }}/rock/poimen-memory:$SHORT_SHA"
-
-      - name: Install Docker CLI
-        run: apt-get update && apt-get install -y docker.io
-
-      - name: Docker login
-        env:
-          REGISTRY_PAT: ${{ secrets.REGISTRY_PAT }}
-        run: |
-          echo "$REGISTRY_PAT" | docker login -u ${{ env.REGISTRY_USER }} --password-stdin ${{ env.REGISTRY }}
+        run: echo "short_sha=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
 
       - name: Build Docker image
         run: |
-          docker build \
-            --tag ${{ env.REGISTRY }}/rock/poimen-memory:${{ steps.sha.outputs.short_sha }} \
-            --tag ${{ env.REGISTRY }}/rock/poimen-memory:latest \
-            -f Dockerfile \
-            .
-          echo "✅ Docker image built"
+          docker build --no-cache --progress=plain \
+            -t "${IMAGE}:${{ steps.sha.outputs.short_sha }}" \
+            -t "${IMAGE}:latest" \
+            -f Dockerfile .
+
+      - name: Registry login
+        run: |
+          echo "${REGISTRY_TOKEN}" | docker login "${REGISTRY}" \
+            --username "${REGISTRY_USER}" --password-stdin
+        env:
+          REGISTRY_USER: ${{ secrets.FORGEJO_REGISTRY_USER }}
+          REGISTRY_TOKEN: ${{ secrets.FORGEJO_REGISTRY_TOKEN }}
 
       - name: Push Docker image
         run: |
-          docker push ${{ env.REGISTRY }}/rock/poimen-memory:${{ steps.sha.outputs.short_sha }}
-          docker push ${{ env.REGISTRY }}/rock/poimen-memory:latest
-          echo "✅ Image pushed to registry"
+          docker push "${IMAGE}:${{ steps.sha.outputs.short_sha }}"
+          docker push "${IMAGE}:latest"
+          echo "✓ Pushed: ${IMAGE}:${{ steps.sha.outputs.short_sha }}"
 
-      - name: Logout from registry
-        if: always()
-        run: docker logout ${{ env.REGISTRY }} || true
+      - 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
-RUN cargo build --release
+# Build the mem binary (offline mode - no DATABASE_URL needed)
+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 from_val != *to_val {
+                    if let Some(to_val) = to.get(key) {
+                        if from_val != to_val {
                             modified.push(DiffField {
-                                name: key,
-                                from_value: Some(from_val),
+                                name: key.clone(),
+                                from_value: Some(from_val.clone()),
                                 to_value: Some(to_val.clone()),
                             });
                         }
                     } else {
                         removed.push(DiffField {
-                            name: key,
-                            from_value: Some(from_val),
+                            name: key.clone(),
+                            from_value: Some(from_val.clone()),
                             to_value: None,
                         });
                     }
                 } else {
                     removed.push(DiffField {
-                        name: key,
-                        from_value: Some(from_val),
+                        name: key.clone(),
+                        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 from_val != *to_val {
+                if let Some(to_val) = to.get(key) {
+                    if from_val != to_val {
                         modified.push(DiffField {
-                            name: key,
-                            from_value: Some(from_val),
+                            name: key.clone(),
+                            from_value: Some(from_val.clone()),
                             to_value: Some(to_val.clone()),
                         });
                     }
                 } else {
                     removed.push(DiffField {
-                        name: key,
-                        from_value: Some(from_val),
+                        name: key.clone(),
+                        from_value: Some(from_val.clone()),
                         to_value: None,
                     });
                 }
             } else {
                 removed.push(DiffField {
-                    name: key,
-                    from_value: Some(from_val),
+                    name: key.clone(),
+                    from_value: Some(from_val.clone()),
                     to_value: None,
                 });
             }