[shared] Replace git.js with dependency "simple-git" (Azure#33967)

- fixes Azure#33942

Author: mikeharder

.github/package-lock.json

@@ -7,13 +7,16 @@
   },
   "dependencies": {
     "@apidevtools/json-schema-ref-parser": "^11.9.3",
+    "debug": "^4.4.0",
     "js-yaml": "^4.1.0",
-    "marked": "^15.0.7"
+    "marked": "^15.0.7",
+    "simple-git": "^3.27.0"
   },
   "devDependencies": {
     "@eslint/js": "^9.22.0",
     "@octokit/webhooks-types": "^7.5.1",
     "@tsconfig/node20": "^20.1.4",
+    "@types/debug": "^4.1.12",
     "@types/github-script": "github:actions/github-script",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^20.0.0",

.github/package.json

@@ -7,7 +7,6 @@
     "./changed-files": "./src/changed-files.js",
     "./equality": "./src/equality.js",
     "./exec": "./src/exec.js",
-    "./git": "./src/git.js",
     "./logger": "./src/logger.js",
     "./readme": "./src/readme.js",
     "./sdk-types": "./src/sdk-types.js",
@@ -20,8 +19,10 @@
   },
   "dependencies": {
     "@apidevtools/json-schema-ref-parser": "^11.9.3",
+    "debug": "^4.4.0",
     "js-yaml": "^4.1.0",
-    "marked": "^15.0.7"
+    "marked": "^15.0.7",
+    "simple-git": "^3.27.0"
   },
   "devDependencies": {
     "@eslint/js": "^9.22.0",

.github/shared/package-lock.json

@@ -1,6 +1,10 @@
 // @ts-check
 
-import { diff } from "./git.js";
+import debug from "debug";
+import { simpleGit } from "simple-git";
+
+// Enable simple-git debug logging to improve console output
+debug.enable("simple-git");
 
 /**
  * @param {Object} [options]
@@ -23,11 +27,11 @@ export async function getChangedFiles(options = {}) {
   // { name: "/foo/baz.js", status: Status.Renamed, previousName: "/foo/bar.js"}.
   // Then add filter functions to filter based on status.  This is more flexible and lets consumers
   // filter based on status with a single call to `git diff`.
-  const result = await diff(baseCommitish, headCommitish, {
-    args: ["--name-only"],
-    cwd,
-    logger: logger,
-  });
+  const result = await simpleGit(cwd).diff([
+    "--name-only",
+    baseCommitish,
+    headCommitish,
+  ]);
 
   const files = result.trim().split("\n");