fix: review comments

Author: dhilipkumars

src/specify_cli/__init__.py

@@ -1090,7 +1090,9 @@ def install_ai_skills(project_path: Path, selected_ai: str, tracker: StepTracker
             if content.startswith("---"):
                 parts = content.split("---", 2)
                 if len(parts) >= 3:
-                    frontmatter = yaml.safe_load(parts[1]) or {}
+                    frontmatter = yaml.safe_load(parts[1])
+                    if not isinstance(frontmatter, dict):
+                        frontmatter = {}
                     body = parts[2].strip()
                 else:
                     # File starts with --- but has no closing ---
@@ -1102,6 +1104,11 @@ def install_ai_skills(project_path: Path, selected_ai: str, tracker: StepTracker
                 body = content
 
             command_name = command_file.stem
+            # Normalize: extracted commands may be named "speckit.<cmd>.md";
+            # strip the "speckit." prefix so skill names stay clean and
+            # SKILL_DESCRIPTIONS lookups work.
+            if command_name.startswith("speckit."):
+                command_name = command_name[len("speckit."):]
             skill_name = f"speckit-{command_name}"
 
             # Create skill directory (additive — never removes existing content)

tests/test_ai_skills.py

@@ -427,7 +427,7 @@ def test_existing_commands_preserved_gemini(self, project_dir, templates_dir, co
         """install_ai_skills must NOT remove pre-existing .gemini/commands files."""
         assert len(list(commands_dir_gemini.glob("speckit.*"))) == 3
 
-        install_ai_skills(project_dir, "claude")
+        install_ai_skills(project_dir, "gemini")
 
         remaining = list(commands_dir_gemini.glob("speckit.*"))
         assert len(remaining) == 3