Expand redemption assist default whitelist

[godnight10061]

Summary

• Add cdk.hybgzs.com and qd.x666.me to the default Redemption Assist URL whitelist.
• Escape dots in the default regex patterns to avoid accidental matches.

Tests

• pnpm test:ci
• pnpm zip:all

Refs

• [Feature]: 大佬这个能添加吗 #281

[dosubot]

Related Documentation

Checked 4 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[coderabbitai]

Walkthrough

This pull request expands the default URL whitelist for RedemptionAssist by adding two new domain patterns (cdk\.hybgzs\.com and qd\.x666\.me) alongside the existing cdk\.linux\.do pattern. Corresponding test files are updated to validate the new configuration and verify that the service correctly allows prompt actions on these common redemption sites.

Changes

Cohort / File(s)	Summary
Default Preferences Configuration services/userPreferences.ts	Expanded DEFAULT_PREFERENCES.RedemptionAssist.urlWhitelist.patterns array from 1 to 3 regex patterns, adding two new whitelisted domains with escaped dot literals.
Test Coverage tests/services/redemptionAssist.test.ts, tests/services/userPreferences.test.ts	Added new test case "allows prompt on common redemption sites by default" to validate handleRedemptionAssistMessage behavior on whitelisted URLs; updated expected default patterns in userPreferences test to match expanded configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat(redemptionAssist): add relaxed code validation support #305: Modifies DEFAULT_PREFERENCES.redemptionAssist default values in the same configuration file.
• feat(preferences): add URL whitelist for redemption assist #274: Updates the same urlWhitelist.patterns array and associated test expectations.

Poem

🐰 Three URLs now hop with glee,
Redemption sites whitelisted, three by three!
From linux lands to mystery mines,
Our patterns leap through dotted lines.
A rabbit's work, precise and fine! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: expanding the redemption assist default whitelist with new URL patterns.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Preview build ready for commit 0b6d568.

Artifacts:

• all-api-hub-3.8.0-chrome.zip
• all-api-hub-3.8.0-firefox.zip
• all-api-hub-3.8.0-sources.zip

Download from workflow run: https://github.com/qixing-jk/all-api-hub/actions/runs/21394184356

This build is for review only and is not released.

[qixing-jk]

Escape dots in the default regex patterns to avoid accidental matches

It is right.

Add cdk.hybgzs.com and qd.x666.me to the default Redemption Assist URL whitelist.

But this one may not be the original intention of the proposer of that issue. Because users often want to add those sites that provide CDK as account sites, like these sites, rather than as external sign-in sites for a specific account.

[godnight10061]

Escape dots in the default regex patterns to avoid accidental matches

It is right.

Add cdk.hybgzs.com and qd.x666.me to the default Redemption Assist URL whitelist.

But this one may not be the original intention of the proposer of that issue. Because users often want to add those sites that provide CDK as account sites, like these sites, rather than as external sign-in sites for a specific account.

Noted. I will wait for further clarification.