Repository navigation
Conversation
Debug mode is how users get pre-releases, and analytics show many already use it, yet its label said developers only and every error popped up an English alert that users read as gibberish. Say what it does and welcome testers in the label, and show the error alert in Chinese with a thank-you and where to report. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017dqWu3YgeVRGwBt5pXDq4A
Reviewer's GuideThis text-only change makes debug mode more approachable for beta testers by updating its settings label and consistently replacing English internal-error alerts with friendly Chinese guidance, without changing runtime behavior. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Deploying xmoj-script-dev-channel with
|
| Latest commit: |
0902fee
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a07b6639.xmoj-script-dev-channel.pages.dev |
| Branch Preview URL: | https://friendly-debug-mode.xmoj-script-dev-channel.pages.dev |
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
Users still turn on SuperDebug and file bug reports about the features it breaks. While it is on, show a banner on every page saying the script talks to a local server, that this is not a bug, with a button to turn it off. Ask for a typed confirmation before enabling it, and keep it out of cloud sync so one mistaken click doesn't spread to every device. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017dqWu3YgeVRGwBt5pXDq4A
There was a problem hiding this comment.
0 issues found across 3 files (changes from recent commits).
Requires human review: Auto-approval blocked by 1 unresolved issue from previous reviews.
View guided diff | Turn on auto-fix | Re-trigger cubic
Sourcery withdrew this approval because the latest commits introduced blocking findings.
There was a problem hiding this comment.
All reported issues were addressed across 2 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
…machine The debug alert text was copied into 73 catch blocks; move it into a DebugAlert(e) helper next to SmartAlert so the wording lives in one place. The SuperDebug banner and prompt said it connects to the developer's machine, but 127.0.0.1 is the user's own computer; say that instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017dqWu3YgeVRGwBt5pXDq4A
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Requires human review: Rewrites debug-mode error alerts and the settings label to recruit beta testers and adds claimed SuperDebug safeguards, but the SuperDebug banner/cloud-sync/confirmation code is in the truncated diff and the copy change expands who runs pre-release builds.
View guided diff | Turn on auto-fix | Re-trigger cubic
What does this PR aim to accomplish?:
Make debug mode inviting instead of scary, so we get more beta testers. Debug mode is how users receive pre-releases, and analytics show many users already turn it on anyway. But the setting said "仅供开发者使用", and every error popped up an English "XMOJ-Script internal error!" alert that users report reading as gibberish.
How does this PR accomplish the above?:
调试模式(仅供开发者使用)→调试模式(抢先体验测试版,并在出错时弹窗提示,欢迎开启帮我们找 bug), which says what the option actually does.SuperDebugis the opposite case: it points the script at a local backend (127.0.0.1:8787) and breaks most features, yet users still turn it on and file bug reports about the breakage. So:<html>so legacybody.innerHTMLrewrites can't remove it). It says this is not a bug and has a "关闭本地调试模式" button that turns it off and reloads.我是开发者, and points beta testers to 调试模式 instead.Does this PR need a documentation update?:
No
By submitting this pull request, I confirm the following:
devfor XMOJ-Script,masterfor everything else).git rebase)/webpages)/webpages)🤖 Generated with Claude Code
https://claude.ai/code/session_017dqWu3YgeVRGwBt5pXDq4A
Summary by Sourcery
Make DebugMode welcoming for beta testers while clearly isolating and restricting SuperDebug to local development use.
New Features:
Bug Fixes:
Enhancements:
Build: