Fix exception sorting operation tags

[martincostello]

Fix SwaggerGeneratorException caused by an InvalidOperationException when attempting to compare OpenApiTagReference instances in a SortedSet<T>.

Resolves #3650.

[Copilot]

Pull Request Overview

This PR fixes a SwaggerGeneratorException caused by attempting to sort OpenApiTagReference instances in a SortedSet<T> without providing a proper comparer, which previously resulted in an InvalidOperationException.

• Introduces a new OpenApiTagComparer class to enable sorting of OpenApiTag and OpenApiTagReference instances
• Updates AnnotationsOperationFilter and AnnotationsDocumentFilter to use the new comparer when creating SortedSet collections
• Updates test fixtures to reflect that tags are now sorted alphabetically

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/Shared/OpenApiTagComparer.cs	New comparer implementation for OpenApiTag and OpenApiTagReference with case-sensitive alphabetical sorting
src/Swashbuckle.AspNetCore.Annotations/AnnotationsOperationFilter.cs	Added comparer parameter to SortedSet<OpenApiTagReference> instantiation
src/Swashbuckle.AspNetCore.Annotations/AnnotationsDocumentFilter.cs	Added comparer parameter to SortedSet<OpenApiTag> instantiation
src/Swashbuckle.AspNetCore.Annotations/Swashbuckle.AspNetCore.Annotations.csproj	Linked the shared OpenApiTagComparer.cs file into the project
test/Swashbuckle.AspNetCore.Annotations.Test/Fixtures/FakeControllerWithSwaggerAnnotations.cs	Updated test fixture from single tag "foobar" to two tags ["foo", "bar"]
test/Swashbuckle.AspNetCore.Annotations.Test/AnnotationsOperationFilterTests.cs	Updated assertion to expect sorted tags ["bar", "foo"]

Comments suppressed due to low confidence (1)

src/Shared/OpenApiTagComparer.cs:130

• Variable obj may be null at this access as suggested by this null check.

        string value = obj?.Name ?? obj.Reference.Id;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

❌ Patch coverage is 65.51724% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.53%. Comparing base (5bb0e76) to head (1da893c).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
src/Shared/OpenApiTagComparer.cs	62.96%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3652      +/-   ##
==========================================
- Coverage   94.75%   94.53%   -0.23%     
==========================================
  Files         110      111       +1     
  Lines        3834     3861      +27     
  Branches      764      778      +14     
==========================================
+ Hits         3633     3650      +17     
- Misses        201      211      +10     

Flag	Coverage Δ
Linux	94.53% <65.51%> (-0.23%)	⬇️
Windows	94.53% <65.51%> (-0.23%)	⬇️
macOS	94.53% <65.51%> (-0.23%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.