ci: Bump version to 0.12.25 [skip ci]

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "flyer-crawler",
-  "version": "0.12.23",
+  "version": "0.12.25",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "flyer-crawler",
-      "version": "0.12.23",
+      "version": "0.12.25",
       "dependencies": {
         "@bull-board/api": "^6.14.2",
         "@bull-board/express": "^6.14.2",

package.json

@@ -1,7 +1,7 @@
 {
   "name": "flyer-crawler",
   "private": true,
-  "version": "0.12.23",
+  "version": "0.12.25",
   "type": "module",
   "scripts": {
     "dev": "concurrently \"npm:start:dev\" \"vite\"",

src/hooks/queries/useUserProfileDataQuery.test.tsx

@@ -404,8 +404,10 @@ describe('useUserProfileDataQuery', () => {
 
     // Both calls should have been made nearly simultaneously (within 50ms)
     // This verifies Promise.all is being used for parallel execution
-    expect(Math.abs((profileCallTime as number) - (achievementsCallTime as number))).toBeLessThan(
+    expect(
-      50,
+      Math.abs(
-    );
+        (profileCallTime as unknown as number) - (achievementsCallTime as unknown as number),
+      ),
+    ).toBeLessThan(50);
   });
 });

src/services/flyerAiProcessor.server.test.ts

@@ -331,7 +331,9 @@ describe('FlyerAiProcessor', () => {
 
       expect(result.needsReview).toBe(true);
       expect(logger.warn).toHaveBeenCalledWith(
-        expect.objectContaining({ qualityIssues: ['Missing both valid_from and valid_to dates'] }),
+        expect.objectContaining({
+          qualityIssues: ['Missing validity dates (valid_from or valid_to)'],
+        }),
         expect.stringContaining('AI response has quality issues.'),
       );
     });
@@ -358,10 +360,10 @@ describe('FlyerAiProcessor', () => {
           qualityIssues: [
             'Missing store name',
             'No items were extracted',
-            'Missing both valid_from and valid_to dates',
+            'Missing validity dates (valid_from or valid_to)',
           ],
         },
-        'AI response has quality issues. Flagging for review. Issues: Missing store name, No items were extracted, Missing both valid_from and valid_to dates',
+        'AI response has quality issues. Flagging for review. Issues: Missing store name, No items were extracted, Missing validity dates (valid_from or valid_to)',
       );
     });
   });

src/services/flyerAiProcessor.server.ts

@@ -99,8 +99,8 @@ export class FlyerAiProcessor {
     }
 
     // 4. Check for flyer validity dates.
-    if (!valid_from && !valid_to) {
+    if (!valid_from || !valid_to) {
-      qualityIssues.push('Missing both valid_from and valid_to dates');
+      qualityIssues.push('Missing validity dates (valid_from or valid_to)');
     }
 
     const needsReview = qualityIssues.length > 0;

src/tests/e2e/receipt-journey.e2e.test.ts

@@ -296,6 +296,23 @@ describe('E2E Receipt Processing Journey', () => {
     expect(reprocessResponse.status).toBe(200);
     expect(reprocessResponse.body.data.message).toContain('reprocessing');
 
+    // Wait for the reprocess job to complete before deleting
+    await poll(
+      async () => {
+        const statusResponse = await getRequest()
+          .get(`/api/v1/receipts/${receipt2Result.rows[0].receipt_id}`)
+          .set('Authorization', `Bearer ${authToken}`);
+        return statusResponse.status === 200
+          ? statusResponse.body
+          : { data: { receipt: { status: 'pending' } } };
+      },
+      (result) => {
+        const status = result.data?.receipt?.status;
+        return status === 'completed' || status === 'failed';
+      },
+      { timeout: 15000, interval: 1000, description: 'receipt reprocessing' },
+    );
+
     // Step 17: Delete the failed receipt
     const deleteResponse = await getRequest()
       .delete(`/api/v1/receipts/${receipt2Result.rows[0].receipt_id}`)