From 3a7efb27b1f49e8595addfcce6cc572216aeac45 Mon Sep 17 00:00:00 2001
From: Bede Carroll <4151340+bedecarroll@users.noreply.github.com>
Date: Sun, 11 May 2025 18:35:01 -0700
Subject: [PATCH 1/4] feat: add option to disable git commit signing

---
 aider/args.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/aider/args.py b/aider/args.py
index 1543bb07e..4bf0bc37f 100644
--- a/aider/args.py
+++ b/aider/args.py
@@ -471,6 +471,12 @@ def get_parser(default_config_files, git_root):
         default=False,
         help="Enable/disable git pre-commit hooks with --no-verify (default: False)",
     )
+    group.add_argument(
+        "--git-commit-no-sign",
+        action=argparse.BooleanOptionalAction,
+        default=False,
+        help="Disable git signing with --no-gpg-sign (default: False)",
+    )
     group.add_argument(
         "--commit",
         action="store_true",

From 294b0faf34c7d1c87e2ffff660ed7894c6d094ca Mon Sep 17 00:00:00 2001
From: Bede Carroll <4151340+bedecarroll@users.noreply.github.com>
Date: Sun, 11 May 2025 18:40:18 -0700
Subject: [PATCH 2/4] feat: add --git-commit-no-sign flag

---
 aider/args.py | 2 +-
 aider/repo.py | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/aider/args.py b/aider/args.py
index 4bf0bc37f..340aa1f00 100644
--- a/aider/args.py
+++ b/aider/args.py
@@ -475,7 +475,7 @@ def get_parser(default_config_files, git_root):
         "--git-commit-no-sign",
         action=argparse.BooleanOptionalAction,
         default=False,
-        help="Disable git signing with --no-gpg-sign (default: False)",
+        help="Disable git commit signing with --no-gpg-sign (default: False)",
     )
     group.add_argument(
         "--commit",
diff --git a/aider/repo.py b/aider/repo.py
index 01652b15f..85f33f677 100644
--- a/aider/repo.py
+++ b/aider/repo.py
@@ -73,6 +73,7 @@ class GitRepo:
         commit_prompt=None,
         subtree_only=False,
         git_commit_verify=True,
+        git_commit_no_sign=False,
         attribute_co_authored_by=False,  # Added parameter
     ):
         self.io = io
@@ -89,6 +90,7 @@ class GitRepo:
         self.commit_prompt = commit_prompt
         self.subtree_only = subtree_only
         self.git_commit_verify = git_commit_verify
+        self.git_commit_no_sign = git_commit_no_sign
         self.ignore_file_cache = {}
 
         if git_dname:
@@ -277,6 +279,8 @@ class GitRepo:
         cmd = ["-m", full_commit_message]
         if not self.git_commit_verify:
             cmd.append("--no-verify")
+        if self.git_commit_no_sign:
+            cmd.append("--no-gpg-sign")
         if fnames:
             fnames = [str(self.abs_root_path(fn)) for fn in fnames]
             for fname in fnames:

From 684eb6e9a52251c8bcd5ef48ff9b82444910396c Mon Sep 17 00:00:00 2001
From: Bede Carroll <4151340+bedecarroll@users.noreply.github.com>
Date: Sun, 11 May 2025 19:02:24 -0700
Subject: [PATCH 3/4] docs: update docs using scripts/update-docs.sh

---
 aider/website/assets/sample.aider.conf.yml | 3 +++
 aider/website/assets/sample.env            | 3 +++
 aider/website/docs/config/aider_conf.md    | 3 +++
 aider/website/docs/config/dotenv.md        | 3 +++
 aider/website/docs/config/options.md       | 9 +++++++++
 5 files changed, 21 insertions(+)

diff --git a/aider/website/assets/sample.aider.conf.yml b/aider/website/assets/sample.aider.conf.yml
index 7fe9459bf..e271cf22d 100644
--- a/aider/website/assets/sample.aider.conf.yml
+++ b/aider/website/assets/sample.aider.conf.yml
@@ -242,6 +242,9 @@
 ## Enable/disable git pre-commit hooks with --no-verify (default: False)
 #git-commit-verify: false
 
+## Disable git commit signing with --no-gpg-sign (default: False)
+#git-commit-no-sign: false
+
 ## Commit all pending changes with a suitable commit message, then exit
 #commit: false
 
diff --git a/aider/website/assets/sample.env b/aider/website/assets/sample.env
index 6d2a032cd..9368330a7 100644
--- a/aider/website/assets/sample.env
+++ b/aider/website/assets/sample.env
@@ -231,6 +231,9 @@
 ## Enable/disable git pre-commit hooks with --no-verify (default: False)
 #AIDER_GIT_COMMIT_VERIFY=false
 
+## Disable git commit signing with --no-gpg-sign (default: False)
+#AIDER_GIT_COMMIT_NO_SIGN=false
+
 ## Commit all pending changes with a suitable commit message, then exit
 #AIDER_COMMIT=false
 
diff --git a/aider/website/docs/config/aider_conf.md b/aider/website/docs/config/aider_conf.md
index 99bccf401..0fc972ffb 100644
--- a/aider/website/docs/config/aider_conf.md
+++ b/aider/website/docs/config/aider_conf.md
@@ -296,6 +296,9 @@ cog.outl("```")
 ## Enable/disable git pre-commit hooks with --no-verify (default: False)
 #git-commit-verify: false
 
+## Disable git commit signing with --no-gpg-sign (default: False)
+#git-commit-no-sign: false
+
 ## Commit all pending changes with a suitable commit message, then exit
 #commit: false
 
diff --git a/aider/website/docs/config/dotenv.md b/aider/website/docs/config/dotenv.md
index 82ad974de..96845761b 100644
--- a/aider/website/docs/config/dotenv.md
+++ b/aider/website/docs/config/dotenv.md
@@ -271,6 +271,9 @@ cog.outl("```")
 ## Enable/disable git pre-commit hooks with --no-verify (default: False)
 #AIDER_GIT_COMMIT_VERIFY=false
 
+## Disable git commit signing with --no-gpg-sign (default: False)
+#AIDER_GIT_COMMIT_NO_SIGN=false
+
 ## Commit all pending changes with a suitable commit message, then exit
 #AIDER_COMMIT=false
 
diff --git a/aider/website/docs/config/options.md b/aider/website/docs/config/options.md
index 368c4ec7e..8bfc10b36 100644
--- a/aider/website/docs/config/options.md
+++ b/aider/website/docs/config/options.md
@@ -58,6 +58,7 @@ usage: aider [-h] [--model] [--openai-api-key] [--anthropic-api-key]
              [--attribute-commit-message-committer | --no-attribute-commit-message-committer]
              [--attribute-co-authored-by | --no-attribute-co-authored-by]
              [--git-commit-verify | --no-git-commit-verify]
+             [--git-commit-no-sign | --no-git-commit-no-sign]
              [--commit] [--commit-prompt] [--dry-run | --no-dry-run]
              [--skip-sanity-check-repo]
              [--watch-files | --no-watch-files] [--lint]
@@ -460,6 +461,14 @@ Aliases:
   - `--git-commit-verify`
   - `--no-git-commit-verify`
 
+### `--git-commit-no-sign`
+Disable git commit signing with --no-gpg-sign (default: False)  
+Default: False  
+Environment variable: `AIDER_GIT_COMMIT_NO_SIGN`  
+Aliases:
+  - `--git-commit-no-sign`
+  - `--no-git-commit-no-sign`
+
 ### `--commit`
 Commit all pending changes with a suitable commit message, then exit  
 Default: False  

From afdbcfdf133af23af966743a80bfe677e3108e4b Mon Sep 17 00:00:00 2001
From: Bede Carroll <4151340+bedecarroll@users.noreply.github.com>
Date: Sun, 11 May 2025 19:03:24 -0700
Subject: [PATCH 4/4] test: add unit tests for new flag

---
 tests/basic/test_repo.py | 52 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 52 insertions(+)

diff --git a/tests/basic/test_repo.py b/tests/basic/test_repo.py
index 0f1e7ed0e..a8ac04c08 100644
--- a/tests/basic/test_repo.py
+++ b/tests/basic/test_repo.py
@@ -598,6 +598,58 @@ class TestRepo(unittest.TestCase):
             self.assertNotIn(str(root_file), tracked_files)
             self.assertNotIn(str(another_subdir_file), tracked_files)
 
+    @patch("aider.repo.GitRepo.get_commit_message")
+    @patch("git.Repo.git")
+    def test_git_commit_no_sign(self, mock_git, mock_get_commit_message):
+        """Test that git_commit_no_sign controls whether --no-gpg-sign is passed to git commit"""
+        mock_get_commit_message.return_value = "Test commit message"
+
+        with GitTemporaryDirectory():
+            # Create a new repo
+            raw_repo = git.Repo()
+
+            # Create a file to commit
+            fname = Path("test_file.txt")
+            fname.write_text("initial content")
+            raw_repo.git.add(str(fname))
+
+            # Do the initial commit
+            raw_repo.git.commit("-m", "Initial commit")
+
+            # Modify the file
+            fname.write_text("modified content")
+            raw_repo.git.add(str(fname))  # Stage the change
+
+            # Create GitRepo with no_sign=True
+            io = InputOutput()
+            git_repo_no_sign = GitRepo(io, None, None, git_commit_no_sign=True)
+
+            # Attempt to commit with no_sign=True
+            # Mock the actual git.commit call on the Git object returned by mock_git
+            mock_git.commit = MagicMock()
+            commit_result = git_repo_no_sign.commit(fnames=[str(fname)], message="Should not sign")
+
+            # Verify that commit was called with --no-gpg-sign
+            mock_git.commit.assert_called_once()
+            call_args, call_kwargs = mock_git.commit.call_args
+            self.assertIn("--no-gpg-sign", call_args[0])
+            self.assertIsNotNone(commit_result)  # Should return commit hash/message tuple
+
+            # Reset mock for the next test case
+            mock_git.commit.reset_mock()
+
+            # Create GitRepo with no_sign=False
+            git_repo_sign = GitRepo(io, None, None, git_commit_no_sign=False)
+
+            # Attempt to commit with no_sign=False
+            commit_result = git_repo_sign.commit(fnames=[str(fname)], message="Should sign")
+
+            # Verify that commit was called WITHOUT --no-gpg-sign
+            mock_git.commit.assert_called_once()
+            call_args, call_kwargs = mock_git.commit.call_args
+            self.assertNotIn("--no-gpg-sign", call_args[0])
+            self.assertIsNotNone(commit_result)  # Should return commit hash/message tuple
+
     @patch("aider.models.Model.simple_send_with_retries")
     def test_noop_commit(self, mock_send):
         mock_send.return_value = '"a good commit message"'