diff --git a/aider/coder.py b/aider/coder.py index 0853bf95c..6dbd5f985 100755 --- a/aider/coder.py +++ b/aider/coder.py @@ -18,7 +18,7 @@ from pathlib import Path import git import openai -# from aider.dump import dump +from aider.dump import dump from aider.getinput import get_input from aider import utils from aider import prompts @@ -450,20 +450,16 @@ class Coder: else: raise ValueError(f"Invalid value for 'which': {which}") + dump(dirty_fnames) + diffs = "" - dirty_fnames = [] - relative_dirty_fnames = [] - for fname in self.abs_fnames: - relative_fname = os.path.relpath(fname, repo.working_tree_dir) + for (abs_fname,relative_fname) in zip(dirty_fnames, relative_dirty_fnames): if self.pretty: these_diffs = repo.git.diff("HEAD", "--color", relative_fname) else: these_diffs = repo.git.diff("HEAD", relative_fname) - if these_diffs: - dirty_fnames.append(fname) - relative_dirty_fnames.append(relative_fname) - diffs += these_diffs + "\n" + diffs += these_diffs + "\n" if not dirty_fnames: self.last_modified = self.get_last_modified() diff --git a/aider/commands.py b/aider/commands.py index 93a0884c1..69974f7b7 100644 --- a/aider/commands.py +++ b/aider/commands.py @@ -73,23 +73,11 @@ class Commands: return if not self.coder.repo.is_dirty(): - self.console.print("[red]No changes to commit.") + self.console.print("[red]No more changes to commit.") return commit_message = args.strip() - if commit_message: - self.coder.repo.git.add( - *[ - os.path.relpath(fname, self.coder.repo.working_tree_dir) - for fname in self.coder.abs_fnames - ] - ) - self.coder.repo.git.commit("-m", commit_message, "--no-verify") - commit_hash = self.coder.repo.head.commit.hexsha[:7] - self.console.print(f"[bright_black]{commit_hash} {commit_message}") - return - - self.coder.commit() + self.coder.commit(message = commit_message, which="repo_files") def cmd_undo(self, args): "Undo the last git commit if it was done by aider" diff --git a/aider/main.py b/aider/main.py index b5282cd7d..3aa2010c1 100644 --- a/aider/main.py +++ b/aider/main.py @@ -59,7 +59,7 @@ def main(): pretty = args.pretty coder = Coder(args.model, fnames, pretty, args.history_file, args.show_diffs) - coder.commit(ask=True, prefix="wip: ") + coder.commit(ask=True, prefix="wip: ", which="repo_files") if args.apply: with open(args.apply, "r") as f: