continue icon indicating copy to clipboard operation
continue copied to clipboard

feat: sonnet lazy apply prompt should not change comments

Open uinstinct opened this issue 2 months ago • 2 comments

Description

Change the lazy apply prompt for claude sonnet models to not change comments.

resolves CON-2084

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Updated the lazy apply prompt for Claude Sonnet models and all edit/apply prompts to preserve comments unless changes require modifying them. Aligns with CON-2084 by preventing unintended comment edits across lazy applies and code edits.

Written for commit 3c85721ef755366371a71c60d732e6382931ac42. Summary will update automatically on new commits.

uinstinct avatar Nov 10 '25 12:11 uinstinct

@uinstinct could we knock this out for all lazy apply prompts in one go? Or if already done add a comment about why only this one?

Sure. Lazy apply prompt only works for sonnet models. Hence only one was changed.

uinstinct avatar Nov 11 '25 03:11 uinstinct

Ah you're right I just mean all apply and edit prompts. Removing comments will be an issue for all of them but especially apply, check intellij buildApplyPrompt, apply manager getApplyPrompt. I would also tweak all edit prompts (core/llm/templates/edit) if they say nothing about comments. I think there's 20 ish prompts in core/llm/templates/edit. Some may not even be reachable anymore, if so can skip

RomneyDa avatar Nov 12 '25 00:11 RomneyDa

:tada: This PR is included in version 1.29.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 18 '25 18:11 sestinj

:tada: This PR is included in version 1.34.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 19 '25 01:11 sestinj

:tada: This PR is included in version 1.5.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 19 '25 04:11 sestinj

:tada: This PR is included in version 1.6.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 20 '25 21:11 sestinj