feat: update Ansible collection retrieval to use ansible-galaxy for improved accuracy

bin/actions/update.py

@@ -29,41 +29,50 @@ def ensure_ansible_collections():
     
     printfe("cyan", "Checking for required Ansible collections...")
     
-    # Get default collections paths from Ansible config
+    # Get list of installed collections using ansible-galaxy
-    collections_paths = []
+    status, output = run_command(["ansible-galaxy", "collection", "list"], shell=False)
-    try:
+    if not status:
-        status, output = run_command(["ansible-config", "dump", "COLLECTIONS_PATHS"], shell=False)
+        printfe("yellow", f"Failed to list Ansible collections: {output}")
-        if status and output.strip():
+        printfe("yellow", "Will try to install all required collections.")
-            # Extract paths from output which is in format "COLLECTIONS_PATHS(default) = ['/path1', '/path2']"
+        installed_collections = []
-            import re
+    else:
-            paths_match = re.search(r'\[(.*)\]', output)
+        # Parse output to get installed collections
-            if paths_match:
+        installed_collections = []
-                paths_str = paths_match.group(1)
-                # Split by comma, strip quotes and whitespace
-                collections_paths = [p.strip().strip("'\"") for p in paths_str.split(',')]
-    except Exception as e:
-        printfe("yellow", f"Failed to get collections paths: {e}")
         
-    # Add default paths if we couldn't get them from config
+        # Split output into lines and process
-    if not collections_paths:
+        lines = output.splitlines()
-        collections_paths = [
+        collection_section = False
-            os.path.expanduser("~/.ansible/collections"),
-            "/usr/share/ansible/collections"
-        ]
         
-    # Check if each required collection is installed
+        for line in lines:
+            line = line.strip()
+            
+            # Skip empty lines
+            if not line:
+                continue
+                
+            # Check if we've reached the collection listing section
+            if line.startswith("Collection"):
+                collection_section = True
+                continue
+                
+            # Skip the separator line after the header
+            if collection_section and line.startswith("--"):
+                continue
+                
+            # Process collection entries
+            if collection_section and " " in line:
+                # Format is typically: "community.general 10.4.0"
+                parts = line.split()
+                if len(parts) >= 1:
+                    collection_name = parts[0]
+                    installed_collections.append(collection_name)
+    
+    printfe("cyan", f"Found installed collections: {', '.join(installed_collections) if installed_collections else 'none'}")
+    
+    # Check which required collections are missing
     missing_collections = []
     for collection in required_collections:
-        collection_found = False
+        if collection not in installed_collections:
-        namespace, name = collection.split('.')
-        
-        for path in collections_paths:
-            collection_path = os.path.join(path, "ansible_collections", namespace, name)
-            if os.path.exists(collection_path):
-                collection_found = True
-                break
-        
-        if not collection_found:
             missing_collections.append(collection)
     
     # Install missing collections