Skip to content

Commit 2b366f7

Browse files
docs(skills): use the ten-director fleet and native tool names in every skill
1 parent ffc2c04 commit 2b366f7

10 files changed

Lines changed: 81 additions & 90 deletions

File tree

‎plugins/corbits-skills/skills/implement/SKILL.md‎

Lines changed: 33 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
---
22
name: implement
3-
description: Disciplined per-commit workflow with Greybeard review, build gates, and Critique loops
3+
description: Disciplined per-commit workflow with Planner review, build gates, and Reviewer loops
44
---
55

66
# Implement
77

8-
A disciplined implementation workflow that produces reviewed, verified commits. Load this skill when you want each commit to go through architectural review, build verification, and code critique before it lands.
8+
A disciplined implementation workflow that produces reviewed, verified commits. Load this skill when you want each commit to go through architectural review, build verification, and code review before it lands.
99

1010
## Prerequisites
1111

@@ -19,43 +19,43 @@ The caller defines what work to do and where the commit boundaries are. This ski
1919

2020
## Tracking Progress
2121

22-
Use `TaskCreate`, `TaskUpdate`, and `TaskList` to track progress throughout the workflow. These tools give the user real-time visibility into what you're doing.
22+
Use `manage_tasks` to track progress throughout the workflow. It gives the user real-time visibility into what you're doing.
2323

2424
### Initial Planning
2525

26-
Before starting implementation, use `TaskCreate` for each commit-sized unit of work from the caller's instructions:
26+
Before starting implementation, use `manage_tasks` to add each commit-sized unit of work from the caller's instructions:
2727

28-
- **subject**: Clear imperative description of the unit of work
28+
- **title**: Clear imperative description of the unit of work
2929
- **description**: Enough context that you could pick it up cold
30-
- **activeForm**: Present continuous form for the spinner (e.g., "Refactoring HTTP client retry logic")
30+
- **active form**: Present continuous form for the status line (e.g., "Refactoring HTTP client retry logic")
3131

3232
### During the Per-Commit Workflow
3333

34-
When you begin a unit of work, mark its task `in_progress` with `TaskUpdate`. As you move through the workflow steps, update the task's `activeForm` to reflect which step you're in:
34+
When you begin a unit of work, mark its task `in_progress` with `manage_tasks`. As you move through the workflow steps, update the task's active form to reflect which step you're in:
3535

36-
- **Step 1**: "Reviewing approach with Greybeard: {subject}"
36+
- **Step 1**: "Reviewing approach with Planner: {subject}"
3737
- **Step 2**: "Implementing: {subject}"
3838
- **Step 3**: "Running build gate: {subject}"
3939
- **Step 4**: "Committing: {subject}"
40-
- **Step 5**: "Running Critique loop: {subject}"
40+
- **Step 5**: "Running Reviewer loop: {subject}"
4141

42-
When the commit lands and Critique is clean, mark the task `completed`.
42+
When the commit lands and Reviewer is clean, mark the task `completed`.
4343

4444
### Discovered Work
4545

46-
If new work surfaces during implementation (Greybeard suggests a preparatory refactor, Critique reveals a missing edge case that warrants its own commit), create a new task with `TaskCreate` and work it through the full per-commit workflow.
46+
If new work surfaces during implementation (Planner suggests a preparatory refactor, Reviewer reveals a missing edge case that warrants its own commit), add a new task with `manage_tasks` and work it through the full per-commit workflow.
4747

4848
## Workflow Per Commit
4949

5050
For each logical unit of work that results in a commit, follow these steps in order. Do not skip steps.
5151

52-
### Step 1: Greybeard Review
52+
### Step 1: Planner Review
5353

54-
Mark the task `in_progress` and set `activeForm` to "Reviewing approach with Greybeard: {subject}".
54+
Mark the task `in_progress` and set the active form to "Reviewing approach with Planner: {subject}".
5555

56-
Before writing any code, describe your implementation approach to Greybeard and ask for feedback.
56+
Before writing any code, describe your implementation approach to Planner and ask for feedback.
5757

58-
**What to send Greybeard:**
58+
**What to send Planner:**
5959

6060
- What you're about to change and why
6161
- Which files you expect to touch
@@ -64,16 +64,16 @@ Before writing any code, describe your implementation approach to Greybeard and
6464

6565
**How to handle feedback:**
6666

67-
- If Greybeard identifies problems with your approach, adjust before proceeding
68-
- If Greybeard suggests a fundamentally different approach, consider it seriously
67+
- If Planner identifies problems with your approach, adjust before proceeding
68+
- If Planner suggests a fundamentally different approach, consider it seriously
6969
- You don't need to agree with every suggestion, but you need a reason to disagree
7070
- Once you're aligned on approach, move to Step 2
7171

72-
Use the `@greybeard` subagent for this step.
72+
Use `spawn_agent(agent="planner")` for this step.
7373

7474
### Step 2: Implement and Test
7575

76-
Update `activeForm` to "Implementing: {subject}".
76+
Update the active form to "Implementing: {subject}".
7777

7878
The order of operations depends on whether you're fixing a bug or building a feature. In both cases, follow the repository's existing test conventions — look at how existing tests are structured, where they live, what framework they use, and match that style. If the repository has no existing tests, ask the caller what test framework and conventions to use before proceeding.
7979

@@ -98,7 +98,7 @@ Keep the scope tight to what was discussed. If you discover additional work is n
9898

9999
### Step 3: Build Gate
100100

101-
Update `activeForm` to "Running build gate: {subject}".
101+
Update the active form to "Running build gate: {subject}".
102102

103103
Run `make` (or the project's equivalent full pipeline: format, lint, build, test).
104104

@@ -111,19 +111,19 @@ Run `make` (or the project's equivalent full pipeline: format, lint, build, test
111111

112112
### Step 4: Commit
113113

114-
Update `activeForm` to "Committing: {subject}".
114+
Update the active form to "Committing: {subject}".
115115

116116
Create the commit. Follow the commit message conventions from the `style` skill. Include the test in the same unit of work as the implementation — same commit when committing — one logical unit — and update the docs when the commit changes documented behavior. Worker-chain branch/PR convention: branch name carries the issue id, the PR body ends with `Fixes CL-…` and carries no AI-attribution lines (CONTRIBUTING: title is a Conventional Commits subject, body is Summary/Verification).
117117

118-
### Step 5: Critique Loop
118+
### Step 5: Reviewer Loop
119119

120-
Update `activeForm` to "Running Critique loop: {subject}".
120+
Update the active form to "Running Reviewer loop: {subject}".
121121

122-
Ask Critique to review the committed change.
122+
Ask Reviewer to review the committed change.
123123

124124
**How to run:**
125125

126-
1. Spawn the `@critique` subagent and ask it to review the output of `git show HEAD`. Include the intent from Step 1 (what the change is meant to accomplish and the approach agreed with Greybeard) so Critique can evaluate whether the implementation matches the plan, not just surface-level quality. Tell Critique to limit its findings to the scope of the current commit -- pre-existing issues in touched files are out of scope.
126+
1. Use `spawn_agent(agent="reviewer")` to ask it to review the output of `git show HEAD`. Include the intent from Step 1 (what the change is meant to accomplish and the approach agreed with Planner) so Reviewer can evaluate whether the implementation matches the plan, not just surface-level quality. Tell Reviewer to limit its findings to the scope of the current commit -- pre-existing issues in touched files are out of scope.
127127
2. Read its findings
128128
3. For each issue marked VERIFIED or HIGH confidence: fix it
129129
4. Re-run the build gate (Step 3) to verify fixes
@@ -140,30 +140,30 @@ Ask Critique to review the committed change.
140140

141141
If the situation calls for more elaborate history surgery, search your available skills for one whose description covers git rebase or branch-history cleanup, and load it. Re-run the build gate after the rebase completes.
142142

143-
6. Ask Critique to review `git show HEAD` again. Re-include the original intent from Step 1 and tell it what you fixed since the last pass so it can focus on verifying the fixes and checking for new issues rather than re-reviewing the entire change from scratch.
144-
7. Repeat until Critique comes back clean or all remaining findings are acknowledged and intentional
143+
6. Ask Reviewer to review `git show HEAD` again. Re-include the original intent from Step 1 and tell it what you fixed since the last pass so it can focus on verifying the fixes and checking for new issues rather than re-reviewing the entire change from scratch.
144+
7. Repeat until Reviewer comes back clean or all remaining findings are acknowledged and intentional
145145

146146
**When to stop looping:**
147147

148-
- Critique reports no issues
148+
- Reviewer reports no issues
149149
- Remaining findings are judgment calls you've consciously decided against, not oversights
150150
- The build passes after the last round of fixes
151151

152152
### Step 6: Next
153153

154-
Mark the current task `completed` with `TaskUpdate`. Move to the next unit of work and return to Step 1.
154+
Mark the current task `completed` with `manage_tasks`. Move to the next unit of work and return to Step 1.
155155

156156
## Guidelines
157157

158-
**Don't shortcut the loop.** The value is in the discipline. Skipping Greybeard "because this change is simple" or skipping Critique "because the build passes" defeats the purpose.
158+
**Don't shortcut the loop.** The value is in the discipline. Skipping Planner "because this change is simple" or skipping Reviewer "because the build passes" defeats the purpose.
159159

160-
**Keep commits focused, but do not drop findings.** When Critique surfaces something outside the current commit's scope, every finding must be assigned one of four dispositions: (a) fix in the current commit, (b) commit it separately on this branch, (c) file a new issue with concrete acceptance criteria, or (d) accept it as-is. "Out of scope" is not a disposition. "Note it for later" is not a disposition unless you also say which of (a)–(d) "later" means.
160+
**Keep commits focused, but do not drop findings.** When Reviewer surfaces something outside the current commit's scope, every finding must be assigned one of four dispositions: (a) fix in the current commit, (b) commit it separately on this branch, (c) file a new issue with concrete acceptance criteria, or (d) accept it as-is. "Out of scope" is not a disposition. "Note it for later" is not a disposition unless you also say which of (a)–(d) "later" means.
161161

162-
**Disposition (d) always requires operator approval** — neither you nor greybeard can drop a finding on your own. For (c), the issue must be filed in this session, with its ID or URL in the status update; a promise to file it later is dropping the work. If you are orchestrated by karen, route the decision through karen's section 9 procedure (consult greybeard, paste his recommendation verbatim, escalate to the operator for any "accept as-is" or any unclear answer). If you are running directly, consult the operator before choosing (c) or (d).
162+
**Disposition (d) always requires operator approval** — neither you nor planner can drop a finding on your own. For (c), the issue must be filed in this session, with its ID or URL in the status update; a promise to file it later is dropping the work. If you are orchestrated by karen, route the decision through karen's section 9 procedure (consult planner, paste his recommendation verbatim, escalate to the operator for any "accept as-is" or any unclear answer). If you are running directly, consult the operator before choosing (c) or (d).
163163

164164
**Build must pass before every commit, amend, and rebase stop.** Never commit code that doesn't compile or pass tests. Fix build failures first, then commit, amend, or continue the rebase.
165165

166-
**Greybeard is for approach, Critique is for execution.** Greybeard reviews your plan before you write code. Critique reviews your code after you write it. Don't conflate the two.
166+
**Planner is for approach, Reviewer is for execution.** Planner reviews your plan before you write code. Reviewer reviews your code after you write it. Don't conflate the two.
167167

168168
## Acknowledgment
169169

‎plugins/corbits-skills/skills/interview/SKILL.md‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
11
---
22
name: interview
33
argument-hint: "<topic>[; <context>]"
4-
description: Conduct an iterative multiple-choice interview using AskUserQuestion. Returns the Q&A inline. Use as a utility when a caller needs structured user input on a topic.
5-
tools:
6-
- AskUserQuestion
4+
description: Conduct an iterative multiple-choice interview using `ask_operator`. Returns the Q&A inline. Use as a utility when a caller needs structured user input on a topic.
75
---
86

97
# Interview
108

11-
Use this skill to gather user input on a topic by asking multiple-choice questions in batches via `AskUserQuestion`. Return the questions and answers in the conversation. The caller decides what to do with them.
9+
Use this skill to gather user input on a topic by asking multiple-choice questions in batches via `ask_operator`. Return the questions and answers in the conversation. The caller decides what to do with them.
1210

1311
This is a utility, not a planner. It does not decide what to build, write any files, or invoke other skills.
1412

@@ -31,7 +29,7 @@ Probe objective and priorities before details. They shape every later question,
3129

3230
### Ask in batches
3331

34-
Each round uses `AskUserQuestion`. Refer to the tool's own documentation for parameter limits and multi-select behavior.
32+
Each round uses `ask_operator`. Refer to the tool's own documentation for parameter limits and multi-select behavior.
3533

3634
**Quality bar for options:**
3735

@@ -95,7 +93,7 @@ After emitting the findings, stop. Do not load other skills, invoke other agents
9593
**Round 1** (3 questions, bundled because none depends on the others):
9694

9795
```
98-
AskUserQuestion([
96+
ask_operator([
9997
{ header: "Goal", question: "What is the primary goal of the notification system?",
10098
options: [
10199
{ label: "Alert on critical events", description: "Errors, security issues, SLA breaches" },

‎plugins/corbits-skills/skills/lexicon/SKILL.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ Linear issues for drift.
1414
## What this is not
1515

1616
- Not a director. There is no `lexicon` director package, no
17-
`src/agent/directors/lexicon/` module, and no Skywalker
17+
`src/agent/directors/lexicon/` module, and no dispatch
1818
classification route. Never call `spawn_agent(agent="lexicon")` —
1919
this skill runs as a `/lexicon` slash playbook on the primary only.
2020
- Not a fleet router. It does not assign identity or dispatch workers.

0 commit comments

Comments
 (0)