Skip to content

feat: add support for options.dereference.onDereference #281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 12, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions .github/workflows/CI-CD.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ on:
branches:
- main
pull_request:
schedule:
- cron: "0 0 1 * *"

jobs:
node_tests:
Expand Down
7 changes: 5 additions & 2 deletions docs/options.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,9 @@ $RefParser.dereference("my-schema.yaml", {
dereference: {
circular: false, // Don't allow circular $refs
excludedPathMatcher: (path) => // Skip dereferencing content under any 'example' key
path.includes("/example/")
path.includes("/example/"),
onDereference: (path, value) => // Callback invoked during dereferencing
console.log(path, value)
}
});
```
Expand Down Expand Up @@ -77,4 +79,5 @@ The `dereference` options control how JSON Schema $Ref Parser will dereference `
|Option(s) |Type |Description
|:---------------------|:-------------------|:------------
|`circular`|`boolean` or `"ignore"`|Determines whether [circular `$ref` pointers](README.md#circular-refs) are handled.<br><br>If set to `false`, then a `ReferenceError` will be thrown if the schema contains any circular references.<br><br> If set to `"ignore"`, then circular references will simply be ignored. No error will be thrown, but the [`$Refs.circular`](refs.md#circular) property will still be set to `true`.
|`excludedPathMatcher`|`(string) => boolean`|A function, called for each path, which can return true to stop this path and all subpaths from being dereferenced further. This is useful in schemas where some subpaths contain literal $ref keys that should not be dereferenced.
|`excludedPathMatcher`|`(string) => boolean`|A function, called for each path, which can return true to stop this path and all subpaths from being dereferenced further. This is useful in schemas where some subpaths contain literal `$ref` keys that should not be dereferenced.
|`onDereference`|`(string, JSONSchemaObjectType) => void`|A function, called immediately after dereferencing, with the resolved JSON Schema value and the `$ref` being dereferenced.
3 changes: 3 additions & 0 deletions lib/dereference.js
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,9 @@ function crawl (obj, path, pathFromRoot, parents, processedObjects, dereferenced
// Avoid pointless mutations; breaks frozen objects to no profit
if (obj[key] !== dereferenced.value) {
obj[key] = dereferenced.value;
if (options.dereference.onDereference) {
options.dereference.onDereference(value.$ref, obj[key]);
}
}
}
else {
Expand Down
11 changes: 10 additions & 1 deletion lib/index.d.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { JSONSchema4, JSONSchema4Type, JSONSchema6, JSONSchema6Type, JSONSchema7, JSONSchema7Type } from "json-schema";
import { JSONSchema4, JSONSchema4Object, JSONSchema4Type, JSONSchema6, JSONSchema6Object, JSONSchema6Type, JSONSchema7, JSONSchema7Object, JSONSchema7Type } from "json-schema";

export = $RefParser;

Expand Down Expand Up @@ -174,6 +174,7 @@ declare class $RefParser {
declare namespace $RefParser {

export type JSONSchema = JSONSchema4 | JSONSchema6 | JSONSchema7;
export type JSONSchemaObject = JSONSchema4Object | JSONSchema6Object | JSONSchema7Object;
export type SchemaCallback = (err: Error | null, schema?: JSONSchema) => any;
export type $RefsCallback = (err: Error | null, $refs?: $Refs) => any;

Expand Down Expand Up @@ -238,6 +239,14 @@ declare namespace $RefParser {
* subpaths contain literal $ref keys that should not be dereferenced.
*/
excludedPathMatcher?(path: string): boolean;

/**
* Callback invoked during dereferencing.
*
* @argument {string} path The path being dereferenced (ie. the `$ref` string).
* @argument {JSONSchemaObject} object The JSON-Schema that the `$ref` resolved to.
*/
onDereference(path: string, value: JSONSchemaObject): void;
};
}

Expand Down
26 changes: 26 additions & 0 deletions test/specs/dereference-callback/dereference-callback.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
"use strict";

const { expect } = require("chai");
const $RefParser = require("../../..");
const { rel } = require("../../utils/path");

describe("Schema with a $ref", () => {
it("should call onDereference", async () => {
let parser = new $RefParser();
const calls = [];
await parser.dereference(
rel("specs/dereference-callback/dereference-callback.yaml"),
{
dereference: {
onDereference (path, object) {
calls.push({ path, object });
},
},
}
);
expect(calls).to.deep.equal([
{ path: "#/definitions/b", object: { $ref: "#/definitions/a" }},
{ path: "#/definitions/a", object: { $ref: "#/definitions/a" }},
]);
});
});
12 changes: 12 additions & 0 deletions test/specs/dereference-callback/dereference-callback.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
title: test
type: object
definitions:
a:
$ref: "#/definitions/b"
b:
$ref: "#/definitions/a"
properties:
c:
type: string
d:
$ref: "#/definitions/a"