Merge pull request #212 from takker99:tree-shakable

perf: Make export identifiers tree-shakable
takker99 · Nov 12, 2024 · 865e7ff · 865e7ff
2 parents 80a0a11 + 24f5870
commit 865e7ff
Show file tree

Hide file tree

Showing 13 changed files with 324 additions and 278 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,7 +1,7 @@
 name: ci
 
 env:
-  DENO_VERSION: 1.x
+  DENO_VERSION: 2.x
 
 on: [push, pull_request]
 
@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: denoland/setup-deno@v1
+      - uses: denoland/setup-deno@v2
         with:
           deno-version: ${{ env.DENO_VERSION }}
       - name: Check fmt & lint & type check & test

diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -2,7 +2,7 @@
 name: publish
 
 env:
-  DENO_VERSION: 1.x
+  DENO_VERSION: 2.x
 
 on:
   push:
@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: denoland/setup-deno@v1
+      - uses: denoland/setup-deno@v2
         with:
           deno-version: ${{ env.DENO_VERSION }}
       - name: Publish on tag

diff --git a/browser/dom/getCachedLines.ts b/browser/dom/getCachedLines.ts
@@ -1,11 +1,14 @@
 import type { Line, Scrapbox } from "@cosense/types/userscript";
 declare const scrapbox: Scrapbox;
 
-let isLatestData = false;
-let lines: typeof scrapbox.Page.lines = null;
+let isLatestData = /* @__PURE__ */ false;
+let lines: Line[] | null = /* @__PURE__ */ null;
 
-scrapbox.addListener("lines:changed", () => isLatestData = false);
-scrapbox.addListener("layout:changed", () => isLatestData = false);
+let initialize: (() => void) | undefined = () => {
+  scrapbox.addListener("lines:changed", () => isLatestData = false);
+  scrapbox.addListener("layout:changed", () => isLatestData = false);
+  initialize = undefined;
+};
 
 /** scrapbox.Page.linesをcacheして取得する
  *
@@ -14,6 +17,7 @@ scrapbox.addListener("layout:changed", () => isLatestData = false);
  * @return `scrapbox.Page.lines`と同じ。常に最新のものが返される
  */
 export const getCachedLines = (): readonly Line[] | null => {
+  initialize?.();
   if (!isLatestData) {
     lines = scrapbox.Page.lines;
     isLatestData = true;

diff --git a/browser/dom/textInputEventListener.ts b/browser/dom/textInputEventListener.ts
@@ -7,11 +7,34 @@ declare const scrapbox: Scrapbox;
  * - second key: listener
  * - value: encoded options
  */
-const listenerMap = new Map<
+const listenerMap = /* @__PURE__ */ new Map<
   keyof HTMLElementEventMap,
   Map<EventListener, Set<number>>
 >();
-const onceListenerMap = new Map<EventListener, Map<number, EventListener>>();
+const onceListenerMap = /* @__PURE__ */ new Map<
+  EventListener,
+  Map<number, EventListener>
+>();
+
+/** re-register event listeners when the layout changes */
+let reRegister: (() => void) | undefined = () => {
+  scrapbox.on("layout:changed", () => {
+    const textinput = textInput();
+    if (!textinput) return;
+    for (const [name, argMap] of listenerMap) {
+      for (const [listener, encodedOptions] of argMap) {
+        for (const encoded of encodedOptions) {
+          textinput.addEventListener(
+            name,
+            listener as EventListener,
+            decode(encoded),
+          );
+        }
+      }
+    }
+  });
+  reRegister = undefined;
+};
 
 /** `#text-input`に対してイベントリスナーを追加する
  *
@@ -30,6 +53,7 @@ export const addTextInputEventListener = <K extends keyof HTMLElementEventMap>(
   ) => unknown,
   options?: boolean | AddEventListenerOptions,
 ): void => {
+  reRegister?.();
   const argMap = listenerMap.get(name) ?? new Map<EventListener, Set<number>>();
   const encodedOptions = argMap.get(listener as EventListener) ?? new Set();
   if (encodedOptions.has(encode(options))) return;
@@ -62,30 +86,14 @@ export const addTextInputEventListener = <K extends keyof HTMLElementEventMap>(
   textinput.addEventListener<K>(name, listener, options);
 };
 
-// re-register event listeners when the layout changes
-scrapbox.on("layout:changed", () => {
-  const textinput = textInput();
-  if (!textinput) return;
-  for (const [name, argMap] of listenerMap) {
-    for (const [listener, encodedOptions] of argMap) {
-      for (const encoded of encodedOptions) {
-        textinput.addEventListener(
-          name,
-          listener as EventListener,
-          decode(encoded),
-        );
-      }
-    }
-  }
-});
-
 export const removeTextInputEventListener = <
   K extends keyof HTMLElementEventMap,
 >(
   name: K,
   listener: (event: HTMLElementEventMap[K]) => unknown,
   options?: boolean | AddEventListenerOptions,
 ): void => {
+  reRegister?.();
   const argMap = listenerMap.get(name);
   if (!argMap) return;
   const encodedOptions = argMap.get(listener as EventListener);