Skip to content

feat: use the duckduckgo search #87

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

treeleaves30760
Copy link

This is an update for #80.
If the firecrawl return empty in any situation, we will use the duckduckgo as a backup option.

Copy link

@Dariton4000 Dariton4000 left a comment

Choose a reason for hiding this comment

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

The changes seem well-structured and logical.

@dzhng
Copy link
Owner

dzhng commented Feb 17, 2025

thanks for this! mind that I don't merge this PR and just leave it open as a reference implementation? I want the repo to be minimal so people can easily fork / modify it

@treeleaves30760
Copy link
Author

This change can be a feature since the firecrawl is so expensive on search, and many people suffer from it.
Also, the change is minimal and well-structured enough to maintain and modify for people.

@firefoxzheng
Copy link

Great! This feature can be easily integrated as an option to the default google search.

@serhii-nakon
Copy link

Support it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants