chore: skip commit if only changs were in the function run (#692)

Author: caroljung-cg

src/codegen/git/repo_operator/repo_operator.py

@@ -446,6 +446,12 @@ def checkout_branch(self, branch_name: str | None, *, remote: bool = False, remo
                 logger.exception(f"Error with Git operations: {e}")
                 raise
 
+    def get_modified_files(self, ref: str | GitCommit) -> list[str]:
+        """Returns a list of modified files in the repo"""
+        self.git_cli.git.add(A=True)
+        diff = self.git_cli.git.diff(ref, "--name-only")
+        return diff.splitlines()
+
     def get_diffs(self, ref: str | GitCommit, reverse: bool = True) -> list[Diff]:
         """Gets all staged diffs"""
         self.git_cli.git.add(A=True)

src/codegen/runner/servers/local_daemon.py

@@ -1,4 +1,5 @@
 import logging
+import os
 from contextlib import asynccontextmanager
 
 from fastapi import FastAPI
@@ -78,6 +79,10 @@ async def run(request: RunFunctionRequest) -> CodemodRunResult:
     diff_req = GetDiffRequest(codemod=Codemod(user_code=request.codemod_source))
     diff_response = await runner.get_diff(request=diff_req)
     if request.commit:
-        if commit_sha := runner.codebase.git_commit(f"[Codegen] {request.function_name}"):
+        changed_files = runner.op.get_modified_files(runner.commit)
+        # if the only changed file is the function that was requested, do not commit
+        if len(changed_files) == 1 and os.path.splitext(os.path.basename(changed_files[0])) == request.function_name:
+            logger.info(f"Skipping commit because only changes to {request.function_name} were made")
+        elif commit_sha := runner.codebase.git_commit(f"[Codegen] {request.function_name}"):
             logger.info(f"Committed changes to {commit_sha.hexsha}")
     return diff_response.result