fix

Author: pashidlos

src/compare/compare.module.ts

@@ -11,7 +11,15 @@ import { StaticModule } from '../static/static.module';
 
 @Module({
   controllers: [OllamaController],
-  providers: [CompareService, PixelmatchService, LookSameService, OdiffService, VlmService, OllamaService, GeminiService],
+  providers: [
+    CompareService,
+    PixelmatchService,
+    LookSameService,
+    OdiffService,
+    VlmService,
+    OllamaService,
+    GeminiService,
+  ],
   imports: [StaticModule],
   exports: [CompareService],
 })

src/compare/libs/vlm/providers/gemini/gemini.service.spec.ts

@@ -82,7 +82,7 @@ describe('GeminiService', () => {
 
       const callArgs = mockGenerateContent.mock.calls[0][0];
       expect(callArgs.contents.length).toBe(expectedPartsCount);
-      
+
       // Verify first image is base64 encoded
       if (images.length > 0) {
         const imagePart = callArgs.contents[1];
@@ -116,13 +116,12 @@ describe('GeminiService', () => {
       ['SDK call fails', { apiKey: 'test-api-key' }, 'API Error'],
     ])('should throw error when %s', async (_, overrides, expectedError) => {
       const config = createConfig(overrides);
-      
+
       if (expectedError === 'API Error') {
         mockGenerateContent.mockRejectedValue(new Error(expectedError));
       }
-      
+
       await expect(service.generate(config, [])).rejects.toThrow(expectedError);
     });
   });
 });
-

src/compare/libs/vlm/providers/gemini/gemini.service.ts

@@ -39,10 +39,7 @@ export class GeminiService implements VlmProvider {
         },
       }));
 
-      const parts = [
-        { text: geminiConfig.prompt },
-        ...imageParts,
-      ];
+      const parts = [{ text: geminiConfig.prompt }, ...imageParts];
 
       const result = await genAI.models.generateContent({
         model: geminiConfig.model,

src/compare/libs/vlm/providers/ollama/ollama.controller.ts

@@ -61,4 +61,3 @@ export class OllamaController {
     return this.ollamaService.generate(config, images);
   }
 }
-

src/compare/libs/vlm/providers/ollama/ollama.service.spec.ts

@@ -122,7 +122,6 @@ describe('OllamaService', () => {
     });
   });
 
-
   describe('listModels', () => {
     it('should return list of models', async () => {
       const mockDate = new Date('2024-01-01');
@@ -179,4 +178,3 @@ describe('OllamaService', () => {
     });
   });
 });
-

src/compare/libs/vlm/vlm-provider.interface.ts

@@ -27,4 +27,3 @@ export interface VlmProviderResponse {
    */
   thinking?: string;
 }
-

src/compare/libs/vlm/vlm.service.spec.ts

@@ -55,16 +55,11 @@ const createImageMocks = () => {
   return {
     image,
     diffImage,
-    getImageMock: jest
-      .fn()
-      .mockReturnValueOnce(image)
-      .mockReturnValueOnce(image)
-      .mockReturnValueOnce(diffImage),
+    getImageMock: jest.fn().mockReturnValueOnce(image).mockReturnValueOnce(image).mockReturnValueOnce(diffImage),
   };
 };
 
 describe('VlmService', () => {
-
   it('should return NO_BASELINE_RESULT when pixelmatch returns no baseline', async () => {
     const pixelmatchGetDiffMock = jest.fn().mockResolvedValue(NO_BASELINE_RESULT);
     const service = await initService({ pixelmatchGetDiffMock });
@@ -109,7 +104,10 @@ describe('VlmService', () => {
     ],
     [
       'keep unresolved when VLM confirms noticeable',
-      { identical: false, description: 'Button text changed from Submit to Send, and user count changed from 12 to 15.' },
+      {
+        identical: false,
+        description: 'Button text changed from Submit to Send, and user count changed from 12 to 15.',
+      },
       TestStatus.unresolved,
       { pixelMisMatchCount: 500, diffPercent: 12.5 },
     ],
@@ -138,7 +136,11 @@ describe('VlmService', () => {
   });
 
   it.each([
-    ['invalid JSON response', { content: 'Invalid JSON response from model' }, { pixelMisMatchCount: 200, diffPercent: 5 }],
+    [
+      'invalid JSON response',
+      { content: 'Invalid JSON response from model' },
+      { pixelMisMatchCount: 200, diffPercent: 5 },
+    ],
     ['API error', null, { pixelMisMatchCount: 300, diffPercent: 7.5 }],
   ])('should handle %s gracefully and return pixelmatch result', async (_, vlmResponse, pixelmatchOverrides) => {
     const pixelmatchResult = createPixelmatchResult(pixelmatchOverrides);

src/compare/libs/vlm/vlm.service.ts

@@ -11,7 +11,7 @@ import { OllamaService } from './providers/ollama/ollama.service';
 import { GeminiService } from './providers/gemini/gemini.service';
 import { PixelmatchService, DEFAULT_CONFIG as PIXELMATCH_DEFAULT_CONFIG } from '../pixelmatch/pixelmatch.service';
 import { PNG } from 'pngjs';
-import z from 'zod/v3'; 
+import z from 'zod/v3';
 
 export const DEFAULT_PROMPT = `You are provided with three images:
 1. First image: baseline screenshot