Skip to content

feat: Create issue tool #2

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: Create issue tool #2

wants to merge 3 commits into from

Conversation

gsr25
Copy link

@gsr25 gsr25 commented Apr 12, 2025

No description provided.

Copy link
Collaborator

@kpsunil97 kpsunil97 left a comment

Choose a reason for hiding this comment

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

Could you please update the description with the added changes? Also, please add the example screenshot with the server running locally.

@@ -0,0 +1 @@
* @kpsunil97
Copy link
Collaborator

Choose a reason for hiding this comment

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

add newline

@@ -0,0 +1 @@
deployable: true
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we need this? this is not deployable.

- `create_issue`: Creates a DevRev issue with a specified title and part. Parameters:
- `title`: The issue title (required)
- `part_name`: Name of the part to associate with the issue (required).
- The issue will be automatically assigned to the currently authenticated user
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we add the owner too which if not provided can be assigned to the current user?

@@ -37,3 +37,56 @@ def make_devrev_request(endpoint: str, payload: Dict[str, Any]) -> requests.Resp
headers=headers,
json=payload
)

def search_part_by_name(part_name: str) -> Tuple[bool, Optional[str], Optional[str]]:
Copy link
Collaborator

Choose a reason for hiding this comment

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

instead allow part as namespace in search tool. agent would figure out the relevant part while searching.

"type": "object",
"properties": {
"title": {"type": "string"},
"part_name": {"type": "string"},
Copy link
Collaborator

Choose a reason for hiding this comment

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

add description too, easier of the fields to be predicted by agent. move part to part id and let agent figure out the part id from the name with the search tool.

@kpsunil97 kpsunil97 mentioned this pull request Apr 25, 2025
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