oh-my-codex 0.8.4 → 0.8.6
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +53 -0
- package/bin/omx.js +2 -2
- package/dist/agents/__tests__/definitions.test.js +5 -1
- package/dist/agents/__tests__/definitions.test.js.map +1 -1
- package/dist/agents/__tests__/native-config.test.js +7 -2
- package/dist/agents/__tests__/native-config.test.js.map +1 -1
- package/dist/agents/definitions.d.ts +5 -2
- package/dist/agents/definitions.d.ts.map +1 -1
- package/dist/agents/definitions.js +117 -30
- package/dist/agents/definitions.js.map +1 -1
- package/dist/agents/native-config.d.ts.map +1 -1
- package/dist/agents/native-config.js +72 -9
- package/dist/agents/native-config.js.map +1 -1
- package/dist/cli/__tests__/prompt-skill-sanitization.test.d.ts +2 -0
- package/dist/cli/__tests__/prompt-skill-sanitization.test.d.ts.map +1 -0
- package/dist/cli/__tests__/prompt-skill-sanitization.test.js +48 -0
- package/dist/cli/__tests__/prompt-skill-sanitization.test.js.map +1 -0
- package/dist/cli/__tests__/team.test.js +75 -1
- package/dist/cli/__tests__/team.test.js.map +1 -1
- package/dist/cli/team.d.ts.map +1 -1
- package/dist/cli/team.js +56 -0
- package/dist/cli/team.js.map +1 -1
- package/dist/config/__tests__/generator-notify.test.js +5 -1
- package/dist/config/__tests__/generator-notify.test.js.map +1 -1
- package/dist/config/generator.js +1 -1
- package/dist/config/generator.js.map +1 -1
- package/dist/hooks/__tests__/notify-fallback-watcher.test.js +43 -0
- package/dist/hooks/__tests__/notify-fallback-watcher.test.js.map +1 -1
- package/dist/hooks/__tests__/notify-hook-all-workers-idle.test.js +50 -0
- package/dist/hooks/__tests__/notify-hook-all-workers-idle.test.js.map +1 -1
- package/dist/hooks/__tests__/notify-hook-team-dispatch.test.js +40 -0
- package/dist/hooks/__tests__/notify-hook-team-dispatch.test.js.map +1 -1
- package/dist/hooks/__tests__/notify-hook-team-leader-nudge.test.js +59 -0
- package/dist/hooks/__tests__/notify-hook-team-leader-nudge.test.js.map +1 -1
- package/dist/hooks/__tests__/notify-hook-worker-idle.test.js +4 -0
- package/dist/hooks/__tests__/notify-hook-worker-idle.test.js.map +1 -1
- package/dist/hooks/__tests__/prompt-guidance-catalog.test.d.ts +2 -0
- package/dist/hooks/__tests__/prompt-guidance-catalog.test.d.ts.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-catalog.test.js +44 -0
- package/dist/hooks/__tests__/prompt-guidance-catalog.test.js.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-contract.test.d.ts +2 -0
- package/dist/hooks/__tests__/prompt-guidance-contract.test.d.ts.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-contract.test.js +56 -0
- package/dist/hooks/__tests__/prompt-guidance-contract.test.js.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-scenarios.test.d.ts +2 -0
- package/dist/hooks/__tests__/prompt-guidance-scenarios.test.d.ts.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-scenarios.test.js +30 -0
- package/dist/hooks/__tests__/prompt-guidance-scenarios.test.js.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-wave-two.test.d.ts +2 -0
- package/dist/hooks/__tests__/prompt-guidance-wave-two.test.d.ts.map +1 -0
- package/dist/hooks/__tests__/prompt-guidance-wave-two.test.js +56 -0
- package/dist/hooks/__tests__/prompt-guidance-wave-two.test.js.map +1 -0
- package/dist/hooks/__tests__/skill-guidance-contract.test.d.ts +2 -0
- package/dist/hooks/__tests__/skill-guidance-contract.test.d.ts.map +1 -0
- package/dist/hooks/__tests__/skill-guidance-contract.test.js +31 -0
- package/dist/hooks/__tests__/skill-guidance-contract.test.js.map +1 -0
- package/dist/mcp/__tests__/team-server-wait.test.d.ts +2 -0
- package/dist/mcp/__tests__/team-server-wait.test.d.ts.map +1 -0
- package/dist/mcp/__tests__/team-server-wait.test.js +155 -0
- package/dist/mcp/__tests__/team-server-wait.test.js.map +1 -0
- package/dist/mcp/team-server.d.ts.map +1 -1
- package/dist/mcp/team-server.js +41 -4
- package/dist/mcp/team-server.js.map +1 -1
- package/dist/notifications/__tests__/tmux.test.js +5 -5
- package/dist/notifications/__tests__/tmux.test.js.map +1 -1
- package/dist/notifications/tmux.js +1 -1
- package/dist/notifications/tmux.js.map +1 -1
- package/dist/team/__tests__/runtime-cli.test.js +6 -0
- package/dist/team/__tests__/runtime-cli.test.js.map +1 -1
- package/dist/team/__tests__/runtime.test.js +2 -1
- package/dist/team/__tests__/runtime.test.js.map +1 -1
- package/dist/team/api-interop.d.ts.map +1 -1
- package/dist/team/api-interop.js +4 -0
- package/dist/team/api-interop.js.map +1 -1
- package/dist/team/contracts.d.ts +1 -1
- package/dist/team/contracts.d.ts.map +1 -1
- package/dist/team/contracts.js +5 -0
- package/dist/team/contracts.js.map +1 -1
- package/dist/team/runtime.d.ts.map +1 -1
- package/dist/team/runtime.js +14 -0
- package/dist/team/runtime.js.map +1 -1
- package/dist/team/state/events.d.ts +18 -1
- package/dist/team/state/events.d.ts.map +1 -1
- package/dist/team/state/events.js +129 -1
- package/dist/team/state/events.js.map +1 -1
- package/dist/team/state/types.d.ts +9 -2
- package/dist/team/state/types.d.ts.map +1 -1
- package/dist/team/state/types.js.map +1 -1
- package/dist/team/state.d.ts +10 -1
- package/dist/team/state.d.ts.map +1 -1
- package/dist/team/state.js +3 -3
- package/dist/team/state.js.map +1 -1
- package/dist/verification/verifier.d.ts +1 -1
- package/dist/verification/verifier.js +1 -1
- package/package.json +1 -1
- package/prompts/analyst.md +15 -1
- package/prompts/api-reviewer.md +14 -0
- package/prompts/architect.md +18 -1
- package/prompts/build-fixer.md +14 -0
- package/prompts/code-reviewer.md +14 -0
- package/prompts/code-simplifier.md +13 -1
- package/prompts/critic.md +17 -1
- package/prompts/debugger.md +14 -0
- package/prompts/dependency-expert.md +16 -2
- package/prompts/designer.md +15 -1
- package/prompts/executor.md +22 -3
- package/prompts/explore.md +14 -0
- package/prompts/git-master.md +14 -0
- package/prompts/information-architect.md +14 -1
- package/prompts/performance-reviewer.md +14 -0
- package/prompts/planner.md +21 -3
- package/prompts/product-analyst.md +14 -1
- package/prompts/product-manager.md +18 -5
- package/prompts/qa-tester.md +15 -1
- package/prompts/quality-reviewer.md +14 -0
- package/prompts/quality-strategist.md +18 -5
- package/prompts/researcher.md +16 -2
- package/prompts/security-reviewer.md +14 -0
- package/prompts/style-reviewer.md +14 -0
- package/prompts/test-engineer.md +14 -0
- package/prompts/ux-researcher.md +14 -1
- package/prompts/verifier.md +15 -0
- package/prompts/vision.md +15 -1
- package/prompts/writer.md +15 -1
- package/scripts/notify-fallback-watcher.js +52 -2
- package/scripts/notify-hook/team-dispatch.js +33 -18
- package/scripts/notify-hook/team-leader-nudge.js +13 -3
- package/scripts/notify-hook/team-worker.js +11 -0
- package/skills/analyze/SKILL.md +15 -2
- package/skills/autopilot/SKILL.md +20 -7
- package/skills/build-fix/SKILL.md +20 -4
- package/skills/code-review/SKILL.md +20 -4
- package/skills/ecomode/SKILL.md +20 -20
- package/skills/frontend-ui-ux/SKILL.md +1 -1
- package/skills/git-master/SKILL.md +1 -1
- package/skills/help/SKILL.md +1 -1
- package/skills/plan/SKILL.md +15 -2
- package/skills/ralph/SKILL.md +26 -13
- package/skills/ralplan/SKILL.md +15 -0
- package/skills/security-review/SKILL.md +20 -4
- package/skills/team/SKILL.md +16 -1
- package/skills/ultraqa/SKILL.md +19 -3
- package/skills/ultrawork/SKILL.md +16 -16
- package/templates/AGENTS.md +8 -2
- package/dist/cli/__tests__/setup-codex-version.test.d.ts +0 -2
- package/dist/cli/__tests__/setup-codex-version.test.d.ts.map +0 -1
- package/dist/cli/__tests__/setup-codex-version.test.js +0 -99
- package/dist/cli/__tests__/setup-codex-version.test.js.map +0 -1
|
@@ -35,6 +35,10 @@ Most non-trivial software tasks require coordinated phases: understanding requir
|
|
|
35
35
|
- If a deep-interview spec exists, use it as high-clarity phase input instead of re-expanding from scratch
|
|
36
36
|
- If input is too vague for reliable expansion, offer/trigger `$deep-interview` first
|
|
37
37
|
- Do not enter expansion/planning/execution-heavy phases until pre-context grounding exists; if fast execution is forced, proceed only with explicit risk notes
|
|
38
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail
|
|
39
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints
|
|
40
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the workflow is grounded
|
|
41
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent
|
|
38
42
|
</Execution_Policy>
|
|
39
43
|
|
|
40
44
|
<Steps>
|
|
@@ -54,19 +58,19 @@ Most non-trivial software tasks require coordinated phases: understanding requir
|
|
|
54
58
|
1. **Phase 0 - Expansion**: Turn the user's idea into a detailed spec
|
|
55
59
|
- If `.omx/specs/deep-interview-*.md` exists for this task: reuse it and skip redundant expansion work
|
|
56
60
|
- If prompt is highly vague: route to `$deep-interview` for Socratic ambiguity-gated clarification
|
|
57
|
-
- Analyst (
|
|
58
|
-
- Architect (
|
|
61
|
+
- Analyst (THOROUGH tier): Extract requirements
|
|
62
|
+
- Architect (THOROUGH tier): Create technical specification
|
|
59
63
|
- Output: `.omx/plans/autopilot-spec.md`
|
|
60
64
|
|
|
61
65
|
2. **Phase 1 - Planning**: Create an implementation plan from the spec
|
|
62
|
-
- Architect (
|
|
63
|
-
- Critic (
|
|
66
|
+
- Architect (THOROUGH tier): Create plan (direct mode, no interview)
|
|
67
|
+
- Critic (THOROUGH tier): Validate plan
|
|
64
68
|
- Output: `.omx/plans/autopilot-impl.md`
|
|
65
69
|
|
|
66
70
|
3. **Phase 2 - Execution**: Implement the plan using Ralph + Ultrawork
|
|
67
|
-
-
|
|
68
|
-
-
|
|
69
|
-
-
|
|
71
|
+
- LOW-tier executor/search roles: Simple tasks
|
|
72
|
+
- STANDARD-tier executor roles: Standard tasks
|
|
73
|
+
- THOROUGH-tier executor/architect roles: Complex tasks
|
|
70
74
|
- Run independent tasks in parallel
|
|
71
75
|
|
|
72
76
|
4. **Phase 3 - QA**: Cycle until all tests pass (UltraQA mode)
|
|
@@ -113,6 +117,15 @@ Use `omx_state` MCP tools for autopilot lifecycle state.
|
|
|
113
117
|
- **On cancellation/cleanup**:
|
|
114
118
|
run `$cancel` (which should call `state_clear(mode="autopilot")`)
|
|
115
119
|
|
|
120
|
+
|
|
121
|
+
## Scenario Examples
|
|
122
|
+
|
|
123
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
124
|
+
|
|
125
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
126
|
+
|
|
127
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
128
|
+
|
|
116
129
|
<Examples>
|
|
117
130
|
<Good>
|
|
118
131
|
User: "autopilot A REST API for a bookstore inventory with CRUD operations using TypeScript"
|
|
@@ -17,7 +17,14 @@ This skill activates when:
|
|
|
17
17
|
|
|
18
18
|
## What It Does
|
|
19
19
|
|
|
20
|
-
|
|
20
|
+
## GPT-5.4 Guidance Alignment
|
|
21
|
+
|
|
22
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
23
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
24
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the build-fix workflow is grounded.
|
|
25
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
26
|
+
|
|
27
|
+
Delegates to the `build-fixer` agent (STANDARD tier) to:
|
|
21
28
|
|
|
22
29
|
1. **Collect Errors**
|
|
23
30
|
- Run the project's type check command (e.g., `tsc --noEmit`, `mypy`, `cargo check`, `go vet`)
|
|
@@ -45,9 +52,9 @@ Delegates to the `build-fixer` agent (Sonnet model) to:
|
|
|
45
52
|
## Agent Delegation
|
|
46
53
|
|
|
47
54
|
```
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
55
|
+
delegate(
|
|
56
|
+
role="build-fixer",
|
|
57
|
+
tier="STANDARD",
|
|
51
58
|
prompt="BUILD FIX TASK
|
|
52
59
|
|
|
53
60
|
Fix all build and TypeScript errors with minimal changes.
|
|
@@ -100,6 +107,15 @@ Verification: [type check command] (exit code 0)
|
|
|
100
107
|
- **Document why** - Comment non-obvious fixes
|
|
101
108
|
- **Test after** - Ensure tests still pass
|
|
102
109
|
|
|
110
|
+
|
|
111
|
+
## Scenario Examples
|
|
112
|
+
|
|
113
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
114
|
+
|
|
115
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
116
|
+
|
|
117
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
118
|
+
|
|
103
119
|
## Use with Other Skills
|
|
104
120
|
|
|
105
121
|
Combine with other skills for comprehensive fixing:
|
|
@@ -17,7 +17,14 @@ This skill activates when:
|
|
|
17
17
|
|
|
18
18
|
## What It Does
|
|
19
19
|
|
|
20
|
-
|
|
20
|
+
## GPT-5.4 Guidance Alignment
|
|
21
|
+
|
|
22
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
23
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
24
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the review is grounded.
|
|
25
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
26
|
+
|
|
27
|
+
Delegates to the `code-reviewer` agent (THOROUGH tier) for deep analysis:
|
|
21
28
|
|
|
22
29
|
1. **Identify Changes**
|
|
23
30
|
- Run `git diff` to find changed files
|
|
@@ -44,9 +51,9 @@ Delegates to the `code-reviewer` agent (Opus model) for deep analysis:
|
|
|
44
51
|
## Agent Delegation
|
|
45
52
|
|
|
46
53
|
```
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
54
|
+
delegate(
|
|
55
|
+
role="code-reviewer",
|
|
56
|
+
tier="THOROUGH",
|
|
50
57
|
prompt="CODE REVIEW TASK
|
|
51
58
|
|
|
52
59
|
Review code changes for quality, security, and maintainability.
|
|
@@ -179,6 +186,15 @@ The code-reviewer agent checks:
|
|
|
179
186
|
**REQUEST CHANGES** - CRITICAL or HIGH issues present
|
|
180
187
|
**COMMENT** - Only LOW/MEDIUM issues, no blocking concerns
|
|
181
188
|
|
|
189
|
+
|
|
190
|
+
## Scenario Examples
|
|
191
|
+
|
|
192
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
193
|
+
|
|
194
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
195
|
+
|
|
196
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
197
|
+
|
|
182
198
|
## Use with Other Skills
|
|
183
199
|
|
|
184
200
|
**With Team:**
|
package/skills/ecomode/SKILL.md
CHANGED
|
@@ -13,9 +13,9 @@ Overrides default model selection to prefer cheaper tiers:
|
|
|
13
13
|
|
|
14
14
|
| Default Tier | Ecomode Override |
|
|
15
15
|
|--------------|------------------|
|
|
16
|
-
|
|
|
17
|
-
|
|
|
18
|
-
| LOW
|
|
16
|
+
| THOROUGH | STANDARD, THOROUGH only if essential |
|
|
17
|
+
| STANDARD | LOW first, STANDARD if needed |
|
|
18
|
+
| LOW | LOW - no change |
|
|
19
19
|
|
|
20
20
|
## What Ecomode Does NOT Do
|
|
21
21
|
|
|
@@ -39,9 +39,9 @@ Ecomode is a modifier that combines with execution modes:
|
|
|
39
39
|
|
|
40
40
|
| Decision | Rule |
|
|
41
41
|
|----------|------|
|
|
42
|
-
| DEFAULT | Start with LOW tier
|
|
43
|
-
| UPGRADE | Escalate to
|
|
44
|
-
| AVOID |
|
|
42
|
+
| DEFAULT | Start with LOW tier for most tasks |
|
|
43
|
+
| UPGRADE | Escalate to STANDARD when LOW tier fails or task requires multi-file reasoning |
|
|
44
|
+
| AVOID | THOROUGH tier - only for planning/critique if essential |
|
|
45
45
|
|
|
46
46
|
## Agent Selection in Ecomode
|
|
47
47
|
|
|
@@ -55,16 +55,16 @@ This provides the complete agent tier matrix, MCP tool assignments, and selectio
|
|
|
55
55
|
|
|
56
56
|
```
|
|
57
57
|
// PREFERRED - Use for most tasks
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
58
|
+
delegate(role="executor", tier="LOW", task="...")
|
|
59
|
+
delegate(role="explore", tier="LOW", task="...")
|
|
60
|
+
delegate(role="architect", tier="LOW", task="...")
|
|
61
61
|
|
|
62
62
|
// FALLBACK - Only if LOW fails
|
|
63
|
-
|
|
64
|
-
|
|
63
|
+
delegate(role="executor", tier="STANDARD", task="...")
|
|
64
|
+
delegate(role="architect", tier="STANDARD", task="...")
|
|
65
65
|
|
|
66
66
|
// AVOID - Only for planning/critique if essential
|
|
67
|
-
|
|
67
|
+
delegate(role="planner", tier="THOROUGH", task="...")
|
|
68
68
|
```
|
|
69
69
|
|
|
70
70
|
## Delegation Enforcement
|
|
@@ -73,10 +73,10 @@ Ecomode maintains all delegation rules from core protocol with cost-optimized ro
|
|
|
73
73
|
|
|
74
74
|
| Action | Delegate To | Model |
|
|
75
75
|
|--------|-------------|-------|
|
|
76
|
-
| Code changes | executor
|
|
77
|
-
| Analysis | architect
|
|
78
|
-
| Search | explore |
|
|
79
|
-
| Documentation | writer |
|
|
76
|
+
| Code changes | executor | LOW / STANDARD |
|
|
77
|
+
| Analysis | architect | LOW |
|
|
78
|
+
| Search | explore | LOW |
|
|
79
|
+
| Documentation | writer | LOW |
|
|
80
80
|
|
|
81
81
|
### Background Execution
|
|
82
82
|
Long-running commands (install, build, test) run in background. Maximum 20 concurrent.
|
|
@@ -84,10 +84,10 @@ Long-running commands (install, build, test) run in background. Maximum 20 concu
|
|
|
84
84
|
## Token Savings Tips
|
|
85
85
|
|
|
86
86
|
1. **Batch similar tasks** to one agent instead of spawning many
|
|
87
|
-
2. **Use explore (
|
|
88
|
-
3. **Prefer executor
|
|
89
|
-
4. **Use writer (
|
|
90
|
-
5. **Avoid
|
|
87
|
+
2. **Use explore (LOW tier)** for file discovery, not architect
|
|
88
|
+
3. **Prefer LOW-tier executor routing** for simple changes - only upgrade if it fails
|
|
89
|
+
4. **Use writer (LOW tier)** for all documentation tasks
|
|
90
|
+
5. **Avoid THOROUGH-tier agents** unless the task genuinely requires deep reasoning
|
|
91
91
|
|
|
92
92
|
## Disabling Ecomode
|
|
93
93
|
|
|
@@ -22,7 +22,7 @@ If ToolSearch finds no MCP tools, use the Codex agent fallback below.
|
|
|
22
22
|
|
|
23
23
|
### Fallback: Codex Agent
|
|
24
24
|
```
|
|
25
|
-
|
|
25
|
+
delegate(role="designer", tier="STANDARD", task="{{ARGUMENTS}}")
|
|
26
26
|
```
|
|
27
27
|
|
|
28
28
|
## Capabilities
|
|
@@ -16,7 +16,7 @@ Routes to the git-master agent for git operations.
|
|
|
16
16
|
## Routing
|
|
17
17
|
|
|
18
18
|
```
|
|
19
|
-
|
|
19
|
+
delegate(role="git-master", tier="STANDARD", task="{{ARGUMENTS}}")
|
|
20
20
|
```
|
|
21
21
|
|
|
22
22
|
## Capabilities
|
package/skills/help/SKILL.md
CHANGED
package/skills/plan/SKILL.md
CHANGED
|
@@ -33,6 +33,10 @@ Jumping into code without understanding requirements leads to rework, scope cree
|
|
|
33
33
|
- Plans must meet quality standards: 80%+ claims cite file/line, 90%+ criteria are testable
|
|
34
34
|
- Consensus mode outputs the final plan by default; add `--interactive` to enable execution handoff
|
|
35
35
|
- Consensus mode uses RALPLAN-DR short mode by default; switch to deliberate mode with `--deliberate` or when the request explicitly signals high risk (auth/security, data migration, destructive/irreversible changes, production incident, compliance/PII, public API breakage)
|
|
36
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail
|
|
37
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints
|
|
38
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the plan is grounded
|
|
39
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent
|
|
36
40
|
</Execution_Policy>
|
|
37
41
|
|
|
38
42
|
<Steps>
|
|
@@ -53,7 +57,7 @@ Jumping into code without understanding requirements leads to rework, scope cree
|
|
|
53
57
|
2. **Ask one focused question** using `AskUserQuestion` for preferences, scope, and constraints
|
|
54
58
|
3. **Gather codebase facts first**: Before asking "what patterns does your code use?", spawn an `explore` agent to find out, then ask informed follow-up questions
|
|
55
59
|
4. **Build on answers**: Each question builds on the previous answer
|
|
56
|
-
5. **Consult Analyst** (
|
|
60
|
+
5. **Consult Analyst** (THOROUGH tier) for hidden requirements, edge cases, and risks
|
|
57
61
|
6. **Create plan** when the user signals readiness: "create the plan", "I'm ready", "make it a work plan"
|
|
58
62
|
|
|
59
63
|
### Direct Mode (detailed requests)
|
|
@@ -127,7 +131,7 @@ Plans are saved to `.omx/plans/`. Drafts go to `.omx/drafts/`.
|
|
|
127
131
|
- Before first MCP tool use, call `ToolSearch("mcp")` to discover deferred MCP tools
|
|
128
132
|
- Use `AskUserQuestion` for preference questions (scope, priority, timeline, risk tolerance) -- provides clickable UI
|
|
129
133
|
- Use plain text for questions needing specific values (port numbers, names, follow-up clarifications)
|
|
130
|
-
- Use `explore` agent (
|
|
134
|
+
- Use the `explore` agent (LOW tier, bounded quick pass) to gather codebase facts before asking the user
|
|
131
135
|
- Use `ask_codex` with `agent_role: "planner"` for planning validation on large-scope plans
|
|
132
136
|
- Use `ask_codex` with `agent_role: "analyst"` for requirements analysis
|
|
133
137
|
- Use `ask_codex` with `agent_role: "critic"` for plan review in consensus and review modes
|
|
@@ -138,6 +142,15 @@ Plans are saved to `.omx/plans/`. Drafts go to `.omx/drafts/`.
|
|
|
138
142
|
- In consensus mode with `--interactive`, on user approval **MUST** invoke `$ralph` for execution (step 9) -- never implement directly in the planning agent
|
|
139
143
|
</Tool_Usage>
|
|
140
144
|
|
|
145
|
+
|
|
146
|
+
## Scenario Examples
|
|
147
|
+
|
|
148
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
149
|
+
|
|
150
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
151
|
+
|
|
152
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
153
|
+
|
|
141
154
|
<Examples>
|
|
142
155
|
<Good>
|
|
143
156
|
Adaptive interview (gathering facts before asking):
|
package/skills/ralph/SKILL.md
CHANGED
|
@@ -35,6 +35,10 @@ Complex tasks often fail silently: partial implementations get declared "done",
|
|
|
35
35
|
- Always pass the `model` parameter explicitly when delegating to agents
|
|
36
36
|
- Read `docs/shared/agent-tiers.md` before first delegation to select correct agent tiers
|
|
37
37
|
- Deliver the full implementation: no scope reduction, no partial completion, no deleting tests to make them pass
|
|
38
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail
|
|
39
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints
|
|
40
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the execution loop is grounded
|
|
41
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent
|
|
38
42
|
</Execution_Policy>
|
|
39
43
|
|
|
40
44
|
<Steps>
|
|
@@ -53,9 +57,9 @@ Complex tasks often fail silently: partial implementations get declared "done",
|
|
|
53
57
|
1. **Review progress**: Check TODO list and any prior iteration state
|
|
54
58
|
2. **Continue from where you left off**: Pick up incomplete tasks
|
|
55
59
|
3. **Delegate in parallel**: Route tasks to specialist agents at appropriate tiers
|
|
56
|
-
- Simple lookups: LOW tier
|
|
57
|
-
- Standard work:
|
|
58
|
-
- Complex analysis:
|
|
60
|
+
- Simple lookups: LOW tier -- "What does this function return?"
|
|
61
|
+
- Standard work: STANDARD tier -- "Add error handling to this module"
|
|
62
|
+
- Complex analysis: THOROUGH tier -- "Debug this race condition"
|
|
59
63
|
4. **Run long operations in background**: Builds, installs, test suites use `run_in_background: true`
|
|
60
64
|
5. **Visual task gate (when screenshot/reference images are present)**:
|
|
61
65
|
- Run `$visual-verdict` **before every next edit**.
|
|
@@ -69,9 +73,9 @@ Complex tasks often fail silently: partial implementations get declared "done",
|
|
|
69
73
|
c. Read the output -- confirm it actually passed
|
|
70
74
|
d. Check: zero pending/in_progress TODO items
|
|
71
75
|
7. **Architect verification** (tiered):
|
|
72
|
-
- <5 files, <100 lines with full tests: STANDARD tier minimum (architect
|
|
73
|
-
- Standard changes: STANDARD tier (architect
|
|
74
|
-
- >20 files or security/architectural changes: THOROUGH tier (architect
|
|
76
|
+
- <5 files, <100 lines with full tests: STANDARD tier minimum (architect role)
|
|
77
|
+
- Standard changes: STANDARD tier (architect role)
|
|
78
|
+
- >20 files or security/architectural changes: THOROUGH tier (architect role)
|
|
75
79
|
- Ralph floor: always at least STANDARD, even for small changes
|
|
76
80
|
8. **On approval**: Run `/cancel` to cleanly exit and clean up all state files
|
|
77
81
|
9. **On rejection**: Fix the issues raised, then re-verify at the same tier
|
|
@@ -101,13 +105,22 @@ Use the `omx_state` MCP server tools (`state_write`, `state_read`, `state_clear`
|
|
|
101
105
|
- **On cancellation/cleanup**:
|
|
102
106
|
run `$cancel` (which should call `state_clear(mode="ralph")`)
|
|
103
107
|
|
|
108
|
+
|
|
109
|
+
## Scenario Examples
|
|
110
|
+
|
|
111
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
112
|
+
|
|
113
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
114
|
+
|
|
115
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
116
|
+
|
|
104
117
|
<Examples>
|
|
105
118
|
<Good>
|
|
106
119
|
Correct parallel delegation:
|
|
107
120
|
```
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
121
|
+
delegate(role="executor", tier="LOW", task="Add type export for UserConfig")
|
|
122
|
+
delegate(role="executor", tier="STANDARD", task="Implement the caching layer for API responses")
|
|
123
|
+
delegate(role="executor", tier="THOROUGH", task="Refactor auth module to support OAuth2 flow")
|
|
111
124
|
```
|
|
112
125
|
Why good: Three independent tasks fired simultaneously at appropriate tiers.
|
|
113
126
|
</Good>
|
|
@@ -118,7 +131,7 @@ Correct verification before completion:
|
|
|
118
131
|
1. Run: npm test → Output: "42 passed, 0 failed"
|
|
119
132
|
2. Run: npm run build → Output: "Build succeeded"
|
|
120
133
|
3. Run: lsp_diagnostics → Output: 0 errors
|
|
121
|
-
4.
|
|
134
|
+
4. Delegate to architect at STANDARD tier → Verdict: "APPROVED"
|
|
122
135
|
5. Run /cancel
|
|
123
136
|
```
|
|
124
137
|
Why good: Fresh evidence at each step, architect verification, then clean exit.
|
|
@@ -133,9 +146,9 @@ Why bad: Uses "should" and "look good" -- no fresh test/build output, no archite
|
|
|
133
146
|
<Bad>
|
|
134
147
|
Sequential execution of independent tasks:
|
|
135
148
|
```
|
|
136
|
-
|
|
137
|
-
|
|
138
|
-
|
|
149
|
+
delegate(executor, LOW, "Add type export") → wait →
|
|
150
|
+
delegate(executor, STANDARD, "Implement caching") → wait →
|
|
151
|
+
delegate(executor, THOROUGH, "Refactor auth")
|
|
139
152
|
```
|
|
140
153
|
Why bad: These are independent tasks that should run in parallel, not sequentially.
|
|
141
154
|
</Bad>
|
package/skills/ralplan/SKILL.md
CHANGED
|
@@ -26,6 +26,13 @@ $ralplan --interactive "task description"
|
|
|
26
26
|
|
|
27
27
|
## Behavior
|
|
28
28
|
|
|
29
|
+
## GPT-5.4 Guidance Alignment
|
|
30
|
+
|
|
31
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
32
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
33
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the consensus-planning flow is grounded.
|
|
34
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
35
|
+
|
|
29
36
|
This skill invokes the Plan skill in consensus mode:
|
|
30
37
|
|
|
31
38
|
```
|
|
@@ -147,3 +154,11 @@ The gate auto-passes when it detects **any** concrete signal. You do not need al
|
|
|
147
154
|
| Want to bypass the gate | Prefix with `force:` or `!` (e.g., `force: ralph fix it`) |
|
|
148
155
|
| Gate does not fire on a vague prompt | The gate only catches prompts with <=15 effective words and no concrete anchors; add more detail or use `$ralplan` explicitly |
|
|
149
156
|
| Redirected to ralplan but want to skip planning | In the ralplan workflow, say "just do it" or "skip planning" to transition directly to execution |
|
|
157
|
+
|
|
158
|
+
## Scenario Examples
|
|
159
|
+
|
|
160
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
161
|
+
|
|
162
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
163
|
+
|
|
164
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
@@ -19,7 +19,14 @@ This skill activates when:
|
|
|
19
19
|
|
|
20
20
|
## What It Does
|
|
21
21
|
|
|
22
|
-
|
|
22
|
+
## GPT-5.4 Guidance Alignment
|
|
23
|
+
|
|
24
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
25
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
26
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the security review is grounded.
|
|
27
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
28
|
+
|
|
29
|
+
Delegates to the `security-reviewer` agent (THOROUGH tier) for deep security analysis:
|
|
23
30
|
|
|
24
31
|
1. **OWASP Top 10 Scan**
|
|
25
32
|
- A01: Broken Access Control
|
|
@@ -61,9 +68,9 @@ Delegates to the `security-reviewer` agent (Opus model) for deep security analys
|
|
|
61
68
|
## Agent Delegation
|
|
62
69
|
|
|
63
70
|
```
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
71
|
+
delegate(
|
|
72
|
+
role="security-reviewer",
|
|
73
|
+
tier="THOROUGH",
|
|
67
74
|
prompt="SECURITY REVIEW TASK
|
|
68
75
|
|
|
69
76
|
Conduct comprehensive security audit of codebase.
|
|
@@ -254,6 +261,15 @@ The security-reviewer agent verifies:
|
|
|
254
261
|
4. **Fix MEDIUM** - Planned (within 1 month)
|
|
255
262
|
5. **Fix LOW** - Backlog (when convenient)
|
|
256
263
|
|
|
264
|
+
|
|
265
|
+
## Scenario Examples
|
|
266
|
+
|
|
267
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
268
|
+
|
|
269
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
270
|
+
|
|
271
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
272
|
+
|
|
257
273
|
## Use with Other Skills
|
|
258
274
|
|
|
259
275
|
**With Team:**
|
package/skills/team/SKILL.md
CHANGED
|
@@ -11,6 +11,13 @@ This skill is operationally sensitive. Treat it as an operator workflow, not a g
|
|
|
11
11
|
|
|
12
12
|
## What This Skill Must Do
|
|
13
13
|
|
|
14
|
+
## GPT-5.4 Guidance Alignment
|
|
15
|
+
|
|
16
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
17
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
18
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the team workflow is grounded.
|
|
19
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
20
|
+
|
|
14
21
|
When user triggers `$team`, the agent must:
|
|
15
22
|
|
|
16
23
|
1. Invoke OMX runtime directly with `omx team ...`
|
|
@@ -139,7 +146,7 @@ Model precedence (highest to lowest):
|
|
|
139
146
|
|
|
140
147
|
Thinking-level rule (critical):
|
|
141
148
|
- **No model-name heuristic mapping.**
|
|
142
|
-
- Team runtime must **not** infer `model_reasoning_effort` from model
|
|
149
|
+
- Team runtime must **not** infer `model_reasoning_effort` from model-name substrings (e.g., `spark`, `high-capability`, `mini`).
|
|
143
150
|
- Worker thinking level is applied **only when explicitly provided** (leader/user launch args or explicit config).
|
|
144
151
|
- If no explicit thinking value is provided, runtime leaves `model_reasoning_effort` unset.
|
|
145
152
|
|
|
@@ -450,3 +457,11 @@ Two cleanup tools exist and must not be confused:
|
|
|
450
457
|
- Worktree provisioning requires a git repository and can fail on branch/path collisions
|
|
451
458
|
- send-keys interactions can be timing-sensitive under load
|
|
452
459
|
- stale panes from prior runs can interfere until manually cleaned
|
|
460
|
+
|
|
461
|
+
## Scenario Examples
|
|
462
|
+
|
|
463
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
464
|
+
|
|
465
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
466
|
+
|
|
467
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
package/skills/ultraqa/SKILL.md
CHANGED
|
@@ -9,6 +9,13 @@ description: QA cycling workflow - test, verify, fix, repeat until goal met
|
|
|
9
9
|
|
|
10
10
|
## Overview
|
|
11
11
|
|
|
12
|
+
## GPT-5.4 Guidance Alignment
|
|
13
|
+
|
|
14
|
+
- Default to concise, evidence-dense progress and completion reporting unless the user or risk level requires more detail.
|
|
15
|
+
- Treat newer user task updates as local overrides for the active workflow branch while preserving earlier non-conflicting constraints.
|
|
16
|
+
- If correctness depends on additional inspection, retrieval, execution, or verification, keep using the relevant tools until the QA cycle is grounded.
|
|
17
|
+
- Continue through clear, low-risk, reversible next steps automatically; ask only when the next step is materially branching, destructive, or preference-dependent.
|
|
18
|
+
|
|
12
19
|
You are now in **ULTRAQA** mode - an autonomous QA cycling workflow that runs until your quality goal is met.
|
|
13
20
|
|
|
14
21
|
**Cycle**: qa-tester → architect verification → fix → repeat
|
|
@@ -39,7 +46,7 @@ If no structured goal provided, interpret the argument as a custom goal.
|
|
|
39
46
|
- `--custom`: Run appropriate command and check for pattern
|
|
40
47
|
- `--interactive`: Use qa-tester for interactive CLI/service testing:
|
|
41
48
|
```
|
|
42
|
-
|
|
49
|
+
delegate(role="qa-tester", tier="STANDARD", task="TEST:
|
|
43
50
|
Goal: [describe what to verify]
|
|
44
51
|
Service: [how to start]
|
|
45
52
|
Test cases: [specific scenarios to verify]")
|
|
@@ -51,7 +58,7 @@ If no structured goal provided, interpret the argument as a custom goal.
|
|
|
51
58
|
|
|
52
59
|
3. **ARCHITECT DIAGNOSIS**: Spawn architect to analyze failure
|
|
53
60
|
```
|
|
54
|
-
|
|
61
|
+
delegate(role="architect", tier="THOROUGH", task="DIAGNOSE FAILURE:
|
|
55
62
|
Goal: [goal type]
|
|
56
63
|
Output: [test/build output]
|
|
57
64
|
Provide root cause and specific fix recommendations.")
|
|
@@ -59,7 +66,7 @@ If no structured goal provided, interpret the argument as a custom goal.
|
|
|
59
66
|
|
|
60
67
|
4. **FIX ISSUES**: Apply architect's recommendations
|
|
61
68
|
```
|
|
62
|
-
|
|
69
|
+
delegate(role="executor", tier="STANDARD", task="FIX:
|
|
63
70
|
Issue: [architect diagnosis]
|
|
64
71
|
Files: [affected files]
|
|
65
72
|
Apply the fix precisely as recommended.")
|
|
@@ -105,6 +112,15 @@ Use `omx_state` MCP tools for UltraQA lifecycle state.
|
|
|
105
112
|
- **For resume detection**:
|
|
106
113
|
`state_read({mode: "ultraqa"})`
|
|
107
114
|
|
|
115
|
+
|
|
116
|
+
## Scenario Examples
|
|
117
|
+
|
|
118
|
+
**Good:** The user says `continue` after the workflow already has a clear next step. Continue the current branch of work instead of restarting or re-asking the same question.
|
|
119
|
+
|
|
120
|
+
**Good:** The user changes only the output shape or downstream delivery step (for example `make a PR`). Preserve earlier non-conflicting workflow constraints and apply the update locally.
|
|
121
|
+
|
|
122
|
+
**Bad:** The user says `continue`, and the workflow restarts discovery or stops before the missing verification/evidence is gathered.
|
|
123
|
+
|
|
108
124
|
## Cancellation
|
|
109
125
|
|
|
110
126
|
User can cancel with `/cancel` which clears the state file.
|
|
@@ -37,9 +37,9 @@ Sequential task execution wastes time when tasks are independent. Ultrawork enab
|
|
|
37
37
|
1. **Read agent reference**: Load `docs/shared/agent-tiers.md` for tier selection
|
|
38
38
|
2. **Classify tasks by independence**: Identify which tasks can run in parallel vs which have dependencies
|
|
39
39
|
3. **Route to correct tiers**:
|
|
40
|
-
- Simple lookups/definitions: LOW tier
|
|
41
|
-
- Standard implementation:
|
|
42
|
-
- Complex analysis/refactoring:
|
|
40
|
+
- Simple lookups/definitions: LOW tier
|
|
41
|
+
- Standard implementation: STANDARD tier
|
|
42
|
+
- Complex analysis/refactoring: THOROUGH tier
|
|
43
43
|
4. **Fire independent tasks simultaneously**: Launch all parallel-safe tasks at once
|
|
44
44
|
5. **Run dependent tasks sequentially**: Wait for prerequisites before launching dependent work
|
|
45
45
|
6. **Background long operations**: Builds, installs, and test suites use `run_in_background: true`
|
|
@@ -50,9 +50,9 @@ Sequential task execution wastes time when tasks are independent. Ultrawork enab
|
|
|
50
50
|
</Steps>
|
|
51
51
|
|
|
52
52
|
<Tool_Usage>
|
|
53
|
-
- Use
|
|
54
|
-
- Use
|
|
55
|
-
- Use
|
|
53
|
+
- Use LOW-tier delegation for simple changes
|
|
54
|
+
- Use STANDARD-tier delegation for standard work
|
|
55
|
+
- Use THOROUGH-tier delegation for complex work
|
|
56
56
|
- Use `run_in_background: true` for package installs, builds, and test suites
|
|
57
57
|
- Use foreground execution for quick status checks and file operations
|
|
58
58
|
</Tool_Usage>
|
|
@@ -74,9 +74,9 @@ Use `omx_state` MCP tools for ultrawork lifecycle state.
|
|
|
74
74
|
<Good>
|
|
75
75
|
Three independent tasks fired simultaneously:
|
|
76
76
|
```
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
77
|
+
delegate(role="executor", tier="LOW", task="Add missing type export for Config interface")
|
|
78
|
+
delegate(role="executor", tier="STANDARD", task="Implement the /api/users endpoint with validation")
|
|
79
|
+
delegate(role="test-engineer", tier="STANDARD", task="Add integration tests for the auth middleware")
|
|
80
80
|
```
|
|
81
81
|
Why good: Independent tasks at appropriate tiers, all fired at once.
|
|
82
82
|
</Good>
|
|
@@ -84,8 +84,8 @@ Why good: Independent tasks at appropriate tiers, all fired at once.
|
|
|
84
84
|
<Good>
|
|
85
85
|
Correct use of background execution:
|
|
86
86
|
```
|
|
87
|
-
|
|
88
|
-
|
|
87
|
+
delegate(role="executor", tier="STANDARD", task="npm install && npm run build", run_in_background=true)
|
|
88
|
+
delegate(role="writer", tier="LOW", task="Update the README with new API endpoints")
|
|
89
89
|
```
|
|
90
90
|
Why good: Long build runs in background while short task runs in foreground.
|
|
91
91
|
</Good>
|
|
@@ -93,9 +93,9 @@ Why good: Long build runs in background while short task runs in foreground.
|
|
|
93
93
|
<Bad>
|
|
94
94
|
Sequential execution of independent work:
|
|
95
95
|
```
|
|
96
|
-
result1 =
|
|
97
|
-
result2 =
|
|
98
|
-
result3 =
|
|
96
|
+
result1 = delegate(executor, LOW, "Add type export") # wait...
|
|
97
|
+
result2 = delegate(executor, STANDARD, "Implement endpoint") # wait...
|
|
98
|
+
result3 = delegate(test-engineer, STANDARD, "Add tests") # wait...
|
|
99
99
|
```
|
|
100
100
|
Why bad: These tasks are independent. Running them sequentially wastes time.
|
|
101
101
|
</Bad>
|
|
@@ -103,9 +103,9 @@ Why bad: These tasks are independent. Running them sequentially wastes time.
|
|
|
103
103
|
<Bad>
|
|
104
104
|
Wrong tier selection:
|
|
105
105
|
```
|
|
106
|
-
|
|
106
|
+
delegate(role="executor", tier="THOROUGH", task="Add a missing semicolon")
|
|
107
107
|
```
|
|
108
|
-
Why bad:
|
|
108
|
+
Why bad: THOROUGH tier is expensive overkill for a trivial fix. Use LOW-tier execution instead.
|
|
109
109
|
</Bad>
|
|
110
110
|
</Examples>
|
|
111
111
|
|