Rename the implement director id to build

Author: TheGreatAxios

scripts/eval-capability.test.ts

@@ -62,9 +62,9 @@ describe("parseArgs", () => {
     expect(pair.model).toBe("bar");
   });
 
-  test("--director implement is parsed", () => {
-    const opts = parseArgs(["--provider", "foo", "--model", "bar", "--director", "implement"]);
-    expect(opts.director).toBe("implement");
+  test("--director build is parsed", () => {
+    const opts = parseArgs(["--provider", "foo", "--model", "bar", "--director", "build"]);
+    expect(opts.director).toBe("build");
   });
 
   test("omitted --director stays undefined", () => {

src/agent/directors/build/index.ts

@@ -0,0 +1 @@
+export { buildDirectorPackage } from "./package.js";

…gent/directors/implement/package.test.ts src/agent/directors/build/package.test.tssrc/agent/directors/implement/package.test.ts renamed to src/agent/directors/build/package.test.ts src/agent/directors/implement/package.test.ts renamed to src/agent/directors/build/package.test.ts

@@ -1,70 +1,70 @@
 import { describe, expect, test } from "bun:test";
-import { implementPackage } from "./package.js";
+import { buildDirectorPackage } from "./package.js";
 
-describe("implementPackage", () => {
+describe("buildDirectorPackage", () => {
   test("id matches directory / registry id", () => {
-    expect(implementPackage.id).toBe("implement");
+    expect(buildDirectorPackage.id).toBe("build");
   });
 
   test("systemPrompt is non-empty and not a Placeholder", () => {
-    expect(implementPackage.systemPrompt.length).toBeGreaterThan(0);
-    expect(implementPackage.systemPrompt.startsWith("Placeholder")).toBe(false);
+    expect(buildDirectorPackage.systemPrompt.length).toBeGreaterThan(0);
+    expect(buildDirectorPackage.systemPrompt.startsWith("Placeholder")).toBe(false);
   });
 
   test("systemPrompt mentions PRIMARY INTENT", () => {
-    expect(implementPackage.systemPrompt).toContain("PRIMARY INTENT");
+    expect(buildDirectorPackage.systemPrompt).toContain("PRIMARY INTENT");
   });
 
   test("spawn.maySpawn is false (leaf)", () => {
-    expect(implementPackage.spawn.maySpawn).toBe(false);
+    expect(buildDirectorPackage.spawn.maySpawn).toBe(false);
   });
 
   test("tools.allow includes product write tools", () => {
-    const allow = implementPackage.tools?.allow ?? [];
+    const allow = buildDirectorPackage.tools?.allow ?? [];
     expect(allow).toContain("write_file");
     expect(allow).toContain("edit_file");
     expect(allow).toContain("delete_file");
   });
 
   test("report.requiredSections includes Summary, Findings, Blockers, Paths", () => {
-    const sections = implementPackage.report.requiredSections;
+    const sections = buildDirectorPackage.report.requiredSections;
     expect(sections).toContain("Summary");
     expect(sections).toContain("Findings");
     expect(sections).toContain("Blockers");
     expect(sections).toContain("Paths");
   });
 
   test("modelRole is implement", () => {
-    expect(implementPackage.modelRole).toBe("implement");
+    expect(buildDirectorPackage.modelRole).toBe("implement");
   });
 
   test("optionalSkills order is style, philosophy, typescript", () => {
-    expect(implementPackage.optionalSkills).toEqual(["style", "philosophy", "typescript"]);
+    expect(buildDirectorPackage.optionalSkills).toEqual(["style", "philosophy", "typescript"]);
   });
 
   test("systemPrompt has DONE GATE for success_criteria", () => {
-    const prompt = implementPackage.systemPrompt;
+    const prompt = buildDirectorPackage.systemPrompt;
     expect(prompt).toContain("DONE GATE");
     expect(prompt).toContain("success_criteria");
     expect(prompt).toMatch(/[Ss]top when/);
   });
 
   test("systemPrompt has VERIFY language", () => {
-    const prompt = implementPackage.systemPrompt;
+    const prompt = buildDirectorPackage.systemPrompt;
     expect(prompt).toContain("VERIFY");
     expect(prompt).toMatch(/typecheck|tests/);
     expect(prompt).toContain("Blockers");
   });
 
   test("systemPrompt has REPORT MAP for criteria and Paths", () => {
-    const prompt = implementPackage.systemPrompt;
+    const prompt = buildDirectorPackage.systemPrompt;
     expect(prompt).toContain("REPORT MAP");
     expect(prompt).toMatch(/success_criteria.*pass|fail|blocked/s);
     expect(prompt).toMatch(/Paths must list files touched/);
   });
 
   test("systemPrompt has API CONTRACT for sync/async preservation", () => {
-    const prompt = implementPackage.systemPrompt;
+    const prompt = buildDirectorPackage.systemPrompt;
     expect(prompt).toContain("API CONTRACT");
     expect(prompt).toMatch(/sync/i);
     expect(prompt).toMatch(/Promise|async/);

src/agent/directors/implement/package.ts src/agent/directors/build/package.tssrc/agent/directors/implement/package.ts renamed to src/agent/directors/build/package.ts

@@ -1,8 +1,8 @@
 import type { DirectorPackage } from "../types.js";
-import { IMPLEMENT_TOOLS } from "../tool-sets.js";
+import { BUILD_TOOLS } from "../tool-sets.js";
 
-export const implementPackage: DirectorPackage = {
-  id: "implement",
+export const buildDirectorPackage: DirectorPackage = {
+  id: "build",
   primaryIntent: "Ship product code with tests to satisfy the brief",
   outOfLane: [
     "architecture gates",
@@ -13,7 +13,7 @@ export const implementPackage: DirectorPackage = {
   ],
   description: "Implementation leaf — edit, verify, report",
   optionalSkills: ["style", "philosophy", "typescript"],
-  tools: { allow: IMPLEMENT_TOOLS },
+  tools: { allow: BUILD_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 60 },
   report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },

src/agent/directors/identity.test.ts

@@ -8,12 +8,12 @@ import { DIRECTOR_REGISTRY } from "./registry.js";
 
 describe("formatDirectorSystemPrompt", () => {
   test("prefixes agent id, model role, and optional skills", () => {
-    const text = formatDirectorSystemPrompt(DIRECTOR_REGISTRY.implement);
-    expect(text.startsWith("Identity: agent id `implement`")).toBe(true);
-    expect(text).toContain('task(agent="implement")');
+    const text = formatDirectorSystemPrompt(DIRECTOR_REGISTRY.build);
+    expect(text.startsWith("Identity: agent id `build`")).toBe(true);
+    expect(text).toContain('task(agent="build")');
     expect(text).toContain("Model role: implement.");
     expect(text).toContain("style, philosophy, typescript");
-    expect(text).toContain(DIRECTOR_REGISTRY.implement.systemPrompt);
+    expect(text).toContain(DIRECTOR_REGISTRY.build.systemPrompt);
   });
 
   test("intern reports no optional skills by default", () => {
@@ -25,7 +25,7 @@ describe("formatDirectorSystemPrompt", () => {
 describe("defaultEffortForDirector", () => {
   test("intern is low; implement is medium; greybeard is high", () => {
     expect(defaultEffortForDirector(DIRECTOR_REGISTRY.intern)).toBe("low");
-    expect(defaultEffortForDirector(DIRECTOR_REGISTRY.implement)).toBe(
+    expect(defaultEffortForDirector(DIRECTOR_REGISTRY.build)).toBe(
       MODEL_ROLE_DEFAULT_EFFORT.implement,
     );
     expect(defaultEffortForDirector(DIRECTOR_REGISTRY.greybeard)).toBe("high");

src/agent/directors/implement/index.ts

@@ -50,7 +50,7 @@ describe("director registry", () => {
   test("intent map defaults (no general)", () => {
     expect(resolveDirector({ intent: "implement" })).toMatchObject({
       ok: true,
-      package: { id: "implement" },
+      package: { id: "build" },
     });
     expect(resolveDirector({ intent: "explore" })).toMatchObject({
       ok: true,
@@ -154,8 +154,8 @@ describe("director registry", () => {
     }
   });
 
-  test("implement mounts product writes; intern is shell-only; other leaves do not spawn", () => {
-    expect(DIRECTOR_REGISTRY.implement.tools?.allow).toEqual(
+  test("build mounts product writes; intern is shell-only; other leaves do not spawn", () => {
+    expect(DIRECTOR_REGISTRY.build.tools?.allow).toEqual(
       expect.arrayContaining(["write_file", "edit_file", "delete_file"]),
     );
     const internAllow = DIRECTOR_REGISTRY.intern.tools?.allow ?? [];

src/agent/directors/registry.test.ts

@@ -7,7 +7,7 @@ import { emilPackage } from "./emil/index.js";
 import { explorePackage } from "./explore/index.js";
 import { gaasbotPackage } from "./gaasbot/index.js";
 import { greybeardPackage } from "./greybeard/index.js";
-import { implementPackage } from "./implement/index.js";
+import { buildDirectorPackage } from "./build/index.js";
 import { internPackage } from "./intern/index.js";
 import { neckbeardPackage } from "./neckbeard/index.js";
 import { planPackage } from "./plan/index.js";
@@ -27,7 +27,7 @@ import {
 
 /** Intent → default director when `task(agent=…)` is omitted. No general director. */
 export const INTENT_DEFAULT_DIRECTOR: Readonly<Record<Exclude<TaskIntent, "general">, DirectorId>> = {
-  implement: "implement",
+  implement: "build",
   explore: "explore",
   plan: "plan",
   review: "critique",
@@ -39,7 +39,7 @@ export const INTENT_DEFAULT_DIRECTOR: Readonly<Record<Exclude<TaskIntent, "gener
  */
 export const DIRECTOR_REGISTRY: Readonly<Record<DirectorId, DirectorPackage>> = {
   skywalker: skywalkerPackage,
-  implement: implementPackage,
+  build: buildDirectorPackage,
   explore: explorePackage,
   plan: planPackage,
   intern: internPackage,

src/agent/directors/registry.ts

@@ -24,7 +24,7 @@ describe("skywalkerPackage", () => {
     expect(skywalkerPackage.spawn.maySpawn).toBe(true);
     expect(skywalkerPackage.spawn.allowlist).toHaveLength(15);
     expect(skywalkerPackage.spawn.allowlist).toEqual([
-      "implement",
+      "build",
       "explore",
       "plan",
       "intern",

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

@@ -23,7 +23,7 @@ Example chains:
 - feature: explore → implement → critique
 - "why / how / is this stalled": answer yourself; at most one explore if a single unknown blocks you
 
-Closed directors (use search_agents / registry; each id matches task(agent="<id>")): implement, explore, plan, intern, critique, greybeard, neckbeard, bruckheimer, gaasbot, draper, emil, brand-reviewer, shakespeare, testsmith, tester.
+Closed directors (use search_agents / registry; each id matches task(agent="<id>")): build, explore, plan, intern, critique, greybeard, neckbeard, bruckheimer, gaasbot, draper, emil, brand-reviewer, shakespeare, testsmith, tester.
 No catch-all worker. If unsure, reclassify — do not spawn a blob agent.
 
 Quick routing:
@@ -131,7 +131,7 @@ Do not reclassify COMMUNICATION as ORCHESTRATION just to justify parallel task s
 # Spawn graph
 
 Skywalker = full closed set. Greybeard = limited spawn only (intern/explore/critique) — not a second primary.
-You may spawn: implement, explore, plan, intern, critique, greybeard, neckbeard, bruckheimer, gaasbot, draper, emil, brand-reviewer, shakespeare, testsmith, tester.
+You may spawn: build, explore, plan, intern, critique, greybeard, neckbeard, bruckheimer, gaasbot, draper, emil, brand-reviewer, shakespeare, testsmith, tester.
 
 When spawning, prefer a typed brief:
 - intent — explore | implement | plan | review
@@ -174,7 +174,7 @@ export const skywalkerPackage: DirectorPackage = {
   spawn: {
     maySpawn: true,
     allowlist: [
-      "implement",
+      "build",
       "explore",
       "plan",
       "intern",