REF: replacing hasRaw in depictBody to content type condition.

RobinTail · RobinTail · commit 8d4b9d382e15 · 2025-04-28T18:28:05.000+02:00
diff --git a/express-zod-api/src/documentation-helpers.ts b/express-zod-api/src/documentation-helpers.ts
@@ -39,9 +39,9 @@ import {
   ucFirst,
 } from "./common-helpers";
 import { InputSource } from "./config-type";
+import { contentTypes } from "./content-type";
 import { ezDateInBrand } from "./date-in-schema";
 import { ezDateOutBrand } from "./date-out-schema";
-import { hasRaw } from "./deep-checks";
 import { DocumentationError } from "./errors";
 import { ezFileBrand } from "./file-schema";
 import { extractObjectSchema, IOSchema } from "./io-schema";
@@ -729,7 +729,7 @@ export const depictBody = ({
     description,
     content: { [mimeType]: media },
   };
-  if (hasRequired || hasRaw(schema)) body.required = true;
+  if (hasRequired || mimeType === contentTypes.raw) body.required = true;
   return body;
 };
 
diff --git a/express-zod-api/tests/documentation-helpers.spec.ts b/express-zod-api/tests/documentation-helpers.spec.ts
@@ -2,6 +2,7 @@ import { JSONSchema } from "@zod/core";
 import { SchemaObject } from "openapi3-ts/oas31";
 import * as R from "ramda";
 import { z } from "zod";
+import { ez } from "../src";
 import {
   OpenAPIContext,
   depictExamples,
@@ -27,6 +28,7 @@ import {
   depictPipeline,
   depictDateIn,
   depictDateOut,
+  depictBody,
 } from "../src/documentation-helpers";
 
 describe("Documentation helpers", () => {
@@ -502,6 +504,19 @@ describe("Documentation helpers", () => {
     });
   });
 
+  describe("depictBody", () => {
+    test("should mark ez.raw() body as required", () => {
+      const body = depictBody({
+        ...requestCtx,
+        schema: ez.raw(),
+        composition: "inline",
+        mimeType: "application/octet-stream", // raw content type
+        paramNames: [],
+      });
+      expect(body.required).toBe(true);
+    });
+  });
+
   describe("depictDateIn", () => {
     test("should set type:string, pattern and format", () => {
       expect(
