-
Notifications
You must be signed in to change notification settings - Fork 162
Add wisent.com to whitelist #1753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughThe Changes
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@whitelist.yaml`:
- Around line 33-34: The whitelist currently includes only the wildcard
"*.wisent.com" which does not match the apex domain; add an explicit entry "url:
wisent.com" alongside "*.wisent.com" so that both the root domain and its
subdomains are allowed (update the whitelist.yaml entries where "*.wisent.com"
is declared to also include the apex "wisent.com").
| - url: nftplus.io | ||
| - url: "*.wisent.com" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the apex domain if you intend to whitelist wisent.com itself.
"*.wisent.com" matches only subdomains, not the root wisent.com. The PR objective calls out both wisent.com and subdomains, so the apex will still be blocked unless explicitly added.
✅ Suggested fix
- url: nftplus.io
+ - url: wisent.com
- url: "*.wisent.com"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - url: nftplus.io | |
| - url: "*.wisent.com" | |
| - url: nftplus.io | |
| - url: wisent.com | |
| - url: "*.wisent.com" |
🤖 Prompt for AI Agents
In `@whitelist.yaml` around lines 33 - 34, The whitelist currently includes only
the wildcard "*.wisent.com" which does not match the apex domain; add an
explicit entry "url: wisent.com" alongside "*.wisent.com" so that both the root
domain and its subdomains are allowed (update the whitelist.yaml entries where
"*.wisent.com" is declared to also include the apex "wisent.com").
Summary
Adding
*.wisent.comto the whitelist.About Wisent
Wisent (https://singularity.wisent.com) is a legitimate DeFi platform for autonomous AI agents. The platform enables:
Details
This is a legitimate project and users are currently seeing "This dApp could be malicious" warnings when connecting their Phantom wallet.
Thank you for reviewing!
Summary by CodeRabbit
Bug Fixes
Updates