@execuro-sw-ecosystem/sw-ecosystem-agentic-harness 0.1.6 → 0.1.7

package/lib/render.mjs

@@ -0,0 +1,364 @@
+// The human view of a result body.
+//
+// Every command still builds exactly the same JSON object it always did — that
+// object is the model. This module is the view: pure `body -> string`
+// functions, no I/O, no process state, so the whole renderer is testable
+// without a terminal and without running an install.
+//
+// Design rules, in the order they matter:
+//
+//   1. Nothing changed is ONE line. A file count is not news to a human, and
+//      "274 unchanged" reads like a failure to someone who typed `install`.
+//   2. Something changed is reported per agent, in the user's vocabulary —
+//      skills and sub-agents by name, config files by path.
+//   3. Conflicts, drift, manual steps and failures are never collapsed into a
+//      count: those are the only cases the user has to act on, so each one
+//      names its file and its remedy.
+//   4. Never name an agent this project has not installed. A Claude Code user
+//      is not told about Codex, Cursor or Copilot.
+//
+// Zero dependencies: bare ANSI, and only when the caller says the stream is a
+// terminal.
+
+/** Human labels for the four supported coding agents. */
+export const LABELS = {
+  'claude-code': 'Claude Code',
+  codex: 'Codex',
+  copilot: 'GitHub Copilot',
+  cursor: 'Cursor',
+};
+
+/** What each agent needs before it reads the configuration we just wrote. */
+export const RESTART = {
+  'claude-code': 'Restart Claude Code, or run `/mcp` in the running session',
+  codex: 'Restart `codex` — .codex/config.toml is read once at startup',
+  copilot: 'Reload the VS Code window (Developer: Reload Window) for .vscode/mcp.json',
+  cursor: 'Restart Cursor so it re-reads .cursor/mcp.json',
+};
+
+function label(id) {
+  return LABELS[id] ?? id;
+}
+
+// ------------------------------------------------------------------ styling
+
+const CODES = { bold: '1', dim: '2', red: '31', green: '32', yellow: '33' };
+
+/** A theme whose functions are the identity when colour is off. */
+function theme(color) {
+  const wrap = (code) => (color ? (s) => `\x1b[${code}m${s}\x1b[0m` : (s) => s);
+  const t = {};
+  for (const [name, code] of Object.entries(CODES)) t[name] = wrap(code);
+  return t;
+}
+
+// --------------------------------------------------------------- primitives
+
+const MAX_NAMES = 6;
+
+/** "a, b, c and 4 more" — a list a person reads, not a dump. */
+function names(list) {
+  const max = MAX_NAMES;
+  const all = [...list];
+  if (all.length <= max) return all.join(', ');
+  return `${all.slice(0, max).join(', ')} and ${all.length - max} more`;
+}
+
+function plural(n, one, many = `${one}s`) {
+  return `${n} ${n === 1 ? one : many}`;
+}
+
+/**
+ * What a target is, in the words a user thinks in. A skill is one thing to
+ * them even though it is a directory of files; a config file is its path.
+ */
+export function classify(target) {
+  const skill = /(?:^|\/)skills\/([^/]+)\//.exec(target);
+  if (skill) return { kind: 'skill', name: skill[1] };
+  const agent = /(?:^|\/)agents\/([^/]+)$/.exec(target);
+  if (agent) return { kind: 'agent', name: agent[1].replace(/(\.agent)?\.(md|toml)$/, '') };
+  return { kind: 'file', name: target };
+}
+
+const VERBS = {
+  create: 'added', created: 'added',
+  update: 'updated', updated: 'updated',
+  removed: 'removed', reverted: 'reverted',
+};
+
+const SHARED = '__shared__';
+
+/**
+ * Group the actions in `states` by agent, then by verb, then by what kind of
+ * thing they are. Returns `[{ id, buckets: Map<verb, {skills, agents, files}> }]`
+ * in the order the agents first appear.
+ */
+export function groupChanges(actions = [], states) {
+  const wanted = new Set(states);
+  const order = [];
+  const groups = new Map();
+  for (const action of actions) {
+    if (!wanted.has(action.state)) continue;
+    const id = !action.host || action.host === 'shared' ? SHARED : action.host;
+    if (!groups.has(id)) { groups.set(id, new Map()); order.push(id); }
+    const buckets = groups.get(id);
+    const verb = VERBS[action.state] ?? action.state;
+    if (!buckets.has(verb)) buckets.set(verb, { skill: new Set(), agent: new Set(), file: new Set() });
+    const bucket = buckets.get(verb);
+    const { kind, name } = classify(action.target);
+    // Several actions can target one config file (one per JSON pointer);
+    // the user changed one file, so it is named once.
+    bucket[kind].add(name);
+    (bucket.targets ??= []).push(action.target);
+  }
+  return order.map((id) => ({ id, buckets: groups.get(id) }));
+}
+
+function bucketLines(verb, bucket, { verbose }, t) {
+  const lines = [];
+  if (verbose) {
+    for (const target of bucket.targets ?? []) lines.push(`    ${t.dim(verb)} ${target}`);
+    return lines;
+  }
+  if (bucket.skill.size) lines.push(`    ${verb} ${plural(bucket.skill.size, 'skill')}: ${names(bucket.skill)}`);
+  if (bucket.agent.size) lines.push(`    ${verb} ${plural(bucket.agent.size, 'sub-agent')}: ${names(bucket.agent)}`);
+  if (bucket.file.size) lines.push(`    ${verb} ${names(bucket.file)}`);
+  return lines;
+}
+
+function groupLines(groups, opts, t) {
+  const lines = [];
+  for (const { id, buckets } of groups) {
+    lines.push(`  ${t.bold(id === SHARED ? 'Project files' : label(id))}`);
+    for (const [verb, bucket] of buckets) lines.push(...bucketLines(verb, bucket, opts, t));
+  }
+  return lines;
+}
+
+// ------------------------------------------------------------------- issues
+
+/** Everything the user has to act on, flattened into one shape. */
+function issues(body) {
+  const found = [];
+  for (const a of body.actions ?? []) {
+    if (a.state === 'conflict' || a.state === 'drift' || a.state === 'failed') {
+      found.push({ kind: a.state, target: a.target, reason: a.reason, remedy: a.remedy });
+    }
+  }
+  for (const d of body.drift ?? []) {
+    found.push({ kind: d.state ?? 'drift', target: d.target, reason: d.reason, remedy: d.remedy });
+  }
+  for (const m of body.manual ?? []) {
+    found.push({ kind: 'manual', target: m.target ?? m.file, reason: m.reason, remedy: m.remedy });
+  }
+  for (const e of body.errors ?? []) {
+    found.push({ kind: 'failed', target: e.target ?? label(e.host), reason: e.error });
+  }
+  // The same file can reach us from both `actions` and `drift`; say it once.
+  const seen = new Set();
+  return found.filter((i) => {
+    const key = `${i.kind}:${i.target}`;
+    if (seen.has(key)) return false;
+    seen.add(key);
+    return true;
+  });
+}
+
+const ISSUE_TITLE = {
+  conflict: 'conflict', drift: 'edited since install',
+  manual: 'manual step', failed: 'failed',
+};
+
+function issueLines(list, t) {
+  if (!list.length) return [];
+  const paint = (kind) => (kind === 'drift' || kind === 'manual' ? t.yellow : t.red);
+  const lines = [`  ${t.bold('Needs your attention')}`];
+  for (const i of list) {
+    lines.push(`    ${paint(i.kind)(ISSUE_TITLE[i.kind] ?? i.kind)}  ${i.target}`);
+    if (i.reason) lines.push(`      ${i.reason}`);
+    if (i.remedy) lines.push(`      ${t.dim('fix:')} ${i.remedy}`);
+  }
+  return lines;
+}
+
+// ------------------------------------------------------------------ restart
+
+function restartLines(body, changedIds, t) {
+  const ids = changedIds.length
+    ? changedIds
+    : (body.agents ?? []).map((a) => a.id).filter((id) => RESTART[id]);
+  const uniq = [...new Set(ids)].filter((id) => RESTART[id]);
+  return uniq.map((id) => `  ${t.dim('→')} ${RESTART[id]}`);
+}
+
+// ------------------------------------------------------------ apply/install
+
+function renderApply(body, opts, t) {
+  const s = body.summary ?? {};
+  const changed = (s.created ?? 0) + (s.updated ?? 0);
+  const problems = issues(body);
+
+  if (changed === 0 && problems.length === 0) {
+    // ONE line. Nothing changed means nothing to restart and nothing to read.
+    return `${t.green('✓')} Shopware agentic harness: already up to date — nothing to change.\n`;
+  }
+
+  const groups = groupChanges(body.actions, ['created', 'updated']);
+  const changedIds = groups.map((g) => g.id).filter((id) => id !== SHARED);
+  const verb = (s.updated ?? 0) === 0 ? 'Installed' : 'Updated';
+  const lines = [];
+  lines.push(changed
+    ? `${t.green('✓')} ${verb} the Shopware agentic harness.`
+    : `${t.yellow('!')} Nothing was written — see below.`);
+  if (groups.length) {
+    lines.push('');
+    lines.push(...groupLines(groups, opts, t));
+  }
+  if (problems.length) {
+    lines.push('');
+    lines.push(...issueLines(problems, t));
+  }
+  if (changed) {
+    lines.push('');
+    lines.push(...restartLines(body, changedIds, t));
+  }
+  return `${lines.join('\n')}\n`;
+}
+
+// -------------------------------------------------------------------- plan
+
+function renderPlan(body, opts, t) {
+  const s = body.summary ?? {};
+  const changes = (s.create ?? 0) + (s.update ?? 0);
+  const problems = issues(body);
+  if (changes === 0 && problems.length === 0) {
+    return `${t.green('✓')} Nothing to do — every target is already up to date.\n`;
+  }
+  const lines = [];
+  // `plan` with nothing selected previews every supported agent. Say so, or
+  // the list silently names agents this project never installed.
+  if (/previews all four/i.test(body.next_step ?? '')) {
+    lines.push(t.dim('No agent selected and no recorded install — previewing every supported agent.'));
+  }
+  lines.push(`${plural(changes, 'change')} would be made. Nothing was written.`);
+  const groups = groupChanges(body.actions, ['create', 'update']);
+  if (groups.length) {
+    lines.push('');
+    lines.push(...groupLines(groups, opts, t));
+  }
+  if (problems.length) {
+    lines.push('');
+    lines.push(...issueLines(problems, t));
+  }
+  if (changes) {
+    lines.push('');
+    lines.push(`  ${t.dim('→')} Run \`sw-ecosystem-agentic-harness apply --yes\` to perform them.`);
+  }
+  return `${lines.join('\n')}\n`;
+}
+
+// ------------------------------------------------------------------ status
+
+/** An agent is only ours to talk about once it has actually been installed. */
+function installedAgents(body) {
+  return (body.agents ?? []).filter((a) => ['ok', 'drift', 'conflict'].includes(a.state));
+}
+
+function renderStatus(body, opts, t) {
+  const mine = installedAgents(body);
+  const problems = issues(body);
+
+  if (!mine.length) {
+    return `${t.yellow('!')} No coding agent is set up in this project yet.\n`
+      + `  ${t.dim('→')} ${body.help?.[0] ?? 'sw-ecosystem-agentic-harness install'}\n`;
+  }
+
+  const worst = ['conflict', 'drift'].find((state) => mine.some((a) => a.state === state));
+  const lines = [];
+  lines.push(worst
+    ? `${t.yellow('!')} Shopware agentic harness ${body.installed_version ?? body.version} — ${plural(problems.length, 'item')} ${problems.length === 1 ? 'needs' : 'need'} attention.`
+    : `${t.green('✓')} Shopware agentic harness ${body.installed_version ?? body.version} — installed and up to date.`);
+
+  for (const a of mine) {
+    const counts = [];
+    if (a.installed?.skills) counts.push(plural(a.installed.skills, 'skill'));
+    if (a.installed?.agents) counts.push(plural(a.installed.agents, 'sub-agent'));
+    lines.push(`  ${t.bold(label(a.id))}${counts.length ? `: ${counts.join(', ')}` : ''}`);
+    for (const note of a.notes ?? []) lines.push(`    ${t.dim(note)}`);
+  }
+
+  if (problems.length) {
+    lines.push('');
+    lines.push(...issueLines(problems, t));
+  }
+
+  const missing = (body.extra_components ?? []).filter((c) => !c.available && c.install);
+  if (missing.length) {
+    lines.push('');
+    lines.push(`  ${t.dim('Optional add-ons not installed:')} ${missing.map((c) => c.install).join(', ')}`);
+  }
+  return `${lines.join('\n')}\n`;
+}
+
+// --------------------------------------------------------------- uninstall
+
+function renderUninstall(body, opts, t) {
+  const s = body.summary ?? {};
+  const gone = (s.removed ?? 0) + (s.reverted ?? 0);
+  if (gone === 0 && !(body.kept ?? []).length) {
+    return `${t.green('✓')} Nothing to remove — this project has no recorded install.\n`;
+  }
+  const lines = [`${t.green('✓')} Removed the Shopware agentic harness — `
+    + `${plural(s.removed ?? 0, 'file')} removed, ${plural(s.reverted ?? 0, 'config block')} reverted.`];
+  if ((body.kept ?? []).length) {
+    lines.push('');
+    lines.push(`  ${t.bold('Kept, because you edited them')}`);
+    for (const k of body.kept) lines.push(`    ${k.target}`);
+  }
+  lines.push('');
+  lines.push(`  ${t.dim('→')} Restart your coding agent(s) so they drop the removed configuration.`);
+  return `${lines.join('\n')}\n`;
+}
+
+// ------------------------------------------------------------------ errors
+
+function renderProblem(body, opts, t) {
+  const lines = [`${t.red('✗')} ${body.error}`];
+  for (const e of body.errors ?? []) {
+    lines.push(`    ${e.target ?? label(e.host)} — ${e.error}`);
+  }
+  const problems = issues({ ...body, errors: [] });
+  if (problems.length) {
+    lines.push('');
+    lines.push(...issueLines(problems, t));
+  }
+  if (body.help?.length) {
+    lines.push('');
+    lines.push(`  ${t.bold('Try')}`);
+    for (const h of body.help) lines.push(`    ${h}`);
+  }
+  return `${lines.join('\n')}\n`;
+}
+
+// ---------------------------------------------------------------- dispatch
+
+const RENDERERS = {
+  install: renderApply,
+  apply: renderApply,
+  plan: renderPlan,
+  status: renderStatus,
+  uninstall: renderUninstall,
+};
+
+/**
+ * `body` is the exact object `--json` prints. `opts.verbose` prints the
+ * per-file detail the JSON carries; `opts.color` enables ANSI.
+ */
+export function render(body, opts = {}) {
+  if (!body || typeof body !== 'object') return '';
+  const t = theme(opts.color === true);
+  if (body.ok === false) return renderProblem(body, opts, t);
+  const renderer = RENDERERS[body.command];
+  if (!renderer) return `${body.next_step ?? ''}\n`;
+  return renderer(body, opts, t);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@execuro-sw-ecosystem/sw-ecosystem-agentic-harness",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Shopware 6 agentic harness: sw-* skills and sub-agents, plus an installer that writes them into Claude Code, Codex, GitHub Copilot and Cursor.",
   "license": "MIT",
   "author": "Execuro UG (haftungsbeschränkt)",

package/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://agent-plugins.org/schemas/1.0.0/plugin.schema.json",
   "name": "sw-ecosystem-agentic-harness",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Shopware 6 agentic harness: sw-* skills and sub-agents for requirements, specs, implementation, verification and documentation, with an installer CLI that writes them into Claude Code, Codex, GitHub Copilot and Cursor. The knowledge-base MCP and the editor tools are separate npm packages.",
   "author": {
     "name": "Execuro UG (haftungsbeschränkt)",

package/skills/sw-setup/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: sw-setup
-description: Print one environment-readiness table for the project's development/test setup — vendor/, Node, the editor CLIs, shopware-cli, the KB MCP, the acceptance-test project, Playwright browsers, its .env, per-plugin test scaffolding, the project wiki, `.gitignore` and whether the SW AH npm packages are current, plus the installer CLI's own configuration/drift status. Stops when every row is ticked. When rows are missing, asks one yes/no question and delegates the fix to the installer CLI. Use when asked to set up the project, check if the environment is ready, or bootstrap the tests. Never designs, implements or verifies a feature itself — that's sw-design-solution, sw-implement-feature and sw-verify-feature, which call this skill first. Never writes a host configuration file itself — that's the installer CLI.
+description: Print one environment-readiness table for the project's development/test setup — vendor/, Node, the editor CLIs, shopware-cli, the KB MCP, the acceptance-test project, Playwright browsers, its .env, per-plugin test scaffolding, the project wiki, `.gitignore` and whether the SW AH npm packages are current, plus the installer CLI's own configuration/drift status. Stops when every row is ticked. When rows are missing, asks one yes/no question and delegates the fix to the installer CLI. Use when asked to set up the project, check if the environment is ready, or bootstrap the tests. Never designs, implements or verifies a feature itself — that's sw-design-solution, sw-implement-feature and sw-verify-feature, which call this skill first. Never writes a coding agent's configuration file itself — that's the installer CLI.
 when_to_use: Trigger phrases — "set up the project", "is the environment ready", "bootstrap the tests", "sw-setup".
 allowed-tools: Read Glob Grep AskUserQuestion Skill Bash mcp__ShopwareDevKnowledgeBase__kb_status Bash(npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest *)
 ---
@@ -40,11 +40,13 @@ table from this run's output — it does not re-run `sw-setup`'s checks.
 Run:
 
 ```
-npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest status
+npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest status --json
 ```
 
-It prints one JSON object (`ok`, `next_step`, `help`, and its own
-`hosts[]`/`drift[]` configuration state). Alongside it, run every
+`--json` is mandatory for every call this skill makes: without it the CLI
+prints a human summary, not the object below. It prints one JSON object
+(`ok`, `next_step`, `help`, and its own `agents[]`/`drift[]` configuration
+state). Alongside it, run every
 environment row's check from `reference/rows.md` in this skill's directory —
 read-only, seconds each: file/directory existence, one version command, one
 `kb_status` call, three `npm view` registry lookups (Package updates row) —
@@ -52,7 +54,7 @@ the one part of step 1 that touches the network, so it is the slow part.
 Never write, never install, never ask a question in this step.
 
 Print one checkbox list, one line per row — the environment rows first, then
-one line per configuration item the CLI's `hosts[]`/`drift[]` reports:
+one line per configuration item the CLI's `agents[]`/`drift[]` reports:
 
 ```
 - [x] vendor/ — shopware/core 6.7.13.0
@@ -88,7 +90,7 @@ For a missing/drifted configuration row, do not research or plan the fix
 yourself — the CLI's `plan` command already has it:
 
 ```
-npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest plan
+npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest plan --json
 ```
 
 List the concrete steps in plain words: environment-row fixes in order
@@ -104,7 +106,7 @@ if you have one (Claude Code: `AskUserQuestion`; Codex: `request_user_input`,
 which is non-blocking outside Plan mode — so end the turn after asking).
 Options:
 - **Yes, run it** — run the environment-row fixes in order, then
-  `npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest apply --yes`
+  `npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest apply --yes --json`
   for the configuration changes, then re-run step 1 and print the final table.
 - **No, stop** — stop; the table from step 1 is already on screen.
 
@@ -124,11 +126,11 @@ Options:
   the corpus ships already built inside the
   `@execuro-sw-ecosystem/sw-dev-knowledge-base-mcp` package. If `platform` is
   not `implemented`, the fix is to check the registration (re-run the
-  installer CLI's `apply --yes`) or update that package — never build a
+  installer CLI's `apply --yes --json`) or update that package — never build a
   corpus here. Neither is replicated in this skill.
-- This skill never writes a host configuration file itself (`.mcp.json`,
-  permission grants, `.gitignore` lines). Every such write goes through the
-  installer CLI's `apply --yes`, run only after the user's yes.
+- This skill never writes a coding agent's configuration file itself
+  (`.mcp.json`, permission grants, `.gitignore` lines). Every such write goes
+  through the installer CLI's `apply --yes --json`, after the user's yes.
 
 ### 5. Report
 
@@ -144,6 +146,6 @@ line:
 
 `reference/rows.md` in this skill's directory — the twelve environment rows:
 check command(s), what "ticked" means, the fix steps verbatim from the
-guideline, and which skills the row blocks. Host configuration state (what
+guideline, and which skills the row blocks. Agent configuration state (what
 gets installed into `.mcp.json`, permissions, `.gitignore`) is not in this
 file — it comes from the installer CLI's `status`/`plan`/`apply` output.

package/skills/sw-setup/reference/rows.md

@@ -2,19 +2,25 @@
 
 Twelve environment rows. Each: check (read-only), what "ticked" means, the fix
 (only run on approval, in this order), and which skills the row blocks when
-unticked. Host configuration files (`.mcp.json` entries, `.gitignore` lines,
-permission grants) are not rows here — the installer CLI owns them and
-reports their state in its own `status`/`plan` output; see `SKILL.md`.
+unticked. Coding-agent configuration files (`.mcp.json` entries, `.gitignore`
+lines, permission grants) are not rows here — the installer CLI owns them and
+reports their state in its own `status --json`/`plan --json` output; see
+`SKILL.md`.
 
-When that `status` call reports no install at all (no lock file, so no
-recorded hosts), the installer CLI's `apply --yes`/`install` has no `--host`
+Every installer-CLI call this skill makes passes `--json`. Without it the CLI
+prints a short human summary instead of the object these rows read — that
+summary is for the person typing the command, never for a skill parsing it.
+
+When that `status --json` call reports no install at all (no lock file, so no
+recorded agents), the installer CLI's `apply --yes`/`install` has no `--agent`
 to reuse and, run from this skill, no terminal to ask on — it would exit 2.
-Do not pass a guessed `--host`. Report this to the user instead: the first
+Do not pass a guessed `--agent`. Report this to the user instead: the first
 install must be run by them directly in a terminal, e.g.
 `npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest install`,
-which then either takes their `--host` or prompts them interactively. Re-run
-this skill's `status` check afterwards — every later `apply --yes` reuses the
-hosts that first run recorded, and this skill can call it directly again.
+which then either takes their `--agent` or prompts them interactively. Re-run
+this skill's `status` check afterwards — every later `apply --yes --json`
+reuses the agents that first run recorded, and this skill can call it
+directly again. Expect `agents[]` in every output this skill reads.
 
 ## 1. vendor/
 
@@ -60,11 +66,11 @@ row never runs an extra component's own install command.
 So this row never blocks readiness. It asks.
 
 - **Check:** read `extra_components[]` from the `status` output the skill already
-  ran in step 1 — each entry's `available`, `installed_for` (the hosts it is
-  placed in), and `install` (the npm command, when missing). Never probe a
+  ran in step 1 — each entry's `available`, `installed_for` (the coding agents
+  it is placed in), and `install` (the npm command, when missing). Never probe a
   path and never run an extra component by hand; the CLI owns both.
 - **Ticked:** `[x]` when an extra component's `available` is true and
-  `installed_for` includes this host; `[-]` when the user has declined it.
+  `installed_for` includes this agent; `[-]` when the user has declined it.
   Only an unanswered offer shows `[ ]`, and even then it does not hold the
   table open — report it as optional and move on.
 - **Fix:** offer each missing extra component inside step 3's single question,
@@ -76,10 +82,10 @@ So this row never blocks readiness. It asks.
   `package` field from `extra_components[]`), then re-run
 
   ```
-  npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest apply --yes
+  npx -y @execuro-sw-ecosystem/sw-ecosystem-agentic-harness@latest apply --yes --json
   ```
 
-  which places one `SKILL.md` per extra component into each selected host's own
+  which places one `SKILL.md` per extra component into each selected agent's own
   skills directory, records it, and removes it again on uninstall. **Never
   run the extra component's own `install-skill`** — with no `--target` it
   defaults to Claude Code's own skills directory, wrong for a Codex, Copilot