fix: dont immediately assume undefined if object with single enum null

Author: maxholman

__tests__/fixtures/test1/models.ts

@@ -3,7 +3,7 @@
  *
  * WARN: Do not edit directly.
  *
- * Generated on 2023-03-29T05:05:24.080Z
+ * Generated on 2023-04-01T07:54:08.728Z
  *
  */
 export type Uuid = string;
@@ -130,19 +130,19 @@ export type BillingAccountCreateRequest = {
   country: BillingCountry;
   timeZone: TimeZone;
   currency: Currency;
-  locale?: BillingLocale | undefined | null | undefined | null | undefined;
-  purchaseOrder?: StringU8 | undefined | null | undefined | null | undefined;
-  taxId?: StringU8 | undefined | null | undefined | null | undefined;
+  locale?: BillingLocale | undefined | null | undefined;
+  purchaseOrder?: StringU8 | undefined | null | undefined;
+  taxId?: StringU8 | undefined | null | undefined;
 };
 export type BillingAccountUpdateRequest = {
   name?: Name | undefined;
   email?: Email | undefined;
   country?: BillingCountry | undefined;
   timeZone?: TimeZone | undefined;
   currency?: Currency | undefined;
-  locale?: BillingLocale | undefined | null | undefined | null | undefined;
-  purchaseOrder?: StringU8 | undefined | null | undefined | null | undefined;
-  taxId?: StringU8 | undefined | null | undefined | null | undefined;
+  locale?: BillingLocale | undefined | null | undefined;
+  purchaseOrder?: StringU8 | undefined | null | undefined;
+  taxId?: StringU8 | undefined | null | undefined;
 };
 export type BillingAccountList = BillingAccount[];
 export type BillingAccountPortalRequest = {
@@ -221,7 +221,7 @@ export type UpdateBillingSubscriptionRequest = {
   trialEndTime?: DateTime | undefined;
 };
 export type UpdateBillingSubscriptionPromoCodeRequest = {
-  promoCode: StringU8 | null | undefined | null;
+  promoCode: StringU8 | null;
 };
 export type BillingSubscriptionPromoCodeLongRunningOperationSuccess = {
   operationId: Uuid;

lib/process-schema.ts

@@ -237,11 +237,12 @@ export function schemaToType(
   }
 
   if (schemaObject.type === 'object') {
+    // type=object and enum null is common openapi nullable workaround
     if (schemaObject.enum?.every((e) => e === null)) {
       return {
         name,
-        hasQuestionToken: true,
-        type: maybeWithUndefined('null', true),
+        hasQuestionToken: false,
+        type: maybeWithUndefined('null', false),
         docs,
       };
     }