ci: Bump version to 0.2.16 [skip ci]

.gitea/workflows/deploy-to-test.yml

@@ -390,15 +390,8 @@ jobs:
 
         run: |
           # Fail-fast check to ensure secrets are configured in Gitea.
-          MISSING_SECRETS=""
-          if [ -z "$DB_HOST" ]; then MISSING_SECRETS="${MISSING_SECRETS} DB_HOST"; fi
-          if [ -z "$DB_USER" ]; then MISSING_SECRETS="${MISSING_SECRETS} DB_USER"; fi
-          if [ -z "$DB_PASSWORD" ]; then MISSING_SECRETS="${MISSING_SECRETS} DB_PASSWORD"; fi
-          if [ -z "$DB_NAME" ]; then MISSING_SECRETS="${MISSING_SECRETS} DB_NAME"; fi
-          if [ -z "$JWT_SECRET" ]; then MISSING_SECRETS="${MISSING_SECRETS} JWT_SECRET"; fi
-
-          if [ ! -z "$MISSING_SECRETS" ]; then
-            echo "ERROR: The following required secrets are missing in Gitea:${MISSING_SECRETS}"
+          if [ -z "$DB_HOST" ] || [ -z "$DB_USER" ] || [ -z "$DB_PASSWORD" ] || [ -z "$DB_NAME" ] || [ -z "$JWT_SECRET" ]; then
+            echo "ERROR: One or more test secrets (DB_*, JWT_SECRET) are not set in Gitea repository settings."
             exit 1
           fi
 

ecosystem.config.cjs

@@ -11,7 +11,6 @@ if (missingSecrets.length > 0) {
   console.warn('\n[ecosystem.config.cjs] ⚠️  WARNING: The following environment variables are MISSING in the shell:');
   missingSecrets.forEach(key => console.warn(`   - ${key}`));
   console.warn('[ecosystem.config.cjs] The application may crash if these are required for startup.\n');
-  process.exit(1); // Fail fast so PM2 doesn't attempt to start a broken app
 } else {
   console.log('[ecosystem.config.cjs] ✅ Critical environment variables are present.');
 }

package-lock.json

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

src/services/aiApiClient.test.ts

@@ -25,9 +25,8 @@ vi.mock('./logger.client', () => ({
 
 // 2. Mock ./apiClient to simply pass calls through to the global fetch.
 vi.mock('./apiClient', async (importOriginal) => {
-  // This is the core logic we want to preserve: it calls the global fetch
-  // which is then intercepted by MSW.
-  const apiFetch = (
+  return {
+    apiFetch: (
       url: string,
       options: RequestInit = {},
       apiOptions: import('./apiClient').ApiOptions = {},
@@ -61,26 +60,6 @@ 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,12 +284,10 @@ describe('FlyerProcessingService', () => {
 
       await expect(service.processJob(job)).rejects.toThrow(conversionError);
 
-      // Use `toHaveBeenLastCalledWith` to check only the final error payload, ignoring earlier progress updates.
-      expect(job.updateProgress).toHaveBeenLastCalledWith({
+      expect(job.updateProgress).toHaveBeenCalledWith({
         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(
@@ -310,14 +308,10 @@ describe('FlyerProcessingService', () => {
         { err: validationError, validationErrors: {}, rawData: {} },
         'AI Data Validation failed.',
       );
-      // Use `toHaveBeenLastCalledWith` to check only the final error payload.
-      // The payload from AiDataValidationError includes validationErrors and rawData.
-      expect(job.updateProgress).toHaveBeenLastCalledWith({
+      expect(job.updateProgress).toHaveBeenCalledWith({
         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(