Revert "fix: support 2020-12 anchors and ref siblings"

This reverts commit f3f7e35.

Author: jonluca

lib/bundle.ts

@@ -97,7 +97,7 @@ function crawl<S extends object = JSONSchema, O extends ParserOptions<S> = Parse
 
   if (obj && typeof obj === "object" && !ArrayBuffer.isView(obj) && !isExcludedPath(pathFromRoot)) {
     const currentScopeBase = dynamicIdScope ? getSchemaBasePath(scopeBase, obj) : scopeBase;
-    if ($Ref.isAllowed$Ref(obj, options, dynamicIdScope)) {
+    if ($Ref.isAllowed$Ref(obj)) {
       inventory$Ref(parent, key, path, currentScopeBase, dynamicIdScope, pathFromRoot, indirections, inventory, $refs, options);
     } else {
       // Crawl the object in a specific order that's optimized for bundling.
@@ -122,7 +122,7 @@ function crawl<S extends object = JSONSchema, O extends ParserOptions<S> = Parse
         const keyPathFromRoot = Pointer.join(pathFromRoot, key);
         const value = obj[key];
 
-        if ($Ref.isAllowed$Ref(value, options, dynamicIdScope)) {
+        if ($Ref.isAllowed$Ref(value)) {
           const valueScopeBase = dynamicIdScope ? getSchemaBasePath(currentScopeBase, value) : currentScopeBase;
           inventory$Ref(
             obj,

lib/dereference.ts

@@ -94,7 +94,7 @@ function crawl<S extends object = JSONSchema, O extends ParserOptions<S> = Parse
       processedObjects.add(obj);
       const currentScopeBase = dynamicIdScope ? getSchemaBasePath(scopeBase, obj) : scopeBase;
 
-      if ($Ref.isAllowed$Ref(obj, options, dynamicIdScope)) {
+      if ($Ref.isAllowed$Ref(obj, options)) {
         dereferenced = dereference$Ref(
           obj,
           path,
@@ -125,7 +125,7 @@ function crawl<S extends object = JSONSchema, O extends ParserOptions<S> = Parse
           const value = obj[key];
           let circular;
 
-          if ($Ref.isAllowed$Ref(value, options, dynamicIdScope)) {
+          if ($Ref.isAllowed$Ref(value, options)) {
             const valueScopeBase = dynamicIdScope ? getSchemaBasePath(currentScopeBase, value) : currentScopeBase;
             dereferenced = dereference$Ref(
               value,
@@ -257,10 +257,18 @@ function dereference$Ref<S extends object = JSONSchema, O extends ParserOptions<
     // If the cached object however is _not_ circular and there are additional keys alongside our
     // `$ref` pointer here we should merge them back in and return that.
     if (!cache.circular) {
-      if (Object.keys($ref).length > 1) {
+      const refKeys = Object.keys($ref);
+      if (refKeys.length > 1) {
+        const extraKeys = {};
+        for (const key of refKeys) {
+          if (key !== "$ref" && !(key in cache.value)) {
+            // @ts-expect-error TS(7053): Element implicitly has an 'any' type because expre... Remove this comment to see the full error message
+            extraKeys[key] = $ref[key];
+          }
+        }
         return {
           circular: cache.circular,
-          value: $Ref.dereference($ref, cache.value, options, dynamicIdScope),
+          value: Object.assign({}, cache.value, extraKeys),
         };
       }
 
@@ -310,7 +318,7 @@ function dereference$Ref<S extends object = JSONSchema, O extends ParserOptions<
   }
 
   // Dereference the JSON reference
-  let dereferencedValue = $Ref.dereference($ref, pointer.value, options, dynamicIdScope);
+  let dereferencedValue = $Ref.dereference($ref, pointer.value, options);
 
   // Crawl the dereferenced value (unless it's circular)
   if (!circular) {

lib/pointer.ts

@@ -308,7 +308,7 @@ function resolveIf$Ref<S extends object = JSONSchema, O extends ParserOptions<S>
 ) {
   // Is the value a JSON reference? (and allowed?)
 
-  if ($Ref.isAllowed$Ref(pointer.value, options, pointer.$ref.dynamicIdScope)) {
+  if ($Ref.isAllowed$Ref(pointer.value, options)) {
     const resolutionBase = pointer.$ref.dynamicIdScope ? pointer.scopeBase : pointer.path;
     const $refPath = url.resolve(resolutionBase, pointer.value.$ref);
 
@@ -326,7 +326,7 @@ function resolveIf$Ref<S extends object = JSONSchema, O extends ParserOptions<S>
       if ($Ref.isExtended$Ref(pointer.value)) {
         // This JSON reference "extends" the resolved value, rather than simply pointing to it.
         // So the resolved path does NOT change.  Just the value does.
-        pointer.value = $Ref.dereference(pointer.value, resolved.value, options, pointer.$ref.dynamicIdScope);
+        pointer.value = $Ref.dereference(pointer.value, resolved.value, options);
         if (pointer.$ref.dynamicIdScope) {
           pointer.scopeBase = getSchemaBasePath(pointer.scopeBase, pointer.value);
         }

lib/ref.ts

@@ -199,18 +199,11 @@ class $Ref<S extends object = JSONSchema, O extends ParserOptions<S> = ParserOpt
    * @param options
    * @returns
    */
-  static isAllowed$Ref<S extends object = JSONSchema>(
-    value: unknown,
-    options?: ParserOptions<S>,
-    allowPlainNameFragments = false,
-  ) {
+  static isAllowed$Ref<S extends object = JSONSchema>(value: unknown, options?: ParserOptions<S>) {
     if (this.is$Ref(value)) {
       if (value.$ref.substring(0, 2) === "#/" || value.$ref === "#") {
         // It's a JSON Pointer reference, which is always allowed
         return true;
-      } else if (allowPlainNameFragments && value.$ref[0] === "#") {
-        // JSON Schema 2019-09+ allows plain-name fragments declared via $anchor/$dynamicAnchor
-        return true;
       } else if (value.$ref[0] !== "#" && (!options || options.resolve?.external)) {
         // It's an external reference, which is allowed by the options
         return true;
@@ -293,12 +286,7 @@ class $Ref<S extends object = JSONSchema, O extends ParserOptions<S> = ParserOpt
     $ref: $Ref<S, O>,
     resolvedValue: S,
     options?: O,
-    useSpecCompliantRefSiblings = false,
   ): S {
-    if ($Ref.isExtended$Ref($ref) && useSpecCompliantRefSiblings) {
-      return preserveRefSiblings($ref, resolvedValue) as S;
-    }
-
     if (resolvedValue && typeof resolvedValue === "object" && $Ref.isExtended$Ref($ref)) {
       const merged = {} as Partial<S>;
       for (const key of Object.keys($ref)) {
@@ -337,27 +325,6 @@ class $Ref<S extends object = JSONSchema, O extends ParserOptions<S> = ParserOpt
   }
 }
 
-function preserveRefSiblings<T>(refValue: Record<string, any>, resolvedValue: T): T {
-  const preserved = {} as Record<string, any>;
-  const existingAllOf = "allOf" in refValue ? refValue.allOf : undefined;
-
-  for (const key of Object.keys(refValue)) {
-    if (key !== "$ref" && key !== "allOf") {
-      preserved[key] = refValue[key];
-    }
-  }
-
-  const allOf = [resolvedValue];
-  if (Array.isArray(existingAllOf)) {
-    allOf.push(...existingAllOf);
-  } else if (existingAllOf !== undefined) {
-    allOf.push(existingAllOf);
-  }
-
-  preserved.allOf = allOf;
-  return preserved as T;
-}
-
 function deepMerge<T>(target: Partial<T>, source: Partial<T>): T {
   //return {...target, ...source};
 

lib/refs.ts

@@ -112,7 +112,7 @@ export default class $Refs<S extends object = JSONSchema, O extends ParserOption
    */
   _get$Ref(path: string) {
     path = url.resolve(this._root$Ref.path!, path);
-    return this._getRef(this._exactAliases[path] || path);
+    return this._getRef(path);
   }
 
   /**
@@ -149,15 +149,6 @@ export default class $Refs<S extends object = JSONSchema, O extends ParserOption
     return $ref;
   }
 
-  _addExactAlias(path: string, targetPath: string) {
-    if (!path || this._exactAliases[path]) {
-      return this._exactAliases[path];
-    }
-
-    this._exactAliases[path] = targetPath;
-    return targetPath;
-  }
-
   /**
    * Resolves the given JSON reference.
    *
@@ -169,14 +160,13 @@ export default class $Refs<S extends object = JSONSchema, O extends ParserOption
    */
   _resolve(path: string, pathFromRoot: string, options?: O) {
     const absPath = url.resolve(this._root$Ref.path!, path);
-    const canonicalPath = this._exactAliases[absPath] || absPath;
-    const $ref = this._getRef(canonicalPath);
+    const $ref = this._getRef(absPath);
 
     if (!$ref) {
       throw new Error(`Error resolving $ref pointer "${path}". \n"${url.stripHash(absPath)}" not found.`);
     }
 
-    return $ref.resolve(canonicalPath, options, path, pathFromRoot);
+    return $ref.resolve(absPath, options, path, pathFromRoot);
   }
 
   /**
@@ -189,8 +179,6 @@ export default class $Refs<S extends object = JSONSchema, O extends ParserOption
 
   _aliases: $RefsMap<S, O> = {};
 
-  _exactAliases: Record<string, string> = {};
-
   /**
    * The {@link $Ref} object that is the root of the JSON schema.
    *
@@ -209,7 +197,6 @@ export default class $Refs<S extends object = JSONSchema, O extends ParserOption
 
     this._$refs = {};
     this._aliases = {};
-    this._exactAliases = {};
 
     // @ts-ignore
     this._root$Ref = null;

lib/util/schema-resources.ts

@@ -38,26 +38,24 @@ export function registerSchemaResources<S extends object = JSONSchema, O extends
 
   const seen = new Set<object>();
 
-  const visit = (node: unknown, scopeBase: string, pointerTokens: string[]) => {
+  const visit = (node: unknown, scopeBase: string) => {
     if (!node || typeof node !== "object" || ArrayBuffer.isView(node) || seen.has(node)) {
       return;
     }
 
     seen.add(node);
 
     const nextScopeBase = getSchemaBasePath(scopeBase, node);
-    const resourcePointerTokens = nextScopeBase === scopeBase ? pointerTokens : [];
     if (nextScopeBase !== scopeBase) {
       $refs._addAlias(nextScopeBase, node as S, pathType, dynamicIdScope);
     }
-    registerAnchorAliases($refs, nextScopeBase, resourcePointerTokens, node);
 
     for (const key of Object.keys(node)) {
-      visit((node as Record<string, unknown>)[key], nextScopeBase, [...resourcePointerTokens, key]);
+      visit((node as Record<string, unknown>)[key], nextScopeBase);
     }
   };
 
-  visit(value, basePath, []);
+  visit(value, basePath);
 }
 
 function getSchemaId(value: unknown): string | undefined {
@@ -73,37 +71,3 @@ function getSchemaId(value: unknown): string | undefined {
 
   return undefined;
 }
-
-function registerAnchorAliases<S extends object = JSONSchema, O extends ParserOptions<S> = ParserOptions<S>>(
-  $refs: $Refs<S, O>,
-  scopeBase: string,
-  pointerTokens: string[],
-  value: unknown,
-) {
-  if (!value || typeof value !== "object" || ArrayBuffer.isView(value)) {
-    return;
-  }
-
-  const resourceBase = url.stripHash(scopeBase);
-  const targetPath = pointerTokens.length > 0 ? joinPointerPath(resourceBase, pointerTokens) : `${resourceBase}#`;
-  const anchors = [
-    (value as { $anchor?: unknown }).$anchor,
-    (value as { $dynamicAnchor?: unknown }).$dynamicAnchor,
-  ];
-
-  for (const anchor of anchors) {
-    if (typeof anchor === "string" && anchor.length > 0) {
-      $refs._addExactAlias(`${resourceBase}#${anchor}`, targetPath);
-    }
-  }
-}
-
-function joinPointerPath(basePath: string, tokens: string[]) {
-  let path = `${basePath}#`;
-
-  for (const token of tokens) {
-    path += `/${token.replace(/~/g, "~0").replace(/\//g, "~1")}`;
-  }
-
-  return path;
-}

test/specs/defs/dereferencedA.json

@@ -30,7 +30,7 @@
               }
             },
             {
-              "type": "object",
+              "type": "number",
               "format": "float"
             }
           ]
@@ -76,7 +76,7 @@
           }
         },
         {
-          "type": "object",
+          "type": "number",
           "format": "float"
         }
       ]
@@ -110,7 +110,7 @@
           }
         },
         {
-          "type": "object",
+          "type": "number",
           "format": "float"
         }
       ]