Use * pointer syntax for property names

Author: jdesrosiers

src/json-schema-errors.js

@@ -58,7 +58,8 @@ const constructErrorIndex = async (outputUnit, schema, errorIndex = {}) => {
 
     const absoluteKeywordLocation = errorOutputUnit.absoluteKeywordLocation
       ?? await toAbsoluteKeywordLocation(schema, /** @type string */ (errorOutputUnit.keywordLocation));
-    const instanceLocation = normalizeInstanceLocation(/** @type string */ (errorOutputUnit.instanceLocation));
+    const instanceLocation = /** @type string */ (errorOutputUnit.instanceLocation)
+      .replace(/^#?\*?/, "#");
 
     errorIndex[absoluteKeywordLocation] ??= {};
     errorIndex[absoluteKeywordLocation][instanceLocation] = true;
@@ -85,12 +86,6 @@ async function toAbsoluteKeywordLocation(schema, keywordLocation) {
   return `${schema.document.baseUri}#${schema.cursor}`;
 }
 
-/** @type {(location: string) => string} */
-function normalizeInstanceLocation(location) {
-  const instanceLocation = location.startsWith("/") || location === "" ? `#${location}` : location;
-  return instanceLocation.replace(/(#|^)\*\//, "$1/");
-}
-
 /** @type API.evaluateSchema */
 export const evaluateSchema = (schemaLocation, instance, context) => {
   const instanceLocation = Instance.uri(instance);
@@ -134,7 +129,7 @@ export const evaluateSchema = (schemaLocation, instance, context) => {
       }
 
       const keywordOutput = keyword.evaluate(keywordValue, instance, keywordContext);
-      const isKeywordValid = !context.errorIndex[keywordLocation]?.[instanceLocation];
+      const isKeywordValid = !context.errorIndex[keywordLocation]?.[instanceLocation.replace(/^#\*/, "#")];
       if (!isKeywordValid) {
         valid = false;
       }

src/normalization-handlers/propertyNames.js

@@ -16,7 +16,6 @@ const propertyNamesNormalizationHandler = {
     }
 
     for (const propertyName of Instance.keys(instance)) {
-      propertyName.pointer = propertyName.pointer.replace(/^\*/, "");
       outputs.push(evaluateSchema(propertyNames, propertyName, context));
     }
 

src/test-suite/tests/propertyNames.json

@@ -16,7 +16,7 @@
           "messageParams": {
             "minLength": "5"
           },
-          "instanceLocation": "#/foo",
+          "instanceLocation": "#*/foo",
           "schemaLocations": ["#/propertyNames/minLength"]
         }
       ]
@@ -34,7 +34,7 @@
           "messageParams": {
             "minLength": "5"
           },
-          "instanceLocation": "#/foo",
+          "instanceLocation": "#*/foo",
           "schemaLocations": ["#/propertyNames/minLength"]
         }
       ]