Skip to content

Conversation

ncapek
Copy link
Contributor

@ncapek ncapek commented Jul 12, 2025

This PR implements the following changes to flow tools:

  1. refactor related to new models in Configuration models
  2. add the conditional flow config schema
  3. updates tools to handle conditional flows

@ncapek ncapek marked this pull request as draft July 12, 2025 22:10
@ncapek ncapek requested a review from vita-stejskal August 1, 2025 14:30
@ncapek ncapek requested a review from mariankrotil August 4, 2025 08:34
Copy link
Contributor

@mariankrotil mariankrotil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving. I still think we might not need to maintain all the classes for the Conditional Flow configuration and using just the schema could be a simpler and more efficient approach.

Additionally, we should assign a new task addressing the refactoring for the client.py module as it got very large to split it into more submodules with individual api models.

@ncapek ncapek merged commit af1dbdf into main Aug 4, 2025
21 of 24 checks passed
@ncapek ncapek deleted the AI-1280-conditional-flows branch August 4, 2025 11:16
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.

5 participants