处理early中的UserAgentData by CodFrm · Pull Request #1045 · scriptscat/scriptcat

CodFrm · 2025-12-02T08:35:06Z

概述 Descriptions

变更内容 Changes

截图 Screenshots

cyfung1031 · 2025-12-02T08:41:10Z

对这个PR没什么意见
不过个人认为，与其搞 early-start, 倒不如所有脚本的注册都能自己跑的content 环境
那就不用另外注入两个 scriptcat-content scriptcat-inject
脚本是 inject-into page 的话就在 content 环境 appendChild(script) 这样做

这样改动是很大，不过下一个版本你可以考虑一下

Copilot

Pull request overview

这个 PR 旨在解决 early-start 脚本中 UserAgentData 的处理问题（修复 #1037），同时将 API 名称从 CAT_ScriptLoaded 更正为 CAT_scriptLoaded（驼峰命名）。

主要变更包括：

在 inject 和 content 环境中通过全局变量传递 UserAgentData，使 early-start 脚本能够立即访问 userAgentData
重构了 ScriptExecutor 类，移除了实例级别的 envInfo 存储，改为通过参数传递
修正了 API 命名规范，将 CAT_ScriptLoaded 改为 CAT_scriptLoaded

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
`src/types/scriptcat.d.ts`	修正 API 名称：`CAT_ScriptLoaded` → `CAT_scriptLoaded`
`src/types/main.d.ts`	添加全局变量声明 `UserAgentData`
`src/template/scriptcat.d.tpl`	模板文件中同步修正 API 名称
`src/inject.ts`	导入 `initEnvInfo` 并传递给 `checkEarlyStartScript` 和 `startScripts`
`src/content.ts`	导入 `initEnvInfo` 并传递给相关方法
`src/app/service/service_worker/runtime.ts`	在 inject 和 content 脚本注入时通过 IIFE 参数传递 UserAgentData；更新 USERSCRIPTS_REGISTER_CONTROL 以强制重新注册
`src/app/service/content/types.ts`	为 `sandboxMode` 字段添加注释说明其为预留字段
`src/app/service/content/script_executor.ts`	添加 `initEnvInfo` 导出，在模块加载时从全局变量初始化；移除实例级 `envInfo` 存储，改为参数传递
`src/app/service/content/inject.ts`	移除 `setEnvInfo` 方法，`startScripts` 方法直接接收 envInfo 参数
`src/app/service/content/content.ts`	`pageLoad` 和 `startScripts` 方法接收并传递 envInfo 参数

Copilot · 2025-12-02T08:54:21Z

src/app/service/service_worker/runtime.ts

  ) {
    // 构建inject.js的脚本注册信息
-    const code = `(function (MessageFlag) {\n${injectJs}\n})('${messageFlag}')`;
+    const code = `(function (MessageFlag,UserAgentData) {\n${injectJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`;


与 content.js 注入代码相同的问题：JSON.stringify(this.userAgentData) 可能在 userAgentData 未正确初始化时导致注入空对象，影响 inject 环境中的 early-start 脚本。建议确保在调用此方法前 userAgentData 已正确初始化。

Suggested change

const code = `(function (MessageFlag,UserAgentData) {\n${injectJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`;

const code = `(function (MessageFlag,UserAgentData) {\n${injectJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData || {})})`;

Copilot · 2025-12-02T08:54:22Z

src/inject.ts

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

-/* global MessageFlag  */
+/* global MessageFlag */


[nitpick] 应该添加 /* global UserAgentData */ 注释来声明全局变量 UserAgentData，与 MessageFlag 的处理方式保持一致。虽然 TypeScript 类型定义中已声明，但全局注释有助于明确该变量的来源。

Suggested change

/* global MessageFlag */

/* global MessageFlag UserAgentData */

Copilot · 2025-12-02T08:54:22Z

src/content.ts

+import { initEnvInfo, ScriptExecutor } from "./app/service/content/script_executor";
 import { randomMessageFlag } from "./pkg/utils/utils";
 import type { Message } from "@Packages/message/types";



[nitpick] content.ts 也使用了全局变量 UserAgentData（通过 initEnvInfo 导入），建议添加 /* global UserAgentData */ 注释来声明，与其他环境保持一致性。

Copilot · 2025-12-02T08:54:23Z

src/app/service/content/script_executor.ts

+} catch {
+  // 如果 UserAgentData 不存在，可能是在非inject/content环境下运行


使用空的 catch 块会静默捕获所有错误，包括可能的代码错误（如变量拼写错误等），这会使调试变得困难。建议：

至少记录错误信息用于调试：catch (e) { console.debug("UserAgentData not available:", e); ... }

或者使用更精确的错误检查：typeof UserAgentData !== 'undefined' ? UserAgentData : {}

Suggested change

} catch {

// 如果 UserAgentData 不存在，可能是在非inject/content环境下运行

} catch (e) {

// 如果 UserAgentData 不存在，可能是在非inject/content环境下运行

console.debug("UserAgentData not available:", e);

Copilot · 2025-12-02T08:54:23Z

src/app/service/service_worker/runtime.ts

@@ -53,7 +53,7 @@ import { scriptToMenu, type TPopupPageLoadInfo } from "./popup_scriptmenu";

 // 避免使用版本号控制导致代码理解混乱
 // 用来清除 UserScript API 里的旧缓存


[nitpick] 更改了 USERSCRIPTS_REGISTER_CONTROL 的值，这将强制所有用户重新注册 UserScript。建议在上方添加注释说明变更原因，例如：

// 修改控制符以强制重新注册，因为需要传递 UserAgentData 到 inject/content 环境 const USERSCRIPTS_REGISTER_CONTROL = "92292a62-4e81-4dc3-87d0-cb0f0cb9883d";

Suggested change

// 用来清除 UserScript API 里的旧缓存

// 用来清除 UserScript API 里的旧缓存

// 修改控制符以强制重新注册，因为需要传递 UserAgentData 到 inject/content 环境

Copilot · 2025-12-02T08:54:23Z

src/app/service/service_worker/runtime.ts

-        js: [{ code: `(function (MessageFlag) {\n${contentJs}\n})('${messageFlag}')` }],
+        js: [
+          {
+            code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`,


使用 JSON.stringify(this.userAgentData) 可能导致问题：

如果 userAgentData 包含 undefined 值，JSON.stringify 会将其省略，导致注入的代码不完整

如果 userAgentData 为空对象 {}（初始化时的默认值），注入的是空对象，early-start 脚本可能获取不到正确的 userAgentData

建议添加错误处理或确保 this.userAgentData 在注入前已正确初始化。可以考虑：

code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData || {})})`,

Suggested change

code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`,

code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData ?? {}, (_, v) => v === undefined ? null : v)})`,

CodFrm · 2025-12-02T09:12:33Z

对这个PR没什么意见不过个人认为，与其搞 early-start, 倒不如所有脚本的注册都能自己跑的content 环境那就不用另外注入两个 scriptcat-content scriptcat-inject 脚本是 inject-into page 的话就在 content 环境 appendChild(script) 这样做

这样改动是很大，不过下一个版本你可以考虑一下

再演进的话，感觉要变成和MV2差不多的方式了，脚本代码发送到 content ，由 content 进行注入，这种做法兼容性广一些（不依赖 chrome.userScript了），但是 early-start 的速度达不到了，始终要获取一次代码

可以考虑做一下，我了解到iOS Edge对于 browser.userScript 的支持因为iOS政策问题，支持不了，但是browser.scripting可以，iOS Edge上架了一个Stay的扩展，似乎就是这种做法，但是我不确定，需要进行验证

cyfung1031 · 2025-12-02T09:42:42Z

对这个PR没什么意见不过个人认为，与其搞 early-start, 倒不如所有脚本的注册都能自己跑的content 环境那就不用另外注入两个 scriptcat-content scriptcat-inject 脚本是 inject-into page 的话就在 content 环境 appendChild(script) 这样做
这样改动是很大，不过下一个版本你可以考虑一下

再演进的话，感觉要变成和MV2差不多的方式了，脚本代码发送到 content ，由 content 进行注入，这种做法兼容性广一些（不依赖 chrome.userScript了），但是 early-start 的速度达不到了，始终要获取一次代码

可以考虑做一下，我了解到iOS Edge对于 browser.userScript 的支持因为iOS政策问题，支持不了，但是browser.scripting可以，iOS Edge上架了一个Stay的扩展，似乎就是这种做法，但是我不确定，需要进行验证

脚本代码发送到 content

但是 early-start 的速度达不到了，始终要获取一次代码

你理解错了

我意思是，

chrome.userScripts.register([{
        id: "script123",
        js: [{ code: `(function (code) { GM = ... new Client ....  
const injectCode = function(){
GM = ...
window.addEventListener("xyz", (event)=>code(GM, event.details.GM_info ...));
}
injectToPage? document.documentElement.appendChild(code) : executeScript(code)
window.dispatchEvent(new CustomEvent("xyz", {details: {GM_info: ....}}))

}  }\n})(${code})` }],
        matches: ["<all_urls>"],
        allFrames: true,
        runAt: "document_start",
        world: "USER_SCRIPT",
        excludeMatches,
        excludeGlobs,
      }
]);

这样吧

反正就是 self contained 的设计

注册会把 server client code 一个脚本处理好

不像现在要等 content 载入才有 pageLoad

CodFrm added 3 commits December 2, 2025 15:31

错误的提示

d83faef

处理early中的UserAgentData

1b11cb0

处理初始化gmInfoEnv

fb30564

CodFrm linked an issue Dec 2, 2025 that may be closed by this pull request

[BUG] early-start 设置后 GM_info.userAgentData 为 undefined #1037

Closed

CodFrm added P0 🚑 需要紧急处理的内容 hotfix 需要尽快更新到扩展商店 labels Dec 2, 2025

错误处理

b594829

CodFrm requested a review from Copilot December 2, 2025 08:48

Copilot started reviewing on behalf of CodFrm December 2, 2025 08:49 View session

Copilot finished reviewing on behalf of CodFrm December 2, 2025 08:52

Copilot AI reviewed Dec 2, 2025

View reviewed changes

CodFrm merged commit b4e08a8 into main Dec 2, 2025
7 of 9 checks passed

CodFrm deleted the fix/1037 branch December 3, 2025 03:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

处理early中的UserAgentData#1045

处理early中的UserAgentData#1045
CodFrm merged 4 commits intomainfrom
fix/1037

CodFrm commented Dec 2, 2025

Uh oh!

cyfung1031 commented Dec 2, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Copilot AI Dec 2, 2025

Uh oh!

Uh oh!

CodFrm commented Dec 2, 2025

Uh oh!

cyfung1031 commented Dec 2, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	const code = `(function (MessageFlag,UserAgentData) {\n${injectJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`;
	const code = `(function (MessageFlag,UserAgentData) {\n${injectJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData \|\| {})})`;

	/* global MessageFlag */
	/* global MessageFlag UserAgentData */

		} catch {
		// 如果 UserAgentData 不存在，可能是在非inject/content环境下运行

		@@ -53,7 +53,7 @@ import { scriptToMenu, type TPopupPageLoadInfo } from "./popup_scriptmenu";

		// 避免使用版本号控制导致代码理解混乱
		// 用来清除 UserScript API 里的旧缓存

	// 用来清除 UserScript API 里的旧缓存
	// 用来清除 UserScript API 里的旧缓存
	// 修改控制符以强制重新注册，因为需要传递 UserAgentData 到 inject/content 环境

	code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData)})`,
	code: `(function (MessageFlag,UserAgentData) {\n${contentJs}\n})('${messageFlag}', ${JSON.stringify(this.userAgentData ?? {}, (_, v) => v === undefined ? null : v)})`,

Comments

Conversation

CodFrm commented Dec 2, 2025

概述 Descriptions

变更内容 Changes

截图 Screenshots

Uh oh!

cyfung1031 commented Dec 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 Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

CodFrm commented Dec 2, 2025

Uh oh!

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

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cyfung1031 commented Dec 2, 2025 •

edited

Loading

cyfung1031 commented Dec 2, 2025 •

edited

Loading