oas 20.8.2 → 20.8.3

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## <small>20.8.3 (2023-05-11)</small>
+
+* feat: split mixed json schema types with booleans out into a oneOf (#763) ([0e1014a](https://github.com/readmeio/oas/commit/0e1014a)), closes [#763](https://github.com/readmeio/oas/issues/763)
+
+
+
 ## <small>20.8.2 (2023-05-04)</small>
 
 * feat: adding a broken skipped test, retaining soeme more jsonschema props ([94436b8](https://github.com/readmeio/oas/commit/94436b8))

package/dist/lib/openapi-to-json-schema.js

@@ -411,7 +411,7 @@ function toJSONSchema(data, opts) {
             if (schema.type.length === 1) {
                 schema.type = schema.type.shift();
             }
-            else if (schema.type.includes('array') || schema.type.includes('object')) {
+            else if (schema.type.includes('array') || schema.type.includes('boolean') || schema.type.includes('object')) {
                 // If we have a `null` type but there's only two types present then we can remove `null`
                 // as an option and flag the whole schema as `nullable`.
                 var isNullable_1 = schema.type.includes('null');
@@ -422,10 +422,11 @@ function toJSONSchema(data, opts) {
                     // If this mixed type has non-primitives then we for convenience of our implementation
                     // we're moving them into a `oneOf`.
                     var nonPrimitives_1 = [];
-                    // Because we're moving an `array` and/or an `object` into a `oneOf` we also want to take
-                    // with it its specific properties that maybe present on our current schema.
+                    // Because arrays, booleans, and objects are not compatible with any other schem type
+                    // other than null we're moving them into an isolated `oneOf`, and as such want to take
+                    // with it its specific properties that may be present on our current schema.
                     Object.entries({
-                        // json-schema.org/understanding-json-schema/reference/array.html
+                        // https://json-schema.org/understanding-json-schema/reference/array.html
                         array: [
                             'additionalItems',
                             'contains',
@@ -437,6 +438,10 @@ function toJSONSchema(data, opts) {
                             'prefixItems',
                             'uniqueItems',
                         ],
+                        // https://json-schema.org/understanding-json-schema/reference/boolean.html
+                        boolean: [
+                        // Booleans don't have any boolean-specific properties.
+                        ],
                         // https://json-schema.org/understanding-json-schema/reference/object.html
                         object: [
                             'additionalProperties',
@@ -471,7 +476,7 @@ function toJSONSchema(data, opts) {
                         });
                         nonPrimitives_1.push(reducedSchema);
                     });
-                    schema.type = schema.type.filter(function (t) { return t !== 'array' && t !== 'object'; });
+                    schema.type = schema.type.filter(function (t) { return t !== 'array' && t !== 'boolean' && t !== 'object'; });
                     if (schema.type.length === 1) {
                         schema.type = schema.type.shift();
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oas",
-  "version": "20.8.2",
+  "version": "20.8.3",
   "description": "Comprehensive tooling for working with OpenAPI definitions",
   "license": "MIT",
   "author": "ReadMe <support@readme.io> (https://readme.com)",

package/src/lib/openapi-to-json-schema.ts

@@ -456,7 +456,7 @@ export default function toJSONSchema(
       // We don't need `type: [<type>]` when we can just as easily make it `type: <type>`.
       if (schema.type.length === 1) {
         schema.type = schema.type.shift();
-      } else if (schema.type.includes('array') || schema.type.includes('object')) {
+      } else if (schema.type.includes('array') || schema.type.includes('boolean') || schema.type.includes('object')) {
         // If we have a `null` type but there's only two types present then we can remove `null`
         // as an option and flag the whole schema as `nullable`.
         const isNullable = schema.type.includes('null');
@@ -468,10 +468,11 @@ export default function toJSONSchema(
           // we're moving them into a `oneOf`.
           const nonPrimitives: any[] = [];
 
-          // Because we're moving an `array` and/or an `object` into a `oneOf` we also want to take
-          // with it its specific properties that maybe present on our current schema.
+          // Because arrays, booleans, and objects are not compatible with any other schem type
+          // other than null we're moving them into an isolated `oneOf`, and as such want to take
+          // with it its specific properties that may be present on our current schema.
           Object.entries({
-            // json-schema.org/understanding-json-schema/reference/array.html
+            // https://json-schema.org/understanding-json-schema/reference/array.html
             array: [
               'additionalItems',
               'contains',
@@ -484,6 +485,11 @@ export default function toJSONSchema(
               'uniqueItems',
             ],
 
+            // https://json-schema.org/understanding-json-schema/reference/boolean.html
+            boolean: [
+              // Booleans don't have any boolean-specific properties.
+            ],
+
             // https://json-schema.org/understanding-json-schema/reference/object.html
             object: [
               'additionalProperties',
@@ -521,7 +527,7 @@ export default function toJSONSchema(
             nonPrimitives.push(reducedSchema);
           });
 
-          schema.type = schema.type.filter(t => t !== 'array' && t !== 'object');
+          schema.type = schema.type.filter(t => t !== 'array' && t !== 'boolean' && t !== 'object');
           if (schema.type.length === 1) {
             schema.type = schema.type.shift();
           }