Merged
Conversation
Adjust tests
Contributor
Author
|
Tests are still failing, but we decided to merge the changes in order to tag a |
AlexMikhalev
added a commit
that referenced
this pull request
Mar 26, 2026
#55 - Add SpawnRequest struct to terraphim_spawner with primary/fallback fields - Add spawn_with_fallback method that tries primary provider first, then falls back - Add circuit_breakers field to AgentOrchestrator for future circuit breaker integration - Migrate spawn_agent to build SpawnRequest and use spawn_with_fallback - Pass fallback_provider and fallback_model from AgentDefinition when configured - All tests pass, clippy clean, cargo fmt applied
3 tasks
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This is not done yet, but we can do an early review of the structure I have in mind.
I think we need to work on making the types more granular to allow for a better design, but for now I have a
KnowledgeGraphBuilder, which takes anIndexMiddlewaresuch as ripgrep or logseq to create aThesaurus