fix: splitting queries incorrectly removed quotes

This made it impossible to use any search queries with quotes in it. Closes #107
pvcnt · Sep 6, 2024 · cd90f2b · cd90f2b
1 parent 183f5ec
commit cd90f2b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/github/src/search.ts b/packages/github/src/search.ts
@@ -20,6 +20,7 @@ export function splitQueries(search: string): string[] {
       query = "";
     } else if (c === '"') {
       quoted = !quoted;
+      query += c;
     } else {
       query += c;
     }

diff --git a/packages/github/tests/search.test.ts b/packages/github/tests/search.test.ts
@@ -128,7 +128,8 @@ test("split a search string into queries", () => {
   expect(splitQueries("a")).toEqual(["a"]);
   expect(splitQueries("a;b")).toEqual(["a", "b"]);
   expect(splitQueries("   a ;  b ")).toEqual(["a", "b"]);
-  expect(splitQueries('"a;b";c')).toEqual(["a;b", "c"]);
+  expect(splitQueries(`"a"`)).toEqual([`"a"`]);
+  expect(splitQueries(`"a;b";c`)).toEqual([`"a;b"`, "c"]);
   expect(splitQueries("a; ;c")).toEqual(["a", "c"]);
 });