Merge pull request microsoft#261095 from microsoft/copilot/fix-261016-2

Fix git diff completions on Windows by removing sed dependency

Author: Tyriar

extensions/terminal-suggest/src/completions/git.ts

@@ -464,36 +464,49 @@ export const gitGenerators = {
 	} satisfies Fig.Generator,
 
 	getStagedFiles: {
-		script: [
-			"bash",
-			"-c",
-			"git --no-optional-locks status --short | sed -ne '/^M /p' -e '/A /p'",
-		],
-		postProcess: postProcessTrackedFiles,
-	},
+		script: ["git", "--no-optional-locks", "status", "--short"],
+		postProcess: (out, context) => {
+			const output = filterMessages(out);
+
+			if (output.startsWith("fatal:")) {
+				return [];
+			}
+
+			const filteredLines = output.split("\n").filter(line => {
+				return line.match(/^M /) || line.match(/A /);
+			});
+
+			return postProcessTrackedFiles(filteredLines.join("\n"), context);
+		},
+	} satisfies Fig.Generator,
 
 	getUnstagedFiles: {
 		script: ["git", "--no-optional-locks", "diff", "--name-only"],
 		splitOn: "\n",
 	} satisfies Fig.Generator,
 
 	getChangedTrackedFiles: {
-		script: function (context) {
+		script: ["git", "--no-optional-locks", "status", "--short"],
+		postProcess: (out, context) => {
+			const output = filterMessages(out);
+
+			if (output.startsWith("fatal:")) {
+				return [];
+			}
+
+			let filteredLines;
 			if (context.includes("--staged") || context.includes("--cached")) {
-				return [
-					"bash",
-					"-c",
-					`git --no-optional-locks status --short | sed -ne '/^M /p' -e '/A /p'`,
-				];
+				filteredLines = output.split("\n").filter(line => {
+					return line.match(/^M /) || line.match(/A /);
+				});
 			} else {
-				return [
-					"bash",
-					"-c",
-					`git --no-optional-locks status --short | sed -ne '/M /p' -e '/A /p'`,
-				];
+				filteredLines = output.split("\n").filter(line => {
+					return line.match(/M /) || line.match(/A /);
+				});
 			}
+
+			return postProcessTrackedFiles(filteredLines.join("\n"), context);
 		},
-		postProcess: postProcessTrackedFiles,
 	} satisfies Fig.Generator,
 };