Skip to content

Added valid_stage_transition tool to suggest possible stage transitions for an object #9

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

Closed
wants to merge 8 commits into from

Conversation

floatingIce91
Copy link
Contributor

Added a new tool which can retrieve the valid stages an object can transition too. This tool is used to add the ability to update stage of an object, using the update_object tool.

@floatingIce91 floatingIce91 requested a review from a team as a code owner May 20, 2025 12:12
@shivam-gupta-devrev shivam-gupta-devrev changed the title Added valid_stage_transition tool and ability to update stage using u… Added valid_stage_transition tool to suggest possible stage transitions for an object May 20, 2025
},
"required": ["id", "type"],
},
),
types.Tool(
name="valid_stage_transitions",
description="Get all the possible stage transitions for a current stage of a given object ID",
Copy link
Contributor

Choose a reason for hiding this comment

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

Update the description to mention that we can do this only for issues & tickets today.

Comment on lines +289 to +291
"custom_schema_fragment_ids": custom_schema_frag_id,
"leaf_type": leaf_type,
"stock_schema_fragment_id": stock_schema_frag_id
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: indentation.

leaf_type = response.json().get("work", {}).get("type", {})

schema_response = make_internal_devrev_request(
"schemas.aggregated.get",
Copy link
Contributor

Choose a reason for hiding this comment

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

@kpsunil97 is it fine to expose internal APIs here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No, we won't have any way to track the clients and provide a resolution in case of a backward incompatibility for an internal endpoint.

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.

3 participants