revert extensions.py and test changes, keep commit.md template fix only

Address PR review: remove hook message changes from extensions.py
(avoids changing $ARGUMENTS semantics for all hooks). The fix is now
entirely in the speckit.git.commit.md command template which instructs
agents to read the config file and documents the message fallback.

Author: mnriem

extensions/git/commands/speckit.git.commit.md

@@ -28,7 +28,7 @@ If the file does not exist, auto-commit is disabled. Exit silently.
 Look under the `auto_commit:` section in the config file you just read:
 
 1. Find the key matching the event name (e.g., `after_tasks:`).
-2. If the event key exists **and** has `enabled: true` → auto-commit is **enabled**. Use the `message` value from that key.
+2. If the event key exists **and** has `enabled: true` → auto-commit is **enabled**. Use the `message` value from that key if present; if `message` is missing or empty, fall back to the default format `"[Spec Kit] Auto-commit <before|after> <command>"`.
 3. If the event key exists **and** has `enabled: false` → auto-commit is **disabled**. Exit silently.
 4. If the event key does **not** exist at all, check `auto_commit.default`:
    - `default: true` → auto-commit is **enabled**. Use a default message `"[Spec Kit] Auto-commit <before|after> <command>"` (e.g., `after_tasks` → `"[Spec Kit] Auto-commit after tasks"`).

src/specify_cli/extensions.py

@@ -2785,18 +2785,16 @@ def format_hook_message(
 
             if optional:
                 lines.append(f"\n**Optional Hook**: {extension}")
-                lines.append(f"Hook event: {event_name}")
                 lines.append(f"Command: `{display_invocation}`")
                 if description:
                     lines.append(f"Description: {description}")
                 lines.append(f"\nPrompt: {prompt}")
-                lines.append(f"To execute: `{display_invocation} {event_name}`")
+                lines.append(f"To execute: `{display_invocation}`")
             else:
                 lines.append(f"\n**Automatic Hook**: {extension}")
-                lines.append(f"Hook event: {event_name}")
-                lines.append(f"Executing: `{display_invocation} {event_name}`")
+                lines.append(f"Executing: `{display_invocation}`")
                 lines.append(f"EXECUTE_COMMAND: {command_text}")
-                lines.append(f"EXECUTE_COMMAND_INVOCATION: {display_invocation} {event_name}")
+                lines.append(f"EXECUTE_COMMAND_INVOCATION: {display_invocation}")
 
         return "\n".join(lines)
 

tests/integrations/test_integration_claude.py

@@ -269,10 +269,9 @@ def test_claude_hooks_render_skill_invocation(self, tmp_path):
             ],
         )
 
-        assert "Hook event: before_plan" in message
-        assert "Executing: `/speckit-plan before_plan`" in message
+        assert "Executing: `/speckit-plan`" in message
         assert "EXECUTE_COMMAND: speckit.plan" in message
-        assert "EXECUTE_COMMAND_INVOCATION: /speckit-plan before_plan" in message
+        assert "EXECUTE_COMMAND_INVOCATION: /speckit-plan" in message
 
     def test_claude_preset_creates_new_skill_without_commands_dir(self, tmp_path):
         from specify_cli import save_init_options

tests/test_extensions.py

@@ -4269,10 +4269,9 @@ def test_kimi_hooks_render_skill_invocation(self, project_dir):
             ],
         )
 
-        assert "Hook event: before_plan" in message
-        assert "Executing: `/skill:speckit-plan before_plan`" in message
+        assert "Executing: `/skill:speckit-plan`" in message
         assert "EXECUTE_COMMAND: speckit.plan" in message
-        assert "EXECUTE_COMMAND_INVOCATION: /skill:speckit-plan before_plan" in message
+        assert "EXECUTE_COMMAND_INVOCATION: /skill:speckit-plan" in message
 
     def test_codex_hooks_render_dollar_skill_invocation(self, project_dir):
         """Codex projects with --ai-skills should render $speckit-* invocations."""
@@ -4310,10 +4309,9 @@ def test_non_skill_command_keeps_slash_invocation(self, project_dir):
             ],
         )
 
-        assert "Hook event: before_tasks" in message
-        assert "Executing: `/pre_tasks_test before_tasks`" in message
+        assert "Executing: `/pre_tasks_test`" in message
         assert "EXECUTE_COMMAND: pre_tasks_test" in message
-        assert "EXECUTE_COMMAND_INVOCATION: /pre_tasks_test before_tasks" in message
+        assert "EXECUTE_COMMAND_INVOCATION: /pre_tasks_test" in message
 
     def test_extension_command_uses_hyphenated_skill_invocation(self, project_dir):
         """Multi-segment extension command ids should map to hyphenated skills."""
@@ -4333,10 +4331,9 @@ def test_extension_command_uses_hyphenated_skill_invocation(self, project_dir):
             ],
         )
 
-        assert "Hook event: after_tasks" in message
-        assert "Executing: `/skill:speckit-test-ext-hello after_tasks`" in message
+        assert "Executing: `/skill:speckit-test-ext-hello`" in message
         assert "EXECUTE_COMMAND: speckit.test-ext.hello" in message
-        assert "EXECUTE_COMMAND_INVOCATION: /skill:speckit-test-ext-hello after_tasks" in message
+        assert "EXECUTE_COMMAND_INVOCATION: /skill:speckit-test-ext-hello" in message
 
     def test_hook_executor_caches_init_options_lookup(self, project_dir, monkeypatch):
         """Init options should be loaded once per executor instance."""
@@ -4371,10 +4368,9 @@ def test_hook_message_falls_back_when_invocation_is_empty(self, project_dir):
             ],
         )
 
-        assert "Hook event: after_tasks" in message
-        assert "Executing: `/<missing command> after_tasks`" in message
+        assert "Executing: `/<missing command>`" in message
         assert "EXECUTE_COMMAND: <missing command>" in message
-        assert "EXECUTE_COMMAND_INVOCATION: /<missing command> after_tasks" in message
+        assert "EXECUTE_COMMAND_INVOCATION: /<missing command>" in message
 
 
 class TestExtensionRemoveCLI: