From 3b2c7c1685211e1254fdb982e7ea8931a056c2ef Mon Sep 17 00:00:00 2001
From: Dotan Simha <dotan@the-guild.dev>
Date: Tue, 13 Jan 2026 17:10:49 +0200
Subject: [PATCH] Simplify unchanged file check in git.ts

Changed the order of the checks. This way only files that were actually changed are going to fail if they are executable/symlinks.

The current piece of code errors on untouched/unmodified/unrelated files.
---
 src/git.ts | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/git.ts b/src/git.ts
index a6e2a80..99e0973 100644
--- a/src/git.ts
+++ b/src/git.ts
@@ -74,6 +74,14 @@ export const commitChangesFromRepo = async ({
       ) {
         return null;
       }
+    
+      const prevOid = await commit?.oid();
+      const currentOid = await workdir?.oid();
+      // Don't include files that haven't changed, and exist in both trees
+      if (prevOid === currentOid && !commit === !workdir) {
+        return null;
+      }
+      
       if (
         (await commit?.mode()) === FILE_MODES.symlink ||
         (await workdir?.mode()) === FILE_MODES.symlink
@@ -87,12 +95,7 @@ export const commitChangesFromRepo = async ({
           `Unexpected executable file at ${filepath}, GitHub API only supports non-executable files and directories. You may need to add this file to .gitignore`,
         );
       }
-      const prevOid = await commit?.oid();
-      const currentOid = await workdir?.oid();
-      // Don't include files that haven't changed, and exist in both trees
-      if (prevOid === currentOid && !commit === !workdir) {
-        return null;
-      }
+
       // Iterate through anything that may be a directory in either the
       // current commit or the working directory
       if (