clarity-pattern-parser 3.0.10 → 3.0.12

package/src/patterns/value/RepeatValue.ts

@@ -76,6 +76,7 @@ export default class RepeatValue extends ValuePattern {
             this.nodes.push(node);
 
             if (node.endIndex === this.cursor.lastIndex()) {
+              this.nodes.length = 0;
               this._processMatch();
               break;
             }

package/src/tests/RepeatComposite.test.ts

@@ -43,4 +43,16 @@ describe("RepeatComposite", () => {
     expect(result?.children[2].value).toBe("B");
     expect(result?.children[2].name).toBe("b");
   });
+
+  test("Cannot end on a divider.", () => {
+    const a = new Literal("a", "A");
+    const b = new Literal("b", "B");
+    const space = new Literal("space", " ");
+    const or = new OrComposite("names", [a, b]);
+
+    const repeat = new RepeatComposite("repeat", or, space);
+    const result = repeat.parse(new Cursor("A B "));
+
+    expect(result).toBe(null);
+  });
 });

package/src/tests/TextSuggester.test.ts

@@ -5,6 +5,9 @@ import Literal from "../patterns/value/Literal";
 import AndValue from "../patterns/value/AndValue";
 import OrValue from "../patterns/value/OrValue";
 import json from "./javascriptPatterns/json";
+import OrComposite from "../patterns/composite/OrComposite";
+import RepeatComposite from "../patterns/composite/RepeatComposite";
+import Cursor from "../Cursor";
 
 describe("TextInspector", () => {
   test("Partial Match", () => {
@@ -214,4 +217,20 @@ describe("TextInspector", () => {
     expect(inspection.match?.text).toBe(`{"blah":0.9`);
     expect(inspection.isComplete).toBe(false);
   });
+
+  test("Suggest another item in the repeat.", () => {
+    const a = new Literal("a", "A");
+    const b = new Literal("b", "B");
+    const space = new Literal("space", " ");
+    const or = new OrComposite("names", [a, b]);
+
+    const repeat = new RepeatComposite("repeat", or, space);
+
+    const result = TextSuggester.suggest("A B ", repeat);
+
+    expect(result.isComplete).toBe(false);
+    expect(result.options.values[0]).toBe("A");
+    expect(result.options.values[1]).toBe("B");
+    expect(result.options.values.length).toBe(2);
+  });
 });