Mount write tools on every closed director

src/agent/codex-tool-proxies.test.ts

@@ -597,8 +597,8 @@ describe("update_plan proxy", () => {
 });
 
 describe("allowDeleteFromCapabilities", () => {
-  test("docs allowlist (no delete_file) → false; build → true", () => {
-    expect(allowDeleteFromCapabilities({ mode: "allow", tools: DOCS_TOOLS })).toBe(false);
+  test("docs allowlist (includes delete_file) → true; build → true", () => {
+    expect(allowDeleteFromCapabilities({ mode: "allow", tools: DOCS_TOOLS })).toBe(true);
     expect(allowDeleteFromCapabilities({ mode: "allow", tools: BUILD_TOOLS })).toBe(true);
     expect(allowDeleteFromCapabilities(undefined)).toBe(true);
     expect(allowDeleteFromCapabilities({ mode: "exclude", tools: ["run_shell"] })).toBe(true);

src/agent/codex-tool-proxies.ts

@@ -55,8 +55,8 @@ export interface CreateCodexToolProxiesOpts {
   runManageTasks: CodexRunManageTasks;
   /**
    * When false, Delete File and Update+Move refuse without calling `delete_file`.
-   * Defaults to true (implement / unconstrained). Docs leaves pass false because
-   * DOCS_TOOLS includes apply_patch but not delete_file.
+   * Defaults to true (implement / unconstrained). Pass false when the
+   * director allowlist omits delete_file (docs leaves mount it today).
    */
   allowDelete?: boolean;
   /**

src/agent/directors/brand-reviewer/package.test.ts

@@ -73,6 +73,7 @@ describe("brandReviewerPackage", () => {
     const allow = brandReviewerPackage.tools?.allow ?? [];
     expect(allow).toContain("write_file");
     expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("systemPrompt mentions DESIGN.md", () => {

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

@@ -26,6 +26,7 @@ describe("bruckheimerPackage", () => {
     const allow = bruckheimerPackage.tools?.allow ?? [];
     expect(allow).toContain("write_file");
     expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is docs", () => {

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

@@ -84,14 +84,14 @@ describe("critiquePackage", () => {
     expect(critiquePackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is review surface without product writes", () => {
+  test("tools.allow is review surface with product writes", () => {
     const allow = critiquePackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
     expect(allow).toContain("read_file");
     expect(allow).not.toContain("use_skill");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is review", () => {

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

@@ -76,13 +76,13 @@ describe("draperPackage", () => {
     expect(draperPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is review surface without product writes", () => {
+  test("tools.allow is review surface with product writes", () => {
     const allow = draperPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
     expect(allow).not.toContain("use_skill");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is review", () => {

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

@@ -73,13 +73,13 @@ describe("emilPackage", () => {
     expect(emilPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is review surface without product writes", () => {
+  test("tools.allow is review surface with product writes", () => {
     const allow = emilPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
     expect(allow).not.toContain("use_skill");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is review", () => {

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

@@ -82,13 +82,13 @@ describe("explorePackage", () => {
     expect(explorePackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is read-only (no product writes)", () => {
+  test("tools.allow mounts product writes (lane: no product edits)", () => {
     const allow = explorePackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
     expect(allow).toContain("grep");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is explore", () => {

src/agent/directors/explore/package.ts

@@ -1,5 +1,5 @@
 import type { DirectorPackage } from "../types.js";
-import { READ_TOOLS } from "../tool-sets.js";
+import { REVIEW_TOOLS } from "../tool-sets.js";
 
 /**
  * Explorer leaf (CL-7020).
@@ -33,7 +33,7 @@ FINDINGS SHAPE: Findings must be a scannable map — key paths, symbols, call fl
 FINISH BIAS: Prefer one thorough pass then report. Expand Findings, change approach, or write the final report — do not keep re-reading the same paths.
 
 OUT OF LANE: product writes, drive-by fixes, shipping features, review severity theater, orchestration, spawning specialists, fleet discovery, becoming Builder/Critic/orchestrator as primary.`,
-  tools: { allow: READ_TOOLS },
+  tools: { allow: REVIEW_TOOLS },
   spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "explore",

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

@@ -67,12 +67,12 @@ describe("gaasbotPackage", () => {
     expect(gaasbotPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("denies product write tools (advice only)", () => {
+  test("mounts product write tools (lane discipline in prompts)", () => {
     const allow = gaasbotPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is plan", () => {

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

@@ -88,13 +88,13 @@ describe("greybeardPackage", () => {
     expect(allow).not.toContain("plan");
   });
 
-  test("tools.allow is orchestrator surface without product writes", () => {
+  test("tools.allow is orchestrator surface with product writes", () => {
     const allow = greybeardPackage.tools?.allow ?? [];
     expect(allow).toContain("task");
     expect(allow).toContain("search_agents");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is review", () => {

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

@@ -17,12 +17,15 @@ describe("internPackage", () => {
     expect(internPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is shell-first minimal surface", () => {
+  test("tools.allow is shell-first with path writes", () => {
     const allow = internPackage.tools?.allow ?? [];
     expect(allow).toContain("run_shell");
     expect(allow).toContain("read_file");
     expect(allow).toContain("list_dir");
-    for (const name of ["write_file", "edit_file", "delete_file", "grep", "search_files", "task"]) {
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
+    for (const name of ["grep", "search_files", "task", "apply_patch"]) {
       expect(allow).not.toContain(name);
     }
   });

src/agent/directors/intern/package.ts

@@ -3,7 +3,7 @@ import { INTERN_TOOLS } from "../tool-sets.js";
 
 /**
  * Mechanical intern leaf (CL-5822).
- * Shell/commands only — no judgment, no exploration, no product writes.
+ * Shell/commands first — no judgment, no exploration; path writes only when the brief requires them.
  */
 export const internPackage: DirectorPackage = {
   id: "intern",

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

@@ -25,12 +25,12 @@ describe("neckbeardPackage", () => {
     expect(neckbeardPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("denies product write tools", () => {
+  test("mounts product write tools", () => {
     const allow = neckbeardPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is review", () => {

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

@@ -66,12 +66,12 @@ describe("planPackage", () => {
     expect(planPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is review surface without product writes", () => {
+  test("tools.allow is review surface with product writes", () => {
     const allow = planPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is plan", () => {

src/agent/directors/registry.test.ts

@@ -97,7 +97,9 @@ describe("director registry", () => {
     expect(explore.description).toContain("agent id: explore");
     expect(explore.capabilities?.mode).toBe("allow");
     expect(explore.capabilities?.tools).toContain("read_file");
-    expect(explore.capabilities?.tools).not.toContain("write_file");
+    expect(explore.capabilities?.tools).toContain("write_file");
+    expect(explore.capabilities?.tools).toContain("edit_file");
+    expect(explore.capabilities?.tools).toContain("delete_file");
     expect(explore.orchestrator).toBe(false);
 
     const grey = packageToProfile(DIRECTOR_REGISTRY.greybeard);
@@ -123,7 +125,7 @@ describe("director registry", () => {
     expect(packageToProfile(g).orchestrator).toBe(true);
   });
 
-  test("review and design leaves deny product write tools", () => {
+  test("closed directors mount product write tools", () => {
     for (const id of [
       "critique",
       "greybeard",
@@ -135,22 +137,30 @@ describe("director registry", () => {
       "testsmith",
       "tester",
       "gaasbot",
+      "intern",
+      "build",
+      "shakespeare",
+      "bruckheimer",
+      "brand-reviewer",
+      "skywalker",
     ] as const) {
       const allow = DIRECTOR_REGISTRY[id].tools?.allow ?? [];
-      expect(allow).not.toContain("write_file");
-      expect(allow).not.toContain("edit_file");
-      expect(allow).not.toContain("delete_file");
+      expect(allow).toContain("write_file");
+      expect(allow).toContain("edit_file");
+      expect(allow).toContain("delete_file");
     }
   });
 
-  test("build mounts product writes; intern is shell-only; other leaves do not spawn", () => {
+  test("build mounts product writes + apply_patch; intern mounts writes without apply_patch; other leaves do not spawn", () => {
     expect(DIRECTOR_REGISTRY.build.tools?.allow).toEqual(
       expect.arrayContaining(["write_file", "edit_file", "delete_file", "apply_patch"]),
     );
     const internAllow = DIRECTOR_REGISTRY.intern.tools?.allow ?? [];
     expect(internAllow).toContain("run_shell");
-    expect(internAllow).not.toContain("write_file");
-    expect(internAllow).not.toContain("edit_file");
+    expect(internAllow).toContain("write_file");
+    expect(internAllow).toContain("edit_file");
+    expect(internAllow).toContain("delete_file");
+    expect(internAllow).not.toContain("apply_patch");
     for (const id of DIRECTOR_IDS) {
       if (id === "skywalker" || id === "greybeard") continue;
       expect(DIRECTOR_REGISTRY[id].spawn.maySpawn).toBe(false);

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

@@ -76,6 +76,7 @@ describe("shakespearePackage", () => {
     const allow = shakespearePackage.tools?.allow ?? [];
     expect(allow).toContain("write_file");
     expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is docs", () => {

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

@@ -60,13 +60,13 @@ describe("testerPackage", () => {
     expect(testerPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is read-only (no product writes)", () => {
+  test("tools.allow mounts product writes (lane: never fix)", () => {
     const allow = testerPackage.tools?.allow ?? [];
     expect(allow).toContain("run_shell");
     expect(allow).toContain("read_file");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is test", () => {

src/agent/directors/tester/package.ts

@@ -1,5 +1,5 @@
 import type { DirectorPackage } from "../types.js";
-import { READ_TOOLS } from "../tool-sets.js";
+import { REVIEW_TOOLS } from "../tool-sets.js";
 
 /**
  * Tester leaf (CL-7026).
@@ -34,7 +34,7 @@ DONE GATE: Stop when the brief's verify ask is answered with evidence OR explici
 REPORT MAP: Findings must map each requested check → pass | fail | blocked, with commands run and key failure excerpts. Paths list suites/files exercised.
 
 OUT OF LANE: fixing product code, "just quickly" fixing, redesigning the suite as Testsmith's primary job, fleet orchestration, architecture essays, exploration maps as primary.`,
-  tools: { allow: READ_TOOLS },
+  tools: { allow: REVIEW_TOOLS },
   spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "test",

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

@@ -21,12 +21,12 @@ describe("testsmithPackage", () => {
     expect(testsmithPackage.spawn.maySpawn).toBe(false);
   });
 
-  test("tools.allow is read-only (no product writes)", () => {
+  test("tools.allow mounts product writes (lane: design only)", () => {
     const allow = testsmithPackage.tools?.allow ?? [];
     expect(allow).toContain("read_file");
-    expect(allow).not.toContain("write_file");
-    expect(allow).not.toContain("edit_file");
-    expect(allow).not.toContain("delete_file");
+    expect(allow).toContain("write_file");
+    expect(allow).toContain("edit_file");
+    expect(allow).toContain("delete_file");
   });
 
   test("modelRole is test", () => {

src/agent/directors/testsmith/package.ts

@@ -1,5 +1,5 @@
 import type { DirectorPackage } from "../types.js";
-import { READ_TOOLS } from "../tool-sets.js";
+import { REVIEW_TOOLS } from "../tool-sets.js";
 
 /**
  * Testsmith: test design specialist — strategy and cases only; never implements product
@@ -29,8 +29,8 @@ Design in the report. Prefer:
 
 OUT OF LANE: fixing production code, becoming the implementer, running the full verify-and-fix loop, fleet orchestration.
 
-Read and search the codebase to ground the design; you have no product-mutation tools.`,
-  tools: { allow: READ_TOOLS },
+Read and search the codebase to ground the design; do not mutate product code.`,
+  tools: { allow: REVIEW_TOOLS },
   spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "test",