feat(agent-coder): extract coder director as a versioned package

Author: TheGreatAxios

agents/coder/package.json

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

src/agent/directors/coder/package.ts agents/coder/src/index.tssrc/agent/directors/coder/package.ts renamed to agents/coder/src/index.ts

@@ -1,12 +1,49 @@
-import type { DirectorPackage } from "../types.js";
-import { BUILD_TOOLS } from "../tool-sets.js";
-
 /**
  * Coder worker: implementation specialist.
  * Minimal safe diffs, root-cause fixes at the proper layer, zero unnecessary
  * abstractions, unit tests landed with changes, repo check gate.
+ *
+ * Ships as the @corbits/agent-coder 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/coder/package.test.ts.
  */
-export const coderPackage: DirectorPackage = {
+export type AgentPackage = {
+  readonly id: "coder";
+  readonly primaryIntent: string;
+  readonly outOfLane: readonly string[];
+  readonly description: string;
+  readonly systemPrompt: string;
+  /** Unset — coder never attaches skill bodies at spawn. */
+  readonly attachedSkills?: readonly string[];
+  /** Unset — coder declares no skill scope. */
+  readonly optionalSkills?: readonly string[];
+  readonly tools: {
+    readonly allow: readonly string[];
+  };
+  readonly spawn: {
+    readonly maySpawn: false;
+  };
+  readonly modelRole: "implement";
+  readonly tier: "leaf";
+};
+
+const CODER_TOOLS = [
+  "read_file",
+  "grep",
+  "search_files",
+  "list_dir",
+  "lsp",
+  "run_shell",
+  "web_fetch",
+  "web_search",
+  "skill_search",
+  "use_skill",
+  "write_file",
+  "edit_file",
+  "delete_file",
+] as const;
+
+export const coderPackage: AgentPackage = {
   id: "coder",
   primaryIntent:
     "Implement the brief in product code — minimal safe diffs, root-cause fixes, tests with change",
@@ -19,10 +56,6 @@ export const coderPackage: DirectorPackage = {
   ],
   description:
     "Implementation specialist — minimal safe diffs, root-cause fixes, tests",
-  tools: { allow: BUILD_TOOLS },
-  spawn: { maySpawn: false },
-  tier: "leaf",
-  modelRole: "implement",
   systemPrompt: `You are CoderDirector (Coder), a specialist in Corbits Code.
 
 PRIMARY INTENT: implement the brief in product code. Edit, verify, report.
@@ -41,4 +74,8 @@ Ship the brief:
 4. Report envelope: use Summary / Findings / Blockers / Paths. In Findings, map each success_criteria item to pass, fail, or blocked with verification evidence. Paths must list every file touched. Do not commit unless the brief explicitly demands it.
 
 OUT OF LANE: pure exploration maps, speculative abstractions, review-only verdicts, mechanical command lists without implementing, fleet orchestration or spawning.`,
+  tools: { allow: [...CODER_TOOLS] },
+  spawn: { maySpawn: false },
+  tier: "leaf",
+  modelRole: "implement",
 };

bun.lock

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

package.json

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

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

@@ -1,6 +1,6 @@
 import type { AgentProfile, CapabilityFilter } from "../profile-types.js";
 import { artistPackage } from "./artist/package.js";
-import { coderPackage } from "./coder/package.js";
+import { coderPackage } from "@corbits/agent-coder";
 import { designerPackage } from "./designer/package.js";
 import { dispatchPackage } from "./dispatch/package.js";
 import { explorerPackage } from "./explorer/package.js";

src/agent/directors/registry.ts

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