Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Sep 29, 2025

  • Implement OpenAPI 3.2.0 server name field with backward compatibility
  • Add comprehensive test coverage for server name serialization/deserialization
  • Update benchmark results for EmptyModels
Original prompt

This section details on the original issue you should resolve

<issue_title>Server - new name field</issue_title>
<issue_description>OAI 3.2.0 defines a new server name field (string?)

We should add this new field to the model, and implement serialization/deserialization.

When serializing to OpenAPI 3.1.0, we should add an x-oai- prefix to make it an extension</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #16

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Server - new name field Implement OpenAPI 3.2.0 server name field with backward compatibility Sep 29, 2025
@Copilot Copilot AI requested a review from baywet September 29, 2025 16:48
Copilot finished work on behalf of baywet September 29, 2025 16:48
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
@baywet baywet marked this pull request as ready for review September 29, 2025 17:09
@baywet baywet enabled auto-merge (squash) September 29, 2025 17:12
@baywet baywet disabled auto-merge September 29, 2025 17:17
@baywet baywet enabled auto-merge (squash) September 29, 2025 17:20
@baywet
Copy link

baywet commented Sep 29, 2025

@copilot run the following commands cd performance/benchmark && dotnet run -c Release EmptyModels and commit the results, do nothing else

Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
auto-merge was automatically disabled September 29, 2025 23:58

Head branch was pushed to by a user without write access

Copy link
Author

Copilot AI commented Sep 29, 2025

@copilot run the following commands cd performance/benchmark && dotnet run -c Release EmptyModels and commit the results, do nothing else

Benchmark results updated in commit d964404.

@Copilot Copilot AI requested a review from baywet September 29, 2025 23:59
Copilot finished work on behalf of baywet September 29, 2025 23:59
@baywet baywet enabled auto-merge September 30, 2025 00:55
@baywet baywet merged commit 9bba39f into feat/oai-3-2-support Sep 30, 2025
9 checks passed
@baywet baywet deleted the copilot/fix-e82b7370-b029-4de8-b196-901596c61b07 branch September 30, 2025 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Server - new name field

3 participants