Fix QuickSight permissions for DataSources and DataSets

- Apply same permissions to DataSources and DataSets as Dashboards
- Renamed dashboard_permissions to asset_permissions for clarity
- All three asset types now get proper Principals and Actions arrays

Author: your-github-username

experimental/SMUS-CICD-pipeline-cli/src/smus_cicd/helpers/quicksight.py

@@ -208,8 +208,8 @@ def import_dashboard(
         else:
             job_id = f"import-{timestamp}"
 
-        # Build dashboard permissions for OverridePermissions
-        dashboard_permissions = {}
+        # Build permissions for OverridePermissions (applies to all asset types)
+        asset_permissions = {}
         if permissions:
             principals = []
             actions = []
@@ -218,18 +218,18 @@ def import_dashboard(
                 actions.extend(perm["actions"])
             # Remove duplicates from actions
             actions = list(set(actions))
-            dashboard_permissions = {"Principals": principals, "Actions": actions}
+            asset_permissions = {"Principals": principals, "Actions": actions}
 
         import_params = {
             "AwsAccountId": aws_account_id,
             "AssetBundleImportJobId": job_id,
             "AssetBundleImportSource": {"Body": _download_bundle(bundle_url)},
             "FailureAction": "ROLLBACK",
             "OverridePermissions": {
-                "DataSources": [{"DataSourceIds": ["*"], "Permissions": {}}],
-                "DataSets": [{"DataSetIds": ["*"], "Permissions": {}}],
+                "DataSources": [{"DataSourceIds": ["*"], "Permissions": asset_permissions}],
+                "DataSets": [{"DataSetIds": ["*"], "Permissions": asset_permissions}],
                 "Dashboards": [
-                    {"DashboardIds": ["*"], "Permissions": dashboard_permissions}
+                    {"DashboardIds": ["*"], "Permissions": asset_permissions}
                 ],
             },
         }