Skip to content

Make owner_type optional and fix add_project_item in consolidated Projects V2 tools #1799

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.

Sign up for GitHub

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
Copilot wants to merge 1 commit into
base: main
Choose a base branch
from
+385 −96
Open

Make owner_type optional and fix add_project_item in consolidated Projects V2 tools #1799

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions pkg/github/__toolsnaps__/assign_copilot_to_issue.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
"type": "string"
},
"custom_instructions": {
"type": "string",
"description": "Optional custom instructions to guide the agent beyond the issue body. Use this to provide additional context, constraints, or guidance that is not captured in the issue description"
"description": "Optional custom instructions to guide the agent beyond the issue body. Use this to provide additional context, constraints, or guidance that is not captured in the issue description",
"type": "string"
},
"issue_number": {
"description": "Issue number",
Expand Down
5 changes: 2 additions & 3 deletions pkg/github/__toolsnaps__/projects_get.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@
"type": "string"
},
"owner": {
"description": "If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.",
"description": "The owner (user or organization login). The name is not case sensitive.",
"type": "string"
},
"owner_type": {
"description": "Owner type",
"description": "Owner type (user or org). If not provided, will be automatically detected.",
"enum": [
"user",
"org"
Expand All @@ -49,7 +49,6 @@
},
"required": [
"method",
"owner_type",
"owner",
"project_number"
],
Expand Down
5 changes: 2 additions & 3 deletions pkg/github/__toolsnaps__/projects_list.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@
"type": "string"
},
"owner": {
"description": "If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.",
"description": "The owner (user or organization login). The name is not case sensitive.",
"type": "string"
},
"owner_type": {
"description": "Owner type",
"description": "Owner type (user or org). If not provided, will automatically try both.",
"enum": [
"user",
"org"
Expand All @@ -57,7 +57,6 @@
},
"required": [
"method",
"owner_type",
"owner"
],
"type": "object"
Expand Down
23 changes: 19 additions & 4 deletions pkg/github/__toolsnaps__/projects_write.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,22 @@
"description": "Add, update, or delete project items in a GitHub Project.",
"inputSchema": {
"properties": {
"issue_number": {
"description": "The issue number (use when item_type is 'issue' for 'add_project_item' method). Provide either issue_number or pull_request_number.",
"type": "number"
},
"item_id": {
"description": "The project item ID. Required for 'update_project_item' and 'delete_project_item' methods. For add_project_item, this is the numeric ID of the issue or pull request to add.",
"description": "The project item ID. Required for 'update_project_item' and 'delete_project_item' methods.",
"type": "number"
},
"item_owner": {
"description": "The owner (user or organization) of the repository containing the issue or pull request. Required for 'add_project_item' method.",
"type": "string"
},
"item_repo": {
"description": "The name of the repository containing the issue or pull request. Required for 'add_project_item' method.",
"type": "string"
},
"item_type": {
"description": "The item's type, either issue or pull_request. Required for 'add_project_item' method.",
"enum": [
Expand All @@ -28,11 +40,11 @@
"type": "string"
},
"owner": {
"description": "If owner_type == user it is the handle for the GitHub user account. If owner_type == org it is the name of the organization. The name is not case sensitive.",
"description": "The project owner (user or organization login). The name is not case sensitive.",
"type": "string"
},
"owner_type": {
"description": "Owner type",
"description": "Owner type (user or org). If not provided, will be automatically detected.",
"enum": [
"user",
"org"
Expand All @@ -43,14 +55,17 @@
"description": "The project's number.",
"type": "number"
},
"pull_request_number": {
"description": "The pull request number (use when item_type is 'pull_request' for 'add_project_item' method). Provide either issue_number or pull_request_number.",
"type": "number"
},
"updated_field": {
"description": "Object consisting of the ID of the project field to update and the new value for the field. To clear the field, set value to null. Example: {\"id\": 123456, \"value\": \"New Value\"}. Required for 'update_project_item' method.",
"type": "object"
}
},
"required": [
"method",
"owner_type",
"owner",
"project_number"
],
Expand Down
1 change: 1 addition & 0 deletions pkg/github/minimal_types.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ type MinimalProject struct {
Number *int `json:"number,omitempty"`
ShortDescription *string `json:"short_description,omitempty"`
DeletedBy *MinimalUser `json:"deleted_by,omitempty"`
OwnerType string `json:"owner_type,omitempty"`
}

// Helper functions
Expand Down
Loading