review findings fixes

Author: M-Hietala

src/azure-cli/azure/cli/command_modules/cognitiveservices/_client_factory.py

@@ -105,10 +105,6 @@ def cf_outbound_rule(cli_ctx, *_):
     return get_cognitiveservices_management_client(cli_ctx).outbound_rule
 
 
-def cf_outbound_rules(cli_ctx, *_):
-    return get_cognitiveservices_management_client(cli_ctx).outbound_rules
-
-
 def cf_account_capability_hosts(cli_ctx, *_):
     return get_cognitiveservices_management_client(cli_ctx).account_capability_hosts
 

src/azure-cli/azure/cli/command_modules/cognitiveservices/_params.py

@@ -706,7 +706,6 @@ def load_arguments(self, _):
                    help='Firewall SKU for the managed network.')
 
     with self.argument_context('cognitiveservices account managed-network update') as c:
-        c.argument('managed_network_name', default=None)
         c.argument('managed_network',
                    options_list=['--managed-network'],
                    arg_type=get_enum_type(['allow_internet_outbound', 'allow_only_approved_outbound']),
@@ -741,7 +740,7 @@ def load_arguments(self, _):
                    help='Destination for the outbound rule. '
                         'For FQDN rules, this is the FQDN string. '
                         'For PrivateEndpoint rules, this is the service resource ID. '
-                        'For ServiceTag rules, provide a JSON string or @file path.')
+                        'For ServiceTag rules, provide a JSON string.')
         c.argument('subresource_target',
                    options_list=['--subresource-target'],
                    help='Subresource target for PrivateEndpoint outbound rules '

src/azure-cli/azure/cli/command_modules/cognitiveservices/custom.py

@@ -2330,14 +2330,23 @@ def _build_outbound_rule(rule_type, category=None, destination=None, subresource
         )
     if normalized_type == 'ServiceTag':
         # ServiceTag requires a structured destination object with serviceTag field
+        # Map camelCase keys (from JSON examples) to snake_case (SDK model kwargs)
+        _service_tag_key_map = {
+            'serviceTag': 'service_tag',
+            'portRanges': 'port_ranges',
+        }
+
+        def _normalize_service_tag_keys(d):
+            return {_service_tag_key_map.get(k, k): v for k, v in d.items()}
+
         if isinstance(destination, ServiceTagOutboundRuleDestination):
             dest_obj = destination
         elif isinstance(destination, dict):
-            dest_obj = ServiceTagOutboundRuleDestination(**destination)
+            dest_obj = ServiceTagOutboundRuleDestination(**_normalize_service_tag_keys(destination))
         elif isinstance(destination, str):
             try:
                 dest_dict = json.loads(destination)
-                dest_obj = ServiceTagOutboundRuleDestination(**dest_dict)
+                dest_obj = ServiceTagOutboundRuleDestination(**_normalize_service_tag_keys(dest_dict))
             except (json.JSONDecodeError, TypeError):
                 dest_obj = ServiceTagOutboundRuleDestination(
                     service_tag=destination,

src/azure-cli/azure/cli/command_modules/cognitiveservices/linter_exclusions.yml

@@ -9,6 +9,11 @@ cognitiveservices account managed-network:
     managed_network_name:
       rule_exclusions:
         - missing_parameter_test_coverage
+cognitiveservices account managed-network update:
+  parameters:
+    managed_network_name:
+      rule_exclusions:
+        - no_parameter_defaults_for_update_commands
 cognitiveservices account connection list:
   parameters:
     category:

src/azure-cli/azure/cli/command_modules/cognitiveservices/tests/latest/test_managed_network.py

@@ -194,20 +194,7 @@ def test_outbound_rule_service_tag(self, resource_group):
 
         sname = self.create_random_name(prefix='cog', length=12)
         rule_name = 'test-st-rule'
-        
-        # Print resource details so we can share with service team
-        import sys
-        from datetime import datetime, timezone
-        print(f'\n=== SERVICE TAG TEST RESOURCES ===', file=sys.stderr)
-        print(f'Subscription:  {self.get_subscription_id()}', file=sys.stderr)
-        print(f'Resource Group: {resource_group}', file=sys.stderr)
-        print(f'Account Name:  {sname}', file=sys.stderr)
-        print(f'Rule Name:     {rule_name}', file=sys.stderr)
-        print(f'Region:        eastus', file=sys.stderr)
-        print(f'API Version:   2025-10-01-preview', file=sys.stderr)
-        print(f'Timestamp:     {datetime.now(timezone.utc).isoformat()}', file=sys.stderr)
-        print(f'=================================\n', file=sys.stderr)
-        
+
         self.kwargs.update({
             'sname': sname,
             'kind': 'AIServices',