fix: account for children making parent fields required

Author: Daniel Playfair Cal

src/schema.js

@@ -178,22 +178,57 @@ const getObjectTypeContent = (schema) => {
 const complexTypeGetter = (schema) => getInlineParseContent(schema);
 const filterContents = (contents, types) => _.filter(contents, (type) => !_.includes(types, type));
 
+const makeAddRequiredToChildSchema = (parentSchema) => (childSchema) => {
+  let required = childSchema.required || [];
+  let properties = childSchema.properties || {};
+
+  // Inherit all the required fields from the parent schema that are defined
+  // either on the parent schema or on the child schema
+  // TODO: any that are defined at grandparents or higher are ignored
+  required = required.concat(
+    (parentSchema.required || []).filter(
+      (key) =>
+        !required.includes(key) && (_.keys(properties).includes(key) || _.keys(parentSchema.properties).includes(key)),
+    ),
+  );
+
+  // Identify properties that are required in the child schema, but
+  // defined only in the parent schema (TODO: this only works one level deep)
+  const parentPropertiesRequiredByChild = required.filter(
+    (key) => !_.keys(childSchema.properties).includes(key) && _.keys(parentSchema.properties).includes(key),
+  );
+
+  // Add such properties to the child so that they can be overriden and made required
+  properties = {
+    ...properties,
+    ...parentPropertiesRequiredByChild.reduce(
+      (additionalProperties, key) => ({
+        ...additionalProperties,
+        [key]: (parentSchema.properties || {})[key],
+      }),
+      {},
+    ),
+  };
+
+  return _.merge(
+    {
+      required: required,
+      properties: properties,
+    },
+    childSchema,
+  );
+};
+
 const complexSchemaParsers = {
   [SCHEMA_TYPES.COMPLEX_ONE_OF]: (schema) => {
     // T1 | T2
-    const combined = _.map(
-      schema.oneOf.map((s) => _.merge({ required: schema.required }, s)),
-      complexTypeGetter,
-    );
+    const combined = _.map(schema.oneOf.map(makeAddRequiredToChildSchema(schema)), complexTypeGetter);
 
     return checkAndAddNull(schema, filterContents(combined, [TS_KEYWORDS.ANY]).join(" | "));
   },
   [SCHEMA_TYPES.COMPLEX_ALL_OF]: (schema) => {
     // T1 & T2
-    const combined = _.map(
-      schema.allOf.map((s) => _.merge({ required: schema.required }, s)),
-      complexTypeGetter,
-    );
+    const combined = _.map(schema.allOf.map(makeAddRequiredToChildSchema(schema)), complexTypeGetter);
     return checkAndAddNull(
       schema,
       filterContents(combined, [...JS_EMPTY_TYPES, ...JS_PRIMITIVE_TYPES, TS_KEYWORDS.ANY]).join(
@@ -203,10 +238,7 @@ const complexSchemaParsers = {
   },
   [SCHEMA_TYPES.COMPLEX_ANY_OF]: (schema) => {
     // T1 | T2 | (T1 & T2)
-    const combined = _.map(
-      schema.anyOf.map((s) => _.merge({ required: schema.required }, s)),
-      complexTypeGetter,
-    );
+    const combined = _.map(makeAddRequiredToChildSchema(schema), complexTypeGetter);
     const nonEmptyTypesCombined = filterContents(combined, [
       ...JS_EMPTY_TYPES,
       ...JS_PRIMITIVE_TYPES,

tests/generated/v3.0/full-swagger-scheme.ts

@@ -10636,9 +10636,41 @@ export class Api<SecurityDataType extends unknown> extends HttpClient<SecurityDa
      */
     gistsUpdate: (
       gistId: string,
-      data: (any | any | null) & {
+      data: (
+        | { description: string }
+        | {
+            files: Record<
+              string,
+              (
+                | { content: string }
+                | { filename: string | null }
+                | object
+                | ({ content: string } & { filename: string | null } & object)
+              ) & { content?: string; filename?: string | null }
+            >;
+          }
+        | ({ description: string } & {
+            files: Record<
+              string,
+              (
+                | { content: string }
+                | { filename: string | null }
+                | object
+                | ({ content: string } & { filename: string | null } & object)
+              ) & { content?: string; filename?: string | null }
+            >;
+          })
+      ) & {
         description?: string;
-        files?: Record<string, (any | any | object | null) & { content?: string; filename?: string | null }>;
+        files?: Record<
+          string,
+          (
+            | { content: string }
+            | { filename: string | null }
+            | object
+            | ({ content: string } & { filename: string | null } & object)
+          ) & { content?: string; filename?: string | null }
+        >;
       },
       params: RequestParams = {},
     ) =>
@@ -15943,17 +15975,29 @@ export class Api<SecurityDataType extends unknown> extends HttpClient<SecurityDa
       owner: string,
       repo: string,
       data: (
-        | { status?: "completed"; [key: string]: any }
-        | { status?: "queued" | "in_progress"; [key: string]: any }
-        | ({ status?: "completed"; [key: string]: any } & { status?: "queued" | "in_progress"; [key: string]: any })
+        | {
+            status?: "completed";
+            conclusion: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
+            name: string;
+            head_sha: string;
+            [key: string]: any;
+          }
+        | { status?: "queued" | "in_progress"; name: string; head_sha: string; [key: string]: any }
+        | ({
+            status?: "completed";
+            conclusion: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
+            name: string;
+            head_sha: string;
+            [key: string]: any;
+          } & { status?: "queued" | "in_progress"; name: string; head_sha: string; [key: string]: any })
       ) & {
-        name?: string;
+        name: string;
         head_sha: string;
         details_url?: string;
         external_id?: string;
         status?: "queued" | "in_progress" | "completed";
         started_at?: string;
-        conclusion: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
+        conclusion?: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
         completed_at?: string;
         output?: {
           title: string;
@@ -16014,9 +16058,17 @@ export class Api<SecurityDataType extends unknown> extends HttpClient<SecurityDa
       repo: string,
       checkRunId: number,
       data: (
-        | { status?: "completed"; [key: string]: any }
+        | {
+            status?: "completed";
+            conclusion: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
+            [key: string]: any;
+          }
         | { status?: "queued" | "in_progress"; [key: string]: any }
-        | ({ status?: "completed"; [key: string]: any } & { status?: "queued" | "in_progress"; [key: string]: any })
+        | ({
+            status?: "completed";
+            conclusion: "success" | "failure" | "neutral" | "cancelled" | "skipped" | "timed_out" | "action_required";
+            [key: string]: any;
+          } & { status?: "queued" | "in_progress"; [key: string]: any })
       ) & {
         name?: string;
         details_url?: string;