only cache array result errors

Author: neverRare

src/array_result.ts

@@ -173,3 +173,7 @@ export function extractArrayResultError(
       }
   }
 }
+export function isArrayResult(errors: ReadonlyArray<unknown>): boolean {
+  return errors.length > 0 &&
+    errors.every((error) => error instanceof ArrayResultError);
+}

src/main.ts

@@ -12,7 +12,7 @@ if (typeof LIVE_RELOAD !== "undefined" && LIVE_RELOAD) {
 import { dictionary } from "../dictionary/dictionary.ts";
 import { asComment } from "../dictionary/misc.ts";
 import PROJECT_DATA from "../project_data.json" with { type: "json" };
-import { ArrayResultError } from "./array_result.ts";
+import { ArrayResultError, isArrayResult } from "./array_result.ts";
 import { loadCustomDictionary } from "./dictionary.ts";
 import { checkLocalStorage, setIgnoreError } from "./local_storage.ts";
 import { flattenError } from "../misc/misc.ts";
@@ -144,7 +144,7 @@ function main(): void {
     try {
       loadCustomDictionary(customDictionary);
     } catch (error) {
-      errorDisplay.innerText = errorsFixable(flattenError(error))
+      errorDisplay.innerText = isArrayResult(flattenError(error))
         ? DICTIONARY_LOADING_FAILED_FIXABLE_MESSAGE
         : DICTIONARY_LOADING_FAILED_UNFIXABLE_MESSAGE;
       // deno-lint-ignore no-console
@@ -276,7 +276,7 @@ function main(): void {
       customDictionaryDialogBox.close();
     } catch (error) {
       const errors = flattenError(error);
-      const message = errorsFixable(errors)
+      const message = isArrayResult(errors)
         ? DICTIONARY_ERROR_FIXABLE_MESSAGE
         : DICTIONARY_ERROR_UNFIXABLE_MESSAGE;
       const errorListMessage = errors
@@ -300,10 +300,6 @@ function extractErrorMessage(error: unknown): string {
     return `${error}`;
   }
 }
-function errorsFixable(errors: ReadonlyArray<unknown>): boolean {
-  return errors.length > 0 &&
-    errors.every((error) => error instanceof ArrayResultError);
-}
 
 if (document.readyState === "loading") {
   document.addEventListener("DOMContentLoaded", main);

src/parser/parser_lib.ts

@@ -1,8 +1,12 @@
 import { assert } from "@std/assert/assert";
 import { MemoizationCacheResult, memoize } from "@std/cache/memoize";
-import { ArrayResult, ArrayResultError } from "../array_result.ts";
+import {
+  ArrayResult,
+  ArrayResultError,
+  isArrayResult,
+} from "../array_result.ts";
 import { Clearable, ClearableCacheSet, Lazy } from "../cache.ts";
-import { throwError } from "../../misc/misc.ts";
+import { flattenError, throwError } from "../../misc/misc.ts";
 
 export type ValueRest<T> = Readonly<{ rest: string; value: T }>;
 export type ParserResult<T> = ArrayResult<ValueRest<T>>;
@@ -18,7 +22,7 @@ export class Parser<T> {
       Parser.addToCache(cache);
       this.rawParser = memoize(this.nonMemoizedParser, {
         cache,
-        errorIsCacheable: () => true,
+        errorIsCacheable: (error) => isArrayResult(flattenError(error)),
       });
     } else {
       this.rawParser = this.nonMemoizedParser;