Merge pull request #782 from jamietanna/fix/jsonpath

deps: upgrade jsonpathly to 3.0.0

Author: paulRbr

package-lock.json

@@ -101,7 +101,7 @@
     "axios": "^1.7.7",
     "chalk": "^5.3.0",
     "debug": "^4.3.7",
-    "jsonpathly": "^2.0.2",
+    "jsonpathly": "^3.0.0",
     "mergician": "^2.0.2",
     "open": "^10.1.0"
   }

package.json

@@ -86,12 +86,7 @@ export class Overlay {
 
     // Take last element from path (which is the thing to act
     // upon)
-    let thingToActUpon: number | string | undefined = explodedPath.pop()
-    // The last element (e.g. '"price"]' or '0]') contains a final ']'
-    // so we need to remove it AND we need to parse the element to
-    // transform the string in either a string or a number
-    thingToActUpon =
-      thingToActUpon === undefined ? '$' : (thingToActUpon = JSON.parse(thingToActUpon.slice(0, -1)) as number | string)
+    const thingToActUpon: number | string = this.pathEntryToKey(explodedPath.pop())
 
     // Reconstruct the stringified path expression targeting the parent
     const parentPath: string = explodedPath.join('][')
@@ -117,6 +112,17 @@ export class Overlay {
     return action.description ? `Action '${action.description}'` : 'Action'
   }
 
+  // The last path entry (e.g. "'price']" or '0]') contains a final
+  // ']' so we need to remove it AND we need to replace single quotes
+  // to double quotes AND FINALLY parse the element to transform the
+  // string in either a string or a number.
+  private pathEntryToKey(pathEntry: string | undefined): number | string {
+    if (pathEntry === undefined) {
+      return '$'
+    }
+    return JSON.parse(pathEntry.slice(0, -1).replaceAll(/(^'|'$)/g, '"'))
+  }
+
   private remove(parent: JSONSchema4Object, property_or_index: number | string): void {
     if (Array.isArray(parent)) {
       parent.splice(property_or_index as number, 1)