fix: address Copilot review feedback on Claude integration

- Remove unused `import re`
- Skip injection if argument-hint already exists in frontmatter
- Add found_description assertion to test_hint_appears_after_description
- Add test_inject_argument_hint_skips_if_already_present test

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Quratulain-bilal

src/specify_cli/integrations/claude/__init__.py

@@ -2,7 +2,6 @@
 
 from __future__ import annotations
 
-import re
 from pathlib import Path
 from typing import Any, TYPE_CHECKING
 
@@ -45,8 +44,25 @@ class ClaudeIntegration(MarkdownIntegration):
 
     @staticmethod
     def inject_argument_hint(content: str, hint: str) -> str:
-        """Insert ``argument-hint`` after the first ``description:`` in YAML frontmatter."""
+        """Insert ``argument-hint`` after the first ``description:`` in YAML frontmatter.
+
+        Skips injection if ``argument-hint:`` already exists in the
+        frontmatter to avoid duplicate keys.
+        """
         lines = content.splitlines(keepends=True)
+
+        # Pre-scan: bail out if argument-hint already present in frontmatter
+        dash_count = 0
+        for line in lines:
+            stripped = line.rstrip("\n\r")
+            if stripped == "---":
+                dash_count += 1
+                if dash_count == 2:
+                    break
+                continue
+            if dash_count == 1 and stripped.startswith("argument-hint:"):
+                return content  # already present
+
         out: list[str] = []
         in_fm = False
         dash_count = 0

tests/integrations/test_integration_claude.py

@@ -77,15 +77,20 @@ def test_hint_appears_after_description(self, tmp_path):
         for f in cmd_files:
             content = f.read_text(encoding="utf-8")
             lines = content.splitlines()
+            found_description = False
             for idx, line in enumerate(lines):
                 if line.startswith("description:"):
+                    found_description = True
                     assert idx + 1 < len(lines), (
                         f"{f.name}: description is last line"
                     )
                     assert lines[idx + 1].startswith("argument-hint:"), (
                         f"{f.name}: argument-hint does not follow description"
                     )
                     break
+            assert found_description, (
+                f"{f.name}: no description: line found in output"
+            )
 
     def test_inject_argument_hint_only_in_frontmatter(self):
         """inject_argument_hint must not modify description: lines in the body."""
@@ -100,7 +105,25 @@ def test_inject_argument_hint_only_in_frontmatter(self):
         )
         result = ClaudeIntegration.inject_argument_hint(content, "Test hint")
         lines = result.splitlines()
-        hint_count = sum(1 for l in lines if l.startswith("argument-hint:"))
+        hint_count = sum(1 for ln in lines if ln.startswith("argument-hint:"))
         assert hint_count == 1, (
             f"Expected exactly 1 argument-hint line, found {hint_count}"
         )
+
+    def test_inject_argument_hint_skips_if_already_present(self):
+        """inject_argument_hint must not duplicate if argument-hint already exists."""
+        from specify_cli.integrations.claude import ClaudeIntegration
+
+        content = (
+            "---\n"
+            "description: My command\n"
+            "argument-hint: Existing hint\n"
+            "---\n"
+            "\n"
+            "Body text\n"
+        )
+        result = ClaudeIntegration.inject_argument_hint(content, "New hint")
+        assert result == content, "Content should be unchanged when hint already exists"
+        lines = result.splitlines()
+        hint_count = sum(1 for ln in lines if ln.startswith("argument-hint:"))
+        assert hint_count == 1