Skip to content
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

feat(tasks): Update execution transition relation #397

Merged
merged 5 commits into from
Jun 22, 2024

Conversation

creatorrr
Copy link
Contributor

@creatorrr creatorrr commented Jun 22, 2024

  • wip
  • wip
  • wip
  • feat(tasks): Update execution transition relation

🚀 This description was created by Ellipsis for commit 6fdd429

Summary:

Added metadata fields to various models and updated corresponding schemas and functions in Python and TypeScript SDKs.

Key points:

  • Added metadata field to ExecutionTransition, Agent, CreateAgentRequest, CreateDoc, Session, User, and related models in sdks/python/julep/api/types.
  • Updated create_execution_transition_query in agents-api/agents_api/models/execution/create_execution_transition.py to include metadata and task_token.
  • Updated get_execution_transition_query in agents-api/agents_api/models/execution/get_execution_transition.py to include metadata, task_token, created_at, and updated_at.
  • Modified migrate_1716939839_task_relations.py to add metadata and task_token fields.
  • Updated TypeScript SDK models and schemas to include metadata fields.
  • Removed specific metadata types in Python SDK and replaced with generic Dict[str, Any].
  • Updated uuid_int_list_to_uuid4 in agents-api/agents_api/common/utils/cozo.py to a standard function definition.

Generated with ❤️ by ellipsis.dev

Diwank Tomer and others added 5 commits June 20, 2024 23:08
wip
Signed-off-by: Diwank Tomer <[email protected]>
wip
Signed-off-by: Diwank Tomer <[email protected]>
wip
Signed-off-by: Julep Developers <[email protected]>
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 6fdd429 in 49 seconds

More details
  • Looked at 2433 lines of code in 62 files
  • Skipped 1 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. agents-api/agents_api/autogen/openapi_model.py:1160
  • Draft comment:
    The changes to the ExecutionTransition model, including the addition of task_token, metadata, created_at, and updated_at, are correctly implemented in both the Python and TypeScript SDKs. This ensures consistency across different platforms and aligns with the updated API specifications.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The PR introduces changes to the ExecutionTransition model in the Python SDK, adding new fields such as task_token, metadata, created_at, and updated_at. These changes are reflected in the TypeScript SDK as well. The PR also updates various schemas in the TypeScript SDK to include these new fields. The changes are consistent across both SDKs, ensuring that the data models remain synchronized between different implementations.

Workflow ID: wflow_HqKzq1ovLrnp72ck


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@whiterabbit1983 whiterabbit1983 merged commit a6b17e9 into dev-tasks Jun 22, 2024
3 of 6 checks passed
@whiterabbit1983 whiterabbit1983 deleted the f/tasks-steps branch June 22, 2024 05:36
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.

2 participants