fix(nested-definitions): add definitions availability for nested definitions

Author: jonluca

src/index.ts

@@ -31,16 +31,26 @@ const handleDefinition = async <T>(
 	if (type) {
 		// Walk just the definitions types
 		const walker = new Walker<T>();
-		await walker.loadSchema({ ...def, $schema: schema['$schema'] } as any, {
-			dereference: true,
-			cloneSchema: true,
-			dereferenceOptions: {
-				dereference: {
-					circular: 'ignore',
+		await walker.loadSchema(
+			{
+				definitions: schema['definitions'] || [],
+				...def,
+				$schema: schema['$schema'],
+			} as any,
+			{
+				dereference: true,
+				cloneSchema: true,
+				dereferenceOptions: {
+					dereference: {
+						circular: 'ignore',
+					},
 				},
-			},
-		});
+			}
+		);
 		await walker.walk(convertSchema, walker.vocabularies.DRAFT_07);
+		if ('definitions' in walker.rootSchema) {
+			delete (<any>walker.rootSchema).definitions;
+		}
 		return walker.rootSchema;
 	} else if (Array.isArray(def)) {
 		// if it's an array, we might want to reconstruct the type;

test/__snapshots__/dereference_schema.test.ts.snap

@@ -0,0 +1,52 @@
+// Vitest Snapshot v1
+
+exports[`throws an error when dereferecing fails 1`] = `
+{
+  "additionalProperties": false,
+  "definitions": {
+    "configvariable": {
+      "additionalProperties": false,
+      "properties": {
+        "default": {
+          "type": "string",
+        },
+        "name": {
+          "pattern": "^[A-Z_]+[A-Z0-9_]*$",
+          "type": "string",
+        },
+        "required": {
+          "default": true,
+          "type": "boolean",
+        },
+      },
+      "required": [
+        "name",
+      ],
+      "type": "object",
+    },
+    "envVarName": {
+      "pattern": "^[A-Z_]+[A-Z0-9_]*$",
+      "type": "string",
+    },
+  },
+  "properties": {
+    "componentId": {
+      "pattern": "^(.*)$",
+      "title": "The component id Schema",
+      "type": "string",
+    },
+    "configurationTemplate": {
+      "items": {
+        "$ref": "#/definitions/configvariable",
+      },
+      "title": "The Configurationtemplate Schema",
+      "type": "array",
+    },
+  },
+  "required": [
+    "componentId",
+  ],
+  "title": "Component Manifest Schema",
+  "type": "object",
+}
+`;

test/dereference_schema.test.ts

@@ -264,3 +264,50 @@ it('throws an error when dereferecing fails', async ({ expect }) => {
 
 	expect(error).have.property('ioErrorCode', 'ENOENT');
 });
+
+it('throws an error when dereferecing fails', async ({ expect }) => {
+	const schema = {
+		definitions: {
+			envVarName: {
+				type: 'string',
+				pattern: '^[A-Z_]+[A-Z0-9_]*$',
+			},
+			configvariable: {
+				type: 'object',
+				properties: {
+					name: { $ref: '#/definitions/envVarName' },
+					default: { type: 'string' },
+					required: { type: 'boolean', default: true },
+				},
+				required: ['name'],
+				additionalProperties: false,
+			},
+		},
+		$schema: 'http://json-schema.org/draft-07/schema#',
+		$id: 'http://example.com/root.json',
+		type: 'object',
+		title: 'Component Manifest Schema',
+		required: ['componentId'],
+		additionalProperties: false,
+		properties: {
+			componentId: {
+				$id: '#/properties/componentId',
+				type: 'string',
+				title: 'The component id Schema',
+				pattern: '^(.*)$',
+			},
+			configurationTemplate: {
+				$id: '#/properties/configurationTemplate',
+				type: 'array',
+				title: 'The Configurationtemplate Schema',
+				items: {
+					$ref: '#/definitions/configvariable',
+				},
+			},
+		},
+	};
+
+	const result = await convert(schema);
+
+	expect(result).toMatchSnapshot();
+});