Skip to content

Fix C# server model filenames and enum XML docs - #11591

Open
Timothee Guerin (timotheeguerin) with Copilot wants to merge 6 commits into
mainfrom
copilot/fix-generated-filename-xml-doc
Open

Fix C# server model filenames and enum XML docs#11591
Timothee Guerin (timotheeguerin) with Copilot wants to merge 6 commits into
mainfrom
copilot/fix-generated-filename-xml-doc

Conversation

Copilot AI commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

The C# server emitter generated model and enum filenames from raw TypeSpec names instead of the C# naming policy. Enum XML documentation was also emitted below [JsonConverter].

Incorrect camel-cased generated filenames

  • File naming

    • Apply the C# naming policy to model and enum filenames.
    • camelModel and camelEnum now generate CamelModel.cs and CamelEnum.cs.
  • Enum documentation

    • Emit XML documentation before declaration attributes:
/// <summary>
/// The kind of entity that created the resource.
/// </summary>
[JsonConverter(typeof(JsonStringEnumConverter))]
public enum CreatedByType
  • Regression coverage
    • Verify generated paths and enum documentation ordering.
    • Update affected generated-output snapshots.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI and others added 2 commits August 7, 2026 14:40
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix incorrect file name and XML documentation in createdByType.cs Fix C# server model filenames and enum XML docs Aug 7, 2026
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@pkg-pr-new

pkg-pr-new Bot commented Aug 7, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-server-csharp@11591

commit: e96aece

@timotheeguerin

Copy link
Copy Markdown
Member

Copilot fix spellcheck

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Copilot fix spellcheck

Fixed the Icamel spellcheck failure in 2df003d. Targeted cspell, formatting, lint, package build, and emitter tests pass.

Comment thread packages/http-server-csharp/test/emitter.test.ts Outdated
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@azure-sdk-automation

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

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.

[Bug]: http-server-csharp: Generated createdbyType.cs has incorrect file name, XML doc

2 participants