Merge pull request #154 from vim-denops/fix-mapping

🐛 Fix parse error on "Nvim builtin"

Author: lambdalisue

denops_std/mapping/mod.ts

@@ -118,7 +118,13 @@ export async function list(
 ): Promise<Mapping[]> {
   const mode = options.mode ?? "";
   const result = await fn.execute(denops, `${mode}map ${lhs}`) as string;
-  return result.split(/\r?\n/).filter((v) => v).map(parse);
+  return result.split(/\r?\n/).flatMap((v) => {
+    try {
+      return [parse(v)];
+    } catch {
+      return [];
+    }
+  }).filter((v) => v);
 }
 
 function forceArray<T>(v: T | T[]): T[] {

denops_std/mapping/parser_test.ts

@@ -105,3 +105,11 @@ Deno.test(`parse() throws an error for invalid record`, () => {
     "Failed to parse a mapping record",
   );
 });
+Deno.test(`parse() throws an error for "Nvim builtin"`, () => {
+  assertThrows(
+    () => {
+      parse("                 Nvim builtin");
+    },
+    "Failed to parse a mapping record",
+  );
+});