add a cache for parameters/responses/operations

packages/http-client-csharp/emitter/src/emitter.ts

@@ -26,7 +26,7 @@ import { Logger } from "./lib/logger.js";
 import { execAsync } from "./lib/utils.js";
 import { _resolveOutputFolder, CSharpEmitterOptions, resolveOptions } from "./options.js";
 import { defaultSDKContextOptions } from "./sdk-context-options.js";
-import { CSharpEmitterContext } from "./sdk-context.js";
+import { createSdkTypeCache, CSharpEmitterContext } from "./sdk-context.js";
 import { CodeModel } from "./type/code-model.js";
 import { Configuration } from "./type/configuration.js";
 
@@ -72,11 +72,7 @@ export async function $onEmit(context: EmitContext<CSharpEmitterOptions>) {
         defaultSDKContextOptions,
       )),
       logger: logger,
-      __typeCache: {
-        types: new Map(),
-        models: new Map(),
-        enums: new Map(),
-      },
+      __typeCache: createSdkTypeCache(),
     };
     if (
       context.program.diagnostics.length > 0 &&

packages/http-client-csharp/emitter/src/lib/client-model-builder.ts

@@ -184,7 +184,6 @@ export function createModel(sdkContext: CSharpEmitterContext): CodeModel {
         Type: parameterType,
         Location: RequestLocation.Uri,
         IsApiVersion: parameter.isApiVersionParam,
-        IsResourceParameter: false,
         IsContentType: false,
         IsRequired: !parameter.optional,
         IsEndpoint: isEndpoint,

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

@@ -7,9 +7,7 @@ import {
   SdkDictionaryExampleValue,
   SdkExampleValue,
   SdkHttpOperationExample,
-  SdkHttpParameter,
   SdkHttpParameterExampleValue,
-  SdkHttpResponse,
   SdkHttpResponseExampleValue,
   SdkModelExampleValue,
   SdkNullExampleValue,
@@ -34,7 +32,6 @@ import {
   InputUnknownExampleValue,
   OperationResponseExample,
 } from "../type/input-examples.js";
-import { InputParameter } from "../type/input-parameter.js";
 import {
   InputArrayType,
   InputDictionaryType,
@@ -43,14 +40,12 @@ import {
   InputPrimitiveType,
   InputUnionType,
 } from "../type/input-type.js";
-import { OperationResponse } from "../type/operation-response.js";
+import { fromSdkHttpOperationResponse } from "./operation-converter.js";
 import { fromSdkType } from "./type-converter.js";
 
 export function fromSdkHttpExamples(
   sdkContext: CSharpEmitterContext,
   examples: SdkHttpOperationExample[],
-  parameterMap: Map<SdkHttpParameter, InputParameter>,
-  responseMap: Map<SdkHttpResponse, OperationResponse>,
 ): InputHttpOperationExample[] {
   return examples.map((example) => fromSdkHttpExample(example));
 
@@ -61,36 +56,26 @@ export function fromSdkHttpExamples(
       description: example.description,
       filePath: example.filePath,
       parameters: example.parameters.map((p) => fromSdkParameterExample(p)),
-      responses: fromSdkOperationResponses(example.responses),
+      responses: example.responses.map((r) => fromSdkOperationResponse(r)),
     };
   }
 
   function fromSdkParameterExample(
     parameter: SdkHttpParameterExampleValue,
   ): InputParameterExampleValue {
     return {
-      parameter: parameterMap.get(parameter.parameter)!,
+      parameter: sdkContext.__typeCache.parameters.get(parameter.parameter)!,
       value: fromSdkExample(parameter.value),
     };
   }
 
-  function fromSdkOperationResponses(
-    responses: SdkHttpResponseExampleValue[],
-  ): OperationResponseExample[] {
-    const result: OperationResponseExample[] = [];
-    for (const response of responses) {
-      result.push(fromSdkOperationResponse(response));
-    }
-    return result;
-  }
-
   function fromSdkOperationResponse(
-    response: SdkHttpResponseExampleValue,
+    responseValue: SdkHttpResponseExampleValue,
   ): OperationResponseExample {
     return {
-      response: responseMap.get(response.response)!,
-      statusCode: response.statusCode,
-      bodyValue: response.bodyValue ? fromSdkExample(response.bodyValue) : undefined,
+      response: fromSdkHttpOperationResponse(sdkContext, responseValue.response),
+      statusCode: responseValue.statusCode,
+      bodyValue: responseValue.bodyValue ? fromSdkExample(responseValue.bodyValue) : undefined,
     };
   }
 

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

@@ -57,8 +57,6 @@ export function fromSdkServiceMethod(
     generateConvenience = false;
   }
 
-  const parameterMap = fromSdkOperationParameters(sdkContext, method.operation, rootApiVersions);
-  const responseMap = fromSdkHttpOperationResponses(sdkContext, method.operation.responses);
   return {
     Name: method.name,
     ResourceName:
@@ -69,8 +67,8 @@ export function fromSdkServiceMethod(
     Summary: method.summary,
     Doc: method.doc,
     Accessibility: method.access,
-    Parameters: [...parameterMap.values()],
-    Responses: [...responseMap.values()],
+    Parameters: fromSdkOperationParameters(sdkContext, method.operation, rootApiVersions),
+    Responses: fromSdkHttpOperationResponses(sdkContext, method.operation.responses),
     HttpMethod: parseHttpRequestMethod(method.operation.verb),
     RequestBodyMediaType: getBodyMediaType(method.operation.bodyParam?.type),
     Uri: uri,
@@ -85,7 +83,7 @@ export function fromSdkServiceMethod(
     CrossLanguageDefinitionId: method.crossLanguageDefinitionId,
     Decorators: method.decorators,
     Examples: method.operation.examples
-      ? fromSdkHttpExamples(sdkContext, method.operation.examples, parameterMap, responseMap)
+      ? fromSdkHttpExamples(sdkContext, method.operation.examples)
       : undefined,
   };
 }
@@ -138,8 +136,8 @@ function fromSdkOperationParameters(
   sdkContext: CSharpEmitterContext,
   operation: SdkHttpOperation,
   rootApiVersions: string[],
-): Map<SdkHttpParameter, InputParameter> {
-  const parameters = new Map<SdkHttpParameter, InputParameter>();
+): InputParameter[] {
+  const parameters: InputParameter[] = [];
   for (const p of operation.parameters) {
     if (p.kind === "cookie") {
       sdkContext.logger.reportDiagnostic({
@@ -150,7 +148,7 @@ function fromSdkOperationParameters(
       return parameters;
     }
     const param = fromSdkHttpOperationParameter(sdkContext, p, rootApiVersions);
-    parameters.set(p, param);
+    parameters.push(param);
   }
 
   if (operation.bodyParam) {
@@ -159,36 +157,41 @@ function fromSdkOperationParameters(
       operation.bodyParam,
       rootApiVersions,
     );
-    parameters.set(operation.bodyParam, bodyParam);
+    parameters.push(bodyParam);
   }
   return parameters;
 }
 
-function fromSdkHttpOperationParameter(
+export function fromSdkHttpOperationParameter(
   sdkContext: CSharpEmitterContext,
   p: SdkPathParameter | SdkQueryParameter | SdkHeaderParameter | SdkBodyParameter,
   rootApiVersions: string[],
 ): InputParameter {
+  let retVar = sdkContext.__typeCache.parameters.get(p);
+  if (retVar) {
+    return retVar;
+  }
+
   const isContentType =
     p.kind === "header" && p.serializedName.toLocaleLowerCase() === "content-type";
   const parameterType = fromSdkType(sdkContext, p.type);
   const format = p.kind === "header" || p.kind === "query" ? p.collectionFormat : undefined;
   const serializedName = p.kind !== "body" ? p.serializedName : p.name;
 
-  // TO-DO: In addition to checking if a path parameter is exploded, we should consider capturing the delimiter for
+  // TODO: In addition to checking if a path parameter is exploded, we should consider capturing the delimiter for
   // any path expansion to ensure the parameter values are delimited correctly during serialization.
   // https://github.com/microsoft/typespec/issues/5561
   const explode = isExplodedParameter(p);
 
-  return {
+  retVar = {
     Name: p.name,
     NameInRequest: p.kind === "header" ? normalizeHeaderName(serializedName) : serializedName,
     Summary: p.summary,
     Doc: p.doc,
     Type: parameterType,
     Location: getParameterLocation(p),
     IsApiVersion:
-      p.name.toLocaleLowerCase() === "apiversion" || p.name.toLocaleLowerCase() === "api-version",
+      p.name.toLocaleLowerCase() === "apiversion" || p.name.toLocaleLowerCase() === "api-version", // TODO -- we should use `isApiVersionParam` instead
     IsContentType: isContentType,
     IsEndpoint: false,
     Explode: explode,
@@ -198,7 +201,10 @@ function fromSdkHttpOperationParameter(
     DefaultValue: getParameterDefaultValue(sdkContext, p.clientDefaultValue, parameterType),
     Decorators: p.decorators,
     SkipUrlEncoding: p.kind === "path" ? p.allowReserved : false,
-  } as InputParameter;
+  };
+
+  sdkContext.__typeCache.updateSdkParameterReferences(p, retVar);
+  return retVar;
 }
 
 function loadLongRunningOperation(
@@ -227,22 +233,38 @@ function loadLongRunningOperation(
 function fromSdkHttpOperationResponses(
   sdkContext: CSharpEmitterContext,
   operationResponses: SdkHttpResponse[],
-): Map<SdkHttpResponse, OperationResponse> {
-  const responses = new Map<SdkHttpResponse, OperationResponse>();
+): OperationResponse[] {
+  const responses: OperationResponse[] = [];
   for (const r of operationResponses) {
-    const range = r.statusCodes;
-    responses.set(r, {
-      StatusCodes: toStatusCodesArray(range),
-      BodyType: r.type ? fromSdkType(sdkContext, r.type) : undefined,
-      BodyMediaType: BodyMediaType.Json,
-      Headers: fromSdkServiceResponseHeaders(sdkContext, r.headers),
-      IsErrorResponse: r.type !== undefined && isErrorModel(sdkContext.program, r.type.__raw!),
-      ContentTypes: r.contentTypes,
-    });
+    responses.push(fromSdkHttpOperationResponse(sdkContext, r));
   }
   return responses;
 }
 
+export function fromSdkHttpOperationResponse(
+  sdkContext: CSharpEmitterContext,
+  sdkResponse: SdkHttpResponse,
+): OperationResponse {
+  let retVar = sdkContext.__typeCache.responses.get(sdkResponse);
+  if (retVar) {
+    return retVar;
+  }
+
+  const range = sdkResponse.statusCodes;
+  retVar = {
+    StatusCodes: toStatusCodesArray(range),
+    BodyType: sdkResponse.type ? fromSdkType(sdkContext, sdkResponse.type) : undefined,
+    BodyMediaType: BodyMediaType.Json,
+    Headers: fromSdkServiceResponseHeaders(sdkContext, sdkResponse.headers),
+    IsErrorResponse:
+      sdkResponse.type !== undefined && isErrorModel(sdkContext.program, sdkResponse.type.__raw!),
+    ContentTypes: sdkResponse.contentTypes,
+  };
+
+  sdkContext.__typeCache.updateSdkResponseReferences(sdkResponse, retVar);
+  return retVar;
+}
+
 function fromSdkServiceResponseHeaders(
   sdkContext: CSharpEmitterContext,
   headers: SdkServiceResponseHeader[],

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

@@ -42,11 +42,11 @@ export function fromSdkType(
   sdkType: SdkType,
   literalTypeContext?: LiteralTypeContext,
 ): InputType {
-  if (sdkContext.__typeCache.types.has(sdkType)) {
-    return sdkContext.__typeCache.types.get(sdkType)!;
+  let retVar = sdkContext.__typeCache.types.get(sdkType);
+  if (retVar) {
+    return retVar;
   }
 
-  let retVar: InputType;
   switch (sdkType.kind) {
     case "nullable":
       const inputType = fromSdkType(sdkContext, sdkType.type);
@@ -104,26 +104,10 @@ export function fromSdkType(
       break;
   }
 
-  updateSdkTypeReferences(sdkContext, sdkType, retVar);
+  sdkContext.__typeCache.updateSdkTypeReferences(sdkType, retVar);
   return retVar;
 }
 
-function updateTypeCache(sdkContext: CSharpEmitterContext, typeName: string, type: InputType) {
-  if (type.kind === "model") {
-    sdkContext.__typeCache.models.set(typeName, type);
-  } else if (type.kind === "enum") {
-    sdkContext.__typeCache.enums.set(typeName, type);
-  }
-}
-
-function updateSdkTypeReferences(
-  sdkContext: CSharpEmitterContext,
-  sdkType: SdkType,
-  inputType: InputType,
-) {
-  sdkContext.__typeCache.types.set(sdkType, inputType);
-}
-
 export function fromSdkModelType(
   sdkContext: CSharpEmitterContext,
   modelType: SdkModelType,
@@ -145,7 +129,7 @@ export function fromSdkModelType(
       decorators: modelType.decorators,
     } as InputModelType;
 
-    updateTypeCache(sdkContext, modelTypeName, inputModelType);
+    sdkContext.__typeCache.updateTypeCache(modelTypeName, inputModelType);
 
     inputModelType.additionalProperties = modelType.additionalProperties
       ? fromSdkType(sdkContext, modelType.additionalProperties)
@@ -250,7 +234,7 @@ export function fromSdkEnumType(
       decorators: enumType.decorators,
     };
     if (addToCollection) {
-      updateTypeCache(sdkContext, enumName, inputEnumType);
+      sdkContext.__typeCache.updateTypeCache(enumName, inputEnumType);
     }
     for (const v of enumType.values) {
       values.push(fromSdkEnumValueType(sdkContext, v));
@@ -373,7 +357,7 @@ function fromSdkConstantType(
       decorators: constantType.decorators,
     };
 
-    updateTypeCache(sdkContext, enumName, enumType);
+    sdkContext.__typeCache.updateTypeCache(enumName, enumType);
 
     values.push({
       kind: "enumvalue",

packages/http-client-csharp/emitter/src/lib/typespec-server.ts

@@ -52,7 +52,6 @@ export function resolveServers(
         Type: inputType,
         Location: RequestLocation.Uri,
         IsApiVersion: name.toLowerCase() === "apiversion" || name.toLowerCase() === "api-version",
-        IsResourceParameter: false,
         IsContentType: false,
         IsRequired: true,
         IsEndpoint: isEndpoint,
@@ -79,7 +78,6 @@ export function resolveServers(
         },
         Location: RequestLocation.Uri,
         IsApiVersion: false,
-        IsResourceParameter: false,
         IsContentType: false,
         IsRequired: true,
         IsEndpoint: true,