Skip to content

fix(engine): apply WebSearch domain filters#338

Open
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-web-search-domain-filters
Open

fix(engine): apply WebSearch domain filters#338
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-web-search-domain-filters

Conversation

@OnlyYu1996
Copy link
Copy Markdown

Fixes PlatformNetwork/bounty-challenge#53257.

Summary

  • apply include_domains as site: filters in the DuckDuckGo query
  • apply exclude_domains as -site: filters in the DuckDuckGo query
  • ignore blank domain entries rather than emitting malformed filters
  • reuse the filtered query in the fallback search URL
  • add focused unit coverage for include/exclude/blank domain handling

Validation

  • cargo +stable fmt --package cortex-engine --check
  • cargo +stable test --package cortex-engine tools::handlers::web_search::tests (5 passed)
  • cargo +stable check --package cortex-engine --lib
  • git diff --check

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant