feat(skills): document copilot workaround in --target=all output

  Installing 3 skills to 4 target(s):
    - claude: /Users/gilbertwong/.claude/skills
    - codex: /Users/gilbertwong/.codex/skills
    - cursor: /Users/gilbertwong/.cursor/skills
    - pi: /Users/gilbertwong/.pi/agent/skills

  Installed: 0
  Skipped:   12 (use --force to overwrite)
  Errors:    0 is documented to install
'to every known per-user target' (pi, claude, cursor, codex)
but NOT copilot, because copilot is per-repo, not per-user
(writes to <repo>/.github/ado-cli/, not ~/.copilot/...).

Users who expected copilot in --target=all were confused.
Now the output prints a one-liner when --target=all is used
and copilot isn't in the target list, telling them how to
install to copilot:

  Note: copilot installs per-repo (to <repo>/.github/ado-cli/).
        Run from inside your repo: ado skills install --target copilot

Also updated the --target option's docstring to make copilot's
per-repo nature more obvious in 'ado skills install --help'.

+ 1 new test (18 total skills tests, was 17).

mix ci: 8/8 steps green. 314 tests pass.

Author: gilbertwong96

CHANGELOG.md

@@ -9,6 +9,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+- **`ado skills install --target=all` now prints a hint about copilot**:
+  copilot is per-repo, not per-user, so it can't be included in
+  `--target=all`. The output now notes this and tells the user
+  how to install to copilot (`cd <repo> && ado skills install
+  --target copilot`). Help text also clarifies copilot's
+  per-repo nature.
 - **Code style cleanups addressing `mix ex_dna` and `mix reach`**
   warnings. Includes:
     * Extracted `confirm_delete/2` and `print_id_name_type_table/1`

lib/ado_cli/cli/skills.ex

@@ -103,15 +103,17 @@ defmodule AdoCli.CLI.Skills do
                 "Where to install: 'pi' (~/.pi/agent/skills/), " <>
                   "'claude' (~/.claude/skills/), 'cursor' (~/.cursor/skills/), " <>
                   "'codex' (~/.codex/skills/), " <>
-                  "'copilot' (requires --repo, writes to <repo>/.github/ado-cli/), " <>
-                  "or a custom absolute path. " <>
-                  "Default: 'all' (installs to every known per-user target)."
+                  "'copilot' (per-repo, requires --repo or cwd to be a git repo; " <>
+                  "writes to <repo>/.github/ado-cli/). " <>
+                  "Default: 'all' (installs to every per-user target above; " <>
+                  "copilot is NOT included because it needs a repo)."
             ],
             repo: [
               type: :string,
               doc:
-                "Path to a local git repository. Required when --target=copilot; " <>
-                  "default: current working directory. Ignored for other targets."
+                "Path to a local git repository. Used by --target=copilot " <>
+                  "(writes to <repo>/.github/ado-cli/); default: current " <>
+                  "working directory. Ignored for other targets."
             ],
             skill: [
               type: :string,
@@ -441,6 +443,17 @@ defmodule AdoCli.CLI.Skills do
       writeln("    - #{name}: #{path}")
     end)
 
+    # Copilot is intentionally not in --target=all because it
+    # installs per-repository (to <repo>/.github/ado-cli/), not
+    # per-user. When --target=all is the default and the user
+    # didn't explicitly ask for copilot, print a one-liner
+    # telling them how to install to it.
+    if Enum.all?(target_dirs, fn {name, _} -> name != "copilot" end) do
+      writeln("")
+      writeln("  Note: copilot installs per-repo (to <repo>/.github/ado-cli/).")
+      writeln("        Run from inside your repo: ado skills install --target copilot")
+    end
+
     writeln("")
 
     summarize_install_results(results)

test/ado_cli/cli/skills_test.exs

@@ -309,6 +309,30 @@ name: ado-cli")
       target_names = decoded["result"]["targets"] |> Enum.map(& &1["name"]) |> Enum.sort()
       assert target_names == ["claude", "codex", "cursor", "pi"]
     end
+
+    test "--target=all output mentions copilot workaround (it's per-repo, not in 'all')", %{
+      repo_dir: _repo_dir
+    } do
+      Skills.install_skills(%{
+        options: %{target: "all", force: false, json: false, skill: nil},
+        arguments: %{}
+      })
+
+      # Drain the writeln/info messages until we find the one that
+      # mentions copilot. The print order is: blank line, header,
+      # target list, copilot hint, blank line, summary.
+      hint = drain_info_until(fn m -> m =~ "ado skills install" end)
+      assert hint =~ "copilot"
+    end
+  end
+
+  # Recursively pull :cli_mate_shell :info messages off the test
+  # process mailbox and return the first one that matches the
+  # given predicate. Asserts (and thus fails the test) if the
+  # mailbox is exhausted before a match.
+  defp drain_info_until(predicate) do
+    assert_receive {:cli_mate_shell, :info, msg}, 1000
+    if predicate.(msg), do: msg, else: drain_info_until(predicate)
   end
 
   describe "install_skills/1 with --target codex" do