test: tighten sync-skills option validation coverage

Signed-off-by: GitHub <noreply@github.com>

Co-authored-by: OhYee <13498329+OhYee@users.noreply.github.com>

Author: Copilot

src/agentrun_cli/commands/sync_skills_cmd.py

@@ -215,17 +215,18 @@ def sync_skills(
 ):
     """Sync platform skills to local AI tool skill directories."""
     if use_claude_code == use_codex:
-        raise click.UsageError("Exactly one of --claude-code or --codex is required.")
+        raise click.UsageError(
+            "You must specify exactly one of --claude-code or --codex."
+        )
     if user_scope == project_scope:
-        raise click.UsageError("Exactly one of --user or --project is required.")
+        raise click.UsageError("You must specify exactly one of --user or --project.")
 
     ai_tool = "claude-code" if use_claude_code else "codex"
     scope = "user" if user_scope else "project"
     target_dir = _resolve_target_dir(ai_tool, scope)
     os.makedirs(target_dir, exist_ok=True)
 
     profile, region = _ctx_cfg(ctx)
-    cfg = build_sdk_config(profile_name=profile, region=region)
     all_skills = _list_platform_skills(profile, region)
     requested_workspaces = set(workspaces or [])
 
@@ -262,6 +263,7 @@ def sync_skills(
 
     downloaded = []
     updated = []
+    cfg = build_sdk_config(profile_name=profile, region=region)
     from agentrun.tool import Tool
 
     for name, remote_updated_at, local_exists in to_sync:

tests/unit/test_sync_skills_cmd.py

@@ -44,12 +44,24 @@ def test_requires_exactly_one_tool(self):
         assert result.exit_code != 0
         assert "--claude-code or --codex" in result.output
 
+    def test_rejects_both_tool_flags(self):
+        runner = CliRunner()
+        result = runner.invoke(sync_skills, ["--claude-code", "--codex", "--user"])
+        assert result.exit_code != 0
+        assert "--claude-code or --codex" in result.output
+
     def test_requires_exactly_one_scope(self):
         runner = CliRunner()
         result = runner.invoke(sync_skills, ["--claude-code"])
         assert result.exit_code != 0
         assert "--user or --project" in result.output
 
+    def test_rejects_both_scope_flags(self):
+        runner = CliRunner()
+        result = runner.invoke(sync_skills, ["--claude-code", "--user", "--project"])
+        assert result.exit_code != 0
+        assert "--user or --project" in result.output
+
 
 class TestSyncSkillsCommand: