Skip to content

Commit f978539

Browse files
committed
Server instructions change
1 parent 4035140 commit f978539

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

internal/githubv4mock/githubv4mock.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ func NewMockedHTTPClient(ms ...Matcher) *http.Client {
167167
return
168168
}
169169
defer func() { _ = r.Body.Close() }()
170-
170+
171171
matcher, ok := matchers[gqlRequest.Query]
172172
if !ok {
173173
http.Error(w, fmt.Sprintf("no matcher found for query %s", gqlRequest.Query), http.StatusNotFound)

pkg/github/instructions.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ Context management:
4848
func getToolsetInstructions(toolset string) string {
4949
switch toolset {
5050
case "pull_requests":
51-
return "## Pull Requests\n\nPR review workflow: Always use 'create_pending_pull_request_review' → 'add_comment_to_pending_review' → 'submit_pending_pull_request_review' for complex reviews with line-specific comments."
51+
return "## Pull Requests\n\nPR review workflow: Always use 'pull_request_review_write' with method 'create' and no 'event' parameter to create a pending review, then 'add_comment_to_pending_review' to add comments, and finally 'pull_request_review_write' with method 'submit_pending' to submit the review for complex reviews with line-specific comments."
5252
case "issues":
5353
return "## Issues\n\nCheck 'list_issue_types' first for organizations to use proper issue types. Use 'search_issues' before creating new issues to avoid duplicates. Always set 'state_reason' when closing issues."
5454
case "discussions":

0 commit comments

Comments
 (0)