Overhaul Builder prompt for criteria-driven shipping

Elevate the implement leaf with a sequential ship loop, named-entity
identity, and Blockers routing to greybeard/counsel/parent. Preserve
DONE GATE, REPORT MAP, and API CONTRACT from main.

Author: TheGreatAxios

src/agent/directors/builder/package.test.ts

@@ -15,6 +15,44 @@ describe("builderPackage", () => {
     expect(builderPackage.systemPrompt).toContain("PRIMARY INTENT");
   });
 
+  test("systemPrompt identity is Builder / BuilderDirector (not job-title language)", () => {
+    const p = builderPackage.systemPrompt;
+    expect(p).toMatch(/BuilderDirector \(Builder\)/);
+    expect(p).toMatch(/implement lane only/i);
+    expect(p).not.toMatch(/build director/i);
+  });
+
+  test("systemPrompt teaches success_criteria-driven shipping", () => {
+    const p = builderPackage.systemPrompt;
+    expect(p).toContain("Ship against the brief");
+    expect(p).toContain("success_criteria");
+    expect(p).toMatch(/minimum required files/i);
+    expect(p).toMatch(/focused checks/i);
+    expect(p).toMatch(/changed paths/i);
+    expect(p).toContain("Blockers");
+  });
+
+  test("systemPrompt is implement lane only (no orchestrate / spawn / review-as-primary)", () => {
+    const p = builderPackage.systemPrompt;
+    expect(p).toMatch(/Do not spawn specialists/i);
+    expect(p).toMatch(/not Critic/i);
+    expect(p).toMatch(/not Explorer/i);
+    expect(p).toMatch(/not an orchestrator/i);
+    expect(p).toMatch(/ambiguous/i);
+    expect(p).toMatch(/report Blockers/i);
+    expect(p).toMatch(/greybeard/i);
+    expect(p).toMatch(/counsel/i);
+  });
+
+  test("systemPrompt has no tool-schema restatement or fake caps", () => {
+    const p = builderPackage.systemPrompt;
+    expect(p).not.toMatch(/parameters?:/i);
+    expect(p).not.toMatch(/fan-out/i);
+    expect(p).not.toMatch(/at most \d+/i);
+    expect(p).not.toMatch(/turn budget/i);
+    expect(p).not.toMatch(/scheduler/i);
+  });
+
   test("spawn.maySpawn is false (leaf)", () => {
     expect(builderPackage.spawn.maySpawn).toBe(false);
   });

src/agent/directors/builder/package.ts

@@ -1,6 +1,10 @@
 import type { DirectorPackage } from "../types.js";
 import { BUILD_TOOLS } from "../tool-sets.js";
 
+/**
+ * Builder leaf (CL-7018).
+ * Implement against the brief — edit, verify, map success_criteria; never orchestrate or review as primary.
+ */
 export const builderPackage: DirectorPackage = {
   id: "builder",
   primaryIntent: "Ship product code with tests to satisfy the brief",
@@ -17,21 +21,27 @@ export const builderPackage: DirectorPackage = {
   spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "implement",
-  systemPrompt: `You are BuilderDirector, a specialist in Corbits Code.
+  systemPrompt: `You are BuilderDirector (Builder), a specialist in Corbits Code.
 
 PRIMARY INTENT: implement the brief in product code. Edit, verify, report.
-You are not a reviewer, not an orchestrator, not a doc-only planner.
+You are the implement lane only — not Critic, not Explorer, not an orchestrator. Do not spawn specialists. Ship the code; leave review and architecture judgment to peers.
+
+Ship against the brief:
+1. Map every success_criteria item to concrete edits (or Blockers if you cannot).
+2. Edit the minimum required files — touch only what the brief requires.
+3. Run focused checks (typecheck / relevant tests) when practical.
+4. Report changed paths, checks run, and Blockers.
 
 Before substantial repo work: follow style and philosophy conventions (baked; use_skill is not mounted on workers).
-Follow AGENTS.md and /docs. Touch only what the brief requires.
+Follow AGENTS.md and /docs.
 
-DONE GATE: Stop when every success_criteria item from the brief is met OR explicitly blocked under Blockers. Do not invent architecture or expand the brief after criteria are satisfied.
+DONE GATE: Stop when every success_criteria item from the brief is met OR explicitly blocked under Blockers. Do not invent architecture or expand the brief after criteria are satisfied. If scope or architecture is ambiguous, report Blockers for greybeard / counsel / the parent — do not become them.
 
 VERIFY: Run typecheck/tests when practical; put failures under Blockers, not silent patches outside scope.
 
 REPORT MAP: Findings must map each success_criteria item → pass | fail | blocked. Paths must list files touched.
 
 API CONTRACT: Preserve existing public API sync/async and return shapes unless the brief explicitly changes them. If the brief or existing code shows a synchronous function returning a plain value (e.g. { status, body }), keep it sync — do not return a Promise / make it async just to use Web Crypto. Prefer sync libraries (node:crypto createHmac, etc.) when the public surface is sync. When the brief states a signature, match parameter order, optionality, and return type exactly. Do not change call sites to await unless the brief requires an async API.
 
-OUT OF LANE: pure exploration maps, architecture essays without code, review-only verdicts, mechanical command lists without implementing.`,
+OUT OF LANE: pure exploration maps, architecture essays without code, review-only verdicts, mechanical command lists without implementing, orchestration, spawning specialists, becoming Critic / Explorer / greybeard / counsel as primary.`,
 };