Merge pull request 'Background fixes & improvements' (#37) from mobile_app_hotfixes into main

Reviewed-on: #37

.gitea/workflows/docker.yaml

@@ -31,6 +31,7 @@ jobs:
     needs: detect-changes
     if: needs.detect-changes.outputs.backend == 'true'
     runs-on: ubuntu-latest
+    timeout-minutes: 15
     steps:
     - uses: actions/checkout@v4
     - name: Setup Node.js
@@ -39,15 +40,18 @@ jobs:
         node-version: '22'
         cache: 'npm'
     - name: Install dependencies
-      run: npm ci
+      run: npm ci --ignore-scripts
+    - name: Install backend native deps
+      run: npm rebuild sqlite3 bcrypt --workspace=farma-clic-backend
     - name: Run tests
-      run: npm test --workspace=farma-clic-backend -- --ci
+      run: npm test --workspace=farma-clic-backend
 
   test-frontend:
     name: Frontend Tests
     needs: detect-changes
     if: needs.detect-changes.outputs.frontend == 'true'
     runs-on: ubuntu-latest
+    timeout-minutes: 10
     steps:
     - uses: actions/checkout@v4
     - name: Setup Node.js
@@ -56,7 +60,7 @@ jobs:
         node-version: '22'
         cache: 'npm'
     - name: Install dependencies
-      run: npm ci
+      run: npm ci --ignore-scripts
     - name: Run tests
       run: npm test --workspace=farma-clic-frontend -- --run --reporter=basic
 

.gitea/workflows/test-branches.yaml

@@ -6,42 +6,82 @@ on:
     - 'main'
 
 jobs:
-  test:
+  detect-changes:
-    name: Run Tests
+    name: Detect Changes
     runs-on: ubuntu-latest
+    outputs:
+      backend: ${{ steps.filter.outputs.backend }}
+      frontend: ${{ steps.filter.outputs.frontend }}
+      frontend-mobile: ${{ steps.filter.outputs.frontend-mobile }}
     steps:
-    - name: Checkout Code
+    - uses: actions/checkout@v4
-      uses: actions/checkout@v4
+    - uses: dorny/paths-filter@v3
+      id: filter
       with:
-        fetch-depth: 2
+        filters: |
-
+          backend:
-    - name: Detect Changes
+            - 'apps/backend/**'
-      id: changes
+            - 'packages/**'
-      run: |
+          frontend:
-        BASE=${{ github.event.before }}
+            - 'apps/frontend/**'
-        [ -z "$BASE" ] || [ "$BASE" = "0000000000000000000000000000000000000000" ] && BASE=$(git rev-parse HEAD~1)
+            - 'packages/**'
-        echo "backend=$(git diff --name-only "$BASE"...HEAD | grep -qE '^(apps/backend/|packages/)' && echo true || echo false)" >> "$GITHUB_OUTPUT"
+          frontend-mobile:
-        echo "frontend=$(git diff --name-only "$BASE"...HEAD | grep -qE '^(apps/frontend/|packages/)' && echo true || echo false)" >> "$GITHUB_OUTPUT"
+            - 'apps/frontend-mobile/**'
-        echo "frontend-mobile=$(git diff --name-only "$BASE"...HEAD | grep -qE '^(apps/frontend-mobile/|packages/)' && echo true || echo false)" >> "$GITHUB_OUTPUT"
+            - 'packages/**'
 
+  test-backend:
+    name: Backend Tests
+    needs: detect-changes
+    if: needs.detect-changes.outputs.backend == 'true'
+    runs-on: ubuntu-latest
+    timeout-minutes: 15
+    steps:
+    - uses: actions/checkout@v4
     - name: Setup Node.js
       uses: actions/setup-node@v4
       with:
         node-version: '22'
         cache: 'npm'
-
+    - name: Install dependencies
-    - name: Install Dependencies
+      run: npm ci --ignore-scripts
-      run: npm ci
+    - name: Install backend native deps
-
+      run: npm rebuild sqlite3 bcrypt --workspace=farma-clic-backend
     - name: Run Backend Tests
-      if: steps.changes.outputs.backend == 'true'
+      run: npm test --workspace=farma-clic-backend
-      run: npm test --workspace=farma-clic-backend -- --ci
 
+  test-frontend:
+    name: Frontend Tests
+    needs: detect-changes
+    if: needs.detect-changes.outputs.frontend == 'true'
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '22'
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci --ignore-scripts
     - name: Run Frontend Tests
-      if: steps.changes.outputs.frontend == 'true'
       run: npm test --workspace=farma-clic-frontend -- --run --reporter=basic
 
+  test-frontend-mobile:
+    name: Frontend Mobile Tests
+    needs: detect-changes
+    if: needs.detect-changes.outputs.frontend-mobile == 'true'
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '22'
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci --ignore-scripts
     - name: Run Frontend Mobile Tests
-      if: steps.changes.outputs.frontend-mobile == 'true'
       run: npm test --workspace=frontend-mobile -- --ci
       continue-on-error: true

.gitignore

@@ -3,6 +3,7 @@ dist/
 dev-dist/
 build/
 .turbo/
+.mimocode/
 *.sqlite
 *.sqlite3
 .env

.mimocode/.cron-lock

@@ -1 +1 @@
-{"pid":10782,"startedAt":1783578681151}
+{"pid":568607,"startedAt":1784046408782}

apps/backend/jest.config.js

@@ -3,4 +3,8 @@ export default {
   transform: {},
   moduleFileExtensions: ['js', 'json'],
   testMatch: ['**/__tests__/**/*.test.js'],
+  testTimeout: 15000,
+  openHandlesTimeout: 5000,
+  // Run test files in parallel (not individual tests within a file)
+  // workerIdleMemoryLimit: '512MB',
 }

apps/backend/package.json

@@ -12,7 +12,7 @@
     "migrate": "node migrate.js",
     "reset-db": "bash reset-db.sh",
     "import-farmacias": "node import-farmacias.js",
-    "test": "NODE_OPTIONS='--experimental-vm-modules' npx jest --forceExit"
+    "test": "NODE_OPTIONS='--experimental-vm-modules' npx jest --ci --forceExit --forceExitTimeout=30000"
   },
   "keywords": [],
   "author": "",

apps/backend/redis-client.js

@@ -5,7 +5,11 @@ import * as appMetrics from './src/metrics.js';
 const redisClient = createClient({
   socket: {
     host: process.env.REDIS_HOST || 'localhost',
-    port: process.env.REDIS_PORT || 6379
+    port: process.env.REDIS_PORT || 6379,
+    reconnectStrategy: (retries) => {
+      if (retries > 10) return new Error('Redis max retries reached');
+      return Math.min(retries * 100, 3000);
+    }
   },
   password: process.env.REDIS_PASSWORD || undefined
 });
@@ -42,7 +46,9 @@ redisClient.setEx = async (...args) => {
   }
 };
 
-// Connect to Redis
+// Connect to Redis — skip in test (services are mocked) or when REDIS_URL is unset
-await redisClient.connect();
+if (process.env.NODE_ENV !== 'test') {
+  await redisClient.connect();
+}
 
 export default redisClient;

apps/backend/server.js

@@ -1,5 +1,8 @@
 // OpenTelemetry SDK — must be imported first to instrument http, express, pg, etc.
-import './src/tracing.js';
+// Skipped in test to avoid gRPC connection attempts to a non-existent collector.
+if (process.env.NODE_ENV !== 'test') {
+  await import('./src/tracing.js');
+}
 
 import express from 'express';
 import redisClient from './redis-client.js';
@@ -19,8 +22,6 @@ import webpush from 'web-push';
 import pino from 'pino';
 import pinoHttp from 'pino-http';
 import multer from 'multer';
-import { createWorker } from 'tesseract.js';
-import { BarcodeDetector } from 'barcode-detector';
 import { searchMedicines, getMedicineDetails, searchOTC } from './cima-service.js';
 import { searchBabyProducts, getBabyProductDetails } from './off-service.js';
 import { runFarmaciaWebhookImport, DEFAULT_FARMACIAS_WEBHOOK, importPharmaciesFromRows } from './farmacias-webhook-import.js';
@@ -914,6 +915,7 @@ app.post('/api/tsi/ocr', upload.single('photo'), async (req, res) => {
 
     console.log('[OCR] Processing image, size:', req.file.size, 'bytes');
 
+    const { createWorker } = await import('tesseract.js');
     const worker = await createWorker('spa+eng');
     const { data } = await worker.recognize(req.file.buffer);
     await worker.terminate();

apps/frontend-mobile/app/(tabs)/_layout.tsx

@@ -55,6 +55,8 @@ export default function TabLayout() {
           fontWeight: '600',
           fontSize: isTablet ? 20 : 17,
         },
+        sceneContainerStyle: { backgroundColor: 'transparent' },
+        sceneStyle: { backgroundColor: 'transparent' },
       }}
     >
       <Tabs.Screen

apps/frontend-mobile/app/_layout.tsx

@@ -50,6 +50,7 @@ function RootLayoutInner() {
           headerStyle: { backgroundColor: colors.card },
           headerTintColor: colors.primary,
           headerTitleStyle: { color: colors.text, fontWeight: '600' },
+          contentStyle: { backgroundColor: 'transparent' },
         }}
       >
         <Stack.Screen name="(tabs)" options={{ headerShown: false }} />
@@ -99,7 +100,7 @@ function ThemedBackground({ children }: { children: React.ReactNode }) {
         resizeMode="cover"
       />
       <View style={[styles.overlay, isDark && styles.overlayDark]} pointerEvents="none" />
-      <View style={styles.content}>
+      <View style={styles.content} pointerEvents="auto">
         {children}
       </View>
     </View>
@@ -127,17 +128,30 @@ const styles = StyleSheet.create({
     flex: 1,
   },
   bgImage: {
-    ...StyleSheet.absoluteFillObject,
+    position: 'absolute',
+    left: 0,
+    right: 0,
+    top: 0,
+    bottom: 0,
+    width: '100%',
+    height: '100%',
     opacity: 0.55,
+    zIndex: 0,
   },
   overlay: {
-    ...StyleSheet.absoluteFillObject,
+    position: 'absolute',
+    left: 0,
+    right: 0,
+    top: 0,
+    bottom: 0,
     backgroundColor: 'rgba(255, 252, 245, 0.48)',
+    zIndex: 1,
   },
   overlayDark: {
     backgroundColor: 'rgba(0, 0, 0, 0.25)',
   },
   content: {
     flex: 1,
+    zIndex: 2,
   },
 });

apps/frontend-mobile/metro.config.js

@@ -0,0 +1,21 @@
+const { getDefaultConfig } = require("expo/metro-config");
+const path = require("path");
+
+const projectRoot = __dirname;
+const monorepoRoot = path.resolve(projectRoot, "../..");
+
+const config = getDefaultConfig(projectRoot);
+
+// Watch the monorepo root so Metro sees all workspace packages
+config.watchFolders = [monorepoRoot];
+
+// Resolve modules from both the workspace and root node_modules
+config.resolver.nodeModulesPaths = [
+  path.resolve(projectRoot, "node_modules"),
+  path.resolve(monorepoRoot, "node_modules"),
+];
+
+// Ensure symlinks are followed (npm workspaces use symlinks)
+config.resolver.unstable_enableSymlinks = true;
+
+module.exports = config;

apps/frontend-mobile/package.json

@@ -4,6 +4,7 @@
   "main": "expo-router/entry",
   "dependencies": {
     "@expo/vector-icons": "^15.0.2",
+    "@grafana/faro-react-native": "^1.3.0",
     "@react-native-async-storage/async-storage": "2.2.0",
     "@tanstack/react-query": "^5.101.2",
     "axios": "^1.18.1",
@@ -13,7 +14,6 @@
     "expo-constants": "~57.0.3",
     "expo-dev-client": "~57.0.5",
     "expo-device": "~7.0.2",
-    "@grafana/faro-react-native": "^1.2.1",
     "expo-image-picker": "~57.0.2",
     "expo-linking": "~57.0.1",
     "expo-local-authentication": "~57.0.0",

package-lock.json

@@ -138,7 +138,7 @@
       "version": "1.0.0",
       "dependencies": {
         "@expo/vector-icons": "^15.0.2",
-        "@grafana/faro-react-native": "^1.2.1",
+        "@grafana/faro-react-native": "^1.3.0",
         "@react-native-async-storage/async-storage": "2.2.0",
         "@tanstack/react-query": "^5.101.2",
         "axios": "^1.18.1",
@@ -416,6 +416,7 @@
       "version": "1.3.0",
       "resolved": "https://registry.npmjs.org/@grafana/faro-react-native/-/faro-react-native-1.3.0.tgz",
       "integrity": "sha512-KeohD3S2xmiukL5YW3mOGDqvFiwpD9IzJ2EtZ50ktEsnA+6EMt2AtJ8RzJfAamGTYqemBpDiw51rqB4ID+C2xg==",
+      "license": "Apache-2.0",
       "dependencies": {
         "@grafana/faro-core": "^2.7.0",
         "@react-native-async-storage/async-storage": "^1.21.0",