@stoplight/json-ref-resolver - npm Package Compare versions

Comparing version 3.0.4 to 3.0.5

crawler.d.ts

@@ -0,4 +1,6 @@
+import { Optional } from '@stoplight/types';
 import { DepGraph } from 'dependency-graph';
 import * as Types from './types';
 export declare class ResolveCrawler implements Types.ICrawler {
+    private _resolved;
     readonly resolvers: Array<Promise<Types.IUriResult>>;
@@ -9,5 +11,5 @@ jsonPointer?: string;
     private _runner;
-    constructor(runner: Types.IResolveRunner, jsonPointer?: string);
+    constructor(runner: Types.IResolveRunner, jsonPointer: Optional<string>, _resolved: Types.IResolveResult);
     computeGraph: (target: any, parentPath?: string[], parentPointer?: string, pointerStack?: string[]) => void;
     private _resolveRef;
 }

crawler.js

@@ -8,3 +8,4 @@ "use strict";
 class ResolveCrawler {
-    constructor(runner, jsonPointer) {
+    constructor(runner, jsonPointer, _resolved) {
+        this._resolved = _resolved;
         this.resolvers = [];
@@ -68,3 +69,17 @@ this.pointerGraph = new dependency_graph_1.DepGraph({ circular: true });
                     const targetPointer = Utils.uriToJSONPointer(ref);
-                    const targetPath = json_1.pointerToPath(targetPointer);
+                    let targetPath;
+                    try {
+                        targetPath = json_1.pointerToPath(targetPointer);
+                    }
+                    catch (_a) {
+                        this._resolved.errors.push({
+                            code: 'PARSE_POINTER',
+                            message: `'${ref}' JSON pointer is invalid`,
+                            uri: this._runner.baseUri,
+                            uriStack: this._runner.uriStack,
+                            pointerStack: [],
+                            path: [],
+                        });
+                        return;
+                    }
                     let referencesParent = targetPath.length > 0;
@@ -71,0 +86,0 @@ for (const i in targetPath) {

package.json

 {
   "name": "@stoplight/json-ref-resolver",
-  "version": "3.0.4",
+  "version": "3.0.5",
   "description": "Recursively resolve JSON pointers and remote authorities.",
@@ -5,0 +5,0 @@ "keywords": [

runner.js

@@ -254,3 +254,3 @@ "use strict";
             }
-            const crawler = new crawler_1.ResolveCrawler(this, jsonPointer);
+            const crawler = new crawler_1.ResolveCrawler(this, jsonPointer, resolved);
             crawler.computeGraph(resolved.result, targetPath, jsonPointer || '');
@@ -257,0 +257,0 @@ let uriResults = [];

types.d.ts

@@ -117,2 +117,3 @@ /// <reference types="urijs" />
     depth: number;
+    uriStack: string[];
     baseUri: uri.URI;
@@ -119,0 +120,0 @@ graph: DepGraph<IGraphNodeData>;

No alert changes

Improved metrics

Total package byte prevSize

80445

increased by1.41%

Lines of code

901

increased by2.04%

No dependency changes