fix: register youcom plugin in core provider registration - #649
fix: register youcom plugin in core provider registration#649korderushi07 wants to merge 1 commit into
Conversation
|
@korderushi07 is attempting to deploy a commit to the corsair Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe core provider registry now registers the You.com provider in its provider list, display-name map, and provider union type. ChangesYou.com provider registration
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
Greptile SummaryThis PR registers the existing You.com plugin in Corsair’s shared provider vocabulary.
Confidence Score: 5/5The PR appears safe to merge with the You.com provider registered consistently across all required core declarations. The added identifier matches the plugin’s existing Important Files Changed
Reviews (1): Last reviewed commit: "fix: register youcom plugin in core prov..." | Re-trigger Greptile |
|
Thanks for the contribution @korderushi07 Closing PR as PR #623 already exist. |
The
youcomplugin (You.com Web Search) was fully implemented atpackages/youcom/but was missing from the core provider registry,so it wasn't recognized by the type system, display-name lookup,
or provider list.
Summary by CodeRabbit