-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add minItems to ToolkitTask actions schema #725
Conversation
d7a63f2
to
781d6e6
Compare
@@ -10,6 +10,133 @@ | |||
|
|||
|
|||
class TestToolTask: | |||
TARGET_TOOLS_SCHEMA = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Originally added this to validate the existence of minItems
in the schema, but now it is being added during prompt rendering. Can remove these tests, but I think there is value in having them.
781d6e6
to
0407d08
Compare
griptape/tasks/actions_subtask.py
Outdated
) | ||
|
||
self.actions.append( | ||
self.__error_to_action(f"Activity input JSON validation error: {error_message}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we raise a schema.SchemaError
instead so it's more consistent with the rest of the validation logic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call, updated.
0407d08
to
b6a524d
Compare
b6a524d
to
332eee0
Compare
No description provided.