[enhance] Workflow Saving Update Existing md Files #3466
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
fengju0213
left a comment
fengju0213
left a comment
There was a problem hiding this comment.
LGTM！ @hesamsheikh sorry for late reply！！
| target_workflow_filename: Optional[str] = Field( | ||
| description=( | ||
| "When operation_mode is 'update', specify the exact filename " | ||
| "(without .md extension) of the loaded workflow to update. " | ||
| "Must match one of the loaded workflow filenames exactly. " | ||
| "Leave empty or null when operation_mode is 'create'." | ||
| ), | ||
| default=None, | ||
| ) |
There was a problem hiding this comment.
In your opinion, do you think if given the file name we can always assume its update ? @hesamsheikh
a7m-1st
left a comment
a7m-1st
left a comment
There was a problem hiding this comment.
LGTM, it looks like the main update is operation_mode update or create 👍
Description
Fixes #3377
If the new process is similar to the workflow process, update it instead of creating new ones. this helps maintain the number of workflow memories.