Skip to content

Commit 118bb94

Browse files
authored
Pull request tool call feedback (#7503)
Fixes #7502
1 parent f47aa3d commit 118bb94

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

src/lm/tools/activePullRequestTool.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ export class ActivePullRequestTool implements vscode.LanguageModelTool<FetchIssu
3434
async prepareInvocation(): Promise<vscode.PreparedToolInvocation> {
3535
const pullRequest = this._findActivePullRequest();
3636
return {
37-
invocationMessage: pullRequest ? vscode.l10n.t('Pull request "{0}"', pullRequest.title) : vscode.l10n.t('Active pull request'),
37+
pastTenseMessage: pullRequest ? vscode.l10n.t('Read pull request "{0}"', pullRequest.title) : vscode.l10n.t('No active pull request'),
38+
invocationMessage: pullRequest ? vscode.l10n.t('Reading pull request "{0}"', pullRequest.title) : vscode.l10n.t('Reading active pull request'),
3839
};
3940
}
4041

@@ -107,7 +108,7 @@ export class ActivePullRequestTool implements vscode.LanguageModelTool<FetchIssu
107108
return copilotSteps;
108109
}
109110

110-
async invoke(options: vscode.LanguageModelToolInvocationOptions<any>, token: vscode.CancellationToken): Promise<vscode.LanguageModelToolResult | undefined> {
111+
async invoke(options: vscode.LanguageModelToolInvocationOptions<any>, token: vscode.CancellationToken): Promise<vscode.ExtendedLanguageModelToolResult | undefined> {
111112
let pullRequest = this._findActivePullRequest();
112113

113114
if (!pullRequest) {
@@ -159,8 +160,9 @@ export class ActivePullRequestTool implements vscode.LanguageModelTool<FetchIssu
159160
})
160161
};
161162

162-
return new vscode.LanguageModelToolResult([new vscode.LanguageModelTextPart(JSON.stringify(pullRequestInfo))]);
163-
163+
const result = new vscode.ExtendedLanguageModelToolResult([new vscode.LanguageModelTextPart(JSON.stringify(pullRequestInfo))]);
164+
result.toolResultDetails = [vscode.Uri.parse(pullRequest.html_url)];
165+
return result;
164166
}
165167

166168
}

0 commit comments

Comments
 (0)