-
Notifications
You must be signed in to change notification settings - Fork 4
Tool use dsrs #106
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
AnthonyRonning
wants to merge
20
commits into
master
Choose a base branch
from
tool-use-dsrs
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Tool use dsrs #106
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
cb1016b
feat: Add tool use with classifier-based web search
AnthonyRonning 15bef03
refactor: remove panic from StorageMessage clone by using oneshot for…
AnthonyRonning 70e5cc4
feat: Replace mock web search with Kagi Search API integration
AnthonyRonning 9543e24
feat: Make web_search tool opt-in via client configuration
AnthonyRonning c10ab81
refactor: Replace kagi-api-rust submodule with minimal in-house client
AnthonyRonning 9203f53
fix: Move assistant message creation after tool execution to fix time…
AnthonyRonning 797abf2
refactor: Enable non-blocking SSE streaming with background orchestrator
AnthonyRonning a847190
docs: Add Kagi Search deployment configuration for AWS Nitro
AnthonyRonning 1796d76
refactor: Auto-prefix Kagi API key with "Bot " in client
AnthonyRonning 9b0cfcd
fix: Make Kagi client optional and improve error handling
AnthonyRonning 3a0dae1
fix: Honor tool_choice parameter before executing web search
AnthonyRonning 4dc3f8e
refactor: Fix race conditions and optimize streaming orchestrator
AnthonyRonning 40e8ec9
fix: Abort requests on storage channel failure during tool execution
AnthonyRonning 77f1c47
refactor: Replace in-memory tool_call tracking with DB lookup
AnthonyRonning 6b60d53
fix: Filter empty content deltas to prevent unnecessary SSE events
AnthonyRonning 1efb399
security: Add user authorization to tool_call lookup
AnthonyRonning 818315a
fix: Guard against integer overflow in tool token counting
AnthonyRonning 2cdaf6a
Update PCRs
AnthonyRonning bfc4cca
Implement first pass at dsrs
AnthonyRonning 7be3d33
feat: Implement DSPy-style modules with custom Adapter for tool class…
AnthonyRonning File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| { | ||
| "HashAlgorithm": "Sha384 { ... }", | ||
| "PCR0": "a771d516c0373f30ae15b3296d188e376345d6d21294b6cc7c5a9d7c10e9989fcd1cf5ff45043644caac19e1644d768d", | ||
| "PCR0": "5d05893f150ce0237d4dbea01f83d1aed7ba8eeaa4239ce9ac17286ad4548cd5aab757aa20c15bd028703b6c3b8ff048", | ||
| "PCR1": "f004075c672258b499f8e88d59701031a3b451f65c7de60c81d09da2b0799272675481ec390527594dd7069cb7de59d7", | ||
| "PCR2": "328f9ceecaa1bac78021274a5dd3f977f2c9ea4c96b1bf70d89b94f0958951da33a47881e2113ec27ae3072c05296f63" | ||
| "PCR2": "22485f598f10762f0a932469c6bbd9a1cd867da14703d7d5031e9f8f591c535ca80f9676706cea27768098297a9e2eb8" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| { | ||
| "HashAlgorithm": "Sha384 { ... }", | ||
| "PCR0": "858fe55e3736da573ab19719b92df9793b106451fcb1750b35e46b788f58ce294a1048795723f57628ec2347ebd0c3a9", | ||
| "PCR0": "c2d7a330881cd2f394d85e3a04def8714d611138560785806f116ea52e8748fcf6f2453bff89977faa9b1c20af6e8778", | ||
| "PCR1": "f004075c672258b499f8e88d59701031a3b451f65c7de60c81d09da2b0799272675481ec390527594dd7069cb7de59d7", | ||
| "PCR2": "82c53aa649f676766a839e93699ecca83f1207dd9ab59590dfba33149cd274dc59afe10ea1af0feaa5aec1f4d840efd3" | ||
| "PCR2": "97845cf3189f0da75d5bf893c91461d3cc8f19c2f8955c4e097f7b69ffe4c18dfc8d55590d503cb73a4dcd64ba225c90" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
logic: dependency points to
mainbranch instead of a specific commit or version tagUsing
mainbranch creates instability - any upstream changes will break your build without warning. Pin to specific commit hash or wait for v0.6.0 release.Prompt To Fix With AI