ci: Bump version to 0.2.18 [skip ci]

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "flyer-crawler",
-  "version": "0.2.15",
+  "version": "0.2.18",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "flyer-crawler",
-      "version": "0.2.15",
+      "version": "0.2.18",
       "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.2.15",
+  "version": "0.2.18",
   "type": "module",
   "scripts": {
     "dev": "concurrently \"npm:start:dev\" \"vite\"",

src/services/aiApiClient.test.ts

@@ -25,8 +25,9 @@ vi.mock('./logger.client', () => ({
 
 // 2. Mock ./apiClient to simply pass calls through to the global fetch.
 vi.mock('./apiClient', async (importOriginal) => {
-  return {
-    apiFetch: (
+  // This is the core logic we want to preserve: it calls the global fetch
+  // which is then intercepted by MSW.
+  const apiFetch = (
       url: string,
       options: RequestInit = {},
       apiOptions: import('./apiClient').ApiOptions = {},
@@ -60,6 +61,26 @@ vi.mock('./apiClient', async (importOriginal) => {
       const request = new Request(fullUrl, options);
       console.log(`[apiFetch MOCK] Executing fetch for URL: ${request.url}.`);
       return fetch(request);
+  };
+
+  return {
+    // The original mock only had apiFetch. We need to add the helpers.
+    apiFetch,
+
+    // These helpers are what aiApiClient.ts actually calls.
+    // Their mock implementation should just call our mocked apiFetch.
+    authedGet: (endpoint: string, options: import('./apiClient').ApiOptions = {}) => {
+      return apiFetch(endpoint, { method: 'GET' }, options);
+    },
+    authedPost: <T>(endpoint: string, body: T, options: import('./apiClient').ApiOptions = {}) => {
+      return apiFetch(
+        endpoint,
+        { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify(body) },
+        options,
+      );
+    },
+    authedPostForm: (endpoint: string, formData: FormData, options: import('./apiClient').ApiOptions = {}) => {
+      return apiFetch(endpoint, { method: 'POST', body: formData }, options);
     },
     // Add a mock for ApiOptions to satisfy the compiler
     ApiOptions: vi.fn(),

src/services/flyerProcessingService.server.test.ts

@@ -284,10 +284,12 @@ describe('FlyerProcessingService', () => {
 
       await expect(service.processJob(job)).rejects.toThrow(conversionError);
 
-      expect(job.updateProgress).toHaveBeenCalledWith({
+      // Use `toHaveBeenLastCalledWith` to check only the final error payload, ignoring earlier progress updates.
+      expect(job.updateProgress).toHaveBeenLastCalledWith({
         errorCode: 'PDF_CONVERSION_FAILED',
         message:
           'The uploaded PDF could not be processed. It might be blank, corrupt, or password-protected.', // This was a duplicate, fixed.
+        stderr: 'pdftocairo error',
       });
       expect(mockCleanupQueue.add).not.toHaveBeenCalled();
       expect(logger.warn).toHaveBeenCalledWith(
@@ -308,10 +310,14 @@ describe('FlyerProcessingService', () => {
         { err: validationError, validationErrors: {}, rawData: {} },
         'AI Data Validation failed.',
       );
-      expect(job.updateProgress).toHaveBeenCalledWith({
+      // Use `toHaveBeenLastCalledWith` to check only the final error payload.
+      // The payload from AiDataValidationError includes validationErrors and rawData.
+      expect(job.updateProgress).toHaveBeenLastCalledWith({
         errorCode: 'AI_VALIDATION_FAILED',
         message:
           "The AI couldn't read the flyer's format. Please try a clearer image or a different flyer.", // This was a duplicate, fixed.
+        validationErrors: {},
+        rawData: {},
       });
       expect(mockCleanupQueue.add).not.toHaveBeenCalled();
       expect(logger.warn).toHaveBeenCalledWith(