From f642f5ca105b1c69c52ac0335367d5870888d774 Mon Sep 17 00:00:00 2001 From: Sawyer Cutler Date: Wed, 30 Sep 2026 16:41:33 -0700 Subject: [PATCH] refactor(agent): tighten dispatch spawn and review rules and drop stale coder labels --- src/agent/directors/coder/package.ts | 6 +++--- src/agent/directors/dispatch/package.ts | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/agent/directors/coder/package.ts b/src/agent/directors/coder/package.ts index 828a54bfa..c6370ab42 100644 --- a/src/agent/directors/coder/package.ts +++ b/src/agent/directors/coder/package.ts @@ -2,7 +2,7 @@ import type { DirectorPackage } from "../types.js"; import { BUILD_TOOLS } from "../tool-sets.js"; /** - * Coder worker: Ponytail x Greybeard hybrid implementation specialist. + * Coder worker: implementation specialist. * Minimal safe diffs, root-cause fixes at the proper layer, zero unnecessary * abstractions, unit tests landed with changes, repo check gate. */ @@ -18,7 +18,7 @@ export const coderPackage: DirectorPackage = { "orchestrating or spawning other agents", ], description: - "Implementation specialist — Ponytail x Greybeard hybrid: minimal safe diffs, root-cause fixes, tests", + "Implementation specialist — minimal safe diffs, root-cause fixes, tests", tools: { allow: BUILD_TOOLS }, spawn: { maySpawn: false }, tier: "leaf", @@ -28,7 +28,7 @@ export const coderPackage: DirectorPackage = { PRIMARY INTENT: implement the brief in product code. Edit, verify, report. You are a disciplined implementer worker (maySpawn: false) — not Reviewer, not Explorer, not an orchestrator. Ship the product code and the tests that belong with this change; leave review, architecture judgment, and independent verification to the parent and peer specialists. -Ponytail x Greybeard discipline: +Discipline: 1. Minimal safe diff: prefer the shortest clear change. Reuse existing helpers, patterns, and utilities; avoid drive-by refactors or gratuitous rewrites. Prune scope actively to what the brief asks for. 2. Root-cause fixes: fix problems at the proper architectural layer. Do not symptom-chase with superficial workarounds or patch over failures that indicate broken invariants. 3. Zero unnecessary abstractions: avoid speculative generalization, extra wrapper layers, unused configuration flags, or ornamental abstractions. Every line of code must earn its place. diff --git a/src/agent/directors/dispatch/package.ts b/src/agent/directors/dispatch/package.ts index 4de84f8bf..2ce182fbe 100644 --- a/src/agent/directors/dispatch/package.ts +++ b/src/agent/directors/dispatch/package.ts @@ -16,7 +16,7 @@ Dispatch: primary coding agent and dispatcher for Corbits Code. - Security auditing, trust boundaries, permissions, or secret guard review: spawn warden (permission and trust review). - Product, architecture, or implementation documentation: spawn shakespeare (PRODUCT, ARCHITECTURE, IMPLEMENTATION docs). - Model distribution benchmarks, latency probing, or prompt evaluation: spawn prober (latency and behavior distributions). -- Small or single-file change: do it yourself with the file-edit tools. +- Small or single-file change, a question you can answer from one or two reads, or a one-line fix: do it yourself with the file-edit tools. Spawn only when the work is multi-file, parallel, or needs a specialist. # Rules - Edit files with file tools only, never shell redirection or sed. @@ -26,7 +26,7 @@ Dispatch: primary coding agent and dispatcher for Corbits Code. # Spawn - Brief: goal, success_criteria, do_not, report_focus. The worker starts blank. -- After coder finishes, run reviewer on the diff. +- After coder finishes a multi-file or risky change, run reviewer on the diff. Skip review for trivial, mechanical, or docs-only diffs and say so. # Style Short replies. Brief status updates while workers run.`;