Skip to content

Commit a1778be

Browse files
committed
Fix prettier formatting for git-worktrees skill
1 parent 84074e7 commit a1778be

4 files changed

Lines changed: 13 additions & 16 deletions

File tree

docs/ARCHITECTURE.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -422,12 +422,12 @@ Each `<base>/<skill-name>/SKILL.md` is one skill. Discovery dedupes by directory
422422

423423
A skill file begins with a YAML frontmatter block, followed by the body that holds the instructions. Discovery parses `description` and `disable-model-invocation`; `loadSkillCommands` also reads `user-invocable`. The skill's identifier (what `use_skill` and `/<skill-name>` take) is its directory name. A skill with no `SKILL.md` or an empty body is skipped.
424424

425-
| Field | Required | Description |
426-
| --------------------------- | ------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
427-
| `description` | yes | One-line summary shown in the prompt's lazy skills listing and the slash picker |
428-
| `name` | conventional | Conventionally matches the directory name; the directory name is what is actually used as the identifier |
429-
| `user-invocable` | no | When `false`, `loadSkillCommands` skips slash synthesis; the skill remains `use_skill` only. Untagged skills still become slashes (marketplace BC) |
430-
| `disable-model-invocation` | no | When `true`, `discoverSkills` omits the skill from the lazy listing (but still claims the name for first-wins). Explicit `resolveSkillBody` / `use_skill("name")` still loads the body. Does not affect slash emission. |
425+
| Field | Required | Description |
426+
| -------------------------- | ------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
427+
| `description` | yes | One-line summary shown in the prompt's lazy skills listing and the slash picker |
428+
| `name` | conventional | Conventionally matches the directory name; the directory name is what is actually used as the identifier |
429+
| `user-invocable` | no | When `false`, `loadSkillCommands` skips slash synthesis; the skill remains `use_skill` only. Untagged skills still become slashes (marketplace BC) |
430+
| `disable-model-invocation` | no | When `true`, `discoverSkills` omits the skill from the lazy listing (but still claims the name for first-wins). Explicit `resolveSkillBody` / `use_skill("name")` still loads the body. Does not affect slash emission. |
431431

432432
There is no skill `type` field required for model invocation — a skill body is plain instruction text. Background libraries (e.g. `git-worktrees`) set both `user-invocable: false` and `disable-model-invocation: true` so they are absent from slash and listing, yet recipes can still `use_skill("git-worktrees")`. `argument-hint` on frontmatter is preserved for the slash picker (greyed arg guidance). Multi-step orchestration is a separate mechanism (see Workflows above), not a skill `type`.
433433

src/agent/directors/tool-sets.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,12 +59,7 @@ export const DOCS_TOOLS = [
5959
export const REVIEW_TOOLS = [...READ_TOOLS, ...PRODUCT_WRITE_TOOLS] as const;
6060

6161
/** Mechanical intern: shell-first + path writes when the brief requires them. */
62-
export const INTERN_TOOLS = [
63-
"run_shell",
64-
"read_file",
65-
"list_dir",
66-
...PRODUCT_WRITE_TOOLS,
67-
] as const;
62+
export const INTERN_TOOLS = ["run_shell", "read_file", "list_dir", ...PRODUCT_WRITE_TOOLS] as const;
6863

6964
/** Nested orchestrator surface (greybeard / package filter): dispatch + path writes. */
7065
export const ORCHESTRATOR_TOOLS = [

src/agent/tools.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,11 @@ import {
3939
type SubAgentProvider,
4040
type SubAgentSessionStore,
4141
} from "../subagent/index.js";
42-
import { createFleetRecords, createSpawnAgentTool, createWaitAgentsTool } from "../subagent/agent-fleet.js";
42+
import {
43+
createFleetRecords,
44+
createSpawnAgentTool,
45+
createWaitAgentsTool,
46+
} from "../subagent/agent-fleet.js";
4347
import {
4448
createCloseAgentTool,
4549
createResumeAgentTool,

tests/unit/corbits-skills-catalog.test.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -148,9 +148,7 @@ test("only background libs carry disable-model-invocation", async () => {
148148
});
149149

150150
test("linear-issue-workflow references use_skill(git-worktrees)", async () => {
151-
const skill = await Bun.file(
152-
join(pluginRoot, "skills/linear-issue-workflow/SKILL.md"),
153-
).text();
151+
const skill = await Bun.file(join(pluginRoot, "skills/linear-issue-workflow/SKILL.md")).text();
154152
expect(skill).toContain('use_skill("git-worktrees")');
155153
expect(skill).not.toContain("git worktree add");
156154
});

0 commit comments

Comments
 (0)