feat: adding agents back to the experimental repo #326
Conversation
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
Pull Request Test Coverage Report for Build 16221580855Details
💛 - Coveralls |
|
This is acting like a feature branch, I open the following branches/PR against this one: |
* initial import * formating and renaming file * solving shadowing issues * solving shadowing issues * solving formating issues * adding type annotation * adding visit count to Tool * updating AgentBreakpoint * removing whitespace * renaming shadow variable names * improving eq in ToolBrekapoint
* adding agents back to the experimental repo * wip: adding breakpoints PoC to Agents * wip * typing * formatting * pylint issues * fix typing * reverting pyproject.toml * fixing bug in component_visits * linting * adding/fixing tests * all tests fixed * cleaning up * adding breakpoints to async version * cleaning up * refactoring tests * adding more tests * cleaning up * formating issues * typing * cleaning * adding async tests * nit * refactoring breakpoints logic to avoid duplicated code * converting to staticmethod * updating Agent + tests to use Breakpoint datastrcutures * updating async tests to use Breakpoint datastrcutures * refactoring tests * refactoring tests * wip: Pipeline accepting Breakpoint and Agentbreakpoint * Pipeline accepting Breakpoint and Agentbreakpoint + fixing tests * fix validate_input issue * updating tests * wip: adding breakpoints PoC to Agents * WIP: testing breakpoints in agent within a pipeline * wip: adding test for multiple breakpoints in Agent and Pipeline * wip: adding test for multiple breakpoints in Agent and Pipeline * adding tests for agent in pipeline * nit * reverting validate * fixing agent in pipeline tests * Update haystack_experimental/components/agents/agent.py Co-authored-by: Sebastian Husch Lee <[email protected]> * having one single BreakpointExecption * parameters * wip: fixing async tests * nit: cleaning * fixing imports * reverting back to a single Breakpoint * updating all tests due to reverting back to a single Breakpoint * updating all tests due to reverting back to a single Breakpoint * WIP: fixing tests * updating tests without break_on_first and properly catching all the execptions * updating integratiosn tests, reverting back to a single break point * enabling back linter rules * wip: resuming a pipeline agent, saving agent_name to state * cleaning up * all tests passing - working on pipeline agent resume from state * updating tests to consider mandatory agent_name when having an Agent breakpoint * serialisation with success of the pipeline where the agent is running plus the agent status serialisations into the the same resume JSON file * updating linting and types due to new changes * updating all tests due to new changes * resuming an Agent within a pipeline * replacing Path by PosixPath for Windows compatability * reverting back to Path * wip * cleaning up - small improvments * cleaning up validate pipeline * simplifying the save state * reducing parameters for save_state agent case * saving all the main pipeline data nested under a single key * more cleaning and small improvments * helper function to handle the processing the state of the host pipeline where agent is running * helper function to handle file saving * adding tests for agent in pipeline break and resume on a tool * refactoring pipeline.run * refactoring pipeline methods * removing hard-coded path for JSON resume files, using pytest tmpfile * wip * review commments/improvments * review commments/improvments + updating tests * Fix validate_breakpoint * moving static methods to breakpoint.py * Update haystack_experimental/core/pipeline/pipeline.py Co-authored-by: Amna Mubashar <[email protected]> * fixing LICENSE header in breakpoint.py * Update haystack_experimental/dataclasses/breakpoints.py Co-authored-by: Sebastian Husch Lee <[email protected]> * Small changes * wip * attending PR comments * attending PR comments * homogenization of var names and function names --------- Co-authored-by: Sebastian Husch Lee <[email protected]> Co-authored-by: Amna Mubashar <[email protected]> Co-authored-by: Sebastian Husch Lee <[email protected]>
sjrl
approved these changes
Jul 14, 2025
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.
Proposed Changes:
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:.