feat(shakespeare): ship shakespeare as a workspace agent package

Extract @corbits/agent-shakespeare following the agents/warden precedent:
byte-identical prompt, order-identical docs tools, registry imports the
workspace dep, in-tree package.ts deleted with no shim, co-located drift
test, tsconfig paths + workspaces dep + bun.lock.

Fixes CL-9807

Author: TheGreatAxios

agents/shakespeare/package.json

@@ -0,0 +1,13 @@
+{
+  "name": "@corbits/agent-shakespeare",
+  "version": "0.1.0",
+  "private": true,
+  "type": "module",
+  "license": "SEE LICENSE IN LICENSE.md",
+  "exports": {
+    ".": {
+      "types": "./src/index.ts",
+      "default": "./src/index.ts"
+    }
+  }
+}

…c/agent/directors/shakespeare/package.ts agents/shakespeare/src/index.tssrc/agent/directors/shakespeare/package.ts renamed to agents/shakespeare/src/index.ts src/agent/directors/shakespeare/package.ts renamed to agents/shakespeare/src/index.ts

@@ -1,12 +1,48 @@
-import type { DirectorPackage } from "../types.js";
-import { DOCS_TOOLS } from "../tool-sets.js";
-
 /**
  * Shakespeare worker (CL-7029).
  * Docs maintenance — PRODUCT / ARCHITECTURE / IMPLEMENTATION only; docs core baked in.
  * Package id/path stays `shakespeare` (global rename is out of scope).
+ *
+ * Ships as the @corbits/agent-shakespeare workspace package: the tool allowlist
+ * lives here so the package stays importable without the app. Drift against
+ * the app build surface fails src/agent/directors/shakespeare/package.test.ts.
  */
-export const shakespearePackage: DirectorPackage = {
+export type AgentPackage = {
+  readonly id: "shakespeare";
+  readonly primaryIntent: string;
+  readonly outOfLane: readonly string[];
+  readonly description: string;
+  readonly systemPrompt: string;
+  /** Unset — shakespeare never attaches skill bodies at spawn. */
+  readonly attachedSkills?: readonly string[];
+  /** Unset — shakespeare declares no skill scope. */
+  readonly optionalSkills?: readonly string[];
+  readonly tools: {
+    readonly allow: readonly string[];
+  };
+  readonly spawn: {
+    readonly maySpawn: false;
+  };
+  readonly modelRole: "docs";
+  readonly tier: "leaf";
+};
+
+const SHAKESPEARE_TOOLS = [
+  "read_file",
+  "grep",
+  "search_files",
+  "list_dir",
+  "lsp",
+  "web_fetch",
+  "web_search",
+  "skill_search",
+  "use_skill",
+  "write_file",
+  "edit_file",
+  "delete_file",
+] as const;
+
+export const shakespearePackage: AgentPackage = {
   id: "shakespeare",
   primaryIntent: "Maintain product, architecture, and implementation docs",
   outOfLane: [
@@ -73,7 +109,7 @@ Confirm what changed and where. Summarize consistency/gap follow-ups. Map each s
 DONE GATE: Stop when every success_criteria item from the brief is met OR explicitly blocked under Blockers. Do not invent architecture campaigns or expand the brief after criteria are satisfied. If the ask needs product code, review, or brand/DESIGN.md, report Blockers — do not become Coder, Reviewer, or Designer.
 
 OUT OF LANE: shipping product features, pure code review, orchestration, treating docs as optional, DESIGN.md ownership, becoming Coder or Reviewer as primary.`,
-  tools: { allow: DOCS_TOOLS },
+  tools: { allow: [...SHAKESPEARE_TOOLS] },
   spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "docs",

bun.lock

@@ -86,6 +86,7 @@
     "tar": "^7.5.1"
   },
   "dependencies": {
+    "@corbits/agent-shakespeare": "workspace:*",
     "@corbits/codex-provider": "0.1.1",
     "@corbits/oauth-core": "0.2.0",
     "@corbits/openai-responses": "0.2.1",

package.json

@@ -1,3 +1,4 @@
+import { shakespearePackage } from "@corbits/agent-shakespeare";
 import type { AgentProfile, CapabilityFilter } from "../profile-types.js";
 import { artistPackage } from "./artist/package.js";
 import { coderPackage } from "./coder/package.js";
@@ -8,7 +9,6 @@ import { plannerPackage } from "./planner/package.js";
 import { proberPackage } from "./prober/package.js";
 import { qaLeadPackage } from "./qa-lead/package.js";
 import { reviewerPackage } from "./reviewer/package.js";
-import { shakespearePackage } from "./shakespeare/package.js";
 import { wardenPackage } from "./warden/package.js";
 import { formatDirectorSystemPrompt } from "./identity.js";
 import {

src/agent/directors/registry.ts

@@ -0,0 +1,27 @@
+import { describe, expect, test } from "bun:test";
+
+import { shakespearePackage } from "@corbits/agent-shakespeare";
+import { DOCS_TOOLS } from "../tool-sets.js";
+import { DIRECTOR_REGISTRY } from "../registry.js";
+import type { DirectorPackage } from "../types.js";
+
+describe("shakespearePackage", () => {
+  test("workspace package satisfies the director contract and feeds the registry", () => {
+    const asDirector: DirectorPackage = shakespearePackage;
+    expect(asDirector.id).toBe("shakespeare");
+    expect(DIRECTOR_REGISTRY.shakespeare).toBe(shakespearePackage);
+    expect([...shakespearePackage.tools.allow]).toEqual([...DOCS_TOOLS]);
+  });
+
+  test("extracted copy stays docs-scoped and non-spawning", () => {
+    expect(shakespearePackage.tools.allow).toContain("write_file");
+    expect(shakespearePackage.tools.allow).toContain("edit_file");
+    expect(shakespearePackage.tools.allow).toContain("delete_file");
+    expect(shakespearePackage.tools.allow).not.toContain("run_shell");
+    expect(shakespearePackage.tools.allow).not.toContain("spawn_agent");
+    expect(shakespearePackage.tools.allow).not.toContain("ask_operator");
+    expect(shakespearePackage.spawn.maySpawn).toBe(false);
+    expect(shakespearePackage.tier).toBe("leaf");
+    expect(shakespearePackage.modelRole).toBe("docs");
+  });
+});

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

@@ -5,6 +5,7 @@
     "composite": false,
     "baseUrl": ".",
     "paths": {
+      "@corbits/agent-shakespeare": ["./agents/shakespeare/src/index.ts"],
       "@corbits/provider-opencode-go": ["./packages/opencode-go/src/index.ts"],
       "@corbits/first-class-providers": [
         "./packages/first-class-providers/src/index.ts"