fix: address review findings for install command

- No-flag usage now exits with code 83 (validation error) instead of 0
- Print "Replacing existing skills" message before rmtree
- Add integration test for install(skills=True) happy path
- Mention `install --skills` in SKILL.md setup section

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

Author: jschfflr

src/qodev_gitlab_cli/commands/install.py

@@ -23,6 +23,7 @@ def _install_skills(target_root: Path | None = None) -> Path:
     source = files("qodev_gitlab_cli") / "skills"
 
     if dest.exists():
+        console.print(f"Replacing existing skills at {dest}")
         shutil.rmtree(dest)
     dest.mkdir(parents=True)
 
@@ -49,10 +50,10 @@ def install(
     skills: Annotated[bool, Parameter(name="--skills", help="Install AI agent skill files", negative="")] = False,
 ) -> None:
     """Install CLI resources into the current workspace."""
-    if skills:
-        dest = _install_skills()
-        console.print(f"[green]Installed skills to {dest}[/green]")
-    else:
-        console.print("Usage: qodev-gitlab install --skills")
-        console.print("")
-        console.print("  --skills  Copy AI agent skill files to .claude/skills/qodev-gitlab/")
+    if not skills:
+        from qodev_gitlab_cli.output import error
+
+        error("No install target specified. Use: qodev-gitlab install --skills", code="validation", exit_code=83)
+
+    dest = _install_skills()
+    console.print(f"[green]Installed skills to {dest}[/green]")

src/qodev_gitlab_cli/skills/SKILL.md

@@ -7,6 +7,9 @@ Agent-friendly CLI for the GitLab API. Designed for AI coding agents with struct
 ```bash
 pip install qodev-gitlab-cli
 export GITLAB_TOKEN="glpat-..."
+
+# Install skill files into the current workspace
+qodev-gitlab install --skills
 ```
 
 The CLI auto-detects the current GitLab project from the git remote. Override with `--project GROUP/NAME` or `-p GROUP/NAME`.

tests/test_install.py

@@ -5,7 +5,9 @@
 from pathlib import Path
 from unittest.mock import patch
 
-from qodev_gitlab_cli.commands.install import _install_skills
+import pytest
+
+from qodev_gitlab_cli.commands.install import _install_skills, install
 
 
 class TestInstallSkills:
@@ -44,11 +46,15 @@ def test_replaces_existing_directory(self, tmp_path: Path) -> None:
         assert not stale.exists()
         assert (dest / "SKILL.md").is_file()
 
-    def test_no_flag_shows_guidance(self) -> None:
-        from qodev_gitlab_cli.commands.install import install
-
-        with patch("qodev_gitlab_cli.commands.install.console") as mock_console:
+    def test_no_flag_exits_with_validation_error(self) -> None:
+        with pytest.raises(SystemExit, match="83"):
             install(skills=False)
 
+    def test_skills_flag_prints_success(self, tmp_path: Path) -> None:
+        with patch("qodev_gitlab_cli.commands.install.console") as mock_console, \
+             patch("qodev_gitlab_cli.commands.install.Path") as mock_path:
+            mock_path.cwd.return_value = tmp_path
+            install(skills=True)
+
         calls = [str(c) for c in mock_console.print.call_args_list]
-        assert any("--skills" in c for c in calls)
+        assert any("Installed skills" in c for c in calls)