fix: reduce dependency missing keyword required

Author: sagold

src/compileSchema.reduceSchema.test.ts

@@ -92,6 +92,16 @@ describe("compileSchema : reduceNode", () => {
             assert.deepEqual(reduced.dynamicId, "#/$defs/schema(dependencies/one,dependencies/two)");
         });
 
+        it("should add required-property from dependency", () => {
+            const node: any = compileSchema({
+                type: "object",
+                properties: { one: { title: "Property One", type: "string" } },
+                dependencies: { one: { required: ["two"], properties: { two: { type: "string" } } } }
+            });
+            const { node: reduced } = node.reduceNode({ one: "" });
+            assert.deepEqual(reduced.schema.required, ["two"]);
+        });
+
         it("should NOT merge dependency when it is not defined", () => {
             const node: any = compileSchema({
                 type: "object",

src/keywords/dependencies.ts

@@ -79,6 +79,10 @@ export function reduceDependencies({ node, data, key, pointer, path }: JsonSchem
                 return true;
             }
 
+            if (Array.isArray(dependency.schema.required)) {
+                required.push(...dependency.schema.required);
+            }
+
             // @note pass on updated required-list to resolve nested dependencies. This is currently supported,
             // but probably not how json-schema spec defines this behaviour (resolve only within sub-schema)
             const reducedDependency = { ...dependency, schema: { ...dependency.schema, required } }.reduceNode(data, {
@@ -88,9 +92,6 @@ export function reduceDependencies({ node, data, key, pointer, path }: JsonSchem
             }).node;
 
             workingNode = mergeNode(workingNode, reducedDependency);
-            if (workingNode.schema.required) {
-                required.push(...workingNode.schema.required);
-            }
 
             // @dynamicId
             const nestedDynamicId = reducedDependency.dynamicId?.replace(node.dynamicId, "") ?? "";
@@ -103,10 +104,6 @@ export function reduceDependencies({ node, data, key, pointer, path }: JsonSchem
         return node;
     }
 
-    // mergedSchema = mergeSchema(node.schema, mergedSchema, "dependencies");
-    // const { node: childNode, error } = node.compileSchema(mergedSchema, node.evaluationPath).reduceNode(data, { path });
-    // return childNode ?? error;
-
     if (required.length === 0) {
         return workingNode;
     }