[WIP] Add support for deterministic naming in model suffix by Copilot · Pull Request #11999 · microsoft/typespec

Copilot · 2026-09-17T17:34:12Z

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

This section details on the original issue you should resolve

<issue_title>Support Deterministic Naming for model suffix Response to Result</issue_title>
<issue_description></issue_description>

Closes Support Deterministic Naming for model suffix Response to Result #11993

Comments on the Issue (you are Copilot in this section)

Fixes Support Deterministic Naming for model suffix Response to Result #11993

azure-pipelines · 2026-09-17T17:34:37Z

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.

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

pkg-pr-new · 2026-09-18T19:47:08Z

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-csharp@11999

commit: 6584968

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Jorge Rangel (jorgerangel-msft) · 2026-09-18T20:04:50Z

+            var resultName = $"{normalizedName[..^responseSuffix.Length]}Result";
+            var inputNamespace = CodeModelGenerator.Instance.InputLibrary.InputNamespace;
+            // Model and enum files share a flat output directory, even across namespaces.
+            return inputNamespace.Models.Any(model => HasConflictingName(model, model.Namespace)) ||


Copilot why do we need to check with conflicting names?

The rename can introduce a collision that wasn’t in the input: WidgetResponse and WidgetResult would both become WidgetResult. Models and enums also share the flat Generated/Models directory, so even different namespaces can collide on filenames. The checks retain WidgetResponse when the target is already occupied, including through a customization, rather than emitting duplicate types or overwriting generated files.

Copilot is that something we have to check though? doesn't our codewriter and type resolution handle that for us ?

Not for this kind of collision. I traced the paths:

CodeWriter qualifies type references, but writes declaration names unchanged. Its unique-name allocation applies to CodeWriterDeclaration identifiers, not model types.

TypeFactory registers the resulting names; it doesn’t allocate a different name on collision.

ModelProvider uses Generated/Models/{Name}.cs, and CSharpGen writes directly to that path.

So namespace qualification handles A.WidgetResult versus B.WidgetResult references, but not two A.WidgetResult declarations—or duplicate output paths. We need prevention somewhere before emission; it doesn’t necessarily have to live in this naming helper.

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Jorge Rangel (jorgerangel-msft)

Copilot I'd like to see unit tests where a model name is not normalized if it is found in the last contract. Similarly, tests to validate that the isExactName flag is honored for model names

…lity Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot · 2026-09-18T20:59:25Z

Copilot I'd like to see unit tests where a model name is not normalized if it is found in the last contract. Similarly, tests to validate that the isExactName flag is honored for model names

Added dedicated IsExactName and last-contract test matrices in 2500c9a, covering suffix/acronym normalization, verbatim exact names, behavior without a prior contract, and name preservation after Reset(). Also added shipped-sibling alias tests in both input orders. All 165 targeted naming/customization tests pass, including the 16 new cases.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:16:32Z

+            return inputNamespace.Models.Any(model => HasConflictingName(model, model.Namespace, resultName)) ||
+                inputNamespace.Enums.Any(@enum => HasConflictingName(@enum, @enum.Namespace, resultName)) ||
+                inputNamespace.Clients.Any(client => HasConflictingName(client, typeNamespace, resultName))
+                    ? normalizedName
+                    : resultName;


Copilot is this check deterministic ? that is, will the outcome of this always be the same given the same set of types ?

This check is deterministic because it only asks whether any model/enum/client reserves the target Result name; the boolean outcome does not depend on traversal order. In 9d543b68, I extracted it into HasConflictingResultName to make that scope clearer.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:17:24Z

+                return false;
+            }
+
+            customType = sourceInputModel.FindForTypeInCurrentCompilation(


Copilot why do we have to call this 3 separate times ?

Addressed in 9d543b68 by consolidating the current-compilation probes behind FindCustomizationType and a single ordered lookup-name helper.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:17:43Z

+                sourceInputModel.FindForTypeInLastContract(otherNamespace, otherName) is null &&
+                (normalizedOtherName == otherName ||
+                    sourceInputModel.FindForTypeInLastContract(otherNamespace, normalizedOtherName) is null);


Copilot why do we have to call this more than once ?

Addressed in 9d543b68 by moving the last-contract probes into HasLastContractName, so the call sites no longer duplicate that lookup logic.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:19:21Z

+        private protected override TypeProvider? BuildCustomCodeView(string? generatedTypeName = null, string? generatedTypeNamespace = null)
+        {
+            var typeNamespace = generatedTypeNamespace ?? BuildNamespace();
+            var typeName = generatedTypeName ?? BuildName();
+            var customCodeView = base.BuildCustomCodeView(typeName, typeNamespace);
+            if (customCodeView is not null || _inputModel.IsExactName)
+            {
+                return customCodeView;
+            }
+
+            var originalName = _inputModel.Name.ToIdentifierName();
+            if (!originalName.EndsWith(ResponseSuffix, StringComparison.Ordinal) ||
+                originalName == typeName ||
+                typeName != NormalizeTypeName(originalName))
+            {
+                return null;
+            }
+
+            return base.BuildCustomCodeView(originalName, typeNamespace) ??
+                base.BuildCustomCodeView(originalName.NormalizeCSharpAcronyms(), typeNamespace);
+        }
+
+        private protected override TypeProvider? BuildLastContractView(string? generatedTypeName = null, string? generatedTypeNamespace = null)
+        {
+            var typeNamespace = generatedTypeNamespace ?? CustomCodeView?.Type.Namespace ?? BuildNamespace();
+            var typeName = generatedTypeName ?? CustomCodeView?.Name ?? BuildName();
+            var lastContractView = base.BuildLastContractView(typeName, typeNamespace);
+            if (lastContractView is not null || _inputModel.IsExactName)
+            {
+                return lastContractView;
+            }
+
+            var originalName = _inputModel.Name.ToIdentifierName();
+            if (!originalName.EndsWith(ResponseSuffix, StringComparison.Ordinal) ||
+                originalName == typeName ||
+                typeName != NormalizeTypeName(originalName))
+            {
+                return null;
+            }


Copilot can we consolidate some of this shared logic into a private helper ?

Consolidated in 9d543b68: custom/current lookup candidates now share FindCustomizationType/GetCustomizationLookupNames, and custom-code plus last-contract response fallbacks share BuildResponseSuffixFallbackView.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:22:10Z

+
+namespace Sample.Models
+{
+    [CodeGenType("GadgetResult")]


Copilot lets consider this other scenario for a unit test:

A model that doesn't have a previous GA surface is called FooModel and contained a custom implmentation with a [CodeGenType("FooResponse")] annotation to do the rename. Will this rename still be honored with our changes? if not, should it ?

Added this coverage in 9d543b68. A custom FooModel with [CodeGenType("FooResponse")] and no last contract is still honored; the generated model uses FooModel through CustomCodeView.

Copilot

🟡 Changes recommended

Collision checks must account for customized CLR names before preserving the Response suffix.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds deterministic C# model naming by converting eligible Response suffixes to Result while preserving compatibility and avoiding known collisions.

Changes:

Adds response-to-result normalization and compatibility lookup logic.
Expands naming, customization, collision, and enum tests.
Regenerates affected C# clients, models, serializers, and paging helpers.

File summaries

File	Description
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/TypeUnionModelFactory.cs	Updates model factory naming.
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/StringsOnly.cs	Uses the renamed result model.
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/Models/GetResult.Serialization.cs	Adds renamed model serialization.
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/Models/GetResult.cs	Renames the generated model.
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/Models/GetResponse.Serialization.cs	Removes obsolete serialization output.
packages/http-client-csharp/generator/TestProjects/Spector/http/type/union/src/Generated/Models/_TypeUnionContext.cs	Registers the renamed model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/SampleTypeSpecModelFactory.cs	Renames model factory methods.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/SampleTypeSpecClient.cs	Updates anonymous-model return types.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Notebooks.cs	Updates notebook return types.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/SampleTypeSpecContext.cs	Registers renamed sample models.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ReturnsAnonymousModelResult.Serialization.cs	Renames serialization members.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ReturnsAnonymousModelResult.cs	Renames the anonymous result model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithStringNextLinkResult.Serialization.cs	Renames string-link serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithStringNextLinkResult.cs	Renames the string-link model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithNextLinkResult.Serialization.cs	Renames next-link serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithNextLinkResult.cs	Renames the next-link model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithContinuationTokenResult.Serialization.cs	Renames token serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithContinuationTokenResult.cs	Renames the token model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithContinuationTokenHeaderResponseResult.Serialization.cs	Renames header-result serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/ListWithContinuationTokenHeaderResponseResult.cs	Renames the header-result model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/GetWidgetMetricsResult.Serialization.cs	Renames metrics serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/GetWidgetMetricsResult.cs	Renames the metrics model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/GetNotebookResult.Serialization.cs	Renames notebook serialization.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Models/GetNotebookResult.cs	Renames the notebook model.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/Metrics.cs	Updates metrics return types.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithStringNextLinkCollectionResultOfT.cs	Updates typed string-link paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithStringNextLinkCollectionResult.cs	Updates string-link paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithStringNextLinkAsyncCollectionResultOfT.cs	Updates typed async paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithStringNextLinkAsyncCollectionResult.cs	Updates async string-link casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithNextLinkCollectionResultOfT.cs	Updates typed next-link casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithNextLinkCollectionResult.cs	Updates next-link paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithNextLinkAsyncCollectionResultOfT.cs	Updates typed async next-link casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithNextLinkAsyncCollectionResult.cs	Updates async next-link casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenHeaderResponseCollectionResultOfT.cs	Updates header paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenHeaderResponseAsyncCollectionResultOfT.cs	Updates async header paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenCollectionResultOfT.cs	Updates typed token paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenCollectionResult.cs	Updates token paging casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenAsyncCollectionResultOfT.cs	Updates typed async token casts.
packages/http-client-csharp/generator/TestProjects/Local/Sample-TypeSpec/src/Generated/CollectionResults/SampleTypeSpecClientGetWithContinuationTokenAsyncCollectionResult.cs	Updates async token paging casts.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixPreservesExistingName/ExistingModels.cs	Supplies existing-contract fixtures.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixPreservesCustomName/CustomizedModels.cs	Supplies customization fixtures.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixIgnoresResultAliasForShippedModel/WidgetResult.cs	Tests result aliases with shipped models.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixIgnoresResultAliasForShippedModel(LastContract)/IPResponse.cs	Supplies the prior-contract fixture.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixAvoidsResultCustomizationAliasCollision/WidgetResult.cs	Tests result-alias collisions.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/TestData/ModelProviderTests/TestBuildName_ResponseSuffixAvoidsCustomizationCollision/WidgetResult.cs	Tests customization collisions.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/ModelProviders/ModelProviderTests.cs	Adds deterministic naming tests.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/Providers/EnumProviders/EnumProviderTests.cs	Confirms enums remain unaffected.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/TypeProvider.cs	Makes type normalization overridable.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/ModelProvider.cs	Implements response-to-result naming.
packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator.ClientModel/test/ReferenceMap/ClientBodyDependencyReferenceMapTests.cs	Updates reference-map expectations.

Review details

Suppressed comments (2)

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/ModelProvider.cs:429

The response-suffix compatibility lookup is duplicated here and again in BuildLastContractView (lines 432-451), including the same candidate-name and fallback sequence. Keeping two copies means a future fix to customization matching can silently diverge from last-contract matching; extract the shared candidate-resolution logic and pass the view-specific lookup into it.

            var originalName = _inputModel.Name.ToIdentifierName();
            if (!originalName.EndsWith(ResponseSuffix, StringComparison.Ordinal) ||
                originalName == typeName ||
                typeName != NormalizeTypeName(originalName))
            {
                return null;
            }

            return base.BuildCustomCodeView(originalName, typeNamespace) ??
                base.BuildCustomCodeView(originalName.NormalizeCSharpAcronyms(), typeNamespace);

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/ModelProvider.cs:344

NormalizeTypeName runs this scan for every Response model, and each sibling can trigger up to three Roslyn compilation lookups below, making provider construction quadratic in the number of input types for large libraries. Cache the current/last-contract customization names (or build a single lookup set) once per generator run instead of querying the compilations inside the nested scan.

            return inputNamespace.Models.Any(model => HasConflictingName(model, model.Namespace, resultName)) ||
                inputNamespace.Enums.Any(@enum => HasConflictingName(@enum, @enum.Namespace, resultName)) ||
                inputNamespace.Clients.Any(client => HasConflictingName(client, typeNamespace, resultName))
                    ? normalizedName
                    : resultName;

Files reviewed: 11/50 changed files
Comments generated: 1
Review effort level: Balanced

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Jorge Rangel (jorgerangel-msft) · 2026-09-18T21:30:55Z

+            var otherName = inputType.IsExactName ? inputType.Name : inputType.Name.ToIdentifierName();
+            // Acronym normalization only changes casing, so this also covers the normalized filename.
+            if (string.Equals(otherName, resultName, StringComparison.OrdinalIgnoreCase))
+            {
+                return true;
+            }
+
+            var sourceInputModel = CodeModelGenerator.Instance.SourceInputModel;
+            if (sourceInputModel.Customization is null)
+            {
+                return false;
+            }
+
+            var otherNamespace = string.IsNullOrEmpty(inputTypeNamespace)
+                ? CodeModelGenerator.Instance.TypeFactory.PrimaryNamespace
+                : CodeModelGenerator.Instance.TypeFactory.GetCleanNameSpace(inputTypeNamespace);
+            var customType = sourceInputModel.FindForTypeInCurrentCompilation(otherNamespace, otherName);
+            if (customType is not null || inputType.IsExactName)
+            {
+                return string.Equals(customType?.Name, resultName, StringComparison.OrdinalIgnoreCase);
+            }


Copilot I believe this is the question I asked earlier. I agree with the assessment here

Agreed and fixed in 65849681. The model collision check now resolves a sibling customization first and only falls back to raw input-name collision when no customization is found. I also added a regression test where WidgetResponse normalizes to WidgetResult while the sibling WidgetResult is customized to CustomWidgetResult.

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Initial plan

62f2a61

Copilot AI assigned Copilot and Jorge Rangel (jorgerangel-msft) Sep 17, 2026

Copilot AI linked an issue Sep 17, 2026 that may be closed by this pull request

Support Deterministic Naming for model suffix Response to Result #11993

Open

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 17, 2026 17:34 View session

Copilot stopped work on behalf of Jorge Rangel (jorgerangel-msft) due to an error September 17, 2026 17:35
Execution failed: Error: Reasoning effort 'max' is not supported for model 'gpt-6-astra'.

Copilot AI requested a review from Jorge Rangel (jorgerangel-msft) September 17, 2026 17:35

Jorge Rangel (jorgerangel-msft) assigned Copilot and unassigned Copilot Sep 18, 2026

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 19:31 View session

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 19:32

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 19:33 View session

test(csharp): cover deterministic Response-to-Result model names

d37edba

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

microsoft-github-policy-service Bot added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Sep 18, 2026

Copilot AI and others added 2 commits September 18, 2026 19:53

feat(csharp): normalize Response model suffixes to Result

4af1e99

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

fix(csharp): account for Result-targeted customization aliases

d9ed5f8

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:04

Jorge Rangel (jorgerangel-msft) reviewed Sep 18, 2026

View reviewed changes

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:06 View session

refactor(csharp): scope response name fallbacks to ModelProvider

8c8f5d2

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:23

Copilot AI requested a review from Jorge Rangel (jorgerangel-msft) September 18, 2026 20:23

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:24 View session

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:26

Jorge Rangel (jorgerangel-msft) reviewed Sep 18, 2026

View reviewed changes

Comment thread ...ges/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/ModelProvider.cs Outdated

Comment thread ...ages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/src/Providers/TypeProvider.cs Outdated

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:28 View session

refactor(csharp): keep response suffix normalization in ModelProvider

dbba17b

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:45

Copilot AI requested a review from Jorge Rangel (jorgerangel-msft) September 18, 2026 20:45

Jorge Rangel (jorgerangel-msft) reviewed Sep 18, 2026

View reviewed changes

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 20:50 View session

refactor(csharp): reduce naming collision lookups and cover compatibi…

2500c9a

…lity Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:05

Copilot AI requested a review from Jorge Rangel (jorgerangel-msft) September 18, 2026 21:05

Jorge Rangel (jorgerangel-msft) requested a balanced review from Copilot September 18, 2026 21:15

Copilot started reviewing on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:16 View session

Jorge Rangel (jorgerangel-msft) reviewed Sep 18, 2026

View reviewed changes

Copilot AI reviewed Sep 18, 2026

View reviewed changes

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:28 View session

refactor(csharp): consolidate response naming lookup helpers

9d543b6

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:39

Copilot AI requested a review from Jorge Rangel (jorgerangel-msft) September 18, 2026 21:39

Copilot started work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:40 View session

fix(csharp): honor customized result model names

6584968

Co-authored-by: jorgerangel-msft <102122018+jorgerangel-msft@users.noreply.github.com>

Copilot finished work on behalf of Jorge Rangel (jorgerangel-msft) September 18, 2026 21:47

Conversation

Copilot AI commented Sep 17, 2026

Comments on the Issue (you are Copilot in this section)

Uh oh!

azure-pipelines Bot commented Sep 17, 2026

Uh oh!

pkg-pr-new Bot commented Sep 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Jorge Rangel (jorgerangel-msft) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Sep 18, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Suppressed comments (2)

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

pkg-pr-new Bot commented Sep 18, 2026 •

edited

Loading