@mandujs/core 0.35.0 → 0.35.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mandujs/core",
-  "version": "0.35.0",
+  "version": "0.35.1",
   "description": "Mandu Framework Core - Spec, Generator, Guard, Runtime",
   "type": "module",
   "main": "./src/index.ts",

package/src/openapi/generator.ts

@@ -140,11 +140,14 @@ export interface OpenAPIDocument {
 export function zodToOpenAPISchema(zodSchema: z.ZodTypeAny): OpenAPISchema {
   const typeName = getZodTypeName(zodSchema);
 
-  // Handle ZodOptional
+  // Handle ZodOptional — optionality is expressed by the parent object's
+  // `required[]` array (or by `parameter.required: false`), NOT by the
+  // field's own schema. Emitting `nullable: true` here would conflate
+  // "may be absent" with "may literally be null", which breaks Postman,
+  // codegen, and Swagger UI (they'd all treat the field as nullable).
   if (typeName === "ZodOptional") {
     const inner = getZodInnerType(zodSchema);
-    const innerSchema = inner ? zodToOpenAPISchema(inner) : {};
-    return { ...innerSchema, nullable: true };
+    return inner ? zodToOpenAPISchema(inner) : {};
   }
 
   // Handle ZodDefault

package/src/openapi/openapi.test.ts

@@ -147,12 +147,14 @@ describe("zodToOpenAPISchema", () => {
   });
 
   describe("modifiers", () => {
-    test("should convert ZodOptional", () => {
+    test("should convert ZodOptional — unwraps without adding nullable (optional != nullable)", () => {
       const schema = z.string().optional();
       const result = zodToOpenAPISchema(schema);
 
       expect(result.type).toBe("string");
-      expect(result.nullable).toBe(true);
+      // Optionality is expressed by the parent object's required[] / parameter.required,
+      // NOT by marking the field nullable. See generator.ts comment.
+      expect(result.nullable).toBeUndefined();
     });
 
     test("should convert ZodNullable", () => {
@@ -163,6 +165,26 @@ describe("zodToOpenAPISchema", () => {
       expect(result.nullable).toBe(true);
     });
 
+    test("should convert ZodOptional(ZodNullable) — inner nullable preserved", () => {
+      const schema = z.string().nullable().optional();
+      const result = zodToOpenAPISchema(schema);
+
+      expect(result.type).toBe("string");
+      expect(result.nullable).toBe(true);
+    });
+
+    test("object with optional field — field omitted from required[] but schema itself is not nullable", () => {
+      const schema = z.object({
+        name: z.string(),
+        age: z.number().optional(),
+      });
+      const result = zodToOpenAPISchema(schema);
+
+      expect(result.type).toBe("object");
+      expect(result.required).toEqual(["name"]);
+      expect(result.properties?.age).toEqual({ type: "number" });
+    });
+
     test("should convert ZodDefault", () => {
       const schema = z.number().default(10);
       const result = zodToOpenAPISchema(schema);