minor refactor

Author: Nick-1234531

src/lib/processSnapshot.ts

@@ -669,7 +669,6 @@ export default async function processSnapshot(snapshot: Snapshot, ctx: Context):
             if (!Array.isArray(processedOptions[ignoreOrSelectBoxes][viewportString])) processedOptions[ignoreOrSelectBoxes][viewportString] = []
 
             for (const selector of selectors) {
-                // Handle coordinates validation
                 if (selector.startsWith('coordinates=')) {
                     const coordString = selector.replace('coordinates=', '');
                     const viewportSize = await page.viewportSize();
@@ -687,7 +686,6 @@ export default async function processSnapshot(snapshot: Snapshot, ctx: Context):
                         continue;
                     }
 
-                    // Coordinates are valid - create a coordinate element
                     const coordinateElement = { 
                         type: 'coordinates', 
                         ...validation.coords
@@ -708,7 +706,6 @@ export default async function processSnapshot(snapshot: Snapshot, ctx: Context):
                 if (locator && typeof locator === 'object' && locator.hasOwnProperty('type') && (locator as any).type === 'coordinates') {
                     const coordLocator = locator as any;
                     const { top, bottom, left, right } = coordLocator;
-                    // Coordinates already validated, push directly
                     processedOptions[ignoreOrSelectBoxes][viewportString].push({
                         left: left,
                         top: top,

src/lib/utils.ts

@@ -518,10 +518,8 @@ export function validateCoordinates(
     viewportString: string
 ): { valid: boolean, error?: string, coords?: { top: number, bottom: number, left: number, right: number } } {
 
-    // Parse coordinates
     const coords = coordString.split(',').map(Number);
 
-    // Check format
     if (coords.length !== 4) {
         return { 
             valid: false, 
@@ -531,15 +529,13 @@ export function validateCoordinates(
 
     const [top, bottom, left, right] = coords;
 
-    // Check if all values are numbers
     if (coords.some(isNaN)) {
         return { 
             valid: false, 
             error: `for snapshot ${snapshotName} viewport ${viewportString}, invalid coordinate values: ${coordString}. All values must be numbers` 
         };
     }
 
-    // Check coordinate bounds
     if (top < 0 || left < 0 || bottom < 0 || right < 0) {
         return { 
             valid: false, 
@@ -561,7 +557,6 @@ export function validateCoordinates(
         };
     }
 
-    // Check viewport bounds
     if (bottom > viewportSize.height) {
         return { 
             valid: false, 
@@ -576,7 +571,6 @@ export function validateCoordinates(
         };
     }
 
-    // All validations passed
     return { 
         valid: true, 
         coords: { top, bottom, left, right }