certs

.github/scripts/issue_triage.py

@@ -1,164 +0,0 @@
-#!/usr/bin/env python
-from __future__ import annotations
-import os, sys, json, datetime, pathlib, textwrap, requests
-from openai import OpenAI
-
-REPO = "voideditor/void"
-CACHE_FILE = pathlib.Path(".github/triage_cache.json")
-STAMP_FILE = pathlib.Path(".github/last_triage.txt")
-
-THEMES_MD = textwrap.dedent("""\
-1. 🔗 LLM Integration & Provider Support
-2. 🖥 App Build & Platform Compatibility
-3. 🎯 Prompt, Token, and Cost Management
-4. 🧩 Editor UX & Interaction Design
-5. 🤖 Agent & Automation Features
-6. ⚙️ System Config & Environment Setup
-7. 🗃 Meta: Feature Comparison, Structure, and Naming
-""").strip()
-
-client  = OpenAI(api_key=os.environ["OPENAI_API_KEY"])
-headers = {"Authorization": f"Bearer {os.environ['GITHUB_TOKEN']}"}
-
-
-# ───────── helpers ────────────────────────────────────────────────────────
-def utc_iso_now() -> str:
-    return datetime.datetime.utcnow().replace(microsecond=0, tzinfo=datetime.timezone.utc).isoformat()
-
-def read_stamp() -> str:
-    return STAMP_FILE.read_text().strip() if STAMP_FILE.exists() else "1970-01-01T00:00:00Z"
-
-def save_stamp():
-    STAMP_FILE.parent.mkdir(parents=True, exist_ok=True)
-    STAMP_FILE.write_text(utc_iso_now())
-
-def load_cache() -> dict[int, str]:
-    return json.loads(CACHE_FILE.read_text()) if CACHE_FILE.exists() else {}
-
-def save_cache(d: dict[int, str]):
-    CACHE_FILE.parent.mkdir(parents=True, exist_ok=True)
-    CACHE_FILE.write_text(json.dumps(d, indent=2))
-
-def fetch_open_issues(since_iso: str | None = None) -> list[dict]:
-    issues, page = [], 1
-    while True:
-        url = (
-            f"https://api.github.com/repos/{REPO}/issues"
-            f"?state=open&per_page=100&page={page}"
-            + (f"&since={since_iso}" if since_iso else "")
-        )
-        chunk = requests.get(url, headers=headers).json()
-        if not chunk or (isinstance(chunk, dict) and chunk.get("message")):
-            break
-        issues.extend(i for i in chunk if "pull_request" not in i)
-        page += 1
-    return issues
-
-
-# ───────── main ───────────────────────────────────────────────────────────
-last_stamp = read_stamp()
-changed    = fetch_open_issues(since_iso=last_stamp)
-
-# Fallback if **nothing** changed AND we have *no* existing output
-if not changed:
-    cache_exists = CACHE_FILE.exists()
-    wiki_exists  = pathlib.Path("wiki/Issue-Categories.md").exists()
-    if not cache_exists or not wiki_exists:
-        # first run or someone wiped the wiki → build from scratch
-        print("⏩ First run or empty wiki — fetching ALL open issues.", file=sys.stderr)
-        changed = fetch_open_issues()         # full list
-    else:
-        print(f"✅ No issues updated since {last_stamp}. Nothing to classify.", file=sys.stderr)
-        save_stamp()
-        sys.exit(0)
-
-# ---------------------------------------------------------------- prompt
-issue_lines = "\n".join(f"- {i['title']} ({i['html_url']})" for i in changed)
-prompt = textwrap.dedent(f"""\
-You are an AI assistant helping triage GitHub issues into exactly 7 predefined themes.
-
-Each issue must go into exactly one of the themes below:
-
-{THEMES_MD}
-
-Format your output in Markdown like:
-## 🎯 Prompt, Token, and Cost Management
-- [#123](https://github.com/org/repo/issues/123) – Title here
-
-Classify these issues:
-{issue_lines}
-""")
-
-resp = client.chat.completions.create(
-    model="gpt-4.1",
-    messages=[{"role": "user", "content": prompt}],
-    temperature=0.2,
-)
-
-md = resp.choices[0].message.content
-
-# ---------------------------------------------------------------- parse GPT
-new_map: dict[int, str] = {}
-current = None
-for ln in md.splitlines():
-    if ln.startswith("##"):
-        current = ln.lstrip("# ").strip()
-    elif ln.lstrip().startswith("- [#"):
-        try:
-            num = int(ln.split("[#")[1].split("]")[0])
-            new_map[num] = current
-        except Exception:
-            pass  # ignore malformed lines
-
-cache = load_cache()
-cache.update(new_map)
-save_cache(cache)
-save_stamp()
-
-# ---------------------------------------------------------------- rebuild wiki
-order = [
-    "🔗 LLM Integration & Provider Support",
-    "🖥 App Build & Platform Compatibility",
-    "🎯 Prompt, Token, and Cost Management",
-    "🧩 Editor UX & Interaction Design",
-    "🤖 Agent & Automation Features",
-    "⚙️ System Config & Environment Setup",
-    "🗃 Meta: Feature Comparison, Structure, and Naming",
-]
-
-sections: dict[str, list[int]] = {t: [] for t in order}
-
-# ── fetch ALL current open issues once  (PRs filtered out) ────────────────
-title_map: dict[int, tuple[str, str]] = {}
-open_now: set[int] = set()
-
-page = 1
-while True:
-    batch = fetch_open_issues(since_iso=None) if page == 1 else []
-    if not batch:
-        break
-    for it in batch:
-        num = it["number"]
-        title_map[num] = (it["title"], it["html_url"])
-        open_now.add(num)
-    page += 1
-
-# 🧹 drop any cached IDs that are no longer open issues (e.g., became a PR or were closed)
-for stale in set(cache) - open_now:
-    del cache[stale]
-save_cache(cache)            # persist cleaned cache
-
-# build sections from cleaned cache
-for num, theme in cache.items():
-    if theme in sections:          # extra safety
-        sections[theme].append(num)
-
-# ---------------------------------------------------------------- print roadmap
-for theme in order:
-    issues = sections[theme]
-    if issues:
-        print(f"## {theme}")
-        for n in sorted(issues):
-            title, url = title_map.get(n, ("(missing)", f"https://github.com/{REPO}/issues/{n}"))
-            print(f"- [#{n}]({url}) – {title}")
-        print()

.github/workflows/triage.yml

@@ -1,60 +0,0 @@
-name: Issue Triage to Wiki
-
-on:
-  workflow_dispatch:
-  schedule:
-    - cron: '0 */6 * * *'    # every 6 hrs (UTC)
-
-jobs:
-  roadmap:
-    runs-on: ubuntu-latest
-
-    steps:
-      # 1️⃣ Check out code (so the script and cache files are present)
-      - name: Checkout code
-        uses: actions/checkout@v3
-        with:
-          fetch-depth: 1    # shallow clone
-
-      # 2️⃣ Set up Python
-      - name: Set up Python
-        uses: actions/setup-python@v4
-        with:
-          python-version: '3.11'
-
-      # 3️⃣ Install dependencies
-      - name: Install Python dependencies
-        run: |
-          pip install openai requests
-
-      # 4️⃣ Clone your fork’s Wiki
-      - name: Clone your fork's Wiki
-        run: |
-          git clone https://x-access-token:${{ secrets.WIKI_TOKEN }}@github.com/${{ github.repository }}.wiki.git wiki
-
-      # 5️⃣ (Optional) Show repo tree for debugging
-      - name: Show repo tree (debug)
-        run: |
-          echo "PWD: $(pwd)"
-          ls -al
-          ls -al .github/scripts || true
-          ls -al void/.github/scripts || true
-
-      # 6️⃣ Generate roadmap and push only if it changed
-      - name: Generate roadmap directly into wiki
-        run: |
-          python .github/scripts/issue_triage.py > wiki/_new.md
-          if ! cmp -s wiki/_new.md wiki/Issue-Categories.md ; then
-            mv wiki/_new.md wiki/Issue-Categories.md
-            cd wiki
-            git config user.name "github-actions[bot]"
-            git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
-            git add Issue-Categories.md
-            git commit -m "Auto-update Issue-Categories.md from GPT triage"
-            git push
-          else
-            echo "No content change – skipping wiki update"
-          fi
-        env:
-          OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
-          GITHUB_TOKEN:   ${{ secrets.WIKI_TOKEN }}

.voidrules

@@ -8,9 +8,3 @@ Look for services and built-in functions that you might need to use to solve the
 In typescript, do NOT cast to types if not neccessary. NEVER lazily cast to 'any'. Find the correct type to apply and use it.
 
 Do not add or remove semicolons to any of my files. Just go with convention and make the least number of changes.
-
-Never modify files outside src/vs/workbench/contrib/void without consulting with the user first.
-
-All types that map from a value A to B should be called bOfA. For example, if you create a hashmap that goes from toolId to toolName, it should be called toolNameOfToolId, etc.
-
-Do not run anything to validate your changes; tell the user what to do instead.

HOW_TO_CONTRIBUTE.md

@@ -20,15 +20,14 @@ Most of Void's code lives in the folder `src/vs/workbench/contrib/void/`.
 
 
 
-## Editing Void's Code
 
-If you're making changes to Void's code as a contributor, you'll want to run a local version of Void to make sure your changes worked. Developer mode lets you do this. Here's how to use it.
+## Building Void
 
-### a. Mac - Prerequisites
+### a. Mac - Build Prerequisites
 
 If you're using a Mac, you need Python and XCode. You probably have these by default.
 
-### b. Windows - Prerequisites
+### b. Windows - Build Prerequisites
 
 If you're using a Windows computer, first get [Visual Studio 2022](https://visualstudio.microsoft.com/thank-you-downloading-visual-studio/?sku=Community) (recommended) or [VS Build Tools](https://visualstudio.microsoft.com/thank-you-downloading-visual-studio/?sku=BuildTools) (not recommended). If you already have both, you might need to run the next few steps on both of them.
 
@@ -43,52 +42,34 @@ Go to the "Individual Components" tab and select:
 
 Finally, click Install.
 
-### c. Linux - Prerequisites
+### c. Linux - Build Prerequisites
 
 First, run `npm install -g node-gyp`. Then:
 
 - Debian (Ubuntu, etc): `sudo apt-get install build-essential g++ libx11-dev libxkbfile-dev libsecret-1-dev libkrb5-dev python-is-python3`.
 - Red Hat (Fedora, etc): `sudo dnf install @development-tools gcc gcc-c++ make libsecret-devel krb5-devel libX11-devel libxkbfile-devel`.
-- SUSE (openSUSE, etc): `sudo zypper install patterns-devel-C-C++-devel_C_C++  krb5-devel libsecret-devel libxkbfile-devel libX11-devel`.
 - Others: see [How to Contribute](https://github.com/microsoft/vscode/wiki/How-to-Contribute).
 
-### Developer Mode Instructions
-
-Here's how to start changing Void's code. These steps cover everything from cloning Void, to opening a Developer Mode window where you can play around with your updates.
+### d. Building Void from inside VSCode
 
 1. `git clone https://github.com/voideditor/void` to clone the repo.
 2. `npm install` to install all dependencies.
-3. Open Void or VSCode, and initialize Developer Mode (this can take ~5 min to finish, it's done when 2 of the 3 spinners turn to check marks):
+3. To build Void, open VSCode. Then:
    - Windows: Press <kbd>Ctrl+Shift+B</kbd>.
    - Mac: Press <kbd>Cmd+Shift+B</kbd>.
    - Linux: Press <kbd>Ctrl+Shift+B</kbd>.
-4. Open the Void Developer Mode window:
+   - This step can take ~5 min. The build is done when you see two check marks (one of the items will continue spinning indefinitely - it compiles our React code).
+4. To run Void:
    - Windows: `./scripts/code.bat`.
    - Mac: `./scripts/code.sh`.
    - Linux: `./scripts/code.sh`.
-5. You're good to start editing Void's code! 
-   - You won't see your changes unless you press <kbd>Ctrl+R</kbd> (<kbd>Cmd+R</kbd>) inside the new window to reload. Alternatively, press <kbd>Ctrl+Shift+P</kbd> and `Reload Window`.
-   - You might want to add the flags `--user-data-dir ./.tmp/user-data --extensions-dir ./.tmp/extensions` to the command in step 4, which lets you reset any IDE changes you made by deleting the `.tmp` folder.
-	- You can kill any of the build scripts by pressing `Ctrl+D` in its terminal. If you press `Ctrl+C` the script will close but will keep running in the background.
+5. Nice-to-knows.
+   - You can always press <kbd>Ctrl+R</kbd> (<kbd>Cmd+R</kbd>) inside the new window to reload and see your new changes. It's faster than <kbd>Ctrl+Shift+P</kbd> and `Reload Window`.
+   - You might want to add the flags `--user-data-dir ./.tmp/user-data --extensions-dir ./.tmp/extensions` to the above run command, which lets you delete the `.tmp` folder to reset any IDE changes you made when testing.
+	- You can kill any of the build scripts by pressing `Ctrl+D` in VSCode terminal. If you press `Ctrl+C` the script will close but will keep running in the background (to open all background scripts, just re-build).
 
 If you get any errors, scroll down for common fixes.
 
-#### Common Fixes
-
-- Make sure you followed the prerequisite steps above.
-- Make sure you have Node version `20.18.2` (the version in `.nvmrc`).
-    - You can do this without changing your global Node version using [nvm](https://github.com/nvm-sh/nvm): run `nvm install`, followed by `nvm use` to install the version in `.nvmrc` locally.
-- Make sure the path to your Void folder does not have any spaces in it.
-- If you get `"TypeError: Failed to fetch dynamically imported module"`, make sure all imports end with `.js`.
-- If you get an error with React, try running `NODE_OPTIONS="--max-old-space-size=8192" npm run buildreact`.
-- If you see missing styles, wait a few seconds and then reload.
-- If you get errors like `npm error libtool:   error: unrecognised option: '-static'`,  when running ./scripts/code.sh, make sure you have GNU libtool instead of BSD libtool (BSD is the default in macos)
-- If you get errors like `The SUID sandbox helper binary was found, but is not configured correctly` when running ./scripts/code.sh, run
-`sudo chown root:root .build/electron/chrome-sandbox && sudo chmod 4755 .build/electron/chrome-sandbox` and then run `./scripts/code.sh` again.
-- If you have any other questions, feel free to [submit an issue](https://github.com/voideditor/void/issues/new). You can also refer to VSCode's complete [How to Contribute](https://github.com/microsoft/vscode/wiki/How-to-Contribute) page.
-
-
-
 #### Building Void from Terminal
 
 To build Void from the terminal instead of from inside VSCode, follow the steps above, but instead of pressing <kbd>Cmd+Shift+B</kbd>, run `npm run watch`. The build is done when you see something like this:
@@ -101,47 +82,50 @@ To build Void from the terminal instead of from inside VSCode, follow the steps
 ```
 
 
+#### Common Fixes
 
-### Distributing
-Void's maintainers distribute Void on our website and in releases. Our build pipeline is a fork of VSCodium, and it works by running GitHub Actions which create the downloadables. The build repo with more instructions lives [here](https://github.com/voideditor/void-builder).
-
-If you want to completely control Void's build pipeline for your own internal usage, which comes with a lot of time cost (and is typically not recommended), see our [`void-builder`](https://github.com/voideditor/void-builder) repo which builds Void and contains a few important notes about auto-updating and rebasing.
+- Make sure you followed the prerequisite steps above.
+- Make sure you have Node version `20.18.2` (the version in `.nvmrc`)!
+    - You can do this easily without touching your base installation with [nvm](https://github.com/nvm-sh/nvm). Simply run `nvm install`, followed by `nvm use` and it will automatically install and use the version specified in `nvmrc`.
+- Make sure that the path to your Void folder does not have any spaces in it.
+- If you get `"TypeError: Failed to fetch dynamically imported module"`, make sure all imports end with `.js`.
+- If you get an error with React, try running `NODE_OPTIONS="--max-old-space-size=8192" npm run buildreact`.
+- If you see missing styles, wait a few seconds and then reload.
+- If you get errors like `npm error libtool:   error: unrecognised option: '-static'`,  when running ./scripts/code.sh, make sure you have GNU libtool instead of BSD libtool (BSD is the default in macos)
+- If you get erorrs like `The SUID sandbox helper binary was found, but is not configured correctly` when running ./scripts/code.sh, run
+`sudo chown root:root .build/electron/chrome-sandbox && sudo chmod 4755 .build/electron/chrome-sandbox` and then run `./scripts/code.sh` again.
+- If you have any other questions, feel free to [submit an issue](https://github.com/voideditor/void/issues/new). You can also refer to VSCode's complete [How to Contribute](https://github.com/microsoft/vscode/wiki/How-to-Contribute) page.
 
 
-#### Building a Local Executible
-We don't usually recommend building a local executible of Void - typically you should follow the steps above to distribute a complete executible with the advantages of VSCodium baked-in, or you should just use Developer Mode to run Void locally which is much faster. If you're certain this is what you want, see details below.
+## Packaging
 
-<details>
-	<summary> Building Locally (not recommended)</summary>
-If you're certain you want to build a local executible of Void, follow these steps. It can take ~25 minutes.
-
-Make sure you've already entered Developer Mode with Void first, then run one of the following commands. This will create a folder named `VSCode-darwin-arm64` or similar outside of the void/ repo (see below). 
+We don't usually recommend packaging. Instead, you should probably just build. If you're sure you want to package Void into an executable app, make sure you've built first, then run one of the following commands. This will create a folder named `VSCode-darwin-arm64` or similar outside of the void/ repo (see below). Be patient - packaging can take ~25 minutes.
 
 
-##### Mac
+### Mac
 - `npm run gulp vscode-darwin-arm64` - most common (Apple Silicon)
 - `npm run gulp vscode-darwin-x64` (Intel)
 
-##### Windows
+### Windows
 - `npm run gulp vscode-win32-x64` - most common
 - `npm run gulp vscode-win32-arm64`
 
-##### Linux
+### Linux
 - `npm run gulp vscode-linux-x64` - most common
 - `npm run gulp vscode-linux-arm64`
 
 
-##### Local Executible Output
+### Output
 
-The local executible will be located in a folder outside of `void/`:
+This will generate a folder outside of `void/`:
 ```bash
 workspace/
 ├── void/   # Your Void fork
 └── VSCode-darwin-arm64/ # Generated output
 ```
 
-</details>
-
+### Distributing
+Void's maintainers distribute Void on our website and in releases. Our build pipeline is a fork of VSCodium, and it works by running GitHub Actions which create the downloadables. The build repo with more instructions lives [here](https://github.com/voideditor/void-builder).
 
 ## Pull Request Guidelines
 
@@ -153,3 +137,32 @@ workspace/
 
 
 
+
+<!--
+# Relevant files
+
+We keep track of all the files we've changed with Void so it's easy to rebase:
+
+Edit: far too many changes to track... this is old
+
+- README.md
+- CONTRIBUTING.md
+- VOID_USEFUL_LINKS.md
+- product.json
+- package.json
+
+- src/vs/workbench/api/common/{extHost.api.impl.ts | extHostApiCommands.ts}
+- src/vs/workbench/workbench.common.main.ts
+- src/vs/workbench/contrib/void/\*
+- extensions/void/\*
+
+- .github/\*
+- .vscode/settings/\*
+- .eslintrc.json
+- build/hygiene.js
+- build/lib/i18n.resources.json
+- build/npm/dirs.js
+
+- vscode.proposed.editorInsets.d.ts - not modified, but code copied
+
+-->

README.md

@@ -1,30 +1,3 @@
-## Void is now deprecated.
-Void is deprecated and no longer accepting contributions.
-Thank you to everyone who contributed, both with lines of code and support from the community. Void remains open source and is still one of the best references to use when forking VS Code.
-
-
-## Download
-
-To view a list of newer Void forks, see [Void Forks](http://github.com/voideditor/void-forks/).
-
-To download an old version of Void, see [Releases](https://github.com/voideditor/void/releases).
-
-## Forking VS Code
-
-If you're forking VS Code, you might still want to reference Void's logic, and see our [Codebase Guide](https://github.com/voideditor/void/blob/main/VOID_CODEBASE_GUIDE.md) and [How to Contribute](https://github.com/voideditor/void/blob/main/HOW_TO_CONTRIBUTE.md).
-
-- We mount React + Tailwind. This is not possible in plain VS Code, and required extending the build pipeline to compile React and [scope](https://github.com/andrewpareles/scope-tailwind) Tailwind ourselves.
-
-- You can copy our GitHub Actions to package, sign, and auto-update Void. VS Code's build pipeline is private, so this is normally very hard.
-
-- Our AI provider code is built from scratch, allowing us to support autocomplete (FIM) and other custom responses. We expose grammars for common `<thinking>` tags, tool tags, etc. Feel free to reference our architecture for using IPC and satisfying CSP.
-
-- Use our custom services to edit files. EditCodeService lets you show diffs as code streams in, even token by token. VoidModelService lets you edit files in the background and syncs OS files with your text buffers.
-
-- Everything we've done is 100% open source. See [repos](https://github.com/orgs/voideditor/repositories) for a complete picture of all the repos that make up Void.
-
-
-
 # Welcome to Void.
 
 <div align="center">
@@ -36,26 +9,29 @@ If you're forking VS Code, you might still want to reference Void's logic, and s
 	/>
 </div>
 
+Void is the open-source Cursor alternative.
+
 Use AI agents on your codebase, checkpoint and visualize changes, and bring any model or host locally. Void sends messages directly to providers without retaining your data.
 
-This repo contains the full sourcecode for Void's Desktop app. If you're new, welcome!
+This repo contains the full sourcecode for Void. If you're new, welcome!
 
 - 🧭 [Website](https://voideditor.com)
 
-- 🚙 [Roadmap](https://github.com/orgs/voideditor/projects/2)
+- 👋 [Discord](https://discord.gg/RSNjgaugJs)
 
-- 🔨 [Contribute](https://github.com/voideditor/void/blob/main/HOW_TO_CONTRIBUTE.md)
+- 🚙 [Project Board](https://github.com/orgs/voideditor/projects/2)
 
 
+## Contributing
+
+1. To get started working on Void, check out our Project Board! You can also see [HOW_TO_CONTRIBUTE](https://github.com/voideditor/void/blob/main/HOW_TO_CONTRIBUTE.md).
+
+2. Feel free to attend a casual weekly meeting in our Discord channel!
 
 
 ## Reference
 
-Void is a fork of the [vscode](https://github.com/microsoft/vscode) repository. For a guide to our codebase, see [VOID_CODEBASE_GUIDE](https://github.com/voideditor/void/blob/main/VOID_CODEBASE_GUIDE.md).
-
-For a guide on how to develop your own version of Void, see [HOW_TO_CONTRIBUTE](https://github.com/voideditor/void/blob/main/HOW_TO_CONTRIBUTE.md) and [void-builder](https://github.com/voideditor/void-builder).
-
-
+Void is a fork of the [vscode](https://github.com/microsoft/vscode) repository. For a guide to the codebase, see [VOID_CODEBASE_GUIDE](https://github.com/voideditor/void/blob/main/VOID_CODEBASE_GUIDE.md).
 
 ## Support
-You can always reach us in our [Discord server](https://discord.gg/RSNjgaugJs) or contact us via email at hello@voideditor.com.
+You can always reach us in our Discord server or contact us via email: hello@voideditor.com.

VOID_CODEBASE_GUIDE.md

@@ -52,10 +52,7 @@ Here's some terminology you might want to know about when working inside VSCode:
 	- You can run actions as a user by pressing Cmd+Shift+P (opens the command pallete), or you can run them internally by using the commandService to call them by ID. We use actions to register keybinding listeners like Cmd+L, Cmd+K, etc. The nice thing about actions is the user can change the keybindings.
 
 
-
-
-
-### Void's LLM Message Pipeline
+### Internal LLM Message Pipeline
 
 Here's a picture of all the dependencies that are relevent between the time you first send a message through Void's sidebar, and the time a request is sent to your provider.
 Sending LLM messages from the main process avoids CSP issues with local providers and lets us use node_modules more easily.
@@ -137,7 +134,7 @@ If you want to know how our build pipeline works, see our build repo [here](http
 
 For additional references, the Void team put together this list of links to get up and running with VSCode.
 <details>
-
+	
 
 #### Links for Beginners
 

eslint.config.js

@@ -842,9 +842,7 @@ export default tseslint.config(
 						'@xterm/xterm',
 						'yauzl',
 						'yazl',
-						'zlib',
-						// Void added this
-						'@modelcontextprotocol/sdk/**'
+						'zlib'
 					]
 				},
 				{

package-lock.json

@@ -17,7 +17,7 @@
         "@microsoft/1ds-core-js": "^3.2.13",
         "@microsoft/1ds-post-js": "^3.2.13",
         "@mistralai/mistralai": "^1.6.0",
-        "@modelcontextprotocol/sdk": "^1.11.2",
+        "@modelcontextprotocol/sdk": "^1.10.2",
         "@parcel/watcher": "2.5.1",
         "@types/semver": "^7.5.8",
         "@vscode/deviceid": "^0.1.1",
@@ -299,14 +299,13 @@
       "dev": true
     },
     "node_modules/@azure/core-auth": {
-      "version": "1.9.0",
-      "resolved": "https://registry.npmjs.org/@azure/core-auth/-/core-auth-1.9.0.tgz",
-      "integrity": "sha512-FPwHpZywuyasDSLMqJ6fhbOK3TqUdviZNF8OqRGA4W5Ewib2lEEZ+pBsYcBa88B2NGO/SEnYPGhyBqNlE8ilSw==",
+      "version": "1.7.2",
+      "resolved": "https://registry.npmjs.org/@azure/core-auth/-/core-auth-1.7.2.tgz",
+      "integrity": "sha512-Igm/S3fDYmnMq1uKS38Ae1/m37B3zigdlZw+kocwEhh5GjyKjPrXKO2J6rzpC1wAxrNil/jX9BJRqBshyjnF3g==",
       "dev": true,
-      "license": "MIT",
       "dependencies": {
         "@azure/abort-controller": "^2.0.0",
-        "@azure/core-util": "^1.11.0",
+        "@azure/core-util": "^1.1.0",
         "tslib": "^2.6.2"
       },
       "engines": {
@@ -449,18 +448,18 @@
       }
     },
     "node_modules/@azure/core-rest-pipeline": {
-      "version": "1.20.0",
-      "resolved": "https://registry.npmjs.org/@azure/core-rest-pipeline/-/core-rest-pipeline-1.20.0.tgz",
-      "integrity": "sha512-ASoP8uqZBS3H/8N8at/XwFr6vYrRP3syTK0EUjDXQy0Y1/AUS+QeIRThKmTNJO2RggvBBxaXDPM7YoIwDGeA0g==",
+      "version": "1.16.0",
+      "resolved": "https://registry.npmjs.org/@azure/core-rest-pipeline/-/core-rest-pipeline-1.16.0.tgz",
+      "integrity": "sha512-CeuTvsXxCUmEuxH5g/aceuSl6w2EugvNHKAtKKVdiX915EjJJxAwfzNNWZreNnbxHZ2fi0zaM6wwS23x2JVqSQ==",
       "dev": true,
-      "license": "MIT",
       "dependencies": {
         "@azure/abort-controller": "^2.0.0",
-        "@azure/core-auth": "^1.8.0",
+        "@azure/core-auth": "^1.4.0",
         "@azure/core-tracing": "^1.0.1",
-        "@azure/core-util": "^1.11.0",
+        "@azure/core-util": "^1.9.0",
         "@azure/logger": "^1.0.0",
-        "@typespec/ts-http-runtime": "^0.2.2",
+        "http-proxy-agent": "^7.0.0",
+        "https-proxy-agent": "^7.0.0",
         "tslib": "^2.6.2"
       },
       "engines": {
@@ -480,14 +479,12 @@
       }
     },
     "node_modules/@azure/core-util": {
-      "version": "1.12.0",
-      "resolved": "https://registry.npmjs.org/@azure/core-util/-/core-util-1.12.0.tgz",
-      "integrity": "sha512-13IyjTQgABPARvG90+N2dXpC+hwp466XCdQXPCRlbWHgd3SJd5Q1VvaBGv6k1BIa4MQm6hAF1UBU1m8QUxV8sQ==",
+      "version": "1.9.0",
+      "resolved": "https://registry.npmjs.org/@azure/core-util/-/core-util-1.9.0.tgz",
+      "integrity": "sha512-AfalUQ1ZppaKuxPPMsFEUdX6GZPB3d9paR9d/TTL7Ow2De8cJaC7ibi7kWVlFAVPCYo31OcnGymc0R89DX8Oaw==",
       "dev": true,
-      "license": "MIT",
       "dependencies": {
         "@azure/abort-controller": "^2.0.0",
-        "@typespec/ts-http-runtime": "^0.2.2",
         "tslib": "^2.6.2"
       },
       "engines": {
@@ -2616,9 +2613,9 @@
       }
     },
     "node_modules/@modelcontextprotocol/sdk": {
-      "version": "1.11.2",
-      "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.11.2.tgz",
-      "integrity": "sha512-H9vwztj5OAqHg9GockCQC06k1natgcxWQSRpQcPJf6i5+MWBzfKkRtxGbjQf0X2ihii0ffLZCRGbYV2f2bjNCQ==",
+      "version": "1.10.2",
+      "resolved": "https://registry.npmjs.org/@modelcontextprotocol/sdk/-/sdk-1.10.2.tgz",
+      "integrity": "sha512-rb6AMp2DR4SN+kc6L1ta2NCpApyA9WYNx3CrTSZvGxq9wH71bRur+zRqPfg0vQ9mjywR7qZdX2RGHOPq3ss+tA==",
       "license": "MIT",
       "dependencies": {
         "content-type": "^1.0.5",
@@ -4260,21 +4257,6 @@
         "url": "https://opencollective.com/typescript-eslint"
       }
     },
-    "node_modules/@typespec/ts-http-runtime": {
-      "version": "0.2.2",
-      "resolved": "https://registry.npmjs.org/@typespec/ts-http-runtime/-/ts-http-runtime-0.2.2.tgz",
-      "integrity": "sha512-Gz/Sm64+Sq/vklJu1tt9t+4R2lvnud8NbTD/ZfpZtMiUX7YeVpCA8j6NSW8ptwcoLL+NmYANwqP8DV0q/bwl2w==",
-      "dev": true,
-      "license": "MIT",
-      "dependencies": {
-        "http-proxy-agent": "^7.0.0",
-        "https-proxy-agent": "^7.0.0",
-        "tslib": "^2.6.2"
-      },
-      "engines": {
-        "node": ">=18.0.0"
-      }
-    },
     "node_modules/@vscode/deviceid": {
       "version": "0.1.1",
       "resolved": "https://registry.npmjs.org/@vscode/deviceid/-/deviceid-0.1.1.tgz",

package.json

@@ -79,7 +79,7 @@
     "@microsoft/1ds-core-js": "^3.2.13",
     "@microsoft/1ds-post-js": "^3.2.13",
     "@mistralai/mistralai": "^1.6.0",
-    "@modelcontextprotocol/sdk": "^1.11.2",
+    "@modelcontextprotocol/sdk": "^1.10.2",
     "@parcel/watcher": "2.5.1",
     "@types/semver": "^7.5.8",
     "@vscode/deviceid": "^0.1.1",

product.json

@@ -1,8 +1,8 @@
 {
 	"nameShort": "Void",
 	"nameLong": "Void",
-	"voidVersion": "1.4.9",
-	"voidRelease": "0044",
+	"voidVersion": "1.3.7",
+	"voidRelease": "0031",
 	"applicationName": "void",
 	"dataFolderName": ".void-editor",
 	"win32MutexName": "voideditor",

src/vs/code/electron-main/app.ts

@@ -130,9 +130,7 @@ import { IVoidUpdateService } from '../../workbench/contrib/void/common/voidUpda
 import { MetricsMainService } from '../../workbench/contrib/void/electron-main/metricsMainService.js';
 import { VoidMainUpdateService } from '../../workbench/contrib/void/electron-main/voidUpdateMainService.js';
 import { LLMMessageChannel } from '../../workbench/contrib/void/electron-main/sendLLMMessageChannel.js';
-import { VoidSCMService } from '../../workbench/contrib/void/electron-main/voidSCMMainService.js';
-import { IVoidSCMService } from '../../workbench/contrib/void/common/voidSCMTypes.js';
-import { MCPChannel } from '../../workbench/contrib/void/electron-main/mcpChannel.js';
+
 /**
  * The main VS Code application. There will only ever be one instance,
  * even if the user starts many instances (e.g. from the command line).
@@ -1104,7 +1102,6 @@ export class CodeApplication extends Disposable {
 		// Void main process services (required for services with a channel for comm between browser and electron-main (node))
 		services.set(IMetricsService, new SyncDescriptor(MetricsMainService, undefined, false));
 		services.set(IVoidUpdateService, new SyncDescriptor(VoidMainUpdateService, undefined, false));
-		services.set(IVoidSCMService, new SyncDescriptor(VoidSCMService, undefined, false));
 
 		// Default Extensions Profile Init
 		services.set(IExtensionsProfileScannerService, new SyncDescriptor(ExtensionsProfileScannerService, undefined, true));
@@ -1246,14 +1243,6 @@ export class CodeApplication extends Disposable {
 		const sendLLMMessageChannel = new LLMMessageChannel(accessor.get(IMetricsService));
 		mainProcessElectronServer.registerChannel('void-channel-llmMessage', sendLLMMessageChannel);
 
-		// Void added this
-		const voidSCMChannel = ProxyChannel.fromService(accessor.get(IVoidSCMService), disposables);
-		mainProcessElectronServer.registerChannel('void-channel-scm', voidSCMChannel);
-
-		// Void added this
-		const mcpChannel = new MCPChannel();
-		mainProcessElectronServer.registerChannel('void-channel-mcp', mcpChannel);
-
 		// Extension Host Debug Broadcasting
 		const electronExtensionHostDebugBroadcastChannel = new ElectronExtensionHostDebugBroadcastChannel(accessor.get(IWindowsMainService));
 		mainProcessElectronServer.registerChannel('extensionhostdebugservice', electronExtensionHostDebugBroadcastChannel);

src/vs/platform/telemetry/common/telemetryService.ts

@@ -163,35 +163,38 @@ export class TelemetryService implements ITelemetryService {
 }
 
 function getTelemetryLevelSettingDescription(): string {
-	const telemetryText = localize('telemetry.telemetryLevelMd', "The default telemetry setting for VS Code (Microsoft). {0} recommends keeping this off.", product.nameLong);
-	// const externalLinksStatement = !product.privacyStatementUrl ?
-	// 	localize("telemetry.docsStatement", "Read more about the [data we collect]({0}).", 'https://aka.ms/vscode-telemetry') :
-	// 	localize("telemetry.docsAndPrivacyStatement", "Read more about the [data we collect]({0}) and our [privacy statement]({1}).", 'https://aka.ms/vscode-telemetry', product.privacyStatementUrl);
-	const restartString = !isWeb ? localize('telemetry.restart', 'Microsoft says \"Some third party extensions might not respect this setting. Consult the specific extension\'s documentation to be sure. A full restart of the application is necessary for crash reporting changes to take effect.\"') : '';
+	const telemetryText = localize('telemetry.telemetryLevelMd', "Controls {0} telemetry, first-party extension telemetry, and participating third-party extension telemetry. Some third party extensions might not respect this setting. Consult the specific extension's documentation to be sure. Telemetry helps us better understand how {0} is performing, where improvements need to be made, and how features are being used.", product.nameLong);
+	const externalLinksStatement = !product.privacyStatementUrl ?
+		localize("telemetry.docsStatement", "Read more about the [data we collect]({0}).", 'https://aka.ms/vscode-telemetry') :
+		localize("telemetry.docsAndPrivacyStatement", "Read more about the [data we collect]({0}) and our [privacy statement]({1}).", 'https://aka.ms/vscode-telemetry', product.privacyStatementUrl);
+	const restartString = !isWeb ? localize('telemetry.restart', 'A full restart of the application is necessary for crash reporting changes to take effect.') : '';
 
+	const crashReportsHeader = localize('telemetry.crashReports', "Crash Reports");
+	const errorsHeader = localize('telemetry.errors', "Error Telemetry");
+	const usageHeader = localize('telemetry.usage', "Usage Data");
 
-	// Void removed these
-	// const crashReportsHeader = localize('telemetry.crashReports', "Crash Reports");
-	// const errorsHeader = localize('telemetry.errors', "Error Telemetry");
-	// const usageHeader = localize('telemetry.usage', "Usage Data");
+	const telemetryTableDescription = localize('telemetry.telemetryLevel.tableDescription', "The following table outlines the data sent with each setting:");
+	const telemetryTable = `
+|       | ${crashReportsHeader} | ${errorsHeader} | ${usageHeader} |
+|:------|:-------------:|:---------------:|:----------:|
+| all   |       ✓       |        ✓        |     ✓      |
+| error |       ✓       |        ✓        |     -      |
+| crash |       ✓       |        -        |     -      |
+| off   |       -       |        -        |     -      |
+`;
 
-	// const telemetryTableDescription = localize('telemetry.telemetryLevel.tableDescription', "The following table outlines the data sent with each setting:");
-	// 	const telemetryTable = `
-	// |       | ${crashReportsHeader} | ${errorsHeader} | ${usageHeader} |
-	// |:------|:-------------:|:---------------:|:----------:|
-	// | all   |       ✓       |        ✓        |     ✓      |
-	// | error |       ✓       |        ✓        |     -      |
-	// | crash |       ✓       |        -        |     -      |
-	// | off   |       -       |        -        |     -      |
-	// `;
-
-	// const deprecatedSettingNote = localize('telemetry.telemetryLevel.deprecated', "****Note:*** If this setting is 'off', no telemetry will be sent regardless of other telemetry settings. If this setting is set to anything except 'off' and telemetry is disabled with deprecated settings, no telemetry will be sent.*");
+	const deprecatedSettingNote = localize('telemetry.telemetryLevel.deprecated', "****Note:*** If this setting is 'off', no telemetry will be sent regardless of other telemetry settings. If this setting is set to anything except 'off' and telemetry is disabled with deprecated settings, no telemetry will be sent.*");
 	const telemetryDescription = `
-${telemetryText}
+${telemetryText} ${externalLinksStatement} ${restartString}
 
-${restartString}
+ 
 
-Void separately records basic usage like the number of messages people are sending. If you'd like to disable Void metrics, you may do so in Void's Settings.
+${telemetryTableDescription}
+${telemetryTable}
+
+ 
+
+${deprecatedSettingNote}
 `;
 
 	return telemetryDescription;

src/vs/workbench/browser/parts/editor/editorGroupWatermark.ts

@@ -26,6 +26,7 @@ import { IViewsService } from '../../../services/views/common/viewsService.js';
 
 /* eslint-disable */ // Void
 import { VOID_CTRL_K_ACTION_ID, VOID_CTRL_L_ACTION_ID } from '../../../contrib/void/browser/actionIDs.js';
+import { VOID_OPEN_SETTINGS_ACTION_ID } from '../../../contrib/void/browser/voidSettingsPane.js';
 import { VIEWLET_ID as REMOTE_EXPLORER_VIEWLET_ID } from '../../../contrib/remote/browser/remoteExplorer.js';
 /* eslint-enable */
 
@@ -305,21 +306,21 @@ export class EditorGroupWatermark extends Disposable {
 					label2.set(keys2);
 				this.currentDisposables.add(label2);
 
-				// const keys3 = this.keybindingService.lookupKeybinding('workbench.action.openGlobalKeybindings');
-				// const button3 = append(recentsBox, $('button'));
-				// button3.textContent = `Void Settings`
-				// button3.style.display = 'block'
-				// button3.style.marginLeft = 'auto'
-				// button3.style.marginRight = 'auto'
-				// button3.classList.add('void-settings-watermark-button')
+				const keys3 = this.keybindingService.lookupKeybinding('workbench.action.openGlobalKeybindings');
+				const button3 = append(recentsBox, $('button'));
+				button3.textContent = `Void Settings`
+				button3.style.display = 'block'
+				button3.style.marginLeft = 'auto'
+				button3.style.marginRight = 'auto'
+				button3.classList.add('void-settings-watermark-button')
 
-				// const label3 = new KeybindingLabel(button3, OS, { renderUnboundKeybindings: true, ...defaultKeybindingLabelStyles });
-				// if (keys3)
-				// 	label3.set(keys3);
-				// button3.onclick = () => {
-				// 	this.commandService.executeCommand(VOID_OPEN_SETTINGS_ACTION_ID)
-				// }
-				// this.currentDisposables.add(label3);
+				const label3 = new KeybindingLabel(button3, OS, { renderUnboundKeybindings: true, ...defaultKeybindingLabelStyles });
+				if (keys3)
+					label3.set(keys3);
+				button3.onclick = () => {
+					this.commandService.executeCommand(VOID_OPEN_SETTINGS_ACTION_ID)
+				}
+				this.currentDisposables.add(label3);
 
 			}
 

src/vs/workbench/contrib/void/browser/chatThreadService.ts

@@ -11,12 +11,12 @@ import { IStorageService, StorageScope, StorageTarget } from '../../../../platfo
 import { URI } from '../../../../base/common/uri.js';
 import { Emitter, Event } from '../../../../base/common/event.js';
 import { ILLMMessageService } from '../common/sendLLMMessageService.js';
-import { chat_userMessageContent, isABuiltinToolName } from '../common/prompt/prompts.js';
+import { chat_userMessageContent, ToolName, } from '../common/prompt/prompts.js';
 import { AnthropicReasoning, getErrorMessage, RawToolCallObj, RawToolParamsObj } from '../common/sendLLMMessageTypes.js';
 import { generateUuid } from '../../../../base/common/uuid.js';
 import { FeatureName, ModelSelection, ModelSelectionOptions } from '../common/voidSettingsTypes.js';
 import { IVoidSettingsService } from '../common/voidSettingsService.js';
-import { approvalTypeOfBuiltinToolName, BuiltinToolCallParams, ToolCallParams, ToolName, ToolResult } from '../common/toolsServiceTypes.js';
+import { approvalTypeOfToolName, ToolCallParams, ToolResultType } from '../common/toolsServiceTypes.js';
 import { IToolsService } from './toolsService.js';
 import { CancellationToken } from '../../../../base/common/cancellation.js';
 import { ILanguageFeaturesService } from '../../../../editor/common/services/languageFeatures.js';
@@ -37,8 +37,6 @@ import { deepClone } from '../../../../base/common/objects.js';
 import { IWorkspaceContextService } from '../../../../platform/workspace/common/workspace.js';
 import { IDirectoryStrService } from '../common/directoryStrService.js';
 import { IFileService } from '../../../../platform/files/common/files.js';
-import { IMCPService } from '../common/mcpService.js';
-import { RawMCPToolCall } from '../common/mcpServiceTypes.js';
 
 
 // related to retrying when LLM message has error
@@ -183,11 +181,10 @@ export type ThreadStreamState = {
 		llmInfo?: undefined;
 		toolInfo: {
 			toolName: ToolName;
-			toolParams: ToolCallParams<ToolName>;
+			toolParams: ToolCallParams[ToolName];
 			id: string;
 			content: string;
 			rawParams: RawToolParamsObj;
-			mcpServerName: string | undefined;
 		};
 		interrupt: Promise<() => void>;
 	} | {
@@ -326,7 +323,6 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 		@IWorkspaceContextService private readonly _workspaceContextService: IWorkspaceContextService,
 		@IDirectoryStrService private readonly _directoryStringService: IDirectoryStrService,
 		@IFileService private readonly _fileService: IFileService,
-		@IMCPService private readonly _mcpService: IMCPService,
 	) {
 		super()
 		this.state = { allThreads: {}, currentThreadId: null as unknown as string } // default state
@@ -449,8 +445,7 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 
 			// if running now but stream state doesn't indicate it (happens if restart Void), cancel that last tool
 			if (lastMessage && lastMessage.role === 'tool' && lastMessage.type === 'running_now') {
-
-				this._updateLatestTool(threadId, { role: 'tool', type: 'rejected', content: lastMessage.content, id: lastMessage.id, rawParams: lastMessage.rawParams, result: null, name: lastMessage.name, params: lastMessage.params, mcpServerName: lastMessage.mcpServerName })
+				this._updateLatestTool(threadId, { role: 'tool', type: 'rejected', content: lastMessage.content, id: lastMessage.id, rawParams: lastMessage.rawParams, result: null, name: lastMessage.name, params: lastMessage.params })
 			}
 
 		}
@@ -537,23 +532,19 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 
 		const lastMsg = thread.messages[thread.messages.length - 1]
 
-		let params: ToolCallParams<ToolName>
+		let params: ToolCallParams[ToolName]
 		if (lastMsg.role === 'tool' && lastMsg.type !== 'invalid_params') {
 			params = lastMsg.params
 		}
 		else return
 
-		const { name, id, rawParams, mcpServerName } = lastMsg
+		const { name, id, rawParams } = lastMsg
 
 		const errorMessage = this.toolErrMsgs.rejected
-		this._updateLatestTool(threadId, { role: 'tool', type: 'rejected', params: params, name: name, content: errorMessage, result: null, id, rawParams, mcpServerName })
+		this._updateLatestTool(threadId, { role: 'tool', type: 'rejected', params: params, name: name, content: errorMessage, result: null, id, rawParams })
 		this._setStreamState(threadId, undefined)
 	}
 
-	private _computeMCPServerOfToolName = (toolName: string) => {
-		return this._mcpService.getMCPTools()?.find(t => t.name === toolName)?.mcpServerName
-	}
-
 	async abortRunning(threadId: string) {
 		const thread = this.state.allThreads[threadId]
 		if (!thread) return // should never happen
@@ -562,13 +553,13 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 		if (this.streamState[threadId]?.isRunning === 'LLM') {
 			const { displayContentSoFar, reasoningSoFar, toolCallSoFar } = this.streamState[threadId].llmInfo
 			this._addMessageToThread(threadId, { role: 'assistant', displayContent: displayContentSoFar, reasoning: reasoningSoFar, anthropicReasoning: null })
-			if (toolCallSoFar) this._addMessageToThread(threadId, { role: 'interrupted_streaming_tool', name: toolCallSoFar.name, mcpServerName: this._computeMCPServerOfToolName(toolCallSoFar.name) })
+			if (toolCallSoFar) this._addMessageToThread(threadId, { role: 'interrupted_streaming_tool', name: toolCallSoFar.name })
 		}
 		// add tool that's running
 		else if (this.streamState[threadId]?.isRunning === 'tool') {
-			const { toolName, toolParams, id, content: content_, rawParams, mcpServerName } = this.streamState[threadId].toolInfo
+			const { toolName, toolParams, id, content: content_, rawParams } = this.streamState[threadId].toolInfo
 			const content = content_ || this.toolErrMsgs.interrupted
-			this._updateLatestTool(threadId, { role: 'tool', name: toolName, params: toolParams, id, content, rawParams, type: 'rejected', result: null, mcpServerName })
+			this._updateLatestTool(threadId, { role: 'tool', name: toolName, params: toolParams, id, content, rawParams, type: 'rejected', result: null })
 		}
 		// reject the tool for the user if relevant
 		else if (this.streamState[threadId]?.isRunning === 'awaiting_user') {
@@ -606,46 +597,36 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 		threadId: string,
 		toolName: ToolName,
 		toolId: string,
-		mcpServerName: string | undefined,
-		opts: { preapproved: true, unvalidatedToolParams: RawToolParamsObj, validatedParams: ToolCallParams<ToolName> } | { preapproved: false, unvalidatedToolParams: RawToolParamsObj },
+		opts: { preapproved: true, unvalidatedToolParams: RawToolParamsObj, validatedParams: ToolCallParams[ToolName] } | { preapproved: false, unvalidatedToolParams: RawToolParamsObj },
 	): Promise<{ awaitingUserApproval?: boolean, interrupted?: boolean }> => {
 
 		// compute these below
-		let toolParams: ToolCallParams<ToolName>
-		let toolResult: ToolResult<ToolName>
+		let toolParams: ToolCallParams[ToolName]
+		let toolResult: Awaited<ToolResultType[typeof toolName]>
 		let toolResultStr: string
 
-		// Check if it's a built-in tool
-		const isBuiltInTool = isABuiltinToolName(toolName)
-
-
 		if (!opts.preapproved) { // skip this if pre-approved
 			// 1. validate tool params
 			try {
-				if (isBuiltInTool) {
-					const params = this._toolsService.validateParams[toolName](opts.unvalidatedToolParams)
-					toolParams = params
-				}
-				else {
-					toolParams = opts.unvalidatedToolParams
-				}
-			}
-			catch (error) {
+				const params = this._toolsService.validateParams[toolName](opts.unvalidatedToolParams)
+				toolParams = params
+			} catch (error) {
 				const errorMessage = getErrorMessage(error)
-				this._addMessageToThread(threadId, { role: 'tool', type: 'invalid_params', rawParams: opts.unvalidatedToolParams, result: null, name: toolName, content: errorMessage, id: toolId, mcpServerName })
+				this._addMessageToThread(threadId, { role: 'tool', type: 'invalid_params', rawParams: opts.unvalidatedToolParams, result: null, name: toolName, content: errorMessage, id: toolId, })
 				return {}
 			}
 			// once validated, add checkpoint for edit
-			if (toolName === 'edit_file') { this._addToolEditCheckpoint({ threadId, uri: (toolParams as BuiltinToolCallParams['edit_file']).uri }) }
-			if (toolName === 'rewrite_file') { this._addToolEditCheckpoint({ threadId, uri: (toolParams as BuiltinToolCallParams['rewrite_file']).uri }) }
+			if (toolName === 'edit_file') { this._addToolEditCheckpoint({ threadId, uri: (toolParams as ToolCallParams['edit_file']).uri }) }
+			if (toolName === 'rewrite_file') { this._addToolEditCheckpoint({ threadId, uri: (toolParams as ToolCallParams['rewrite_file']).uri }) }
 
 			// 2. if tool requires approval, break from the loop, awaiting approval
 
-			const approvalType = isBuiltInTool ? approvalTypeOfBuiltinToolName[toolName] : 'MCP tools'
+
+			const approvalType = approvalTypeOfToolName[toolName]
 			if (approvalType) {
 				const autoApprove = this._settingsService.state.globalSettings.autoApprove[approvalType]
 				// add a tool_request because we use it for UI if a tool is loading (this should be improved in the future)
-				this._addMessageToThread(threadId, { role: 'tool', type: 'tool_request', content: '(Awaiting user permission...)', result: null, name: toolName, params: toolParams, id: toolId, rawParams: opts.unvalidatedToolParams, mcpServerName })
+				this._addMessageToThread(threadId, { role: 'tool', type: 'tool_request', content: '(Awaiting user permission...)', result: null, name: toolName, params: toolParams, id: toolId, rawParams: opts.unvalidatedToolParams })
 				if (!autoApprove) {
 					return { awaitingUserApproval: true }
 				}
@@ -657,12 +638,9 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 
 
 
-
-
-
 		// 3. call the tool
 		// this._setStreamState(threadId, { isRunning: 'tool' }, 'merge')
-		const runningTool = { role: 'tool', type: 'running_now', name: toolName, params: toolParams, content: '(value not received yet...)', result: null, id: toolId, rawParams: opts.unvalidatedToolParams, mcpServerName } as const
+		const runningTool = { role: 'tool', type: 'running_now', name: toolName, params: toolParams, content: '(value not received yet...)', result: null, id: toolId, rawParams: opts.unvalidatedToolParams } as const
 		this._updateLatestTool(threadId, runningTool)
 
 
@@ -672,28 +650,13 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 		try {
 
 			// set stream state
-			this._setStreamState(threadId, { isRunning: 'tool', interrupt: interruptorPromise, toolInfo: { toolName, toolParams, id: toolId, content: 'interrupted...', rawParams: opts.unvalidatedToolParams, mcpServerName } })
+			this._setStreamState(threadId, { isRunning: 'tool', interrupt: interruptorPromise, toolInfo: { toolName, toolParams, id: toolId, content: 'interrupted...', rawParams: opts.unvalidatedToolParams } })
 
-			if (isBuiltInTool) {
-				const { result, interruptTool } = await this._toolsService.callTool[toolName](toolParams as any)
-				const interruptor = () => { interrupted = true; interruptTool?.() }
-				resolveInterruptor(interruptor)
+			const { result, interruptTool } = await this._toolsService.callTool[toolName](toolParams as any)
+			const interruptor = () => { interrupted = true; interruptTool?.() }
+			resolveInterruptor(interruptor)
 
-				toolResult = await result
-			}
-			else {
-				const mcpTools = this._mcpService.getMCPTools()
-				const mcpTool = mcpTools?.find(t => t.name === toolName)
-				if (!mcpTool) { throw new Error(`MCP tool ${toolName} not found`) }
-
-				resolveInterruptor(() => { })
-
-				toolResult = (await this._mcpService.callMCPTool({
-					serverName: mcpTool.mcpServerName ?? 'unknown_mcp_server',
-					toolName: toolName,
-					params: toolParams
-				})).result
-			}
+			toolResult = await result
 
 			if (interrupted) { return { interrupted: true } } // the tool result is added where we interrupt, not here
 		}
@@ -702,27 +665,21 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 			if (interrupted) { return { interrupted: true } } // the tool result is added where we interrupt, not here
 
 			const errorMessage = getErrorMessage(error)
-			this._updateLatestTool(threadId, { role: 'tool', type: 'tool_error', params: toolParams, result: errorMessage, name: toolName, content: errorMessage, id: toolId, rawParams: opts.unvalidatedToolParams, mcpServerName })
+			this._updateLatestTool(threadId, { role: 'tool', type: 'tool_error', params: toolParams, result: errorMessage, name: toolName, content: errorMessage, id: toolId, rawParams: opts.unvalidatedToolParams })
 			return {}
 		}
 
 		// 4. stringify the result to give to the LLM
 		try {
-			if (isBuiltInTool) {
-				toolResultStr = this._toolsService.stringOfResult[toolName](toolParams as any, toolResult as any)
-			}
-			// For MCP tools, handle the result based on its type
-			else {
-				toolResultStr = this._mcpService.stringifyResult(toolResult as RawMCPToolCall)
-			}
+			toolResultStr = this._toolsService.stringOfResult[toolName](toolParams as any, toolResult as any)
 		} catch (error) {
 			const errorMessage = this.toolErrMsgs.errWhenStringifying(error)
-			this._updateLatestTool(threadId, { role: 'tool', type: 'tool_error', params: toolParams, result: errorMessage, name: toolName, content: errorMessage, id: toolId, rawParams: opts.unvalidatedToolParams, mcpServerName })
+			this._updateLatestTool(threadId, { role: 'tool', type: 'tool_error', params: toolParams, result: errorMessage, name: toolName, content: errorMessage, id: toolId, rawParams: opts.unvalidatedToolParams })
 			return {}
 		}
 
 		// 5. add to history and keep going
-		this._updateLatestTool(threadId, { role: 'tool', type: 'success', params: toolParams, result: toolResult, name: toolName, content: toolResultStr, id: toolId, rawParams: opts.unvalidatedToolParams, mcpServerName })
+		this._updateLatestTool(threadId, { role: 'tool', type: 'success', params: toolParams, result: toolResult, name: toolName, content: toolResultStr, id: toolId, rawParams: opts.unvalidatedToolParams })
 		return {}
 	};
 
@@ -757,7 +714,7 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 
 		// before enter loop, call tool
 		if (callThisToolFirst) {
-			const { interrupted } = await this._runToolCall(threadId, callThisToolFirst.name, callThisToolFirst.id, callThisToolFirst.mcpServerName, { preapproved: true, unvalidatedToolParams: callThisToolFirst.rawParams, validatedParams: callThisToolFirst.params })
+			const { interrupted } = await this._runToolCall(threadId, callThisToolFirst.name, callThisToolFirst.id, { preapproved: true, unvalidatedToolParams: callThisToolFirst.rawParams, validatedParams: callThisToolFirst.params })
 			if (interrupted) {
 				this._setStreamState(threadId, undefined)
 				this._addUserCheckpoint({ threadId })
@@ -866,7 +823,7 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 						const { error } = llmRes
 						const { displayContentSoFar, reasoningSoFar, toolCallSoFar } = this.streamState[threadId].llmInfo
 						this._addMessageToThread(threadId, { role: 'assistant', displayContent: displayContentSoFar, reasoning: reasoningSoFar, anthropicReasoning: null })
-						if (toolCallSoFar) this._addMessageToThread(threadId, { role: 'interrupted_streaming_tool', name: toolCallSoFar.name, mcpServerName: this._computeMCPServerOfToolName(toolCallSoFar.name) })
+						if (toolCallSoFar) this._addMessageToThread(threadId, { role: 'interrupted_streaming_tool', name: toolCallSoFar.name })
 
 						this._setStreamState(threadId, { isRunning: undefined, error })
 						this._addUserCheckpoint({ threadId })
@@ -883,10 +840,7 @@ class ChatThreadService extends Disposable implements IChatThreadService {
 
 				// call tool if there is one
 				if (toolCall) {
-					const mcpTools = this._mcpService.getMCPTools()
-					const mcpTool = mcpTools?.find(t => t.name === toolCall.name)
-
-					const { awaitingUserApproval, interrupted } = await this._runToolCall(threadId, toolCall.name, toolCall.id, mcpTool?.mcpServerName, { preapproved: false, unvalidatedToolParams: toolCall.rawParams })
+					const { awaitingUserApproval, interrupted } = await this._runToolCall(threadId, toolCall.name, toolCall.id, { preapproved: false, unvalidatedToolParams: toolCall.rawParams })
 					if (interrupted) {
 						this._setStreamState(threadId, undefined)
 						return
@@ -1346,7 +1300,7 @@ We only need to do it for files that were edited since `from`, ie files between
 			}
 			// URIs of files that have been read
 			else if (m.role === 'tool' && m.type === 'success' && m.name === 'read_file') {
-				const params = m.params as BuiltinToolCallParams['read_file']
+				const params = m.params as ToolCallParams['read_file']
 				addURI(params.uri)
 			}
 		}

src/vs/workbench/contrib/void/browser/convertToLLMMessageService.ts

@@ -7,7 +7,7 @@ import { IWorkspaceContextService } from '../../../../platform/workspace/common/
 import { IEditorService } from '../../../services/editor/common/editorService.js';
 import { ChatMessage } from '../common/chatThreadServiceTypes.js';
 import { getIsReasoningEnabledState, getReservedOutputTokenSpace, getModelCapabilities } from '../common/modelCapabilities.js';
-import { reParsedToolXMLString, chat_systemMessage } from '../common/prompt/prompts.js';
+import { reParsedToolXMLString, chat_systemMessage, ToolName } from '../common/prompt/prompts.js';
 import { AnthropicLLMChatMessage, AnthropicReasoning, GeminiLLMChatMessage, LLMChatMessage, LLMFIMMessage, OpenAILLMChatMessage, RawToolParamsObj } from '../common/sendLLMMessageTypes.js';
 import { IVoidSettingsService } from '../common/voidSettingsService.js';
 import { ChatMode, FeatureName, ModelSelection, ProviderName } from '../common/voidSettingsTypes.js';
@@ -16,8 +16,6 @@ import { ITerminalToolService } from './terminalToolService.js';
 import { IVoidModelService } from '../common/voidModelService.js';
 import { URI } from '../../../../base/common/uri.js';
 import { EndOfLinePreference } from '../../../../editor/common/model.js';
-import { ToolName } from '../common/toolsServiceTypes.js';
-import { IMCPService } from '../common/mcpService.js';
 
 export const EMPTY_MESSAGE = '(empty message)'
 
@@ -457,8 +455,8 @@ const prepareGeminiMessages = (messages: AnthropicLLMChatMessage[]) => {
 						return { text: c.text }
 					}
 					else if (c.type === 'tool_use') {
-						latestToolName = c.name
-						return { functionCall: { id: c.id, name: c.name, args: c.input } }
+						latestToolName = c.name as ToolName
+						return { functionCall: { id: c.id, name: c.name as ToolName, args: c.input } }
 					}
 					else return null
 				}).filter(m => !!m)
@@ -540,7 +538,6 @@ class ConvertToLLMMessageService extends Disposable implements IConvertToLLMMess
 		@ITerminalToolService private readonly terminalToolService: ITerminalToolService,
 		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
 		@IVoidModelService private readonly voidModelService: IVoidModelService,
-		@IMCPService private readonly mcpService: IMCPService,
 	) {
 		super()
 	}
@@ -590,10 +587,8 @@ class ConvertToLLMMessageService extends Disposable implements IConvertToLLMMess
 
 		const includeXMLToolDefinitions = !specialToolFormat
 
-		const mcpTools = this.mcpService.getMCPTools()
-
 		const persistentTerminalIDs = this.terminalToolService.listPersistentTerminalIds()
-		const systemMessage = chat_systemMessage({ workspaceFolders, openedURIs, directoryStr, activeURI, persistentTerminalIDs, chatMode, mcpTools, includeXMLToolDefinitions })
+		const systemMessage = chat_systemMessage({ workspaceFolders, openedURIs, directoryStr, activeURI, persistentTerminalIDs, chatMode, includeXMLToolDefinitions })
 		return systemMessage
 	}
 
@@ -678,15 +673,13 @@ class ConvertToLLMMessageService extends Disposable implements IConvertToLLMMess
 			contextWindow,
 			supportsSystemMessage,
 		} = getModelCapabilities(providerName, modelName, overridesOfModel)
-
-		const { disableSystemMessage } = this.voidSettingsService.state.globalSettings;
-		const fullSystemMessage = await this._generateChatMessagesSystemMessage(chatMode, specialToolFormat)
-		const systemMessage = disableSystemMessage ? '' : fullSystemMessage;
+		const systemMessage = await this._generateChatMessagesSystemMessage(chatMode, specialToolFormat)
 
 		const modelSelectionOptions = this.voidSettingsService.state.optionsOfModelSelection['Chat'][modelSelection.providerName]?.[modelSelection.modelName]
 
 		// Get combined AI instructions
 		const aiInstructions = this._getCombinedAIInstructions();
+
 		const isReasoningEnabled = getIsReasoningEnabledState('Chat', providerName, modelName, modelSelectionOptions, overridesOfModel)
 		const reservedOutputTokenSpace = getReservedOutputTokenSpace(providerName, modelName, { isReasoningEnabled, overridesOfModel })
 		const llmMessages = this._chatMessagesToSimpleMessages(chatMessages)

src/vs/workbench/contrib/void/browser/editCodeService.ts

@@ -1179,11 +1179,6 @@ class EditCodeService extends Disposable implements IEditCodeService {
 			this._onDidChangeStreamingInDiffZone.fire({ uri, diffareaid: diffZone.diffareaid })
 			this._refreshStylesAndDiffsInURI(uri)
 			onFinishEdit()
-
-			// auto accept
-			if (this._settingsService.state.globalSettings.autoAcceptLLMChanges) {
-				this.acceptOrRejectAllDiffAreas({ uri, removeCtrlKs: false, behavior: 'accept' })
-			}
 		}
 
 
@@ -1223,11 +1218,6 @@ class EditCodeService extends Disposable implements IEditCodeService {
 			this._onDidChangeStreamingInDiffZone.fire({ uri, diffareaid: diffZone.diffareaid })
 			this._refreshStylesAndDiffsInURI(uri)
 			onFinishEdit()
-
-			// auto accept
-			if (this._settingsService.state.globalSettings.autoAcceptLLMChanges) {
-				this.acceptOrRejectAllDiffAreas({ uri, removeCtrlKs: false, behavior: 'accept' })
-			}
 		}
 
 		this._writeURIText(uri, newContent, 'wholeFileRange', { shouldRealignDiffAreas: true })
@@ -1458,11 +1448,6 @@ class EditCodeService extends Disposable implements IEditCodeService {
 			}
 			this._refreshStylesAndDiffsInURI(uri)
 			onFinishEdit()
-
-			// auto accept
-			if (this._settingsService.state.globalSettings.autoAcceptLLMChanges) {
-				this.acceptOrRejectAllDiffAreas({ uri, removeCtrlKs: false, behavior: 'accept' })
-			}
 		}
 
 		// throws
@@ -1747,11 +1732,6 @@ class EditCodeService extends Disposable implements IEditCodeService {
 				this._deleteTrackingZone(trackingZone)
 
 			onFinishEdit()
-
-			// auto accept
-			if (this._settingsService.state.globalSettings.autoAcceptLLMChanges) {
-				this.acceptOrRejectAllDiffAreas({ uri, removeCtrlKs: false, behavior: 'accept' })
-			}
 		}
 
 		const onError = (e: { message: string; fullError: Error | null; }) => {

src/vs/workbench/contrib/void/browser/quickEditActions.ts

@@ -13,7 +13,7 @@ import { roundRangeToLines } from './sidebarActions.js';
 import { VOID_CTRL_K_ACTION_ID } from './actionIDs.js';
 import { localize2 } from '../../../../nls.js';
 import { IMetricsService } from '../common/metricsService.js';
-import { ContextKeyExpr } from '../../../../platform/contextkey/common/contextkey.js';
+
 
 export type QuickEditPropsType = {
 	diffareaid: number,
@@ -42,7 +42,6 @@ registerAction2(class extends Action2 {
 			keybinding: {
 				primary: KeyMod.CtrlCmd | KeyCode.KeyK,
 				weight: KeybindingWeight.VoidExtension,
-				when: ContextKeyExpr.deserialize('editorFocus && !terminalFocus'),
 			}
 		});
 	}

src/vs/workbench/contrib/void/browser/react/src/sidebar-tsx/SidebarChat.tsx

@@ -13,7 +13,7 @@ import { ChatMarkdownRender, ChatMessageLocation, getApplyBoxId } from '../markd
 import { URI } from '../../../../../../../base/common/uri.js';
 import { IDisposable } from '../../../../../../../base/common/lifecycle.js';
 import { ErrorDisplay } from './ErrorDisplay.js';
-import { BlockCode, TextAreaFns, VoidCustomDropdownBox, VoidInputBox2, VoidSlider, VoidSwitch, VoidDiffEditor } from '../util/inputs.js';
+import { BlockCode, TextAreaFns, VoidCustomDropdownBox, VoidInputBox2, VoidSlider, VoidSwitch } from '../util/inputs.js';
 import { ModelDropdown, } from '../void-settings-tsx/ModelDropdown.js';
 import { PastThreadsList } from './SidebarThreadSelector.js';
 import { VOID_CTRL_L_ACTION_ID } from '../../../actionIDs.js';
@@ -24,18 +24,16 @@ import { WarningBox } from '../void-settings-tsx/WarningBox.js';
 import { getModelCapabilities, getIsReasoningEnabledState } from '../../../../common/modelCapabilities.js';
 import { AlertTriangle, File, Ban, Check, ChevronRight, Dot, FileIcon, Pencil, Undo, Undo2, X, Flag, Copy as CopyIcon, Info, CirclePlus, Ellipsis, CircleEllipsis, Folder, ALargeSmall, TypeOutline, Text } from 'lucide-react';
 import { ChatMessage, CheckpointEntry, StagingSelectionItem, ToolMessage } from '../../../../common/chatThreadServiceTypes.js';
-import { approvalTypeOfBuiltinToolName, BuiltinToolCallParams, BuiltinToolName, ToolName, LintErrorItem, ToolApprovalType, toolApprovalTypes } from '../../../../common/toolsServiceTypes.js';
+import { approvalTypeOfToolName, LintErrorItem, ToolApprovalType, toolApprovalTypes, ToolCallParams } from '../../../../common/toolsServiceTypes.js';
 import { CopyButton, EditToolAcceptRejectButtonsHTML, IconShell1, JumpToFileButton, JumpToTerminalButton, StatusIndicator, StatusIndicatorForApplyButton, useApplyStreamState, useEditToolStreamState } from '../markdown/ApplyBlockHoverButtons.js';
 import { IsRunningType } from '../../../chatThreadService.js';
 import { acceptAllBg, acceptBorder, buttonFontSize, buttonTextColor, rejectAllBg, rejectBg, rejectBorder } from '../../../../common/helpers/colors.js';
-import { builtinToolNames, isABuiltinToolName, MAX_FILE_CHARS_PAGE, MAX_TERMINAL_INACTIVE_TIME } from '../../../../common/prompt/prompts.js';
+import { MAX_FILE_CHARS_PAGE, MAX_TERMINAL_INACTIVE_TIME, ToolName, toolNames } from '../../../../common/prompt/prompts.js';
 import { RawToolCallObj } from '../../../../common/sendLLMMessageTypes.js';
 import ErrorBoundary from './ErrorBoundary.js';
 import { ToolApprovalTypeSwitch } from '../void-settings-tsx/Settings.js';
 
 import { persistentTerminalNameOfId } from '../../../terminalToolService.js';
-import { removeMCPToolNamePrefix } from '../../../../common/mcpServiceTypes.js';
-
 
 
 export const IconX = ({ size, className = '', ...props }: { size: number, className?: string } & React.SVGProps<SVGSVGElement>) => {
@@ -664,89 +662,81 @@ export const SelectedFiles = (
 					key={thisKey}
 					className={`flex flex-col space-y-[1px]`}
 				>
-					{/* tooltip for file path */}
-					<span className="truncate overflow-hidden text-ellipsis"
-						data-tooltip-id='void-tooltip'
-						data-tooltip-content={getRelative(selection.uri, accessor)}
-						data-tooltip-place='top'
-						data-tooltip-delay-show={3000}
+					{/* summarybox */}
+					<div
+						className={`
+							flex items-center gap-1 relative
+							px-1
+							w-fit h-fit
+							select-none
+							text-xs text-nowrap
+							border rounded-sm
+							${isThisSelectionProspective ? 'bg-void-bg-1 text-void-fg-3 opacity-80' : 'bg-void-bg-1 hover:brightness-95 text-void-fg-1'}
+							${isThisSelectionProspective
+								? 'border-void-border-2'
+								: 'border-void-border-1'
+							}
+							hover:border-void-border-1
+							transition-all duration-150
+						`}
+						onClick={() => {
+							if (type !== 'staging') return; // (never)
+							if (isThisSelectionProspective) { // add prospective selection to selections
+								setSelections([...selections, selection])
+							}
+							else if (selection.type === 'File') { // open files
+								voidOpenFileFn(selection.uri, accessor);
+
+								const wasAddedAsCurrentFile = selection.state.wasAddedAsCurrentFile
+								if (wasAddedAsCurrentFile) {
+									// make it so the file is added permanently, not just as the current file
+									const newSelection: StagingSelectionItem = { ...selection, state: { ...selection.state, wasAddedAsCurrentFile: false } }
+									setSelections([
+										...selections.slice(0, i),
+										newSelection,
+										...selections.slice(i + 1)
+									])
+								}
+							}
+							else if (selection.type === 'CodeSelection') {
+								voidOpenFileFn(selection.uri, accessor, selection.range);
+							}
+							else if (selection.type === 'Folder') {
+								// TODO!!! reveal in tree
+							}
+						}}
 					>
-						{/* summarybox */}
-						<div
-							className={`
-								flex items-center gap-1 relative
-								px-1
-								w-fit h-fit
-								select-none
-								text-xs text-nowrap
-								border rounded-sm
-								${isThisSelectionProspective ? 'bg-void-bg-1 text-void-fg-3 opacity-80' : 'bg-void-bg-1 hover:brightness-95 text-void-fg-1'}
-								${isThisSelectionProspective
-									? 'border-void-border-2'
-									: 'border-void-border-1'
-								}
-								hover:border-void-border-1
-								transition-all duration-150
-							`}
-							onClick={() => {
-								if (type !== 'staging') return; // (never)
-								if (isThisSelectionProspective) { // add prospective selection to selections
-									setSelections([...selections, selection])
-								}
-								else if (selection.type === 'File') { // open files
-									voidOpenFileFn(selection.uri, accessor);
+						{<SelectionIcon size={10} />}
 
-									const wasAddedAsCurrentFile = selection.state.wasAddedAsCurrentFile
-									if (wasAddedAsCurrentFile) {
-										// make it so the file is added permanently, not just as the current file
-										const newSelection: StagingSelectionItem = { ...selection, state: { ...selection.state, wasAddedAsCurrentFile: false } }
-										setSelections([
-											...selections.slice(0, i),
-											newSelection,
-											...selections.slice(i + 1)
-										])
-									}
-								}
-								else if (selection.type === 'CodeSelection') {
-									voidOpenFileFn(selection.uri, accessor, selection.range);
-								}
-								else if (selection.type === 'Folder') {
-									// TODO!!! reveal in tree
-								}
-							}}
-						>
-							{<SelectionIcon size={10} />}
+						{ // file name and range
+							getBasename(selection.uri.fsPath)
+							+ (selection.type === 'CodeSelection' ? ` (${selection.range[0]}-${selection.range[1]})` : '')
+						}
 
-							{ // file name and range
-								getBasename(selection.uri.fsPath)
-								+ (selection.type === 'CodeSelection' ? ` (${selection.range[0]}-${selection.range[1]})` : '')
-							}
+						{selection.type === 'File' && selection.state.wasAddedAsCurrentFile && messageIdx === undefined && currentURI?.fsPath === selection.uri.fsPath ?
+							<span className={`text-[8px] 'void-opacity-60 text-void-fg-4`}>
+								{`(Current File)`}
+							</span>
+							: null
+						}
 
-							{selection.type === 'File' && selection.state.wasAddedAsCurrentFile && messageIdx === undefined && currentURI?.fsPath === selection.uri.fsPath ?
-								<span className={`text-[8px] 'void-opacity-60 text-void-fg-4`}>
-									{`(Current File)`}
-								</span>
-								: null
-							}
-
-							{type === 'staging' && !isThisSelectionProspective ? // X button
-								<div // box for making it easier to click
-									className='cursor-pointer z-1 self-stretch flex items-center justify-center'
-									onClick={(e) => {
-										e.stopPropagation(); // don't open/close selection
-										if (type !== 'staging') return;
-										setSelections([...selections.slice(0, i), ...selections.slice(i + 1)])
-									}}
-								>
-									<IconX
-										className='stroke-[2]'
-										size={10}
-									/>
-								</div>
-								: <></>
-							}
-						</div>
-					</span>
+						{type === 'staging' && !isThisSelectionProspective ? // X button
+							<div // box for making it easier to click
+								className='cursor-pointer z-1 self-stretch flex items-center justify-center'
+								onClick={(e) => {
+									e.stopPropagation(); // don't open/close selection
+									if (type !== 'staging') return;
+									setSelections([...selections.slice(0, i), ...selections.slice(i + 1)])
+								}}
+							>
+								<IconX
+									className='stroke-[2]'
+									size={10}
+								/>
+							</div>
+							: <></>
+						}
+					</div>
 				</div>
 
 			})}
@@ -917,14 +907,11 @@ const EditTool = ({ toolMessage, threadId, messageIdx, content }: Parameters<Res
 	const desc1OnClick = () => voidOpenFileFn(params.uri, accessor)
 	const componentParams: ToolHeaderParams = { title, desc1, desc1OnClick, desc1Info, isError, icon, isRejected, }
 
-
-	const editToolType = toolMessage.name === 'edit_file' ? 'diff' : 'rewrite'
 	if (toolMessage.type === 'running_now' || toolMessage.type === 'tool_request') {
 		componentParams.children = <ToolChildrenWrapper className='bg-void-bg-3'>
 			<EditToolChildren
 				uri={params.uri}
 				code={content}
-				type={editToolType}
 			/>
 		</ToolChildrenWrapper>
 		// JumpToFileButton removed in favor of FileLinkText
@@ -949,7 +936,6 @@ const EditTool = ({ toolMessage, threadId, messageIdx, content }: Parameters<Res
 			<EditToolChildren
 				uri={params.uri}
 				code={content}
-				type={editToolType}
 			/>
 		</ToolChildrenWrapper>
 
@@ -1129,7 +1115,7 @@ const UserMessageComponent = ({ chatMessage, messageIdx, isCheckpointGhost, curr
 			if (e.key === 'Escape') {
 				onCloseEdit()
 			}
-			if (e.key === 'Enter' && !e.shiftKey && !e.nativeEvent.isComposing) {
+			if (e.key === 'Enter' && !e.shiftKey) {
 				onSubmit()
 			}
 		}
@@ -1402,7 +1388,7 @@ const loadingTitleWrapper = (item: React.ReactNode): React.ReactNode => {
 	</span>
 }
 
-const titleOfBuiltinToolName = {
+const titleOfToolName = {
 	'read_file': { done: 'Read file', proposed: 'Read file', running: loadingTitleWrapper('Reading file') },
 	'ls_dir': { done: 'Inspected folder', proposed: 'Inspect folder', running: loadingTitleWrapper('Inspecting folder') },
 	'get_dir_tree': { done: 'Inspected folder tree', proposed: 'Inspect folder tree', running: loadingTitleWrapper('Inspecting folder tree') },
@@ -1420,42 +1406,21 @@ const titleOfBuiltinToolName = {
 
 	'read_lint_errors': { done: `Read lint errors`, proposed: 'Read lint errors', running: loadingTitleWrapper('Reading lint errors') },
 	'search_in_file': { done: 'Searched in file', proposed: 'Search in file', running: loadingTitleWrapper('Searching in file') },
-} as const satisfies Record<BuiltinToolName, { done: any, proposed: any, running: any }>
+} as const satisfies Record<ToolName, { done: any, proposed: any, running: any }>
 
 
-const getTitle = (toolMessage: Pick<ChatMessage & { role: 'tool' }, 'name' | 'type' | 'mcpServerName'>): React.ReactNode => {
+const getTitle = (toolMessage: Pick<ChatMessage & { role: 'tool' }, 'name' | 'type'>): React.ReactNode => {
 	const t = toolMessage
+	if (!toolNames.includes(t.name as ToolName)) return t.name // good measure
 
-	// non-built-in title
-	if (!builtinToolNames.includes(t.name as BuiltinToolName)) {
-		// descriptor of Running or Ran etc
-		const descriptor =
-			t.type === 'success' ? 'Called'
-				: t.type === 'running_now' ? 'Calling'
-					: t.type === 'tool_request' ? 'Call'
-						: t.type === 'rejected' ? 'Call'
-							: t.type === 'invalid_params' ? 'Call'
-								: t.type === 'tool_error' ? 'Call'
-									: 'Call'
-
-
-		const title = `${descriptor} ${toolMessage.mcpServerName || 'MCP'}`
-		if (t.type === 'running_now' || t.type === 'tool_request')
-			return loadingTitleWrapper(title)
-		return title
-	}
-
-	// built-in title
-	else {
-		const toolName = t.name as BuiltinToolName
-		if (t.type === 'success') return titleOfBuiltinToolName[toolName].done
-		if (t.type === 'running_now') return titleOfBuiltinToolName[toolName].running
-		return titleOfBuiltinToolName[toolName].proposed
-	}
+	const toolName = t.name as ToolName
+	if (t.type === 'success') return titleOfToolName[toolName].done
+	if (t.type === 'running_now') return titleOfToolName[toolName].running
+	return titleOfToolName[toolName].proposed
 }
 
 
-const toolNameToDesc = (toolName: BuiltinToolName, _toolParams: BuiltinToolCallParams[BuiltinToolName] | undefined, accessor: ReturnType<typeof useAccessor>): {
+const toolNameToDesc = (toolName: ToolName, _toolParams: ToolCallParams[ToolName] | undefined, accessor: ReturnType<typeof useAccessor>): {
 	desc1: React.ReactNode,
 	desc1Info?: string,
 } => {
@@ -1466,95 +1431,95 @@ const toolNameToDesc = (toolName: BuiltinToolName, _toolParams: BuiltinToolCallP
 
 	const x = {
 		'read_file': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['read_file']
+			const toolParams = _toolParams as ToolCallParams['read_file']
 			return {
 				desc1: getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			};
 		},
 		'ls_dir': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['ls_dir']
+			const toolParams = _toolParams as ToolCallParams['ls_dir']
 			return {
 				desc1: getFolderName(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			};
 		},
 		'search_pathnames_only': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['search_pathnames_only']
+			const toolParams = _toolParams as ToolCallParams['search_pathnames_only']
 			return {
 				desc1: `"${toolParams.query}"`,
 			}
 		},
 		'search_for_files': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['search_for_files']
+			const toolParams = _toolParams as ToolCallParams['search_for_files']
 			return {
 				desc1: `"${toolParams.query}"`,
 			}
 		},
 		'search_in_file': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['search_in_file'];
+			const toolParams = _toolParams as ToolCallParams['search_in_file'];
 			return {
 				desc1: `"${toolParams.query}"`,
 				desc1Info: getRelative(toolParams.uri, accessor),
 			};
 		},
 		'create_file_or_folder': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['create_file_or_folder']
+			const toolParams = _toolParams as ToolCallParams['create_file_or_folder']
 			return {
 				desc1: toolParams.isFolder ? getFolderName(toolParams.uri.fsPath) ?? '/' : getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			}
 		},
 		'delete_file_or_folder': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['delete_file_or_folder']
+			const toolParams = _toolParams as ToolCallParams['delete_file_or_folder']
 			return {
 				desc1: toolParams.isFolder ? getFolderName(toolParams.uri.fsPath) ?? '/' : getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			}
 		},
 		'rewrite_file': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['rewrite_file']
+			const toolParams = _toolParams as ToolCallParams['rewrite_file']
 			return {
 				desc1: getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			}
 		},
 		'edit_file': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['edit_file']
+			const toolParams = _toolParams as ToolCallParams['edit_file']
 			return {
 				desc1: getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
 			}
 		},
 		'run_command': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['run_command']
+			const toolParams = _toolParams as ToolCallParams['run_command']
 			return {
 				desc1: `"${toolParams.command}"`,
 			}
 		},
 		'run_persistent_command': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['run_persistent_command']
+			const toolParams = _toolParams as ToolCallParams['run_persistent_command']
 			return {
 				desc1: `"${toolParams.command}"`,
 			}
 		},
 		'open_persistent_terminal': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['open_persistent_terminal']
+			const toolParams = _toolParams as ToolCallParams['open_persistent_terminal']
 			return { desc1: '' }
 		},
 		'kill_persistent_terminal': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['kill_persistent_terminal']
+			const toolParams = _toolParams as ToolCallParams['kill_persistent_terminal']
 			return { desc1: toolParams.persistentTerminalId }
 		},
 		'get_dir_tree': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['get_dir_tree']
+			const toolParams = _toolParams as ToolCallParams['get_dir_tree']
 			return {
 				desc1: getFolderName(toolParams.uri.fsPath) ?? '/',
 				desc1Info: getRelative(toolParams.uri, accessor),
 			}
 		},
 		'read_lint_errors': () => {
-			const toolParams = _toolParams as BuiltinToolCallParams['read_lint_errors']
+			const toolParams = _toolParams as ToolCallParams['read_lint_errors']
 			return {
 				desc1: getBasename(toolParams.uri.fsPath),
 				desc1Info: getRelative(toolParams.uri, accessor),
@@ -1625,9 +1590,9 @@ const ToolRequestAcceptRejectButtons = ({ toolName }: { toolName: ToolName }) =>
 		</button>
 	)
 
-	const approvalType = isABuiltinToolName(toolName) ? approvalTypeOfBuiltinToolName[toolName] : 'MCP tools'
+	const approvalType = approvalTypeOfToolName[toolName]
 	const approvalToggle = approvalType ? <div key={approvalType} className="flex items-center ml-2 gap-x-1">
-		<ToolApprovalTypeSwitch size='xs' approvalType={approvalType} desc={`Auto-approve ${approvalType}`} />
+		<ToolApprovalTypeSwitch size='xs' approvalType={approvalType} desc='Auto-approve' />
 	</div> : null
 
 	return <div className="flex gap-2 mx-0.5 items-center">
@@ -1639,7 +1604,7 @@ const ToolRequestAcceptRejectButtons = ({ toolName }: { toolName: ToolName }) =>
 
 export const ToolChildrenWrapper = ({ children, className }: { children: React.ReactNode, className?: string }) => {
 	return <div className={`${className ? className : ''} cursor-default select-none`}>
-		<div className='px-2 min-w-full overflow-hidden'>
+		<div className='px-2 min-w-full'>
 			{children}
 		</div>
 	</div>
@@ -1668,18 +1633,12 @@ export const ListableToolItem = ({ name, onClick, isSmall, className, showDot }:
 
 
 
-const EditToolChildren = ({ uri, code, type }: { uri: URI | undefined, code: string, type: 'diff' | 'rewrite' }) => {
-
-	const content = type === 'diff' ?
-		<VoidDiffEditor uri={uri} searchReplaceBlocks={code} />
-		: <ChatMarkdownRender string={`\`\`\`\n${code}\n\`\`\``} codeURI={uri} chatMessageLocation={undefined} />
-
+const EditToolChildren = ({ uri, code }: { uri: URI | undefined, code: string }) => {
 	return <div className='!select-text cursor-auto'>
 		<SmallProseWrapper>
-			{content}
+			<ChatMarkdownRender string={code} codeURI={uri} chatMessageLocation={undefined} />
 		</SmallProseWrapper>
 	</div>
-
 }
 
 
@@ -1730,9 +1689,9 @@ const EditToolHeaderButtons = ({ applyBoxId, uri, codeStr, toolName, threadId }:
 
 
 
-const InvalidTool = ({ toolName, message, mcpServerName }: { toolName: ToolName, message: string, mcpServerName: string | undefined }) => {
+const InvalidTool = ({ toolName, message }: { toolName: ToolName, message: string }) => {
 	const accessor = useAccessor()
-	const title = getTitle({ name: toolName, type: 'invalid_params', mcpServerName })
+	const title = getTitle({ name: toolName, type: 'invalid_params' })
 	const desc1 = 'Invalid parameters'
 	const icon = null
 	const isError = true
@@ -1746,9 +1705,9 @@ const InvalidTool = ({ toolName, message, mcpServerName }: { toolName: ToolName,
 	return <ToolHeaderWrapper {...componentParams} />
 }
 
-const CanceledTool = ({ toolName, mcpServerName }: { toolName: ToolName, mcpServerName: string | undefined }) => {
+const CanceledTool = ({ toolName }: { toolName: ToolName }) => {
 	const accessor = useAccessor()
-	const title = getTitle({ name: toolName, type: 'rejected', mcpServerName })
+	const title = getTitle({ name: toolName, type: 'rejected' })
 	const desc1 = ''
 	const icon = null
 	const isRejected = true
@@ -1860,61 +1819,9 @@ const CommandTool = ({ toolMessage, type, threadId }: { threadId: string } & ({
 	</>
 }
 
-type WrapperProps<T extends ToolName> = { toolMessage: Exclude<ToolMessage<T>, { type: 'invalid_params' }>, messageIdx: number, threadId: string }
-const MCPToolWrapper = ({ toolMessage }: WrapperProps<string>) => {
-	const accessor = useAccessor()
-	const mcpService = accessor.get('IMCPService')
 
-	const title = getTitle(toolMessage)
-	const desc1 = removeMCPToolNamePrefix(toolMessage.name)
-	const icon = null
-
-
-	if (toolMessage.type === 'running_now') return null // do not show running
-
-	const isError = false
-	const isRejected = toolMessage.type === 'rejected'
-	const { rawParams, params } = toolMessage
-	const componentParams: ToolHeaderParams = { title, desc1, isError, icon, isRejected, }
-
-	const paramsStr = JSON.stringify(params, null, 2)
-	componentParams.desc2 = <CopyButton codeStr={paramsStr} toolTipName={`Copy inputs: ${paramsStr}`} />
-
-	componentParams.info = !toolMessage.mcpServerName ? 'MCP tool not found' : undefined
-
-	// Add copy inputs button in desc2
-
-
-	if (toolMessage.type === 'success' || toolMessage.type === 'tool_request') {
-		const { result } = toolMessage
-		const resultStr = result ? mcpService.stringifyResult(result) : 'null'
-		componentParams.children = <ToolChildrenWrapper>
-			<SmallProseWrapper>
-				<ChatMarkdownRender
-					string={`\`\`\`json\n${resultStr}\n\`\`\``}
-					chatMessageLocation={undefined}
-					isApplyEnabled={false}
-					isLinkDetectionEnabled={true}
-				/>
-			</SmallProseWrapper>
-		</ToolChildrenWrapper>
-	}
-	else if (toolMessage.type === 'tool_error') {
-		const { result } = toolMessage
-		componentParams.bottomChildren = <BottomChildren title='Error'>
-			<CodeChildren>
-				{result}
-			</CodeChildren>
-		</BottomChildren>
-	}
-
-	return <ToolHeaderWrapper {...componentParams} />
-
-}
-
-type ResultWrapper<T extends ToolName> = (props: WrapperProps<T>) => React.ReactNode
-
-const builtinToolNameToComponent: { [T in BuiltinToolName]: { resultWrapper: ResultWrapper<T>, } } = {
+type ResultWrapper<T extends ToolName> = (props: { toolMessage: Exclude<ToolMessage<T>, { type: 'invalid_params' }>, messageIdx: number, threadId: string }) => React.ReactNode
+const toolNameToComponent: { [T in ToolName]: { resultWrapper: ResultWrapper<T>, } } = {
 	'read_file': {
 		resultWrapper: ({ toolMessage }) => {
 			const accessor = useAccessor()
@@ -2350,12 +2257,12 @@ const builtinToolNameToComponent: { [T in BuiltinToolName]: { resultWrapper: Res
 	},
 	'rewrite_file': {
 		resultWrapper: (params) => {
-			return <EditTool {...params} content={params.toolMessage.params.newContent} />
+			return <EditTool {...params} content={`${'```\n'}${params.toolMessage.params.newContent}${'\n```'}`} />
 		}
 	},
 	'edit_file': {
 		resultWrapper: (params) => {
-			return <EditTool {...params} content={params.toolMessage.params.searchReplaceBlocks} />
+			return <EditTool {...params} content={`${'```\n'}${params.toolMessage.params.searchReplaceBlocks}${'\n```'}`} />
 		}
 	},
 
@@ -2535,15 +2442,11 @@ const _ChatBubble = ({ threadId, chatMessage, currCheckpointIdx, isCommitted, me
 
 		if (chatMessage.type === 'invalid_params') {
 			return <div className={`${isCheckpointGhost ? 'opacity-50' : ''}`}>
-				<InvalidTool toolName={chatMessage.name} message={chatMessage.content} mcpServerName={chatMessage.mcpServerName} />
+				<InvalidTool toolName={chatMessage.name} message={chatMessage.content} />
 			</div>
 		}
 
-		const toolName = chatMessage.name
-		const isBuiltInTool = isABuiltinToolName(toolName)
-		const ToolResultWrapper = isBuiltInTool ? builtinToolNameToComponent[toolName]?.resultWrapper as ResultWrapper<ToolName>
-			: MCPToolWrapper as ResultWrapper<ToolName>
-
+		const ToolResultWrapper = toolNameToComponent[chatMessage.name]?.resultWrapper as ResultWrapper<ToolName>
 		if (ToolResultWrapper)
 			return <>
 				<div className={`${isCheckpointGhost ? 'opacity-50' : ''}`}>
@@ -2563,7 +2466,7 @@ const _ChatBubble = ({ threadId, chatMessage, currCheckpointIdx, isCommitted, me
 
 	else if (role === 'interrupted_streaming_tool') {
 		return <div className={`${isCheckpointGhost ? 'opacity-50' : ''}`}>
-			<CanceledTool toolName={chatMessage.name} mcpServerName={chatMessage.mcpServerName} />
+			<CanceledTool toolName={chatMessage.name} />
 		</div>
 	}
 
@@ -2843,13 +2746,12 @@ const CommandBarInChat = () => {
 
 const EditToolSoFar = ({ toolCallSoFar, }: { toolCallSoFar: RawToolCallObj }) => {
 
-	if (!isABuiltinToolName(toolCallSoFar.name)) return null
 
 	const accessor = useAccessor()
 
 	const uri = toolCallSoFar.rawParams.uri ? URI.file(toolCallSoFar.rawParams.uri) : undefined
 
-	const title = titleOfBuiltinToolName[toolCallSoFar.name].proposed
+	const title = titleOfToolName[toolCallSoFar.name].proposed
 
 	const uriDone = toolCallSoFar.doneParams.includes('uri')
 	const desc1 = <span className='flex items-center'>
@@ -2870,11 +2772,12 @@ const EditToolSoFar = ({ toolCallSoFar, }: { toolCallSoFar: RawToolCallObj }) =>
 		<EditToolChildren
 			uri={uri}
 			code={toolCallSoFar.rawParams.search_replace_blocks ?? toolCallSoFar.rawParams.new_content ?? ''}
-			type={'rewrite'} // as it streams, show in rewrite format, don't make a diff editor
 		/>
 		<IconLoading />
 	</ToolHeaderWrapper>
 
+
+
 }
 
 
@@ -2953,13 +2856,13 @@ export const SidebarChat = () => {
 
 	// resolve mount info
 	const isResolved = chatThreadsState.allThreads[threadId]?.state.mountedInfo?.mountedIsResolvedRef.current
+
 	useEffect(() => {
 		if (isResolved) return
 		chatThreadsState.allThreads[threadId]?.state.mountedInfo?._whenMountedResolver?.({
 			textAreaRef: textAreaRef,
 			scrollToBottom: () => scrollToBottom(scrollContainerRef),
 		})
-
 	}, [chatThreadsState, threadId, textAreaRef, scrollContainerRef, isResolved])
 
 
@@ -3056,7 +2959,7 @@ export const SidebarChat = () => {
 		setInstructionsAreEmpty(!newStr)
 	}, [setInstructionsAreEmpty])
 	const onKeyDown = useCallback((e: KeyboardEvent<HTMLTextAreaElement>) => {
-		if (e.key === 'Enter' && !e.shiftKey && !e.nativeEvent.isComposing) {
+		if (e.key === 'Enter' && !e.shiftKey) {
 			onSubmit()
 		} else if (e.key === 'Escape' && isRunning) {
 			onAbort()

src/vs/workbench/contrib/void/browser/react/src/util/inputs.tsx

@@ -20,11 +20,6 @@ import { URI } from '../../../../../../../base/common/uri.js';
 import { getBasename, getFolderName } from '../sidebar-tsx/SidebarChat.js';
 import { ChevronRight, File, Folder, FolderClosed, LucideProps } from 'lucide-react';
 import { StagingSelectionItem } from '../../../../common/chatThreadServiceTypes.js';
-import { DiffEditorWidget } from '../../../../../../../editor/browser/widget/diffEditor/diffEditorWidget.js';
-import { extractSearchReplaceBlocks, ExtractedSearchReplaceBlock } from '../../../../common/helpers/extractCodeFromResult.js';
-import { IAccessibilitySignalService } from '../../../../../../../platform/accessibilitySignal/browser/accessibilitySignalService.js';
-import { IEditorProgressService } from '../../../../../../../platform/progress/common/progress.js';
-import { detectLanguage } from '../../../../common/helpers/languageHelpers.js';
 
 
 // type guard
@@ -956,11 +951,11 @@ export const VoidInputBox = ({ onChangeText, onCreateInstance, inputBoxRef, plac
 
 	const contextViewProvider = accessor.get('IContextViewService')
 	return <WidgetComponent
+		ctor={InputBox}
 		className='
 			bg-void-bg-1
 			@@void-force-child-placeholder-void-fg-1
 		'
-		ctor={InputBox}
 		propsFn={useCallback((container) => [
 			container,
 			contextViewProvider,
@@ -996,7 +991,8 @@ export const VoidInputBox = ({ onChangeText, onCreateInstance, inputBoxRef, plac
 				inputBoxRef.current = instance;
 
 			return disposables
-		}, [onChangeText, onCreateInstance, inputBoxRef])}
+		}, [onChangeText, onCreateInstance, inputBoxRef])
+		}
 	/>
 };
 
@@ -1845,154 +1841,3 @@ export const VoidButtonBgDarken = ({ children, disabled, onClick, className }: {
 // };
 
 
-
-
-const SingleDiffEditor = ({ block, lang }: { block: ExtractedSearchReplaceBlock, lang: string | undefined }) => {
-	const accessor = useAccessor();
-	const modelService = accessor.get('IModelService');
-	const instantiationService = accessor.get('IInstantiationService');
-	const languageService = accessor.get('ILanguageService');
-
-	const languageSelection = useMemo(() => languageService.createById(lang), [lang, languageService]);
-
-	// Create models for original and modified
-	const originalModel = useMemo(() =>
-		modelService.createModel(block.orig, languageSelection),
-		[block.orig, languageSelection, modelService]
-	);
-	const modifiedModel = useMemo(() =>
-		modelService.createModel(block.final, languageSelection),
-		[block.final, languageSelection, modelService]
-	);
-
-	// Clean up models on unmount
-	useEffect(() => {
-		return () => {
-			originalModel.dispose();
-			modifiedModel.dispose();
-		};
-	}, [originalModel, modifiedModel]);
-
-	// Imperatively mount the DiffEditorWidget
-	const divRef = useRef<HTMLDivElement | null>(null);
-	const editorRef = useRef<any>(null);
-
-	useEffect(() => {
-		if (!divRef.current) return;
-		// Create the diff editor instance
-		const editor = instantiationService.createInstance(
-			DiffEditorWidget,
-			divRef.current,
-			{
-				automaticLayout: true,
-				readOnly: true,
-				renderSideBySide: true,
-				minimap: { enabled: false },
-				lineNumbers: 'off',
-				scrollbar: {
-					vertical: 'hidden',
-					horizontal: 'auto',
-					verticalScrollbarSize: 0,
-					horizontalScrollbarSize: 8,
-					alwaysConsumeMouseWheel: false,
-					ignoreHorizontalScrollbarInContentHeight: true,
-				},
-				hover: { enabled: false },
-				folding: false,
-				selectionHighlight: false,
-				renderLineHighlight: 'none',
-				overviewRulerLanes: 0,
-				hideCursorInOverviewRuler: true,
-				overviewRulerBorder: false,
-				glyphMargin: false,
-				stickyScroll: { enabled: false },
-				scrollBeyondLastLine: false,
-				renderGutterMenu: false,
-				renderIndicators: false,
-			},
-			{ originalEditor: { isSimpleWidget: true }, modifiedEditor: { isSimpleWidget: true } }
-		);
-		editor.setModel({ original: originalModel, modified: modifiedModel });
-
-		// Calculate the height based on content
-		const updateHeight = () => {
-			const contentHeight = Math.max(
-				originalModel.getLineCount() * 19, // approximate line height
-				modifiedModel.getLineCount() * 19
-			) + 19 * 2 + 1; // add padding
-
-			// Set reasonable min/max heights
-			const height = Math.min(Math.max(contentHeight, 100), 300);
-			if (divRef.current) {
-				divRef.current.style.height = `${height}px`;
-				editor.layout();
-			}
-		};
-
-		updateHeight();
-		editorRef.current = editor;
-
-		// Update height when content changes
-		const disposable1 = originalModel.onDidChangeContent(() => updateHeight());
-		const disposable2 = modifiedModel.onDidChangeContent(() => updateHeight());
-
-		return () => {
-			disposable1.dispose();
-			disposable2.dispose();
-			editor.dispose();
-			editorRef.current = null;
-		};
-	}, [originalModel, modifiedModel, instantiationService]);
-
-	return (
-		<div className="w-full bg-void-bg-3 @@bg-editor-style-override" ref={divRef} />
-	);
-};
-
-
-
-
-
-/**
- * ToolDiffEditor mounts a native VSCode DiffEditorWidget to show a diff between original and modified code blocks.
- * Props:
- *   - uri: URI of the file (for language detection, etc)
- *   - searchReplaceBlocks: string in search/replace format (from LLM)
- *   - language?: string (optional, fallback to 'plaintext')
- */
-export const VoidDiffEditor = ({ uri, searchReplaceBlocks, language }: { uri?: any, searchReplaceBlocks: string, language?: string }) => {
-	const accessor = useAccessor();
-	const languageService = accessor.get('ILanguageService');
-
-	// Extract all blocks
-	const blocks = extractSearchReplaceBlocks(searchReplaceBlocks);
-
-	// Use detectLanguage for language detection if not provided
-	let lang = language;
-	if (!lang && blocks.length > 0) {
-		lang = detectLanguage(languageService, { uri: uri ?? null, fileContents: blocks[0].orig });
-	}
-
-	// If no blocks, show empty state
-	if (blocks.length === 0) {
-		return <div className="w-full p-4 text-void-fg-4 text-sm">No changes found</div>;
-	}
-
-	// Display all blocks
-	return (
-		<div className="w-full flex flex-col gap-2">
-			{blocks.map((block, index) => (
-				<div key={index} className="w-full">
-					{blocks.length > 1 && (
-						<div className="text-void-fg-4 text-xs mb-1 px-1">
-							Change {index + 1} of {blocks.length}
-						</div>
-					)}
-					<SingleDiffEditor block={block} lang={lang} />
-				</div>
-			))}
-		</div>
-	);
-};
-
-

src/vs/workbench/contrib/void/browser/react/src/util/services.tsx

@@ -4,8 +4,8 @@
  *--------------------------------------------------------------------------------------*/
 
 import React, { useState, useEffect, useCallback } from 'react'
-import { MCPUserState, RefreshableProviderName, SettingsOfProvider } from '../../../../../../../workbench/contrib/void/common/voidSettingsTypes.js'
-import { DisposableStore, IDisposable } from '../../../../../../../base/common/lifecycle.js'
+import { RefreshableProviderName, SettingsOfProvider } from '../../../../../../../workbench/contrib/void/common/voidSettingsTypes.js'
+import { IDisposable } from '../../../../../../../base/common/lifecycle.js'
 import { VoidSettingsState } from '../../../../../../../workbench/contrib/void/common/voidSettingsService.js'
 import { ColorScheme } from '../../../../../../../platform/theme/common/theme.js'
 import { RefreshModelStateOfProvider } from '../../../../../../../workbench/contrib/void/common/refreshModelService.js'
@@ -21,6 +21,7 @@ import { IThemeService } from '../../../../../../../platform/theme/common/themeS
 import { ILLMMessageService } from '../../../../common/sendLLMMessageService.js';
 import { IRefreshModelService } from '../../../../../../../workbench/contrib/void/common/refreshModelService.js';
 import { IVoidSettingsService } from '../../../../../../../workbench/contrib/void/common/voidSettingsService.js';
+import { IVoidCertificateService } from '../../../../common/voidCertificateService.js';
 import { IExtensionTransferService } from '../../../../../../../workbench/contrib/void/browser/extensionTransferService.js'
 
 import { IInstantiationService } from '../../../../../../../platform/instantiation/common/instantiation.js'
@@ -51,9 +52,6 @@ import { IConvertToLLMMessageService } from '../../../convertToLLMMessageService
 import { ITerminalService } from '../../../../../terminal/browser/terminal.js'
 import { ISearchService } from '../../../../../../services/search/common/search.js'
 import { IExtensionManagementService } from '../../../../../../../platform/extensionManagement/common/extensionManagement.js'
-import { IMCPService } from '../../../../common/mcpService.js';
-import { IStorageService, StorageScope } from '../../../../../../../platform/storage/common/storage.js'
-import { OPT_OUT_KEY } from '../../../../common/storageKeys.js'
 
 
 // normally to do this you'd use a useEffect that calls .onDidChangeState(), but useEffect mounts too late and misses initial state changes
@@ -81,8 +79,6 @@ const ctrlKZoneStreamingStateListeners: Set<(diffareaid: number, s: boolean) =>
 const commandBarURIStateListeners: Set<(uri: URI) => void> = new Set();
 const activeURIListeners: Set<(uri: URI | null) => void> = new Set();
 
-const mcpListeners: Set<() => void> = new Set()
-
 
 // must call this before you can use any of the hooks below
 // this should only be called ONCE! this is the only place you don't need to dispose onDidChange. If you use state.onDidChange anywhere else, make sure to dispose it!
@@ -100,10 +96,9 @@ export const _registerServices = (accessor: ServicesAccessor) => {
 		editCodeService: accessor.get(IEditCodeService),
 		voidCommandBarService: accessor.get(IVoidCommandBarService),
 		modelService: accessor.get(IModelService),
-		mcpService: accessor.get(IMCPService),
 	}
 
-	const { settingsStateService, chatThreadsStateService, refreshModelService, themeService, editCodeService, voidCommandBarService, modelService, mcpService } = stateServices
+	const { settingsStateService, chatThreadsStateService, refreshModelService, themeService, editCodeService, voidCommandBarService, modelService } = stateServices
 
 
 
@@ -170,11 +165,6 @@ export const _registerServices = (accessor: ServicesAccessor) => {
 		})
 	)
 
-	disposables.push(
-		mcpService.onDidChangeState(() => {
-			mcpListeners.forEach(l => l())
-		})
-	)
 
 
 	return disposables
@@ -196,6 +186,7 @@ const getReactAccessor = (accessor: ServicesAccessor) => {
 		IVoidSettingsService: accessor.get(IVoidSettingsService),
 		IEditCodeService: accessor.get(IEditCodeService),
 		IChatThreadService: accessor.get(IChatThreadService),
+		IVoidCertificateService: accessor.get(IVoidCertificateService),
 
 		IInstantiationService: accessor.get(IInstantiationService),
 		ICodeEditorService: accessor.get(ICodeEditorService),
@@ -226,9 +217,6 @@ const getReactAccessor = (accessor: ServicesAccessor) => {
 		ITerminalService: accessor.get(ITerminalService),
 		IExtensionManagementService: accessor.get(IExtensionManagementService),
 		IExtensionTransferService: accessor.get(IExtensionTransferService),
-		IMCPService: accessor.get(IMCPService),
-
-		IStorageService: accessor.get(IStorageService),
 
 	} as const
 	return reactAccessor
@@ -390,39 +378,3 @@ export const useActiveURI = () => {
 
 
 
-
-export const useMCPServiceState = () => {
-	const accessor = useAccessor()
-	const mcpService = accessor.get('IMCPService')
-	const [s, ss] = useState(mcpService.state)
-	useEffect(() => {
-		const listener = () => { ss(mcpService.state) }
-		mcpListeners.add(listener);
-		return () => { mcpListeners.delete(listener) };
-	}, []);
-	return s
-}
-
-
-
-export const useIsOptedOut = () => {
-	const accessor = useAccessor()
-	const storageService = accessor.get('IStorageService')
-
-	const getVal = useCallback(() => {
-		return storageService.getBoolean(OPT_OUT_KEY, StorageScope.APPLICATION, false)
-	}, [storageService])
-
-	const [s, ss] = useState(getVal())
-
-	useEffect(() => {
-		const disposables = new DisposableStore();
-		const d = storageService.onDidChangeValue(StorageScope.APPLICATION, OPT_OUT_KEY, disposables)(e => {
-			ss(getVal())
-		})
-		disposables.add(d)
-		return () => disposables.clear()
-	}, [storageService, getVal])
-
-	return s
-}

src/vs/workbench/contrib/void/browser/react/src/void-onboarding/VoidOnboarding.tsx

@@ -100,7 +100,7 @@ const tabNames = ['Free', 'Paid', 'Local'] as const;
 type TabName = typeof tabNames[number] | 'Cloud/Other';
 
 // Data for cloud providers tab
-const cloudProviders: ProviderName[] = ['googleVertex', 'liteLLM', 'microsoftAzure', 'awsBedrock', 'openAICompatible'];
+const cloudProviders: ProviderName[] = ['googleVertex', 'liteLLM', 'microsoftAzure', 'openAICompatible'];
 
 // Data structures for provider tabs
 const providerNamesOfTab: Record<TabName, ProviderName[]> = {
@@ -123,7 +123,6 @@ const featureNameMap: { display: string, featureName: FeatureName }[] = [
 	{ display: 'Quick Edit', featureName: 'Ctrl+K' },
 	{ display: 'Autocomplete', featureName: 'Autocomplete' },
 	{ display: 'Fast Apply', featureName: 'Apply' },
-	{ display: 'Source Control', featureName: 'SCM' },
 ];
 
 const AddProvidersPage = ({ pageIndex, setPageIndex }: { pageIndex: number, setPageIndex: (index: number) => void }) => {

src/vs/workbench/contrib/void/browser/react/src/void-tooltip/VoidTooltip.tsx

@@ -50,8 +50,6 @@ export const VoidTooltip = () => {
 					padding: 0px 8px;
 					border-radius: 6px;
 					z-index: 999999;
-					max-width: 300px;
-					word-wrap: break-word;
 				}
 
 				#void-tooltip {

src/vs/workbench/contrib/void/browser/sidebarActions.ts

@@ -172,7 +172,7 @@ registerAction2(class extends Action2 {
 		const oldUI = await oldThread?.state.mountedInfo?.whenMounted
 
 		const oldSelns = oldThread?.state.stagingSelections
-		const oldVal = oldUI?.textAreaRef?.current?.value
+		const oldVal = oldUI?.textAreaRef.current?.value
 
 		// open and focus new thread
 		chatThreadsService.openNewThread()

src/vs/workbench/contrib/void/browser/terminalToolService.ts

@@ -297,12 +297,11 @@ export class TerminalToolService extends Disposable implements ITerminalToolServ
 
 
 			const cmdCap = await this._waitForCommandDetectionCapability(terminal)
-			// if (!cmdCap) throw new Error(`There was an error using the terminal: CommandDetection capability did not mount yet. Please try again in a few seconds or report this to the Void team.`)
+			if (!cmdCap) throw new Error(`There was an error using the terminal: CommandDetection capability did not mount yet. Please try again in a few seconds or report this to the Void team.`)
 
 			// Prefer the structured command-detection capability when available
 
 			const waitUntilDone = new Promise<void>(resolve => {
-				if (!cmdCap) return
 				const l = cmdCap.onCommandFinished(cmd => {
 					if (resolveReason) return // already resolved
 					resolveReason = { type: 'done', exitCode: cmd.exitCode ?? 0 };
@@ -347,20 +346,20 @@ export class TerminalToolService extends Disposable implements ITerminalToolServ
 			await Promise.any([waitUntilDone, waitUntilInterrupt])
 				.finally(() => disposables.forEach(d => d.dispose()))
 
-
-
-			// read result if timed out, since we didn't get it (could clean this code up but it's ok)
-			if (resolveReason?.type === 'timeout') {
-				const terminalId = isPersistent ? params.persistentTerminalId : params.terminalId
-				result = await this.readTerminal(terminalId)
-			}
-
 			if (!isPersistent) {
 				interrupt()
 			}
 
 			if (!resolveReason) throw new Error('Unexpected internal error: Promise.any should have resolved with a reason.')
 
+			// read result if timed out, since we didn't get it (could clean this code up but it's ok)
+			if (resolveReason.type === 'timeout') {
+				const terminalId = isPersistent ? params.persistentTerminalId : params.terminalId
+				result = await this.readTerminal(terminalId)
+			}
+
+
+
 			if (!isPersistent) result = `$ ${command}\n${result}`
 			result = removeAnsiEscapeCodes(result)
 			// trim

src/vs/workbench/contrib/void/browser/toolsService.ts

@@ -8,7 +8,7 @@ import { QueryBuilder } from '../../../services/search/common/queryBuilder.js'
 import { ISearchService } from '../../../services/search/common/search.js'
 import { IEditCodeService } from './editCodeServiceInterface.js'
 import { ITerminalToolService } from './terminalToolService.js'
-import { LintErrorItem, BuiltinToolCallParams, BuiltinToolResultType, BuiltinToolName } from '../common/toolsServiceTypes.js'
+import { LintErrorItem, ToolCallParams, ToolResultType } from '../common/toolsServiceTypes.js'
 import { IVoidModelService } from '../common/voidModelService.js'
 import { EndOfLinePreference } from '../../../../editor/common/model.js'
 import { IVoidCommandBarService } from './voidCommandBarService.js'
@@ -16,15 +16,20 @@ import { computeDirectoryTree1Deep, IDirectoryStrService, stringifyDirectoryTree
 import { IMarkerService, MarkerSeverity } from '../../../../platform/markers/common/markers.js'
 import { timeout } from '../../../../base/common/async.js'
 import { RawToolParamsObj } from '../common/sendLLMMessageTypes.js'
-import { MAX_CHILDREN_URIs_PAGE, MAX_FILE_CHARS_PAGE, MAX_TERMINAL_BG_COMMAND_TIME, MAX_TERMINAL_INACTIVE_TIME } from '../common/prompt/prompts.js'
+import { MAX_CHILDREN_URIs_PAGE, MAX_FILE_CHARS_PAGE, MAX_TERMINAL_BG_COMMAND_TIME, MAX_TERMINAL_INACTIVE_TIME, ToolName } from '../common/prompt/prompts.js'
 import { IVoidSettingsService } from '../common/voidSettingsService.js'
 import { generateUuid } from '../../../../base/common/uuid.js'
 
 
 // tool use for AI
-type ValidateBuiltinParams = { [T in BuiltinToolName]: (p: RawToolParamsObj) => BuiltinToolCallParams[T] }
-type CallBuiltinTool = { [T in BuiltinToolName]: (p: BuiltinToolCallParams[T]) => Promise<{ result: BuiltinToolResultType[T] | Promise<BuiltinToolResultType[T]>, interruptTool?: () => void }> }
-type BuiltinToolResultToString = { [T in BuiltinToolName]: (p: BuiltinToolCallParams[T], result: Awaited<BuiltinToolResultType[T]>) => string }
+
+
+
+
+type ValidateParams = { [T in ToolName]: (p: RawToolParamsObj) => ToolCallParams[T] }
+type CallTool = { [T in ToolName]: (p: ToolCallParams[T]) => Promise<{ result: ToolResultType[T] | Promise<ToolResultType[T]>, interruptTool?: () => void }> }
+type ToolResultToString = { [T in ToolName]: (p: ToolCallParams[T], result: Awaited<ToolResultType[T]>) => string }
+
 
 
 const isFalsy = (u: unknown) => {
@@ -42,29 +47,8 @@ const validateStr = (argName: string, value: unknown) => {
 const validateURI = (uriStr: unknown) => {
 	if (uriStr === null) throw new Error(`Invalid LLM output: uri was null.`)
 	if (typeof uriStr !== 'string') throw new Error(`Invalid LLM output format: Provided uri must be a string, but it's a(n) ${typeof uriStr}. Full value: ${JSON.stringify(uriStr)}.`)
-
-	// Check if it's already a full URI with scheme (e.g., vscode-remote://, file://, etc.)
-	// Look for :// pattern which indicates a scheme is present
-	// Examples of supported URIs:
-	// - vscode-remote://wsl+Ubuntu/home/user/file.txt (WSL)
-	// - vscode-remote://ssh-remote+myserver/home/user/file.txt (SSH)
-	// - file:///home/user/file.txt (local file with scheme)
-	// - /home/user/file.txt (local file path, will be converted to file://)
-	// - C:\Users\file.txt (Windows local path, will be converted to file://)
-	if (uriStr.includes('://')) {
-		try {
-			const uri = URI.parse(uriStr)
-			return uri
-		} catch (e) {
-			// If parsing fails, it's a malformed URI
-			throw new Error(`Invalid URI format: ${uriStr}. Error: ${e}`)
-		}
-	} else {
-		// No scheme present, treat as file path
-		// This handles regular file paths like /home/user/file.txt or C:\Users\file.txt
-		const uri = URI.file(uriStr)
-		return uri
-	}
+	const uri = URI.file(uriStr)
+	return uri
 }
 
 const validateOptionalURI = (uriStr: unknown) => {
@@ -126,9 +110,9 @@ const checkIfIsFolder = (uriStr: string) => {
 
 export interface IToolsService {
 	readonly _serviceBrand: undefined;
-	validateParams: ValidateBuiltinParams;
-	callTool: CallBuiltinTool;
-	stringOfResult: BuiltinToolResultToString;
+	validateParams: ValidateParams;
+	callTool: CallTool;
+	stringOfResult: ToolResultToString;
 }
 
 export const IToolsService = createDecorator<IToolsService>('ToolsService');
@@ -137,9 +121,9 @@ export class ToolsService implements IToolsService {
 
 	readonly _serviceBrand: undefined;
 
-	public validateParams: ValidateBuiltinParams;
-	public callTool: CallBuiltinTool;
-	public stringOfResult: BuiltinToolResultToString;
+	public validateParams: ValidateParams;
+	public callTool: CallTool;
+	public stringOfResult: ToolResultToString;
 
 	constructor(
 		@IFileService fileService: IFileService,
@@ -154,6 +138,7 @@ export class ToolsService implements IToolsService {
 		@IMarkerService private readonly markerService: IMarkerService,
 		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
 	) {
+
 		const queryBuilder = instantiationService.createInstance(QueryBuilder);
 
 		this.validateParams = {
@@ -461,6 +446,7 @@ export class ToolsService implements IToolsService {
 				await this.terminalToolService.killPersistentTerminal(persistentTerminalId)
 				return { result: {} }
 			},
+
 		}
 
 
@@ -564,6 +550,7 @@ export class ToolsService implements IToolsService {
 			kill_persistent_terminal: (params, _result) => {
 				return `Successfully closed terminal "${params.persistentTerminalId}".`;
 			},
+
 		}
 
 

src/vs/workbench/contrib/void/browser/void.contribution.ts

@@ -61,9 +61,6 @@ import './miscWokrbenchContrib.js'
 // register file service (for explorer context menu)
 import './fileService.js'
 
-// register source control management
-import './voidSCMService.js'
-
 // ---------- common (unclear if these actually need to be imported, because they're already imported wherever they're used) ----------
 
 // llmMessage

src/vs/workbench/contrib/void/browser/voidSCMService.ts

@@ -1,230 +0,0 @@
-/*--------------------------------------------------------------------------------------
- *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
- *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
- *--------------------------------------------------------------------------------------*/
-
-import { ThemeIcon } from '../../../../base/common/themables.js'
-import { localize2 } from '../../../../nls.js'
-import { Action2, MenuId, registerAction2 } from '../../../../platform/actions/common/actions.js'
-import { ContextKeyExpr, IContextKey, IContextKeyService } from '../../../../platform/contextkey/common/contextkey.js'
-import { ISCMService } from '../../scm/common/scm.js'
-import { ProxyChannel } from '../../../../base/parts/ipc/common/ipc.js'
-import { IVoidSCMService } from '../common/voidSCMTypes.js'
-import { IMainProcessService } from '../../../../platform/ipc/common/mainProcessService.js'
-import { IVoidSettingsService } from '../common/voidSettingsService.js'
-import { IConvertToLLMMessageService } from './convertToLLMMessageService.js'
-import { ILLMMessageService } from '../common/sendLLMMessageService.js'
-import { ModelSelection, OverridesOfModel, ModelSelectionOptions } from '../common/voidSettingsTypes.js'
-import { gitCommitMessage_systemMessage, gitCommitMessage_userMessage } from '../common/prompt/prompts.js'
-import { LLMChatMessage } from '../common/sendLLMMessageTypes.js'
-import { generateUuid } from '../../../../base/common/uuid.js'
-import { ThrottledDelayer } from '../../../../base/common/async.js'
-import { CancellationError, isCancellationError } from '../../../../base/common/errors.js'
-import { registerSingleton, InstantiationType } from '../../../../platform/instantiation/common/extensions.js'
-import { createDecorator, ServicesAccessor } from '../../../../platform/instantiation/common/instantiation.js'
-import { Disposable } from '../../../../base/common/lifecycle.js'
-import { INotificationService } from '../../../../platform/notification/common/notification.js'
-
-interface ModelOptions {
-	modelSelection: ModelSelection | null
-	modelSelectionOptions?: ModelSelectionOptions
-	overridesOfModel: OverridesOfModel
-}
-
-export interface IGenerateCommitMessageService {
-	readonly _serviceBrand: undefined
-	generateCommitMessage(): Promise<void>
-	abort(): void
-}
-
-export const IGenerateCommitMessageService = createDecorator<IGenerateCommitMessageService>('voidGenerateCommitMessageService');
-
-const loadingContextKey = 'voidSCMGenerateCommitMessageLoading'
-
-class GenerateCommitMessageService extends Disposable implements IGenerateCommitMessageService {
-	readonly _serviceBrand: undefined;
-	private readonly execute = new ThrottledDelayer(300)
-	private llmRequestId: string | null = null
-	private currentRequestId: string | null = null
-	private voidSCM: IVoidSCMService
-	private loadingContextKey: IContextKey<boolean>
-
-	constructor(
-		@ISCMService private readonly scmService: ISCMService,
-		@IMainProcessService mainProcessService: IMainProcessService,
-		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
-		@IConvertToLLMMessageService private readonly convertToLLMMessageService: IConvertToLLMMessageService,
-		@ILLMMessageService private readonly llmMessageService: ILLMMessageService,
-		@IContextKeyService private readonly contextKeyService: IContextKeyService,
-		@INotificationService private readonly notificationService: INotificationService
-	) {
-		super()
-		this.loadingContextKey = this.contextKeyService.createKey(loadingContextKey, false)
-		this.voidSCM = ProxyChannel.toService<IVoidSCMService>(mainProcessService.getChannel('void-channel-scm'))
-	}
-
-	override dispose() {
-		this.execute.dispose()
-		super.dispose()
-	}
-
-	async generateCommitMessage() {
-		this.loadingContextKey.set(true)
-		this.execute.trigger(async () => {
-			const requestId = generateUuid()
-			this.currentRequestId = requestId
-
-
-			try {
-				const { path, repo } = this.gitRepoInfo()
-				const [stat, sampledDiffs, branch, log] = await Promise.all([
-					this.voidSCM.gitStat(path),
-					this.voidSCM.gitSampledDiffs(path),
-					this.voidSCM.gitBranch(path),
-					this.voidSCM.gitLog(path)
-				])
-
-				if (!this.isCurrentRequest(requestId)) { throw new CancellationError() }
-
-				const modelSelection = this.voidSettingsService.state.modelSelectionOfFeature['SCM'] ?? null
-				const modelSelectionOptions = modelSelection ? this.voidSettingsService.state.optionsOfModelSelection['SCM'][modelSelection?.providerName]?.[modelSelection.modelName] : undefined
-				const overridesOfModel = this.voidSettingsService.state.overridesOfModel
-
-				const modelOptions: ModelOptions = { modelSelection, modelSelectionOptions, overridesOfModel }
-
-				const prompt = gitCommitMessage_userMessage(stat, sampledDiffs, branch, log)
-
-				const simpleMessages = [{ role: 'user', content: prompt } as const]
-				const { messages, separateSystemMessage } = this.convertToLLMMessageService.prepareLLMSimpleMessages({
-					simpleMessages,
-					systemMessage: gitCommitMessage_systemMessage,
-					modelSelection: modelOptions.modelSelection,
-					featureName: 'SCM',
-				})
-
-				const commitMessage = await this.sendLLMMessage(messages, separateSystemMessage!, modelOptions)
-
-				if (!this.isCurrentRequest(requestId)) { throw new CancellationError() }
-
-				repo.input.setValue(commitMessage, false)
-			} catch (error) {
-				this.onError(error)
-			} finally {
-				if (this.isCurrentRequest(requestId)) {
-					this.loadingContextKey.set(false)
-				}
-			}
-		})
-	}
-
-	abort() {
-		if (this.llmRequestId) {
-			this.llmMessageService.abort(this.llmRequestId)
-		}
-		this.execute.cancel()
-		this.loadingContextKey.set(false)
-		this.currentRequestId = null
-	}
-
-	private gitRepoInfo() {
-		const repo = Array.from(this.scmService.repositories || []).find((r: any) => r.provider.contextValue === 'git')
-		if (!repo) { throw new Error('No git repository found') }
-		if (!repo.provider.rootUri?.fsPath) { throw new Error('No git repository root path found') }
-		return { path: repo.provider.rootUri.fsPath, repo }
-	}
-
-	/** LLM Functions */
-
-	private sendLLMMessage(messages: LLMChatMessage[], separateSystemMessage: string, modelOptions: ModelOptions): Promise<string> {
-		return new Promise((resolve, reject) => {
-
-			this.llmRequestId = this.llmMessageService.sendLLMMessage({
-				messagesType: 'chatMessages',
-				messages,
-				separateSystemMessage,
-				chatMode: null,
-				modelSelection: modelOptions.modelSelection,
-				modelSelectionOptions: modelOptions.modelSelectionOptions,
-				overridesOfModel: modelOptions.overridesOfModel,
-				onText: () => { },
-				onFinalMessage: (params: { fullText: string }) => {
-					const match = params.fullText.match(/<output>([\s\S]*?)<\/output>/i)
-					const commitMessage = match ? match[1].trim() : ''
-					resolve(commitMessage)
-				},
-				onError: (error) => {
-					console.error(error)
-					reject(error)
-				},
-				onAbort: () => {
-					reject(new CancellationError())
-				},
-				logging: { loggingName: 'VoidSCM - Commit Message' },
-			})
-		})
-	}
-
-
-	/** Request Helpers */
-
-	private isCurrentRequest(requestId: string) {
-		return requestId === this.currentRequestId
-	}
-
-
-	/** UI Functions */
-
-	private onError(error: any) {
-		if (!isCancellationError(error)) {
-			console.error(error)
-			this.notificationService.error(localize2('voidFailedToGenerateCommitMessage', 'Failed to generate commit message.').value)
-		}
-	}
-}
-
-class GenerateCommitMessageAction extends Action2 {
-	constructor() {
-		super({
-			id: 'void.generateCommitMessageAction',
-			title: localize2('voidCommitMessagePrompt', 'Void: Generate Commit Message'),
-			icon: ThemeIcon.fromId('sparkle'),
-			tooltip: localize2('voidCommitMessagePromptTooltip', 'Void: Generate Commit Message'),
-			f1: true,
-			menu: [{
-				id: MenuId.SCMInputBox,
-				when: ContextKeyExpr.and(ContextKeyExpr.equals('scmProvider', 'git'), ContextKeyExpr.equals(loadingContextKey, false)),
-				group: 'inline'
-			}]
-		})
-	}
-
-	async run(accessor: ServicesAccessor): Promise<void> {
-		const generateCommitMessageService = accessor.get(IGenerateCommitMessageService)
-		generateCommitMessageService.generateCommitMessage()
-	}
-}
-
-class LoadingGenerateCommitMessageAction extends Action2 {
-	constructor() {
-		super({
-			id: 'void.loadingGenerateCommitMessageAction',
-			title: localize2('voidCommitMessagePromptCancel', 'Void: Cancel Commit Message Generation'),
-			icon: ThemeIcon.fromId('stop-circle'),
-			tooltip: localize2('voidCommitMessagePromptCancelTooltip', 'Void: Cancel Commit Message Generation'),
-			f1: false, //Having a cancel command in the command palette is more confusing than useful.
-			menu: [{
-				id: MenuId.SCMInputBox,
-				when: ContextKeyExpr.and(ContextKeyExpr.equals('scmProvider', 'git'), ContextKeyExpr.equals(loadingContextKey, true)),
-				group: 'inline'
-			}]
-		})
-	}
-	async run(accessor: ServicesAccessor): Promise<void> {
-		const generateCommitMessageService = accessor.get(IGenerateCommitMessageService)
-		generateCommitMessageService.abort()
-	}
-}
-
-registerAction2(GenerateCommitMessageAction)
-registerAction2(LoadingGenerateCommitMessageAction)
-registerSingleton(IGenerateCommitMessageService, GenerateCommitMessageService, InstantiationType.Delayed)

src/vs/workbench/contrib/void/browser/voidUpdateActions.ts

@@ -8,7 +8,7 @@ import Severity from '../../../../base/common/severity.js';
 import { ServicesAccessor } from '../../../../editor/browser/editorExtensions.js';
 import { localize2 } from '../../../../nls.js';
 import { Action2, registerAction2 } from '../../../../platform/actions/common/actions.js';
-import { INotificationActions, INotificationHandle, INotificationService } from '../../../../platform/notification/common/notification.js';
+import { INotificationActions, INotificationService } from '../../../../platform/notification/common/notification.js';
 import { IMetricsService } from '../common/metricsService.js';
 import { IVoidUpdateService } from '../common/voidUpdateService.js';
 import { IWorkbenchContribution, registerWorkbenchContribution2, WorkbenchPhase } from '../../../common/contributions.js';
@@ -20,7 +20,7 @@ import { IAction } from '../../../../base/common/actions.js';
 
 
 
-const notifyUpdate = (res: VoidCheckUpdateRespose & { message: string }, notifService: INotificationService, updateService: IUpdateService): INotificationHandle => {
+const notifyUpdate = (res: VoidCheckUpdateRespose & { message: string }, notifService: INotificationService, updateService: IUpdateService) => {
 	const message = res?.message || 'This is a very old version of Void, please download the latest version! [Void Editor](https://voideditor.com/download-beta)!'
 
 	let actions: INotificationActions | undefined
@@ -119,21 +119,18 @@ const notifyUpdate = (res: VoidCheckUpdateRespose & { message: string }, notifSe
 		progress: actions ? { worked: 0, total: 100 } : undefined,
 		actions: actions,
 	})
-
-	return notifController
 	// const d = notifController.onDidClose(() => {
 	// 	notifyYesUpdate(notifService, res)
 	// 	d.dispose()
 	// })
 }
-const notifyErrChecking = (notifService: INotificationService): INotificationHandle => {
+const notifyErrChecking = (notifService: INotificationService) => {
 	const message = `Void Error: There was an error checking for updates. If this persists, please get in touch or reinstall Void [here](https://voideditor.com/download-beta)!`
-	const notifController = notifService.notify({
+	notifService.notify({
 		severity: Severity.Info,
 		message: message,
 		sticky: true,
 	})
-	return notifController
 }
 
 
@@ -143,35 +140,30 @@ const performVoidCheck = async (
 	voidUpdateService: IVoidUpdateService,
 	metricsService: IMetricsService,
 	updateService: IUpdateService,
-): Promise<INotificationHandle | null> => {
+) => {
 
 	const metricsTag = explicit ? 'Manual' : 'Auto'
 
 	metricsService.capture(`Void Update ${metricsTag}: Checking...`, {})
 	const res = await voidUpdateService.check(explicit)
 	if (!res) {
-		const notifController = notifyErrChecking(notifService);
+		notifyErrChecking(notifService);
 		metricsService.capture(`Void Update ${metricsTag}: Error`, { res })
-		return notifController
 	}
 	else {
 		if (res.message) {
-			const notifController = notifyUpdate(res, notifService, updateService)
+			notifyUpdate(res, notifService, updateService)
 			metricsService.capture(`Void Update ${metricsTag}: Yes`, { res })
-			return notifController
 		}
 		else {
 			metricsService.capture(`Void Update ${metricsTag}: No`, { res })
-			return null
+			return
 		}
 	}
 }
 
 
 // Action
-let lastNotifController: INotificationHandle | null = null
-
-
 registerAction2(class extends Action2 {
 	constructor() {
 		super({
@@ -185,15 +177,7 @@ registerAction2(class extends Action2 {
 		const notifService = accessor.get(INotificationService)
 		const metricsService = accessor.get(IMetricsService)
 		const updateService = accessor.get(IUpdateService)
-
-		const currNotifController = lastNotifController
-
-		const newController = await performVoidCheck(true, notifService, voidUpdateService, metricsService, updateService)
-
-		if (newController) {
-			currNotifController?.close()
-			lastNotifController = newController
-		}
+		performVoidCheck(true, notifService, voidUpdateService, metricsService, updateService)
 	}
 })
 

src/vs/workbench/contrib/void/common/chatThreadServiceTypes.ts

@@ -5,32 +5,31 @@
 
 import { URI } from '../../../../base/common/uri.js';
 import { VoidFileSnapshot } from './editCodeServiceTypes.js';
+import { ToolName } from './prompt/prompts.js';
 import { AnthropicReasoning, RawToolParamsObj } from './sendLLMMessageTypes.js';
-import { ToolCallParams, ToolName, ToolResult } from './toolsServiceTypes.js';
+import { ToolCallParams, ToolResultType } from './toolsServiceTypes.js';
 
 export type ToolMessage<T extends ToolName> = {
 	role: 'tool';
 	content: string; // give this result to LLM (string of value)
 	id: string;
 	rawParams: RawToolParamsObj;
-	mcpServerName: string | undefined; // the server name at the time of the call
 } & (
 		// in order of events:
 		| { type: 'invalid_params', result: null, name: T, }
 
-		| { type: 'tool_request', result: null, name: T, params: ToolCallParams<T>, }  // params were validated, awaiting user
+		| { type: 'tool_request', result: null, name: T, params: ToolCallParams[T], }  // params were validated, awaiting user
 
-		| { type: 'running_now', result: null, name: T, params: ToolCallParams<T>, }
+		| { type: 'running_now', result: null, name: T, params: ToolCallParams[T], }
 
-		| { type: 'tool_error', result: string, name: T, params: ToolCallParams<T>, } // error when tool was running
-		| { type: 'success', result: Awaited<ToolResult<T>>, name: T, params: ToolCallParams<T>, }
-		| { type: 'rejected', result: null, name: T, params: ToolCallParams<T> }
+		| { type: 'tool_error', result: string, name: T, params: ToolCallParams[T], } // error when tool was running
+		| { type: 'success', result: Awaited<ToolResultType[T]>, name: T, params: ToolCallParams[T], }
+		| { type: 'rejected', result: null, name: T, params: ToolCallParams[T] }
 	) // user rejected
 
 export type DecorativeCanceledTool = {
 	role: 'interrupted_streaming_tool';
 	name: ToolName;
-	mcpServerName: string | undefined; // the server name at the time of the call
 }
 
 

src/vs/workbench/contrib/void/common/directoryStrService.ts

@@ -9,7 +9,7 @@ import { registerSingleton, InstantiationType } from '../../../../platform/insta
 import { createDecorator } from '../../../../platform/instantiation/common/instantiation.js';
 import { IFileService, IFileStat } from '../../../../platform/files/common/files.js';
 import { IWorkspaceContextService } from '../../../../platform/workspace/common/workspace.js';
-import { ShallowDirectoryItem, BuiltinToolCallParams, BuiltinToolResultType } from './toolsServiceTypes.js';
+import { ShallowDirectoryItem, ToolCallParams, ToolResultType } from './toolsServiceTypes.js';
 import { MAX_CHILDREN_URIs_PAGE, MAX_DIRSTR_CHARS_TOTAL_BEGINNING, MAX_DIRSTR_CHARS_TOTAL_TOOL } from './prompt/prompts.js';
 
 
@@ -76,7 +76,7 @@ export const computeDirectoryTree1Deep = async (
 	fileService: IFileService,
 	rootURI: URI,
 	pageNumber: number = 1,
-): Promise<BuiltinToolResultType['ls_dir']> => {
+): Promise<ToolResultType['ls_dir']> => {
 	const stat = await fileService.resolve(rootURI, { resolveMetadata: false });
 	if (!stat.isDirectory) {
 		return { children: null, hasNextPage: false, hasPrevPage: false, itemsRemaining: 0 };
@@ -107,7 +107,7 @@ export const computeDirectoryTree1Deep = async (
 	};
 };
 
-export const stringifyDirectoryTree1Deep = (params: BuiltinToolCallParams['ls_dir'], result: BuiltinToolResultType['ls_dir']): string => {
+export const stringifyDirectoryTree1Deep = (params: ToolCallParams['ls_dir'], result: ToolResultType['ls_dir']): string => {
 	if (!result.children) {
 		return `Error: ${params.uri} is not a directory`;
 	}

src/vs/workbench/contrib/void/common/mcpService.ts

@@ -1,360 +0,0 @@
-/*--------------------------------------------------------------------------------------
- *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
- *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
- *--------------------------------------------------------------------------------------*/
-
-import { URI } from '../../../../base/common/uri.js';
-import { Disposable } from '../../../../base/common/lifecycle.js';
-import { registerSingleton, InstantiationType } from '../../../../platform/instantiation/common/extensions.js';
-import { createDecorator } from '../../../../platform/instantiation/common/instantiation.js';
-import { IFileService } from '../../../../platform/files/common/files.js';
-import { IPathService } from '../../../services/path/common/pathService.js';
-import { IEditorService } from '../../../services/editor/common/editorService.js';
-import { IProductService } from '../../../../platform/product/common/productService.js';
-import { VSBuffer } from '../../../../base/common/buffer.js';
-import { IChannel } from '../../../../base/parts/ipc/common/ipc.js';
-import { IMainProcessService } from '../../../../platform/ipc/common/mainProcessService.js';
-import { MCPServerOfName, MCPConfigFileJSON, MCPServer, MCPToolCallParams, RawMCPToolCall, MCPServerEventResponse } from './mcpServiceTypes.js';
-import { Event, Emitter } from '../../../../base/common/event.js';
-import { InternalToolInfo } from './prompt/prompts.js';
-import { IVoidSettingsService } from './voidSettingsService.js';
-import { MCPUserStateOfName } from './voidSettingsTypes.js';
-
-
-type MCPServiceState = {
-	mcpServerOfName: MCPServerOfName,
-	error: string | undefined, // global parsing error
-}
-
-export interface IMCPService {
-	readonly _serviceBrand: undefined;
-	revealMCPConfigFile(): Promise<void>;
-	toggleServerIsOn(serverName: string, isOn: boolean): Promise<void>;
-
-	readonly state: MCPServiceState; // NOT persisted
-	onDidChangeState: Event<void>;
-
-	getMCPTools(): InternalToolInfo[] | undefined;
-	callMCPTool(toolData: MCPToolCallParams): Promise<{ result: RawMCPToolCall }>;
-	stringifyResult(result: RawMCPToolCall): string
-}
-
-export const IMCPService = createDecorator<IMCPService>('mcpConfigService');
-
-
-
-const MCP_CONFIG_FILE_NAME = 'mcp.json';
-const MCP_CONFIG_SAMPLE = { mcpServers: {} }
-const MCP_CONFIG_SAMPLE_STRING = JSON.stringify(MCP_CONFIG_SAMPLE, null, 2);
-
-
-// export interface MCPCallToolOfToolName {
-// 	[toolName: string]: (params: any) => Promise<{
-// 		result: any | Promise<any>,
-// 		interruptTool?: () => void
-// 	}>;
-// }
-
-
-class MCPService extends Disposable implements IMCPService {
-	_serviceBrand: undefined;
-
-
-	private readonly channel: IChannel // MCPChannel
-
-	// list of MCP servers pulled from mcpChannel
-	state: MCPServiceState = {
-		mcpServerOfName: {},
-		error: undefined,
-	}
-
-	// Emitters for server events
-	private readonly _onDidChangeState = new Emitter<void>();
-	public readonly onDidChangeState = this._onDidChangeState.event;
-
-	// private readonly _onLoadingServersChange = new Emitter<MCPServerEventLoadingParam>();
-	// public readonly onLoadingServersChange = this._onLoadingServersChange.event;
-
-	constructor(
-		@IFileService private readonly fileService: IFileService,
-		@IPathService private readonly pathService: IPathService,
-		@IProductService private readonly productService: IProductService,
-		@IEditorService private readonly editorService: IEditorService,
-		@IMainProcessService private readonly mainProcessService: IMainProcessService,
-		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
-	) {
-		super();
-		this.channel = this.mainProcessService.getChannel('void-channel-mcp')
-
-
-		const onEvent = (e: MCPServerEventResponse) => {
-			// console.log('GOT EVENT', e)
-			this._setMCPServerState(e.response.name, e.response.newServer)
-		}
-		this._register((this.channel.listen('onAdd_server') satisfies Event<MCPServerEventResponse>)(onEvent));
-		this._register((this.channel.listen('onUpdate_server') satisfies Event<MCPServerEventResponse>)(onEvent));
-		this._register((this.channel.listen('onDelete_server') satisfies Event<MCPServerEventResponse>)(onEvent));
-
-		this._initialize();
-	}
-
-
-	private async _initialize() {
-		try {
-			await this.voidSettingsService.waitForInitState;
-
-			// Create .mcpConfig if it doesn't exist
-			const mcpConfigUri = await this._getMCPConfigFilePath();
-			const fileExists = await this._configFileExists(mcpConfigUri);
-			if (!fileExists) {
-				await this._createMCPConfigFile(mcpConfigUri);
-				console.log('MCP Config file created:', mcpConfigUri.toString());
-			}
-			await this._addMCPConfigFileWatcher();
-			await this._refreshMCPServers();
-		} catch (error) {
-			console.error('Error initializing MCPService:', error);
-		}
-	}
-
-	private readonly _setMCPServerState = async (serverName: string, newServer: MCPServer | undefined) => {
-		if (newServer === undefined) {
-			// Remove the server from the state
-			const { [serverName]: removed, ...remainingServers } = this.state.mcpServerOfName;
-			this.state = {
-				...this.state,
-				mcpServerOfName: remainingServers
-			}
-		} else {
-			// Add or update the server
-			this.state = {
-				...this.state,
-				mcpServerOfName: {
-					...this.state.mcpServerOfName,
-					[serverName]: newServer
-				}
-			}
-		}
-		this._onDidChangeState.fire();
-	}
-
-	private readonly _setHasError = async (errMsg: string | undefined) => {
-		this.state = {
-			...this.state,
-			error: errMsg,
-		}
-		this._onDidChangeState.fire();
-	}
-
-	// Create the file/directory if it doesn't exist
-	private async _createMCPConfigFile(mcpConfigUri: URI): Promise<void> {
-		await this.fileService.createFile(mcpConfigUri.with({ path: mcpConfigUri.path }));
-		const buffer = VSBuffer.fromString(MCP_CONFIG_SAMPLE_STRING);
-		await this.fileService.writeFile(mcpConfigUri, buffer);
-	}
-
-
-	private async _addMCPConfigFileWatcher(): Promise<void> {
-		const mcpConfigUri = await this._getMCPConfigFilePath();
-		this._register(
-			this.fileService.watch(mcpConfigUri)
-		)
-
-		this._register(this.fileService.onDidFilesChange(async e => {
-			if (!e.contains(mcpConfigUri)) return
-			await this._refreshMCPServers();
-		}));
-	}
-
-	// Client-side functions
-
-	public async revealMCPConfigFile(): Promise<void> {
-		try {
-			const mcpConfigUri = await this._getMCPConfigFilePath();
-			await this.editorService.openEditor({
-				resource: mcpConfigUri,
-				options: {
-					pinned: true,
-					revealIfOpened: true,
-				}
-			});
-		} catch (error) {
-			console.error('Error opening MCP config file:', error);
-		}
-	}
-
-	public getMCPTools(): InternalToolInfo[] | undefined {
-		const allTools: InternalToolInfo[] = []
-		for (const serverName in this.state.mcpServerOfName) {
-			const server = this.state.mcpServerOfName[serverName];
-			server.tools?.forEach(tool => {
-				allTools.push({
-					description: tool.description || '',
-					params: this._transformInputSchemaToParams(tool.inputSchema),
-					name: tool.name,
-					mcpServerName: serverName,
-				})
-			})
-		}
-		if (allTools.length === 0) return undefined
-		return allTools
-	}
-
-	private _transformInputSchemaToParams(inputSchema?: Record<string, any>): { [paramName: string]: { description: string } } {
-
-		// Check if inputSchema is valid
-		if (!inputSchema || !inputSchema.properties) return {};
-
-		const params: { [paramName: string]: { description: string } } = {};
-		Object.keys(inputSchema.properties).forEach(paramName => {
-			const propertyValues = inputSchema.properties[paramName];
-
-			// Check if propertyValues is not an object
-			if (typeof propertyValues !== 'object') {
-				console.warn(`Invalid property value for ${paramName}: expected object, got ${typeof propertyValues}`);
-				return; // in forEach the return is equivalent to continue
-			}
-
-			// Add the parameter to the params object
-			params[paramName] = {
-				description: JSON.stringify(propertyValues.description || '', null, 2) || '',
-			}
-		});
-		return params;
-	}
-
-	private async _getMCPConfigFilePath(): Promise<URI> {
-		const appName = this.productService.dataFolderName
-		const userHome = await this.pathService.userHome();
-		const uri = URI.joinPath(userHome, appName, MCP_CONFIG_FILE_NAME)
-		return uri
-	}
-
-	private async _configFileExists(mcpConfigUri: URI): Promise<boolean> {
-		try {
-			await this.fileService.stat(mcpConfigUri);
-			return true;
-		} catch (error) {
-			return false;
-		}
-	}
-
-
-	private async _parseMCPConfigFile(): Promise<MCPConfigFileJSON | null> {
-		const mcpConfigUri = await this._getMCPConfigFilePath();
-		try {
-			const fileContent = await this.fileService.readFile(mcpConfigUri);
-			const contentString = fileContent.value.toString();
-			const configFileJson = JSON.parse(contentString);
-			if (!configFileJson.mcpServers) {
-				throw new Error('Missing mcpServers property');
-			}
-			return configFileJson as MCPConfigFileJSON;
-		} catch (error) {
-			const fullError = `Error parsing MCP config file: ${error}`;
-			this._setHasError(fullError)
-			return null;
-		}
-	}
-
-
-	// Handle server state changes
-	private async _refreshMCPServers(): Promise<void> {
-
-		this._setHasError(undefined)
-
-		const newConfigFileJSON = await this._parseMCPConfigFile();
-		if (!newConfigFileJSON) { console.log(`Not setting state: MCP config file not found`); return }
-		if (!newConfigFileJSON?.mcpServers) { console.log(`Not setting state: MCP config file did not have an 'mcpServers' field`); return }
-
-
-		const oldConfigFileNames = Object.keys(this.state.mcpServerOfName)
-		const newConfigFileNames = Object.keys(newConfigFileJSON.mcpServers)
-
-		const addedServerNames = newConfigFileNames.filter(serverName => !oldConfigFileNames.includes(serverName)); // in new and not in old
-		const removedServerNames = oldConfigFileNames.filter(serverName => !newConfigFileNames.includes(serverName)); // in old and not in new
-
-		// set isOn to any new servers in the config
-		const addedUserStateOfName: MCPUserStateOfName = {}
-		for (const name of addedServerNames) { addedUserStateOfName[name] = { isOn: true } }
-		await this.voidSettingsService.addMCPUserStateOfNames(addedUserStateOfName);
-
-		// delete isOn for any servers that no longer show up in the config
-		await this.voidSettingsService.removeMCPUserStateOfNames(removedServerNames);
-
-		// set all servers to loading
-		for (const serverName in newConfigFileJSON.mcpServers) {
-			this._setMCPServerState(serverName, { status: 'loading', tools: [] })
-		}
-		const updatedServerNames = Object.keys(newConfigFileJSON.mcpServers).filter(serverName => !addedServerNames.includes(serverName) && !removedServerNames.includes(serverName))
-
-		this.channel.call('refreshMCPServers', {
-			mcpConfigFileJSON: newConfigFileJSON,
-			addedServerNames,
-			removedServerNames,
-			updatedServerNames,
-			userStateOfName: this.voidSettingsService.state.mcpUserStateOfName,
-		})
-	}
-
-	stringifyResult(result: RawMCPToolCall): string {
-		let toolResultStr: string
-		if (result.event === 'text') {
-			toolResultStr = result.text
-		} else if (result.event === 'image') {
-			toolResultStr = `[Image: ${result.image.mimeType}]`
-		} else if (result.event === 'audio') {
-			toolResultStr = `[Audio content]`
-		} else if (result.event === 'resource') {
-			toolResultStr = `[Resource content]`
-		} else {
-			toolResultStr = JSON.stringify(result)
-		}
-		return toolResultStr
-	}
-
-	// toggle MCP server and update isOn in void settings
-	public async toggleServerIsOn(serverName: string, isOn: boolean): Promise<void> {
-		this._setMCPServerState(serverName, { status: 'loading', tools: [] })
-
-		await this.voidSettingsService.setMCPServerState(serverName, { isOn });
-		this.channel.call('toggleMCPServer', { serverName, isOn })
-	}
-
-
-	public async callMCPTool(toolData: MCPToolCallParams): Promise<{ result: RawMCPToolCall }> {
-		const result = await this.channel.call<RawMCPToolCall>('callTool', toolData);
-		if (result.event === 'error') {
-			throw new Error(`Error: ${result.text}`)
-		}
-		return { result };
-	}
-
-	// public getMCPToolFns(): MCPToolResultType {
-	// 	const tools = this.getMCPTools();
-	// 	const toolFns: MCPToolResultType = {};
-
-	// 	tools.forEach((tool) => {
-	// 		const name = tool.name;
-	// 		// Define the tool call function
-	// 		const toolFn = async (params: {
-	// 			serverName: string,
-	// 			toolName: string,
-	// 			args: any
-	// 		}) => {
-	// 			const { serverName, toolName, args } = params;
-	// 			const response = await this.callMCPTool({
-	// 				serverName,
-	// 				toolName,
-	// 				params: args,
-	// 			});
-	// 			return { result: response }
-	// 		};
-	// 		toolFns[name] = toolFn;
-	// 	});
-
-	// 	return toolFns
-	// }
-}
-
-registerSingleton(IMCPService, MCPService, InstantiationType.Eager);

src/vs/workbench/contrib/void/common/mcpServiceTypes.ts

@@ -1,242 +0,0 @@
-/**
- * mcp-response-types.ts
- * --------------------------------------------------
- * **Pure** TypeScript interfaces (no external imports)
- * describing the JSON-RPC response shapes for:
- *
- *   1. tools/list      -> ToolsListResponse
- *   2. prompts/list    -> PromptsListResponse
- *   3. tools/call      -> ToolCallResponse
- *
- * They are distilled directly from the official MCP
- * 2025‑03‑26 specification:
- *   • Tools list response examples
- *   • Prompts list response examples
- *   • Tool call response examples
- *
- * Use them to get full IntelliSense when working with
- * @modelcontextprotocol/inspector‑cli responses.
- */
-
-
-/* -------------------------------------------------- */
-/* Core JSON‑RPC envelope                              */
-/* -------------------------------------------------- */
-
-// export interface JsonRpcSuccess<T> {
-// 	/** JSON‑RPC version – always '2.0' */
-// 	jsonrpc: '2.0';
-// 	/** Request identifier echoed back by the server */
-// 	id: string | number | null;
-// 	/** The successful result payload */
-// 	result: T;
-// }
-
-/* -------------------------------------------------- */
-/* Utility: pagination                                 */
-/* -------------------------------------------------- */
-
-// export interface Paginated {
-// 	/** Opaque cursor for fetching the next page */
-// 	nextCursor?: string;
-// }
-
-/* -------------------------------------------------- */
-/* 1. tools/list                                       */
-/* -------------------------------------------------- */
-
-export interface MCPTool {
-	/** Unique tool identifier */
-	name: string;
-	/** Human‑readable description */
-	description?: string;
-	/** JSON schema describing expected arguments */
-	inputSchema?: Record<string, unknown>;
-	/** Free‑form annotations describing behaviour, security, etc. */
-	annotations?: Record<string, unknown>;
-}
-
-// export interface ToolsListResult extends Paginated {
-// 	tools: MCPTool[];
-// }
-
-// export type ToolsListResponse = JsonRpcSuccess<ToolsListResult>;
-
-/* -------------------------------------------------- */
-/* 2. prompts/list                                     */
-/* -------------------------------------------------- */
-
-// export interface PromptArgument {
-// 	name: string;
-// 	description?: string;
-// 	/** Whether the argument is required */
-// 	required?: boolean;
-// }
-
-// export interface Prompt {
-// 	name: string;
-// 	description?: string;
-// 	arguments?: PromptArgument[];
-// }
-
-// export interface PromptsListResult extends Paginated {
-// 	prompts: Prompt[];
-// }
-
-// export type PromptsListResponse = JsonRpcSuccess<PromptsListResult>;
-
-/* -------------------------------------------------- */
-/* 3. tools/call                                       */
-/* -------------------------------------------------- */
-
-/** Additional resource structure that can be embedded in tool results */
-// export interface Resource {
-// 	uri: string;
-// 	mimeType: string;
-// 	/** Either plain‑text or base64‑encoded binary data */
-// 	text?: string;
-// 	data?: string;
-// }
-
-/** Individual content items returned by a tool */
-// export type ToolContent =
-// 	| { type: 'text'; text: string }
-// 	| { type: 'image'; data: string; mimeType: string }
-// 	| { type: 'audio'; data: string; mimeType: string }
-// 	| { type: 'resource'; resource: Resource };
-
-// export interface ToolCallResult {
-// 	/** List of content parts (text, images, resources, etc.) */
-// 	content: ToolContent[];
-// 	/** True if the tool itself encountered a domain‑level error */
-// 	isError?: boolean;
-// }
-
-// export type ToolCallResponse = JsonRpcSuccess<ToolCallResult>;
-
-// MCP SERVER CONFIG FILE TYPES -----------------------------
-
-export interface MCPConfigFileEntryJSON {
-	// Command-based server properties
-	command?: string;
-	args?: string[];
-	env?: Record<string, string>;
-
-	// URL-based server properties
-	url?: URL;
-	headers?: Record<string, string>;
-}
-
-export interface MCPConfigFileJSON {
-	mcpServers: Record<string, MCPConfigFileEntryJSON>;
-}
-
-
-// SERVER EVENT TYPES ------------------------------------------
-
-export type MCPServer = {
-	// Command-based server properties
-	tools: MCPTool[],
-	status: 'loading' | 'success' | 'offline',
-	command?: string,
-	error?: string,
-} | {
-	tools?: undefined,
-	status: 'error',
-	command?: string,
-	error: string,
-}
-
-export interface MCPServerOfName {
-	[serverName: string]: MCPServer;
-}
-
-export type MCPServerEvent = {
-	name: string;
-	prevServer?: MCPServer;
-	newServer?: MCPServer;
-}
-export type MCPServerEventResponse = { response: MCPServerEvent }
-
-export interface MCPConfigFileParseErrorResponse {
-	response: {
-		type: 'config-file-error';
-		error: string | null;
-	}
-}
-
-
-// export type MCPServerResponse = MCPAddResponse | MCPUpdateResponse | MCPDeleteResponse | MCPLoadingResponse;
-
-// Event parameter types
-// export type MCPServerEventAddParam = { response: MCPAddResponse };
-// export type MCPServerEventUpdateParam = { response: MCPUpdateResponse };
-// export type MCPServerEventDeleteParam = { response: MCPDeleteResponse };
-// export type MCPServerEventLoadingParam = { response: MCPLoadingResponse };
-
-// Event Param union type
-// export type MCPServerEventParam = MCPServerEventAddParam | MCPServerEventUpdateParam | MCPServerEventDeleteParam | MCPServerEventLoadingParam;
-
-// TOOL CALL EVENT TYPES ------------------------------------------
-
-type MCPToolResponseType = 'text' | 'image' | 'audio' | 'resource' | 'error';
-
-type ResponseImageTypes = 'image/png' | 'image/jpeg' | 'image/gif' | 'image/webp' | 'image/svg+xml' | 'image/bmp' | 'image/tiff' | 'image/vnd.microsoft.icon';
-
-interface ImageData {
-	data: string;
-	mimeType: ResponseImageTypes;
-}
-
-interface MCPToolResponseBase {
-	toolName: string;
-	serverName?: string;
-	event: MCPToolResponseType;
-	text?: string;
-	image?: ImageData;
-}
-
-type MCPToolResponseConstraints = {
-	'text': {
-		image?: never;
-		text: string;
-	};
-	'error': {
-		image?: never;
-		text: string;
-	};
-	'image': {
-		text?: never;
-		image: ImageData;
-	};
-	'audio': {
-		text?: never;
-		image?: never;
-	};
-	'resource': {
-		text?: never;
-		image?: never;
-	}
-}
-
-type MCPToolEventResponse<T extends MCPToolResponseType> = Omit<MCPToolResponseBase, 'event' | keyof MCPToolResponseConstraints> & MCPToolResponseConstraints[T] & { event: T };
-
-// Response types
-export type MCPToolTextResponse = MCPToolEventResponse<'text'>;
-export type MCPToolErrorResponse = MCPToolEventResponse<'error'>;
-export type MCPToolImageResponse = MCPToolEventResponse<'image'>;
-export type MCPToolAudioResponse = MCPToolEventResponse<'audio'>;
-export type MCPToolResourceResponse = MCPToolEventResponse<'resource'>;
-export type RawMCPToolCall = MCPToolTextResponse | MCPToolErrorResponse | MCPToolImageResponse | MCPToolAudioResponse | MCPToolResourceResponse;
-
-export interface MCPToolCallParams {
-	serverName: string;
-	toolName: string;
-	params: Record<string, unknown>;
-}
-
-
-
-export const removeMCPToolNamePrefix = (name: string) => {
-	return name.split('_').slice(1).join('_')
-}

src/vs/workbench/contrib/void/common/metricsService.ts

@@ -14,7 +14,6 @@ import { INotificationService } from '../../../../platform/notification/common/n
 export interface IMetricsService {
 	readonly _serviceBrand: undefined;
 	capture(event: string, params: Record<string, any>): void;
-	setOptOut(val: boolean): void;
 	getDebuggingProperties(): Promise<object>;
 }
 
@@ -39,11 +38,6 @@ export class MetricsService implements IMetricsService {
 		this.metricsService.capture(...params);
 	}
 
-	setOptOut(...params: Parameters<IMetricsService['setOptOut']>) {
-		this.metricsService.setOptOut(...params);
-	}
-
-
 	// anything transmitted over a channel must be async even if it looks like it doesn't have to be
 	async getDebuggingProperties(): Promise<object> {
 		return this.metricsService.getDebuggingProperties()

src/vs/workbench/contrib/void/common/modelCapabilities.ts

@@ -60,12 +60,6 @@ export const defaultProviderSettings = {
 		apiKey: '',
 		azureApiVersion: '2024-05-01-preview',
 	},
-	awsBedrock: {
-		apiKey: '',
-		region: 'us-east-1', // add region setting
-		endpoint: '', // optionally allow overriding default
-	},
-
 } as const
 
 
@@ -84,19 +78,14 @@ export const defaultModelsOfProvider = {
 		// 'gpt-4o-mini',
 	],
 	anthropic: [ // https://docs.anthropic.com/en/docs/about-claude/models
-		'claude-opus-4-0',
-		'claude-sonnet-4-0',
 		'claude-3-7-sonnet-latest',
 		'claude-3-5-sonnet-latest',
 		'claude-3-5-haiku-latest',
 		'claude-3-opus-latest',
 	],
 	xAI: [ // https://docs.x.ai/docs/models?cluster=us-east-1
-		'grok-2',
-		'grok-3',
-		'grok-3-mini',
-		'grok-3-fast',
-		'grok-3-mini-fast'
+		'grok-2-latest',
+		'grok-3-latest',
 	],
 	gemini: [ // https://ai.google.dev/gemini-api/docs/models/gemini
 		'gemini-2.5-pro-exp-03-25',
@@ -117,14 +106,11 @@ export const defaultModelsOfProvider = {
 
 	openRouter: [ // https://openrouter.ai/models
 		// 'anthropic/claude-3.7-sonnet:thinking',
-		'anthropic/claude-opus-4',
-		'anthropic/claude-sonnet-4',
 		'qwen/qwen3-235b-a22b',
 		'anthropic/claude-3.7-sonnet',
 		'anthropic/claude-3.5-sonnet',
 		'deepseek/deepseek-r1',
 		'deepseek/deepseek-r1-zero:free',
-		'mistralai/devstral-small:free'
 		// 'openrouter/quasar-alpha',
 		// 'google/gemini-2.5-pro-preview-03-25',
 		// 'mistralai/codestral-2501',
@@ -142,7 +128,6 @@ export const defaultModelsOfProvider = {
 	],
 	mistral: [ // https://docs.mistral.ai/getting-started/models/models_overview/
 		'codestral-latest',
-		'devstral-small-latest',
 		'mistral-large-latest',
 		'mistral-medium-latest',
 		'ministral-3b-latest',
@@ -151,7 +136,6 @@ export const defaultModelsOfProvider = {
 	openAICompatible: [], // fallback
 	googleVertex: [],
 	microsoftAzure: [],
-	awsBedrock: [],
 	liteLLM: [],
 
 
@@ -204,20 +188,14 @@ export type VoidStaticModelInfo = { // not stateful
 // if you change the above type, remember to update the Settings link
 
 
-
-export const modelOverrideKeys = [
-	'contextWindow',
-	'reservedOutputTokenSpace',
-	'supportsSystemMessage',
-	'specialToolFormat',
-	'supportsFIM',
-	'reasoningCapabilities',
-	'additionalOpenAIPayload'
-] as const
-
-export type ModelOverrides = Pick<
-	VoidStaticModelInfo,
-	(typeof modelOverrideKeys)[number]
+export type ModelOverrides = Pick<VoidStaticModelInfo,
+	| 'contextWindow'
+	| 'reservedOutputTokenSpace'
+	| 'specialToolFormat'
+	| 'supportsSystemMessage'
+	| 'supportsFIM'
+	| 'reasoningCapabilities'
+	| 'additionalOpenAIPayload'
 >
 
 
@@ -279,12 +257,6 @@ const openSourceModelOptions_assumingOAICompat = {
 		reasoningCapabilities: false,
 		contextWindow: 32_000, reservedOutputTokenSpace: 4_096,
 	},
-	'devstral': {
-		supportsFIM: false,
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: false,
-		contextWindow: 131_000, reservedOutputTokenSpace: 8_192,
-	},
 	'openhands-lm-32b': { // https://www.all-hands.dev/blog/introducing-openhands-lm-32b----a-strong-open-coding-agent-model
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
@@ -399,28 +371,22 @@ const extensiveModelOptionsFallback: VoidStaticProviderInfo['modelOptionsFallbac
 		: VoidStaticModelInfo & { modelName: string, recognizedModelName: string } => {
 
 		const opts = obj[recognizedModelName]
-		const supportsSystemMessage = opts.supportsSystemMessage === 'separated'
-			? 'system-role'
-			: opts.supportsSystemMessage
-
 		return {
 			recognizedModelName,
 			modelName,
 			...opts,
-			supportsSystemMessage: supportsSystemMessage,
+			supportsSystemMessage: opts.supportsSystemMessage ? 'system-role' : false,
 			cost: { input: 0, output: 0 },
 			downloadable: false,
 			...fallbackKnownValues
-		};
+		}
 	}
-
 	if (lower.includes('gemini') && (lower.includes('2.5') || lower.includes('2-5'))) return toFallback(geminiModelOptions, 'gemini-2.5-pro-exp-03-25')
 
 	if (lower.includes('claude-3-5') || lower.includes('claude-3.5')) return toFallback(anthropicModelOptions, 'claude-3-5-sonnet-20241022')
 	if (lower.includes('claude')) return toFallback(anthropicModelOptions, 'claude-3-7-sonnet-20250219')
 
-	if (lower.includes('grok2') || lower.includes('grok2')) return toFallback(xAIModelOptions, 'grok-2')
-	if (lower.includes('grok')) return toFallback(xAIModelOptions, 'grok-3')
+	if (lower.includes('grok')) return toFallback(xAIModelOptions, 'grok-2')
 
 	if (lower.includes('deepseek-r1') || lower.includes('deepseek-reasoner')) return toFallback(openSourceModelOptions_assumingOAICompat, 'deepseekR1')
 	if (lower.includes('deepseek') && lower.includes('v2')) return toFallback(openSourceModelOptions_assumingOAICompat, 'deepseekCoderV2')
@@ -440,7 +406,6 @@ const extensiveModelOptionsFallback: VoidStaticProviderInfo['modelOptionsFallbac
 	if (lower.includes('qwq')) { return toFallback(openSourceModelOptions_assumingOAICompat, 'qwq') }
 	if (lower.includes('phi4')) return toFallback(openSourceModelOptions_assumingOAICompat, 'phi4')
 	if (lower.includes('codestral')) return toFallback(openSourceModelOptions_assumingOAICompat, 'codestral')
-	if (lower.includes('devstral')) return toFallback(openSourceModelOptions_assumingOAICompat, 'devstral')
 
 	if (lower.includes('gemma')) return toFallback(openSourceModelOptions_assumingOAICompat, 'gemma')
 
@@ -493,40 +458,6 @@ const anthropicModelOptions = {
 			reasoningSlider: { type: 'budget_slider', min: 1024, max: 8192, default: 1024 }, // they recommend batching if max > 32_000. we cap at 8192 because above is typically not necessary (often even buggy)
 		},
 
-	},
-	'claude-opus-4-20250514': {
-		contextWindow: 200_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 15.00, cache_read: 1.50, cache_write: 18.75, output: 30.00 },
-		downloadable: false,
-		supportsFIM: false,
-		specialToolFormat: 'anthropic-style',
-		supportsSystemMessage: 'separated',
-		reasoningCapabilities: {
-			supportsReasoning: true,
-			canTurnOffReasoning: true,
-			canIOReasoning: true,
-			reasoningReservedOutputTokenSpace: 8192, // can bump it to 128_000 with beta mode output-128k-2025-02-19
-			reasoningSlider: { type: 'budget_slider', min: 1024, max: 8192, default: 1024 }, // they recommend batching if max > 32_000. we cap at 8192 because above is typically not necessary (often even buggy)
-		},
-
-	},
-	'claude-sonnet-4-20250514': {
-		contextWindow: 200_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 3.00, cache_read: 0.30, cache_write: 3.75, output: 6.00 },
-		downloadable: false,
-		supportsFIM: false,
-		specialToolFormat: 'anthropic-style',
-		supportsSystemMessage: 'separated',
-		reasoningCapabilities: {
-			supportsReasoning: true,
-			canTurnOffReasoning: true,
-			canIOReasoning: true,
-			reasoningReservedOutputTokenSpace: 8192, // can bump it to 128_000 with beta mode output-128k-2025-02-19
-			reasoningSlider: { type: 'budget_slider', min: 1024, max: 8192, default: 1024 }, // they recommend batching if max > 32_000. we cap at 8192 because above is typically not necessary (often even buggy)
-		},
-
 	},
 	'claude-3-5-sonnet-20241022': {
 		contextWindow: 200_000,
@@ -586,10 +517,6 @@ const anthropicSettings: VoidStaticProviderInfo = {
 	modelOptionsFallback: (modelName) => {
 		const lower = modelName.toLowerCase()
 		let fallbackName: keyof typeof anthropicModelOptions | null = null
-		if (lower.includes('claude-4-opus') || lower.includes('claude-opus-4')) fallbackName = 'claude-opus-4-20250514'
-		if (lower.includes('claude-4-sonnet') || lower.includes('claude-sonnet-4')) fallbackName = 'claude-sonnet-4-20250514'
-
-
 		if (lower.includes('claude-3-7-sonnet')) fallbackName = 'claude-3-7-sonnet-20250219'
 		if (lower.includes('claude-3-5-sonnet')) fallbackName = 'claude-3-5-sonnet-20241022'
 		if (lower.includes('claude-3-5-haiku')) fallbackName = 'claude-3-5-haiku-20241022'
@@ -703,16 +630,6 @@ const openAIModelOptions = { // https://platform.openai.com/docs/pricing
 } as const satisfies { [s: string]: VoidStaticModelInfo }
 
 
-// https://platform.openai.com/docs/guides/reasoning?api-mode=chat
-const openAICompatIncludeInPayloadReasoning = (reasoningInfo: SendableReasoningInfo) => {
-	if (!reasoningInfo?.isReasoningEnabled) return null
-	if (reasoningInfo.type === 'effort_slider_value') {
-		return { reasoning_effort: reasoningInfo.reasoningEffort }
-	}
-	return null
-
-}
-
 const openAISettings: VoidStaticProviderInfo = {
 	modelOptions: openAIModelOptions,
 	modelOptionsFallback: (modelName) => {
@@ -725,7 +642,17 @@ const openAISettings: VoidStaticProviderInfo = {
 		return null
 	},
 	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
+		input: {
+			// https://platform.openai.com/docs/guides/reasoning?api-mode=chat
+			includeInPayload: (reasoningInfo) => {
+				if (!reasoningInfo?.isReasoningEnabled) return null
+
+				if (reasoningInfo.type === 'effort_slider_value') {
+					return { reasoning_effort: reasoningInfo.reasoningEffort }
+				}
+				return null
+			}
+		},
 	},
 }
 
@@ -740,7 +667,6 @@ const xAIModelOptions = {
 		downloadable: false,
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
-		specialToolFormat: 'openai-style',
 		reasoningCapabilities: false,
 	},
 	'grok-3': {
@@ -750,7 +676,6 @@ const xAIModelOptions = {
 		downloadable: false,
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
-		specialToolFormat: 'openai-style',
 		reasoningCapabilities: false,
 	},
 	'grok-3-fast': {
@@ -760,7 +685,6 @@ const xAIModelOptions = {
 		downloadable: false,
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
-		specialToolFormat: 'openai-style',
 		reasoningCapabilities: false,
 	},
 	// only mini supports thinking
@@ -771,7 +695,6 @@ const xAIModelOptions = {
 		downloadable: false,
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
-		specialToolFormat: 'openai-style',
 		reasoningCapabilities: { supportsReasoning: true, canTurnOffReasoning: false, canIOReasoning: false, reasoningSlider: { type: 'effort_slider', values: ['low', 'high'], default: 'low' } },
 	},
 	'grok-3-mini-fast': {
@@ -781,7 +704,6 @@ const xAIModelOptions = {
 		downloadable: false,
 		supportsFIM: false,
 		supportsSystemMessage: 'system-role',
-		specialToolFormat: 'openai-style',
 		reasoningCapabilities: { supportsReasoning: true, canTurnOffReasoning: false, canIOReasoning: false, reasoningSlider: { type: 'effort_slider', values: ['low', 'high'], default: 'low' } },
 	},
 } as const satisfies { [s: string]: VoidStaticModelInfo }
@@ -792,15 +714,11 @@ const xAISettings: VoidStaticProviderInfo = {
 		const lower = modelName.toLowerCase()
 		let fallbackName: keyof typeof xAIModelOptions | null = null
 		if (lower.includes('grok-2')) fallbackName = 'grok-2'
-		if (lower.includes('grok-3')) fallbackName = 'grok-3'
-		if (lower.includes('grok')) fallbackName = 'grok-3'
 		if (fallbackName) return { modelName: fallbackName, recognizedModelName: fallbackName, ...xAIModelOptions[fallbackName] }
 		return null
 	},
 	// same implementation as openai
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-	},
+	providerReasoningIOSettings: openAISettings.providerReasoningIOSettings,
 }
 
 
@@ -919,7 +837,7 @@ const geminiModelOptions = { // https://ai.google.dev/gemini-api/docs/pricing
 
 const geminiSettings: VoidStaticProviderInfo = {
 	modelOptions: geminiModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
+	modelOptionsFallback: (modelName) => { return null }
 }
 
 
@@ -945,12 +863,11 @@ const deepseekModelOptions = {
 
 const deepseekSettings: VoidStaticProviderInfo = {
 	modelOptions: deepseekModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
 	providerReasoningIOSettings: {
 		// reasoning: OAICompat +  response.choices[0].delta.reasoning_content // https://api-docs.deepseek.com/guides/reasoning_model
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
 		output: { nameOfFieldInDelta: 'reasoning_content' },
 	},
+	modelOptionsFallback: (modelName) => { return null }
 }
 
 
@@ -985,33 +902,6 @@ const mistralModelOptions = { // https://mistral.ai/products/la-plateforme#prici
 		supportsSystemMessage: 'system-role',
 		reasoningCapabilities: false,
 	},
-	'magistral-medium-latest': {
-		contextWindow: 256_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 0.30, output: 0.90 }, // TODO: check this
-		supportsFIM: true,
-		downloadable: { sizeGb: 13 },
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: { supportsReasoning: true, canIOReasoning: true, canTurnOffReasoning: false, openSourceThinkTags: ['<think>', '</think>'] },
-	},
-	'magistral-small-latest': {
-		contextWindow: 40_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 0.30, output: 0.90 }, // TODO: check this
-		supportsFIM: true,
-		downloadable: { sizeGb: 13 },
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: { supportsReasoning: true, canIOReasoning: true, canTurnOffReasoning: false, openSourceThinkTags: ['<think>', '</think>'] },
-	},
-	'devstral-small-latest': { //https://openrouter.ai/mistralai/devstral-small:free
-		contextWindow: 131_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 0, output: 0 },
-		supportsFIM: false,
-		downloadable: { sizeGb: 14 }, //https://ollama.com/library/devstral
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: false,
-	},
 	'ministral-8b-latest': { // ollama 'mistral'
 		contextWindow: 131_000,
 		reservedOutputTokenSpace: 4_096,
@@ -1035,9 +925,6 @@ const mistralModelOptions = { // https://mistral.ai/products/la-plateforme#prici
 const mistralSettings: VoidStaticProviderInfo = {
 	modelOptions: mistralModelOptions,
 	modelOptionsFallback: (modelName) => { return null },
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-	},
 }
 
 
@@ -1081,13 +968,11 @@ const groqModelOptions = { // https://console.groq.com/docs/models, https://groq
 	},
 } as const satisfies { [s: string]: VoidStaticModelInfo }
 const groqSettings: VoidStaticProviderInfo = {
-	modelOptions: groqModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
 	providerReasoningIOSettings: {
-		// Must be set to either parsed or hidden when using tool calling https://console.groq.com/docs/reasoning
 		input: {
 			includeInPayload: (reasoningInfo) => {
 				if (!reasoningInfo?.isReasoningEnabled) return null
+
 				if (reasoningInfo.type === 'budget_slider_value') {
 					return { reasoning_format: 'parsed' }
 				}
@@ -1095,7 +980,9 @@ const groqSettings: VoidStaticProviderInfo = {
 			}
 		},
 		output: { nameOfFieldInDelta: 'reasoning' },
-	},
+	}, // Must be set to either parsed or hidden when using tool calling https://console.groq.com/docs/reasoning
+	modelOptions: groqModelOptions,
+	modelOptionsFallback: (modelName) => { return null }
 }
 
 
@@ -1104,10 +991,7 @@ const googleVertexModelOptions = {
 } as const satisfies Record<string, VoidStaticModelInfo>
 const googleVertexSettings: VoidStaticProviderInfo = {
 	modelOptions: googleVertexModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-	},
+	modelOptionsFallback: (modelName) => { return null }
 }
 
 // ---------------- MICROSOFT AZURE ----------------
@@ -1115,22 +999,7 @@ const microsoftAzureModelOptions = {
 } as const satisfies Record<string, VoidStaticModelInfo>
 const microsoftAzureSettings: VoidStaticProviderInfo = {
 	modelOptions: microsoftAzureModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-	},
-}
-
-// ---------------- AWS BEDROCK ----------------
-const awsBedrockModelOptions = {
-} as const satisfies Record<string, VoidStaticModelInfo>
-
-const awsBedrockSettings: VoidStaticProviderInfo = {
-	modelOptions: awsBedrockModelOptions,
-	modelOptionsFallback: (modelName) => { return null },
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-	},
+	modelOptionsFallback: (modelName) => { return null }
 }
 
 
@@ -1199,67 +1068,42 @@ const ollamaModelOptions = {
 		supportsSystemMessage: 'system-role',
 		reasoningCapabilities: { supportsReasoning: true, canIOReasoning: false, canTurnOffReasoning: false, openSourceThinkTags: ['<think>', '</think>'] },
 	},
-	'devstral:latest': {
-		contextWindow: 131_000,
-		reservedOutputTokenSpace: 8_192,
-		cost: { input: 0, output: 0 },
-		downloadable: { sizeGb: 14 },
-		supportsFIM: false,
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: false,
-	},
 
 } as const satisfies Record<string, VoidStaticModelInfo>
 
-export const ollamaRecommendedModels = ['qwen2.5-coder:1.5b', 'llama3.1', 'qwq', 'deepseek-r1', 'devstral:latest'] as const satisfies (keyof typeof ollamaModelOptions)[]
+export const ollamaRecommendedModels = ['qwen2.5-coder:1.5b', 'llama3.1', 'qwq', 'deepseek-r1'] as const satisfies (keyof typeof ollamaModelOptions)[]
 
 
 const vLLMSettings: VoidStaticProviderInfo = {
+	// reasoning: OAICompat + response.choices[0].delta.reasoning_content // https://docs.vllm.ai/en/stable/features/reasoning_outputs.html#streaming-chat-completions
+	providerReasoningIOSettings: { output: { nameOfFieldInDelta: 'reasoning_content' }, },
 	modelOptionsFallback: (modelName) => extensiveModelOptionsFallback(modelName, { downloadable: { sizeGb: 'not-known' } }),
-	modelOptions: {},
-	providerReasoningIOSettings: {
-		// reasoning: OAICompat + response.choices[0].delta.reasoning_content // https://docs.vllm.ai/en/stable/features/reasoning_outputs.html#streaming-chat-completions
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-		output: { nameOfFieldInDelta: 'reasoning_content' },
-	},
+	modelOptions: {}, // TODO
 }
 
 const lmStudioSettings: VoidStaticProviderInfo = {
+	providerReasoningIOSettings: { output: { needsManualParse: true }, },
 	modelOptionsFallback: (modelName) => extensiveModelOptionsFallback(modelName, { downloadable: { sizeGb: 'not-known' }, contextWindow: 4_096 }),
-	modelOptions: {},
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-		output: { needsManualParse: true },
-	},
+	modelOptions: {}, // TODO
 }
 
 const ollamaSettings: VoidStaticProviderInfo = {
+	// reasoning: we need to filter out reasoning <think> tags manually
+	providerReasoningIOSettings: { output: { needsManualParse: true }, },
 	modelOptionsFallback: (modelName) => extensiveModelOptionsFallback(modelName, { downloadable: { sizeGb: 'not-known' } }),
 	modelOptions: ollamaModelOptions,
-	providerReasoningIOSettings: {
-		// reasoning: we need to filter out reasoning <think> tags manually
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-		output: { needsManualParse: true },
-	},
 }
 
 const openaiCompatible: VoidStaticProviderInfo = {
+	// reasoning: we have no idea what endpoint they used, so we can't consistently parse out reasoning
 	modelOptionsFallback: (modelName) => extensiveModelOptionsFallback(modelName),
 	modelOptions: {},
-	providerReasoningIOSettings: {
-		// reasoning: we have no idea what endpoint they used, so we can't consistently parse out reasoning
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-		output: { nameOfFieldInDelta: 'reasoning_content' },
-	},
 }
 
 const liteLLMSettings: VoidStaticProviderInfo = { // https://docs.litellm.ai/docs/reasoning_content
+	providerReasoningIOSettings: { output: { nameOfFieldInDelta: 'reasoning_content' } },
 	modelOptionsFallback: (modelName) => extensiveModelOptionsFallback(modelName, { downloadable: { sizeGb: 'not-known' } }),
-	modelOptions: {},
-	providerReasoningIOSettings: {
-		input: { includeInPayload: openAICompatIncludeInPayloadReasoning },
-		output: { nameOfFieldInDelta: 'reasoning_content' },
-	},
+	modelOptions: {}, // TODO
 }
 
 
@@ -1326,24 +1170,6 @@ const openRouterModelOptions_assumingOpenAICompat = {
 		cost: { input: 0.8, output: 2.4 },
 		downloadable: false,
 	},
-	'anthropic/claude-opus-4': {
-		contextWindow: 200_000,
-		reservedOutputTokenSpace: null,
-		cost: { input: 15.00, output: 75.00 },
-		downloadable: false,
-		supportsFIM: false,
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: false,
-	},
-	'anthropic/claude-sonnet-4': {
-		contextWindow: 200_000,
-		reservedOutputTokenSpace: null,
-		cost: { input: 15.00, output: 75.00 },
-		downloadable: false,
-		supportsFIM: false,
-		supportsSystemMessage: 'system-role',
-		reasoningCapabilities: false,
-	},
 	'anthropic/claude-3.7-sonnet:thinking': {
 		contextWindow: 200_000,
 		reservedOutputTokenSpace: null,
@@ -1385,14 +1211,6 @@ const openRouterModelOptions_assumingOpenAICompat = {
 		downloadable: false,
 		reasoningCapabilities: false,
 	},
-	'mistralai/devstral-small:free': {
-		...openSourceModelOptions_assumingOAICompat.devstral,
-		contextWindow: 130_000,
-		reservedOutputTokenSpace: null,
-		cost: { input: 0, output: 0 },
-		downloadable: false,
-		reasoningCapabilities: false,
-	},
 	'qwen/qwen-2.5-coder-32b-instruct': {
 		...openSourceModelOptions_assumingOAICompat['qwen2.5coder'],
 		contextWindow: 33_000,
@@ -1410,17 +1228,8 @@ const openRouterModelOptions_assumingOpenAICompat = {
 } as const satisfies { [s: string]: VoidStaticModelInfo }
 
 const openRouterSettings: VoidStaticProviderInfo = {
-	modelOptions: openRouterModelOptions_assumingOpenAICompat,
-	modelOptionsFallback: (modelName) => {
-		const res = extensiveModelOptionsFallback(modelName)
-		// openRouter does not support gemini-style, use openai-style instead
-		if (res?.specialToolFormat === 'gemini-style') {
-			res.specialToolFormat = 'openai-style'
-		}
-		return res
-	},
+	// reasoning: OAICompat + response.choices[0].delta.reasoning : payload should have {include_reasoning: true} https://openrouter.ai/announcements/reasoning-tokens-for-thinking-models
 	providerReasoningIOSettings: {
-		// reasoning: OAICompat + response.choices[0].delta.reasoning : payload should have {include_reasoning: true} https://openrouter.ai/announcements/reasoning-tokens-for-thinking-models
 		input: {
 			// https://openrouter.ai/docs/use-cases/reasoning-tokens
 			includeInPayload: (reasoningInfo) => {
@@ -1444,6 +1253,16 @@ const openRouterSettings: VoidStaticProviderInfo = {
 		},
 		output: { nameOfFieldInDelta: 'reasoning' },
 	},
+	modelOptions: openRouterModelOptions_assumingOpenAICompat,
+	// TODO!!! send a query to openrouter to get the price, etc.
+	modelOptionsFallback: (modelName) => {
+		const res = extensiveModelOptionsFallback(modelName)
+		// openRouter does not support gemini-style, use openai-style instead
+		if (res?.specialToolFormat === 'gemini-style') {
+			res.specialToolFormat = 'openai-style'
+		}
+		return res
+	},
 }
 
 
@@ -1473,7 +1292,6 @@ const modelSettingsOfProvider: { [providerName in ProviderName]: VoidStaticProvi
 
 	googleVertex: googleVertexSettings,
 	microsoftAzure: microsoftAzureSettings,
-	awsBedrock: awsBedrockSettings,
 } as const
 
 

src/vs/workbench/contrib/void/common/prompt/prompts.ts

@@ -9,7 +9,7 @@ import { IDirectoryStrService } from '../directoryStrService.js';
 import { StagingSelectionItem } from '../chatThreadServiceTypes.js';
 import { os } from '../helpers/systemInfo.js';
 import { RawToolParamsObj } from '../sendLLMMessageTypes.js';
-import { approvalTypeOfBuiltinToolName, BuiltinToolCallParams, BuiltinToolName, BuiltinToolResultType, ToolName } from '../toolsServiceTypes.js';
+import { approvalTypeOfToolName, ToolCallParams, ToolResultType } from '../toolsServiceTypes.js';
 import { ChatMode } from '../voidSettingsTypes.js';
 
 // Triple backtick wrapper used throughout the prompts for code blocks
@@ -147,8 +147,6 @@ export type InternalToolInfo = {
 	params: {
 		[paramName: string]: { description: string }
 	},
-	// Only if the tool is from an MCP server
-	mcpServerName?: string,
 }
 
 
@@ -183,197 +181,191 @@ export type SnakeCaseKeys<T extends Record<string, any>> = {
 
 
 
-export const builtinTools: {
-	[T in keyof BuiltinToolCallParams]: {
-		name: string;
-		description: string;
-		// more params can be generated than exist here, but these params must be a subset of them
-		params: Partial<{ [paramName in keyof SnakeCaseKeys<BuiltinToolCallParams[T]>]: { description: string } }>
+// export const voidTools = {
+export const voidTools
+	: {
+		[T in keyof ToolCallParams]: {
+			name: string;
+			description: string;
+			// more params can be generated than exist here, but these params must be a subset of them
+			params: Partial<{ [paramName in keyof SnakeCaseKeys<ToolCallParams[T]>]: { description: string } }>
+		}
 	}
-} = {
-	// --- context-gathering (read/search/list) ---
+	= {
+		// --- context-gathering (read/search/list) ---
 
-	read_file: {
-		name: 'read_file',
-		description: `Returns full contents of a given file.`,
-		params: {
-			...uriParam('file'),
-			start_line: { description: 'Optional. Do NOT fill this field in unless you were specifically given exact line numbers to search. Defaults to the beginning of the file.' },
-			end_line: { description: 'Optional. Do NOT fill this field in unless you were specifically given exact line numbers to search. Defaults to the end of the file.' },
-			...paginationParam,
+		read_file: {
+			name: 'read_file',
+			description: `Returns full contents of a given file.`,
+			params: {
+				...uriParam('file'),
+				start_line: { description: 'Optional. Do NOT fill this field in unless you were specifically given exact line numbers to search. Defaults to the beginning of the file.' },
+				end_line: { description: 'Optional. Do NOT fill this field in unless you were specifically given exact line numbers to search. Defaults to the end of the file.' },
+				...paginationParam,
+			},
 		},
-	},
 
-	ls_dir: {
-		name: 'ls_dir',
-		description: `Lists all files and folders in the given URI.`,
-		params: {
-			uri: { description: `Optional. The FULL path to the ${'folder'}. Leave this as empty or "" to search all folders.` },
-			...paginationParam,
+		ls_dir: {
+			name: 'ls_dir',
+			description: `Lists all files and folders in the given URI.`,
+			params: {
+				uri: { description: `Optional. The FULL path to the ${'folder'}. Leave this as empty or "" to search all folders.` },
+				...paginationParam,
+			},
 		},
-	},
 
-	get_dir_tree: {
-		name: 'get_dir_tree',
-		description: `This is a very effective way to learn about the user's codebase. Returns a tree diagram of all the files and folders in the given folder. `,
-		params: {
-			...uriParam('folder')
+		get_dir_tree: {
+			name: 'get_dir_tree',
+			description: `This is a very effective way to learn about the user's codebase. Returns a tree diagram of all the files and folders in the given folder. `,
+			params: {
+				...uriParam('folder')
+			}
+		},
+
+		// pathname_search: {
+		// 	name: 'pathname_search',
+		// 	description: `Returns all pathnames that match a given \`find\`-style query over the entire workspace. ONLY searches file names. ONLY searches the current workspace. You should use this when looking for a file with a specific name or path. ${paginationHelper.desc}`,
+
+		search_pathnames_only: {
+			name: 'search_pathnames_only',
+			description: `Returns all pathnames that match a given query (searches ONLY file names). You should use this when looking for a file with a specific name or path.`,
+			params: {
+				query: { description: `Your query for the search.` },
+				include_pattern: { description: 'Optional. Only fill this in if you need to limit your search because there were too many results.' },
+				...paginationParam,
+			},
+		},
+
+
+
+		search_for_files: {
+			name: 'search_for_files',
+			description: `Returns a list of file names whose content matches the given query. The query can be any substring or regex.`,
+			params: {
+				query: { description: `Your query for the search.` },
+				search_in_folder: { description: 'Optional. Leave as blank by default. ONLY fill this in if your previous search with the same query was truncated. Searches descendants of this folder only.' },
+				is_regex: { description: 'Optional. Default is false. Whether the query is a regex.' },
+				...paginationParam,
+			},
+		},
+
+		// add new search_in_file tool
+		search_in_file: {
+			name: 'search_in_file',
+			description: `Returns an array of all the start line numbers where the content appears in the file.`,
+			params: {
+				...uriParam('file'),
+				query: { description: 'The string or regex to search for in the file.' },
+				is_regex: { description: 'Optional. Default is false. Whether the query is a regex.' }
+			}
+		},
+
+		read_lint_errors: {
+			name: 'read_lint_errors',
+			description: `Use this tool to view all the lint errors on a file.`,
+			params: {
+				...uriParam('file'),
+			},
+		},
+
+		// --- editing (create/delete) ---
+
+		create_file_or_folder: {
+			name: 'create_file_or_folder',
+			description: `Create a file or folder at the given path. To create a folder, the path MUST end with a trailing slash.`,
+			params: {
+				...uriParam('file or folder'),
+			},
+		},
+
+		delete_file_or_folder: {
+			name: 'delete_file_or_folder',
+			description: `Delete a file or folder at the given path.`,
+			params: {
+				...uriParam('file or folder'),
+				is_recursive: { description: 'Optional. Return true to delete recursively.' }
+			},
+		},
+
+		edit_file: {
+			name: 'edit_file',
+			description: `Edit the contents of a file. You must provide the file's URI as well as a SINGLE string of SEARCH/REPLACE block(s) that will be used to apply the edit.`,
+			params: {
+				...uriParam('file'),
+				search_replace_blocks: { description: replaceTool_description }
+			},
+		},
+
+		rewrite_file: {
+			name: 'rewrite_file',
+			description: `Edits a file, deleting all the old contents and replacing them with your new contents. Use this tool if you want to edit a file you just created.`,
+			params: {
+				...uriParam('file'),
+				new_content: { description: `The new contents of the file. Must be a string.` }
+			},
+		},
+		run_command: {
+			name: 'run_command',
+			description: `Runs a terminal command and waits for the result (times out after ${MAX_TERMINAL_INACTIVE_TIME}s of inactivity). ${terminalDescHelper}`,
+			params: {
+				command: { description: 'The terminal command to run.' },
+				cwd: { description: cwdHelper },
+			},
+		},
+
+		run_persistent_command: {
+			name: 'run_persistent_command',
+			description: `Runs a terminal command in the persistent terminal that you created with open_persistent_terminal (results after ${MAX_TERMINAL_BG_COMMAND_TIME} are returned, and command continues running in background). ${terminalDescHelper}`,
+			params: {
+				command: { description: 'The terminal command to run.' },
+				persistent_terminal_id: { description: 'The ID of the terminal created using open_persistent_terminal.' },
+			},
+		},
+
+
+
+		open_persistent_terminal: {
+			name: 'open_persistent_terminal',
+			description: `Use this tool when you want to run a terminal command indefinitely, like a dev server (eg \`npm run dev\`), a background listener, etc. Opens a new terminal in the user's environment which will not awaited for or killed.`,
+			params: {
+				cwd: { description: cwdHelper },
+			}
+		},
+
+
+		kill_persistent_terminal: {
+			name: 'kill_persistent_terminal',
+			description: `Interrupts and closes a persistent terminal that you opened with open_persistent_terminal.`,
+			params: { persistent_terminal_id: { description: `The ID of the persistent terminal.` } }
 		}
-	},
-
-	// pathname_search: {
-	// 	name: 'pathname_search',
-	// 	description: `Returns all pathnames that match a given \`find\`-style query over the entire workspace. ONLY searches file names. ONLY searches the current workspace. You should use this when looking for a file with a specific name or path. ${paginationHelper.desc}`,
-
-	search_pathnames_only: {
-		name: 'search_pathnames_only',
-		description: `Returns all pathnames that match a given query (searches ONLY file names). You should use this when looking for a file with a specific name or path.`,
-		params: {
-			query: { description: `Your query for the search.` },
-			include_pattern: { description: 'Optional. Only fill this in if you need to limit your search because there were too many results.' },
-			...paginationParam,
-		},
-	},
 
 
+		// go_to_definition
+		// go_to_usages
 
-	search_for_files: {
-		name: 'search_for_files',
-		description: `Returns a list of file names whose content matches the given query. The query can be any substring or regex.`,
-		params: {
-			query: { description: `Your query for the search.` },
-			search_in_folder: { description: 'Optional. Leave as blank by default. ONLY fill this in if your previous search with the same query was truncated. Searches descendants of this folder only.' },
-			is_regex: { description: 'Optional. Default is false. Whether the query is a regex.' },
-			...paginationParam,
-		},
-	},
-
-	// add new search_in_file tool
-	search_in_file: {
-		name: 'search_in_file',
-		description: `Returns an array of all the start line numbers where the content appears in the file.`,
-		params: {
-			...uriParam('file'),
-			query: { description: 'The string or regex to search for in the file.' },
-			is_regex: { description: 'Optional. Default is false. Whether the query is a regex.' }
-		}
-	},
-
-	read_lint_errors: {
-		name: 'read_lint_errors',
-		description: `Use this tool to view all the lint errors on a file.`,
-		params: {
-			...uriParam('file'),
-		},
-	},
-
-	// --- editing (create/delete) ---
-
-	create_file_or_folder: {
-		name: 'create_file_or_folder',
-		description: `Create a file or folder at the given path. To create a folder, the path MUST end with a trailing slash.`,
-		params: {
-			...uriParam('file or folder'),
-		},
-	},
-
-	delete_file_or_folder: {
-		name: 'delete_file_or_folder',
-		description: `Delete a file or folder at the given path.`,
-		params: {
-			...uriParam('file or folder'),
-			is_recursive: { description: 'Optional. Return true to delete recursively.' }
-		},
-	},
-
-	edit_file: {
-		name: 'edit_file',
-		description: `Edit the contents of a file. You must provide the file's URI as well as a SINGLE string of SEARCH/REPLACE block(s) that will be used to apply the edit.`,
-		params: {
-			...uriParam('file'),
-			search_replace_blocks: { description: replaceTool_description }
-		},
-	},
-
-	rewrite_file: {
-		name: 'rewrite_file',
-		description: `Edits a file, deleting all the old contents and replacing them with your new contents. Use this tool if you want to edit a file you just created.`,
-		params: {
-			...uriParam('file'),
-			new_content: { description: `The new contents of the file. Must be a string.` }
-		},
-	},
-	run_command: {
-		name: 'run_command',
-		description: `Runs a terminal command and waits for the result (times out after ${MAX_TERMINAL_INACTIVE_TIME}s of inactivity). ${terminalDescHelper}`,
-		params: {
-			command: { description: 'The terminal command to run.' },
-			cwd: { description: cwdHelper },
-		},
-	},
-
-	run_persistent_command: {
-		name: 'run_persistent_command',
-		description: `Runs a terminal command in the persistent terminal that you created with open_persistent_terminal (results after ${MAX_TERMINAL_BG_COMMAND_TIME} are returned, and command continues running in background). ${terminalDescHelper}`,
-		params: {
-			command: { description: 'The terminal command to run.' },
-			persistent_terminal_id: { description: 'The ID of the terminal created using open_persistent_terminal.' },
-		},
-	},
+	} satisfies { [T in keyof ToolResultType]: InternalToolInfo }
 
 
+export type ToolName = keyof ToolResultType
+export const toolNames = Object.keys(voidTools) as ToolName[]
 
-	open_persistent_terminal: {
-		name: 'open_persistent_terminal',
-		description: `Use this tool when you want to run a terminal command indefinitely, like a dev server (eg \`npm run dev\`), a background listener, etc. Opens a new terminal in the user's environment which will not awaited for or killed.`,
-		params: {
-			cwd: { description: cwdHelper },
-		}
-	},
+type ToolParamNameOfTool<T extends ToolName> = keyof (typeof voidTools)[T]['params']
+export type ToolParamName = { [T in ToolName]: ToolParamNameOfTool<T> }[ToolName]
 
+const toolNamesSet = new Set<string>(toolNames)
 
-	kill_persistent_terminal: {
-		name: 'kill_persistent_terminal',
-		description: `Interrupts and closes a persistent terminal that you opened with open_persistent_terminal.`,
-		params: { persistent_terminal_id: { description: `The ID of the persistent terminal.` } }
-	}
-
-
-	// go_to_definition
-	// go_to_usages
-
-} satisfies { [T in keyof BuiltinToolResultType]: InternalToolInfo }
-
-
-
-
-export const builtinToolNames = Object.keys(builtinTools) as BuiltinToolName[]
-const toolNamesSet = new Set<string>(builtinToolNames)
-export const isABuiltinToolName = (toolName: string): toolName is BuiltinToolName => {
+export const isAToolName = (toolName: string): toolName is ToolName => {
 	const isAToolName = toolNamesSet.has(toolName)
 	return isAToolName
 }
 
-
-
-
-
-export const availableTools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[] | undefined) => {
-
-	const builtinToolNames: BuiltinToolName[] | undefined = chatMode === 'normal' ? undefined
-		: chatMode === 'gather' ? (Object.keys(builtinTools) as BuiltinToolName[]).filter(toolName => !(toolName in approvalTypeOfBuiltinToolName))
-			: chatMode === 'agent' ? Object.keys(builtinTools) as BuiltinToolName[]
+export const availableTools = (chatMode: ChatMode) => {
+	const toolNames: ToolName[] | undefined = chatMode === 'normal' ? undefined
+		: chatMode === 'gather' ? (Object.keys(voidTools) as ToolName[]).filter(toolName => !(toolName in approvalTypeOfToolName))
+			: chatMode === 'agent' ? Object.keys(voidTools) as ToolName[]
 				: undefined
 
-	const effectiveBuiltinTools = builtinToolNames?.map(toolName => builtinTools[toolName]) ?? undefined
-	const effectiveMCPTools = chatMode === 'agent' ? mcpTools : undefined
-
-	const tools: InternalToolInfo[] | undefined = !(builtinToolNames || mcpTools) ? undefined
-		: [
-			...effectiveBuiltinTools ?? [],
-			...effectiveMCPTools ?? [],
-		]
-
+	const tools: InternalToolInfo[] | undefined = toolNames?.map(toolName => voidTools[toolName])
 	return tools
 }
 
@@ -390,7 +382,7 @@ const toolCallDefinitionsXMLString = (tools: InternalToolInfo[]) => {
 }
 
 export const reParsedToolXMLString = (toolName: ToolName, toolParams: RawToolParamsObj) => {
-	const params = Object.keys(toolParams).map(paramName => `<${paramName}>${toolParams[paramName]}</${paramName}>`).join('\n')
+	const params = Object.keys(toolParams).map(paramName => `<${paramName}>${toolParams[paramName as ToolParamName]}</${paramName}>`).join('\n')
 	return `\
     <${toolName}>${!params ? '' : `\n${params}`}
     </${toolName}>`
@@ -399,8 +391,8 @@ export const reParsedToolXMLString = (toolName: ToolName, toolParams: RawToolPar
 
 /* We expect tools to come at the end - not a hard limit, but that's just how we process them, and the flow makes more sense that way. */
 // - You are allowed to call multiple tools by specifying them consecutively. However, there should be NO text or writing between tool calls or after them.
-const systemToolsXMLPrompt = (chatMode: ChatMode, mcpTools: InternalToolInfo[] | undefined) => {
-	const tools = availableTools(chatMode, mcpTools)
+const systemToolsXMLPrompt = (chatMode: ChatMode) => {
+	const tools = availableTools(chatMode)
 	if (!tools || tools.length === 0) return null
 
 	const toolXMLDefinitions = (`\
@@ -425,7 +417,7 @@ const systemToolsXMLPrompt = (chatMode: ChatMode, mcpTools: InternalToolInfo[] |
 // ======================================================== chat (normal, gather, agent) ========================================================
 
 
-export const chat_systemMessage = ({ workspaceFolders, openedURIs, activeURI, persistentTerminalIDs, directoryStr, chatMode: mode, mcpTools, includeXMLToolDefinitions }: { workspaceFolders: string[], directoryStr: string, openedURIs: string[], activeURI: string | undefined, persistentTerminalIDs: string[], chatMode: ChatMode, mcpTools: InternalToolInfo[] | undefined, includeXMLToolDefinitions: boolean }) => {
+export const chat_systemMessage = ({ workspaceFolders, openedURIs, activeURI, persistentTerminalIDs, directoryStr, chatMode: mode, includeXMLToolDefinitions }: { workspaceFolders: string[], directoryStr: string, openedURIs: string[], activeURI: string | undefined, persistentTerminalIDs: string[], chatMode: ChatMode, includeXMLToolDefinitions: boolean }) => {
 	const header = (`You are an expert coding ${mode === 'agent' ? 'agent' : 'assistant'} whose job is \
 ${mode === 'agent' ? `to help the user develop, run, and make changes to their codebase.`
 			: mode === 'gather' ? `to search, understand, and reference files in the user's codebase.`
@@ -459,7 +451,7 @@ ${directoryStr}
 </files_overview>`)
 
 
-	const toolDefinitions = includeXMLToolDefinitions ? systemToolsXMLPrompt(mode, mcpTools) : null
+	const toolDefinitions = includeXMLToolDefinitions ? systemToolsXMLPrompt(mode) : null
 
 	const details: string[] = []
 
@@ -575,24 +567,11 @@ export const messageOfSelection = async (
 ) => {
 	const lineNumAddition = (range: [number, number]) => ` (lines ${range[0]}:${range[1]})`
 
-	if (s.type === 'CodeSelection') {
+	if (s.type === 'File' || s.type === 'CodeSelection') {
 		const { val } = await readFile(opts.fileService, s.uri, DEFAULT_FILE_SIZE_LIMIT)
-		const lines = val?.split('\n')
-
-		const innerVal = lines?.slice(s.range[0] - 1, s.range[1]).join('\n')
-		const content = !lines ? ''
-			: `${tripleTick[0]}${s.language}\n${innerVal}\n${tripleTick[1]}`
-		const str = `${s.uri.fsPath}${lineNumAddition(s.range)}:\n${content}`
-		return str
-	}
-	else if (s.type === 'File') {
-		const { val } = await readFile(opts.fileService, s.uri, DEFAULT_FILE_SIZE_LIMIT)
-
-		const innerVal = val
-		const content = val === null ? ''
-			: `${tripleTick[0]}${s.language}\n${innerVal}\n${tripleTick[1]}`
-
-		const str = `${s.uri.fsPath}:\n${content}`
+		const lineNumAdd = s.type === 'CodeSelection' ? lineNumAddition(s.range) : ''
+		const content = val === null ? 'null' : `${tripleTick[0]}${s.language}\n${val}\n${tripleTick[1]}`
+		const str = `${s.uri.fsPath}${lineNumAdd}:\n${content}`
 		return str
 	}
 	else if (s.type === 'Folder') {
@@ -993,77 +972,3 @@ Store Result: After computing fib(n), the result is stored in memo for future re
 ## END EXAMPLES
 
 */
-
-
-// ======================================================== scm ========================================================================
-
-export const gitCommitMessage_systemMessage = `
-You are an expert software engineer AI assistant responsible for writing clear and concise Git commit messages that summarize the **purpose** and **intent** of the change. Try to keep your commit messages to one sentence. If necessary, you can use two sentences.
-
-You always respond with:
-- The commit message wrapped in <output> tags
-- A brief explanation of the reasoning behind the message, wrapped in <reasoning> tags
-
-Example format:
-<output>Fix login bug and improve error handling</output>
-<reasoning>This commit updates the login handler to fix a redirect issue and improves frontend error messages for failed logins.</reasoning>
-
-Do not include anything else outside of these tags.
-Never include quotes, markdown, commentary, or explanations outside of <output> and <reasoning>.`.trim()
-
-
-/**
- * Create a user message for the LLM to generate a commit message. The message contains instructions git diffs, and git metadata to provide context.
- *
- * @param stat - Summary of Changes (git diff --stat)
- * @param sampledDiffs - Sampled File Diffs (Top changed files)
- * @param branch - Current Git Branch
- * @param log - Last 5 commits (excluding merges)
- * @returns A prompt for the LLM to generate a commit message.
- *
- * @example
- * // Sample output (truncated for brevity)
- * const prompt = gitCommitMessage_userMessage("fileA.ts | 10 ++--", "diff --git a/fileA.ts...", "main", "abc123|Fix bug|2025-01-01\n...")
- *
- * // Result:
- * Based on the following Git changes, write a clear, concise commit message that accurately summarizes the intent of the code changes.
- *
- * Section 1 - Summary of Changes (git diff --stat):
- * fileA.ts | 10 ++--
- *
- * Section 2 - Sampled File Diffs (Top changed files):
- * diff --git a/fileA.ts b/fileA.ts
- * ...
- *
- * Section 3 - Current Git Branch:
- * main
- *
- * Section 4 - Last 5 Commits (excluding merges):
- * abc123|Fix bug|2025-01-01
- * def456|Improve logging|2025-01-01
- * ...
- */
-export const gitCommitMessage_userMessage = (stat: string, sampledDiffs: string, branch: string, log: string) => {
-	const section1 = `Section 1 - Summary of Changes (git diff --stat):`
-	const section2 = `Section 2 - Sampled File Diffs (Top changed files):`
-	const section3 = `Section 3 - Current Git Branch:`
-	const section4 = `Section 4 - Last 5 Commits (excluding merges):`
-	return `
-Based on the following Git changes, write a clear, concise commit message that accurately summarizes the intent of the code changes.
-
-${section1}
-
-${stat}
-
-${section2}
-
-${sampledDiffs}
-
-${section3}
-
-${branch}
-
-${section4}
-
-${log}`.trim()
-}

src/vs/workbench/contrib/void/common/sendLLMMessageService.ts

@@ -13,7 +13,6 @@ import { generateUuid } from '../../../../base/common/uuid.js';
 import { Event } from '../../../../base/common/event.js';
 import { Disposable } from '../../../../base/common/lifecycle.js';
 import { IVoidSettingsService } from './voidSettingsService.js';
-import { IMCPService } from './mcpService.js';
 
 // calls channel to implement features
 export const ILLMMessageService = createDecorator<ILLMMessageService>('llmMessageService');
@@ -62,7 +61,6 @@ export class LLMMessageService extends Disposable implements ILLMMessageService
 		@IMainProcessService private readonly mainProcessService: IMainProcessService, // used as a renderer (only usable on client side)
 		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
 		// @INotificationService private readonly notificationService: INotificationService,
-		@IMCPService private readonly mcpService: IMCPService,
 	) {
 		super()
 
@@ -118,8 +116,6 @@ export class LLMMessageService extends Disposable implements ILLMMessageService
 
 		const { settingsOfProvider, } = this.voidSettingsService.state
 
-		const mcpTools = this.mcpService.getMCPTools()
-
 		// add state for request id
 		const requestId = generateUuid();
 		this.llmMessageHooks.onText[requestId] = onText
@@ -133,7 +129,6 @@ export class LLMMessageService extends Disposable implements ILLMMessageService
 			requestId,
 			settingsOfProvider,
 			modelSelection,
-			mcpTools,
 		} satisfies MainSendLLMMessageParams);
 
 		return requestId

src/vs/workbench/contrib/void/common/sendLLMMessageTypes.ts

@@ -3,8 +3,7 @@
  *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
  *--------------------------------------------------------------------------------------*/
 
-import { InternalToolInfo } from './prompt/prompts.js'
-import { ToolName, ToolParamName } from './toolsServiceTypes.js'
+import { ToolName, ToolParamName } from './prompt/prompts.js'
 import { ChatMode, ModelSelection, ModelSelectionOptions, OverridesOfModel, ProviderName, RefreshableProviderName, SettingsOfProvider } from './voidSettingsTypes.js'
 
 
@@ -79,12 +78,12 @@ export type LLMFIMMessage = {
 
 
 export type RawToolParamsObj = {
-	[paramName in ToolParamName<ToolName>]?: string;
+	[paramName in ToolParamName]?: string;
 }
 export type RawToolCallObj = {
 	name: ToolName;
 	rawParams: RawToolParamsObj;
-	doneParams: ToolParamName<ToolName>[];
+	doneParams: ToolParamName[];
 	id: string;
 	isDone: boolean;
 };
@@ -134,7 +133,6 @@ export type SendLLMMessageParams = {
 	overridesOfModel: OverridesOfModel | undefined;
 
 	settingsOfProvider: SettingsOfProvider;
-	mcpTools: InternalToolInfo[] | undefined;
 } & SendLLMType
 
 

src/vs/workbench/contrib/void/common/storageKeys.ts

@@ -17,7 +17,3 @@ export const VOID_SETTINGS_STORAGE_KEY = 'void.settingsServiceStorageII'
 
 // 1.0.3
 export const THREAD_STORAGE_KEY = 'void.chatThreadStorageII'
-
-
-
-export const OPT_OUT_KEY = 'void.app.optOutAll'

src/vs/workbench/contrib/void/common/toolsServiceTypes.ts

@@ -1,7 +1,5 @@
 import { URI } from '../../../../base/common/uri.js'
-import { RawMCPToolCall } from './mcpServiceTypes.js';
-import { builtinTools } from './prompt/prompts.js';
-import { RawToolParamsObj } from './sendLLMMessageTypes.js';
+import { ToolName } from './prompt/prompts.js';
 
 
 
@@ -18,7 +16,7 @@ export type ShallowDirectoryItem = {
 }
 
 
-export const approvalTypeOfBuiltinToolName: Partial<{ [T in BuiltinToolName]?: 'edits' | 'terminal' | 'MCP tools' }> = {
+export const approvalTypeOfToolName: Partial<{ [T in ToolName]?: 'edits' | 'terminal' }> = {
 	'create_file_or_folder': 'edits',
 	'delete_file_or_folder': 'edits',
 	'rewrite_file': 'edits',
@@ -30,19 +28,16 @@ export const approvalTypeOfBuiltinToolName: Partial<{ [T in BuiltinToolName]?: '
 }
 
 
-export type ToolApprovalType = NonNullable<(typeof approvalTypeOfBuiltinToolName)[keyof typeof approvalTypeOfBuiltinToolName]>;
-
-
-export const toolApprovalTypes = new Set<ToolApprovalType>([
-	...Object.values(approvalTypeOfBuiltinToolName),
-	'MCP tools',
-])
-
 
+// {{add: define new type for approval types}}
+export type ToolApprovalType = NonNullable<(typeof approvalTypeOfToolName)[keyof typeof approvalTypeOfToolName]>;
 
+export const toolApprovalTypes = new Set<ToolApprovalType>(
+	Object.values(approvalTypeOfToolName).filter((v): v is ToolApprovalType => v !== undefined)
+)
 
 // PARAMS OF TOOL CALL
-export type BuiltinToolCallParams = {
+export type ToolCallParams = {
 	'read_file': { uri: URI, startLine: number | null, endLine: number | null, pageNumber: number },
 	'ls_dir': { uri: URI, pageNumber: number },
 	'get_dir_tree': { uri: URI },
@@ -63,7 +58,7 @@ export type BuiltinToolCallParams = {
 }
 
 // RESULT OF TOOL CALL
-export type BuiltinToolResultType = {
+export type ToolResultType = {
 	'read_file': { fileContents: string, totalFileLen: number, totalNumLines: number, hasNextPage: boolean },
 	'ls_dir': { children: ShallowDirectoryItem[] | null, hasNextPage: boolean, hasPrevPage: boolean, itemsRemaining: number },
 	'get_dir_tree': { str: string, },
@@ -83,15 +78,3 @@ export type BuiltinToolResultType = {
 	'kill_persistent_terminal': {},
 }
 
-
-export type ToolCallParams<T extends BuiltinToolName | (string & {})> = T extends BuiltinToolName ? BuiltinToolCallParams[T] : RawToolParamsObj
-export type ToolResult<T extends BuiltinToolName | (string & {})> = T extends BuiltinToolName ? BuiltinToolResultType[T] : RawMCPToolCall
-
-export type BuiltinToolName = keyof BuiltinToolResultType
-
-type BuiltinToolParamNameOfTool<T extends BuiltinToolName> = keyof (typeof builtinTools)[T]['params']
-export type BuiltinToolParamName = { [T in BuiltinToolName]: BuiltinToolParamNameOfTool<T> }[BuiltinToolName]
-
-
-export type ToolName = BuiltinToolName | (string & {})
-export type ToolParamName<T extends ToolName> = T extends BuiltinToolName ? BuiltinToolParamNameOfTool<T> : string

src/vs/workbench/contrib/void/common/voidCertificateService.ts

@@ -0,0 +1,202 @@
+/*--------------------------------------------------------------------------------------
+ *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
+ *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
+ *--------------------------------------------------------------------------------------*/
+
+import { Disposable } from '../../../../base/common/lifecycle.js';
+import { createDecorator } from '../../../../platform/instantiation/common/instantiation.js';
+import { registerSingleton, InstantiationType } from '../../../../platform/instantiation/common/extensions.js';
+import { ILogService } from '../../../../platform/log/common/log.js';
+import { IVoidSettingsService } from './voidSettingsService.js';
+import { IRequestService } from '../../../../platform/request/common/request.js';
+import { Emitter } from '../../../../base/common/event.js';
+import { CancellationToken } from '../../../../base/common/cancellation.js';
+import { IRequestContext } from '../../../../base/parts/request/common/request.js';
+import { IFileService } from '../../../../platform/files/common/files.js';
+import { URI } from '../../../../base/common/uri.js';
+
+/**
+ * Service for managing and applying custom certificate paths for secure requests.
+ */
+export interface IVoidCertificateService {
+	readonly _serviceBrand: undefined;
+
+	/**
+	 * Get all configured custom certificates as an array of file URIs.
+	 */
+	getCustomCertificates(): URI[];
+
+	/**
+	 * Add a new custom certificate path.
+	 */
+	addCustomCertificate(certificatePath: URI): Promise<void>;
+
+	/**
+	 * Remove a custom certificate path.
+	 */
+	removeCustomCertificate(certificatePath: URI): Promise<void>;
+
+	/**
+	 * Verify that a certificate path exists and is readable.
+	 */
+	verifyCertificatePath(certificatePath: URI): Promise<boolean>;
+
+	/**
+	 * Get all certificate contents as a concatenated string for use with HTTPS requests.
+	 */
+	getCertificateContents(): Promise<string[]>;
+}
+
+export const IVoidCertificateService = createDecorator<IVoidCertificateService>('VoidCertificateService');
+
+/**
+ * Service implementation for managing custom certificates.
+ */
+export class VoidCertificateService extends Disposable implements IVoidCertificateService {
+	readonly _serviceBrand: undefined;
+
+	private readonly _onCertificatesChanged = new Emitter<void>();
+
+	constructor(
+		@IVoidSettingsService private readonly voidSettingsService: IVoidSettingsService,
+		@ILogService private readonly logService: ILogService,
+		@IRequestService private readonly requestService: IRequestService,
+		@IFileService private readonly fileService: IFileService,
+	) {
+		super();
+
+		// Override the original request service to include our custom certificates
+		this._overrideRequestService();
+	}
+
+	/**
+	 * Get all configured custom certificates
+	 */
+	getCustomCertificates(): URI[] {
+		const certificatePaths = this.voidSettingsService.state.globalSettings.customRootCertificates || [];
+		return certificatePaths.map(path => URI.parse(path));
+	}
+
+	/**
+	 * Add a new custom certificate path
+	 */
+	async addCustomCertificate(certificatePath: URI): Promise<void> {
+		// Verify the certificate path exists
+		const isValid = await this.verifyCertificatePath(certificatePath);
+		if (!isValid) {
+			throw new Error(`Certificate file not found or not readable: ${certificatePath.toString()}`);
+		}
+
+		// Get current certificates
+		const currentCertificates = this.getCustomCertificates();
+
+		// Check if already exists
+		if (currentCertificates.some(cert => cert.toString() === certificatePath.toString())) {
+			return; // Already exists, nothing to do
+		}
+
+		// Add the new certificate path
+		const newCertificates = [...currentCertificates, certificatePath];
+		await this.voidSettingsService.setGlobalSetting('customRootCertificates', newCertificates.map(uri => uri.toString()));
+
+		this._onCertificatesChanged.fire();
+		this.logService.info(`Added custom certificate: ${certificatePath.toString()}`);
+	}
+
+	/**
+	 * Remove a custom certificate path
+	 */
+	async removeCustomCertificate(certificatePath: URI): Promise<void> {
+		const currentCertificates = this.getCustomCertificates();
+
+		// Remove the certificate
+		const newCertificates = currentCertificates.filter(cert => cert.toString() !== certificatePath.toString());
+
+		// Update settings
+		await this.voidSettingsService.setGlobalSetting('customRootCertificates', newCertificates.map(uri => uri.toString()));
+
+		this._onCertificatesChanged.fire();
+		this.logService.info(`Removed custom certificate: ${certificatePath.toString()}`);
+	}
+
+	/**
+	 * Verify a certificate path exists and is readable
+	 */
+	async verifyCertificatePath(certificatePath: URI): Promise<boolean> {
+		try {
+			// Check if file exists and is readable
+			const stats = await this.fileService.stat(certificatePath);
+			if (!stats.isFile) {
+				return false;
+			}
+
+			// Check if we can read the file
+			await this.fileService.readFile(certificatePath);
+			return true;
+		} catch (error) {
+			this.logService.error(`Error verifying certificate path: ${error}`);
+			return false;
+		}
+	}
+
+	/**
+	 * Get all certificate contents
+	 */
+	async getCertificateContents(): Promise<string[]> {
+		const certificatePaths = this.getCustomCertificates();
+		const contents: string[] = [];
+
+		for (const certPath of certificatePaths) {
+			try {
+				if (await this.verifyCertificatePath(certPath)) {
+					const fileContent = await this.fileService.readFile(certPath);
+					contents.push(fileContent.value.toString());
+				}
+			} catch (error) {
+				this.logService.error(`Error reading certificate ${certPath.toString()}: ${error}`);
+			}
+		}
+
+		return contents;
+	}
+
+	/**
+	 * Override the original request service to include our custom certificates
+	 */
+	private _overrideRequestService(): void {
+		// Store the original request method
+		const originalRequest = this.requestService.request.bind(this.requestService);
+
+		// Override the request method to inject our certificates
+		// @ts-ignore - We're monkey patching the request service
+		this.requestService.request = async (options: any, token: CancellationToken): Promise<IRequestContext> => {
+			// Only add certificates for HTTPS requests
+			if (options.url?.startsWith('https://')) {
+				try {
+					// Load system certificates
+					const systemCerts = await this.requestService.loadCertificates();
+
+					// Load our custom certificates
+					const customCertContents = await this.getCertificateContents();
+
+					if (customCertContents.length > 0) {
+						// Create custom CA option
+						// Documentation in Node.js: https://nodejs.org/api/https.html#https_https_request_options_callback
+						// The 'ca' option can be a string, Buffer, or array of strings/Buffers
+						(options as any).ca = [...systemCerts, ...customCertContents];
+
+						this.logService.debug(`Added ${customCertContents.length} custom certificates to request to ${options.url}`);
+					}
+				} catch (error) {
+					this.logService.error(`Error adding custom certificates: ${error}`);
+				}
+			}
+
+			// Forward to the original request implementation
+			return originalRequest(options, token);
+		};
+	}
+}
+
+// Register the service
+registerSingleton(IVoidCertificateService, VoidCertificateService, InstantiationType.Eager);

src/vs/workbench/contrib/void/common/voidSCMTypes.ts

@@ -1,36 +0,0 @@
-/*--------------------------------------------------------------------------------------
- *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
- *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
- *--------------------------------------------------------------------------------------*/
-
-import { createDecorator } from '../../../../platform/instantiation/common/instantiation.js';
-
-export interface IVoidSCMService {
-	readonly _serviceBrand: undefined;
-	/**
-	 * Get git diff --stat
-	 *
-	 * @param path Path to the git repository
-	 */
-	gitStat(path: string): Promise<string>
-	/**
-	 * Get git diff --stat for the top 10 most significantly changed files according to lines added/removed
-	 *
-	 * @param path Path to the git repository
-	 */
-	gitSampledDiffs(path: string): Promise<string>
-	/**
-	 * Get the current git branch
-	 *
-	 * @param path Path to the git repository
-	 */
-	gitBranch(path: string): Promise<string>
-	/**
-	 * Get the last 5 commits excluding merges
-	 *
-	 * @param path Path to the git repository
-	 */
-	gitLog(path: string): Promise<string>
-}
-
-export const IVoidSCMService = createDecorator<IVoidSCMService>('voidSCMService')

src/vs/workbench/contrib/void/common/voidSettingsService.ts

@@ -13,7 +13,7 @@ import { IStorageService, StorageScope, StorageTarget } from '../../../../platfo
 import { IMetricsService } from './metricsService.js';
 import { defaultProviderSettings, getModelCapabilities, ModelOverrides } from './modelCapabilities.js';
 import { VOID_SETTINGS_STORAGE_KEY } from './storageKeys.js';
-import { defaultSettingsOfProvider, FeatureName, ProviderName, ModelSelectionOfFeature, SettingsOfProvider, SettingName, providerNames, ModelSelection, modelSelectionsEqual, featureNames, VoidStatefulModelInfo, GlobalSettings, GlobalSettingName, defaultGlobalSettings, ModelSelectionOptions, OptionsOfModelSelection, ChatMode, OverridesOfModel, defaultOverridesOfModel, MCPUserStateOfName as MCPUserStateOfName, MCPUserState } from './voidSettingsTypes.js';
+import { defaultSettingsOfProvider, FeatureName, ProviderName, ModelSelectionOfFeature, SettingsOfProvider, SettingName, providerNames, ModelSelection, modelSelectionsEqual, featureNames, VoidStatefulModelInfo, GlobalSettings, GlobalSettingName, defaultGlobalSettings, ModelSelectionOptions, OptionsOfModelSelection, ChatMode, OverridesOfModel, defaultOverridesOfModel } from './voidSettingsTypes.js';
 
 
 // name is the name in the dropdown
@@ -43,7 +43,6 @@ export type VoidSettingsState = {
 	readonly optionsOfModelSelection: OptionsOfModelSelection;
 	readonly overridesOfModel: OverridesOfModel;
 	readonly globalSettings: GlobalSettings;
-	readonly mcpUserStateOfName: MCPUserStateOfName; // user-controlled state of MCP servers
 
 	readonly _modelOptions: ModelOption[] // computed based on the two above items
 }
@@ -63,7 +62,6 @@ export interface IVoidSettingsService {
 	setModelSelectionOfFeature: SetModelSelectionOfFeatureFn;
 	setOptionsOfModelSelection: SetOptionsOfModelSelection;
 	setGlobalSetting: SetGlobalSettingFn;
-	// setMCPServerStates: (newStates: MCPServerStates) => Promise<void>;
 
 	// setting to undefined CLEARS it, unlike others:
 	setOverridesOfModel(providerName: ProviderName, modelName: string, overrides: Partial<ModelOverrides> | undefined): Promise<void>;
@@ -75,10 +73,6 @@ export interface IVoidSettingsService {
 	toggleModelHidden(providerName: ProviderName, modelName: string): void;
 	addModel(providerName: ProviderName, modelName: string): void;
 	deleteModel(providerName: ProviderName, modelName: string): boolean;
-
-	addMCPUserStateOfNames(userStateOfName: MCPUserStateOfName): Promise<void>;
-	removeMCPUserStateOfNames(serverNames: string[]): Promise<void>;
-	setMCPServerState(serverName: string, state: MCPUserState): Promise<void>;
 }
 
 
@@ -116,7 +110,6 @@ export const modelFilterOfFeatureName: {
 	'Chat': { filter: o => true, emptyMessage: null, },
 	'Ctrl+K': { filter: o => true, emptyMessage: null, },
 	'Apply': { filter: o => true, emptyMessage: null, },
-	'SCM': { filter: o => true, emptyMessage: null, },
 }
 
 
@@ -214,12 +207,11 @@ const _validatedModelState = (state: Omit<VoidSettingsState, '_modelOptions'>):
 const defaultState = () => {
 	const d: VoidSettingsState = {
 		settingsOfProvider: deepClone(defaultSettingsOfProvider),
-		modelSelectionOfFeature: { 'Chat': null, 'Ctrl+K': null, 'Autocomplete': null, 'Apply': null, 'SCM': null },
+		modelSelectionOfFeature: { 'Chat': null, 'Ctrl+K': null, 'Autocomplete': null, 'Apply': null },
 		globalSettings: deepClone(defaultGlobalSettings),
-		optionsOfModelSelection: { 'Chat': {}, 'Ctrl+K': {}, 'Autocomplete': {}, 'Apply': {}, 'SCM': {} },
+		optionsOfModelSelection: { 'Chat': {}, 'Ctrl+K': {}, 'Autocomplete': {}, 'Apply': {} },
 		overridesOfModel: deepClone(defaultOverridesOfModel),
 		_modelOptions: [], // computed later
-		mcpUserStateOfName: {},
 	}
 	return d
 }
@@ -263,7 +255,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 		await this._storeState()
 		this._onDidChangeState.fire()
 		this._onUpdate_syncApplyToChat()
-		this._onUpdate_syncSCMToChat()
 	}
 	async resetState() {
 		await this.dangerousSetState(defaultState())
@@ -281,17 +272,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 
 			// autoapprove is now an obj not a boolean (1.2.5)
 			if (typeof readS.globalSettings.autoApprove === 'boolean') readS.globalSettings.autoApprove = {}
-
-			// 1.3.5 add source control feature
-			if (readS.modelSelectionOfFeature && !readS.modelSelectionOfFeature['SCM']) {
-				readS.modelSelectionOfFeature['SCM'] = deepClone(readS.modelSelectionOfFeature['Chat'])
-				readS.optionsOfModelSelection['SCM'] = deepClone(readS.optionsOfModelSelection['Chat'])
-			}
-			// add disableSystemMessage feature
-			if (readS.globalSettings.disableSystemMessage === undefined) readS.globalSettings.disableSystemMessage = false;
-			
-			// add autoAcceptLLMChanges feature
-			if (readS.globalSettings.autoAcceptLLMChanges === undefined) readS.globalSettings.autoAcceptLLMChanges = false;
 		}
 		catch (e) {
 			readS = defaultState()
@@ -381,7 +361,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 
 		const newGlobalSettings = this.state.globalSettings
 		const newOverridesOfModel = this.state.overridesOfModel
-		const newMCPUserStateOfName = this.state.mcpUserStateOfName
 
 		const newState = {
 			modelSelectionOfFeature: newModelSelectionOfFeature,
@@ -389,7 +368,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 			settingsOfProvider: newSettingsOfProvider,
 			globalSettings: newGlobalSettings,
 			overridesOfModel: newOverridesOfModel,
-			mcpUserStateOfName: newMCPUserStateOfName,
 		}
 
 		this.state = _validatedModelState(newState)
@@ -403,10 +381,7 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 	private _onUpdate_syncApplyToChat() {
 		// if sync is turned on, sync (call this whenever Chat model or !!sync changes)
 		this.setModelSelectionOfFeature('Apply', deepClone(this.state.modelSelectionOfFeature['Chat']))
-	}
 
-	private _onUpdate_syncSCMToChat() {
-		this.setModelSelectionOfFeature('SCM', deepClone(this.state.modelSelectionOfFeature['Chat']))
 	}
 
 	setGlobalSetting: SetGlobalSettingFn = async (settingName, newVal) => {
@@ -423,8 +398,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 
 		// hooks
 		if (this.state.globalSettings.syncApplyToChat) this._onUpdate_syncApplyToChat()
-		if (this.state.globalSettings.syncSCMToChat) this._onUpdate_syncSCMToChat()
-
 	}
 
 
@@ -444,9 +417,7 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 
 		// hooks
 		if (featureName === 'Chat') {
-			// When Chat model changes, update synced features
-			this._onUpdate_syncApplyToChat()
-			this._onUpdate_syncSCMToChat()
+			if (this.state.globalSettings.syncApplyToChat) this._onUpdate_syncApplyToChat()
 		}
 	}
 
@@ -560,55 +531,6 @@ class VoidSettingsService extends Disposable implements IVoidSettingsService {
 		return true
 	}
 
-	// MCP Server State
-	private _setMCPUserStateOfName = async (newStates: MCPUserStateOfName) => {
-		const newState: VoidSettingsState = {
-			...this.state,
-			mcpUserStateOfName: {
-				...this.state.mcpUserStateOfName,
-				...newStates
-			}
-		};
-		this.state = _validatedModelState(newState);
-		await this._storeState();
-		this._onDidChangeState.fire();
-		this._metricsService.capture('Set MCP Server States', { newStates });
-	}
-
-	addMCPUserStateOfNames = async (newMCPStates: MCPUserStateOfName) => {
-		const { mcpUserStateOfName: mcpServerStates } = this.state
-		const newMCPServerStates = {
-			...mcpServerStates,
-			...newMCPStates,
-		}
-		await this._setMCPUserStateOfName(newMCPServerStates)
-		this._metricsService.capture('Add MCP Servers', { servers: Object.keys(newMCPStates).join(', ') });
-	}
-
-	removeMCPUserStateOfNames = async (serverNames: string[]) => {
-		const { mcpUserStateOfName: mcpServerStates } = this.state
-		const newMCPServerStates = {
-			...mcpServerStates,
-		}
-		serverNames.forEach(serverName => {
-			if (serverName in newMCPServerStates) {
-				delete newMCPServerStates[serverName]
-			}
-		})
-		await this._setMCPUserStateOfName(newMCPServerStates)
-		this._metricsService.capture('Remove MCP Servers', { servers: serverNames.join(', ') });
-	}
-
-	setMCPServerState = async (serverName: string, state: MCPUserState) => {
-		const { mcpUserStateOfName } = this.state
-		const newMCPServerStates = {
-			...mcpUserStateOfName,
-			[serverName]: state,
-		}
-		await this._setMCPUserStateOfName(newMCPServerStates)
-		this._metricsService.capture('Update MCP Server State', { serverName, state });
-	}
-
 }
 
 

src/vs/workbench/contrib/void/common/voidSettingsTypes.ts

@@ -33,7 +33,7 @@ export type VoidStatefulModelInfo = { // <-- STATEFUL
 	modelName: string,
 	type: 'default' | 'autodetected' | 'custom';
 	isHidden: boolean, // whether or not the user is hiding it (switched off)
-}
+}  // TODO!!! eventually we'd want to let the user change supportsFIM, etc on the model themselves
 
 
 
@@ -103,9 +103,6 @@ export const displayInfoOfProviderName = (providerName: ProviderName): DisplayIn
 	else if (providerName === 'microsoftAzure') {
 		return { title: 'Microsoft Azure OpenAI', }
 	}
-	else if (providerName === 'awsBedrock') {
-		return { title: 'AWS Bedrock', }
-	}
 
 	throw new Error(`descOfProviderName: Unknown provider name: "${providerName}"`)
 }
@@ -123,7 +120,6 @@ export const subTextMdOfProviderName = (providerName: ProviderName): string => {
 	if (providerName === 'openAICompatible') return `Use any provider that's OpenAI-compatible (use this for llama.cpp and more).`
 	if (providerName === 'googleVertex') return 'You must authenticate before using Vertex with Void. Read more about endpoints [here](https://cloud.google.com/vertex-ai/generative-ai/docs/multimodal/call-vertex-using-openai-library), and regions [here](https://cloud.google.com/vertex-ai/docs/general/locations#available-regions).'
 	if (providerName === 'microsoftAzure') return 'Read more about endpoints [here](https://learn.microsoft.com/en-us/rest/api/aifoundry/model-inference/get-chat-completions/get-chat-completions?view=rest-aifoundry-model-inference-2024-05-01-preview&tabs=HTTP), and get your API key [here](https://learn.microsoft.com/en-us/azure/search/search-security-api-keys?tabs=rest-use%2Cportal-find%2Cportal-query#find-existing-keys).'
-	if (providerName === 'awsBedrock') return 'Connect via a LiteLLM proxy or the AWS [Bedrock-Access-Gateway](https://github.com/aws-samples/bedrock-access-gateway). LiteLLM Bedrock setup docs are [here](https://docs.litellm.ai/docs/providers/bedrock).'
 	if (providerName === 'ollama') return 'Read more about custom [Endpoints here](https://github.com/ollama/ollama/blob/main/docs/faq.md#how-can-i-expose-ollama-on-my-network).'
 	if (providerName === 'vLLM') return 'Read more about custom [Endpoints here](https://docs.vllm.ai/en/latest/getting_started/quickstart.html#openai-compatible-server).'
 	if (providerName === 'lmStudio') return 'Read more about custom [Endpoints here](https://lmstudio.ai/docs/app/api/endpoints/openai).'
@@ -155,8 +151,7 @@ export const displayInfoOfSettingName = (providerName: ProviderName, settingName
 											providerName === 'mistral' ? 'api-key...' :
 												providerName === 'googleVertex' ? 'AIzaSy...' :
 													providerName === 'microsoftAzure' ? 'key-...' :
-														providerName === 'awsBedrock' ? 'key-...' :
-															'',
+														'',
 
 			isPasswordField: true,
 		}
@@ -170,16 +165,14 @@ export const displayInfoOfSettingName = (providerName: ProviderName, settingName
 							providerName === 'googleVertex' ? 'baseURL' :
 								providerName === 'microsoftAzure' ? 'baseURL' :
 									providerName === 'liteLLM' ? 'baseURL' :
-										providerName === 'awsBedrock' ? 'Endpoint' :
-											'(never)',
+										'(never)',
 
 			placeholder: providerName === 'ollama' ? defaultProviderSettings.ollama.endpoint
 				: providerName === 'vLLM' ? defaultProviderSettings.vLLM.endpoint
 					: providerName === 'openAICompatible' ? 'https://my-website.com/v1'
 						: providerName === 'lmStudio' ? defaultProviderSettings.lmStudio.endpoint
 							: providerName === 'liteLLM' ? 'http://localhost:4000'
-								: providerName === 'awsBedrock' ? 'http://localhost:4000/v1'
-									: '(never)',
+								: '(never)',
 
 
 		}
@@ -192,9 +185,7 @@ export const displayInfoOfSettingName = (providerName: ProviderName, settingName
 		return {
 			title: 'Region',
 			placeholder: providerName === 'googleVertex' ? defaultProviderSettings.googleVertex.region
-				: providerName === 'awsBedrock'
-					? defaultProviderSettings.awsBedrock.region
-					: ''
+				: ''
 		}
 	}
 	else if (settingName === 'azureApiVersion') {
@@ -231,9 +222,12 @@ export const displayInfoOfSettingName = (providerName: ProviderName, settingName
 	}
 
 	throw new Error(`displayInfo: Unknown setting name: "${settingName}"`)
+
 }
 
 
+
+
 const defaultCustomSettings: Record<CustomSettingName, undefined> = {
 	apiKey: undefined,
 	endpoint: undefined,
@@ -346,12 +340,6 @@ export const defaultSettingsOfProvider: SettingsOfProvider = {
 		...modelInfoOfDefaultModelNames(defaultModelsOfProvider.microsoftAzure),
 		_didFillInProviderSettings: undefined,
 	},
-	awsBedrock: { // aggregator (serves models from multiple providers)
-		...defaultCustomSettings,
-		...defaultProviderSettings.awsBedrock,
-		...modelInfoOfDefaultModelNames(defaultModelsOfProvider.awsBedrock),
-		_didFillInProviderSettings: undefined,
-	},
 }
 
 
@@ -362,7 +350,7 @@ export const modelSelectionsEqual = (m1: ModelSelection, m2: ModelSelection) =>
 }
 
 // this is a state
-export const featureNames = ['Chat', 'Ctrl+K', 'Autocomplete', 'Apply', 'SCM'] as const
+export const featureNames = ['Chat', 'Ctrl+K', 'Autocomplete', 'Apply'] as const
 export type ModelSelectionOfFeature = Record<(typeof featureNames)[number], ModelSelection | null>
 export type FeatureName = keyof ModelSelectionOfFeature
 
@@ -377,9 +365,6 @@ export const displayInfoOfFeatureName = (featureName: FeatureName) => {
 		return 'Chat'
 	else if (featureName === 'Apply')
 		return 'Apply'
-	// source control:
-	else if (featureName === 'SCM')
-		return 'Commit Message Generator'
 	else
 		throw new Error(`Feature Name ${featureName} not allowed`)
 }
@@ -443,15 +428,13 @@ export type GlobalSettings = {
 	aiInstructions: string;
 	enableAutocomplete: boolean;
 	syncApplyToChat: boolean;
-	syncSCMToChat: boolean;
 	enableFastApply: boolean;
 	chatMode: ChatMode;
 	autoApprove: { [approvalType in ToolApprovalType]?: boolean };
 	showInlineSuggestions: boolean;
 	includeToolLintErrors: boolean;
 	isOnboardingComplete: boolean;
-	disableSystemMessage: boolean;
-	autoAcceptLLMChanges: boolean;
+	customRootCertificates: string[]; // Paths to custom root certificates
 }
 
 export const defaultGlobalSettings: GlobalSettings = {
@@ -459,15 +442,13 @@ export const defaultGlobalSettings: GlobalSettings = {
 	aiInstructions: '',
 	enableAutocomplete: false,
 	syncApplyToChat: true,
-	syncSCMToChat: true,
 	enableFastApply: true,
 	chatMode: 'agent',
 	autoApprove: {},
 	showInlineSuggestions: true,
 	includeToolLintErrors: true,
 	isOnboardingComplete: false,
-	disableSystemMessage: false,
-	autoAcceptLLMChanges: false,
+	customRootCertificates: [],
 }
 
 export type GlobalSettingName = keyof GlobalSettings
@@ -512,13 +493,3 @@ export type OverridesOfModel = {
 const overridesOfModel = {} as OverridesOfModel
 for (const providerName of providerNames) { overridesOfModel[providerName] = {} }
 export const defaultOverridesOfModel = overridesOfModel
-
-
-
-export interface MCPUserStateOfName {
-	[serverName: string]: MCPUserState | undefined;
-}
-
-export interface MCPUserState {
-	isOn: boolean;
-}

src/vs/workbench/contrib/void/electron-main/llmMessage/extractGrammar.ts

@@ -5,9 +5,8 @@
 
 import { generateUuid } from '../../../../../base/common/uuid.js'
 import { endsWithAnyPrefixOf, SurroundingsRemover } from '../../common/helpers/extractCodeFromResult.js'
-import { availableTools, InternalToolInfo } from '../../common/prompt/prompts.js'
+import { availableTools, InternalToolInfo, ToolName, ToolParamName } from '../../common/prompt/prompts.js'
 import { OnFinalMessage, OnText, RawToolCallObj, RawToolParamsObj } from '../../common/sendLLMMessageTypes.js'
-import { ToolName, ToolParamName } from '../../common/toolsServiceTypes.js'
 import { ChatMode } from '../../common/voidSettingsTypes.js'
 
 
@@ -165,15 +164,15 @@ const findIndexOfAny = (fullText: string, matches: string[]) => {
 
 
 type ToolOfToolName = { [toolName: string]: InternalToolInfo | undefined }
-const parseXMLPrefixToToolCall = <T extends ToolName,>(toolName: T, toolId: string, str: string, toolOfToolName: ToolOfToolName): RawToolCallObj => {
+const parseXMLPrefixToToolCall = (toolName: ToolName, toolId: string, str: string, toolOfToolName: ToolOfToolName): RawToolCallObj => {
 	const paramsObj: RawToolParamsObj = {}
-	const doneParams: ToolParamName<T>[] = []
+	const doneParams: ToolParamName[] = []
 	let isDone = false
 
 	const getAnswer = (): RawToolCallObj => {
 		// trim off all whitespace at and before first \n and after last \n for each param
 		for (const p in paramsObj) {
-			const paramName = p as ToolParamName<T>
+			const paramName = p as ToolParamName
 			const orig = paramsObj[paramName]
 			if (orig === undefined) continue
 			paramsObj[paramName] = trimBeforeAndAfterNewLines(orig)
@@ -203,16 +202,16 @@ const parseXMLPrefixToToolCall = <T extends ToolName,>(toolName: T, toolId: stri
 
 	const pm = new SurroundingsRemover(str)
 
-	const allowedParams = Object.keys(toolOfToolName[toolName]?.params ?? {}) as ToolParamName<T>[]
+	const allowedParams = Object.keys(toolOfToolName[toolName]?.params ?? {}) as ToolParamName[]
 	if (allowedParams.length === 0) return getAnswer()
-	let latestMatchedOpenParam: null | ToolParamName<T> = null
+	let latestMatchedOpenParam: null | ToolParamName = null
 	let n = 0
 	while (true) {
 		n += 1
 		if (n > 10) return getAnswer() // just for good measure as this code is early
 
 		// find the param name opening tag
-		let matchedOpenParam: null | ToolParamName<T> = null
+		let matchedOpenParam: null | ToolParamName = null
 		for (const paramName of allowedParams) {
 			const removed = pm.removeFromStartUntilFullMatch(`<${paramName}>`, true)
 			if (removed) {
@@ -261,14 +260,11 @@ const parseXMLPrefixToToolCall = <T extends ToolName,>(toolName: T, toolId: stri
 }
 
 export const extractXMLToolsWrapper = (
-	onText: OnText,
-	onFinalMessage: OnFinalMessage,
-	chatMode: ChatMode | null,
-	mcpTools: InternalToolInfo[] | undefined,
+	onText: OnText, onFinalMessage: OnFinalMessage, chatMode: ChatMode | null
 ): { newOnText: OnText, newOnFinalMessage: OnFinalMessage } => {
 
 	if (!chatMode) return { newOnText: onText, newOnFinalMessage: onFinalMessage }
-	const tools = availableTools(chatMode, mcpTools)
+	const tools = availableTools(chatMode)
 	if (!tools) return { newOnText: onText, newOnFinalMessage: onFinalMessage }
 
 	const toolOfToolName: ToolOfToolName = {}

src/vs/workbench/contrib/void/electron-main/llmMessage/sendLLMMessage.impl.ts

@@ -7,7 +7,7 @@
 /* eslint-disable */
 import Anthropic from '@anthropic-ai/sdk';
 import { Ollama } from 'ollama';
-import OpenAI, { ClientOptions, AzureOpenAI } from 'openai';
+import OpenAI, { ClientOptions } from 'openai';
 import { MistralCore } from '@mistralai/mistralai/core.js';
 import { fimComplete } from '@mistralai/mistralai/funcs/fimComplete.js';
 import { Tool as GeminiTool, FunctionDeclaration, GoogleGenAI, ThinkingConfig, Schema, Type } from '@google/genai';
@@ -18,7 +18,7 @@ import { AnthropicLLMChatMessage, GeminiLLMChatMessage, LLMChatMessage, LLMFIMMe
 import { ChatMode, displayInfoOfProviderName, ModelSelectionOptions, OverridesOfModel, ProviderName, SettingsOfProvider } from '../../common/voidSettingsTypes.js';
 import { getSendableReasoningInfo, getModelCapabilities, getProviderCapabilities, defaultProviderSettings, getReservedOutputTokenSpace } from '../../common/modelCapabilities.js';
 import { extractReasoningWrapper, extractXMLToolsWrapper } from './extractGrammar.js';
-import { availableTools, InternalToolInfo } from '../../common/prompt/prompts.js';
+import { availableTools, InternalToolInfo, isAToolName, ToolParamName, voidTools } from '../../common/prompt/prompts.js';
 import { generateUuid } from '../../../../../base/common/uuid.js';
 
 const getGoogleApiKey = async () => {
@@ -48,7 +48,6 @@ type SendChatParams_Internal = InternalCommonMessageParams & {
 	messages: LLMChatMessage[];
 	separateSystemMessage: string | undefined;
 	chatMode: ChatMode | null;
-	mcpTools: InternalToolInfo[] | undefined;
 }
 type SendFIMParams_Internal = InternalCommonMessageParams & { messages: LLMFIMMessage; separateSystemMessage: string | undefined; }
 export type ListParams_Internal<ModelResponse> = ModelListParams<ModelResponse>
@@ -115,36 +114,10 @@ const newOpenAICompatibleSDK = async ({ settingsOfProvider, providerName, includ
 	}
 	else if (providerName === 'microsoftAzure') {
 		// https://learn.microsoft.com/en-us/rest/api/aifoundry/model-inference/get-chat-completions/get-chat-completions?view=rest-aifoundry-model-inference-2024-05-01-preview&tabs=HTTP
-		//  https://github.com/openai/openai-node?tab=readme-ov-file#microsoft-azure-openai
 		const thisConfig = settingsOfProvider[providerName]
-		const endpoint = `https://${thisConfig.project}.openai.azure.com/`;
-		const apiVersion = thisConfig.azureApiVersion ?? '2024-04-01-preview';
-		const options = { endpoint, apiKey: thisConfig.apiKey, apiVersion };
-		return new AzureOpenAI({ ...options, ...commonPayloadOpts });
+		const baseURL = `https://${thisConfig.project}.services.ai.azure.com/api/models/chat/completions?api-version=${thisConfig.azureApiVersion}`
+		return new OpenAI({ baseURL: baseURL, apiKey: thisConfig.apiKey, ...commonPayloadOpts })
 	}
-	else if (providerName === 'awsBedrock') {
-		/**
-		  * We treat Bedrock as *OpenAI-compatible only through a proxy*:
-		  *   • LiteLLM default → http://localhost:4000/v1
-		  *   • Bedrock-Access-Gateway → https://<api-id>.execute-api.<region>.amazonaws.com/openai/
-		  *
-		  * The native Bedrock runtime endpoint
-		  *   https://bedrock-runtime.<region>.amazonaws.com
-		  * is **NOT** OpenAI-compatible, so we do *not* fall back to it here.
-		  */
-		const { endpoint, apiKey } = settingsOfProvider.awsBedrock
-
-		// ① use the user-supplied proxy if present
-		// ② otherwise default to local LiteLLM
-		let baseURL = endpoint || 'http://localhost:4000/v1'
-
-		// Normalize: make sure we end with “/v1”
-		if (!baseURL.endsWith('/v1'))
-			baseURL = baseURL.replace(/\/+$/, '') + '/v1'
-
-		return new OpenAI({ baseURL, apiKey, ...commonPayloadOpts })
-	}
-
 
 	else if (providerName === 'deepseek') {
 		const thisConfig = settingsOfProvider[providerName]
@@ -230,8 +203,8 @@ const toOpenAICompatibleTool = (toolInfo: InternalToolInfo) => {
 	} satisfies OpenAI.Chat.Completions.ChatCompletionTool
 }
 
-const openAITools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[] | undefined) => {
-	const allowedTools = availableTools(chatMode, mcpTools)
+const openAITools = (chatMode: ChatMode) => {
+	const allowedTools = availableTools(chatMode)
 	if (!allowedTools || Object.keys(allowedTools).length === 0) return null
 
 	const openAITools: OpenAI.Chat.Completions.ChatCompletionTool[] = []
@@ -241,36 +214,29 @@ const openAITools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[] | u
 	return openAITools
 }
 
-
-// convert LLM tool call to our tool format
-const rawToolCallObjOfParamsStr = (name: string, toolParamsStr: string, id: string): RawToolCallObj | null => {
+const rawToolCallObjOf = (name: string, toolParamsStr: string, id: string): RawToolCallObj | null => {
+	if (!isAToolName(name)) return null
+	const rawParams: RawToolParamsObj = {}
 	let input: unknown
-	try { input = JSON.parse(toolParamsStr) }
-	catch (e) { return null }
-
+	try {
+		input = JSON.parse(toolParamsStr)
+	}
+	catch (e) {
+		return null
+	}
 	if (input === null) return null
 	if (typeof input !== 'object') return null
-
-	const rawParams: RawToolParamsObj = input
-	return { id, name, rawParams, doneParams: Object.keys(rawParams), isDone: true }
-}
-
-
-const rawToolCallObjOfAnthropicParams = (toolBlock: Anthropic.Messages.ToolUseBlock): RawToolCallObj | null => {
-	const { id, name, input } = toolBlock
-
-	if (input === null) return null
-	if (typeof input !== 'object') return null
-
-	const rawParams: RawToolParamsObj = input
-	return { id, name, rawParams, doneParams: Object.keys(rawParams), isDone: true }
+	for (const paramName in voidTools[name].params) {
+		rawParams[paramName as ToolParamName] = (input as any)[paramName]
+	}
+	return { id, name, rawParams, doneParams: Object.keys(rawParams) as ToolParamName[], isDone: true }
 }
 
 
 // ------------ OPENAI-COMPATIBLE ------------
 
 
-const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, modelName: modelName_, _setAborter, providerName, chatMode, separateSystemMessage, overridesOfModel, mcpTools }: SendChatParams_Internal) => {
+const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, modelName: modelName_, _setAborter, providerName, chatMode, separateSystemMessage, overridesOfModel }: SendChatParams_Internal) => {
 	const {
 		modelName,
 		specialToolFormat,
@@ -290,17 +256,13 @@ const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onE
 	}
 
 	// tools
-	const potentialTools = openAITools(chatMode, mcpTools)
+	const potentialTools = chatMode !== null ? openAITools(chatMode) : null
 	const nativeToolsObj = potentialTools && specialToolFormat === 'openai-style' ?
 		{ tools: potentialTools } as const
 		: {}
 
 	// instance
 	const openai: OpenAI = await newOpenAICompatibleSDK({ providerName, settingsOfProvider, includeInPayload })
-	if (providerName === 'microsoftAzure') {
-		// Required to select the model
-		(openai as AzureOpenAI).deploymentName = modelName;
-	}
 	const options: OpenAI.Chat.Completions.ChatCompletionCreateParamsStreaming = {
 		model: modelName,
 		messages: messages as any,
@@ -321,7 +283,7 @@ const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onE
 
 	// manually parse out tool results if XML
 	if (!specialToolFormat) {
-		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode, mcpTools)
+		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode)
 		onText = newOnText
 		onFinalMessage = newOnFinalMessage
 	}
@@ -366,7 +328,7 @@ const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onE
 				onText({
 					fullText: fullTextSoFar,
 					fullReasoning: fullReasoningSoFar,
-					toolCall: !toolName ? undefined : { name: toolName, rawParams: {}, isDone: false, doneParams: [], id: toolId },
+					toolCall: isAToolName(toolName) ? { name: toolName, rawParams: {}, isDone: false, doneParams: [], id: toolId } : undefined,
 				})
 
 			}
@@ -375,7 +337,7 @@ const _sendOpenAICompatibleChat = async ({ messages, onText, onFinalMessage, onE
 				onError({ message: 'Void: Response from model was empty.', fullError: null })
 			}
 			else {
-				const toolCall = rawToolCallObjOfParamsStr(toolName, toolParamsStr, toolId)
+				const toolCall = rawToolCallObjOf(toolName, toolParamsStr, toolId)
 				const toolCallObj = toolCall ? { toolCall } : {}
 				onFinalMessage({ fullText: fullTextSoFar, fullReasoning: fullReasoningSoFar, anthropicReasoning: null, ...toolCallObj });
 			}
@@ -441,8 +403,8 @@ const toAnthropicTool = (toolInfo: InternalToolInfo) => {
 	} satisfies Anthropic.Messages.Tool
 }
 
-const anthropicTools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[] | undefined) => {
-	const allowedTools = availableTools(chatMode, mcpTools)
+const anthropicTools = (chatMode: ChatMode) => {
+	const allowedTools = availableTools(chatMode)
 	if (!allowedTools || Object.keys(allowedTools).length === 0) return null
 
 	const anthropicTools: Anthropic.Messages.ToolUnion[] = []
@@ -452,10 +414,20 @@ const anthropicTools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[]
 	return anthropicTools
 }
 
-
+const anthropicToolToRawToolCallObj = (toolBlock: Anthropic.Messages.ToolUseBlock): RawToolCallObj | null => {
+	const { id, name, input } = toolBlock
+	if (!isAToolName(name)) return null
+	const rawParams: RawToolParamsObj = {}
+	if (input === null) return null
+	if (typeof input !== 'object') return null
+	for (const paramName in voidTools[name].params) {
+		rawParams[paramName as ToolParamName] = (input as any)[paramName]
+	}
+	return { id, name, rawParams, doneParams: Object.keys(rawParams) as ToolParamName[], isDone: true }
+}
 
 // ------------ ANTHROPIC ------------
-const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, overridesOfModel, modelName: modelName_, _setAborter, separateSystemMessage, chatMode, mcpTools }: SendChatParams_Internal) => {
+const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, overridesOfModel, modelName: modelName_, _setAborter, separateSystemMessage, chatMode }: SendChatParams_Internal) => {
 	const {
 		modelName,
 		specialToolFormat,
@@ -472,7 +444,7 @@ const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessag
 	const maxTokens = getReservedOutputTokenSpace(providerName, modelName_, { isReasoningEnabled: !!reasoningInfo?.isReasoningEnabled, overridesOfModel })
 
 	// tools
-	const potentialTools = anthropicTools(chatMode, mcpTools)
+	const potentialTools = chatMode !== null ? anthropicTools(chatMode) : null
 	const nativeToolsObj = potentialTools && specialToolFormat === 'anthropic-style' ?
 		{ tools: potentialTools, tool_choice: { type: 'auto' } } as const
 		: {}
@@ -496,7 +468,7 @@ const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessag
 
 	// manually parse out tool results if XML
 	if (!specialToolFormat) {
-		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode, mcpTools)
+		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode)
 		onText = newOnText
 		onFinalMessage = newOnFinalMessage
 	}
@@ -513,7 +485,7 @@ const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessag
 		onText({
 			fullText,
 			fullReasoning,
-			toolCall: !fullToolName ? undefined : { name: fullToolName, rawParams: {}, isDone: false, doneParams: [], id: 'dummy' },
+			toolCall: isAToolName(fullToolName) ? { name: fullToolName, rawParams: {}, isDone: false, doneParams: [], id: 'dummy' } : undefined,
 		})
 	}
 	// there are no events for tool_use, it comes in at the end
@@ -563,11 +535,8 @@ const sendAnthropicChat = async ({ messages, providerName, onText, onFinalMessag
 	stream.on('finalMessage', (response) => {
 		const anthropicReasoning = response.content.filter(c => c.type === 'thinking' || c.type === 'redacted_thinking')
 		const tools = response.content.filter(c => c.type === 'tool_use')
-		// console.log('TOOLS!!!!!!', JSON.stringify(tools, null, 2))
-		// console.log('TOOLS!!!!!!', JSON.stringify(response, null, 2))
-		const toolCall = tools[0] && rawToolCallObjOfAnthropicParams(tools[0])
+		const toolCall = tools[0] && anthropicToolToRawToolCallObj(tools[0])
 		const toolCallObj = toolCall ? { toolCall } : {}
-
 		onFinalMessage({ fullText, fullReasoning, anthropicReasoning, ...toolCallObj })
 	})
 	// on error
@@ -700,8 +669,8 @@ const toGeminiFunctionDecl = (toolInfo: InternalToolInfo) => {
 	} satisfies FunctionDeclaration
 }
 
-const geminiTools = (chatMode: ChatMode | null, mcpTools: InternalToolInfo[] | undefined): GeminiTool[] | null => {
-	const allowedTools = availableTools(chatMode, mcpTools)
+const geminiTools = (chatMode: ChatMode): GeminiTool[] | null => {
+	const allowedTools = availableTools(chatMode)
 	if (!allowedTools || Object.keys(allowedTools).length === 0) return null
 	const functionDecls: FunctionDeclaration[] = []
 	for (const t in allowedTools ?? {}) {
@@ -727,7 +696,6 @@ const sendGeminiChat = async ({
 	providerName,
 	modelSelectionOptions,
 	chatMode,
-	mcpTools,
 }: SendChatParams_Internal) => {
 
 	if (providerName !== 'gemini') throw new Error(`Sending Gemini chat, but provider was ${providerName}`)
@@ -753,7 +721,7 @@ const sendGeminiChat = async ({
 			: undefined
 
 	// tools
-	const potentialTools = geminiTools(chatMode, mcpTools)
+	const potentialTools = chatMode !== null ? geminiTools(chatMode) : undefined
 	const toolConfig = potentialTools && specialToolFormat === 'gemini-style' ?
 		potentialTools
 		: undefined
@@ -764,7 +732,7 @@ const sendGeminiChat = async ({
 
 	// manually parse out tool results if XML
 	if (!specialToolFormat) {
-		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode, mcpTools)
+		const { newOnText, newOnFinalMessage } = extractXMLToolsWrapper(onText, onFinalMessage, chatMode)
 		onText = newOnText
 		onFinalMessage = newOnFinalMessage
 	}
@@ -811,7 +779,7 @@ const sendGeminiChat = async ({
 				onText({
 					fullText: fullTextSoFar,
 					fullReasoning: fullReasoningSoFar,
-					toolCall: !toolName ? undefined : { name: toolName, rawParams: {}, isDone: false, doneParams: [], id: toolId },
+					toolCall: isAToolName(toolName) ? { name: toolName, rawParams: {}, isDone: false, doneParams: [], id: toolId } : undefined,
 				})
 			}
 
@@ -820,7 +788,7 @@ const sendGeminiChat = async ({
 				onError({ message: 'Void: Response from model was empty.', fullError: null })
 			} else {
 				if (!toolId) toolId = generateUuid() // ids are empty, but other providers might expect an id
-				const toolCall = rawToolCallObjOfParamsStr(toolName, toolParamsStr, toolId)
+				const toolCall = rawToolCallObjOf(toolName, toolParamsStr, toolId)
 				const toolCallObj = toolCall ? { toolCall } : {}
 				onFinalMessage({ fullText: fullTextSoFar, fullReasoning: fullReasoningSoFar, anthropicReasoning: null, ...toolCallObj });
 			}
@@ -932,12 +900,6 @@ export const sendLLMMessageToProviderImplementation = {
 		sendFIM: null,
 		list: null,
 	},
-	awsBedrock: {
-		sendChat: (params) => _sendOpenAICompatibleChat(params),
-		sendFIM: null,
-		list: null,
-	},
-
 } satisfies CallFnOfProvider
 
 

src/vs/workbench/contrib/void/electron-main/llmMessage/sendLLMMessage.ts

@@ -23,7 +23,6 @@ export const sendLLMMessage = async ({
 	overridesOfModel,
 	chatMode,
 	separateSystemMessage,
-	mcpTools,
 }: SendLLMMessageParams,
 
 	metricsService: IMetricsService
@@ -108,7 +107,7 @@ export const sendLLMMessage = async ({
 		}
 		const { sendFIM, sendChat } = implementation
 		if (messagesType === 'chatMessages') {
-			await sendChat({ messages: messages_, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, overridesOfModel, modelName, _setAborter, providerName, separateSystemMessage, chatMode, mcpTools })
+			await sendChat({ messages: messages_, onText, onFinalMessage, onError, settingsOfProvider, modelSelectionOptions, overridesOfModel, modelName, _setAborter, providerName, separateSystemMessage, chatMode })
 			return
 		}
 		if (messagesType === 'FIMMessage') {

src/vs/workbench/contrib/void/electron-main/mcpChannel.ts

@@ -1,395 +0,0 @@
-/*--------------------------------------------------------------------------------------
- *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
- *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
- *--------------------------------------------------------------------------------------*/
-
-// registered in app.ts
-// can't make a service responsible for this, because it needs
-// to be connected to the main process and node dependencies
-
-import { IServerChannel } from '../../../../base/parts/ipc/common/ipc.js';
-import { Emitter, Event } from '../../../../base/common/event.js';
-import { Client } from '@modelcontextprotocol/sdk/client/index.js';
-import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js';
-import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js';
-import { SSEClientTransport } from '@modelcontextprotocol/sdk/client/sse.js';
-import { MCPConfigFileJSON, MCPConfigFileEntryJSON, MCPServer, RawMCPToolCall, MCPToolErrorResponse, MCPServerEventResponse, MCPToolCallParams, removeMCPToolNamePrefix } from '../common/mcpServiceTypes.js';
-import { Transport } from '@modelcontextprotocol/sdk/shared/transport.js';
-import { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
-import { MCPUserStateOfName } from '../common/voidSettingsTypes.js';
-
-const getClientConfig = (serverName: string) => {
-	return {
-		name: `${serverName}-client`,
-		version: '0.1.0',
-		// debug: true,
-	}
-}
-
-type MCPServerNonError = MCPServer & { status: Omit<MCPServer['status'], 'error'> }
-type MCPServerError = MCPServer & { status: 'error' }
-
-
-
-type ClientInfo = {
-	_client: Client, // _client is the client that connects with an mcp client. We're calling mcp clients "server" everywhere except here for naming consistency.
-	mcpServerEntryJSON: MCPConfigFileEntryJSON,
-	mcpServer: MCPServerNonError,
-} | {
-	_client?: undefined,
-	mcpServerEntryJSON: MCPConfigFileEntryJSON,
-	mcpServer: MCPServerError,
-}
-
-type InfoOfClientId = {
-	[clientId: string]: ClientInfo
-}
-
-export class MCPChannel implements IServerChannel {
-
-	private readonly infoOfClientId: InfoOfClientId = {}
-	private readonly _refreshingServerNames: Set<string> = new Set()
-
-	// mcp emitters
-	private readonly mcpEmitters = {
-		serverEvent: {
-			onAdd: new Emitter<MCPServerEventResponse>(),
-			onUpdate: new Emitter<MCPServerEventResponse>(),
-			onDelete: new Emitter<MCPServerEventResponse>(),
-		}
-	} satisfies {
-		serverEvent: {
-			onAdd: Emitter<MCPServerEventResponse>,
-			onUpdate: Emitter<MCPServerEventResponse>,
-			onDelete: Emitter<MCPServerEventResponse>,
-		}
-	}
-
-	constructor(
-	) { }
-
-	// browser uses this to listen for changes
-	listen(_: unknown, event: string): Event<any> {
-
-		// server events
-		if (event === 'onAdd_server') return this.mcpEmitters.serverEvent.onAdd.event;
-		else if (event === 'onUpdate_server') return this.mcpEmitters.serverEvent.onUpdate.event;
-		else if (event === 'onDelete_server') return this.mcpEmitters.serverEvent.onDelete.event;
-		// else if (event === 'onLoading_server') return this.mcpEmitters.serverEvent.onChangeLoading.event;
-
-		// tool call events
-
-		// handle unknown events
-		else throw new Error(`Event not found: ${event}`);
-	}
-
-	// browser uses this to call (see this.channel.call() in mcpConfigService.ts for all usages)
-	async call(_: unknown, command: string, params: any): Promise<any> {
-		try {
-			if (command === 'refreshMCPServers') {
-				await this._refreshMCPServers(params)
-			}
-			else if (command === 'closeAllMCPServers') {
-				await this._closeAllMCPServers()
-			}
-			else if (command === 'toggleMCPServer') {
-				await this._toggleMCPServer(params.serverName, params.isOn)
-			}
-			else if (command === 'callTool') {
-				const p: MCPToolCallParams = params
-				const response = await this._safeCallTool(p.serverName, p.toolName, p.params)
-				return response
-			}
-			else {
-				throw new Error(`Void sendLLM: command "${command}" not recognized.`)
-			}
-		}
-		catch (e) {
-			console.error('mcp channel: Call Error:', e)
-		}
-	}
-
-	// server functions
-
-
-	private async _refreshMCPServers(params: { mcpConfigFileJSON: MCPConfigFileJSON, userStateOfName: MCPUserStateOfName, addedServerNames: string[], removedServerNames: string[], updatedServerNames: string[] }) {
-
-		const {
-			mcpConfigFileJSON,
-			userStateOfName,
-			addedServerNames,
-			removedServerNames,
-			updatedServerNames,
-		} = params
-
-		const { mcpServers: mcpServersJSON } = mcpConfigFileJSON
-
-		const allChanges: { type: 'added' | 'removed' | 'updated', serverName: string }[] = [
-			...addedServerNames.map(n => ({ serverName: n, type: 'added' }) as const),
-			...removedServerNames.map(n => ({ serverName: n, type: 'removed' }) as const),
-			...updatedServerNames.map(n => ({ serverName: n, type: 'updated' }) as const),
-		]
-
-		await Promise.all(
-			allChanges.map(async ({ serverName, type }) => {
-
-				// check if already refreshing
-				if (this._refreshingServerNames.has(serverName)) return
-				this._refreshingServerNames.add(serverName)
-
-				const prevServer = this.infoOfClientId[serverName]?.mcpServer;
-
-				// close and delete the old client
-				if (type === 'removed' || type === 'updated') {
-					await this._closeClient(serverName)
-					delete this.infoOfClientId[serverName]
-					this.mcpEmitters.serverEvent.onDelete.fire({ response: { prevServer, name: serverName, } })
-				}
-
-				// create a new client
-				if (type === 'added' || type === 'updated') {
-					const clientInfo = await this._createClient(mcpServersJSON[serverName], serverName, userStateOfName[serverName]?.isOn)
-					this.infoOfClientId[serverName] = clientInfo
-					this.mcpEmitters.serverEvent.onAdd.fire({ response: { newServer: clientInfo.mcpServer, name: serverName, } })
-				}
-			})
-		)
-
-		allChanges.forEach(({ serverName, type }) => {
-			this._refreshingServerNames.delete(serverName)
-		})
-
-	}
-
-	private async _createClientUnsafe(server: MCPConfigFileEntryJSON, serverName: string, isOn: boolean): Promise<ClientInfo> {
-
-		const clientConfig = getClientConfig(serverName)
-		const client = new Client(clientConfig)
-		let transport: Transport;
-		let info: MCPServerNonError;
-
-		if (server.url) {
-			// first try HTTP, fall back to SSE
-			try {
-				transport = new StreamableHTTPClientTransport(server.url);
-				await client.connect(transport);
-				console.log(`Connected via HTTP to ${serverName}`);
-				const { tools } = await client.listTools()
-				const toolsWithUniqueName = tools.map(({ name, ...rest }) => ({ name: this._addUniquePrefix(name), ...rest }))
-				info = {
-					status: isOn ? 'success' : 'offline',
-					tools: toolsWithUniqueName,
-					command: server.url.toString(),
-				}
-			} catch (httpErr) {
-				console.warn(`HTTP failed for ${serverName}, trying SSE…`, httpErr);
-				transport = new SSEClientTransport(server.url);
-				await client.connect(transport);
-				const { tools } = await client.listTools()
-				const toolsWithUniqueName = tools.map(({ name, ...rest }) => ({ name: this._addUniquePrefix(name), ...rest }))
-				console.log(`Connected via SSE to ${serverName}`);
-				info = {
-					status: isOn ? 'success' : 'offline',
-					tools: toolsWithUniqueName,
-					command: server.url.toString(),
-				}
-			}
-		} else if (server.command) {
-			// console.log('ENV DATA: ', server.env)
-			transport = new StdioClientTransport({
-				command: server.command,
-				args: server.args,
-				env: {
-					...server.env,
-					...process.env
-				} as Record<string, string>,
-			});
-
-			await client.connect(transport)
-
-			// Get the tools from the server
-			const { tools } = await client.listTools()
-			const toolsWithUniqueName = tools.map(({ name, ...rest }) => ({ name: this._addUniquePrefix(name), ...rest }))
-
-			// Create a full command string for display
-			const fullCommand = `${server.command} ${server.args?.join(' ') || ''}`
-
-			// Format server object
-			info = {
-				status: isOn ? 'success' : 'offline',
-				tools: toolsWithUniqueName,
-				command: fullCommand,
-			}
-
-		} else {
-			throw new Error(`No url or command for server ${serverName}`);
-		}
-
-
-		return { _client: client, mcpServerEntryJSON: server, mcpServer: info }
-	}
-
-	private _addUniquePrefix(base: string) {
-		return `${Math.random().toString(36).slice(2, 8)}_${base}`;
-	}
-
-	private async _createClient(serverConfig: MCPConfigFileEntryJSON, serverName: string, isOn = true): Promise<ClientInfo> {
-		try {
-			const c: ClientInfo = await this._createClientUnsafe(serverConfig, serverName, isOn)
-			return c
-		} catch (err) {
-			console.error(`❌ Failed to connect to server "${serverName}":`, err)
-			const fullCommand = !serverConfig.command ? '' : `${serverConfig.command} ${serverConfig.args?.join(' ') || ''}`
-			const c: MCPServerError = { status: 'error', error: err + '', command: fullCommand, }
-			return { mcpServerEntryJSON: serverConfig, mcpServer: c, }
-		}
-	}
-
-	private async _closeAllMCPServers() {
-		for (const serverName in this.infoOfClientId) {
-			await this._closeClient(serverName)
-			delete this.infoOfClientId[serverName]
-		}
-		console.log('Closed all MCP servers');
-	}
-
-	private async _closeClient(serverName: string) {
-		const info = this.infoOfClientId[serverName]
-		if (!info) return
-		const { _client: client } = info
-		if (client) {
-			await client.close()
-		}
-		console.log(`Closed MCP server ${serverName}`);
-	}
-
-
-	private async _toggleMCPServer(serverName: string, isOn: boolean) {
-		const prevServer = this.infoOfClientId[serverName]?.mcpServer
-		// Handle turning on the server
-		if (isOn) {
-			// this.mcpEmitters.serverEvent.onChangeLoading.fire(getLoadingServerObject(serverName, isOn))
-			const clientInfo = await this._createClientUnsafe(this.infoOfClientId[serverName].mcpServerEntryJSON, serverName, isOn)
-			this.mcpEmitters.serverEvent.onUpdate.fire({
-				response: {
-					name: serverName,
-					newServer: clientInfo.mcpServer,
-					prevServer: prevServer,
-				}
-			})
-		}
-		// Handle turning off the server
-		else {
-			// this.mcpEmitters.serverEvent.onChangeLoading.fire(getLoadingServerObject(serverName, isOn))
-			this._closeClient(serverName)
-			delete this.infoOfClientId[serverName]._client
-
-			this.mcpEmitters.serverEvent.onUpdate.fire({
-				response: {
-					name: serverName,
-					newServer: {
-						status: 'offline',
-						tools: [],
-						command: '',
-						// Explicitly set error to undefined to reset the error state
-						error: undefined,
-					},
-					prevServer: prevServer,
-				}
-			})
-		}
-	}
-
-	// tool call functions
-
-	private async _callTool(serverName: string, toolName: string, params: any): Promise<RawMCPToolCall> {
-		const server = this.infoOfClientId[serverName]
-		if (!server) throw new Error(`Server ${serverName} not found`)
-		const { _client: client } = server
-		if (!client) throw new Error(`Client for server ${serverName} not found`)
-
-		// Call the tool with the provided parameters
-		const response = await client.callTool({
-			name: removeMCPToolNamePrefix(toolName),
-			arguments: params
-		})
-		const { content } = response as CallToolResult
-		const returnValue = content[0]
-
-		if (returnValue.type === 'text') {
-			// handle text response
-
-			if (response.isError) {
-				throw new Error(`Tool call error: ${returnValue.text}`)
-			}
-
-			// handle success
-			return {
-				event: 'text',
-				text: returnValue.text,
-				toolName,
-				serverName,
-			}
-		}
-
-		// if (returnValue.type === 'audio') {
-		// 	// handle audio response
-		// }
-
-		// if (returnValue.type === 'image') {
-		// 	// handle image response
-		// }
-
-		// if (returnValue.type === 'resource') {
-		// 	// handle resource response
-		// }
-
-		throw new Error(`Tool call error: We don\'t support ${returnValue.type} tool response yet for tool ${toolName} on server ${serverName}`)
-	}
-
-	// tool call error wrapper
-	private async _safeCallTool(serverName: string, toolName: string, params: any): Promise<RawMCPToolCall> {
-		try {
-			const response = await this._callTool(serverName, toolName, params)
-			return response
-		} catch (err) {
-
-			let errorMessage: string;
-
-			if (typeof err === 'object' && err !== null && err['code']) {
-				const code = err.code
-				let codeDescription = ''
-				if (code === -32700)
-					codeDescription = 'Parse Error';
-				if (code === -32600)
-					codeDescription = 'Invalid Request';
-				if (code === -32601)
-					codeDescription = 'Method Not Found';
-				if (code === -32602)
-					codeDescription = 'Invalid Parameters';
-				if (code === -32603)
-					codeDescription = 'Internal Error';
-				errorMessage = `${codeDescription}. Full response:\n${JSON.stringify(err, null, 2)}`
-			}
-			// Check if it's an MCP error with a code
-			else if (typeof err === 'string') {
-				// String error
-				errorMessage = err;
-			} else {
-				// Unknown error format
-				errorMessage = JSON.stringify(err, null, 2);
-			}
-
-			const fullErrorMessage = `❌ Failed to call tool "${toolName}" on server "${serverName}": ${errorMessage}`;
-			const errorResponse: MCPToolErrorResponse = {
-				event: 'error',
-				text: fullErrorMessage,
-				toolName,
-				serverName,
-			}
-			return errorResponse
-		}
-	}
-}
-
-

src/vs/workbench/contrib/void/electron-main/metricsMainService.ts

@@ -13,7 +13,6 @@ import { IApplicationStorageMainService } from '../../../../platform/storage/ele
 
 import { IMetricsService } from '../common/metricsService.js';
 import { PostHog } from 'posthog-node'
-import { OPT_OUT_KEY } from '../common/storageKeys.js';
 
 
 const os = isWindows ? 'windows' : isMacintosh ? 'mac' : isLinux ? 'linux' : null
@@ -30,8 +29,6 @@ const osInfo = _getOSInfo()
 
 // we'd like to use devDeviceId on telemetryService, but that gets sanitized by the time it gets here as 'someValue.devDeviceId'
 
-
-
 export class MetricsMainService extends Disposable implements IMetricsService {
 	_serviceBrand: undefined;
 
@@ -122,18 +119,7 @@ export class MetricsMainService extends Disposable implements IMetricsService {
 			distinctId: this.distinctId,
 			properties: this._initProperties,
 		}
-
-		const didOptOut = this._appStorage.getBoolean(OPT_OUT_KEY, StorageScope.APPLICATION, false)
-
-		console.log('User is opted out of basic Void metrics?', didOptOut)
-		if (didOptOut) {
-			this.client.optOut()
-		}
-		else {
-			this.client.optIn()
-			this.client.identify(identifyMessage)
-		}
-
+		this.client.identify(identifyMessage)
 
 		console.log('Void posthog metrics info:', JSON.stringify(identifyMessage, null, 2))
 	}
@@ -141,18 +127,10 @@ export class MetricsMainService extends Disposable implements IMetricsService {
 
 	capture: IMetricsService['capture'] = (event, params) => {
 		const capture = { distinctId: this.distinctId, event, properties: params } as const
-		// console.log('full capture:', this.distinctId)
+		// console.log('full capture:', capture)
 		this.client.capture(capture)
 	}
 
-	setOptOut: IMetricsService['setOptOut'] = (newVal: boolean) => {
-		if (newVal) {
-			this._appStorage.store(OPT_OUT_KEY, 'true', StorageScope.APPLICATION, StorageTarget.MACHINE)
-		}
-		else {
-			this._appStorage.remove(OPT_OUT_KEY, StorageScope.APPLICATION)
-		}
-	}
 
 	async getDebuggingProperties() {
 		return this._initProperties

src/vs/workbench/contrib/void/electron-main/voidSCMMainService.ts

@@ -1,82 +0,0 @@
-/*--------------------------------------------------------------------------------------
- *  Copyright 2025 Glass Devtools, Inc. All rights reserved.
- *  Licensed under the Apache License, Version 2.0. See LICENSE.txt for more information.
- *--------------------------------------------------------------------------------------*/
-
-import { promisify } from 'util'
-import { exec as _exec } from 'child_process'
-import { IVoidSCMService } from '../common/voidSCMTypes.js'
-
-interface NumStat {
-	file: string
-	added: number
-	removed: number
-}
-
-const exec = promisify(_exec)
-
-//8000 and 10 were chosen after some experimentation on small-to-moderately sized changes
-const MAX_DIFF_LENGTH = 8000
-const MAX_DIFF_FILES = 10
-
-const git = async (command: string, path: string): Promise<string> => {
-	const { stdout, stderr } = await exec(`${command}`, { cwd: path })
-	if (stderr) {
-		throw new Error(stderr)
-	}
-	return stdout.trim()
-}
-
-const getNumStat = async (path: string, useStagedChanges: boolean): Promise<NumStat[]> => {
-	const staged = useStagedChanges ? '--staged' : ''
-	const output = await git(`git diff --numstat ${staged}`, path)
-	return output
-		.split('\n')
-		.map((line) => {
-			const [added, removed, file] = line.split('\t')
-			return {
-				file,
-				added: parseInt(added, 10) || 0,
-				removed: parseInt(removed, 10) || 0,
-			}
-		})
-}
-
-const getSampledDiff = async (file: string, path: string, useStagedChanges: boolean): Promise<string> => {
-	const staged = useStagedChanges ? '--staged' : ''
-	const diff = await git(`git diff --unified=0 --no-color ${staged} -- "${file}"`, path)
-	return diff.slice(0, MAX_DIFF_LENGTH)
-}
-
-const hasStagedChanges = async (path: string): Promise<boolean> => {
-	const output = await git('git diff --staged --name-only', path)
-	return output.length > 0
-}
-
-export class VoidSCMService implements IVoidSCMService {
-	readonly _serviceBrand: undefined
-
-	async gitStat(path: string): Promise<string> {
-		const useStagedChanges = await hasStagedChanges(path)
-		const staged = useStagedChanges ? '--staged' : ''
-		return git(`git diff --stat ${staged}`, path)
-	}
-
-	async gitSampledDiffs(path: string): Promise<string> {
-		const useStagedChanges = await hasStagedChanges(path)
-		const numStatList = await getNumStat(path, useStagedChanges)
-		const topFiles = numStatList
-			.sort((a, b) => (b.added + b.removed) - (a.added + a.removed))
-			.slice(0, MAX_DIFF_FILES)
-		const diffs = await Promise.all(topFiles.map(async ({ file }) => ({ file, diff: await getSampledDiff(file, path, useStagedChanges) })))
-		return diffs.map(({ file, diff }) => `==== ${file} ====\n${diff}`).join('\n\n')
-	}
-
-	gitBranch(path: string): Promise<string> {
-		return git('git branch --show-current', path)
-	}
-
-	gitLog(path: string): Promise<string> {
-		return git('git log --pretty=format:"%h|%s|%ad" --date=short --no-merges -n 5', path)
-	}
-}

src/vs/workbench/contrib/void/electron-main/voidUpdateMainService.ts

@@ -32,12 +32,6 @@ export class VoidMainUpdateService extends Disposable implements IVoidUpdateServ
 			return { message: null } as const
 		}
 
-		// if disabled and not explicitly checking, return early
-		if (this._updateService.state.type === StateType.Disabled) {
-			if (!explicit)
-				return { message: null } as const
-		}
-
 		this._updateService.checkForUpdates(false) // implicity check, then handle result ourselves
 
 		console.log('updateState', this._updateService.state)