git-cai-cli 0.16.1__tar.gz → 0.16.2__tar.gz

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/.github/workflows/python-tests.yml

@@ -99,7 +99,7 @@ jobs:
 
       # MegaLinter
       - name: MegaLinter
-        uses: oxsecurity/megalinter@42bb470545e359597e7f12156947c436e4e3fb9a
+        uses: oxsecurity/megalinter@15e5b45552097e318c93de385779ce3b1084052c # v10.0.0
         id: ml
         env:
           VALIDATE_ALL_CODEBASE: false

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/.github/workflows/release_aur.yml

@@ -9,6 +9,9 @@ on:
         description: "Release version (e.g. 0.6.5)"
         required: true
 
+permissions:
+  contents: read
+
 jobs:
   aur:
     runs-on: ubuntu-slim
@@ -48,8 +51,8 @@ jobs:
           echo "Downloading tag: ${TAG}"
 
           curl -fL \
-            -o cai-${VERSION}.tar.gz \
-            https://github.com/thorstenfoltz/cai/archive/refs/tags/${TAG}.tar.gz
+            -o "cai-${VERSION}.tar.gz" \
+            "https://github.com/thorstenfoltz/cai/archive/refs/tags/${TAG}.tar.gz"
 
       - name: Compute sha256sum
         id: sha

git_cai_cli-0.16.2/.linters/.checkov.yml

@@ -0,0 +1,7 @@
+# skip-path only filters directory scans. On pull requests MegaLinter passes
+# the changed files to checkov with --file, which ignores skip-path, so
+# workflow findings must be suppressed per check instead.
+# CKV_GHA_7: workflow_dispatch inputs must be empty; release_aur.yml needs a
+# manual version input to republish a release.
+skip-check:
+  - CKV_GHA_7

git_cai_cli-0.16.2/.linters/.proselintrc.json

@@ -0,0 +1,32 @@
+{
+  "checks": {
+    "annotations": false,
+    "archaism": false,
+    "cliches": true,
+    "dates_times": true,
+    "hedging": true,
+    "industrial_language": false,
+    "lexical_illusions": false,
+    "malapropisms": true,
+    "misc": true,
+    "misc.phrasal_adjectives": false,
+    "mixed_metaphors": true,
+    "mondegreens": true,
+    "needless_variants": false,
+    "nonwords": true,
+    "oxymorons": true,
+    "psychology": true,
+    "redundancy": true,
+    "restricted": false,
+    "skunked_terms": true,
+    "social_awareness": true,
+    "spelling": true,
+    "spelling.athletes": false,
+    "terms": true,
+    "typography": true,
+    "typography.symbols": false,
+    "uncomparables": true,
+    "weasel_words": true,
+    "weasel_words.very": false
+  }
+}

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/.mega-linter.yml

@@ -2,20 +2,26 @@
 # See all available variables at https://megalinter.io/configuration/
 # and in linters documentation
 
+ADDITIONAL_EXCLUDED_DIRECTORIES:
+  # Build/test caches: gitignored, but project-mode linters scan them anyway.
+  - __pycache__
+  - .mypy_cache
+  - .pytest_cache
+  - .ruff_cache
+  - .benchmarks
 APPLY_FIXES: none # all, none, or list of linter keys
 BASH_SHELLCHECK_ARGUMENTS: -e "SC2162"
 CLEAR_REPORT_FOLDER: true
 DISABLE_ERRORS: false
 ENABLE_LINTERS:
+  - ACTION_ACTIONLINT
   - BASH_EXEC
   - BASH_SHELLCHECK
   - BASH_SHFMT
   - JSON_JSONLINT
   - JSON_PRETTIER
   - JSON_V8R
-  - MAKEFILE_CHECKMAKE
   - MARKDOWN_MARKDOWNLINT
-  - MARKDOWN_MARKDOWN_LINK_CHECK
   - MARKDOWN_MARKDOWN_TABLE_FORMATTER
   - PYTHON_BLACK
   - PYTHON_BANDIT
@@ -26,9 +32,8 @@ ENABLE_LINTERS:
   - PYTHON_PYRIGHT
   - PYTHON_RUFF
   - PYTHON_RUFF_FORMAT
+  - REPOSITORY_BETTERLEAKS
   - REPOSITORY_CHECKOV
-  - REPOSITORY_GITLEAKS
-  - REPOSITORY_KICS
   - REPOSITORY_LS_LINT
   - REPOSITORY_SECRETLINT
   - REPOSITORY_SEMGREP
@@ -48,7 +53,8 @@ IGNORE_GITIGNORED_FILES: true
 LINTER_RULES_PATH: .linters
 LOG_LEVEL: INFO
 MARKDOWN_DEFAULT_STYLE: markdownlint
-MARKDOWN_MARKDOWN_LINK_CHECK_RULES_PATH: .linters
+# Pins the Docker image for local runs to the version the CI workflow pins.
+MEGALINTER_VERSION: v10.0.0
 PARALLEL: true
 PRINT_ALPACA: false
 PYTHON_BANDIT_RULES_PATH: .linters
@@ -68,8 +74,10 @@ REPOSITORY_SEMGREP_RULESETS:
     "p/secure-defaults",
   ]
 SECURITY_SUGGESTIONS: false
+# Lob API keys look like `test_<alnum>`, so every pytest function name is
+# reported as a verified secret. No other detector is affected.
+REPOSITORY_TRUFFLEHOG_ARGUMENTS: --exclude-detectors=Lob
 SHOW_ELAPSED_TIME: true
 SHOW_SKIPPED_LINTERS: true
 SPELL_PROSELINT_RULES_PATH: .linters
-SPELL_VALE_RULES_PATH: .linters
 VALIDATE_ALL_CODEBASE: false

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/Makefile

@@ -42,10 +42,10 @@ clean: ## Clean cache of uv and delete virtual environment
 
 lint:
 	@sh ./.linters/check_git_branch_name.sh
-	@npx mega-linter-runner
+	@npx mega-linter-runner@v10
 
 lint-fix: ## Lints the code using sqlfluff and fixes the issues
-	@npx mega-linter-runner --fix
+	@npx mega-linter-runner@v10 --fix
 
 test: ## Runs tests
 	@$(UV) run pytest

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: git-cai-cli
-Version: 0.16.1
+Version: 0.16.2
 Summary: Use LLM to create git commit messages
 Author-email: Thorsten Foltz <thorsten.foltz@live.com>
 License-Expression: MIT

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/cai_config.yml

@@ -16,6 +16,7 @@ pr_to_file: false
 pr_file_name: PR_DESCRIPTION.md
 pr_prompt_file: "/home/thorsten/.config/cai/pr_prompt.md"
 stats: true
+signoff: true
 anthropic:
   model: claude-haiku-4-5
   temperature: 0
@@ -37,7 +38,7 @@ ollama:
   temperature: 0
   timeout: 300
 openai:
-  model: gpt-5.4-mini
+  model: gpt-5.6-luna
 xai:
   model: grok-4.3
   temperature: 0

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/docs/man/git-cai.1

@@ -2,12 +2,12 @@
 .\"     Title: git-cai
 .\"    Author: Thorsten Foltz
 .\" Generator: Asciidoctor 2.0.26
-.\"      Date: 2026-07-28
+.\"      Date: 2026-08-06
 .\"    Manual: \ \&
 .\"    Source: \ \&
 .\"  Language: English
 .\"
-.TH "GIT\-CAI" "1" "2026-07-28" "\ \&" "\ \&"
+.TH "GIT\-CAI" "1" "2026-08-06" "\ \&" "\ \&"
 .ie \n(.g .ds Aq \(aq
 .el       .ds Aq '
 .ss \n[.ss] 0

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.16.1'
-__version_tuple__ = version_tuple = (0, 16, 1)
+__version__ = version = '0.16.2'
+__version_tuple__ = version_tuple = (0, 16, 2)
 
-__commit_id__ = commit_id = 'geec8479e2'
+__commit_id__ = commit_id = 'g2674674a6'

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli/core/config.py

@@ -40,7 +40,7 @@ DEFAULT_CONFIG: dict[str, Any] = {
         # ``max_tokens`` is still read as a legacy alias (see llm.py)
         "max_output_tokens": 32768,
     },
-    "openai": {"model": "gpt-5.4-mini"},
+    "openai": {"model": "gpt-5.6-luna"},
     "deepseek": {"model": "deepseek-chat", "temperature": 0},
     "gemini": {"model": "gemini-3.1-flash-lite", "temperature": 0},
     "groq": {"model": "openai/gpt-oss-20b", "temperature": 0},

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli/core/gitutils.py

@@ -259,8 +259,15 @@ def append_signoff(message: str, identity: tuple[str, str] | None = None) -> str
     if not stripped:
         return trailer
 
-    last_line = stripped.rsplit("\n", 1)[-1]
-    if _TRAILER_LINE_RE.match(last_line):
+    # Only join an existing trailer block directly: a block is the final
+    # paragraph and must consist solely of ``Key: value`` lines. A message
+    # with a single paragraph never qualifies — a lone conventional subject
+    # ("feat: add x") looks like a trailer line, and gluing the sign-off to
+    # it makes git stop recognizing the trailer at all.
+    paragraphs = stripped.split("\n\n")
+    if len(paragraphs) > 1 and all(
+        _TRAILER_LINE_RE.match(line) for line in paragraphs[-1].splitlines()
+    ):
         return f"{stripped}\n{trailer}"
 
     return f"{stripped}\n\n{trailer}"

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli/core/llm.py

@@ -37,7 +37,9 @@ log = logging.getLogger(__name__)
 
 
 # Models that reject any non-default ``temperature`` (return HTTP 400).
-# Extend this list as new restricted models appear.
+# Pure fast path: it saves the rejected request. Models missing here are
+# still handled — ``_http_post`` retries once without the field — so add
+# an entry only when a warning shows up in practice.
 _TEMPERATURE_UNSUPPORTED_PATTERNS = [
     re.compile(r"^gpt-5", re.IGNORECASE),  # OpenAI GPT-5 family (incl. mini/nano)
     re.compile(r"^o[134](-|$)", re.IGNORECASE),  # OpenAI o1/o3/o4 reasoning series
@@ -123,14 +125,46 @@ def _get_http_session() -> requests.Session:
     return _build_retrying_session()
 
 
+def _strip_temperature(payload: Any) -> bool:
+    """Remove every ``temperature`` key from a request body, in place.
+
+    Providers nest it differently — top level for the OpenAI-compatible
+    APIs and Anthropic, ``generationConfig`` for Gemini, ``options`` for
+    Ollama — so recurse into nested dicts. Returns True if anything was
+    removed.
+    """
+    if not isinstance(payload, dict):
+        return False
+    removed = payload.pop("temperature", None) is not None
+    for value in payload.values():
+        if _strip_temperature(value):
+            removed = True
+    return removed
+
+
 def _http_post(*args, **kwargs):
     """POST via the module-level retrying session.
 
     Single patch-point for tests; never uses ``requests.post`` directly so
     every provider call benefits from urllib3 retry/backoff on transient
     failures (429 / 5xx).
+
+    Models that reject ``temperature`` answer 400 instead of ignoring the
+    field, and new ones keep appearing, so a hardcoded model list can only
+    ever be a fast path (see ``_TEMPERATURE_UNSUPPORTED_PATTERNS``). When
+    the provider's own error names the field, drop it and retry once
+    rather than failing the run.
     """
-    return _get_http_session().post(*args, **kwargs)
+    response = _get_http_session().post(*args, **kwargs)
+    if response.status_code == 400 and "temperature" in response.text.lower():
+        if _strip_temperature(kwargs.get("json")):
+            log.warning(
+                "Provider rejected the configured temperature (HTTP 400); "
+                "retrying without it. Remove 'temperature' from this "
+                "model's config block to skip the extra request."
+            )
+            response = _get_http_session().post(*args, **kwargs)
+    return response
 
 
 def load_prompt_file(

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli/core/squash.py

@@ -409,7 +409,17 @@ def squash_branch(
             log.info("Squash summary generated in %.2fs", elapsed)
             generator.record_elapsed(int(elapsed * 1000))
 
-        # 4) Let user edit the summary without making a commit yet
+        # 4) Let user edit the summary without making a commit yet.
+        # The trailer is added before the editor opens — same as commit mode,
+        # so the user sees (and can drop) it instead of it appearing only
+        # after the squash commit was already written.
+        if apply_signoff:
+            try:
+                summary_message = append_signoff(summary_message)
+            except RuntimeError as e:
+                log.error("%s", e)
+                sys.exit(1)
+
         log.info(
             "Opening editor for final squash commit message. Save = continue, exit w/o save = cancel."
         )
@@ -449,13 +459,6 @@ def squash_branch(
         finally:
             tf_name.unlink(missing_ok=True)
 
-        if apply_signoff:
-            try:
-                final_message = append_signoff(final_message)
-            except RuntimeError as e:
-                log.error("%s", e)
-                sys.exit(1)
-
         # 5) Perform squash
         subprocess.run(["git", "reset", "--soft", merge_base], check=True)
         subprocess.run(["git", "commit", "-m", final_message], check=True)

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: git-cai-cli
-Version: 0.16.1
+Version: 0.16.2
 Summary: Use LLM to create git commit messages
 Author-email: Thorsten Foltz <thorsten.foltz@live.com>
 License-Expression: MIT

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli.egg-info/SOURCES.txt

@@ -24,9 +24,8 @@ uv.lock
 .linters/.checkov.yml
 .linters/.flake8
 .linters/.ls-lint.yml
-.linters/.markdown-link-check.json
 .linters/.markdownlint.json
-.linters/.proselintrc
+.linters/.proselintrc.json
 .linters/.pylintrc
 .linters/.yamllint.yml
 .linters/check_git_branch_name.sh

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/src/git_cai_cli.egg-info/scm_file_list.json

@@ -1,105 +1,104 @@
 {
   "files": [
-    ".gitattributes",
-    ".semgrepignore",
-    ".mega-linter.yml",
-    "Makefile",
-    ".trivyignore",
-    "pyproject.toml",
-    "cai_config.yml",
-    ".markdownlintignore",
-    "LICENSE",
-    "CLAUDE.md",
-    "README.md",
-    "uv.lock",
-    ".gitignore",
     ".caiignore",
-    ".github/workflows/python-tests.yml",
-    ".github/workflows/release_aur.yml",
-    ".github/workflows/release.yml",
-    ".github/cd/PKGBUILD",
+    ".gitattributes",
     ".github/cd/.SRCINFO",
+    ".github/cd/PKGBUILD",
     ".github/ci/_version.py",
     ".github/ci/cai_config.ci.yml",
     ".github/ci/tokens.ci.yml",
+    ".github/workflows/python-tests.yml",
+    ".github/workflows/release.yml",
+    ".github/workflows/release_aur.yml",
+    ".gitignore",
+    ".linters/.bandit.yml",
+    ".linters/.checkov.yml",
+    ".linters/.flake8",
+    ".linters/.ls-lint.yml",
+    ".linters/.markdownlint.json",
+    ".linters/.proselintrc.json",
+    ".linters/.pylintrc",
+    ".linters/.yamllint.yml",
+    ".linters/check_git_branch_name.sh",
+    ".linters/lychee.toml",
+    ".linters/pyrightconfig.json",
+    ".markdownlintignore",
+    ".mega-linter.yml",
+    ".semgrepignore",
+    ".trivyignore",
+    "CLAUDE.md",
+    "LICENSE",
+    "Makefile",
+    "README.md",
+    "cai_config.yml",
+    "docs/git-cai.txt",
+    "docs/man/git-cai.1",
+    "pyproject.toml",
+    "src/git_cai_cli/__init__.py",
+    "src/git_cai_cli/cli/__init__.py",
+    "src/git_cai_cli/cli/cli.py",
+    "src/git_cai_cli/cli/helptext.py",
+    "src/git_cai_cli/cli/modes.py",
+    "src/git_cai_cli/core/__init__.py",
+    "src/git_cai_cli/core/changelog.py",
+    "src/git_cai_cli/core/completion.py",
+    "src/git_cai_cli/core/config.py",
+    "src/git_cai_cli/core/doctor.py",
+    "src/git_cai_cli/core/explain.py",
+    "src/git_cai_cli/core/generation.py",
+    "src/git_cai_cli/core/gitutils.py",
+    "src/git_cai_cli/core/init.py",
+    "src/git_cai_cli/core/languages.py",
+    "src/git_cai_cli/core/llm.py",
+    "src/git_cai_cli/core/options.py",
+    "src/git_cai_cli/core/pr.py",
+    "src/git_cai_cli/core/prompts_fallback.py",
+    "src/git_cai_cli/core/release.py",
+    "src/git_cai_cli/core/secrets.py",
+    "src/git_cai_cli/core/spinner.py",
+    "src/git_cai_cli/core/split.py",
+    "src/git_cai_cli/core/squash.py",
+    "src/git_cai_cli/core/stats.py",
+    "src/git_cai_cli/core/validate.py",
+    "src/git_cai_cli/main.py",
     "tests/conftest.py",
-    "tests/integration/test_options_integration.py",
-    "tests/integration/test_modes_integration.py",
+    "tests/integration/test_cli_integration.py",
+    "tests/integration/test_config_integration.py",
     "tests/integration/test_gitutils_integration.py",
+    "tests/integration/test_modes_integration.py",
+    "tests/integration/test_options_integration.py",
     "tests/integration/test_pr_integration.py",
-    "tests/integration/test_config_integration.py",
     "tests/integration/test_squash_integration.py",
-    "tests/integration/test_cli_integration.py",
+    "tests/unit/test_amend.py",
+    "tests/unit/test_branch_context.py",
+    "tests/unit/test_changelog.py",
     "tests/unit/test_classification.py",
-    "tests/unit/test_prompt_loading.py",
-    "tests/unit/test_stats.py",
+    "tests/unit/test_cli.py",
+    "tests/unit/test_completion.py",
+    "tests/unit/test_config.py",
     "tests/unit/test_conventional.py",
-    "tests/unit/test_branch_context.py",
-    "tests/unit/test_signoff.py",
     "tests/unit/test_doctor.py",
+    "tests/unit/test_explain.py",
     "tests/unit/test_generation.py",
-    "tests/unit/test_squash.py",
-    "tests/unit/test_print.py",
-    "tests/unit/test_options.py",
-    "tests/unit/test_llm.py",
-    "tests/unit/test_changelog.py",
+    "tests/unit/test_gitutils.py",
     "tests/unit/test_helptext.py",
+    "tests/unit/test_init.py",
+    "tests/unit/test_llm.py",
+    "tests/unit/test_main.py",
+    "tests/unit/test_modes.py",
+    "tests/unit/test_options.py",
     "tests/unit/test_pr.py",
-    "tests/unit/test_secrets.py",
-    "tests/unit/test_split.py",
-    "tests/unit/test_explain.py",
+    "tests/unit/test_print.py",
+    "tests/unit/test_prompt_loading.py",
     "tests/unit/test_release.py",
-    "tests/unit/test_amend.py",
-    "tests/unit/test_gitutils.py",
+    "tests/unit/test_secrets.py",
     "tests/unit/test_set_config.py",
-    "tests/unit/test_completion.py",
-    "tests/unit/test_main.py",
-    "tests/unit/test_modes.py",
-    "tests/unit/test_validate.py",
-    "tests/unit/test_cli.py",
+    "tests/unit/test_signoff.py",
     "tests/unit/test_spinner.py",
-    "tests/unit/test_init.py",
-    "tests/unit/test_config.py",
-    "src/git_cai_cli/main.py",
-    "src/git_cai_cli/__init__.py",
-    "src/git_cai_cli/cli/modes.py",
-    "src/git_cai_cli/cli/__init__.py",
-    "src/git_cai_cli/cli/cli.py",
-    "src/git_cai_cli/cli/helptext.py",
-    "src/git_cai_cli/core/languages.py",
-    "src/git_cai_cli/core/prompts_fallback.py",
-    "src/git_cai_cli/core/split.py",
-    "src/git_cai_cli/core/pr.py",
-    "src/git_cai_cli/core/completion.py",
-    "src/git_cai_cli/core/llm.py",
-    "src/git_cai_cli/core/stats.py",
-    "src/git_cai_cli/core/config.py",
-    "src/git_cai_cli/core/release.py",
-    "src/git_cai_cli/core/explain.py",
-    "src/git_cai_cli/core/__init__.py",
-    "src/git_cai_cli/core/spinner.py",
-    "src/git_cai_cli/core/options.py",
-    "src/git_cai_cli/core/changelog.py",
-    "src/git_cai_cli/core/validate.py",
-    "src/git_cai_cli/core/secrets.py",
-    "src/git_cai_cli/core/init.py",
-    "src/git_cai_cli/core/generation.py",
-    "src/git_cai_cli/core/squash.py",
-    "src/git_cai_cli/core/doctor.py",
-    "src/git_cai_cli/core/gitutils.py",
-    "docs/git-cai.txt",
-    "docs/man/git-cai.1",
-    ".linters/.checkov.yml",
-    ".linters/.pylintrc",
-    ".linters/.ls-lint.yml",
-    ".linters/pyrightconfig.json",
-    ".linters/.markdown-link-check.json",
-    ".linters/check_git_branch_name.sh",
-    ".linters/lychee.toml",
-    ".linters/.markdownlint.json",
-    ".linters/.yamllint.yml",
-    ".linters/.proselintrc",
-    ".linters/.flake8",
-    ".linters/.bandit.yml"
+    "tests/unit/test_split.py",
+    "tests/unit/test_squash.py",
+    "tests/unit/test_stats.py",
+    "tests/unit/test_validate.py",
+    "uv.lock"
   ]
 }

git_cai_cli-0.16.2/src/git_cai_cli.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.16.2",
+  "distance": 0,
+  "node": "g2674674a6ae699ea810dde77263bb4e195d1d625",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-08-15"
+}

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/tests/unit/test_llm.py

@@ -8,8 +8,10 @@ from unittest.mock import MagicMock, patch
 import pytest
 from git_cai_cli.core.llm import (
     CommitMessageGenerator,
+    _http_post,
     _model_rejects_temperature,
     _resolve_temperature,
+    _strip_temperature,
 )
 
 
@@ -1383,6 +1385,65 @@ def test_ollama_start_skips_start_new_session_on_windows(monkeypatch, generator)
 # ---------------------------------------------------------------------------
 
 
+def test_strip_temperature_removes_nested_occurrences():
+    payload = {
+        "model": "m",
+        "temperature": 0,
+        "generationConfig": {"temperature": 0.7, "topK": 1},
+        "options": {"temperature": 0},
+    }
+    assert _strip_temperature(payload) is True
+    assert payload == {"model": "m", "generationConfig": {"topK": 1}, "options": {}}
+    # nothing left to remove -> no pointless retry
+    assert _strip_temperature(payload) is False
+
+
+def test_http_post_retries_without_temperature_on_400(caplog):
+    """A model not covered by the known-list rejects the field with 400;
+    the request is replayed once without it instead of failing the run."""
+    rejected = MagicMock(
+        status_code=400,
+        text=(
+            '{"error":{"message":"Unsupported value: \'temperature\'",'
+            '"param":"temperature"}}'
+        ),
+    )
+    accepted = MagicMock(status_code=200, text="{}")
+    session = MagicMock()
+    session.post.side_effect = [rejected, accepted]
+    payload = {"model": "future-model", "temperature": 0}
+
+    with (
+        patch(
+            f"{CommitMessageGenerator.__module__}._get_http_session",
+            return_value=session,
+        ),
+        caplog.at_level(logging.WARNING),
+    ):
+        result = _http_post("https://api.example.com/v1", json=payload)
+
+    assert result is accepted
+    assert session.post.call_count == 2
+    assert session.post.call_args_list[1].kwargs["json"] == {"model": "future-model"}
+    assert "retrying without it" in caplog.text
+
+
+def test_http_post_does_not_retry_on_unrelated_400():
+    rejected = MagicMock(
+        status_code=400, text='{"error":{"message":"model not found"}}'
+    )
+    session = MagicMock()
+    session.post.return_value = rejected
+
+    with patch(
+        f"{CommitMessageGenerator.__module__}._get_http_session", return_value=session
+    ):
+        result = _http_post("https://api.example.com/v1", json={"temperature": 0})
+
+    assert result is rejected
+    assert session.post.call_count == 1
+
+
 def test_model_rejects_temperature_openai_gpt5():
     assert _model_rejects_temperature("gpt-5.4-mini") is True
     assert _model_rejects_temperature("gpt-5.2") is True

{git_cai_cli-0.16.1 → git_cai_cli-0.16.2}/tests/unit/test_signoff.py

@@ -116,6 +116,20 @@ def test_append_signoff_when_other_signoff_already_present():
     )
 
 
+def test_append_signoff_separates_single_line_conventional_subject():
+    """A lone subject like ``feat: x`` matches the trailer pattern but is
+    not a trailer block — gluing the sign-off to it makes git drop the
+    trailer entirely (``git interpret-trailers --parse`` returns nothing)."""
+    out = append_signoff("feat: add signoff", identity=("Alice", "alice@example.com"))
+    assert out == "feat: add signoff\n\nSigned-off-by: Alice <alice@example.com>"
+
+
+def test_append_signoff_ignores_trailer_lookalike_at_end_of_body():
+    msg = "Add thing\n\nBody line\nNote: this is prose, not a trailer block"
+    out = append_signoff(msg, identity=("Alice", "alice@example.com"))
+    assert out == f"{msg}\n\nSigned-off-by: Alice <alice@example.com>"
+
+
 def test_append_signoff_on_empty_message_just_returns_trailer():
     out = append_signoff("", identity=("Alice", "alice@example.com"))
     assert out == "Signed-off-by: Alice <alice@example.com>"