Background - fix issue with background agent that adds a file

[lszomoru]

Related microsoft/vscode#288028

[Copilot]

Pull request overview

This PR fixes an issue where the background agent incorrectly passes an originalUri when creating a ChatSessionChangedFile for newly added files. When a file is added (status INDEX_ADDED), there is no original version to compare against, so the originalUri should be undefined.

Changes:

• Added conditional logic to set originalUri to undefined when the git change status is INDEX_ADDED (value 1)
• Applied the same fix in two code paths: one for uncommitted changes and one for committed changes

Comments suppressed due to low confidence (1)

src/extension/chatSessions/vscode-node/chatSessionWorktreeServiceImpl.ts:290

• Consider importing the Status enum from the git API and using Status.INDEX_ADDED instead of the magic number 1. This would make the code more maintainable and less prone to errors. The Status enum is available from the git extension API types.

				change.status !== 1 /* INDEX_ADDED */
					? toGitUri(change.originalUri, worktreeProperties.baseCommit)
					: undefined);

[Copilot]

Consider importing the Status enum from the git API and using Status.INDEX_ADDED instead of the magic number 1. This would make the code more maintainable and less prone to errors. The Status enum is available from the git extension API types.

This issue also appears in the following locations of the same file:

• line 288