Skip to content

Commit 897b71e

Browse files
committed
fix(cli): scope skills install to targets that support skills
Drops the six rules-file-only targets that have no native skills directory (they route to the manual install path), prints the install summary only when something was written, and fixes two markdown lint warnings in the skill files.
1 parent e48038c commit 897b71e

3 files changed

Lines changed: 9 additions & 23 deletions

File tree

packages/cli-v3/skills/authoring-chat-agent/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ frontend, narrow `useChat` with `InferChatUIMessage<typeof myChat>` from `@trigg
217217

218218
`chat.agent` accepts hooks that fire in a fixed per-turn order:
219219

220-
```
220+
```text
221221
onValidateMessages -> hydrateMessages -> onChatStart (chat's first message only)
222222
-> onTurnStart -> run() -> onBeforeTurnComplete -> onTurnComplete
223223
```

packages/cli-v3/skills/chat-agent-advanced/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,7 @@ TS helpers `SSEStreamSubscription` and `controlSubtype(headers)` (documented in
348348
> Note on docs vocabulary: agent-side examples in some docs still use the legacy
349349
> `trigger:turn-complete` chunk type. That is the agent-emit vocabulary. A custom **reader** must
350350
> filter on the `trigger-control` header, not on `chunk.type`.
351-
351+
>
352352
> MCP-driven agent chats (`list_agents`, `start_agent_chat`, `send_agent_message`,
353353
> `close_agent_chat`) are MCP server tools used from Claude Code / Cursor, not importable SDK
354354
> functions. See `/mcp-tools#agent-chat-tools`.

packages/cli-v3/src/commands/skills.ts

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,10 @@ import { pathExists, readFile, safeWriteFile } from "../utilities/fileSystem.js"
2525
import { printStandloneInitialBanner } from "../utilities/initialBanner.js";
2626
import { logger } from "../utilities/logger.js";
2727

28-
const targets = [
29-
"claude-code",
30-
"cursor",
31-
"vscode",
32-
"windsurf",
33-
"gemini-cli",
34-
"cline",
35-
"agents.md",
36-
"amp",
37-
"kilo",
38-
"ruler",
39-
] as const;
28+
// Only tools with a native agent-skills directory. Rules-file-only tools (windsurf,
29+
// gemini-cli, cline, amp, kilo, ruler) don't support the Agent Skills format yet, so
30+
// they fall under the "Unsupported target" manual path rather than silently no-op.
31+
const targets = ["claude-code", "cursor", "vscode", "agents.md"] as const;
4032

4133
type TargetLabels = {
4234
[key in (typeof targets)[number]]: string;
@@ -45,14 +37,8 @@ type TargetLabels = {
4537
const targetLabels: TargetLabels = {
4638
"claude-code": "Claude Code",
4739
cursor: "Cursor",
48-
vscode: "VSCode",
49-
windsurf: "Windsurf",
50-
"gemini-cli": "Gemini CLI",
51-
cline: "Cline",
40+
vscode: "VSCode (Copilot)",
5241
"agents.md": "AGENTS.md (OpenAI Codex CLI, Jules, OpenCode)",
53-
amp: "Sourcegraph AMP",
54-
kilo: "Kilo Code",
55-
ruler: "Ruler",
5642
};
5743

5844
type SupportedTargets = (typeof targets)[number];
@@ -74,7 +60,7 @@ export function configureSkillsCommand(program: Command) {
7460
.description("Install the Trigger.dev agent skills into your coding agent")
7561
.option(
7662
"--target <targets...>",
77-
"Choose the target (or targets) to install the Trigger.dev skills into. We currently support: " +
63+
"Choose the target (or targets) to install the Trigger.dev skills into. Native install is supported for: " +
7864
targets.join(", ")
7965
)
8066
.option(
@@ -242,7 +228,7 @@ async function installSkills(manifest: RulesManifest, opts: SkillsWizardOptions)
242228
}
243229
}
244230

245-
if (results.length > 0) {
231+
if (results.some((r) => r.installations.length > 0 || r.pointer)) {
246232
log.step("Installed the following skills:");
247233

248234
for (const r of results) {

0 commit comments

Comments
 (0)