Revert "Git - adopt the new package to use copy-on-write for the worktree include files (#299583)"

[lszomoru]

This reverts commit 950ab07.

[Copilot]

Pull request overview

This PR reverts a prior change in the built-in Git extension related to copying “worktree include files”, removing the dependency on @vscode/fs-copyfile and returning to Node’s built-in copy implementation.

Changes:

• Remove @vscode/fs-copyfile dependency (and its lockfile entries).
• Switch worktree include file copying from @vscode/fs-copyfile to fsPromises.cp with COPYFILE_FICLONE.
• Revert the path-minimization logic used to decide what to copy into the worktree.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File	Description
extensions/git/src/repository.ts	Reverts worktree include file path selection and uses fsPromises.cp for copying into worktrees.
extensions/git/package.json	Drops the @vscode/fs-copyfile dependency.
extensions/git/package-lock.json	Removes @vscode/fs-copyfile and node-addon-api from the lockfile.

Files not reviewed (1)

• extensions/git/package-lock.json: Language not supported

Comments suppressed due to low confidence (1)

extensions/git/src/repository.ts:1941

• In _getWorktreeIncludePaths, the upward traversal loop uses !gitIgnoredFiles.has(dir) as its stop condition. Since gitIgnoredFiles only contains file paths (not directories), this condition will almost never stop early, so every file walks all the way up to this.root even if that directory chain was already processed. Consider using !gitIgnoredPaths.has(dir) (or a separate visited set) so directories already added short-circuit the traversal, reducing redundant work on large file sets.

		for (const filePath of gitIgnoredFiles) {
			let dir = path.dirname(filePath);
			while (dir !== this.root && !gitIgnoredFiles.has(dir)) {
				gitIgnoredPaths.add(dir);
				dir = path.dirname(dir);
			}