feat!: not parse invalid bracket notation (#175)

Author: dinwwwh

packages/client/src/openapi/bracket-notation.test.ts

@@ -17,8 +17,13 @@ describe('bracketNotation', () => {
     expect(serializer.parsePath('a[b]c[d')).toEqual(['a[b]c[d'])
     expect(serializer.parsePath('a[[b]]')).toEqual(['a', '[b]'])
     expect(serializer.parsePath('a\\[[b]]')).toEqual(['a[', 'b]'])
+    expect(serializer.parsePath('abc[]')).toEqual(['abc', ''])
+
     expect(serializer.parsePath('abc[def')).toEqual(['abc[def'])
-    expect(serializer.parsePath('abc[d][ef')).toEqual(['abc', 'd][ef'])
+    expect(serializer.parsePath('abc[d][ef')).toEqual(['abc[d][ef'])
+    expect(serializer.parsePath('abc[d][')).toEqual(['abc[d]['])
+    expect(serializer.parsePath('abc[')).toEqual(['abc['])
+    expect(serializer.parsePath('abc]')).toEqual(['abc]'])
   })
 
   it.each([

packages/client/src/openapi/bracket-notation.ts

@@ -112,20 +112,26 @@ export class BracketNotationSerializer {
   parsePath(path: string): string[] {
     const segments: string[] = []
 
+    let inBrackets = false
     let currentSegment = ''
     let backslashCount = 0
 
     for (let i = 0; i < path.length; i++) {
       const char = path[i]!
       const nextChar = path[i + 1]
 
-      if (char === ']' && (nextChar === undefined || nextChar === '[') && backslashCount % 2 === 0) {
+      if (inBrackets && char === ']' && (nextChar === undefined || nextChar === '[') && backslashCount % 2 === 0) {
+        if (nextChar === undefined) {
+          inBrackets = false
+        }
+
         segments.push(currentSegment)
         currentSegment = ''
         i++
       }
 
       else if (segments.length === 0 && char === '[' && backslashCount % 2 === 0) {
+        inBrackets = true
         segments.push(currentSegment)
         currentSegment = ''
       }
@@ -140,14 +146,7 @@ export class BracketNotationSerializer {
       }
     }
 
-    if (!segments.length) {
-      segments.push(currentSegment)
-    }
-    else if (currentSegment) {
-      segments[segments.length - 1] += segments.length === 1 ? `[${currentSegment}` : `][${currentSegment}`
-    }
-
-    return segments
+    return inBrackets || segments.length === 0 ? [path] : segments
   }
 }