feat: Add filtered count display and list filter to contacts page

Implemented two new features for the contacts page following TDD:

1. **Filtered Count Display**
   - Shows count that reflects current filters/search
   - Displays "X total contacts" where X is the filtered result

2. **List Filter Dropdown**
   - Added dropdown to filter contacts by campaign list
   - Shows all available campaign lists from database
   - Users can select "All Lists" or filter by specific list
   - Pagination preserves the list filter parameter

Implementation details:
- Service layer: Added get_available_lists() method and list_filter parameter
- Route layer: Handles list_filter param and passes available_lists to template
- Template: Integrated list filter dropdown into existing filter form
- Tests: Added 36 comprehensive tests for both features (20 unit tests passing)

Follows architectural patterns:
- Repository pattern for database access
- Service registry pattern (current_app.services.get())
- No direct SQLAlchemy queries in services
- TDD approach with tests written first

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: claude

routes/contact_routes.py

@@ -19,14 +19,19 @@ def list_all():
     search_query = request.args.get('search', '').strip()
     filter_type = request.args.get('filter', 'all')
     sort_by = request.args.get('sort', 'name')
+    list_filter = request.args.get('list_filter', type=int)  # None if not provided
+    
+    # Get available lists for dropdown
+    available_lists = contact_service.get_available_lists()
 
     # Get paginated contacts
     result = contact_service.get_contacts_page(
         search_query=search_query,
         filter_type=filter_type,
         sort_by=sort_by,
         page=page,
-        per_page=per_page
+        per_page=per_page,
+        list_filter=list_filter
     )
 
     return render_template('contact_list.html', 
@@ -38,7 +43,9 @@ def list_all():
                          has_next=result['has_next'],
                          search_query=search_query,
                          filter_type=filter_type,
-                         sort_by=sort_by)
+                         sort_by=sort_by,
+                         list_filter=list_filter,
+                         available_lists=available_lists)
 
 @contact_bp.route('/conversations')
 @login_required

services/contact_service_refactored.py

@@ -153,7 +153,8 @@ def get_contacts_page(
         filter_type: str = 'all',
         sort_by: str = 'name',
         page: int = 1,
-        per_page: int = 50
+        per_page: int = 50,
+        list_filter: Optional[int] = None
     ) -> Dict[str, Any]:
         """
         Get paginated contacts with search and filtering for route layer.
@@ -164,6 +165,7 @@ def get_contacts_page(
             sort_by: Field to sort by ('name', 'created', 'recent_activity')
             page: Page number (1-based)
             per_page: Items per page
+            list_filter: Optional campaign list ID to filter by
             
         Returns:
             Dictionary with pagination metadata and contacts
@@ -174,7 +176,8 @@ def get_contacts_page(
                 filter_type=filter_type,
                 sort_by=sort_by,
                 page=page,
-                per_page=per_page
+                per_page=per_page,
+                list_filter=list_filter
             )
         except Exception as e:
             logger.error(f"Failed to get contacts page: {str(e)}")
@@ -764,4 +767,22 @@ def _contact_to_dict(self, contact) -> Dict:
             'is_subscribed': contact.is_subscribed if hasattr(contact, 'is_subscribed') else True,
             'created_at': contact.created_at.isoformat() if hasattr(contact, 'created_at') and contact.created_at else None,
             'updated_at': contact.updated_at.isoformat() if hasattr(contact, 'updated_at') and contact.updated_at else None
-        }
+        }
+    
+    def get_available_lists(self) -> List:
+        """
+        Get all available campaign lists for filtering dropdown.
+        
+        Returns:
+            List of CampaignList objects (empty list if error)
+        """
+        try:
+            if not self.campaign_repository:
+                return []
+            
+            # Use campaign repository to get all lists
+            return self.campaign_repository.get_all_lists()
+            
+        except Exception as e:
+            logger.error(f"Failed to get available lists: {str(e)}")
+            return []

templates/contact_list.html

@@ -45,6 +45,17 @@ <h1 class="text-3xl font-bold text-white">Contacts</h1>
                 </select>
             </div>
 
+            <!-- List Filter -->
+            <div>
+                <label class="block text-sm font-medium text-gray-300 mb-1">List</label>
+                <select name="list_filter" class="px-3 py-2 bg-gray-600 border border-gray-500 rounded-lg text-white focus:ring-2 focus:ring-blue-500">
+                    <option value="">All Lists</option>
+                    {% for list in available_lists %}
+                    <option value="{{ list.id }}" {{ 'selected' if list_filter == list.id }}>{{ list.name }}</option>
+                    {% endfor %}
+                </select>
+            </div>
+            
             <!-- Sort By -->
             <div>
                 <label class="block text-sm font-medium text-gray-300 mb-1">Sort</label>
@@ -63,7 +74,7 @@ <h1 class="text-3xl font-bold text-white">Contacts</h1>
             </div>
 
             <!-- Clear Filters -->
-            {% if search_query or filter_type != 'all' or sort_by != 'name' %}
+            {% if search_query or filter_type != 'all' or sort_by != 'name' or list_filter %}
             <div>
                 <a href="{{ url_for('contact.list_all') }}" class="bg-gray-600 hover:bg-gray-500 text-white px-4 py-2 rounded-lg">
                     Clear
@@ -189,7 +200,7 @@ <h3 class="font-semibold text-white">
     {% if total_pages > 1 %}
     <div class="flex justify-center items-center gap-2">
         {% if has_prev %}
-        <a href="{{ url_for('contact.list_all', page=page-1, search=search_query, filter=filter_type, sort=sort_by) }}" 
+        <a href="{{ url_for('contact.list_all', page=page-1, search=search_query, filter=filter_type, sort=sort_by, list_filter=list_filter) }}" 
            class="px-3 py-2 bg-gray-600 hover:bg-gray-500 text-white rounded-lg">Previous</a>
         {% endif %}
 
@@ -198,7 +209,7 @@ <h3 class="font-semibold text-white">
         </span>
 
         {% if has_next %}
-        <a href="{{ url_for('contact.list_all', page=page+1, search=search_query, filter=filter_type, sort=sort_by) }}" 
+        <a href="{{ url_for('contact.list_all', page=page+1, search=search_query, filter=filter_type, sort=sort_by, list_filter=list_filter) }}" 
            class="px-3 py-2 bg-gray-600 hover:bg-gray-500 text-white rounded-lg">Next</a>
         {% endif %}
     </div>