feat(http-client-csharp): add InputOperation.OperationId

- add `InputOperation.OperationId` - invoke `resolveOperationId()` to initialize it part of Azure/autorest.csharp#5117
microsoft · Jan 3, 2025 · 45c7044 · 45c7044
1 parent 254aba7
commit 45c7044
Show file tree

Hide file tree

Showing 54 changed files with 664 additions and 0 deletions.
diff --git a/packages/http-client-csharp/emitter/src/lib/operation-converter.ts b/packages/http-client-csharp/emitter/src/lib/operation-converter.ts
@@ -41,6 +41,7 @@ import { getExternalDocs, getOperationId } from "./decorators.js";
 import { fromSdkHttpExamples } from "./example-converter.js";
 import { Logger } from "./logger.js";
 import { isSdkPathParameter } from "./utils.js";
+import { resolveOperationId } from "@typespec/openapi";
 
 export function fromSdkServiceMethod(
   method: SdkServiceMethod<SdkHttpOperation>,
@@ -74,6 +75,7 @@ export function fromSdkServiceMethod(
       getResourceOperation(sdkContext.program, method.operation.__raw.operation)?.resourceType
         .name ??
       getOperationGroupName(sdkContext, method.operation, sdkContext.sdkPackage.rootNamespace),
+    OperationId: resolveOperationId(sdkContext.program, method.operation.__raw.operation),
     Deprecated: getDeprecated(sdkContext.program, method.__raw!),
     Summary: method.summary,
     Doc: method.doc,

diff --git a/packages/http-client-csharp/emitter/src/type/input-operation.ts b/packages/http-client-csharp/emitter/src/type/input-operation.ts
@@ -19,6 +19,7 @@ export interface Paging {
 export interface InputOperation {
   Name: string;
   ResourceName?: string;
+  OperationId: string;
   Summary?: string;
   Deprecated?: string;
   Doc?: string;

diff --git a/...ent-csharp/generator/TestProjects/CadlRanch/http/authentication/api-key/tspCodeModel.json b/...ent-csharp/generator/TestProjects/CadlRanch/http/authentication/api-key/tspCodeModel.json
@@ -44,6 +44,7 @@
      "$id": "6",
      "Name": "valid",
      "ResourceName": "ApiKey",
+     "OperationId": "valid",
      "Doc": "Check whether client is authenticated",
      "Accessibility": "public",
      "Parameters": [],
@@ -72,6 +73,7 @@
      "$id": "8",
      "Name": "invalid",
      "ResourceName": "ApiKey",
+     "OperationId": "invalid",
      "Doc": "Check whether client is authenticated.",
      "Accessibility": "public",
      "Parameters": [

diff --git a/...csharp/generator/TestProjects/CadlRanch/http/authentication/http/custom/tspCodeModel.json b/...csharp/generator/TestProjects/CadlRanch/http/authentication/http/custom/tspCodeModel.json
@@ -44,6 +44,7 @@
      "$id": "6",
      "Name": "valid",
      "ResourceName": "Custom",
+     "OperationId": "valid",
      "Doc": "Check whether client is authenticated",
      "Accessibility": "public",
      "Parameters": [],
@@ -72,6 +73,7 @@
      "$id": "8",
      "Name": "invalid",
      "ResourceName": "Custom",
+     "OperationId": "invalid",
      "Doc": "Check whether client is authenticated.",
      "Accessibility": "public",
      "Parameters": [

diff --git a/...ient-csharp/generator/TestProjects/CadlRanch/http/authentication/oauth2/tspCodeModel.json b/...ient-csharp/generator/TestProjects/CadlRanch/http/authentication/oauth2/tspCodeModel.json
@@ -44,6 +44,7 @@
      "$id": "6",
      "Name": "valid",
      "ResourceName": "OAuth2",
+     "OperationId": "valid",
      "Doc": "Check whether client is authenticated",
      "Accessibility": "public",
      "Parameters": [],
@@ -72,6 +73,7 @@
      "$id": "8",
      "Name": "invalid",
      "ResourceName": "OAuth2",
+     "OperationId": "invalid",
      "Doc": "Check whether client is authenticated. Will return an invalid bearer error.",
      "Accessibility": "public",
      "Parameters": [

diff --git a/...lient-csharp/generator/TestProjects/CadlRanch/http/authentication/union/tspCodeModel.json b/...lient-csharp/generator/TestProjects/CadlRanch/http/authentication/union/tspCodeModel.json
@@ -14,6 +14,7 @@
      "$id": "3",
      "Name": "validKey",
      "ResourceName": "Union",
+     "OperationId": "validKey",
      "Doc": "Check whether client is authenticated",
      "Accessibility": "public",
      "Parameters": [],
@@ -42,6 +43,7 @@
      "$id": "5",
      "Name": "validToken",
      "ResourceName": "Union",
+     "OperationId": "validToken",
      "Doc": "Check whether client is authenticated",
      "Accessibility": "public",
      "Parameters": [],

diff --git a/...s/http-client-csharp/generator/TestProjects/CadlRanch/http/encode/bytes/tspCodeModel.json b/...s/http-client-csharp/generator/TestProjects/CadlRanch/http/encode/bytes/tspCodeModel.json
@@ -192,6 +192,7 @@
      "$id": "23",
      "Name": "default",
      "ResourceName": "Query",
+     "OperationId": "Query_default",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -242,6 +243,7 @@
      "$id": "27",
      "Name": "base64",
      "ResourceName": "Query",
+     "OperationId": "Query_base64",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -292,6 +294,7 @@
      "$id": "31",
      "Name": "base64url",
      "ResourceName": "Query",
+     "OperationId": "Query_base64url",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -342,6 +345,7 @@
      "$id": "35",
      "Name": "base64urlArray",
      "ResourceName": "Query",
+     "OperationId": "Query_base64urlArray",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -452,6 +456,7 @@
      "$id": "47",
      "Name": "default",
      "ResourceName": "Property",
+     "OperationId": "Property_default",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -561,6 +566,7 @@
      "$id": "56",
      "Name": "base64",
      "ResourceName": "Property",
+     "OperationId": "Property_base64",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -670,6 +676,7 @@
      "$id": "65",
      "Name": "base64url",
      "ResourceName": "Property",
+     "OperationId": "Property_base64url",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -779,6 +786,7 @@
      "$id": "74",
      "Name": "base64urlArray",
      "ResourceName": "Property",
+     "OperationId": "Property_base64urlArray",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -932,6 +940,7 @@
      "$id": "89",
      "Name": "default",
      "ResourceName": "Header",
+     "OperationId": "Header_default",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -982,6 +991,7 @@
      "$id": "93",
      "Name": "base64",
      "ResourceName": "Header",
+     "OperationId": "Header_base64",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1032,6 +1042,7 @@
      "$id": "97",
      "Name": "base64url",
      "ResourceName": "Header",
+     "OperationId": "Header_base64url",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1082,6 +1093,7 @@
      "$id": "101",
      "Name": "base64urlArray",
      "ResourceName": "Header",
+     "OperationId": "Header_base64urlArray",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1192,6 +1204,7 @@
      "$id": "113",
      "Name": "default",
      "ResourceName": "RequestBody",
+     "OperationId": "RequestBody_default",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1273,6 +1286,7 @@
      "$id": "120",
      "Name": "octetStream",
      "ResourceName": "RequestBody",
+     "OperationId": "RequestBody_octetStream",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1352,6 +1366,7 @@
      "$id": "127",
      "Name": "customContentType",
      "ResourceName": "RequestBody",
+     "OperationId": "RequestBody_customContentType",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1431,6 +1446,7 @@
      "$id": "134",
      "Name": "base64",
      "ResourceName": "RequestBody",
+     "OperationId": "RequestBody_base64",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1512,6 +1528,7 @@
      "$id": "141",
      "Name": "base64url",
      "ResourceName": "RequestBody",
+     "OperationId": "RequestBody_base64url",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1637,6 +1654,7 @@
      "$id": "154",
      "Name": "default",
      "ResourceName": "ResponseBody",
+     "OperationId": "ResponseBody_default",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1703,6 +1721,7 @@
      "$id": "160",
      "Name": "octetStream",
      "ResourceName": "ResponseBody",
+     "OperationId": "ResponseBody_octetStream",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1787,6 +1806,7 @@
      "$id": "169",
      "Name": "customContentType",
      "ResourceName": "ResponseBody",
+     "OperationId": "ResponseBody_customContentType",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1871,6 +1891,7 @@
      "$id": "178",
      "Name": "base64",
      "ResourceName": "ResponseBody",
+     "OperationId": "ResponseBody_base64",
      "Accessibility": "public",
      "Parameters": [
       {
@@ -1937,6 +1958,7 @@
      "$id": "184",
      "Name": "base64url",
      "ResourceName": "ResponseBody",
+     "OperationId": "ResponseBody_base64url",
      "Accessibility": "public",
      "Parameters": [
       {