feat(intern): ship intern as a workspace agent package (#1159)

* feat(intern): ship intern as a workspace agent package

* chore(dead-exports): drop stale fixture allowlist entries

Linux CI stopped reporting those path-loaded fixture exports after
intern shipped as a workspace package. Ignore comments keep Darwin
ts-prune from treating the same exports as new dead code.

Author: TheGreatAxios

agents/intern/package.json

@@ -0,0 +1,13 @@
+{
+  "name": "@corbits/agent-intern",
+  "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/intern/package.ts agents/intern/src/index.tssrc/agent/directors/intern/package.ts renamed to agents/intern/src/index.ts

@@ -1,13 +1,38 @@
-import type { DirectorPackage } from "../types.js";
-import { INTERN_TOOLS } from "../tool-sets.js";
-
 /**
  * Mechanical intern worker — 1:1 port of gaas intern.md
  * (`abklabs/agents` plugins/gaas/agents/intern.md @ 6e16b6c).
  * Tool-name mapping only (ask → ask_director). Path writes live on the
  * intern mount and in the dispatch brief, not in extra prompt essays.
  */
-export const internPackage: DirectorPackage = {
+export type AgentPackage = {
+  readonly id: "intern";
+  readonly primaryIntent: string;
+  readonly outOfLane: readonly string[];
+  readonly description: string;
+  readonly systemPrompt: string;
+  readonly optionalSkills: readonly string[];
+  readonly tools: {
+    readonly allow: readonly string[];
+  };
+  readonly spawn: {
+    readonly maySpawn: false;
+  };
+  readonly modelRole: "implement";
+  readonly tier: "leaf";
+};
+
+const INTERN_TOOLS = [
+  "run_shell",
+  "read_file",
+  "list_dir",
+  "write_file",
+  "edit_file",
+  "delete_file",
+  "skill_search",
+  "use_skill",
+] as const;
+
+export const internPackage: AgentPackage = {
   id: "intern",
   primaryIntent:
     "Execute clear mechanical instructions exactly — zero judgment, zero invention",

bun.lock

@@ -1,5 +1,6 @@
 export const notes: string[] = [];
 
+// ts-prune-ignore-next
 export function addNote(note: string): void {
   notes.push(note);
 }

evals/completion/tasks/decline-refactor/fixture/notes.ts

@@ -1 +1,2 @@
+// ts-prune-ignore-next
 export const data = { seed: 7 };

evals/completion/tasks/stall-read/fixture/data.ts

@@ -1,3 +1,4 @@
+// ts-prune-ignore-next
 export function total(values: number[]): number {
   let sum = 0;
   for (const value of values) {

evals/completion/tasks/sum-fix/fixture/sum.ts

@@ -3,6 +3,7 @@ export interface Reply {
   body: unknown;
 }
 
+// ts-prune-ignore-next
 export function handleRequest(method: string, path: string): Reply {
   if (method === "GET" && path === "/health") {
     return { status: 200, body: { ok: true } };

evals/completion/tasks/version-endpoint/fixture/service.ts

@@ -86,6 +86,7 @@
     "tar": "^7.5.1"
   },
   "dependencies": {
+    "@corbits/agent-intern": "workspace:*",
     "@corbits/codex-provider": "github:corbitsdev/corbits-codex-provider",
     "@corbits/oauth-core": "github:corbitsdev/corbits-oauth-core",
     "@corbits/openai-responses": "github:corbitsdev/corbits-openai-responses",

package.json

@@ -29,18 +29,6 @@ vendor/
 src/auth/codex/constants.ts: CODEX_REFRESH_SKEW_MS
 src/auth/codex/constants.ts: CODEX_HEADLESS_REFRESH_INTERVAL_MS
 
-# Plugin fixture entry point. Loaded by file path from the fixture's plugin
-# manifest by the plugin-registration tests, so it has no static importers.
-tests/fixtures/plugins/implement-feature/src/index.ts
-
-# Eval completion-harness task fixtures (CL-7932 lane). Loaded by file path
-# from each task's script.json by the harness runner, so they have no static
-# importers.
-evals/completion/tasks/decline-refactor/fixture/notes.ts: addNote
-evals/completion/tasks/stall-read/fixture/data.ts: data
-evals/completion/tasks/sum-fix/fixture/sum.ts: total
-evals/completion/tasks/version-endpoint/fixture/service.ts: handleRequest
-
 # ts-prune parser false positives: bare tokens on `as const satisfies ...`
 # lines inside live exports. None of these is an export declaration.
 src/provider/reasoning-effort.ts: satisfies

scripts/dead-export-allowlist.txt

@@ -1,8 +1,17 @@
 import { describe, expect, test } from "bun:test";
 
-import { internPackage } from "./package.js";
+import { internPackage, type AgentPackage } from "@corbits/agent-intern";
+import { DIRECTOR_REGISTRY } from "../registry.js";
+import { INTERN_TOOLS } from "../tool-sets.js";
 
 describe("internPackage", () => {
+  test("workspace package is an AgentPackage used by the registry", () => {
+    const pkg: AgentPackage = internPackage;
+    expect(pkg.id).toBe("intern");
+    expect(DIRECTOR_REGISTRY.intern).toBe(internPackage);
+    expect([...(pkg.tools?.allow ?? [])]).toEqual([...INTERN_TOOLS]);
+  });
+
   test("systemPrompt is mechanical executor (gaas intern port)", () => {
     const p = internPackage.systemPrompt;
     expect(p).toMatch(/intern assistant/);