Skip to content

Commit 24d065c

Browse files
fix: update documentation to use load_skills instead of skills parameter (code-yeongyu#1088)
All documentation, agent prompts, and skill descriptions were still referencing the old 'skills' parameter name for delegate_task, but the tool implementation requires 'load_skills' (renamed in commit aa2b052). This caused confusion and errors for users following the docs. Fixes code-yeongyu#1008 Co-authored-by: sisyphus-dev-ai <sisyphus-dev-ai@users.noreply.github.com>
1 parent fd72ce5 commit 24d065c

8 files changed

Lines changed: 38 additions & 38 deletions

File tree

docs/category-skill-guide.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,12 @@ A Skill is a mechanism that injects **specialized knowledge (Context)** and **to
7070

7171
### Usage
7272

73-
Add desired skill names to the `skills` array.
73+
Add desired skill names to the `load_skills` array.
7474

7575
```typescript
7676
delegate_task(
7777
category="quick",
78-
skills=["git-master"],
78+
load_skills=["git-master"],
7979
prompt="Commit current changes. Follow commit message style."
8080
)
8181
```
@@ -110,17 +110,17 @@ You can create powerful specialized agents by combining Categories and Skills.
110110

111111
### 🎨 The Designer (UI Implementation)
112112
- **Category**: `visual-engineering`
113-
- **Skills**: `["frontend-ui-ux", "playwright"]`
113+
- **load_skills**: `["frontend-ui-ux", "playwright"]`
114114
- **Effect**: Implements aesthetic UI and verifies rendering results directly in browser.
115115

116116
### 🏗️ The Architect (Design Review)
117117
- **Category**: `ultrabrain`
118-
- **Skills**: `[]` (pure reasoning)
118+
- **load_skills**: `[]` (pure reasoning)
119119
- **Effect**: Leverages GPT-5.2's logical reasoning for in-depth system architecture analysis.
120120

121121
### ⚡ The Maintainer (Quick Fixes)
122122
- **Category**: `quick`
123-
- **Skills**: `["git-master"]`
123+
- **load_skills**: `["git-master"]`
124124
- **Effect**: Uses cost-effective models to quickly fix code and generate clean commits.
125125

126126
---
@@ -131,7 +131,7 @@ When delegating, **clear and specific** prompts are essential. Include these 7 e
131131

132132
1. **TASK**: What needs to be done? (single objective)
133133
2. **EXPECTED OUTCOME**: What is the deliverable?
134-
3. **REQUIRED SKILLS**: Which skills should be used?
134+
3. **REQUIRED SKILLS**: Which skills should be loaded via `load_skills`?
135135
4. **REQUIRED TOOLS**: Which tools must be used? (whitelist)
136136
5. **MUST DO**: What must be done (constraints)
137137
6. **MUST NOT DO**: What must never be done

docs/configurations.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ Available agents: `oracle`, `librarian`, `explore`, `multimodal-looker`
160160
Oh My OpenCode includes built-in skills that provide additional capabilities:
161161

162162
- **playwright**: Browser automation with Playwright MCP. Use for web scraping, testing, screenshots, and browser interactions.
163-
- **git-master**: Git expert for atomic commits, rebase/squash, and history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with `delegate_task(category='quick', skills=['git-master'], ...)` to save context.
163+
- **git-master**: Git expert for atomic commits, rebase/squash, and history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with `delegate_task(category='quick', load_skills=['git-master'], ...)` to save context.
164164

165165
Disable built-in skills via `disabled_skills` in `~/.config/opencode/oh-my-opencode.json` or `.opencode/oh-my-opencode.json`:
166166

docs/guide/understanding-orchestration-system.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -326,13 +326,13 @@ Skills prepend specialized instructions to subagent prompts:
326326
// Category + Skill combination
327327
delegate_task(
328328
category="visual-engineering",
329-
skills=["frontend-ui-ux"], // Adds UI/UX expertise
329+
load_skills=["frontend-ui-ux"], // Adds UI/UX expertise
330330
prompt="..."
331331
)
332332

333333
delegate_task(
334334
category="general",
335-
skills=["playwright"], // Adds browser automation expertise
335+
load_skills=["playwright"], // Adds browser automation expertise
336336
prompt="..."
337337
)
338338
```
@@ -341,8 +341,8 @@ delegate_task(
341341

342342
| Before | After |
343343
|--------|-------|
344-
| Hardcoded: `frontend-ui-ux-engineer` (Gemini 3 Pro) | `category="visual-engineering" + skills=["frontend-ui-ux"]` |
345-
| One-size-fits-all | `category="visual-engineering" + skills=["unity-master"]` |
344+
| Hardcoded: `frontend-ui-ux-engineer` (Gemini 3 Pro) | `category="visual-engineering" + load_skills=["frontend-ui-ux"]` |
345+
| One-size-fits-all | `category="visual-engineering" + load_skills=["unity-master"]` |
346346
| Model bias | Category-based: model abstraction eliminates bias |
347347

348348
---
@@ -365,7 +365,7 @@ sequenceDiagram
365365
366366
Note over Orchestrator: Prompt Structure:<br/>1. TASK (exact checkbox)<br/>2. EXPECTED OUTCOME<br/>3. REQUIRED SKILLS<br/>4. REQUIRED TOOLS<br/>5. MUST DO<br/>6. MUST NOT DO<br/>7. CONTEXT + Wisdom
367367
368-
Orchestrator->>Junior: delegate_task(category, skills, prompt)
368+
Orchestrator->>Junior: delegate_task(category, load_skills, prompt)
369369
370370
Junior->>Junior: Create todos, execute
371371
Junior->>Junior: Verify (lsp_diagnostics, tests)

sisyphus-prompt.md

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -239,7 +239,7 @@ Ask yourself:
239239
I will use delegate_task with:
240240
- **Category**: [selected-category-name]
241241
- **Why this category**: [how category description matches task domain]
242-
- **Skills**: [list of selected skills]
242+
- **load_skills**: [list of selected skills]
243243
- **Skill evaluation**:
244244
- [skill-1]: INCLUDED because [reason based on skill description]
245245
- [skill-2]: OMITTED because [reason why skill domain doesn't apply]
@@ -256,7 +256,7 @@ I will use delegate_task with:
256256
I will use delegate_task with:
257257
- **Category**: [category-name]
258258
- **Why this category**: Category description says "[quote description]" which matches this task's requirements
259-
- **Skills**: ["skill-a", "skill-b"]
259+
- **load_skills**: ["skill-a", "skill-b"]
260260
- **Skill evaluation**:
261261
- skill-a: INCLUDED - description says "[quote]" which applies to this task
262262
- skill-b: INCLUDED - description says "[quote]" which is needed here
@@ -265,7 +265,7 @@ I will use delegate_task with:
265265

266266
delegate_task(
267267
category="[category-name]",
268-
skills=["skill-a", "skill-b"],
268+
load_skills=["skill-a", "skill-b"],
269269
prompt="..."
270270
)
271271
```
@@ -276,12 +276,12 @@ delegate_task(
276276
I will use delegate_task with:
277277
- **Agent**: [agent-name]
278278
- **Reason**: This requires [agent's specialty] based on agent description
279-
- **Skills**: [] (agents have built-in expertise)
279+
- **load_skills**: [] (agents have built-in expertise)
280280
- **Expected Outcome**: [what agent should return]
281281

282282
delegate_task(
283283
subagent_type="[agent-name]",
284-
skills=[],
284+
load_skills=[],
285285
prompt="..."
286286
)
287287
```
@@ -292,21 +292,21 @@ delegate_task(
292292
I will use delegate_task with:
293293
- **Agent**: explore
294294
- **Reason**: Need to find all authentication implementations across the codebase - this is contextual grep
295-
- **Skills**: []
295+
- **load_skills**: []
296296
- **Expected Outcome**: List of files containing auth patterns
297297

298298
delegate_task(
299299
subagent_type="explore",
300300
run_in_background=true,
301-
skills=[],
301+
load_skills=[],
302302
prompt="Find all authentication implementations in the codebase"
303303
)
304304
```
305305
306306
**WRONG: No Skill Evaluation**
307307
308308
```
309-
delegate_task(category="...", skills=[], prompt="...") // Where's the justification?
309+
delegate_task(category="...", load_skills=[], prompt="...") // Where's the justification?
310310
```
311311
312312
**WRONG: Vague Category Selection**
@@ -329,11 +329,11 @@ I'll use this category because it seems right.
329329
```typescript
330330
// CORRECT: Always background, always parallel
331331
// Contextual Grep (internal)
332-
delegate_task(subagent_type="explore", run_in_background=true, skills=[], prompt="Find auth implementations in our codebase...")
333-
delegate_task(subagent_type="explore", run_in_background=true, skills=[], prompt="Find error handling patterns here...")
332+
delegate_task(subagent_type="explore", run_in_background=true, load_skills=[], prompt="Find auth implementations in our codebase...")
333+
delegate_task(subagent_type="explore", run_in_background=true, load_skills=[], prompt="Find error handling patterns here...")
334334
// Reference Grep (external)
335-
delegate_task(subagent_type="librarian", run_in_background=true, skills=[], prompt="Find JWT best practices in official docs...")
336-
delegate_task(subagent_type="librarian", run_in_background=true, skills=[], prompt="Find how production apps handle auth in Express...")
335+
delegate_task(subagent_type="librarian", run_in_background=true, load_skills=[], prompt="Find JWT best practices in official docs...")
336+
delegate_task(subagent_type="librarian", run_in_background=true, load_skills=[], prompt="Find how production apps handle auth in Express...")
337337
// Continue working immediately. Collect with background_output when needed.
338338
339339
// WRONG: Sequential or blocking
@@ -416,7 +416,7 @@ Skills inject specialized instructions into the subagent. Read the description t
416416
For EVERY skill listed above, ask yourself:
417417
> "Does this skill's expertise domain overlap with my task?"
418418
419-
- If YES → INCLUDE in `skills=[...]`
419+
- If YES → INCLUDE in `load_skills=[...]`
420420
- If NO → You MUST justify why (see below)
421421

422422
**STEP 3: Justify Omissions**
@@ -444,14 +444,14 @@ SKILL EVALUATION for "[skill-name]":
444444
```typescript
445445
delegate_task(
446446
category="[selected-category]",
447-
skills=["skill-1", "skill-2"], // Include ALL relevant skills
447+
load_skills=["skill-1", "skill-2"], // Include ALL relevant skills
448448
prompt="..."
449449
)
450450
```
451451

452452
**ANTI-PATTERN (will produce poor results):**
453453
```typescript
454-
delegate_task(category="...", skills=[], prompt="...") // Empty skills without justification
454+
delegate_task(category="...", load_skills=[], prompt="...") // Empty load_skills without justification
455455
```
456456
### Delegation Table:
457457

@@ -724,7 +724,7 @@ If the user's approach seems problematic:
724724
| **Error Handling** | Empty catch blocks `catch(e) {}` |
725725
| **Testing** | Deleting failing tests to "pass" |
726726
| **Search** | Firing agents for single-line typos or obvious syntax errors |
727-
| **Delegation** | Using `skills=[]` without justifying why no skills apply |
727+
| **Delegation** | Using `load_skills=[]` without justifying why no skills apply |
728728
| **Debugging** | Shotgun debugging, random changes |
729729
## Soft Guidelines
730730

src/agents/atlas.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ Categories spawn \`Sisyphus-Junior-{category}\` with optimized settings:
5858
${categoryRows.join("\n")}
5959
6060
\`\`\`typescript
61-
delegate_task(category="[category-name]", skills=[...], prompt="...")
61+
delegate_task(category="[category-name]", load_skills=[...], prompt="...")
6262
\`\`\``
6363
}
6464

@@ -84,12 +84,12 @@ ${skillRows.join("\n")}
8484
**MANDATORY: Evaluate ALL skills for relevance to your task.**
8585
8686
Read each skill's description and ask: "Does this skill's domain overlap with my task?"
87-
- If YES: INCLUDE in skills=[...]
87+
- If YES: INCLUDE in load_skills=[...]
8888
- If NO: You MUST justify why in your pre-delegation declaration
8989
9090
**Usage:**
9191
\`\`\`typescript
92-
delegate_task(category="[category]", skills=["skill-1", "skill-2"], prompt="...")
92+
delegate_task(category="[category]", load_skills=["skill-1", "skill-2"], prompt="...")
9393
\`\`\`
9494
9595
**IMPORTANT:**
@@ -102,7 +102,7 @@ function buildDecisionMatrix(agents: AvailableAgent[], userCategories?: Record<s
102102
const allCategories = { ...DEFAULT_CATEGORIES, ...userCategories }
103103

104104
const categoryRows = Object.entries(allCategories).map(([name]) =>
105-
`| ${getCategoryDescription(name, userCategories)} | \`category="${name}", skills=[...]\` |`
105+
`| ${getCategoryDescription(name, userCategories)} | \`category="${name}", load_skills=[...]\` |`
106106
)
107107

108108
const agentRows = agents.map((a) => {

src/agents/sisyphus.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -144,11 +144,11 @@ ${librarianSection}
144144
\`\`\`typescript
145145
// CORRECT: Always background, always parallel
146146
// Contextual Grep (internal)
147-
delegate_task(subagent_type="explore", run_in_background=true, skills=[], prompt="Find auth implementations in our codebase...")
148-
delegate_task(subagent_type="explore", run_in_background=true, skills=[], prompt="Find error handling patterns here...")
147+
delegate_task(subagent_type="explore", run_in_background=true, load_skills=[], prompt="Find auth implementations in our codebase...")
148+
delegate_task(subagent_type="explore", run_in_background=true, load_skills=[], prompt="Find error handling patterns here...")
149149
// Reference Grep (external)
150-
delegate_task(subagent_type="librarian", run_in_background=true, skills=[], prompt="Find JWT best practices in official docs...")
151-
delegate_task(subagent_type="librarian", run_in_background=true, skills=[], prompt="Find how production apps handle auth in Express...")
150+
delegate_task(subagent_type="librarian", run_in_background=true, load_skills=[], prompt="Find JWT best practices in official docs...")
151+
delegate_task(subagent_type="librarian", run_in_background=true, load_skills=[], prompt="Find how production apps handle auth in Express...")
152152
// Continue working immediately. Collect with background_output when needed.
153153
154154
// WRONG: Sequential or blocking

src/features/builtin-skills/git-master/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: git-master
3-
description: "MUST USE for ANY git operations. Atomic commits, rebase/squash, history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with delegate_task(category='quick', skills=['git-master'], ...) to save context. Triggers: 'commit', 'rebase', 'squash', 'who wrote', 'when was X added', 'find the commit that'."
3+
description: "MUST USE for ANY git operations. Atomic commits, rebase/squash, history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with delegate_task(category='quick', load_skills=['git-master'], ...) to save context. Triggers: 'commit', 'rebase', 'squash', 'who wrote', 'when was X added', 'find the commit that'."
44
---
55

66
# Git Master Agent

src/features/builtin-skills/skills.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ Interpret creatively and make unexpected choices that feel genuinely designed fo
9595
const gitMasterSkill: BuiltinSkill = {
9696
name: "git-master",
9797
description:
98-
"MUST USE for ANY git operations. Atomic commits, rebase/squash, history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with delegate_task(category='quick', skills=['git-master'], ...) to save context. Triggers: 'commit', 'rebase', 'squash', 'who wrote', 'when was X added', 'find the commit that'.",
98+
"MUST USE for ANY git operations. Atomic commits, rebase/squash, history search (blame, bisect, log -S). STRONGLY RECOMMENDED: Use with delegate_task(category='quick', load_skills=['git-master'], ...) to save context. Triggers: 'commit', 'rebase', 'squash', 'who wrote', 'when was X added', 'find the commit that'.",
9999
template: `# Git Master Agent
100100
101101
You are a Git expert combining three specializations:

0 commit comments

Comments
 (0)