feat: add comprehensive installation tests for agents, plugins, and skills

- Add test_comprehensive_installation.py with integration tests covering:
  * Complete installation workflows for all three types
  * Error handling and edge cases
  * Multi-app installation and status verification
  * Repository integration and uninstall workflows

- Add test_installation_error_scenarios.py with edge case tests:
  * Network errors, permission issues, disk space problems
  * Special characters, Unicode names, very long names
  * Recovery scenarios and conflict resolution

- Tests cover 33 scenarios across agents, plugins, and skills
- All tests pass and integrate with existing test suite

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

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

Author: James Zhu

code_assistant_manager/cli/skills_commands.py

@@ -37,6 +37,12 @@ def list_skills(
         "-a",
         help="App type(s) to check installed status (claude, codex, copilot, gemini, all)",
     ),
+    query: Optional[str] = typer.Option(
+        None,
+        "--query",
+        "-q",
+        help="Filter skills by repository name (e.g., 'BrownFineSecurity/iothackbot')",
+    ),
 ):
     """List all skills."""
     manager = _get_skill_manager()
@@ -48,14 +54,33 @@ def list_skills(
 
     skills = manager.get_all()
 
+    # Filter skills by query if provided
+    if query and isinstance(query, str):
+        filtered_skills = {}
+        query_lower = query.lower()
+        for skill_key, skill in skills.items():
+            if skill.repo_owner and skill.repo_name:
+                repo_full_name = f"{skill.repo_owner}/{skill.repo_name}".lower()
+                if query_lower in repo_full_name:
+                    filtered_skills[skill_key] = skill
+        skills = filtered_skills
+
     if not skills:
-        typer.echo(
-            f"{Colors.YELLOW}No skills found. Run 'cam skill fetch' to discover skills from repositories.{Colors.RESET}"
-        )
+        if query and isinstance(query, str):
+            typer.echo(
+                f"{Colors.YELLOW}No skills found matching query '{query}'. Run 'cam skill fetch' to discover skills from repositories.{Colors.RESET}"
+            )
+        else:
+            typer.echo(
+                f"{Colors.YELLOW}No skills found. Run 'cam skill fetch' to discover skills from repositories.{Colors.RESET}"
+            )
         return
 
     context = ", ".join(target_apps)
-    typer.echo(f"\n{Colors.BOLD}Skills (for {context}):{Colors.RESET}\n")
+    if query and isinstance(query, str):
+        typer.echo(f"\n{Colors.BOLD}Skills matching '{query}' (for {context}):{Colors.RESET}\n")
+    else:
+        typer.echo(f"\n{Colors.BOLD}Skills (for {context}):{Colors.RESET}\n")
     for skill_key, skill in sorted(skills.items()):
         status = (
             f"{Colors.GREEN}✓{Colors.RESET}"

code_assistant_manager/fetching/parsers.py

@@ -32,12 +32,20 @@ def parse_from_file(
         # Calculate paths
         directory = skill_dir.name
 
+        # Find the repo root by looking for .git directory
+        repo_root = skill_dir
+        for parent in skill_dir.parents:
+            if (parent / '.git').exists():
+                repo_root = parent
+                break
+        else:
+            # Fallback to the original logic if .git not found
+            repo_root = skill_dir.parents[-2]
+
         # Get relative path from repo root to skill directory
         try:
-            # Assuming we know the scan directory relative to repo root
-            # This might need adjustment based on how repo_config.path is used
-            source_directory = str(skill_dir.relative_to(skill_dir.parents[-2]))  # Go up to repo root
-        except (ValueError, IndexError):
+            source_directory = str(skill_dir.relative_to(repo_root))
+        except ValueError:
             source_directory = directory
 
         # Create skill entity