Skip to content

Enable Replace String Tool For VSC Model C.#2344

Merged
karthiknadig merged 1 commit intomicrosoft:mainfrom
AbdelrahmanAbouelenin:ababouelenin/vsc_model_enable_replace_string
Dec 3, 2025
Merged

Enable Replace String Tool For VSC Model C.#2344
karthiknadig merged 1 commit intomicrosoft:mainfrom
AbdelrahmanAbouelenin:ababouelenin/vsc_model_enable_replace_string

Conversation

@AbdelrahmanAbouelenin
Copy link
Contributor

Enabling modelSupportsReplaceString and modelSupportsMultiReplaceString for vsc model c

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enables replace string tool support for a subset of VSC models (Model C) by adding a new model category and updating the modelSupportsMultiReplaceString capability check. This allows these specific models to use the multi_replace_string_in_file edit tool, expanding the set of models that can perform advanced string replacement operations.

Key changes:

  • Added new VSC Model C category with two model hashes
  • Updated modelSupportsMultiReplaceString to include VSC Model C
  • Created isVSCModelC helper function following existing patterns

karthiknadig
karthiknadig previously approved these changes Dec 3, 2025
@vs-code-engineering vs-code-engineering bot added this to the November 2025 milestone Dec 3, 2025
TylerLeonhardt
TylerLeonhardt previously approved these changes Dec 3, 2025
@AbdelrahmanAbouelenin AbdelrahmanAbouelenin force-pushed the ababouelenin/vsc_model_enable_replace_string branch from 9181aac to f562fd0 Compare December 3, 2025 05:23
@karthiknadig karthiknadig added this pull request to the merge queue Dec 3, 2025
Merged via the queue into microsoft:main with commit f3d4790 Dec 3, 2025
6 checks passed
hediet pushed a commit that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants