DO NOT MERGE - review PR before submission upstream

[baywet]

No description provided.

To address the issue, replace the loop:

foreach(var type in list)
{
    if (type is not null)
    {
        // logic
    }
}

with a loop that explicitly filters non-null values before iteration, using LINQ's .Where():

foreach(var type in list.Where(type => type is not null))
{
    // logic
}

This reduces nesting, improves readability, and explicitly communicates the intent to filter out nulls. Only the relevant region inside the dictionary initializer at lines 190-210 (in the block for "type") needs changing.

No new imports or other method/variable definitions are needed, as System.Linq is already imported.

---

The best way to fix this pattern matching issue is to restructure the switch expression to properly handle the intended cases. If OpenApiSpecVersion.OpenApi3_2 should have its own branch, then that branch should be written as a separate case. If all other enum values should fall into a generic default, then only the discard arm (_) should be used for the fallback. In this code, to retain the ability to match OpenApiSpecVersion.OpenApi3_2 specifically, use two arms: one for that explicit value, and one (_) as a fallback for anything else. Therefore, line 176 should be split into two lines: one matching only OpenApiSpecVersion.OpenApi3_2, and the next with the discard pattern _.

Edit the switch expression as follows:

• Replace line 176

  OpenApiSpecVersion.OpenApi3_2 or _ => _standardHttp32MethodsNames,
  

  with

  OpenApiSpecVersion.OpenApi3_2 => _standardHttp32MethodsNames,
  _ => _standardHttp32MethodsNames,
  

OR, since both branches do the same thing, just use the discard (_) as the only branch:

_ => _standardHttp32MethodsNames

However, this latter approach may lose clarity; if the intention is to handle OpenApiSpecVersion.OpenApi3_2 specifically, splitting makes it clearer for future modifications.

No new imports or method definitions are required.

[baywet]

closing in favour of microsoft#2529