update claude settings

AI Prompts:
claude-sonnet-4-6: <ide_opened_file>The user opened the file ./CHANGES.md in the IDE. This may or may not be related to the current task.</ide_opened_file> add a version variable to the .claude/settings.json when updated that reflects the package version
claude-sonnet-4-6: <ide_selection>The user selected the lines 5 to 12 from ./ai_prompt_auto_commit/data/claude_settings.json:         "hooks": [           {             "id": "ai-prompt-auto-commit",             "type": "command",             "command": "input=$(cat); ts=$(date +%Y-%m-%dT%H-%M-%S); model=$(printf '%s' \"$input\" | jq -r '.model // \"claude-sonnet-4-6\"'); root=$(git rev-parse --show-toplevel 2>/dev/null || pwd); mkdir -p \"$root/.prompts\"; printf '%s' \"$input\" | jq -r '.prompt' > \"$root/.prompts/${ts}_${model}.md\""           }         ]   This may or may not be related to the current task.</ide_selection> should look like          "hooks": [           {             "id": "ai-prompt-auto-commit",     "version": ...,             "type": "command",             "command": "input=$(cat); ts=$(date +%Y-%m-%dT%H-%M-%S); model=$(printf '%s' \"$input\" | jq -r '.model // \"claude-sonnet-4-6\"'); root=$(git rev-parse --show-toplevel 2>/dev/null || pwd); mkdir -p \"$root/.prompts\"; printf '%s' \"$input\" | jq -r '.prompt' > \"$root/.prompts/${ts}_${model}.md\""           }         ]
claude-sonnet-4-6: <ide_selection>The user selected the lines 15 to 15 from ./ai_prompt_auto_commit/prepare_repository.py: get_default_claude_settings  This may or may not be related to the current task.</ide_selection> fix, use and test get_default_claude_settings
claude-sonnet-4-6: <ide_selection>The user selected the lines 64 to 64 from ./ai_prompt_auto_commit/prepare_repository.py: h["version"] = package_version  This may or may not be related to the current task.</ide_selection> now test that prepare_repository updates  the hook code when it is run again h["version"] = package_version is not enough . the whole hooks content needs updating

Author: niccokunzmann

CHANGES.md

@@ -4,8 +4,11 @@
 
 ## v0.0.6
 
-- Create .prompts if absent
-- `prepare-ai-repository` hook now adds .prompts to the `/.gitignore`
+- Create .prompts if absent in `.clauded/settings.json`
+- `prepare-ai-repository` hook
+  - adds .prompts to the `/.gitignore`
+  - updates the hook `.claude/settings.json`
+  - adds a version to the hook
 
 ## v0.0.5
 

ai_prompt_auto_commit/prepare_repository.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import importlib.metadata
 import importlib.resources
 import json
 import sys
@@ -11,17 +12,26 @@
 from .common import PROMPTS_DIRECTORY
 
 
-def prepare_repository() -> int:
+def get_default_claude_settings() -> dict:
+    """Return the bundled claude_settings.json with the package version injected into the hook."""
+    ref = importlib.resources.files("ai_prompt_auto_commit.data").joinpath("claude_settings.json")
+    settings = json.loads(ref.read_text(encoding="utf-8"))
+    package_version = importlib.metadata.version("ai-prompt-auto-commit")
+    settings["hooks"]["UserPromptSubmit"][0]["hooks"][0]["version"] = package_version
+    return settings
+
+def prepare_repository(
+    prompts_directory:str = PROMPTS_DIRECTORY,) -> int:
     """Set up .prompts/, and .claude/settings.json in the target repo."""
     repo_root = common._repo_root()
 
     # Create PROMPTS_DIRECTORY
-    prompts_dir = repo_root / PROMPTS_DIRECTORY
+    prompts_dir = repo_root / prompts_directory
     prompts_dir.mkdir(parents=True, exist_ok=True)
 
     # Add .prompts/ to the root .gitignore
     root_gitignore = repo_root / ".gitignore"
-    pattern = f"/{PROMPTS_DIRECTORY}/"
+    pattern = f"/{prompts_directory}/"
     existing = root_gitignore.read_text(encoding="utf-8").splitlines() if root_gitignore.exists() else []
     if pattern not in existing:
         with root_gitignore.open("a", encoding="utf-8") as fh:
@@ -31,10 +41,10 @@ def prepare_repository() -> int:
         print(f"{root_gitignore} already contains '{pattern}'")
 
     # Install the UserPromptSubmit hook into .claude/settings.json
-    ref = importlib.resources.files("ai_prompt_auto_commit.data").joinpath("claude_settings.json")
-    bundled = json.loads(ref.read_text(encoding="utf-8"))
+    bundled = get_default_claude_settings()
     hook_def = bundled["hooks"]["UserPromptSubmit"][0]["hooks"][0]
     hook_id = hook_def["id"]
+    package_version = hook_def["version"]
 
     claude_dest = repo_root / ".claude"
     dest_file = claude_dest / "settings.json"
@@ -47,18 +57,23 @@ def prepare_repository() -> int:
         for matcher in matchers
         for h in matcher.get("hooks", [])
     )
-
     if already_installed:
-        print(f"Hook '{hook_id}' already present in {dest_file}")
+        for matcher in matchers:
+            hooks_list = matcher.get("hooks", [])
+            for i, h in enumerate(hooks_list):
+                if h.get("id") == hook_id:
+                    hooks_list[i] = hook_def
+        print(f"Updated hook '{hook_id}' to version {package_version} in {dest_file}")
     else:
         if matchers:
             matchers[0].setdefault("hooks", []).append(hook_def)
         else:
             matchers.append({"hooks": [hook_def]})
-        claude_dest.mkdir(parents=True, exist_ok=True)
-        dest_file.write_text(json.dumps(settings, indent=2) + "\n", encoding="utf-8")
         print(f"Inserted hook '{hook_id}' into {dest_file}")
 
+    claude_dest.mkdir(parents=True, exist_ok=True)
+    dest_file.write_text(json.dumps(settings, indent=2) + "\n", encoding="utf-8")
+
     # Ensure prepare-commit-msg and post-commit hooks are installed.
     # Derive them from the existing pre-commit hook file by swapping --hook-type.
     hooks_dir = repo_root / ".git" / "hooks"

tests/test_prepare_repository.py

@@ -2,11 +2,12 @@
 
 from __future__ import annotations
 
+import importlib.metadata
 import json
 from pathlib import Path
 
 from ai_prompt_auto_commit.common import PROMPTS_DIRECTORY
-from ai_prompt_auto_commit.prepare_repository import prepare_repository
+from ai_prompt_auto_commit.prepare_repository import get_default_claude_settings, prepare_repository
 
 
 # ---------------------------------------------------------------------------
@@ -17,32 +18,47 @@ def test_prompts_dir_created(repo: Path) -> None:
     assert (repo / PROMPTS_DIRECTORY).is_dir()
 
 
-def test_prompts_gitignore_contains_wildcard(repo: Path) -> None:
-    gitignore = repo / PROMPTS_DIRECTORY / ".gitignore"
-    assert gitignore.exists()
-    assert "*" in gitignore.read_text(encoding="utf-8").splitlines()
-
-
-def test_prompts_gitignore_not_duplicated(repo: Path) -> None:
-    prepare_repository()  # second run — fixture already ran once
-    gitignore = repo / PROMPTS_DIRECTORY / ".gitignore"
-    assert gitignore.read_text(encoding="utf-8").splitlines().count("*") == 1
-
-
 # ---------------------------------------------------------------------------
 # top-level .gitignore
 # ---------------------------------------------------------------------------
 
-def test_gitignore_is_not_touched(repo: Path) -> None:
-    assert not (repo / ".gitignore").exists()
+def test_root_gitignore_contains_prompts_pattern(repo: Path) -> None:
+    gitignore = repo / ".gitignore"
+    assert gitignore.exists()
+    assert f"/{PROMPTS_DIRECTORY}/" in gitignore.read_text(encoding="utf-8").splitlines()
+
 
-def test_gitignore_existing_content_preserved(repo: Path) -> None:
-    # Overwrite with pre-existing content and re-run
+def test_root_gitignore_pattern_not_duplicated(repo: Path) -> None:
+    prepare_repository()  # second run
+    gitignore = repo / ".gitignore"
+    lines = gitignore.read_text(encoding="utf-8").splitlines()
+    assert lines.count(f"/{PROMPTS_DIRECTORY}/") == 1
+
+
+def test_root_gitignore_existing_content_preserved(repo: Path) -> None:
     old_content = "*.pyc\n__pycache__\n"
     (repo / ".gitignore").write_text(old_content, encoding="utf-8")
     prepare_repository()
     content = (repo / ".gitignore").read_text(encoding="utf-8")
-    assert content == old_content
+    assert content.startswith(old_content)
+    assert f"/{PROMPTS_DIRECTORY}/" in content
+
+
+# ---------------------------------------------------------------------------
+# get_default_claude_settings
+# ---------------------------------------------------------------------------
+
+def test_get_default_claude_settings_returns_hook() -> None:
+    settings = get_default_claude_settings()
+    hooks = settings["hooks"]["UserPromptSubmit"][0]["hooks"]
+    assert any(h.get("id") == "ai-prompt-auto-commit" for h in hooks)
+
+
+def test_get_default_claude_settings_version_matches_package() -> None:
+    settings = get_default_claude_settings()
+    hook = settings["hooks"]["UserPromptSubmit"][0]["hooks"][0]
+    expected = importlib.metadata.version("ai-prompt-auto-commit")
+    assert hook["version"] == expected
 
 # ---------------------------------------------------------------------------
 # .claude/settings.json
@@ -96,5 +112,41 @@ def test_claude_settings_hook_not_duplicated(repo: Path) -> None:
     assert _hook_ids(settings).count("ai-prompt-auto-commit") == 1
 
 
+def test_claude_settings_hook_has_version(repo: Path) -> None:
+    dest = repo / ".claude" / "settings.json"
+    settings = json.loads(dest.read_text(encoding="utf-8"))
+    hook = next(
+        h for matcher in settings["hooks"]["UserPromptSubmit"]
+        for h in matcher.get("hooks", [])
+        if h.get("id") == "ai-prompt-auto-commit"
+    )
+    expected = importlib.metadata.version("ai-prompt-auto-commit")
+    assert hook["version"] == expected
+
+
+def test_claude_settings_hook_updated_on_rerun(repo: Path) -> None:
+    dest = repo / ".claude" / "settings.json"
+    # Corrupt the existing hook with stale content
+    settings = json.loads(dest.read_text(encoding="utf-8"))
+    for matcher in settings["hooks"]["UserPromptSubmit"]:
+        for h in matcher.get("hooks", []):
+            if h.get("id") == "ai-prompt-auto-commit":
+                h["version"] = "0.0.0"
+                h["command"] = "stale command"
+                h["extra_stale_key"] = "should be removed"
+    dest.write_text(json.dumps(settings, indent=2), encoding="utf-8")
+    prepare_repository()
+    settings = json.loads(dest.read_text(encoding="utf-8"))
+    hook = next(
+        h for matcher in settings["hooks"]["UserPromptSubmit"]
+        for h in matcher.get("hooks", [])
+        if h.get("id") == "ai-prompt-auto-commit"
+    )
+    expected = get_default_claude_settings()["hooks"]["UserPromptSubmit"][0]["hooks"][0]
+    assert hook == expected
+    assert "extra_stale_key" not in hook
+
+
 def test_prepare_repository_returns_zero(repo: Path) -> None:
     assert prepare_repository() == 0
+    assert repo.is_dir()