Print warning when unable to resolve instance location

jirutka · Jun 25, 2024 · 7978b6e · 7978b6e
1 parent acafb2b
commit 7978b6e
Showing 1 changed file with 14 additions and 7 deletions.
diff --git a/src/commands/validate.ts b/src/commands/validate.ts
@@ -247,11 +247,18 @@ function withInstanceLocation<T extends ValidationError>(
   errors: T[],
   file: ParsedFile,
 ): Required<T, 'instanceLocation'>[] {
-  return errors.map(err => ({
-    ...err,
-    instanceLocation: {
-      filename: file.filename,
-      ...file.locate(err.instancePath.split('/').slice(1).map(unescapeJsonPointer)),
-    },
-  }))
+  return errors.map(err => {
+    const location = file.locate(err.instancePath.split('/').slice(1).map(unescapeJsonPointer))
+    if (!location) {
+      // This shouldn't happen...
+      console.warn(`Warning: Unable to resolve instance location: ${err.instancePath}`)
+    }
+    return {
+      ...err,
+      instanceLocation: {
+        filename: file.filename,
+        ...location,
+      },
+    }
+  })
 }