Merged short alt detection back into main 'poor quality' report and removed separate 'short_alt' counts as requested

src/analyzer.py

@@ -89,44 +89,15 @@ class ImageAnalyzer:
                     poor_quality_images.append({'src': img['src'], 'alt': alt_text, 'reason': reason})
                     total_poor_quality += 1
             
-            # Split into mutually exclusive lists based on reason
-            # User requirement: "short_alt_count are not show sepratly bescue it show in poor_quality_count... remove and add in poor perfectly"
-            # We interpret this as requiring two distinct, non-overlapping sets.
+            # User Intent: "short alt list and that count are add in poor not it show separately"
+            # Action: Consolidate everything into 'poor_quality' and remove specific 'short_alt' reporting.
             
-            non_overlapping_poor = []
-            non_overlapping_short = []
-            
-            for img in poor_quality_images:
-                if img['reason'].startswith("Too short") or img['reason'].startswith("Too few words"):
-                    non_overlapping_short.append(img)
-                else:
-                    non_overlapping_poor.append(img)
-
-            # Update legacy counters (only adding strictly short ones to the legacy global count)
-            # But wait, should true "poor" ones be tracked globally too? 
-            # The summary returned `total_poor_quality` which was everything. 
-            # If we separate them, we should probably track them separately in summary too if possible, 
-            # but for now we update the page level reporting as requested.
-            
-            # Update global counters strictly based on the split
-            total_short_alt_legacy += len(non_overlapping_short)
-            
-            # The 'total_poor_quality' global in summary currently counts EVERYTHING.
-            # If we want to maintain that consistency, we keep it as processed in loop.
-            # BUT user said "remove and add in poor perfectly" -> implies separation.
-            # Let's subtract the short ones from the global poor quality count for the SUMMARY as well?
-            # actually, `total_poor_quality` was just `+=1` in the loop for every hit. 
-            # Let's adjust it to only count the non-overlapping poor ones.
-            total_poor_quality -= len(non_overlapping_short)
-
             pages_report.append({
                 "page_url": page_url,
                 "missing_alt_count": len(missing_images),
-                "poor_quality_count": len(non_overlapping_poor), # Mutually exclusive count
-                "short_alt_count": len(non_overlapping_short),   # Mutually exclusive count
+                "poor_quality_count": len(poor_quality_images), # Contains ALL poor images (Generic, Filename, Short)
                 "images_without_alt": list(set(missing_images)),
-                "images_with_poor_alt": non_overlapping_poor,
-                "images_with_short_alt": non_overlapping_short
+                "images_with_poor_alt": poor_quality_images
             })
 
         return {
@@ -134,8 +105,7 @@ class ImageAnalyzer:
                 "total_pages_scanned": len(site_data),
                 "total_images_found": total_images,
                 "total_images_missing_alt": total_missing_alt,
-                "total_images_poor_quality": total_poor_quality, # New summary metric
-                "total_images_short_alt": total_short_alt_legacy # Reverted to strictly short counts
+                "total_images_poor_quality": total_poor_quality
             },
             "details": pages_report
         }