feat: skip matcher when query is empty

Author: lambdalisue

builtin/matcher/fzf.ts

@@ -35,8 +35,13 @@ export function fzf(options: FzfOptions = {}): Matcher {
   const forward = options.forward ?? true;
 
   return defineMatcher(async function* (_denops, { items, query }, { signal }) {
+    if (query.trim() === "") {
+      yield* items;
+      return;
+    }
+
     // Split query into individual terms, ignoring empty strings
-    const terms = query.split(/\s+/).filter((v) => v.length > 0);
+    const terms = query.trim().split(/\s+/).filter((v) => v.length > 0);
 
     // deno-lint-ignore no-explicit-any
     const filter = async (items: readonly IdItem<any>[], term: string) => {

builtin/matcher/regexp.ts

@@ -12,8 +12,13 @@ import { getByteLength } from "../../util/stringutil.ts";
  */
 export function regexp(): Matcher {
   return defineMatcher(async function* (_denops, { query, items }, { signal }) {
+    if (query.trim() === "") {
+      yield* items;
+      return;
+    }
+
     // Create a RegExp from the query with global matching enabled
-    const pattern = new RegExp(query, "g");
+    const pattern = new RegExp(query.trim(), "g");
 
     // Iterate over each item, applying the regular expression
     for await (const item of items) {

builtin/matcher/substring.ts

@@ -46,9 +46,15 @@ export function substring(options: SubstringOptions = {}): Matcher {
 
   return defineMatcher(
     async function* (_denops, { query, items }, { signal }) {
+      if (query.trim() === "") {
+        yield* items;
+        return;
+      }
+
       const ignoreCase = shouldIgnoreCase(query);
       const norm = (v: string): string => (ignoreCase ? v.toLowerCase() : v);
       const terms = query
+        .trim()
         .split(/\s+/)
         .filter((v) => v.length > 0)
         .map(norm);