Skip to content

Adding smart edit#2832

Closed
SimplyYourAverageDev wants to merge 0 commit into
anomalyco:devfrom
SimplyYourAverageDev:dev
Closed

Adding smart edit#2832
SimplyYourAverageDev wants to merge 0 commit into
anomalyco:devfrom
SimplyYourAverageDev:dev

Conversation

@SimplyYourAverageDev
Copy link
Copy Markdown

This PR adds "smart edit" which basically uses Morph V3 Large (an LLM specified for applying code edits) so that the main LLM can sometimes apply edits that are hard to describe.

@jenslys
Copy link
Copy Markdown

jenslys commented Sep 28, 2025

Does this supersede #1916 ?

@SimplyYourAverageDev
Copy link
Copy Markdown
Author

Does this supersede #1916 ?

Well I'm not sure, their implementation only supports Morph API and also doesn't do diff at all so...

@SimplyYourAverageDev
Copy link
Copy Markdown
Author

SimplyYourAverageDev commented Sep 30, 2025

@thdxr @adamdotdevin Any chance you would consier this merge?

@zenyr
Copy link
Copy Markdown
Contributor

zenyr commented Oct 7, 2025

Thanks for implementing the smart_edit tool—it seems great for handling complex code edits! Just a concern: since it uses OpenRouter's morph/morph-v3-large model, users not set up with OpenRouter might run into failures if the main LLM picks this tool, which could disrupt their experience. Perhaps we could make the tool available only when OpenRouter is configured, or adjust the LLM's tool selection to avoid preferring it over the existing edit tool?

@bhaktatejas922
Copy link
Copy Markdown
Contributor

#1916
@adamdotdevin either of these implementations look good to you?

@bhaktatejas922
Copy link
Copy Markdown
Contributor

@thdxr ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants