Skip to content

Conversation

Traveler07
Copy link
Contributor

[1、提高脚本的稳定性,优化脚本的流畅度。
2、提供OCR支持
3、允许自定义操作延迟
4、脚本名变更为中文

@physligl
Copy link
Member

physligl commented Oct 9, 2025

你干嘛修改脚本名

@physligl physligl requested a review from Copilot October 9, 2025 09:39
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the AutoPermission script to "世界权限设置" (World Permission Settings) with enhanced functionality including OCR support, custom operation delays, and improved stability.

  • Renamed the script from English to Chinese and moved to a new directory structure
  • Added OCR-based operation mode as an alternative to fixed position clicking
  • Introduced configurable operation delay settings for better reliability

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
repo/js/世界权限设置/settings.json New configuration file with work mode selection, permission options, and custom delay settings
repo/js/世界权限设置/manifest.json Updated manifest with version bump to 2.0
repo/js/世界权限设置/main.js Complete rewrite with OCR support and configurable delays
repo/js/AutoPermission/settings.json Removed old configuration file
repo/js/AutoPermission/main.js Removed old implementation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

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

Missing spaces around operators. Should be domainName === undefined and domainName = "不允许加入".

Suggested change
if(domainName===undefined){
domainName="不允许加入"
if (domainName === undefined) {
domainName = "不允许加入"

Copilot uses AI. Check for mistakes.

Copy link

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

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

Missing spaces around operators and after if keyword. Should be if (settings.workMode === "基于固定位置") {.

Suggested change
if(settings.workMode==="基于固定位置"){
if (settings.workMode === "基于固定位置") {

Copilot uses AI. Check for mistakes.

Copy link

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

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

Missing space after else keyword. Should be else {.

Suggested change
else{
else {

Copilot uses AI. Check for mistakes.

Copy link

Copilot AI Oct 9, 2025

Choose a reason for hiding this comment

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

Hard-coded sleep duration should use the configurable delay from settings.inputValue when available.

Suggested change
await sleep(1000);
await sleep(settings.inputValue !== undefined ? settings.inputValue : 1000);

Copilot uses AI. Check for mistakes.

@Traveler07
Copy link
Contributor Author

你干嘛修改脚本名

因为我忽然发现,bettergi是面向国服用户的,用中文更合适。

@zaodonganqi
Copy link
Collaborator

你干嘛修改脚本名

因为我忽然发现,bettergi是面向国服用户的,用中文更合适。

其实一样

@zaodonganqi zaodonganqi merged commit 09356bc into babalae:main Oct 9, 2025
1 check passed
@Traveler07
Copy link
Contributor Author

你干嘛修改脚本名

因为我忽然发现,bettergi是面向国服用户的,用中文更合适。

其实一样

还是不一样的,仓库里看到一堆中文名和看到一堆英文名感受完全不一样,至少我英文比较差,还需要一个翻译过程。

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.

3 participants