[高优先] 重新修订 `GM_registerMenuCommand` 相关代码设计 #790

cyfung1031 · 2025-10-02T10:57:19Z

概述

close #787 關連 #580

现有的设计有问题。在以下脚本可验证。
PR后，脚本的执行会跟TM完全一致。

问题有好几个，难以一一说明。你跑下面脚本看看就知道。

脚本中有三个可改动设定
checkSubFrameIdSequence：网页有iframe的话，改为true会改为检查子母frame菜单显示问题。
intervalChanging：预设false. 改成true后会进行每秒修改选单项目。
skipClickCheck: 预设中，脚本会一步一步引导用户执行及检查脚本运作是否跟TM一致。true的话不进行。

全false的话是检查最基本的项目。请在没有 iframe 的简单页面进行。打开 DevTools 即可理解要怎么测试。
然后改 checkSubFrameIdSequence 为 true, 查看一下 top window 和 iframe window 同时执行的情况。点一下菜单项目吧。
然后改 intervalChanging 为 true. 先在没有 iframe 的简单页面看一下。旧版本的话，popup菜单显示没有即时更新，要关一下开一下才行。这跟TM不一样。TM是即时更新。
然后右键点一下看看。我发现TM的右键菜单很正常但SC的右键菜单总是不显示 interval-n-xxxx
最终找到的解决方法是，浏览器contextMenu的项目id需要穏定。穏定后就解决问题了
最后是在有 iframe 的以 intervalChanging true 跑一下。因为top window 跟iframe window的时间有差异，所以菜单会一时四个一时两个（ interval-m-1001 跟 interval-m-1002 不合并）。这个在PR后也是跟TM一致了

这个PR把这些复杂的问题都全部解决了，完全跟TM一致

代码部份尽量加注释避免你看不懂。

// ==UserScript==
// @name         GM_registerMenuCommand Example
// @namespace    http://tampermonkey.net/
// @version      1.0
// @description  Simple demo for GM_registerMenuCommand
// @match        *://*/*
// @grant        GM_registerMenuCommand
// @grant        GM_unregisterMenuCommand
// ==/UserScript==

(async function () {
  'use strict';

  const checkSubFrameIdSequence = false;

  const intervalChanging = false;

  const skipClickCheck = false;

  let myResolve = () => { };
  const waitNext = async () => {
    await new Promise((resolve) => {
      myResolve = () => { setTimeout(resolve, 50) };
    });
  };

  const waitActions = async (...messages) => {
    if (skipClickCheck) return;
    messages = messages.flat();
    for (const message of messages) {
      console.log(message);
      await waitNext();
    }
  };

  const isInSubFrame = () => {

    try {
      return window.top !== window;
    } catch {
      return true;
    }
  }

  if (intervalChanging) {
    // TM: 在打开菜单时，显示会不断改变
    let i = 1000;
    let p = 0;
    setInterval(() => {
      if (p) GM_unregisterMenuCommand(p);
      i++;
      GM_registerMenuCommand(`interval-m-${i}`, () => { console.log(`${i}`); }, {id: "m"});
      p = GM_registerMenuCommand(`interval-n-${i}`, () => { console.log(`${i}`); });
    }, 1000);
    // return;
  }
  if (checkSubFrameIdSequence) {
    const key = Math.floor(Math.random() * 99999 + 99999).toString();

    let arr = [];
    arr.push(
      GM_registerMenuCommand("test", () => { console.log(`${key}-1`); }),
      GM_registerMenuCommand("test", () => { console.log(`${key}-2`); }),
      GM_registerMenuCommand("test", () => { console.log(`${key}-3`); })
    );
    if (isInSubFrame()) {
      arr.push(GM_registerMenuCommand("test-sub", () => { console.log(`${key}-sub`); }));
    } else {
      arr.push(GM_registerMenuCommand("test-main", () => { console.log(`${key}-main`); }));
    }
    arr.push(GM_registerMenuCommand(`test-${location.origin}`, () => { console.log(`${key}-origin`); }));
    console.log(`checkSubFrameIdSequence (key=${key}, frame=${isInSubFrame()})`, arr.join("..."));
    // return;
  }

  let obj1 = { id: "abc" };

  const r01 = GM_registerMenuCommand("MenuReg abc-1", () => {

    console.log("abc-1");
    myResolve();
  }, obj1);

  const r02 = GM_registerMenuCommand("MenuReg abc-2", () => {

    console.log("abc-2");
    myResolve();
  }, obj1);

  console.log("abc-1 id === abc", r01 === "abc");
  console.log("abc-2 id === abc", r02 === "abc");

  // there shall be only "MenuReg abc-2" in the menu.
  await waitActions("There shall be only 'MenuReg abc-2'. Click it to continue.");

  GM_registerMenuCommand("MenuReg abc-1", () => {

    console.log("abc-1.abd");
    myResolve();
  }, { id: "abd" });

  GM_registerMenuCommand("MenuReg abc-2", () => {

    console.log("abc-2.abe");
    myResolve();
  }, { id: "abe" });


  // there shall be only "MenuReg abc-1" and "MenuReg abc-2" in the menu.
  await waitActions("There shall be 'MenuReg abc-2' and 'MenuReg abc-1'. Click either them to continue.");


  GM_registerMenuCommand("MenuReg abc-2", () => {

    console.log("abc-2.abf");
    myResolve();
  }, { id: "abf", accessKey: "h" });

  // there shall be only "MenuReg abc-1" and "MenuReg abc-2" in the menu.
  await waitActions("There shall be 'MenuReg abc-2', 'MenuReg abc-1' and 'MenuReg abc-2 (H)'. Click either them to continue.");

  GM_unregisterMenuCommand("abc");
  GM_unregisterMenuCommand("abd");
  GM_unregisterMenuCommand("abe");
  GM_unregisterMenuCommand("abf");



  const p10 = GM_registerMenuCommand("MenuReg D-23", () => {

    console.log(110);
    myResolve();
  }, "b");


  const p20 = GM_registerMenuCommand("MenuReg D-23", () => {

    console.log(120);
    myResolve();
  }, "b");

  console.log("p10 === 1", p10 === 1);
  console.log("p20 === 2", p20 === 2);

  // MenuReg D-23 clicking shall give both 110 and 120
  await waitActions("Click [MenuReg D-23] -> 110, 120");


  const p30 = GM_registerMenuCommand("MenuReg D-26", () => {

    console.log(130);
    myResolve();
  }, { id: "2" });
  console.log("p30 === '2'", p30 === "2");

  // MenuReg D-23 clicking shall give 110
  // MenuReg D-26 clicking shall give 130

  await waitActions("Click [MenuReg D-23] -> 110", "Click [MenuReg D-26] -> 130");


  const p32 = GM_registerMenuCommand("MenuReg D-26", () => {

    console.log(210);
    myResolve();
  }, { id: 2 });
  console.log("p32 === 2", p32 === 2);

  // MenuReg D-23 clicking shall give 110
  // MenuReg D-26 clicking shall give 210

  await waitActions("Click [MenuReg D-23] -> 110", "Click [MenuReg D-26] -> 210");


  const p33 = GM_registerMenuCommand("MenuReg D-26", () => {

    console.log(220);
    myResolve();
  }, { id: 3 });
  console.log("p33 === 3", p33 === 3);

  // MenuReg D-23 clicking shall give 110
  // MenuReg D-26 clicking shall give 210 220

  await waitActions("Click [MenuReg D-23] -> 110", "Click [MenuReg D-26] -> 210, 220");



  const p34 = GM_registerMenuCommand("MenuReg D-26", () => {

    console.log(230);
    myResolve();
  }, { id: "4" });
  console.log("p34 === '4'", p34 === "4");

  // MenuReg D-23 clicking shall give 110
  // MenuReg D-26 clicking shall give 210 220 230
  await waitActions("Click [MenuReg D-23] -> 110", "Click [MenuReg D-26] -> 210, 220, 230");

  GM_unregisterMenuCommand("4");


  // MenuReg D-23 clicking shall give 110
  // MenuReg D-26 clicking shall give 210 220
  await waitActions("Click [MenuReg D-23] -> 110", "Click [MenuReg D-26] -> 210, 220");



  const p40 = GM_registerMenuCommand("MenuReg D-40", () => {

    console.log(601);
  });

  const p50 = GM_registerMenuCommand("MenuReg D-50", () => {

    console.log(602);
  });
  console.log("p40, p50", [p40, p50]); // TM gives 3&4



})().finally(() => {
  console.log("finish");
});

变更内容

重新修订 GM_registerMenuCommand 相关代码设计

截图

Copilot

Pull Request Overview

这个 PR 重新设计了 GM_registerMenuCommand 相关代码的架构，以提高与 Tampermonkey (TM) 的兼容性。重点解决了菜单项唯一性、显示更新以及跨页面/框架的同步问题。

主要变更：

重新设计菜单项的键值系统，使用稳定的 groupKey 来合并相同语义的菜单项
实现实时菜单更新机制，通过消息传递即时同步菜单状态变化
修正 Chrome contextMenu API 的稳定性问题，使用固定 UUID 池避免菜单 ID 冲突

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/pages/store/AppContext.tsx	改进消息订阅的类型安全性，添加泛型支持与清理优化
src/pages/popup/App.tsx	添加菜单实时更新监听机制，优化脚本列表排序逻辑
src/pages/options/routes/Setting.tsx	统一添加消息订阅清理逻辑，保持代码风格一致
src/pages/options/routes/ScriptList.tsx	为消息订阅添加类型注解，确保类型安全
src/pages/components/ScriptMenuList/index.tsx	重构菜单项分组逻辑，支持 groupKey 聚合与实时更新
src/app/service/service_worker/types.ts	定义新的菜单项类型结构，支持 groupKey 和唯一键系统
src/app/service/service_worker/popup.ts	核心菜单管理逻辑重写，实现稳定的 contextMenu 管理
src/app/service/service_worker/gm_api.ts	更新 GM API 参数类型，适配新的菜单键值系统
src/app/service/service_worker/client.ts	修改客户端接口以使用新的菜单键值参数
src/app/service/queue.ts	更新消息队列类型定义，支持新的菜单键值结构
src/app/service/content/gm_api.ts	重写内容脚本的菜单注册逻辑，支持环境隔离与唯一键
src/app/cache.ts	添加注释说明返回值的边界情况处理

Comments suppressed due to low confidence (1)

src/app/service/service_worker/popup.ts:1

这个注释实际上属于 src/app/cache.ts 文件，而不是 popup.ts。注释位置错误可能会误导开发者。

import { type IMessageQueue } from "@Packages/message/message_queue";

Copilot · 2025-10-03T09:00:20Z

src/pages/popup/App.tsx

+  //@ts-ignore
+  b.enable - a.enable ||


使用 //@ts-ignore 注释不是好的做法。应该明确处理类型问题，比如将 enable 转换为数字或使用条件表达式。

Suggested change

//@ts-ignore

b.enable - a.enable ||

Number(b.enable) - Number(a.enable) ||

Copilot · 2025-10-03T09:00:20Z

src/pages/popup/App.tsx

      url = new URL(currentUrl);
    } catch (_: any) {
-      // ignore error
+      // 容错：URL 解析失败时忽略错误（不影响后续 UI）


[nitpick] 注释中的"容错"应为"容错处理"或"错误处理"，以更准确地表达意图。

Suggested change

// 容错：URL 解析失败时忽略错误（不影响后续 UI）

// 容错处理：URL 解析失败时忽略错误（不影响后续 UI）

Copilot跟ChatGPT有不一样的看法呢

src/pages/popup/App.tsx

src/pages/components/ScriptMenuList/index.tsx

src/app/service/service_worker/popup.ts

Copilot · 2025-10-03T09:00:21Z

src/app/service/content/gm_api.ts

+    delete options.id; // id不直接储存在options (id 影响 groupKey 操作)
+    if (providedId === undefined) providedId = this.menuIdCounter! += 1; // 如无指定，使用累计器id
+    const ret = providedId;
+    // providedId = (typeof providedId === "number" ? `n${providedId}` : `t${providedId}`) as string;


有重复的注释掉的代码和相同的TODO注释。应该删除注释掉的代码，并考虑将TODO注释提取为统一的文档或issue。

Suggested change

// providedId = (typeof providedId === "number" ? `n${providedId}` : `t${providedId}`) as string;

将TODO注释提取为统一的文档或issue。

b779c49 3b6f520

Copilot · 2025-10-03T09:00:22Z

src/app/service/content/gm_api.ts

-    this.sendMessage("GM_unregisterMenuCommand", [id]);
+  GM_unregisterMenuCommand(menuId: TScriptMenuItemID): void {
+    execEnvInit(this);
+    // let menuKey = (typeof menuId === "number" ? `n${menuId}` : `t${menuId}`) as string;


有重复的注释掉的代码和相同的TODO注释。应该删除注释掉的代码，并考虑将TODO注释提取为统一的文档或issue。

Suggested change

// let menuKey = (typeof menuId === "number" ? `n${menuId}` : `t${menuId}`) as string;

将TODO注释提取为统一的文档或issue。

b779c49 3b6f520

Co-authored-by: Copilot <[email protected]>

…and_codefix

cyfung1031 · 2025-10-07T09:52:54Z

先记一下：未有考虑 ServiceWorker 热重启问题。 contextMenuIds 要储 chrome.storage.session

或固定ID？再看看

…and_codefix

Copilot

Pull Request Overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (2)

src/app/service/content/gm_api.ts:1

应该避免使用 @ts-ignore，建议使用适当的类型转换或修正类型定义以消除类型错误。

import type { Message, MessageConnect } from "@Packages/message/types";

src/app/service/content/gm_api.ts:1

应该避免使用 @ts-ignore，建议使用适当的类型转换或修正类型定义以消除类型错误。

import type { Message, MessageConnect } from "@Packages/message/types";

src/pages/components/ScriptMenuList/index.tsx

CodFrm · 2025-10-09T11:53:39Z

这一段注释是不是有问题？这个pr的结果依旧是 3&4，不过这个逻辑也可以

  const p40 = GM_registerMenuCommand("MenuReg D-40", () => {

    console.log(601);
  });

  const p50 = GM_registerMenuCommand("MenuReg D-50", () => {

    console.log(602);
  });
  console.log("p40, p50", [p40, p50]); // TM gives 3&4, but SC should skip 3 and give 4 and 5 (or skip both 3&4)

checkSubFrameIdSequence 我理解，其实就是不同id，但是同名的菜单的触发逻辑，都会触发一次

CodFrm · 2025-10-09T12:01:26Z

src/app/service/content/gm_api.ts

+  // 内容脚本执行环境识别符，用于区分 mainframe / subframe 等环境并作为 menu key 的命名空间。
+  // 由 execEnvInit() 以 randomMessageFlag() 生成，避免跨 frame 的 ID 碰撞。
+  // (同一环境跨脚本也不一样)
+  contentEnvKey: string | undefined;


可以使用 runFlag ，不过另外再生成一个也没问题，语义上不同，只是提一下有 runFlag 也可以用于区分不同的环境/脚本

src/app/service/content/gm_api.ts

src/app/service/service_worker/popup.ts

CodFrm · 2025-10-09T12:53:40Z

src/app/cache.ts

      });
    unlock();
-    return newValue!;
+    return newValue!; // 必须注意当 value 为 「undefined, null, "", 0」 时，newValue 是 undefined


啥意思？

CodFrm · 2025-10-09T13:07:38Z

src/app/service/service_worker/popup.ts

+
+  // 标记需要同步后，若成功写回快取，再触发实际菜单重建（避免多次小变更重复重建）。
+  async onMenuCommandsChanged(tabId: number, uuid: string) {
+    menuRegisterNeedUpdate = true;


menuRegisterNeedUpdate 的设计感觉意义不大， onMenuCommandsChanged -> syncMenuCommandsToSessionStore 是顺序的，也没有另外的地方有调用

CodFrm · 2025-10-09T13:23:06Z

src/app/service/service_worker/popup.ts

+    let menus = menuRegister.get(mrKey) as ScriptMenuItem[];
+    if (!menus) {
+      menus = [] as ScriptMenuItem[];
+      menuRegister.set(mrKey, menus);


menuRegister的设计也有一些多余，没有地方有效用到，可读性不佳，而且如果service_worker重启，会丢失信息

我明白了，如果并发的话，会有问题，会推送多次菜单到popup，导致闪动

// GM_registerMenuCommand 是同步函数。 // 所以流程是：先在 popup.ts 即时更新「无锁的记录」，再回写到「有锁记录」交给 Popup/App.tsx。 // 这样可以避免新增/删除操作的次序冲突。 // 外部（popup.tsx）只会读取 menu items，不会直接修改（若要修改，必须透过 popup.ts）。

這裡有寫

cyfung1031 · 2025-10-09T13:42:50Z

这一段注释是不是有问题？这个pr的结果依旧是 3&4，不过这个逻辑也可以

我最初想法是，应该要跳过id, 但之后再想了一下，跟TM就好，简单。实际使用问题是开发人员的问题
我开PR时忘记改掉comment. 较早时 1楼的代码有修正了这个注释

checkSubFrameIdSequence 我理解，其实就是不同id，但是同名的菜单的触发逻辑，都会触发一次

TM的做法就是把相同按钮，不论是同一frame 还是不同frame, 都合并在一个
按的时候就会几个一起按下
这是TM故意的设计吧
反正就是跟随TM。

…1/scriptcat into pr/cyfung1031/790

cyfung1031 · 2025-10-09T13:54:36Z

79ac078 的 popup.ts 改動要退回

CodFrm · 2025-10-09T14:00:04Z

79ac078 的 popup.ts 改動要退回

6445c0a 我修改为了其它方式

cyfung1031 · 2025-10-10T05:33:24Z

79ac078 的 popup.ts 改动要退回

6445c0a 我修改为了其它方式

你这个写法用了 unregisterMenuCommandMessages 和 registerMenuCommandMessages
会造成的次序不一
效能或是维护都不好

~~正如我之前的写法所言~~
~~script.menus 只会在 service_worker/popup.ts 的 unregisterMenuCommand 和 registerMenuCommand 中进行修改~~

~~addScriptRunNumber 不会进行 script.menus 修改~~
~~chrome.tabs.onRemoved.addListener 那边也可以清 cache，避免remove后再设回~~

~~原写法没问题~~

~~不然也可以把 script.menus 分拆出来~~
~~这样就更清晰了~~

我不坚持原案了
把你的代码修了一下提了 PR
#826

CodFrm · 2025-10-10T08:05:12Z

79ac078 的 popup.ts 改动要退回

6445c0a 我修改为了其它方式

你这个写法用了 unregisterMenuCommandMessages 和 registerMenuCommandMessages

会造成的次序不一

效能或是维护都不好

正如我之前的写法所言

script.menus 只会在 service_worker/popup.ts 的 unregisterMenuCommand 和 registerMenuCommand 中进行修改

addScriptRunNumber 不会进行 script.menus 修改

chrome.tabs.onRemoved.addListener 那边也可以清 cache，避免remove後再設回

原写法没问题

不然也可以把 script.menus 分拆出来

这样就更清晰了

次序（顺序？)应该是没问题的

性能可以考虑在添加队列后使用 await Promise / setTimeout 将线程所有权抛出，然后再判断一下长度是否为0，来减少后续的储存操作

CodFrm · 2025-10-13T14:27:08Z

发现了一个问题，我使用沉浸式翻译，他会注册两个同样的菜单，然后点击的时候，重复触发了事件，导致点击菜单没有翻译效果，一个简单的示例：

// ==UserScript==
// @name         test menu
// @namespace    https://docs.scriptcat.org/
// @version      0.1.0
// @description  try to take over the world!
// @author       You
// @match        https://github.com/CodFrm
// @icon         https://www.google.com/s2/favicons?sz=64&domain=github.com
// @grant        GM_registerMenuCommand
// ==/UserScript==


for (let i = 0; i < 2; i++) {
    GM_registerMenuCommand("菜单", () => {
        console.log("应该只输出一次");
    })
}

TM也是如此，但是直觉感觉不应该如此，同名菜单只触发一次似乎更合理一点，如果多iframe，就多iframe中触发，每个iframe中触发一次

cyfung1031 · 2025-10-13T22:43:06Z

发现了一个问题，我使用沉浸式翻译，他会注册两个同样的菜单，然后点击的时候，重复触发了事件，导致点击菜单没有翻译效果，一个简单的示例：
// ==UserScript==
// @name         test menu
// @namespace    https://docs.scriptcat.org/
// @version      0.1.0
// @description  try to take over the world!
// @author       You
// @match        https://github.com/CodFrm
// @icon         https://www.google.com/s2/favicons?sz=64&domain=github.com
// @grant        GM_registerMenuCommand
// ==/UserScript==


for (let i = 0; i < 2; i++) {
    GM_registerMenuCommand("菜单", () => {
        console.log("应该只输出一次");
    })
}
TM也是如此，但是直觉感觉不应该如此，同名菜单只触发一次似乎更合理一点，如果多iframe，就多iframe中触发，每个iframe中触发一次

你是指

GM_registerMenuCommand("菜单", () => {
  console.log("应该只输出一次");
 }, {id: "menu-01"})

吧？

TM沒有錯
GM_registerMenuCommand 兩次就是兩個
如果你指明了 id 就會是一個

cyfung1031 · 2025-10-13T22:50:58Z

@CodFrm

cyfung1031 added 4 commits October 2, 2025 19:41

重新修订 GM_registerMenuCommand 相关代码设计

09e7392

修正 Popup tsx 状态更新问题

4c854d0

简化 grouppedMenus 代码

ad453bd

小修正

e414b51

CodFrm requested a review from Copilot October 3, 2025 08:58

Copilot AI reviewed Oct 3, 2025

View reviewed changes

cyfung1031 and others added 7 commits October 3, 2025 21:19

Copilot说得对

3125f3c

Co-authored-by: Copilot <[email protected]>

Copilot说得对

a04c139

Co-authored-by: Copilot <[email protected]>

Copilot说得对

7e8981d

Co-authored-by: Copilot <[email protected]>

将TODO注释提取为统一的文档或issue。

b779c49

将TODO注释提取为统一的文档或issue。

3b6f520

lint

27d828d

整理注释成 VSCode IntelliSense 版

f7750a1

cyfung1031 force-pushed the GM_registerMenuCommand_codefix branch from 2b8cf7c to f7750a1 Compare October 3, 2025 12:57

cyfung1031 added 3 commits October 5, 2025 17:21

Merge branch 'main' into GM_registerMenuCommand_codefix

f099b1e

merge修正

cc57167

[script, grouppedMenus]);

0f7280f

cyfung1031 changed the title ~~重新修订 GM_registerMenuCommand 相关代码设计~~ [高优先] 重新修订 GM_registerMenuCommand 相关代码设计 Oct 7, 2025

Merge remote-tracking branch 'upstream/main' into GM_registerMenuComm…

8dbc5ec

…and_codefix

CodFrm added the P0 🚑 需要紧急处理的内容 label Oct 7, 2025

cyfung1031 added 2 commits October 8, 2025 09:42

Merge remote-tracking branch 'upstream/main' into GM_registerMenuComm…

2344f7a

…and_codefix

防止热重启打乱 id

4f676f8

cyfung1031 force-pushed the GM_registerMenuCommand_codefix branch from 64291a7 to 4f676f8 Compare October 8, 2025 01:57

cyfung1031 and others added 6 commits October 8, 2025 11:03

不需要 contextMenuConvArr

2d869ba

處理一下 chrome.contextMenus.create 的 Runtime lastError

05582cd

添加单元测试

d337b86

Merge remote-tracking branch 'origin/main' into pr/cyfung1031/790

59d74bf

减少 JSON.stringify 的呼叫

48824f0

中文

4831c68

CodFrm requested a review from Copilot October 9, 2025 05:47

Copilot AI reviewed Oct 9, 2025

View reviewed changes

src/pages/components/ScriptMenuList/index.tsx Show resolved Hide resolved

cyfung1031 mentioned this pull request Oct 9, 2025

【高优先】重构 GM_registerMenuCommand 设计：稳定 menu item ID、同步更新、跨 iframe 一致性，行为对齐 Tampermonkey #820

Closed

4 tasks

增加单元测试

69c7456

CodFrm reviewed Oct 9, 2025

View reviewed changes

CodFrm added 2 commits October 9, 2025 21:23

Update src/app/service/content/gm_api.ts

276e2a6

Update src/app/service/service_worker/popup.ts

831503c

CodFrm added 2 commits October 9, 2025 21:43

删除menuRegisterNeedUpdate 和 menuRegister逻辑

79ac078

Merge branch 'GM_registerMenuCommand_codefix' of github.com:cyfung103…

dc14f0f

…1/scriptcat into pr/cyfung1031/790

修复并发问题

6445c0a

修改为shift

b8b1e7d

CodFrm merged commit a71cfe4 into scriptscat:main Oct 9, 2025
2 of 3 checks passed

CodFrm mentioned this pull request Oct 9, 2025

二级菜单&分隔线 #785

Closed

CodFrm mentioned this pull request Oct 13, 2025

重构菜单消息推送代码 #837

Merged

cyfung1031 deleted the GM_registerMenuCommand_codefix branch October 17, 2025 13:34

	//@ts-ignore
	b.enable - a.enable \|\|
	Number(b.enable) - Number(a.enable) \|\|

	// 容错：URL 解析失败时忽略错误（不影响后续 UI）
	// 容错处理：URL 解析失败时忽略错误（不影响后续 UI）

	// providedId = (typeof providedId === "number" ? `n${providedId}` : `t${providedId}`) as string;

[高优先] 重新修订 GM_registerMenuCommand 相关代码设计 #790

[高优先] 重新修订 GM_registerMenuCommand 相关代码设计 #790

Uh oh!

Conversation

cyfung1031 commented Oct 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概述

变更内容

截图

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

cyfung1031 Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cyfung1031 commented Oct 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

CodFrm commented Oct 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

CodFrm Oct 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cyfung1031 commented Oct 9, 2025

Uh oh!

cyfung1031 commented Oct 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CodFrm commented Oct 9, 2025

Uh oh!

Uh oh!

cyfung1031 commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CodFrm commented Oct 10, 2025

Uh oh!

[高优先] 重新修订 `GM_registerMenuCommand` 相关代码设计 #790

[高优先] 重新修订 `GM_registerMenuCommand` 相关代码设计 #790

cyfung1031 commented Oct 2, 2025 •

edited

Loading

cyfung1031 Oct 3, 2025 •

edited

Loading

cyfung1031 commented Oct 7, 2025 •

edited

Loading

CodFrm commented Oct 9, 2025 •

edited

Loading

CodFrm Oct 9, 2025 •

edited

Loading

cyfung1031 commented Oct 9, 2025 •

edited

Loading

cyfung1031 commented Oct 10, 2025 •

edited

Loading

CodFrm commented Oct 13, 2025 •

edited

Loading