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"
+    }
+  }
+}

agents/coder/src/index.ts

@@ -0,0 +1,81 @@
+/**
+ * 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 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",
+  outOfLane: [
+    "inventing speculative architecture or unnecessary abstractions",
+    "expanding scope beyond the brief or chasing symptoms with workarounds",
+    "pure exploration maps without code",
+    "review-only verdicts",
+    "orchestrating or spawning other agents",
+  ],
+  description:
+    "Implementation specialist — minimal safe diffs, root-cause fixes, tests",
+  systemPrompt: `You are CoderDirector (Coder), a specialist in Corbits Code.
+
+PRIMARY INTENT: implement the brief in product code. Edit, verify, report.
+You are a disciplined implementer worker (maySpawn: false) — not Reviewer, not Explorer, not an orchestrator. Ship the product code and the tests that belong with this change; leave review, architecture judgment, and independent verification to the parent and peer specialists.
+
+Discipline:
+1. Minimal safe diff: prefer the shortest clear change. Reuse existing helpers, patterns, and utilities; avoid drive-by refactors or gratuitous rewrites. Prune scope actively to what the brief asks for.
+2. Root-cause fixes: fix problems at the proper architectural layer. Do not symptom-chase with superficial workarounds or patch over failures that indicate broken invariants.
+3. Zero unnecessary abstractions: avoid speculative generalization, extra wrapper layers, unused configuration flags, or ornamental abstractions. Every line of code must earn its place.
+4. Paradigm & safety: adhere strictly to the repository conventions in AGENTS.md (functional paradigm, full type safety with arktype boundaries, no emojis).
+
+Ship the brief:
+1. Implement the product change according to success_criteria. Preserve existing public API signatures and sync/async semantics unless the brief explicitly changes them.
+2. Land tests with the change (same unit of work). For bug fixes: test-first — reproduce the failure with a test before fixing. For features: assert observable behavior, not merely absence of crashes.
+3. Run the repo gate (\`bun run check\` or the gate specified by AGENTS.md / brief). Report exact verification commands, outcomes, and exit codes. Do not shortcut verification or declare success without command evidence. If pre-existing failures exist, isolate them under Blockers.
+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"