Agents - use pathspec file when adding files to the temporary index to help with performance

[lszomoru]

No description provided.

[github-actions]

Screenshot Changes

Base: 1eb30178 Current: bf3d9794

Changed (2)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Light

Before	After

editor/inlineCompletions/other/JumpToHint/Dark

Before	After

[Copilot]

Pull request overview

Improves performance of Agent Sessions’ “temp index” workflows by staging only the set of uncommitted files (via a generated pathspec file) instead of staging the entire working directory when computing diffs/snapshots.

Changes:

• Added getUncommittedFilePaths() helper to derive a deduped list of repo-relative changed paths.
• Updated worktree/workspace diff collection to use git add -A --pathspec-from-file=... when untracked files are present.
• Updated checkpoint creation to stage via a pathspec file rather than git add -A -- ..

Show a summary per file

File	Description
extensions/copilot/src/platform/git/vscode-node/utils.ts	Adds helper to compute repo-relative uncommitted paths for use in pathspec files.
extensions/copilot/src/extension/chatSessions/vscode-node/chatSessionWorktreeServiceImpl.ts	Uses the helper + pathspec file when staging into the temp index for diff stats in worktree isolation mode.
extensions/copilot/src/extension/chatSessions/vscode-node/chatSessionWorktreeCheckpointServiceImpl.ts	Uses the helper + pathspec file when staging into the temp index for checkpoint snapshot creation.
extensions/copilot/src/extension/chatSessions/vscode-node/chatSessionWorkspaceFolderServiceImpl.ts	Uses the helper + pathspec file when staging into the temp index for diff stats in workspace isolation mode.

Copilot's findings

Comments suppressed due to low confidence (1)

extensions/copilot/src/platform/git/vscode-node/utils.ts:121

• The paths returned here are later passed to git add via --pathspec-from-file. Git pathspecs treat *, ?, and [] as glob characters by default, so a file name containing those characters can match unintended paths. Consider emitting literal pathspecs (e.g. prefix each entry with :(literal)), or otherwise ensure the entries are interpreted literally when written to the pathspec file.

	const relativePaths = coalesce(Array.from(resources)
		.map(uri => relativePath(repository.rootUri, uri)));

	// Git expects forward slashes even on Windows
	return relativePaths.map(p => p.replace(/\\/g, '/'));

• Files reviewed: 4/4 changed files
• Comments generated: 3