Skip to content

Commit daacdaa

Browse files
committed
Include PR number in PR tools
Fixes #7884
1 parent 227200b commit daacdaa

1 file changed

Lines changed: 16 additions & 3 deletions

File tree

src/lm/tools/activePullRequestTool.ts

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,25 @@ export abstract class PullRequestTool implements vscode.LanguageModelTool<FetchI
2828
return !!pullRequest && this.copilotRemoteAgentManager.enabled && COPILOT_LOGINS.includes(pullRequest.author.login);
2929
}
3030

31+
private _getPullRequestLabel(pullRequest: PullRequestModel): string {
32+
return `${pullRequest.title} (#${pullRequest.number})`;
33+
}
34+
3135
async prepareInvocation(): Promise<vscode.PreparedToolInvocation> {
3236
const pullRequest = this._findActivePullRequest();
37+
if (!pullRequest) {
38+
return {
39+
pastTenseMessage: vscode.l10n.t('No active pull request'),
40+
invocationMessage: vscode.l10n.t('Reading active pull request'),
41+
confirmationMessages: { title: this._confirmationTitle(), message: vscode.l10n.t('Allow reading the details of the active pull request?') },
42+
};
43+
}
44+
45+
const label = this._getPullRequestLabel(pullRequest);
3346
return {
34-
pastTenseMessage: pullRequest ? vscode.l10n.t('Read pull request "{0}"', pullRequest.title) : vscode.l10n.t('No active pull request'),
35-
invocationMessage: pullRequest ? vscode.l10n.t('Reading pull request "{0}"', pullRequest.title) : vscode.l10n.t('Reading active pull request'),
36-
confirmationMessages: { title: this._confirmationTitle(), message: pullRequest ? vscode.l10n.t('Allow reading the details of "{0}"?', pullRequest.title) : vscode.l10n.t('Allow reading the details of the active pull request?') },
47+
pastTenseMessage: vscode.l10n.t('Read pull request "{0}"', label),
48+
invocationMessage: vscode.l10n.t('Reading pull request "{0}"', label),
49+
confirmationMessages: { title: this._confirmationTitle(), message: vscode.l10n.t('Allow reading the details of "{0}"?', label) },
3750
};
3851
}
3952

0 commit comments

Comments
 (0)