Switching to git log approach (#254)

Author: coryodaniel

__tests__/bundle_publish.test.ts

@@ -31,7 +31,7 @@ describe("bundle_publish", () => {
     it("should return true when directory has changes in HEAD commit", async () => {
       mockedExec.exec
         .mockImplementationOnce(async (cmd, args, options) => {
-          // diff-tree returns changed files
+          // git log returns changed files
           if (options?.listeners?.stdout) {
             options.listeners.stdout(Buffer.from("test-bundle/massdriver.yaml\ntest-bundle/src/main.tf\n"))
           }
@@ -41,15 +41,15 @@ describe("bundle_publish", () => {
 
       await bundlePublish()
 
-      // Verify diff-tree was called to check HEAD commit
+      // Verify git log was called to check HEAD commit
       expect(mockedExec.exec).toHaveBeenCalledWith(
         "git",
-        ["diff-tree", "--no-commit-id", "--name-only", "-r", "HEAD", "--", "test-bundle"],
+        ["log", "--format=", "--name-only", "-1", "HEAD", "--", "test-bundle"],
         expect.objectContaining({
           ignoreReturnCode: true,
-          silent: true,
           listeners: expect.objectContaining({
-            stdout: expect.any(Function)
+            stdout: expect.any(Function),
+            stderr: expect.any(Function)
           })
         })
       )
@@ -69,7 +69,7 @@ describe("bundle_publish", () => {
     it("should skip publish when no changes detected in HEAD commit", async () => {
       mockedExec.exec
         .mockImplementationOnce(async (cmd, args, options) => {
-          // diff-tree returns no files (empty output)
+          // git log returns no files (empty output)
           if (options?.listeners?.stdout) {
             options.listeners.stdout(Buffer.from(""))
           }

dist/bundle_publish/index.js

@@ -8,35 +8,41 @@ import * as exec from "@actions/exec"
 const hasChangesInDirectory = async (
   directory: string
 ): Promise<boolean> => {
+  core.info(`====== [hasChanges] START ======`)
   core.info(`[hasChanges] Checking for changes in directory: ${directory}`)
 
-  // Check what files changed in the current commit (HEAD)
-  // This works even with shallow clones (fetch-depth: 1)
-  core.info(`[hasChanges] Checking files changed in HEAD commit in ${directory}...`)
+  // Use git log to get files changed in HEAD commit - works with shallow clones
+  core.info(`[hasChanges] Running: git log --format="" --name-only -1 HEAD -- ${directory}`)
 
   let changedFiles = ""
-  const diffTreeExitCode = await exec.exec(
+  await exec.exec(
     "git",
-    ["diff-tree", "--no-commit-id", "--name-only", "-r", "HEAD", "--", directory],
+    ["log", "--format=", "--name-only", "-1", "HEAD", "--", directory],
     {
       ignoreReturnCode: true,
-      silent: true,
       listeners: {
         stdout: (data: Buffer) => {
           changedFiles += data.toString()
+        },
+        stderr: (data: Buffer) => {
+          core.info(`[hasChanges] git stderr: ${data.toString()}`)
         }
       }
     }
   )
 
-  // diff-tree returns 0 even if no files changed, so check the output
+  core.info(`[hasChanges] Raw output length: ${changedFiles.length}`)
+  core.info(`[hasChanges] Raw output (first 200 chars): ${changedFiles.substring(0, 200)}`)
+  
+  // Check if any files were returned
   if (changedFiles.trim().length > 0) {
     const fileCount = changedFiles.trim().split('\n').length
     core.info(`[hasChanges] ✓ Found ${fileCount} changed file(s) in ${directory} in HEAD commit`)
     return true
   }
 
   core.info(`[hasChanges] ✗ No changes detected in ${directory} in HEAD commit`)
+  core.info(`====== [hasChanges] END ======`)
   return false
 }