fix: Address PR review feedback

- Fix syntax error: Remove duplicate return statement (line 2218)
- Fix grammar: 'are yet supported' -> 'are not yet supported'
- Remove commented-out code
- Fix dry-run allow_insecure to show actual config value
- Fix choice_index undefined variable with proper error handling
- Remove duplicate 'import re' statements
- Fix gpu_profile undefined variable by reading from raw YAML
- Revert version from 1.2.0b6 to 1.2.0b5 (maintainers handle releases)
- Remove premium-ingress params that came from upstream rebase

Addresses feedback from @Greedygre and Copilot AI review

Author: Simon Jakesch

src/containerapp/azext_containerapp/_compose_utils.py

@@ -104,7 +104,7 @@ def warn_about_unsupported_build_configuration(compose_service):
                                  "target", "secrets", "tags"]
     if compose_service.build is not None:
         config_list = resolve_configuration_element_list(compose_service, unsupported_configuration, 'build')
-        message = "These build configuration settings from the docker-compose file are yet supported."
+        message = "These build configuration settings from the docker-compose file are not yet supported."
         message += " Currently, we support supplying a build context and optionally target Dockerfile for a service."
         message += " See https://aka.ms/containerapp/compose/build_support for more information or to add feedback."
         if len(config_list) >= 1:
@@ -314,12 +314,15 @@ def resolve_memory_configuration_from_service(service):
 
 
 def resolve_port_or_expose_list(ports, name):
-    if len(ports) > 1:
+    if len(ports) == 1:
+        return ports[0]
+    elif len(ports) > 1:
         message = f"You have more than one {name} mapping defined in your docker-compose file."
         message += " Which port would you like to use? "
         choice_index = prompt_choice_list(message, ports)
-
-    return ports[choice_index]
+        return ports[choice_index]
+    else:
+        raise ValueError(f"No {name} mappings defined.")
 
 
 def resolve_ingress_and_target_port(service):

src/containerapp/azext_containerapp/_params.py

@@ -521,17 +521,6 @@ def load_arguments(self, _):
         c.argument('mode', arg_type=get_enum_type(['single', 'multiple', 'labels']), help="The active revisions mode for the container app.")
         c.argument('target_label', help="The label to apply to new revisions. Required for revision mode 'labels'.", is_preview=True)
 
-    with self.argument_context('containerapp env premium-ingress') as c:
-        c.argument('resource_group_name', arg_type=resource_group_name_type, id_part=None)
-        c.argument('name', options_list=['--name', '-n'], help="The name of the managed environment.")
-        c.argument('workload_profile_name', options_list=['--workload-profile-name', '-w'], help="The workload profile to run ingress replicas on. This profile must not be shared with any container app or job.")
-        c.argument('min_replicas', options_list=['--min-replicas'], type=int, deprecate_info=c.deprecate(hide=True, expiration='2.79.0'), help="The workload profile minimum instances is used instead.")
-        c.argument('max_replicas', options_list=['--max-replicas'], type=int, deprecate_info=c.deprecate(hide=True, expiration='2.79.0'), help="The workload profile maximum instances is used instead.")
-        c.argument('termination_grace_period', options_list=['--termination-grace-period', '-t'], type=int, help="Time in seconds to drain requests during ingress shutdown. Default 500, minimum 0, maximum 3600.")
-        c.argument('request_idle_timeout', options_list=['--request-idle-timeout'], type=int, help="Timeout in minutes for idle requests. Default 4, minimum 4, maximum 30.")
-        c.argument('header_count_limit', options_list=['--header-count-limit'], type=int, help="Limit of http headers per request. Default 100, minimum 1.")
-
-    # Compose
     with self.argument_context('containerapp compose create') as c:
         c.argument('dry_run', options_list=['--dry-run'], arg_type=get_three_state_flag(),
                    help='Preview deployment without creating actual Azure resources. Generates a detailed report of container apps, workload profiles, role assignments, and environment variable injections.')

src/containerapp/azext_containerapp/custom.py

@@ -1814,7 +1814,6 @@ def create_containerapps_from_compose(cmd,  # pylint: disable=R0914
                         gateway_fqdn = gateway_fqdn.replace('https://', f'{gateway_protocol}://')
 
                         # Extract and preserve the path from the original URL (e.g., /sse from http://mcp-gateway:8811/sse)
-                        import re
                         path_match = re.search(r'://mcp-gateway(?::\d+)?(/[^\s]*)', var_value)
                         preserved_path = path_match.group(1) if path_match else ''
 
@@ -1843,7 +1842,6 @@ def create_containerapps_from_compose(cmd,  # pylint: disable=R0914
                         gateway_fqdn = gateway_fqdn.replace('https://', f'{gateway_protocol}://')
 
                         # Extract and preserve the path from the original URL
-                        import re
                         path_match = re.search(r'://mcp-gateway(?::\d+)?(/[^\s]*)', value)
                         preserved_path = path_match.group(1) if path_match else ''
 
@@ -2133,14 +2131,12 @@ def create_containerapps_from_compose(cmd,  # pylint: disable=R0914
                 replicas=replicas,
                 is_models_service=(service_name == 'models'),
                 is_mcp_gateway=is_mcp_gateway,
-                gpu_type=gpu_profile.get('type') if (service_name == 'models' and 'gpu_profile' in locals()) else None,
+                gpu_type=raw_service_yaml.get('x-azure-deployment', {}).get('workloadProfiles', {}).get('workloadProfileType') if service_name == 'models' else None,
                 raw_service=raw_service_yaml,
                 models_config=models
             )
             dry_run_services.append(service_config)
 
-            # if service_name == 'models':
-            # dry_run_has_models = True
             if is_mcp_gateway:
                 dry_run_has_gateway = True
 
@@ -2215,7 +2211,7 @@ def create_containerapps_from_compose(cmd,  # pylint: disable=R0914
         )
 
         # Return empty list (no actual deployment)
-        return None  # Dry-run complete, no resources created    return containerapps_from_compose
+        return None  # Dry-run complete, no resources created
 
 
 def set_workload_profile(cmd, resource_group_name, env_name, workload_profile_name, workload_profile_type=None, min_nodes=None, max_nodes=None):