fix: show error when yank/paste on chrome (#93)

script/build

@@ -12,6 +12,7 @@ const buildScripts = async (browser) => {
   await build({
     define: {
       'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV),
+      'process.env.BROWSER': JSON.stringify(browser),
     },
     entryPoints: {
       console: "src/console/index.tsx",

src/background/controllers/OperationController.ts

@@ -21,6 +21,6 @@ export default class OperationController {
       props: Record<string, string | number | boolean>;
     }
   ): Promise<void> {
-    this.operationUseCase.run(ctx, name, props, repeat);
+    await this.operationUseCase.run(ctx, name, props, repeat);
   }
 }

src/background/di.ts

@@ -12,7 +12,10 @@ import { FindClientImpl } from "./clients/FindClient";
 import { ConsoleFrameClientImpl } from "./clients/ConsoleFrameClient";
 import { FindRepositoryImpl } from "./repositories/FindRepository";
 import { ReadyFrameRepositoryImpl } from "./repositories/ReadyFrameRepository";
-import { ClipboardRepositoryImpl } from "./repositories/ClipboardRepository";
+import {
+  FirefoxClipboardRepositoryImpl,
+  ChromeClipboardRepositoryImpl,
+} from "./repositories/ClipboardRepository";
 import { PropertySettingsImpl } from "./settings/PropertySettings";
 import { SearchEngineSettingsImpl } from "./settings/SearchEngineSettings";
 import { TransientSettingsRepotiory } from "./settings/SettingsRepository";
@@ -43,7 +46,11 @@ container.bind("NavigateClient").to(NavigateClientImpl);
 container.bind("ConsoleClient").to(ConsoleClientImpl);
 container.bind("ConsoleFrameClient").to(ConsoleFrameClientImpl);
 container.bind("ReadyFrameRepository").to(ReadyFrameRepositoryImpl);
-container.bind("ClipboardRepository").to(ClipboardRepositoryImpl);
+if (process.env.BROWSER === "firefox") {
+  container.bind("ClipboardRepository").to(FirefoxClipboardRepositoryImpl);
+} else {
+  container.bind("ClipboardRepository").to(ChromeClipboardRepositoryImpl);
+}
 container.bind("PropertySettings").to(PropertySettingsImpl);
 container.bind("SearchEngineSettings").to(SearchEngineSettingsImpl);
 container.bind("KeyCaptureClient").to(KeyCaptureClientImpl);

src/background/messaging/BackgroundMessageListener.ts

@@ -6,6 +6,7 @@ import OperationController from "../controllers/OperationController";
 import KeyController from "../controllers/KeyController";
 import ConsoleController from "../controllers/ConsoleController";
 import FindController from "../controllers/FindController";
+import ConsoleClient from "../clients/ConsoleClient";
 import { ReceiverWithContext } from "../../messaging";
 import type { Schema } from "../../messaging/schema/background";
 import RequestContext from "./RequestContext";
@@ -31,7 +32,9 @@ export default class BackgroundMessageListener {
     @inject(ConsoleController)
     consoleController: ConsoleController,
     @inject(FindController)
-    findController: FindController
+    findController: FindController,
+    @inject("ConsoleClient")
+    private readonly consoleClient: ConsoleClient
   ) {
     this.receiver
       .route("background.operation")
@@ -93,18 +96,19 @@ export default class BackgroundMessageListener {
           const style = "background-color: purple; color: white; padding: 4px;";
           console.debug("%cRECEIVE%c %s %o", style, "", type, args);
         }
-        const ret = this.receiver.receive(ctx, type, args);
-        Promise.resolve(ret)
+
+        Promise.resolve()
+          .then(() => this.receiver.receive(ctx, type, args))
           .then(sendResponse)
           .catch((err) => {
             console.error(err);
             if (!sender.tab || !sender.tab.id) {
               return;
             }
-            chrome.tabs.sendMessage(sender.tab.id, {
-              type: "console.show.error",
-              text: err.message,
-            });
+            if (typeof err.message !== "string") {
+              return;
+            }
+            this.consoleClient.showError(sender.tab.id, err.message);
           });
         return true;
       }

src/background/repositories/ClipboardRepository.ts

@@ -7,7 +7,7 @@ export default interface ClipboardRepository {
 }
 
 @injectable()
-export class ClipboardRepositoryImpl implements ClipboardRepository {
+export class FirefoxClipboardRepositoryImpl implements ClipboardRepository {
   async read(): Promise<string> {
     const value = await navigator.clipboard.readText();
     return value;
@@ -17,3 +17,14 @@ export class ClipboardRepositoryImpl implements ClipboardRepository {
     await navigator.clipboard.writeText(value);
   }
 }
+
+@injectable()
+export class ChromeClipboardRepositoryImpl implements ClipboardRepository {
+  async read(): Promise<string> {
+    throw new Error("Chrome does not support clipboard access");
+  }
+
+  async write(_value: string): Promise<void> {
+    throw new Error("Chrome does not support clipboard access");
+  }
+}

src/manifest.chrome.json

@@ -23,8 +23,6 @@
     "sessions",
     "storage",
     "tabs",
-    "clipboardRead",
-    "clipboardWrite",
     "notifications",
     "bookmarks"
   ],