@azure-tools/typespec-ts 0.56.0-dev.9 → 0.56.1-dev.0

package/src/modular/helpers/naming-helpers.ts

@@ -1,7 +1,12 @@
-import { SdkClientType, SdkServiceOperation } from "@azure-tools/typespec-client-generator-core";
-import { SdkContext } from "../../utils/interfaces.js";
+import {
+  getClientNameOverride,
+  type SdkClientType,
+  type SdkServiceOperation,
+} from "@azure-tools/typespec-client-generator-core";
+import pluralize from "pluralize";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { NameType, normalizeName, ReservedModelNames } from "../../utils/name-utils.js";
-import { ServiceOperation } from "../../utils/operation-util.js";
+import type { ServiceOperation } from "../../utils/operation-util.js";
 
 export function getClientName(client: SdkClientType<SdkServiceOperation>): string {
   return client.name.replace(/Client$/, "");
@@ -12,19 +17,47 @@ export function getClassicalClientName(client: SdkClientType<SdkServiceOperation
 }
 
 export interface GuardedName {
+  /** The name used for the generated API-layer function (e.g. `deleteConversation`). */
   name: string;
+  /** The name used for the public method exposed on clients and operation groups. */
+  propertyName: string;
   fixme?: string[];
 }
 
 export function getOperationName(
   operation: ServiceOperation,
   dpgContext?: SdkContext,
+  prefixes: string[] = [],
 ): GuardedName {
-  const norm = normalizeName(operation.name, NameType.Method, true);
+  const name = normalizeName(operation.name, NameType.Method, true);
+  const propertyName = normalizeName(operation.name, NameType.Property);
   const isDataplane = dpgContext !== undefined && !dpgContext.emitterOptions?.azureArm;
-  if (isReservedName(operation.name, NameType.Method) && isDataplane) {
+  // An explicit `@clientName` override is an intentional naming choice by the user, so we
+  // honor it verbatim and skip the reserved-word disambiguation (and its `@fixme`). The
+  // public method keeps the reserved word (e.g. `delete`), while the generated API-layer
+  // function stays guarded (e.g. `$delete`) because a reserved word is not a valid function
+  // binding in JavaScript.
+  const hasClientNameOverride =
+    dpgContext !== undefined &&
+    operation.__raw !== undefined &&
+    getClientNameOverride(dpgContext, operation.__raw) !== undefined;
+  if (isReservedName(operation.name, NameType.Method) && isDataplane && !hasClientNameOverride) {
+    const suffix = getReservedNameGroupSuffix(prefixes);
+    if (suffix) {
+      // Disambiguate the reserved word by suffixing the singularized operation group
+      // name, e.g. `delete` in the `Conversations` group becomes `deleteConversation`.
+      const disambiguated = normalizeName(`${operation.name}_${suffix}`, NameType.Method);
+      return {
+        name: disambiguated,
+        propertyName: disambiguated,
+      };
+    }
+    // There is no operation group to disambiguate the reserved word with (e.g. a top-level
+    // operation), so keep the guarded name (e.g. `$delete`) and surface a fixme asking the
+    // user to override the generated name.
     return {
-      name: norm,
+      name,
+      propertyName,
       fixme: [
         `${operation.name} is a reserved word that cannot be used as an operation name. 
         Please add @clientName("clientName") or @clientName("<JS-Specific-Name>", "javascript") 
@@ -33,10 +66,25 @@ export function getOperationName(
     };
   }
   return {
-    name: norm,
+    name,
+    propertyName,
   };
 }
 
+/**
+ * Builds the suffix used to disambiguate a reserved-word operation name from the
+ * operation group it belongs to. The innermost operation group name is singularized
+ * so that, for example, the `Conversations` group yields `Conversation`.
+ */
+function getReservedNameGroupSuffix(prefixes: string[]): string | undefined {
+  const groups = prefixes.filter((prefix) => prefix && prefix.length > 0);
+  const innermostGroup = groups[groups.length - 1];
+  if (!innermostGroup) {
+    return undefined;
+  }
+  return pluralize.singular(normalizeName(innermostGroup, NameType.Interface));
+}
+
 export function isReservedName(name: string, nameType: NameType): boolean {
   return ReservedModelNames.some(
     (reservedName) =>

package/src/modular/helpers/operation-helpers.ts

@@ -2,30 +2,30 @@ import {
   getClientOptions,
   isHttpMetadata,
   isReadOnly,
-  SdkBodyParameter,
-  SdkClientType,
-  SdkConstantType,
-  SdkEnumType,
-  SdkHttpOperation,
-  SdkHttpParameter,
-  SdkLroPagingServiceMethod,
-  SdkLroServiceMethod,
-  SdkMethod,
-  SdkMethodParameter,
-  SdkModelPropertyType,
-  SdkModelType,
-  SdkPagingServiceMethod,
-  SdkServiceResponseHeader,
-  SdkType,
+  type SdkBodyParameter,
+  type SdkClientType,
+  type SdkConstantType,
+  type SdkEnumType,
+  type SdkHttpOperation,
+  type SdkHttpParameter,
+  type SdkLroPagingServiceMethod,
+  type SdkLroServiceMethod,
+  type SdkMethod,
+  type SdkMethodParameter,
+  type SdkModelPropertyType,
+  type SdkModelType,
+  type SdkPagingServiceMethod,
+  type SdkServiceResponseHeader,
+  type SdkType,
 } from "@azure-tools/typespec-client-generator-core";
-import { NoTarget, Program } from "@typespec/compiler";
+import { NoTarget, type Program } from "@typespec/compiler";
 import { isHeader, isMetadata } from "@typespec/http";
 import {
-  FunctionDeclarationStructure,
-  OptionalKind,
-  ParameterDeclarationStructure,
+  type FunctionDeclarationStructure,
+  type OptionalKind,
+  type ParameterDeclarationStructure,
   StructureKind,
-  TypeAliasDeclarationStructure,
+  type TypeAliasDeclarationStructure,
 } from "ts-morph";
 import { useContext } from "../../context-manager.js";
 import { useSdkTypes } from "../../framework/hooks/sdk-types.js";
@@ -33,7 +33,7 @@ import { useDependencies } from "../../framework/hooks/use-dependencies.js";
 import { resolveReference } from "../../framework/reference.js";
 import { refkey } from "../../framework/refkey.js";
 import { reportDiagnostic } from "../../lib.js";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { isAzureCoreErrorType } from "../../utils/model-utils.js";
 import { NameType, normalizeName } from "../../utils/name-utils.js";
 import {
@@ -46,9 +46,9 @@ import {
   isMultipartPayload,
   isXmlPayload,
   KnownCollectionFormat,
-  ServiceOperation,
+  type ServiceOperation,
 } from "../../utils/operation-util.js";
-import { AzurePollingDependencies } from "../external-dependencies.js";
+import { AzureCoreDependencies, AzurePollingDependencies } from "../external-dependencies.js";
 import {
   buildModelDeserializer,
   buildPropertyDeserializer,
@@ -66,11 +66,10 @@ import {
   getPropertyWithOverrides,
   isNormalUnion,
   isSpecialHandledUnion,
-  ModelOverrideOptions,
+  type ModelOverrideOptions,
 } from "../serialization/serialize-utils.js";
 import {
   PagingHelpers,
-  PlatformTypeHelpers,
   PollingHelpers,
   SerializationHelpers,
   StorageCompatHelpers,
@@ -112,7 +111,7 @@ export function getSendPrivateFunction(
 ): OptionalKind<FunctionDeclarationStructure> {
   const operation = method[1];
   const parameters = getOperationSignatureParameters(dpgContext, method, clientType);
-  const { name } = getOperationName(operation);
+  const { name } = getOperationName(operation, dpgContext, method[0]);
   const dependencies = useDependencies();
 
   const functionStatement: OptionalKind<FunctionDeclarationStructure> = {
@@ -169,7 +168,7 @@ export function getDeserializePrivateFunction(
   method: [string[], ServiceOperation],
 ): OptionalKind<FunctionDeclarationStructure> {
   const operation = method[1];
-  const { name } = getOperationName(operation);
+  const { name } = getOperationName(operation, context, method[0]);
   const dependencies = useDependencies();
   const PathUncheckedResponseReference = resolveReference(dependencies.PathUncheckedResponse);
 
@@ -213,11 +212,11 @@ export function getDeserializePrivateFunction(
     };
   } else if (response.type) {
     // When response.optional is true, some HTTP responses have no body (e.g. 204), so
-    // the return type must include undefined to reflect that possibility.
+    // the return type must include void to reflect that possibility.
     const baseType = getTypeExpression(context, response.type);
     returnType = {
       name: (response as any).name ?? "",
-      type: response.optional ? `${baseType} | undefined` : baseType,
+      type: response.optional ? `${baseType} | void` : baseType,
     };
   } else if (isHeadAsBooleanOperation(operation)) {
     returnType = { name: "", type: "boolean" };
@@ -247,7 +246,7 @@ export function getDeserializePrivateFunction(
   statements.push(`const expectedStatuses = ${getExpectedStatuses(operation)};`);
   statements.push(
     `if(!expectedStatuses.includes(result.status)){`,
-    `${getExceptionThrowStatement(context, operation)}`,
+    `${getExceptionThrowStatement(context, method)}`,
     "}",
   );
   const deserializedType =
@@ -405,15 +404,15 @@ export function getDeserializePrivateFunction(
       }
       if (deserializeFunctionName) {
         if (needsBodyGuard) {
-          // Use ternary form: return result.body ? deserializer(result.body) : undefined
           statements.push(
-            `return ${deserializedRoot} ? ${deserializeFunctionName}(${deserializedRoot})${multipartCastSuffix} : undefined`,
-          );
-        } else {
-          statements.push(
-            `return ${deserializeFunctionName}(${deserializedRoot})${multipartCastSuffix}`,
+            `if (!${deserializedRoot}) {
+            return;
+          }`,
           );
         }
+        statements.push(
+          `return ${deserializeFunctionName}(${deserializedRoot})${multipartCastSuffix}`,
+        );
       } else if (isAzureCoreErrorType(context.program, deserializedType.__raw)) {
         statements.push(`return ${deserializedRoot}${multipartCastSuffix}`);
       } else if (isHeadAsBooleanOperation(operation)) {
@@ -461,8 +460,9 @@ export function getDeserializePrivateFunction(
  */
 export function getDeserializeHeadersPrivateFunction(
   context: SdkContext,
-  operation: ServiceOperation,
+  method: [string[], ServiceOperation],
 ): OptionalKind<FunctionDeclarationStructure> | undefined {
+  const operation = method[1];
   const responseHeaders = getResponseHeaders(operation.operation.responses);
   const isResponseHeadersEnabled = context.emitterOptions?.includeHeadersInResponse === true;
   const isStorageCompatEnabled = context.emitterOptions?.enableStorageCompat === true;
@@ -472,7 +472,7 @@ export function getDeserializeHeadersPrivateFunction(
     return undefined;
   }
 
-  const { name } = getOperationName(operation);
+  const { name } = getOperationName(operation, context, method[0]);
   const dependencies = useDependencies();
   const PathUncheckedResponseReference = resolveReference(dependencies.PathUncheckedResponse);
 
@@ -615,8 +615,9 @@ function getExceptionResponseHeaders(
  */
 export function getDeserializeExceptionHeadersPrivateFunction(
   context: SdkContext,
-  operation: ServiceOperation,
+  method: [string[], ServiceOperation],
 ): OptionalKind<FunctionDeclarationStructure> | undefined {
+  const operation = method[1];
   const isResponseHeadersEnabled = context.emitterOptions?.includeHeadersInResponse === true;
   if (!isResponseHeadersEnabled) {
     return undefined;
@@ -627,7 +628,7 @@ export function getDeserializeExceptionHeadersPrivateFunction(
     return undefined;
   }
 
-  const { name } = getOperationName(operation);
+  const { name } = getOperationName(operation, context, method[0]);
   const dependencies = useDependencies();
   const PathUncheckedResponseReference = resolveReference(dependencies.PathUncheckedResponse);
 
@@ -667,7 +668,8 @@ function getExceptionDeserializeExpr(exception: ExceptionThrowDetail): string {
   return `isXml ? ${exception.xmlDeserializer}(result.body) : ${exception.deserializer}(result.body)`;
 }
 
-function getExceptionThrowStatement(context: SdkContext, operation: ServiceOperation) {
+function getExceptionThrowStatement(context: SdkContext, method: [string[], ServiceOperation]) {
+  const operation = method[1];
   const statements = [];
   const createRestErrorReference = resolveReference(useDependencies().createRestError);
   const { customized, defaultDeserializer, defaultXmlDeserializer, defaultIsXmlOnly } =
@@ -678,7 +680,7 @@ function getExceptionThrowStatement(context: SdkContext, operation: ServiceOpera
   // Check if exception headers function exists and build the call
   const exceptionHeaders = getExceptionResponseHeaders(operation.operation.exceptions);
   const hasExceptionHeaders = isResponseHeadersEnabled && exceptionHeaders.length > 0;
-  const { name: opName } = getOperationName(operation);
+  const { name: opName } = getOperationName(operation, context, method[0]);
   const exceptionHeadersCall = hasExceptionHeaders
     ? `error.details = {...(error.details as any), ..._${opName}DeserializeExceptionHeaders(result)};`
     : undefined;
@@ -949,13 +951,13 @@ export function getOperationFunction(
       const baseCompositeType = buildCompositeResponseType(context, type, responseHeaders);
       returnType = {
         name: (type as any).name ?? "",
-        type: response.optional ? `${baseCompositeType} | undefined` : baseCompositeType,
+        type: response.optional ? `${baseCompositeType} | void` : baseCompositeType,
       };
     } else {
       const baseType = getTypeExpression(context, type!);
       returnType = {
         name: (type as any).name ?? "",
-        type: response.optional ? `${baseType} | undefined` : baseType,
+        type: response.optional ? `${baseType} | void` : baseType,
       };
     }
   } else if (hasHeaderOnlyResponse && isResponseHeadersEnabled) {
@@ -996,14 +998,14 @@ export function getOperationFunction(
     }
   }
 
-  const { name, fixme = [] } = getOperationName(operation, context);
+  const { name, propertyName, fixme = [] } = getOperationName(operation, context, method[0]);
   const functionStatement = {
     kind: StructureKind.Function,
     docs: [...getDocsFromDescription(operation.doc), ...getFixmeForMultilineDocs(fixme)],
     isAsync: true,
     isExported: true,
     name,
-    propertyName: normalizeName(operation.name, NameType.Property),
+    propertyName,
     parameters,
     returnType: `Promise<${finalReturnType}>`,
   };
@@ -1043,7 +1045,7 @@ export function getOperationFunction(
     statements.push(`const ${streamableMethodVarName} = _${name}Send(${sendParameterList});`);
     const binaryHelper =
       wrapReturn && wrapReturnIsBinary
-        ? SerializationHelpers.getBinaryStreamResponse
+        ? AzureCoreDependencies["getBinaryStreamResponse"]
         : SerializationHelpers.getBinaryResponse;
     statements.push(
       `const ${resultVarName} = await ${resolveReference(binaryHelper)}(${streamableMethodVarName});`,
@@ -1120,7 +1122,7 @@ function getLroOnlyOperationFunction(
     clientType,
   );
   const returnType = buildLroReturnType(context, operation);
-  const { name, fixme = [] } = getOperationName(operation, context);
+  const { name, propertyName, fixme = [] } = getOperationName(operation, context, method[0]);
   const pollerLikeReference = resolveReference(AzurePollingDependencies.PollerLike);
   const operationStateReference = resolveReference(AzurePollingDependencies.OperationState);
 
@@ -1140,7 +1142,7 @@ function getLroOnlyOperationFunction(
     isAsync: false,
     isExported: true,
     name,
-    propertyName: normalizeName(operation.name, NameType.Property),
+    propertyName,
     isLro: true,
     lroFinalReturnType: effectiveReturnTypeName,
     parameters,
@@ -1194,7 +1196,7 @@ function getLroAndPagingOperationFunction(
 } {
   const operation = method[1];
   const parameters = getOperationSignatureParameters(context, method, clientType);
-  const { name, fixme = [] } = getOperationName(operation, context);
+  const { name, propertyName, fixme = [] } = getOperationName(operation, context, method[0]);
 
   const returnType = buildLroPagingReturnType(context, operation);
 
@@ -1247,7 +1249,7 @@ function getLroAndPagingOperationFunction(
     isLroPaging: true,
     lropagingFinalReturnType: returnType.type,
     name,
-    propertyName: normalizeName(operation.name, NameType.Property),
+    propertyName,
     parameters,
     returnType: `${refs.pagedIterator}<${returnType.type}>`,
     statements: [
@@ -1324,7 +1326,7 @@ function getPagingOnlyOperationFunction(
       type: getTypeExpression(context, type.valueType),
     };
   }
-  const { name, fixme = [] } = getOperationName(operation, context);
+  const { name, propertyName, fixme = [] } = getOperationName(operation, context, method[0]);
   const pagedAsyncIterableIteratorReference = resolveReference(
     PagingHelpers.PagedAsyncIterableIterator,
   );
@@ -1335,7 +1337,7 @@ function getPagingOnlyOperationFunction(
     isAsync: false,
     isExported: true,
     name,
-    propertyName: normalizeName(operation.name, NameType.Property),
+    propertyName,
     parameters,
     returnType: `${pagedAsyncIterableIteratorReference}<${returnType.type}>`,
   };
@@ -3003,7 +3005,7 @@ export function buildNonModelResponseTypeDeclaration(
   let typeBody: string;
 
   if (isBinary) {
-    const nodeReadableStreamRef = resolveReference(PlatformTypeHelpers.NodeReadableStream);
+    const nodeReadableStreamRef = resolveReference(AzureCoreDependencies["NodeReadableStream"]);
     typeBody = `{
       /**
        * BROWSER ONLY

package/src/modular/helpers/type-helpers.ts

@@ -1,4 +1,4 @@
-import {
+import type {
   SdkBodyParameter,
   SdkCredentialParameter,
   SdkEndpointParameter,
@@ -6,11 +6,11 @@ import {
   SdkModelType,
   SdkType,
 } from "@azure-tools/typespec-client-generator-core";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { NameType, normalizeName } from "../../utils/name-utils.js";
 import {
   getPropertyWithOverrides,
-  ModelOverrideOptions,
+  type ModelOverrideOptions,
 } from "../serialization/serialize-utils.js";
 import { getAllAncestors, getAllProperties } from "./operation-helpers.js";
 

package/src/modular/interfaces.ts

@@ -1,4 +1,4 @@
-import { ClientOptions } from "../interfaces.js";
+import type { ClientOptions } from "../interfaces.js";
 
 export interface ModularOptions {
   sourceRoot: string;

package/src/modular/serialization/build-deserializer-function.ts

@@ -1,19 +1,19 @@
 import {
-  SdkArrayType,
-  SdkDictionaryType,
-  SdkModelPropertyType,
-  SdkModelType,
-  SdkType,
-  SdkUnionType,
+  type SdkArrayType,
+  type SdkDictionaryType,
+  type SdkModelPropertyType,
+  type SdkModelType,
+  type SdkType,
+  type SdkUnionType,
   UsageFlags,
 } from "@azure-tools/typespec-client-generator-core";
 import { NoTarget } from "@typespec/compiler";
-import { FunctionDeclarationStructure, StructureKind } from "ts-morph";
+import { type FunctionDeclarationStructure, StructureKind } from "ts-morph";
 import { useContext } from "../../context-manager.js";
 import { resolveReference } from "../../framework/reference.js";
 import { refkey } from "../../framework/refkey.js";
 import { reportDiagnostic } from "../../lib.js";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { isAzureCoreErrorType } from "../../utils/model-utils.js";
 import { NameType, normalizeName } from "../../utils/name-utils.js";
 import { getAdditionalPropertiesName, normalizeModelName } from "../emit-models.js";
@@ -29,7 +29,7 @@ import {
   getAllDiscriminatedValues,
   isDiscriminatedUnion,
   isSupportedSerializeType,
-  ModelSerializeOptions,
+  type ModelSerializeOptions,
 } from "./serialize-utils.js";
 
 export function buildPropertyDeserializer(

package/src/modular/serialization/build-serializer-function.ts

@@ -1,20 +1,20 @@
 import {
-  SdkArrayType,
-  SdkDictionaryType,
-  SdkModelPropertyType,
-  SdkModelType,
-  SdkType,
-  SdkUnionType,
+  type SdkArrayType,
+  type SdkDictionaryType,
+  type SdkModelPropertyType,
+  type SdkModelType,
+  type SdkType,
+  type SdkUnionType,
   UsageFlags,
 } from "@azure-tools/typespec-client-generator-core";
 import { NoTarget } from "@typespec/compiler";
 import { isOrExtendsHttpFile } from "@typespec/http";
-import { FunctionDeclarationStructure, StructureKind } from "ts-morph";
+import { type FunctionDeclarationStructure, StructureKind } from "ts-morph";
 import { useContext } from "../../context-manager.js";
 import { resolveReference } from "../../framework/reference.js";
 import { refkey } from "../../framework/refkey.js";
 import { reportDiagnostic } from "../../lib.js";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { isAzureCoreErrorType } from "../../utils/model-utils.js";
 import { NameType, normalizeName } from "../../utils/name-utils.js";
 import { getAdditionalPropertiesName, normalizeModelName } from "../emit-models.js";
@@ -31,7 +31,7 @@ import {
   getAllDiscriminatedValues,
   isDiscriminatedUnion,
   isSupportedSerializeType,
-  ModelSerializeOptions,
+  type ModelSerializeOptions,
 } from "./serialize-utils.js";
 
 export function buildPropertySerializer(

package/src/modular/serialization/build-xml-serializer-function.ts

@@ -3,20 +3,20 @@
 
 import {
   isReadOnly,
-  SdkModelPropertyType,
-  SdkModelType,
-  SdkPackage,
-  SdkType,
+  type SdkModelPropertyType,
+  type SdkModelType,
+  type SdkPackage,
+  type SdkType,
   UsageFlags,
 } from "@azure-tools/typespec-client-generator-core";
 import { NoTarget } from "@typespec/compiler";
 import { isMetadata } from "@typespec/http";
-import { FunctionDeclarationStructure, StructureKind } from "ts-morph";
+import { type FunctionDeclarationStructure, StructureKind } from "ts-morph";
 import { useDependencies } from "../../framework/hooks/use-dependencies.js";
 import { resolveReference } from "../../framework/reference.js";
 import { refkey } from "../../framework/refkey.js";
 import { reportDiagnostic } from "../../lib.js";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { isAzureCoreErrorType } from "../../utils/model-utils.js";
 import { NameType } from "../../utils/name-utils.js";
 import { getAdditionalPropertiesName, normalizeModelName } from "../emit-models.js";
@@ -24,7 +24,7 @@ import { getAllAncestors, getAllProperties } from "../helpers/operation-helpers.
 import { getAdditionalPropertiesType } from "../helpers/type-helpers.js";
 import { XmlHelpers } from "../static-helpers-metadata.js";
 import { normalizeModelPropertyName } from "../type-expressions/get-type-expression.js";
-import { isSupportedSerializeType, ModelSerializeOptions } from "./serialize-utils.js";
+import { isSupportedSerializeType, type ModelSerializeOptions } from "./serialize-utils.js";
 
 /**
  * Checks if a model type has XML serialization options defined

package/src/modular/serialization/serialize-utils.ts

@@ -1,4 +1,4 @@
-import {
+import type {
   SdkModelPropertyType,
   SdkModelType,
   SdkType,

package/src/modular/static-helpers-metadata.ts

@@ -59,11 +59,6 @@ export const SerializationHelpers = {
     name: "getBinaryResponse",
     location: "serialization/get-binary-response.ts",
   },
-  getBinaryStreamResponse: {
-    kind: "function",
-    name: "getBinaryStreamResponse",
-    location: "serialization/get-binary-stream-response.ts",
-  },
   areAllPropsUndefined: {
     kind: "function",
     name: "areAllPropsUndefined",
@@ -150,14 +145,6 @@ export const MultipartHelpers = {
   },
 } as const;
 
-export const PlatformTypeHelpers = {
-  NodeReadableStream: {
-    kind: "typeAlias",
-    name: "NodeReadableStream",
-    location: "platform-types.ts",
-  },
-} as const;
-
 export const CloudSettingHelpers = {
   AzureClouds: {
     kind: "enum",

package/src/modular/type-expressions/get-credential-expression.ts

@@ -1,4 +1,4 @@
-import { SdkCredentialType } from "@azure-tools/typespec-client-generator-core";
+import type { SdkCredentialType } from "@azure-tools/typespec-client-generator-core";
 import { useDependencies } from "../../framework/hooks/use-dependencies.js";
 import { resolveReference } from "../../framework/reference.js";
 

package/src/modular/type-expressions/get-enum-expression.ts

@@ -1,7 +1,7 @@
-import { SdkEnumType } from "@azure-tools/typespec-client-generator-core";
+import type { SdkEnumType } from "@azure-tools/typespec-client-generator-core";
 import { resolveReference } from "../../framework/reference.js";
-import { SdkContext } from "../../utils/interfaces.js";
-import { EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
+import type { SdkContext } from "../../utils/interfaces.js";
+import { type EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
 import { shouldEmitInline } from "./utils.js";
 
 export function getEnumExpression(

package/src/modular/type-expressions/get-model-expression.ts

@@ -1,10 +1,10 @@
 import {
-  EmitTypeOptions,
+  type EmitTypeOptions,
   getTypeExpression,
   normalizeModelPropertyName,
 } from "./get-type-expression.js";
 
-import {
+import type {
   SdkModelPropertyType,
   SdkModelType,
   SdkServiceResponseHeader,
@@ -12,7 +12,7 @@ import {
 import { useContext } from "../../context-manager.js";
 import { resolveReference } from "../../framework/reference.js";
 import { refkey } from "../../framework/refkey.js";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { MultipartHelpers } from "../static-helpers-metadata.js";
 import { shouldEmitInline } from "./utils.js";
 

package/src/modular/type-expressions/get-nullable-expression.ts

@@ -1,7 +1,7 @@
-import { SdkNullableType } from "@azure-tools/typespec-client-generator-core";
+import type { SdkNullableType } from "@azure-tools/typespec-client-generator-core";
 import { resolveReference } from "../../framework/reference.js";
-import { SdkContext } from "../../utils/interfaces.js";
-import { EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
+import type { SdkContext } from "../../utils/interfaces.js";
+import { type EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
 import { shouldEmitInline } from "./utils.js";
 
 export function getNullableExpression(

package/src/modular/type-expressions/get-type-expression.ts

@@ -1,10 +1,10 @@
-import {
+import type {
   SdkHttpParameter,
   SdkModelPropertyType,
   SdkServiceResponseHeader,
   SdkType,
 } from "@azure-tools/typespec-client-generator-core";
-import { SdkContext } from "../../utils/interfaces.js";
+import type { SdkContext } from "../../utils/interfaces.js";
 import { NameType, normalizeName } from "../../utils/name-utils.js";
 import { getCredentialExpression } from "./get-credential-expression.js";
 import { getEnumExpression } from "./get-enum-expression.js";

package/src/modular/type-expressions/get-union-expression.ts

@@ -1,7 +1,7 @@
-import { SdkUnionType } from "@azure-tools/typespec-client-generator-core";
+import type { SdkUnionType } from "@azure-tools/typespec-client-generator-core";
 import { resolveReference } from "../../framework/reference.js";
-import { SdkContext } from "../../utils/interfaces.js";
-import { EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
+import type { SdkContext } from "../../utils/interfaces.js";
+import { type EmitTypeOptions, getTypeExpression } from "./get-type-expression.js";
 import { shouldEmitInline } from "./utils.js";
 
 export function getUnionExpression(

package/src/modular/type-expressions/utils.ts

@@ -1,4 +1,4 @@
-import { EmitTypeOptions } from "./get-type-expression.js";
+import type { EmitTypeOptions } from "./get-type-expression.js";
 
 export function shouldEmitInline(
   type: { isGeneratedName?: boolean },