Skip to content

Tool replacement/management robustness#11

Open
fu5ha wants to merge 1 commit into
code-yeongyu:mainfrom
fu5ha:tool-replace-robustness
Open

Tool replacement/management robustness#11
fu5ha wants to merge 1 commit into
code-yeongyu:mainfrom
fu5ha:tool-replace-robustness

Conversation

@fu5ha
Copy link
Copy Markdown

@fu5ha fu5ha commented May 13, 2026

Summary

Right now, when /reloading a session with the apply_patch tool active and the standard edit and write tools already removed, the tool management/replacement logic thinks that there's no editor write or apply_patch tool and then doesn't add the apply_patch tool back.

For simplicity of logic, this PR takes the stance that if a user doesn't want any edit tools they should deactivate this extension:

The new logic is, if this extension is active, it will always add apply_patch when GPT-family model is active, and will remove apply_patch and add standard edit and write tools when GPT-family model is not active.


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

@fu5ha fu5ha requested a review from code-yeongyu as a code owner May 13, 2026 16:19
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

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.

1 participant