@app-vitals/agent-lib 0.16.2 → 1.0.0

package/default-specialty.unit.test.ts

@@ -20,7 +20,7 @@ describe("default specialty resolution", () => {
       join(import.meta.dir, "..", "agent-types"),
     );
     expect(
-      existsSync(join(SHIPPED_AGENT_TYPES_DIR, "specialist", "manifest.yaml")),
+      existsSync(join(SHIPPED_AGENT_TYPES_DIR, "specialist", "templates")),
     ).toBe(true);
   });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@app-vitals/agent-lib",
-  "version": "0.16.2",
+  "version": "1.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/app-vitals/agent.git",
@@ -14,7 +14,6 @@
   "exports": {
     "./admin-types": "./admin-types.ts",
     "./chat-notify": "./chat-notify.ts",
-    "./claim-ttl": "./claim-ttl.ts",
     "./claude-timeout": "./claude-timeout.ts",
     "./cli-flags": "./cli-flags.ts",
     "./clone-plan": "./clone-plan.ts",
@@ -29,11 +28,10 @@
     "./redact": "./redact.ts",
     "./request-context": "./request-context.ts",
     "./secret-env-vars": "./secret-env-vars.ts",
-    "./default-account": "./default-account.ts",
     "./default-agent-env": "./default-agent-env.ts",
     "./default-specialty": "./default-specialty.ts",
+    "./specialist-type": "./specialist-type.ts",
     "./sentry": "./sentry.ts",
-    "./task-store-types": "./task-store-types.ts",
     "./web/*": "./web/*"
   },
   "dependencies": {

package/specialist-type.ts

@@ -0,0 +1,29 @@
+/**
+ * lib/specialist-type.ts
+ *
+ * The platform's one built-in agent type, as a plain typed record. Replaces
+ * the former agent-types/specialist/manifest.yaml (and its strict Zod
+ * schema/loader): the values below are exactly what that manifest declared
+ * and that anything still read from it. Shared by admin (tools floor,
+ * required-env reporting, type validity) and runtime (template lookup).
+ */
+
+export interface SpecialtyType {
+  /** The value stored in Agent.specialty. */
+  name: string;
+  displayName: string;
+  /** Workspace template directory, relative to the repo/image root. */
+  templatesDir: string;
+  /** Tool patterns seeded as AgentTool rows on create. */
+  tools: readonly string[];
+  /** Env keys an agent of this type must have; reported when missing. */
+  requiredEnv: readonly string[];
+}
+
+export const SPECIALIST_TYPE: SpecialtyType = {
+  name: "specialist",
+  displayName: "Specialist Agent",
+  templatesDir: "agent-types/specialist/templates/",
+  tools: ["Bash", "WebSearch", "WebFetch", "Agent"],
+  requiredEnv: ["CLAUDE_CODE_OAUTH_TOKEN"],
+};

package/specialist-type.unit.test.ts

@@ -0,0 +1,45 @@
+/**
+ * Contract test: the specialist type record carries the exact values the
+ * retired agent-types/specialist/manifest.yaml declared, so existing
+ * specialist agents keep the same tools floor, required-env reporting and
+ * template seeding.
+ */
+
+import { describe, expect, it } from "bun:test";
+import { existsSync } from "node:fs";
+import { join } from "node:path";
+import { SPECIALIST_TYPE } from "./specialist-type.ts";
+
+describe("SPECIALIST_TYPE", () => {
+  it("keeps the old manifest's identity and tools floor", () => {
+    expect(SPECIALIST_TYPE.name).toBe("specialist");
+    expect(SPECIALIST_TYPE.displayName).toBe("Specialist Agent");
+    expect([...SPECIALIST_TYPE.tools]).toEqual([
+      "Bash",
+      "WebSearch",
+      "WebFetch",
+      "Agent",
+    ]);
+  });
+
+  it("keeps the old manifest's required env keys", () => {
+    expect([...SPECIALIST_TYPE.requiredEnv]).toEqual([
+      "CLAUDE_CODE_OAUTH_TOKEN",
+    ]);
+  });
+
+  it("points templatesDir at the shipped specialist templates", () => {
+    expect(SPECIALIST_TYPE.templatesDir).toBe(
+      "agent-types/specialist/templates/",
+    );
+    const dir = join(import.meta.dir, "..", SPECIALIST_TYPE.templatesDir);
+    for (const f of [
+      "CLAUDE.md.template",
+      "SOUL.md.template",
+      "IDENTITY.md.template",
+      "SPECIALTY.md.template",
+    ]) {
+      expect(existsSync(join(dir, f))).toBe(true);
+    }
+  });
+});

package/claim-ttl.ts

@@ -1,12 +0,0 @@
-import { DEFAULT_CLAUDE_TIMEOUT_MS } from "./claude-timeout.ts";
-
-/** Extra headroom added on top of the session timeout (5min). */
-export const CLAIM_TTL_BUFFER_MS = 300_000;
-
-/**
- * Default claim TTL: how long a claim (Task or PR review/patch/deploy) remains
- * valid without a heartbeat before it's eligible for reaping. Must exceed
- * DEFAULT_CLAUDE_TIMEOUT_MS so a claim isn't reaped mid-session.
- */
-export const DEFAULT_CLAIM_TTL_MS =
-  DEFAULT_CLAUDE_TIMEOUT_MS + CLAIM_TTL_BUFFER_MS;

package/claim-ttl.unit.test.ts

@@ -1,26 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { CLAIM_TTL_BUFFER_MS, DEFAULT_CLAIM_TTL_MS } from "./claim-ttl.ts";
-import { DEFAULT_CLAUDE_TIMEOUT_MS } from "./claude-timeout.ts";
-
-describe("claim-ttl constants", () => {
-  test("CLAIM_TTL_BUFFER_MS is 300_000 (5min)", () => {
-    expect(CLAIM_TTL_BUFFER_MS).toBe(300_000);
-  });
-
-  test("DEFAULT_CLAIM_TTL_MS is 3_900_000 (65min)", () => {
-    expect(DEFAULT_CLAIM_TTL_MS).toBe(3_900_000);
-  });
-
-  test("DEFAULT_CLAIM_TTL_MS is computed as the sum of the other two constants", () => {
-    expect(DEFAULT_CLAIM_TTL_MS).toBe(
-      DEFAULT_CLAUDE_TIMEOUT_MS + CLAIM_TTL_BUFFER_MS,
-    );
-  });
-
-  // Relocated from claude-timeout.unit.test.ts (AEP-4.1): claim-ttl is the
-  // task-store side, so the cross-module invariant belongs here and the
-  // platform-set lib test no longer imports it.
-  test("DEFAULT_CLAIM_TTL_MS exceeds DEFAULT_CLAUDE_TIMEOUT_MS", () => {
-    expect(DEFAULT_CLAIM_TTL_MS).toBeGreaterThan(DEFAULT_CLAUDE_TIMEOUT_MS);
-  });
-});

package/default-account.ts

@@ -1,5 +0,0 @@
-/**
- * The account every pre-multi-tenancy task-store row belongs to. Never use
- * NULL for task-store accountId — the column is NOT NULL DEFAULT 'default'.
- */
-export const DEFAULT_ACCOUNT_ID = "default";

package/default-account.unit.test.ts

@@ -1,8 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { DEFAULT_ACCOUNT_ID } from "./default-account.ts";
-
-describe("DEFAULT_ACCOUNT_ID", () => {
-  it("is the literal 'default' matching the migration's column default", () => {
-    expect(DEFAULT_ACCOUNT_ID).toBe("default");
-  });
-});