Fix binary body

packages/typespec-test/test/batch_modular/generated/typespec-ts/src/api/operations.ts

@@ -12678,9 +12678,7 @@ export async function _getNodeRemoteDesktopFileDeserialize(
     throw createRestError(result);
   }
 
-  return typeof result.body === "string"
-    ? stringToUint8Array(result.body, "base64")
-    : result.body;
+  return result.body as any;
 }
 
 /**
@@ -13433,9 +13431,7 @@ export async function _getNodeFileDeserialize(
     throw createRestError(result);
   }
 
-  return typeof result.body === "string"
-    ? stringToUint8Array(result.body, "base64")
-    : result.body;
+  return result.body as any;
 }
 
 /** Returns the content of the specified Compute Node file. */

packages/typespec-test/test/contoso/tspconfig.yaml

@@ -1,9 +1,6 @@
 emit:
   - "@azure-tools/typespec-ts"
-  - "@azure-tools/typespec-autorest"
 options:
-  "@azure-tools/typespec-autorest":
-    "emitter-output-dir": "{project-root}/generated/openapi"
   "@azure-tools/typespec-ts":
     "emitter-output-dir": "{project-root}/generated/typespec-ts"
     azureSdkForJs: false

packages/typespec-test/test/loadtesting_modular/spec/client.tsp

@@ -32,7 +32,7 @@ interface AdministrationOperations {
 
 #suppress "@azure-tools/typespec-azure-core/use-standard-operations"
 @client({
-  name: "LoadTestRunClient",
+  name: "SubNamespace.LoadTestRunClient",
   service: Microsoft.LoadTestService,
 })
 interface TestRunOperations {

packages/typespec-test/test/schemaRegistry/generated/typespec-ts/src/api/schemaOperations/index.ts

@@ -15,7 +15,6 @@ import {
   PathUncheckedResponse,
   createRestError,
 } from "@azure-rest/core-client";
-import { stringToUint8Array, uint8ArrayToString } from "@azure/core-util";
 import {
   PagedAsyncIterableIterator,
   buildPagedAsyncIterator,
@@ -90,9 +89,7 @@ export async function _getSchemaByIdDeserialize(
     throw createRestError(result);
   }
 
-  return typeof result.body === "string"
-    ? stringToUint8Array(result.body, "base64")
-    : result.body;
+  return result.body as any;
 }
 
 /** Gets a registered schema by its unique ID.  Azure Schema Registry guarantees that ID is unique within a namespace. Operation response type is based on serialization of schema requested. */
@@ -179,9 +176,7 @@ export async function _getSchemaByVersionDeserialize(
     throw createRestError(result);
   }
 
-  return typeof result.body === "string"
-    ? stringToUint8Array(result.body, "base64")
-    : result.body;
+  return result.body as any;
 }
 
 /** Gets one specific version of one schema. */
@@ -219,7 +214,7 @@ export function _getSchemaIdByContentSend(
     .post({
       ...operationOptionsToRequestParameters(options),
       contentType: contentType,
-      body: uint8ArrayToString(schemaContent, "base64"),
+      body: schemaContent,
     });
 }
 
@@ -271,7 +266,7 @@ export function _registerSchemaSend(
     .put({
       ...operationOptionsToRequestParameters(options),
       contentType: contentType,
-      body: uint8ArrayToString(content, "base64"),
+      body: content,
     });
 }
 

packages/typespec-ts/src/modular/buildCodeModel.ts

@@ -479,7 +479,7 @@ function emitBodyParameter(
       type,
       location: "body",
       ...base,
-      isBinaryPayload: isBinaryPayload(context, body.type, contentTypes)
+      isBinaryPayload: isBinaryPayload(context, body.type)
     };
   }
   return undefined;
@@ -643,11 +643,7 @@ function emitResponse(
     discriminator: "basic",
     type: type,
     isBinaryPayload: innerResponse.body?.type
-      ? isBinaryPayload(
-          context,
-          innerResponse.body?.type,
-          innerResponse.body?.contentTypes![0] ?? "application/json"
-        )
+      ? isBinaryPayload(context, innerResponse.body?.type)
       : false
   };
 

packages/typespec-ts/src/modular/modularCodeModel.ts

@@ -20,7 +20,6 @@ export interface ModularCodeModel {
   options: RLCOptions;
   modularOptions: ModularOptions;
   namespace?: string;
-  subnamespaceToClients?: any;
   clients: Client[];
   types: Type[];
   project: Project;

packages/typespec-ts/src/transform/transformResponses.ts

@@ -249,9 +249,7 @@ function transformBody(
     if (!body || isVoidType(body.type)) {
       continue;
     }
-    const hasBinaryContent = body.contentTypes.some((contentType) =>
-      isBinaryPayload(dpgContext, body.type, contentType)
-    );
+    const hasBinaryContent = isBinaryPayload(dpgContext, body.type);
     if (hasBinaryContent) {
       typeSet.add(getBinaryType([SchemaContext.Output]));
       descriptions.add("Value may contain any sequence of octets");

packages/typespec-ts/src/utils/operationUtil.ts

@@ -33,7 +33,6 @@ import {
   HttpStatusCodesEntry
 } from "@typespec/http";
 import { SdkContext } from "./interfaces.js";
-import { KnownMediaType, knownMediaType } from "./mediaTypes.js";
 import { isByteOrByteUnion } from "./modelUtils.js";
 import { getOperationNamespaceInterfaceName } from "./namespaceUtils.js";
 import { resolveReference } from "../framework/reference.js";
@@ -171,18 +170,9 @@ export function isDefinedStatusCode(statusCodes: HttpStatusCodesEntry) {
   return statusCodes !== "*";
 }
 
-export function isBinaryPayload(
-  dpgContext: SdkContext,
-  body: Type,
-  contentType: string | string[]
-) {
-  const knownMediaTypes: KnownMediaType[] = (
-    Array.isArray(contentType) ? contentType : [contentType]
-  ).map((ct) => knownMediaType(ct));
-  for (const type of knownMediaTypes) {
-    if (type === KnownMediaType.Binary && isByteOrByteUnion(dpgContext, body)) {
-      return true;
-    }
+export function isBinaryPayload(dpgContext: SdkContext, body: Type) {
+  if (isByteOrByteUnion(dpgContext, body)) {
+    return true;
   }
   return false;
 }