From 6d8d88879350c7a84613afa282b69f57be2ecc7c Mon Sep 17 00:00:00 2001
From: snipsnipsnip <snipsnipsnip@localhost.simplelogin.com>
Date: Thu, 23 Oct 2025 23:31:08 +0900
Subject: [PATCH 1/2] docs: tweak the redirect implementation in webpage

I've noticed that we can implement redirection without enabling
the `executeScript` option.
---
 .github/site/index.mjs | 28 ++++++++++++++++++++++++----
 1 file changed, 24 insertions(+), 4 deletions(-)

diff --git a/.github/site/index.mjs b/.github/site/index.mjs
index fdc8250..855382f 100644
--- a/.github/site/index.mjs
+++ b/.github/site/index.mjs
@@ -11,14 +11,32 @@ mermaid.initialize({ startOnLoad: false });
  * @param {string} path Absolute path (/ points to the repository root) to the resource
  * @returns {string} The URL to the resource
  */
-const urlFor = (type, path) => `https://github.com/exteditor/ghostbird/${type}/main/${encodeURIComponent(path.replace(/^[/]+/, ''))}`;
+const urlFor = (type, path) => `https://github.com/exteditor/ghostbird/${encodeURIComponent(type)}/main/${encodeURIComponent(path.replace(/^[/]+/, ''))}`;
 
 /**
  * Build a Markdown text that redirects to a URL
  * @param {string} url The URL to redirect to
- * @returns {string} a Markdown text
+ * @returns {string} A Markdown text
  */
-const redirectTo = (url) => `Redirecting to ${url}...\n\n<script>\nlocation.href = "${url}"${'</'}script>`;
+const redirectTo = (url) => JSON.stringify({redirectTo: url});
+
+/**
+ * Does redirect if the text is an instruction for it.
+ * @param {string} text A Markdown text that may contain a redirect instruction
+ * @returns {string} Returns the `text` as-is if it doesn't contain redirect instruction
+ */
+function tryRedirect(text) {
+  try {
+    if (text.startsWith('{"redirectTo":"')) {
+      const { redirectTo } = JSON.parse(text);
+      location.replace(redirectTo);
+      return "Redirecting...";
+    }
+  } catch (e) {
+    console.warn("failed to parse the redirect instruction", e);
+  }
+  return text;
+}
 
 /**
  * Add a footer to the Markdown text
@@ -46,7 +64,7 @@ window.$docsify = {
   logo: 'ext/blue.svg',
   formatUpdated: '{YYYY}-{MM}-{DD}',
   relativePath: true,
-  executeScript: true,
+  executeScript: false,
   homepage: "homepage.md",
   coverpage: "coverpage.md",
   loadNavbar: "navbar.md",
@@ -56,6 +74,7 @@ window.$docsify = {
   themeColor: '#0b9dd6',
   routes: {
     '/README': (route) => redirectTo(urlFor('blob', `${route}.md`)),
+    '/LICENSE': (route) => redirectTo(urlFor('blob', route)),
     '/[-._/a-zA-Z]*[.][a-zA-Z]+$': (route) => redirectTo(urlFor('blob', route)),
     '/[-._/a-zA-Z]+/$': (route) => redirectTo(urlFor('tree', route)),
   },
@@ -80,6 +99,7 @@ window.$docsify = {
   },
   plugins: [
     (hook, vm) => {
+      hook.beforeEach((text) => tryRedirect(text));
       hook.beforeEach((text) => addFooter(text, vm));
       hook.doneEach(() => mermaid.run());
     },

From 9e8b2156bf5c6c66c621be66662266ca7f751cf7 Mon Sep 17 00:00:00 2001
From: snipsnipsnip <snipsnipsnip@localhost.simplelogin.com>
Date: Fri, 24 Oct 2025 23:39:52 +0900
Subject: [PATCH 2/2] refactor: `await` at top-level for the main loop in
 `options.ts`

---
 src/app-options/options_event_router.ts |  2 +-
 src/root/options.ts                     | 22 +++++++++++++++-------
 2 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/app-options/options_event_router.ts b/src/app-options/options_event_router.ts
index 6a46114..6716f9f 100644
--- a/src/app-options/options_event_router.ts
+++ b/src/app-options/options_event_router.ts
@@ -5,7 +5,7 @@ export class OptionsEventRouter {
 
   constructor(readonly optionsStore: IOptionsStore) {}
 
-  initOptions(elem: GhostbirdOptionsElement): Promise<void> {
+  initOptions(elem: GhostbirdOptionsElement): Promise<never> {
     return elem.startSync(this.optionsStore)
   }
 }
diff --git a/src/root/options.ts b/src/root/options.ts
index 0f1ea87..ac3689f 100644
--- a/src/root/options.ts
+++ b/src/root/options.ts
@@ -1,10 +1,10 @@
 /** @file Entrypoint of the options page */
 
-import { GhostbirdOptionsElement } from "src/app-options/ghostbird_options_element"
-import { OptionsEventRouter } from "src/app-options/options_event_router"
+import { GhostbirdOptionsElement, OptionsEventRouter } from "src/app-options"
 import { default as optionsSyncCtor } from "webext-options-sync"
 import { startupOptions } from "./startup/startup_options"
 
+/** Initialize the event router for the options page */
 function prepareRouter(): OptionsEventRouter {
   const startup = startupOptions({
     optionsSyncCtor,
@@ -12,17 +12,25 @@ function prepareRouter(): OptionsEventRouter {
   return startup(OptionsEventRouter)
 }
 
-console.log("starting", import.meta.url)
+/** Wait for the options page to be ready and start processing events */
+async function runRouterAsync(): Promise<never> {
+  await customElements.whenDefined("ghostbird-options")
 
-customElements.whenDefined("ghostbird-options").then(async () => {
   const router = prepareRouter()
   let [opt] = document.getElementsByTagName("ghostbird-options")
   if (!opt) {
     throw Error("unexpected html state")
   }
-  await router.initOptions(opt as GhostbirdOptionsElement)
-})
+
+  return router.initOptions(opt as GhostbirdOptionsElement)
+}
+
+console.info("starting", import.meta.url)
+
+const mainPromise: Promise<never> = runRouterAsync()
 
 customElements.define("ghostbird-options", GhostbirdOptionsElement)
 
-console.log("started", import.meta.url)
+console.info("started", import.meta.url)
+
+await mainPromise