Skip to content

feat: Durable Web Search golem:web-search API Across Different Providers in Rust #46

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

Closed
wants to merge 15 commits into from

Conversation

Rutik7066
Copy link
Contributor

@Rutik7066 Rutik7066 commented Jun 20, 2025

Added Tavily, Serper, Brave & Google custom search.
Demo link : https://drive.google.com/file/d/16RxMlgtNzEp9Z1vYN-JZskMXBHARhuAQ/view?usp=sharing
Note: Microsoft Bing is not added. Microsoft will discontinue the bing api support after 11th August 2025
Ref : https://www.microsoft.com/en-us/bing/apis/bing-web-search-api

claim #34
/fixes #34

@Rutik7066 Rutik7066 marked this pull request as ready for review June 24, 2025 20:02
@Rutik7066
Copy link
Contributor Author

Rutik7066 commented Jun 25, 2025

@iambenkay I need your assistance in figuring out why the CI is failing. It's important to resolve this CI issue.

@Rutik7066
Copy link
Contributor Author

same issue here as well #28

@iambenkay
Copy link
Contributor

iambenkay commented Jun 26, 2025

I spent some time looking into this.
The way golem app build works, it discovers the components based on all the included golem.yaml files in the subdirectories and tries to deploy them. test:helper does not need any wasm library so it naturally works. test:llm is the one we are building before running the golem build so it works.
websearch is the exception here, since it does not get built into a wasm component before attempting to run golem build.

The immediate solution I see is to change the instruction for build to be cargo make build instead of cargo make --cwd llm build-ollama in ci.yaml.
That way all the components get built. There may be some way to tell golem to only build components in some directories but I have not figured that out yet.

@Rutik7066
Copy link
Contributor Author

I spent some time looking into this. The way golem app build works, it discovers the components based on all the included golem.yaml files in the subdirectories and tries to deploy them. test:helper does not need any wasm library so it naturally works. test:llm is the one we are building before running the golem build so it works. websearch is the exception here, since it does not get built into a wasm component before attempting to run golem build.

The immediate solution I see is to change the instruction for build to be cargo make build instead of cargo make --cwd llm build-ollama in ci.yaml. That way all the components get built. There may be some way to tell golem to only build components in some directories but I have not figured that out yet.

I found the issue this config make golem build all providers /home/daytona/golem-llm/test/golem.yaml
but current quick solution is to build all wasm and test ollma

@Rutik7066
Copy link
Contributor Author

@vigoo @jdegoes All checks are passing. PR is ready to review.

@Rutik7066 Rutik7066 force-pushed the web-search branch 3 times, most recently from 6468343 to a806ff1 Compare July 5, 2025 14:26
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.

Implement Durable Web Search golem:web-search API Across Different Providers in Rust
2 participants