Skip to content

版本升級,代碼修正#499

Closed
cyfung1031 wants to merge 3 commits intoscriptscat:mainfrom
cyfung1031:patch_fix11
Closed

版本升級,代碼修正#499
cyfung1031 wants to merge 3 commits intoscriptscat:mainfrom
cyfung1031:patch_fix11

Conversation

@cyfung1031
Copy link
Copy Markdown
Collaborator

  • 執行 chrome.userScripts.configureWorld 時會出現 chrome.userScripts undefined 的情況。估計是在 chrome://XXXXX 時沒有chrome.userScripts. 加了logger.error

  • parseFavicons 時有 DOMException 問題發生。換了更快的Regex. 但好像還有DOMException. 不詳

  • package的版本都升級了一次。i18next保留在23.x.x, vitest保留在2.x.x

  • esbuild 強制指定 0.25.5 (現時最新)

  • globfetch-blob 強制指定在 10.4.5 及 4.0.0 (穏定最新)

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Jul 6, 2025

为什么要强制指定,而且unocss的版本看起来有问题,单元测试也没有通过

}

messageFlag = await this.getAndGenMessageFlag();
if (!chrome.userScripts) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没有必要的判断吧,如果不支持的话,应该是提示,在之前已经有检查了

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不清楚是不是Brave問題還是什麼。間中後台的錯誤中會在 chrome.userScripts.configureWorld 那句報 chrome.userScripts undefined

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果chrome.userScripts是undefined的话,应该都不会到这个流程来才是

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2025-07-09 at 9 45 49

最新版的後台還是偶爾發生這錯誤。沒注意到是什麼操作

@cyfung1031
Copy link
Copy Markdown
Collaborator Author

cyfung1031 commented Jul 6, 2025

为什么要强制指定

pnpm install的話

 WARN  4 deprecated subdependencies found: glob@7.2.3, inflight@1.0.6, node-domexception@1.0.0, popper.js@1.16.1

依存中使用的 glob 很舊。現在的穏定版本是10.4.5 (一年前)

(glob 7.2.3 導致 glob@7.2.3, inflight@1.0.6 過舊)
(node-domexception 和 popper.js 是沒辦法的)

esbuild 的話有多個引用,而且版本可能不一致。現在pnpm-lock 裡面是0.21.5和0.23.1並存。esbuild 一般都用最新,不然會一堆打包問題。(對應的node版本之類)。能打包就行。

"jsdom>parse5" 可以去除。影響很小。(已刪)

单元测试也没有通过

fetch-blob 4.0.0 刪了一個依存。好像沒有那個依存就不行吧。我刪了 4.0.0 的指定。能通過了

而且unocss的版本看起来有问题

unocss 版本的問題見此 - https://github.com/unocss/unocss/releases/tag/v65.4.0

升級沒有問題

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Jul 6, 2025

只是warn的话,我觉得没必要处理,避免导致其它问题

我的打包工具使用的rspack,并不是vite,只是用了vitest,可能会有引用esbuild

@cyfung1031
Copy link
Copy Markdown
Collaborator Author

退回PR 499
PR 502 時,一併更新 favicon.ts。不進行版本改動

@cyfung1031 cyfung1031 closed this Jul 6, 2025
This was referenced Jul 6, 2025
@cyfung1031 cyfung1031 deleted the patch_fix11 branch July 8, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants