Skip to content

Commit 7c9d2c1

Browse files
Remove duplicate chat refs and ensure to include summary text if there is a prompt (#2316)
* Remove duplicate chat refs and ensure to include summary text if there is a prompt * Update src/extension/agents/copilotcli/common/delegationSummaryService.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 30f15d8 commit 7c9d2c1

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/extension/agents/copilotcli/common/delegationSummaryService.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,9 @@ export class ChatDelegationSummaryService implements IChatDelegationSummaryServi
7373
return undefined;
7474
}
7575
const uri = URI.from({ scheme: SummaryFileScheme, path: l10n.t("summary"), query: sessionId });
76-
const prompt = message.substring(0, index).trimEnd() || l10n.t('Complete the task as described in the {0}', `[summary](${uri.toString()})`);
76+
const promptSuffix = l10n.t('Complete the task as described in the {0}', `[summary](${uri.toString()})`);
77+
const promptPrefix = message.substring(0, index).trimEnd() || '';
78+
const prompt = promptPrefix ? `${promptPrefix}\n\n${promptSuffix}` : promptSuffix;
7779
const summary = message.substring(index);
7880
this._summaries.set(uri, summary);
7981
const reference: ChatPromptReference = {

src/extension/agents/copilotcli/node/copilotcliPromptResolver.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export class CopilotCLIPromptResolver {
3737
* @returns
3838
*/
3939
public async resolvePrompt(request: vscode.ChatRequest, prompt: string | undefined, additionalReferences: vscode.ChatPromptReference[], isIsolationEnabled: boolean, token: vscode.CancellationToken): Promise<{ prompt: string; attachments: Attachment[] }> {
40-
const references = request.references.concat(additionalReferences);
40+
const references = request.references.concat(additionalReferences.filter(ref => !request.references.includes(ref)));
4141
prompt = prompt ?? request.prompt;
4242
if (prompt.startsWith('/')) {
4343
return { prompt, attachments: [] }; // likely a slash command, don't modify

0 commit comments

Comments
 (0)