Skip to content

Commit c7e7ef5

Browse files
authored
Merge pull request microsoft#273113 from microsoft/tyriar/multi-line
Remove multi-line hint from terminal tool desc
2 parents 03827b4 + 838f5bf commit c7e7ef5

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ function createPowerShellModelDescription(shell: string): string {
9595

9696
const genericDescription = `
9797
Command Execution:
98-
- Does NOT support multi-line commands
9998
- Use && to chain simple commands on one line
10099
- Prefer pipelines | over temporary files for data flow
101100
- Never create a sub-shell (eg. bash -c "command") unless explicitly asked

src/vs/workbench/contrib/terminalContrib/chatAgentTools/test/electron-browser/treeSitterCommandParser.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ suite('TreeSitterCommandParser', () => {
9090
test('mixed quote types', () => t('echo "hello \`world\`" && echo \'test\'', ['echo "hello \`world\`"', 'world', 'echo \'test\'']));
9191
test('deeply nested structures', () => t('echo $(echo $(echo $(echo nested))) && ls', ['echo $(echo $(echo $(echo nested)))', 'echo $(echo $(echo nested))', 'echo $(echo nested)', 'echo nested', 'ls']));
9292
test('unicode command names', () => t('测试命令 && echo done', ['测试命令', 'echo done']));
93+
test('multi-line', () => t('echo a\necho b', ['echo a', 'echo b']));
9394
});
9495

9596
// TODO: These should be common but the pwsh grammar doesn't handle && yet https://github.com/microsoft/vscode/issues/272704

0 commit comments

Comments
 (0)