[http-client-csharp] Remove unused InputOperation.ResourceName (#11973)

`InputOperation.ResourceName` is a legacy AutoRest field that no longer
determines client names and has no consumers.

- **Input model:** Remove the property, constructor and `Update`
parameters, and deserialization handling.
- **Emitter:** Stop emitting `resourceName` and remove the helper used
solely to compute it.
- **Compatibility:** Older code models containing `resourceName` remain
readable; regression coverage verifies this and the field’s absence from
emitted operations.

<!-- START COPILOT CODING AGENT SUFFIX -->

- Fixes #8855

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
Co-authored-by: jolov <jolov@microsoft.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cc0b1381-ff7e-4957-a7cb-273c935c22d1

Author: 3 people

packages/http-client-csharp/emitter/src/lib/operation-converter.ts

@@ -4,7 +4,6 @@
 import type {
   SdkBodyParameter,
   SdkBuiltInKinds,
-  SdkContext,
   SdkHeaderParameter,
   SdkHttpOperation,
   SdkHttpParameter,
@@ -41,7 +40,6 @@ import {
 } from "@typespec/compiler";
 import { unsafe_getEventDefinitions } from "@typespec/events/experimental";
 import type { HttpStatusCodeRange } from "@typespec/http";
-import { getResourceOperation } from "@typespec/rest";
 import { isTerminalEvent } from "@typespec/sse";
 import type { CSharpEmitterContext } from "../sdk-context.js";
 import { collectionFormatToDelimMap } from "../type/collection-format.js";
@@ -76,7 +74,7 @@ import type { OperationResponse } from "../type/operation-response.js";
 import { RequestLocation } from "../type/request-location.js";
 import { parseHttpRequestMethod } from "../type/request-method.js";
 import { ResponseLocation } from "../type/response-location.js";
-import { getExternalDocs, getOperationId } from "./decorators.js";
+import { getExternalDocs } from "./decorators.js";
 import { fromSdkHttpExamples } from "./example-converter.js";
 import { getExperimentalDetails } from "./experimental.js";
 import { createDiagnostic } from "./lib.js";
@@ -221,10 +219,6 @@ export function fromSdkServiceMethodOperation(
   operation = {
     name: method.name,
     isExactName: method.isExactName,
-    resourceName:
-      getResourceOperation(sdkContext.program, method.operation.__raw.operation)?.resourceType
-        .name ??
-      getOperationGroupName(sdkContext, method.operation, getClientNamespaceString(sdkContext)!),
     deprecated: getDeprecated(sdkContext.program, method.__raw!),
     summary: method.summary,
     doc: method.doc,
@@ -1109,28 +1103,6 @@ function getParameterScope(
         : InputParameterScope.Method;
 }
 
-function getOperationGroupName(
-  context: SdkContext,
-  operation: SdkHttpOperation,
-  namespace: string,
-): string {
-  const explicitOperationId = getOperationId(context, operation.__raw.operation);
-  if (explicitOperationId) {
-    const ids: string[] = explicitOperationId.split("_");
-    if (ids.length > 1) {
-      return ids.slice(0, -2).join("_");
-    }
-  }
-
-  if (operation.__raw.operation.interface) {
-    return operation.__raw.operation.interface.name;
-  }
-  if (operation.__raw.operation.namespace) {
-    return operation.__raw.operation.namespace.name;
-  }
-  return namespace;
-}
-
 // TODO: remove after https://github.com/Azure/typespec-azure/issues/1227 is fixed
 function normalizeHeaderName(name: string): string {
   switch (name.toLocaleLowerCase()) {

packages/http-client-csharp/emitter/src/type/input-operation.ts

@@ -10,7 +10,6 @@ import type { RequestMethod } from "./request-method.js";
 export interface InputOperation {
   name: string;
   isExactName?: boolean;
-  resourceName?: string;
   summary?: string;
   deprecated?: string;
   doc?: string;

packages/http-client-csharp/emitter/test/Unit/operation-converter.test.ts

@@ -1,6 +1,7 @@
 import type { TestHost } from "@typespec/compiler/testing";
 import { ok, strictEqual } from "assert";
 import { beforeEach, describe, it } from "vitest";
+import { serializeCodeModel } from "../../src/code-model-writer.js";
 import { createModel } from "../../src/lib/client-model-builder.js";
 import type { InputEnumType } from "../../src/type/input-type.js";
 import { RequestLocation } from "../../src/type/request-location.js";
@@ -18,6 +19,57 @@ describe("Operation Converter", () => {
     runner = await createEmitterTestHost();
   });
 
+  it.each([
+    {
+      scope: "namespace",
+      code: `@get @route("/widgets") op getWidget(): string;`,
+      path: "/widgets",
+    },
+    {
+      scope: "interface",
+      code: `interface Widgets {
+        @get @route("/widgets") getWidget(): string;
+      }`,
+      path: "/widgets",
+    },
+    {
+      scope: "REST resource",
+      code: `
+        @resource("widgets")
+        model Widget {
+          @key id: string;
+        }
+        @readsResource(Widget)
+        @get @route("/widgets/{id}")
+        op getWidget(@path id: string): Widget;
+      `,
+      path: "/widgets/{id}",
+    },
+  ])(
+    "omits resourceName for $scope operations and serialized code models",
+    async ({ code, path }) => {
+      const program = await typeSpecCompile(code, runner);
+      const context = createEmitterContext(program);
+      const sdkContext = await createCSharpSdkContext(context);
+      const [root] = createModel(sdkContext);
+      strictEqual(root.clients.length, 1);
+      const client = root.clients[0];
+      const methods = [
+        ...client.methods,
+        ...(client.children ?? []).flatMap((child) => child.methods),
+      ];
+
+      strictEqual(methods.length, 1);
+      const operation = methods[0].operation;
+      ok(operation);
+      strictEqual(operation.name, "getWidget");
+      strictEqual(operation.httpMethod, "GET");
+      strictEqual(operation.path, path);
+      strictEqual("resourceName" in operation, false);
+      strictEqual(serializeCodeModel(sdkContext, root).includes('"resourceName"'), false);
+    },
+  );
+
   describe("Operation with embedded non-body parameters in request body", () => {
     describe("With header in request model", () => {
       it("Header parameter in model should not be present in service method", async () => {
@@ -62,6 +114,7 @@ describe("Operation Converter", () => {
         // validate operation
         const operation = root.clients[0].methods[0].operation;
         ok(operation);
+        strictEqual("resourceName" in operation, false);
         strictEqual(operation.parameters.length, 4);
 
         // content type parameter

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator.Input/src/InputTypes/InputOperation.cs

@@ -13,7 +13,6 @@ public class InputOperation
     {
         public InputOperation(
             string name,
-            string? resourceName,
             string? summary,
             string? doc,
             string? deprecated,
@@ -32,7 +31,6 @@ public InputOperation(
             string? ns)
         {
             Name = name;
-            ResourceName = resourceName;
             Summary = summary;
             Doc = doc;
             Deprecated = deprecated;
@@ -53,7 +51,6 @@ public InputOperation(
 
         public InputOperation() : this(
             name: string.Empty,
-            resourceName: null,
             summary: string.Empty,
             doc: string.Empty,
             deprecated: null,
@@ -83,7 +80,6 @@ public InputOperation() : this(
         /// Gets the original name of the operation as defined in the TypeSpec before any mutations.
         /// </summary>
         public string? OriginalName { get; internal set; }
-        public string? ResourceName { get; internal set; }
         public string? Summary { get; internal set; }
         public string? Doc { get; internal set; }
         public string? Deprecated { get; internal set; }
@@ -109,7 +105,6 @@ public InputOperation() : this(
 
         public void Update(
             string? name = null,
-            string? resourceName = null,
             string? summary = null,
             string? doc = null,
             string? deprecated = null,
@@ -132,10 +127,6 @@ public void Update(
             {
                 Name = name;
             }
-            if (resourceName != null)
-            {
-                ResourceName = resourceName;
-            }
             if (summary != null)
             {
                 Summary = summary;

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator.Input/src/InputTypes/Serialization/InputOperationConverter.cs

@@ -37,7 +37,6 @@ public override void Write(Utf8JsonWriter writer, InputOperation value, JsonSeri
             resolver.AddReference(id, operation);
 
             string? name = null;
-            string? resourceName = null;
             string? summary = null;
             string? doc = null;
             string? deprecated = null;
@@ -63,7 +62,6 @@ public override void Write(Utf8JsonWriter writer, InputOperation value, JsonSeri
             {
                 var isKnownProperty = reader.TryReadString("name", ref name)
                     || reader.TryReadBoolean("isExactName", ref isExactName)
-                    || reader.TryReadString("resourceName", ref resourceName)
                     || reader.TryReadString("summary", ref summary)
                     || reader.TryReadString("doc", ref doc)
                     || reader.TryReadString("deprecated", ref deprecated)
@@ -93,7 +91,6 @@ public override void Write(Utf8JsonWriter writer, InputOperation value, JsonSeri
             operation.Name = name ?? throw new JsonException("InputOperation must have name");
             operation.IsExactName = isExactName;
             operation.OriginalName = name;
-            operation.ResourceName = resourceName;
             operation.Summary = summary;
             operation.Doc = doc;
             operation.Deprecated = deprecated;

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator.Input/test/TypeSpecInputConverterTests.cs

@@ -10,6 +10,39 @@ namespace Microsoft.TypeSpec.Generator.Input.Tests
 {
     public class TypeSpecInputConverterTests
     {
+        [TestCase(false)]
+        [TestCase(true)]
+        public void LoadsOperationWithoutResourceName(bool includeLegacyResourceName)
+        {
+            var json = $$"""
+                {
+                  "$id": "1",
+                  "name": "getWidget",
+                  {{(includeLegacyResourceName ? "\"resourceName\": \"Widget\"," : "")}}
+                  "summary": "Gets a widget",
+                  "httpMethod": "GET",
+                  "uri": "https://example.com",
+                  "path": "/widgets",
+                  "crossLanguageDefinitionId": "Test.getWidget"
+                }
+                """;
+            var options = new JsonSerializerOptions
+            {
+                Converters = { new InputOperationConverter(new TypeSpecReferenceHandler()) }
+            };
+
+            var operation = JsonSerializer.Deserialize<InputOperation>(json, options);
+
+            Assert.IsNotNull(operation);
+            Assert.AreEqual("getWidget", operation!.Name);
+            Assert.AreEqual("Gets a widget", operation.Summary);
+            Assert.AreEqual("GET", operation.HttpMethod);
+            Assert.AreEqual("https://example.com", operation.Uri);
+            Assert.AreEqual("/widgets", operation.Path);
+            Assert.AreEqual("Test.getWidget", operation.CrossLanguageDefinitionId);
+            Assert.IsNull(typeof(InputOperation).GetProperty("ResourceName"));
+        }
+
         [TestCase("model", "\"properties\": []")]
         [TestCase("enum", "\"values\": [], \"valueType\": {\"kind\":\"string\"}")]
         [TestCase("string", "\"crossLanguageDefinitionId\": \"External.Value\"")]

packages/http-client-csharp/generator/Microsoft.TypeSpec.Generator/test/common/InputFactory.cs

@@ -762,7 +762,6 @@ public static InputOperation Operation(
         {
             var operation = new InputOperation(
                 name,
-                null,
                 "",
                 $"{name} description",
                 null,