-
Notifications
You must be signed in to change notification settings - Fork 3.5k
chore: edit file tool improvements #7708
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@@ -1103,6 +1103,7 @@ export interface Tool { | |||
basePolicy: ToolPolicy, | |||
parsedArgs: Record<string, unknown>, | |||
) => ToolPolicy; | |||
skipToolArgsDisplay?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd usually advise negative names such as 'skip', 'doNot', 'shouldNot' etc. Instead, how about 'displayToolArgs' that defaults to true? Otherwise, this may result in expressions such as !!skipToolArgsDisplay
which are difficult to reason about.
@@ -135,6 +136,7 @@ interface StyledMarkdownPreviewProps { | |||
toolCallId?: string; | |||
expandCodeblocks?: boolean; | |||
collapsible?: boolean; | |||
removePadding?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto - instead of removedPadding
, you can name this padded
and default it to true
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments about naming
Description
Fix the edit file tool.
resolves CON-2405
AI Code Review
@continue-general-review
or@continue-detailed-review
Checklist
Screen recording or screenshot
before
after
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Streamlined the Edit File tool UI and added a way for tools to hide noisy args. Also removed padding around Create/Edit file code blocks for a tighter look. Resolves CON-2405.
New Features
Bug Fixes