Expected Final Selection After Running Delete Duplicate Lines

[josephxiao8]

Fixes #234796

[josephxiao8]

@microsoft-github-policy-service agree

[Copilot]

Pull request overview

This PR fixes a selection column calculation bug in the "Delete Duplicate Lines" action. The issue occurred because the final selection's end column was using the string length directly instead of the proper 1-indexed column position (length + 1), which is the VS Code convention where getLineMaxColumn() returns lineLength + 1.

Key Changes

• Updated DeleteDuplicateLinesAction to use lines[lines.length - 1].length + 1 for the final selection's end column, aligning with VS Code's 1-indexed column system
• Corrected test expectations to match the fixed selection behavior

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/vs/editor/contrib/linesOperations/browser/linesOperations.ts	Fixed final selection end column calculation by adding +1 to account for 1-indexed columns
src/vs/editor/contrib/linesOperations/test/browser/linesOperations.test.ts	Updated test expectations for selection end columns from length to length+1 (line "omicron" from 7→8, line "beta" from 4→5)