Delete inert report.requiredSections from director packages

Every director declared report.requiredSections, but nothing read it
except its own definition sites and a test assertion checking it
matched a hardcoded list. It never drove hasReportEnvelope() (which
independently hardcodes the four headings in report.ts) and was never
rendered into any prompt — the real enforcement is
buildSubAgentReportContract(), spliced unconditionally into the base
prompt. CL-6946 will replace the whole four-heading envelope with a
schema-validated submit_result, so building this into a real mechanism
now would be thrown away; delete the dead config instead.

Added a test pinning that the one real mechanism stays internally
consistent: buildSubAgentReportContract()'s rendered headings satisfy
hasReportEnvelope().

Author: TheGreatAxios

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

@@ -33,14 +33,6 @@ describe("brandReviewerPackage", () => {
     expect(brandReviewerPackage.systemPrompt).not.toMatch(/authz/i);
   });
 
-  test("report.requiredSections covers the leaf envelope", () => {
-    const sections = brandReviewerPackage.report.requiredSections;
-    expect(sections).toContain("Summary");
-    expect(sections).toContain("Findings");
-    expect(sections).toContain("Blockers");
-    expect(sections).toContain("Paths");
-  });
-
   test("modelRole is docs", () => {
     expect(brandReviewerPackage.modelRole).toBe("docs");
   });

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

@@ -17,7 +17,6 @@ export const brandReviewerPackage: DirectorPackage = {
   tools: { allow: DOCS_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 40 },
-  report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },
   modelRole: "docs",
   systemPrompt: `You are BrandReviewerDirector, a specialist in Corbits Code.
 

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

@@ -29,12 +29,6 @@ describe("bruckheimerPackage", () => {
     expect(bruckheimerPackage.writePaths).toBeUndefined();
   });
 
-  test("report requires envelope sections", () => {
-    for (const section of ["Summary", "Findings", "Blockers", "Paths"]) {
-      expect(bruckheimerPackage.report.requiredSections).toContain(section);
-    }
-  });
-
   test("modelRole is docs", () => {
     expect(bruckheimerPackage.modelRole).toBe("docs");
   });

src/agent/directors/bruckheimer/package.ts

@@ -18,7 +18,6 @@ export const bruckheimerPackage: DirectorPackage = {
   tools: { allow: DOCS_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 40 },
-  report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },
   modelRole: "docs",
   systemPrompt: `You are BruckheimerDirector, a specialist in Corbits Code.
 

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

@@ -27,14 +27,6 @@ describe("buildDirectorPackage", () => {
     expect(allow).toContain("apply_patch");
   });
 
-  test("report.requiredSections includes Summary, Findings, Blockers, Paths", () => {
-    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(buildDirectorPackage.modelRole).toBe("implement");
   });

src/agent/directors/build/package.ts

@@ -16,7 +16,6 @@ export const buildDirectorPackage: DirectorPackage = {
   tools: { allow: BUILD_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 60 },
-  report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },
   modelRole: "implement",
   systemPrompt: `You are BuildDirector, a specialist in Corbits Code.
 

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

@@ -64,14 +64,6 @@ describe("critiquePackage", () => {
     expect(allow).not.toContain("delete_file");
   });
 
-  test("report.requiredSections covers the leaf envelope", () => {
-    const sections = critiquePackage.report.requiredSections;
-    expect(sections).toContain("Summary");
-    expect(sections).toContain("Findings");
-    expect(sections).toContain("Blockers");
-    expect(sections).toContain("Paths");
-  });
-
   test("modelRole is review", () => {
     expect(critiquePackage.modelRole).toBe("review");
   });

src/agent/directors/critique/package.ts

@@ -20,7 +20,6 @@ export const critiquePackage: DirectorPackage = {
   tools: { allow: REVIEW_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 45 },
-  report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },
   modelRole: "review",
   systemPrompt: `You are CritiqueDirector, a specialist in Corbits Code.
 

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

@@ -28,14 +28,6 @@ describe("draperPackage", () => {
     expect(allow).not.toContain("delete_file");
   });
 
-  test("report.requiredSections covers the leaf envelope", () => {
-    const sections = draperPackage.report.requiredSections;
-    expect(sections).toContain("Summary");
-    expect(sections).toContain("Findings");
-    expect(sections).toContain("Blockers");
-    expect(sections).toContain("Paths");
-  });
-
   test("modelRole is review", () => {
     expect(draperPackage.modelRole).toBe("review");
   });

src/agent/directors/draper/package.ts

@@ -19,7 +19,6 @@ export const draperPackage: DirectorPackage = {
   tools: { allow: REVIEW_TOOLS },
   spawn: { maySpawn: false },
   nudge: { maxTurns: 40 },
-  report: { requiredSections: ["Summary", "Findings", "Blockers", "Paths"] },
   modelRole: "review",
   systemPrompt: `You are DraperDirector, a specialist in Corbits Code.