Commit Graph

474 Commits

Author SHA1 Message Date
Raymond Zhao
9a6ef82833 chore: suppress CodeQL warnings from fixtures (#602) 2025-08-14 17:32:25 +00:00
Matt Bierner
827b844361 Log path to chunks db (#601) 2025-08-14 16:49:45 +00:00
Megan Rogge
e3aa82bb19 fix #261659 (#600) 2025-08-14 16:36:10 +00:00
Christof Marti
563d2798d6 Remove unused dependency (#599) 2025-08-14 15:22:23 +00:00
Matt Bierner
21aba584ae Unify embeddings paths (#595)
* Unify embeddings paths

Our code has two embeddings paths for legacy reasons:

- The capi api based one that only supports text3small. This is used everywhere except `#codebase`

- The new github embeddings endpoint based. This one supports additional embedding types and options. However it is currently only used in `#codebase`

This change switches everything to use the new github embeddings endpoint instead

* Fix static deps in tests
2025-08-14 14:34:41 +00:00
Don Jayamanne
1b307495ba Treat markdown cells in Nobteooks as separate text docs (#597) 2025-08-14 09:14:18 +00:00
Don Jayamanne
91c4a8b881 Tests for tranlating positions with markdown cells included (#596) 2025-08-14 09:13:57 +00:00
Benjamin Christopher Simmonds
05ae67c48e cleanup property translation (#598) 2025-08-14 07:58:26 +00:00
Connor Peet
83803f9192 tools: add multi-replace-string tool (#593)
* add multi edit tool

* comment out reminder

* comment out reminder

* fix multi_replace_string_in_file issue in intents

* add multi_replace_string_in_file related instructions

* add more instructions

* remove next tool prediction parsing

* remove next_tool_prediction from toolSchemaNormalizer.ts

* add initial user reminder

* add hasMultiReplaceString

* add hasMultiReplaceString

* add references to MultiReplaceStringTool

* Delete test_implementation.js

* Delete test_dummy_parameter.js

* Delete test_required_dummy.js

* Delete verify_implementation.js

* Delete src/extension/tools/test/common/addNextToolPredictionParameter.spec.ts

* Fix formatting of multi-replace string reminder

* tidy up and exp

* eng: refactor multi and single edit tools

---------

Co-authored-by: Yevhen Mohylevskyy <yevhenmohylevskyy@Yevhens-MacBook-Pro.local>
Co-authored-by: Ubuntu <yevhen@ubuntu22-vm.1kpbk2tnedsetibfv2d2keui5d.xx.internal.cloudapp.net>
Co-authored-by: yemohyleyemohyle <127880594+yemohyleyemohyle@users.noreply.github.com>
2025-08-13 23:39:09 +00:00
Matt Bierner
33400a0e64 Fallback to in-memory sqllite db in all cases (#591)
This lets up remove a bunch of logic for the old  disk cache because it only needs to be read and then eventually deleted
2025-08-13 23:36:50 +00:00
Rob Lourens
76434b47a3 Remove this instruction (#592) 2025-08-13 23:13:35 +00:00
Matt Bierner
5a54ce2b95 Add better support for new embeddings model (#589) 2025-08-13 22:38:51 +00:00
Christof Marti
fb15e054b1 Log additional information as JSON (#585) 2025-08-13 20:18:33 +00:00
Connor Peet
565ea9543b eng: disable nuget mcp tests for now (#588)
These depend on .net being installed the machine which is not a requirement for vscode chat developers.

Ref: https://github.com/microsoft/vscode/issues/261524
2025-08-13 20:12:03 +00:00
Bhavya U
84c9c4d560 Fix up exp config for gpt-5 alt prompts (#587)
* Fix up exp config for gpt-5 alt prompts

* Add experimental support for GPT-5 alternate prompt configuration
2025-08-13 18:57:17 +00:00
Benjamin Christopher Simmonds
5710037626 translate the properties so github can handle them (#586) 2025-08-13 17:17:16 +00:00
Joel Verhagen
05e64e6d69 Use .NET SDK to search for NuGet packages, emit events, add tests for all package types (#546)
* Extract server.json from NuGet package during assisted install

* Use .NET CLI as much as possible

* Add tests, use GitHub Actions for .NET setup

* Localize user facing strings

* Update src/extension/mcp/vscode-node/nuget.ts

* Update src/extension/mcp/vscode-node/commands.ts

* switch to enum

* Use IFetcherService for npm, PyPI, and Docker Hub, use local package source for NuGet
2025-08-13 17:04:03 +00:00
Alex Ross
3434ba9180 Open and active PR may not be the same (#584)
Part of microsoft/vscode-pull-request-github#7332
2025-08-13 15:46:28 +00:00
Isidor Nikolic
ec9e85369e update auto string (#566) 2025-08-13 12:30:14 +00:00
Don Jayamanne
8d5210888d Remove explanation and code mapper in notebook Edit tool (#583) 2025-08-13 09:36:39 +00:00
Don Jayamanne
ff1e17926c Fallback to notebook editing tool for notebook failures (#581) 2025-08-13 09:20:18 +00:00
Don Jayamanne
725ce254d3 Ensure code markers are not present when editing cells (#582) 2025-08-13 09:20:07 +00:00
Don Jayamanne
fa0c4728b7 Ensure only enabled notebook tools are referenced (#578) 2025-08-13 07:03:38 +00:00
Bhavya U
38690b2f0a Refine model descriptions for VS Code tools to enhance clarity and usage guidelines (#579) 2025-08-13 06:28:47 +00:00
Justin Chen
ea6d7ca865 don't send random thinking tokens all the time (#576)
* don't send random thinking tokens

* change other classes too

* update tests

* cleanup

* fix tests

* fix ts check
2025-08-13 06:10:22 +00:00
Megan Rogge
64a0cbb6d5 allow tasks w dependencies to be run by the agent (#574)
* allow tasks w dependencies to be run by the agent

* dependsOn
2025-08-12 23:08:15 +00:00
Matt Bierner
6bd7610c19 Log which embedding type is used (#558) 2025-08-12 22:10:09 +00:00
Justin Chen
fb9981037b use image uploader in chat (#499)
* image uploader in chat extension

* fix comment

* cleanup

* use capi client + extract into own service

* capi version bump + injected service

* cleanup

* add service in promptrenderer

* add null image service for tets

* add null service in vscode node too

* trying to remove unsued service in prompt renderer

* cleanup spacing

* use urlREquestMetaData instead

* remove vendor

* remove whitespace

* some more cleanup

* more robust mimetype check

* add exp setting
2025-08-12 21:36:49 +00:00
Connor Peet
ad4d063779 prompts: avoid wrapping file attachments in code fences (#569)
* prompts: avoid wrapping file attachments in code fences

We previously included attached files like this

```
<attachment id="test.js" filePath="/home/jola/test.js">
\```javascript
bar

asd (
\```
</attachment>
```

Which, very reasonably, caused the model to get confused and think that
the file actually contained those backticks. This change removes that
so that it's just

```
<attachment id="test.js" filePath="/home/jola/test.js">
bar

asd (
</attachment>
```

Refs https://github.com/microsoft/vscode/issues/260772#issuecomment-3176793591

Busted all the caches with associated baseline noise.

* fixup
2025-08-12 19:06:46 +00:00
Matt Bierner
c5e4fdf11e Clarify that simple browser only supports http and https uris (#571)
Fixes microsoft/vscode#261082
2025-08-12 18:42:27 +00:00
Bhavya U
5eb47c8b98 Add GPT-5 specific safety rules and update agent prompt (#570) 2025-08-12 18:40:05 +00:00
Bhavya U
bb7f5fef4e add support for alternate prompt configuration (#564)
* add support for alternate prompts configuration

* Fix up setting

* update configuration key path

* refactor: enhance GPT5 prompt instructions for clarity and user engagement

* feat: add configuration option to skip reminder instructions in user messages

* refactor: update output formatting instructions for clarity and consistency

* fix: correct configuration key name for GPT-5 alternate prompts
2025-08-12 07:01:38 +00:00
Bhavya U
53e7471f83 Add GPT-5 specific identity rule to agent prompt (#565) 2025-08-12 05:07:15 +00:00
Rob Lourens
40186ec82c This log is noisy and seems unnecessary (#562) 2025-08-12 02:30:03 +00:00
Avinash Sridhar
4d72283d35 Add mode to conversation model messages (#517)
* Add mode to conversation model messages

This updates the conversation.message with source=model to include `mode` details in the telemetry

* Add "mode" field to snapshot responses and update imports in tests
2025-08-12 01:44:36 +00:00
Rob Lourens
a4712f5dc4 ToolNames cleanup (#561)
* ToolNames cleanup

* Restore this name

* Put back

* Put this back
2025-08-12 01:37:15 +00:00
Bhavya U
9cff36b6ee update model family checks (#556)
* fix: update model family checks

* fix: simplify model family checks
2025-08-12 00:56:30 +00:00
Shaun Miller
800683118f Remove the message copilot_cache_control field from a context where there is no understanding of it (#554)
* Remove copilot_cache_control field from message body when calling endpoints directly. It's a CAPI field and some LLMs will issue an error on this field because it isn't a recognized field.

* remove unneeded conditional expressions
2025-08-12 00:09:33 +00:00
Bhavya U
b6bb7bbb48 fix: update guidance for task execution and identity disclosure in agent prompts (#560) 2025-08-11 23:13:32 +00:00
Daniel Imms
e30b068b22 Move terminal selection/command tools to core (#545)
* Move terminal selection/command tools to core

Part of microsoft/vscode#259260

* Update baseline
2025-08-11 19:11:17 +00:00
Logan Ramos
c462c8ee64 Fix API key not persisting (#549) 2025-08-11 17:43:12 +00:00
Bhavya U
125692d6b6 add CoreManageTodoList tool to packagejson tools for testing (#548)
* add CoreManageTodoList tool to packagejson tools for testing

* fix: update references from CoreTodoListTool to CoreManageTodoList
2025-08-11 16:07:26 +00:00
Connor Peet
aaddf573d4 fix: referencing wrong edit tool with no string_replace (#547)
No model actually uses this config currently, but fix the instructions nevertheless

Closes https://github.com/microsoft/vscode/issues/258888
2025-08-11 15:20:11 +00:00
Don Jayamanne
8bf241b6f1 Merge notebook and text observable document interfaces (#544) 2025-08-11 10:41:03 +00:00
Don Jayamanne
d2713c5e3e Add some simple NES tests for alt format of notebooks (#543)
* Add some simple NES tests for alt format of notebooks

* Updates

* Updates
2025-08-11 10:40:52 +00:00
Don Jayamanne
4286142f1f Enable notebook summary when notebook edit tool is used (#540) 2025-08-11 10:40:40 +00:00
Benjamin Christopher Simmonds
1e803a5033 fixes https://github.com/microsoft/vscode/issues/260925 (#541) 2025-08-11 08:12:34 +00:00
Rob Lourens
7e0880b08b Don't add empty #thinking sections to logs (#537) 2025-08-10 06:04:52 +00:00
Rob Lourens
890fcfcf4e Add debug view filters for NES requests (#536)
* Initial plan

* Add NES requests filter implementation

Co-authored-by: roblourens <323878+roblourens@users.noreply.github.com>

* Refine NES request identification logic to avoid false positives

Co-authored-by: roblourens <323878+roblourens@users.noreply.github.com>

* Simplify

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: roblourens <323878+roblourens@users.noreply.github.com>
2025-08-10 05:17:53 +00:00
Burke Holland
f11dd07cb2 Add core Beast Mode functionality to GPT-4.1/5 agent prompt (#467)
* Add GPT-4.1 agent prompt and enhance tool capabilities

Implemented the GPT-4.1 specific agent prompt to improve multi-step task execution and problem-solving. Updated the DefaultAgentPrompt to conditionally render the new GPT41AgentPrompt based on the model family. Enhanced the tool capabilities detection function to streamline tool usage and added a new tool name for managing todo lists.

* Add setting for new prompt, tweaks

* Cleanup

---------

Co-authored-by: Rob Lourens <roblourens@gmail.com>
2025-08-09 22:55:04 +00:00