Skip to content

Commit 04ead50

Browse files
authored
mcp: inline multiple message roles in chat, for now (microsoft#250071)
1 parent 33ac9ec commit 04ead50

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

src/vs/workbench/contrib/chat/browser/contrib/chatInputCompletions.ts

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -663,14 +663,19 @@ class StartParameterizedPromptAction extends Action2 {
663663
}
664664
};
665665

666+
const hasMultipleRoles = messages.some(m => m.role !== messages[0].role);
666667
let input = '';
667668
for (const message of messages) {
668-
if (message.role === 'assistant') {
669-
continue; // would we ever support these?
670-
}
671669
switch (message.content.type) {
672670
case 'text':
673-
input += (input ? '\n' : '') + message.content.text;
671+
if (input) {
672+
input += '\n\n';
673+
}
674+
if (hasMultipleRoles) {
675+
input += `--${message.role.toUpperCase()}\n`;
676+
}
677+
678+
input += message.content.text;
674679
break;
675680
case 'resource':
676681
if ('text' in message.content.resource) {

0 commit comments

Comments
 (0)