Fix skill installation to accept both stored keys and install key format from list command

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: James Zhu

code_assistant_manager/skills/manager.py

@@ -229,9 +229,25 @@ def get_all(self) -> Dict[str, Skill]:
         return self._load_skills()
 
     def get(self, skill_key: str) -> Optional[Skill]:
-        """Get a specific skill."""
+        """Get a specific skill by key (can be actual key or install key format repo:directory)."""
         skills = self._load_skills()
-        return skills.get(skill_key)
+
+        # First try direct lookup with the provided key
+        skill = skills.get(skill_key)
+        if skill:
+            return skill
+
+        # If not found, try to find by install key format (repo:directory)
+        if ":" in skill_key:
+            repo_part, directory_part = skill_key.rsplit(":", 1)
+            # Look for skills that match this repo and directory
+            for stored_skill in skills.values():
+                if (stored_skill.repo_owner and stored_skill.repo_name and
+                    f"{stored_skill.repo_owner}/{stored_skill.repo_name}" == repo_part and
+                    stored_skill.directory == directory_part):
+                    return stored_skill
+
+        return None
 
     def create(self, skill: Skill) -> None:
         """Create a new skill."""
@@ -259,10 +275,24 @@ def upsert(self, skill: Skill) -> None:
         logger.info(f"Upserted skill: {skill.key}")
 
     def delete(self, skill_key: str) -> None:
-        """Delete a skill."""
+        """Delete a skill by key (can be actual key or install key format repo:directory)."""
         skills = self._load_skills()
+
+        # First try direct lookup with the provided key
         if skill_key not in skills:
-            raise ValueError(f"Skill with key '{skill_key}' not found")
+            # If not found, try to find by install key format (repo:directory)
+            if ":" in skill_key:
+                repo_part, directory_part = skill_key.rsplit(":", 1)
+                # Look for skills that match this repo and directory
+                for stored_key, skill in skills.items():
+                    if (skill.repo_owner and skill.repo_name and
+                        f"{skill.repo_owner}/{skill.repo_name}" == repo_part and
+                        skill.directory == directory_part):
+                        skill_key = stored_key
+                        break
+                else:
+                    raise ValueError(f"Skill with key '{skill_key}' not found")
+
         del skills[skill_key]
         self._save_skills(skills)
         logger.info(f"Deleted skill: {skill_key}")
@@ -294,13 +324,32 @@ def install(self, skill_key: str, app_type: str = "claude") -> Path:
         """Install a skill for a specific app.
 
         Args:
-            skill_key: The skill identifier
+            skill_key: The skill identifier (can be actual key or install key format repo:directory)
             app_type: The app type to install to
 
         Returns:
             Path to the installed skill directory
+
+        Raises:
+            ValueError: If skill_key is not found
         """
         skills = self._load_skills()
+
+        # First try direct lookup with the provided key
+        if skill_key not in skills:
+            # If not found, try to find by install key format (repo:directory)
+            if ":" in skill_key:
+                repo_part, directory_part = skill_key.rsplit(":", 1)
+                # Look for skills that match this repo and directory
+                for stored_key, skill in skills.items():
+                    if (skill.repo_owner and skill.repo_name and
+                        f"{skill.repo_owner}/{skill.repo_name}" == repo_part and
+                        skill.directory == directory_part):
+                        skill_key = stored_key
+                        break
+                else:
+                    raise ValueError(f"Skill with key '{skill_key}' not found")
+
         if skill_key not in skills:
             raise ValueError(f"Skill with key '{skill_key}' not found")
 
@@ -320,10 +369,26 @@ def uninstall(self, skill_key: str, app_type: str = "claude") -> None:
         """Uninstall a skill from a specific app.
 
         Args:
-            skill_key: The skill identifier
+            skill_key: The skill identifier (can be actual key or install key format repo:directory)
             app_type: The app type to uninstall from
         """
         skills = self._load_skills()
+
+        # First try direct lookup with the provided key
+        if skill_key not in skills:
+            # If not found, try to find by install key format (repo:directory)
+            if ":" in skill_key:
+                repo_part, directory_part = skill_key.rsplit(":", 1)
+                # Look for skills that match this repo and directory
+                for stored_key, skill in skills.items():
+                    if (skill.repo_owner and skill.repo_name and
+                        f"{skill.repo_owner}/{skill.repo_name}" == repo_part and
+                        skill.directory == directory_part):
+                        skill_key = stored_key
+                        break
+                else:
+                    raise ValueError(f"Skill with key '{skill_key}' not found")
+
         if skill_key not in skills:
             raise ValueError(f"Skill with key '{skill_key}' not found")