fix(textchecker-element): remove unused code

textlint · Aug 2, 2020 · 040724d · 040724d
1 parent 6625de3
commit 040724d
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 8 deletions.
diff --git a/packages/textchecker-element/src/text-checker-element.ts b/packages/textchecker-element/src/text-checker-element.ts
@@ -144,7 +144,7 @@ export class TextCheckerElement extends HTMLElement {
         //          non-visible = drop from `rectItems`
         const annotationItemsByDescendingOrder = annotationItems.slice().reverse();
         let stopSearchAboveIsNotVisible = false;
-        const rectItems = annotationItemsByDescendingOrder.flatMap((annotation, index) => {
+        const rectItems = annotationItemsByDescendingOrder.flatMap((annotation) => {
             // already the annotation is not visible, skip it
             if (stopSearchAboveIsNotVisible) {
                 return [];

diff --git a/packages/webextension/app/scripts/background/textlint.ts b/packages/webextension/app/scripts/background/textlint.ts
@@ -101,25 +101,40 @@ export const createTextlintWorker = (defaultWorkerUrl: string | URL = "download/
             } as TextlintWorkerCommandFix);
         });
     };
+    const log = (...args: any[]) => {
+        console.log("[Background]", ...args);
+    };
     return {
         createLintEngine({ ext }: { ext: string }) {
             const lintEngine: LintEngineAPI = {
-                lintText: ({ text }) => lintText({ text, ext }),
+                lintText: ({ text }) =>
+                    lintText({ text, ext }).then((result) => {
+                        log("lintText", result);
+                        return result;
+                    }),
                 fixText: async ({ text, message }): Promise<{ output: string }> => {
                     if (!message.fix || !message.fix.range) {
                         return { output: text };
                     }
                     // replace fix.range[0, 1] with fix.text
-                    return {
+                    const result = {
                         output:
                             text.slice(0, message.fix.range[0]) + message.fix.text + text.slice(message.fix.range[1])
                     };
+                    log("fixText", result);
+                    return result;
                 },
                 fixAll({ text }: { text: string }): Promise<TextlintFixResult> {
-                    return fixText({ text, ext });
+                    return fixText({ text, ext }).then((result) => {
+                        log("fixAll", result);
+                        return result;
+                    });
                 },
                 fixRule({ text, message }: { text: string; message: TextlintMessage }): Promise<TextlintFixResult> {
-                    return fixText({ text, message, ext });
+                    return fixText({ text, message, ext }).then((result) => {
+                        log("fixRule", result);
+                        return result;
+                    });
                 }
             };
             return lintEngine;

diff --git a/packages/webextension/app/scripts/contentScript.ts b/packages/webextension/app/scripts/contentScript.ts
@@ -1,5 +1,7 @@
-// @ts-ignore - replace webcomponent to shim
-import "@webcomponents/custom-elements";
+// Load webcomponents polyfill for Chrome Extension
+if (window.customElements === null || window.customElements === undefined) {
+    require("@webcomponents/custom-elements");
+}
 import { browser } from "webextension-polyfill-ts";
 import { attachToTextArea, LintEngineAPI } from "textchecker-element";
 import { createEndpoint } from "comlink-extension";

diff --git a/packages/webextension/tsconfig.json b/packages/webextension/tsconfig.json
@@ -8,7 +8,7 @@
     "outDir": "./lib/",
     "target": "ES2015",
     "sourceMap": true,
-    "declaration": true,
+    "declaration": false,
     "jsx": "preserve",
     "lib": [
       "esnext",