review changes

Author: camilamacedo86

internal/operator-controller/controllers/clusterextension_reconcile_steps.go

@@ -174,7 +174,10 @@ func handleResolutionError(ctx context.Context, c client.Client, state *reconcil
 		catalogsExist, catalogCheckErr := CheckCatalogsExist(ctx, c, ext)
 		if catalogCheckErr != nil {
 			msg := fmt.Sprintf("failed to resolve bundle: %v", err)
-			catalogName := getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+			var catalogName string
+			if ext.Spec.Source.Catalog != nil {
+				catalogName = getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+			}
 			l.Error(catalogCheckErr, "error checking if ClusterCatalogs exist, will retry resolution",
 				"resolutionError", err,
 				"packageName", getPackageName(ext),
@@ -189,7 +192,10 @@ func handleResolutionError(ctx context.Context, c client.Client, state *reconcil
 			// ClusterCatalogs exist but resolution failed - likely a transient issue (ClusterCatalog updating, cache stale, etc.)
 			// Retry resolution instead of falling back
 			msg := fmt.Sprintf("failed to resolve bundle, retrying: %v", err)
-			catalogName := getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+			var catalogName string
+			if ext.Spec.Source.Catalog != nil {
+				catalogName = getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+			}
 			l.Error(err, "resolution failed but matching ClusterCatalogs exist - retrying instead of falling back",
 				"packageName", getPackageName(ext),
 				"catalogName", catalogName)
@@ -203,7 +209,10 @@ func handleResolutionError(ctx context.Context, c client.Client, state *reconcil
 		// The controller watches ClusterCatalog resources, so when ClusterCatalogs become available again,
 		// a reconcile will be triggered automatically, allowing the extension to upgrade.
 		msg := fmt.Sprintf("continuing to maintain current installation at version %s due to catalog unavailability", state.revisionStates.Installed.Version)
-		catalogName := getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+		var catalogName string
+		if ext.Spec.Source.Catalog != nil {
+			catalogName = getCatalogNameFromSelector(ext.Spec.Source.Catalog.Selector)
+		}
 		l.Error(err, "matching ClusterCatalogs unavailable or deleted - falling back to installed bundle to maintain workload",
 			"packageName", getPackageName(ext),
 			"catalogName", catalogName,