Peel Azure.AI.AgentServer.Core and Azure.ResourceManager.Cdn off the NoWarn skip-list (Azure#59691)

* Peel Azure.AI.AgentServer.Core off the NoWarn skip-list

Drop the project-wide <NoWarn>AZC0100;AZC0004</NoWarn> block from src/Azure.AI.AgentServer.Core.csproj and move both suppressions to eng/analyzerallowlist/Azure.AI.AgentServer.Core.txt with their architectural justifications:

- AZC0100 (no ConfigureAwait(false)): server-side hosting library on the ASP.NET Core request pipeline, where there is no UI / SynchronizationContext to capture.

- AZC0004 (no sync variant): async-only surface by design - hosting and pipeline glue (port binding, graceful shutdown, OpenTelemetry, AgentHostBuilder) where sync variants would deadlock under ASP.NET Core or duplicate code without value. Same exemption as AMQP-based SDKs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Peel Azure.ResourceManager.Cdn off the NoWarn skip-list

Drop the project-wide <NoWarn>CS1591</NoWarn> from src/Azure.ResourceManager.Cdn.csproj.

CS1591 fired on 66 backward-compatibility shim members across 36 files in src/Customization/Models/. These are [EditorBrowsable(EditorBrowsableState.Never)] discriminator-aliases and legacy constructors preserved from the pre-TypeSpec API; they are hidden from IntelliSense by design. A single-line summary referencing the file-level explanatory comment is added to each undocumented member.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: m-nash

eng/NoWarnSkipValidation.txt

@@ -16,7 +16,6 @@
 #
 # To remove a project from this list, see the workflow in eng/analyzerallowlist/README.md.
 Azure.AI.Agents.Persistent
-Azure.AI.AgentServer.Core
 Azure.AI.AgentServer.Invocations
 Azure.AI.AgentServer.Responses
 Azure.AI.AnomalyDetector
@@ -139,7 +138,6 @@ Azure.Provisioning.WebPubSub
 Azure.ResourceManager
 Azure.ResourceManager.AppService
 Azure.ResourceManager.Batch
-Azure.ResourceManager.Cdn
 Azure.ResourceManager.Chaos
 Azure.ResourceManager.Compute
 Azure.ResourceManager.ContainerRegistry

eng/analyzerallowlist/Azure.AI.AgentServer.Core.txt

@@ -0,0 +1,19 @@
+# Azure.AI.AgentServer.Core approved analyzer suppressions.
+# See eng/analyzerallowlist/README.md for the file format and review policy.
+
+# AZC0100 fires on awaits that omit ConfigureAwait(false). This package is a
+# server-side hosting library that runs on the ASP.NET Core request pipeline,
+# where there is no UI / SynchronizationContext to capture; ConfigureAwait(false)
+# is at best unnecessary and at worst incorrect for hosted-service / middleware
+# scenarios. The Azure.Core guideline that AZC0100 enforces is targeted at
+# client libraries called from arbitrary contexts and does not apply here.
+# See AGENTS.md §0.VI for the broader rationale across AgentServer packages.
+nowarn:AZC0100
+
+# AZC0004 fires on async APIs that lack a corresponding sync variant. This
+# package only exposes async APIs by design — its surface area is hosting and
+# pipeline glue (port binding, graceful shutdown, OpenTelemetry hooks,
+# AgentHostBuilder) where sync variants would either deadlock under ASP.NET
+# Core or duplicate code without value. This is the same exemption granted to
+# AMQP-based SDKs whose I/O model is intrinsically async.
+nowarn:AZC0004

sdk/agentserver/Azure.AI.AgentServer.Core/src/Azure.AI.AgentServer.Core.csproj

@@ -8,10 +8,6 @@
     <PackageTags>Microsoft Azure AI Agent Server Core ASP.NET Core OpenTelemetry</PackageTags>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <Nullable>enable</Nullable>
-    <!-- AZC0100: Server-side hosting library on ASP.NET Core async pipeline — ConfigureAwait(false) is unnecessary and harmful.
-         AZC0004: Async-only API — no sync variants needed (same exemption as AMQP-based SDKs).
-         See AGENTS.md §0.VI for rationale. -->
-    <NoWarn>$(NoWarn);AZC0100;AZC0004</NoWarn>
   </PropertyGroup>
   <ItemGroup>
     <FrameworkReference Include="Microsoft.AspNetCore.App" />

sdk/cdn/Azure.ResourceManager.Cdn/src/Azure.ResourceManager.Cdn.csproj

@@ -7,6 +7,5 @@
     <Description>Microsoft Azure Resource Manager client SDK for Azure resource provider Microsoft.Cdn.</Description>
     <PackageTags>azure;management;arm;resource manager;cdn</PackageTags>
     <DisableEnhancedAnalysis>true</DisableEnhancedAnalysis>
-    <NoWarn>$(NoWarn);CS1591;</NoWarn>
   </PropertyGroup>
 </Project>

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/CacheExpirationActionProperties.cs

@@ -14,12 +14,14 @@ namespace Azure.ResourceManager.Cdn.Models
     // The old constructor and ActionType property (bridging to TypeName) are preserved here, marked as EditorBrowsable.Never.
     public partial class CacheExpirationActionProperties
     {
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CacheExpirationActionProperties(CacheExpirationActionType actionType, CacheBehaviorSetting cacheBehavior, CdnCacheLevel cacheType) : this(cacheBehavior, cacheType)
         {
             ActionType = actionType;
         }
 
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CacheExpirationActionType ActionType
         {

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/CacheKeyQueryStringActionProperties.cs

@@ -14,12 +14,14 @@ namespace Azure.ResourceManager.Cdn.Models
     // The old constructor and ActionType property (bridging to TypeName) are preserved here, marked as EditorBrowsable.Never.
     public partial class CacheKeyQueryStringActionProperties
     {
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CacheKeyQueryStringActionProperties(CacheKeyQueryStringActionType actionType, QueryStringBehavior queryStringBehavior) : this(queryStringBehavior)
         {
             ActionType = actionType;
         }
 
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CacheKeyQueryStringActionType ActionType
         {

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/CanMigrateContent.cs

@@ -17,6 +17,7 @@ namespace Azure.ResourceManager.Cdn.Models
     public partial class CanMigrateContent
     {
         // Backward compatibility: old API used ctor(WritableSubResource)
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CanMigrateContent(WritableSubResource classicResourceReference) : this()
         {

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/CanMigrateResult.cs

@@ -12,6 +12,7 @@ namespace Azure.ResourceManager.Cdn.Models
     // The old string Id property is preserved here (delegating to ResourceId?.ToString()) and marked as EditorBrowsable.Never.
     public partial class CanMigrateResult
     {
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public string Id => ResourceId?.ToString();
     }

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/CdnCertificateSource.cs

@@ -14,12 +14,14 @@ namespace Azure.ResourceManager.Cdn.Models
     // The old API is preserved here and bridges to TypeName.
     public partial class CdnCertificateSource
     {
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CdnCertificateSource(CdnCertificateSourceType sourceType, CdnManagedCertificateType certificateType) : this(certificateType)
         {
             SourceType = sourceType;
         }
 
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public CdnCertificateSourceType SourceType
         {

sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/Models/ClientPortMatchCondition.cs

@@ -14,12 +14,14 @@ namespace Azure.ResourceManager.Cdn.Models
     // marked as EditorBrowsable.Never.
     public partial class ClientPortMatchCondition
     {
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public ClientPortMatchCondition(ClientPortMatchConditionType conditionType, ClientPortOperator clientPortOperator) : this(clientPortOperator)
         {
             ConditionType = conditionType;
         }
 
+        /// <summary> Backward-compatibility shim retained when the model was regenerated from TypeSpec; hidden from IntelliSense. See the file-level comment for details. </summary>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public ClientPortMatchConditionType ConditionType
         {