Skip to content

Conversation

pjhampton
Copy link

@pjhampton pjhampton commented Sep 19, 2025

Description

Adds a new environment variable EDITOR_DISABLE_BACKUP that allows users to skip the creation of .bak backup files during editor operations. When set to "true", the editor tool will no longer create backup files for str_replace, pattern_replace, and insert commands, which can be useful in automated environments or when backup files are not desired.

The feature maintains backward compatibility - backup files are still created by default unless explicitly disabled.

Related Issues

n/a

Documentation PR

n/a

Type of Change

Bug fix
New Tool
Breaking change
Documentation update
Other (please describe): enhancement

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@pjhampton pjhampton requested a review from a team as a code owner September 19, 2025 12:25
@pjhampton pjhampton changed the title feat(editor): add EDITOR_DISABLE_BACKUP option to skip backup file cr… feat(editor): add option to skip backup file creation Sep 19, 2025
Copy link
Member

@cagataycali cagataycali left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! When multi agent systems working on same files, .bak files become friction than utility, with your changes that will no longer be a problem!

@pjhampton
Copy link
Author

pjhampton commented Sep 26, 2025

Ofc @cagataycali - thanks for the review. How can I get this merged/release so I can make use of it?

@pjhampton
Copy link
Author

Hey @cagataycali - kind nudge :)

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.

3 participants