Added removal of conflicting containers

Author: dementii-priadko

postgres_ai

@@ -286,11 +286,77 @@ check_services_running() {
     log_success "Monitoring services are running"
 }
 
+# Clean up any existing containers that might conflict
+cleanup_existing_containers() {
+    log_info "Checking for existing containers that might conflict..."
+    
+    # List of container names that might conflict
+    local container_names=(
+        "sink-postgres"
+        "sink-prometheus" 
+        "pgwatch-postgres"
+        "pgwatch-prometheus"
+        "grafana"
+        "flask-backend"
+        "target-db"
+        "postgres-reports"
+        "sources-generator"
+    )
+    
+    # Check for containers matching our service names
+    local existing_containers=""
+    
+    for container_name in "${container_names[@]}"; do
+        local found_containers
+        found_containers=$(docker ps -a --filter "name=$container_name" --format "{{.Names}}" 2>/dev/null || true)
+        if [ -n "$found_containers" ]; then
+            if [ -n "$existing_containers" ]; then
+                existing_containers="$existing_containers"$'\n'"$found_containers"
+            else
+                existing_containers="$found_containers"
+            fi
+        fi
+    done
+    
+    if [ -n "$existing_containers" ]; then
+        log_warning "Found existing containers from previous installations:"
+        echo "$existing_containers" | sed 's/^/  • /'
+        echo
+        log_warning "These containers may conflict with the new installation."
+        log_warning "Removing them will stop any running services and delete their data."
+        echo
+        
+        read -p "Are you sure you want to remove these existing containers? (Y/n): " -n 1 -r
+        echo
+        
+        if [[ ! $REPLY =~ ^[Nn]$ ]]; then
+            log_info "Stopping and removing existing containers..."
+            
+            # Stop containers first
+            echo "$existing_containers" | xargs -r docker stop 2>/dev/null || true
+            
+            # Remove containers
+            echo "$existing_containers" | xargs -r docker rm 2>/dev/null || true
+            
+            # Remove any orphaned volumes
+            docker volume prune -f 2>/dev/null || true
+            
+            log_success "Existing containers removed"
+        else
+            log_warning "Keeping existing containers - this may cause conflicts during installation"
+            log_info "If you encounter issues, run '$0 reset' to clean up manually"
+        fi
+    else
+        log_success "No conflicting containers found"
+    fi
+}
+
 # Comprehensive precheck for installation commands
 precheck_for_install() {
     check_prerequisites
     check_docker_running
     check_system_resources
+    cleanup_existing_containers
 }
 
 # Precheck for service management commands
@@ -1187,13 +1253,28 @@ reset_environment() {
         cd "$SCRIPT_DIR"
 
         log_info "Stopping services and removing data..."
-        $compose_cmd -f "$COMPOSE_FILE" down -v
+        
+        # Stop and remove containers with volumes
+        $compose_cmd -f "$COMPOSE_FILE" down -v --remove-orphans
+        
+        # Force remove any remaining containers by name pattern
+        local remaining_containers
+        remaining_containers=$(docker ps -a --filter "name=postgres-ai-mon" --format "{{.Names}}" 2>/dev/null || true)
+        
+        if [ -n "$remaining_containers" ]; then
+            log_info "Removing any remaining containers..."
+            echo "$remaining_containers" | xargs -r docker stop 2>/dev/null || true
+            echo "$remaining_containers" | xargs -r docker rm 2>/dev/null || true
+        fi
+        
+        # Remove any orphaned volumes
+        docker volume ls --filter "name=postgres-ai-mon" --format "{{.Name}}" | xargs -r docker volume rm 2>/dev/null || true
 
         # Remove any generated config files
         rm -f "$SCRIPT_DIR/config/pgwatch-postgres/sources.yml"
         rm -f "$SCRIPT_DIR/config/pgwatch-prometheus/sources.yml"
 
-        log_success "Environment reset completed"
+        log_success "Environment reset completed - all containers and data removed"
     else
         log_info "Reset cancelled"
     fi