"Accept All suggested changes" from the file editor doesn't work

[sshveta]

1. Ran analysis on coolstore app , resolutions found in resource.java and producer.java.
2. View Changes , it opens the file editor in comparison mode where you can accept suggested changes .
3. CLick on "Accept all changes in right " does not start new analysis.

Attached video :

2025-01-31.11-30-55.mp4

[rszwajko]

@sshveta
It's a known limitation. When using built-in merge/diff editors we require 2 actions after resolving conflicts:

1. the user needs to save the file (the editor will stay in dirty/modified state until such action is performed)
2. the user needs to manually discard the file from the list (applying would overwrite the the changes).

The reasons:

1. both editors are part of VS Code and we don't have much control over them - we could use a limited subset of their capabilities (like Continue) OR use an external tool for resolving the conflicts.
2. we are restricted by being SCM-agnostic - we could detect that underlying file has changed and mark the file as dirty but this requires some assumptions about source version control system (i.e. git hashes to identify start commit) or implementing equivalent functionality.

CC: @dymurray @jwmatthews