Fix/12 human readable errors#18
Merged
Merged
Conversation
…ntext builder, reasoning controller, and tool registry - Added `AgentLoop` for managing multi-turn interactions and tool calls. - Introduced `ApprovalGate` to handle tool execution policies and user approvals. - Created `ContextBuilder` for generating prompts and managing conversation context. - Developed `ReasoningController` to extract and manage reasoning from model responses. - Established `ToolRegistry` to unify action tools and introspection tools with their policies. - Defined `ToolPolicy` and `ToolDescriptor` for managing tool behaviors and attributes. - Added comprehensive tests for harness components, ensuring functionality without a live LLM or FarmBot.
- Added a model picker to allow users to select from available assistant models. - Improved action execution feedback by displaying results after approval or rejection. - Updated the planning service to support multiple LLM providers and model configurations. - Refactored action summaries to include additional parameters like water_pin for move_path actions. - Removed deprecated send_message action and its associated logic. - Enhanced safety checks for water_pin parameter in move actions. - Added tests for LLM provider functionality and API endpoints.
- Catch unhandled action errors in POST /actions and return meaningful HTTPExceptions (502 for FarmBot connection issues, 500 with real cause). - Add ApiResult.error_message() to extract detail/error/raw body text. - Replace all raw st.error(body) calls in the UI with the new helper. - Add tests for action endpoint errors and ApiResult message extraction. Fixes #12
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.
No description provided.