@markjaquith/agency 3.2.13 → 3.2.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@markjaquith/agency",
-  "version": "3.2.13",
+  "version": "3.2.15",
   "description": "Manage agentic work across repositories with durable workbases",
   "keywords": [
     "agents",
@@ -22,10 +22,15 @@
     "cli.ts",
     "cli-main.ts",
     "src",
+    "!src/**/*.test.ts",
+    "!src/**/*.test.tsx",
+    "!src/test-utils.ts",
     "pi-extensions",
+    "!pi-extensions/**/*.test.ts",
     "scripts/install-pi-extension.ts",
     "schemas",
-    "fixtures/protocol",
+    "fixtures/protocol/success.json",
+    "fixtures/protocol/error.json",
     "README.md",
     "LICENSE"
   ],

package/src/workbase/schemas.ts

@@ -42,6 +42,18 @@ export const WorkStatus = Schema.Literal(
 
 const IsoTimestamp = NonEmptyString.pipe(
 	Schema.pattern(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d{3})?Z$/),
+	Schema.filter(
+		(value) => {
+			const timestamp = Date.parse(value)
+			return (
+				Number.isFinite(timestamp) &&
+				new Date(timestamp).toISOString() === value
+			)
+		},
+		{
+			message: () => "Expected a canonical ISO-8601 timestamp",
+		},
+	),
 )
 
 const GitCommit = Schema.String.pipe(Schema.pattern(/^[a-f0-9]{40}$/))

package/fixtures/protocol/orchestration-recipes.json

@@ -1,53 +0,0 @@
-[
-  ["context", "--task", "checkout", "--phase", "ui", "--json"],
-  ["worktree", "inspect", "checkout", "ui", "--json"],
-  ["next", "--json"],
-  ["graph", "--ready", "--json"],
-  [
-    "task",
-    "create",
-    "investigate-checkout",
-    "--purpose",
-    "investigation",
-    "--repo",
-    "agency",
-    "--json"
-  ],
-  [
-    "task",
-    "handoff",
-    "investigate-checkout",
-    "implement-checkout",
-    "--repo",
-    "agency",
-    "--json"
-  ],
-  [
-    "work",
-    "prepare",
-    "--task",
-    "checkout",
-    "--phase",
-    "ui",
-    "--dry-run",
-    "--json"
-  ],
-  ["work", "prepare", "--task", "checkout", "--phase", "ui", "--json"],
-  ["work", "tasks/checkout/phases/ui", "--agent", "opencode"],
-  ["sync", "--dry-run", "--json"],
-  ["sync", "--json"],
-  [
-    "task",
-    "status",
-    "implement-checkout",
-    "done",
-    "--no-pull-request",
-    "--summary",
-    "Completed without a pull request",
-    "--json"
-  ],
-  ["pr", "create", "checkout", "ui", "--json"],
-  ["sync", "checkout", "ui", "--json"],
-  ["doctor", "--json"],
-  ["worktree", "repair", "checkout", "ui", "--dry-run", "--json"]
-]

package/pi-extensions/agency.test.ts

@@ -1,117 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test"
-import { mkdir } from "node:fs/promises"
-import { join } from "node:path"
-import { cleanupTempDir, createTempDir } from "../src/test-utils"
-import agencyExtension from "./agency"
-
-describe("global Pi extension", () => {
-	let root: string
-
-	beforeEach(async () => {
-		root = await createTempDir()
-	})
-
-	afterEach(async () => cleanupTempDir(root))
-
-	const load = (response?: object) => {
-		const handlers = new Map<string, (...args: any[]) => any>()
-		const pi = {
-			exec: async () => ({
-				code: response ? 0 : 1,
-				stdout: response ? JSON.stringify(response) : "",
-				stderr: "",
-			}),
-			on: (name: string, handler: (...args: any[]) => any) => {
-				handlers.set(name, handler)
-			},
-		}
-		agencyExtension(pi as never)
-		return handlers
-	}
-
-	test("no-ops outside Agency workbases without invoking the CLI", async () => {
-		let executions = 0
-		const handlers = new Map<string, (...args: any[]) => any>()
-		agencyExtension({
-			exec: async () => {
-				executions += 1
-				return { code: 1, stdout: "", stderr: "" }
-			},
-			on: (name: string, handler: (...args: any[]) => any) => {
-				handlers.set(name, handler)
-			},
-		} as never)
-
-		expect(
-			await handlers.get("resources_discover")?.({ cwd: root }),
-		).toBeUndefined()
-		expect(
-			await handlers.get("before_agent_start")?.(
-				{ systemPrompt: "Base" },
-				{ cwd: root },
-			),
-		).toBeUndefined()
-		expect(executions).toBe(0)
-	})
-
-	test("uses CLI context from nested directories for skills and authority", async () => {
-		const task = join(root, "tasks", "example")
-		const nested = join(task, "code", "agency", "src")
-		const checkout = join(task, "code", "agency")
-		await Bun.write(join(root, "agency.json"), '{"version":2}\n')
-		await mkdir(join(root, ".agency"), { recursive: true })
-		await Bun.write(
-			join(root, ".agency", "AGENTS.md"),
-			"# Managed instructions\n",
-		)
-		await mkdir(join(checkout, ".agents", "skills"), { recursive: true })
-		await mkdir(nested, { recursive: true })
-
-		const handlers = load({
-			ok: true,
-			result: {
-				workbase: { root },
-				target: {
-					kind: "task",
-					taskId: "example",
-					path: join(task, "TASK.md"),
-				},
-				authority: { mode: "execution", writable: { checkoutPath: checkout } },
-				documents: { task: { data: { status: "working" } } },
-				validation: { valid: true },
-			},
-		})
-		const resources = await handlers.get("resources_discover")?.({
-			cwd: nested,
-		})
-		expect(resources.skillPaths).toEqual([join(checkout, ".agents", "skills")])
-
-		const prompt = await handlers.get("before_agent_start")?.(
-			{ systemPrompt: "Base" },
-			{ cwd: nested },
-		)
-		expect(prompt.systemPrompt).toContain("# Managed instructions")
-		expect(prompt.systemPrompt).toContain(
-			`${checkout} as the default implementation directory`,
-		)
-	})
-
-	test("rejects CLI context for a different workbase", async () => {
-		await Bun.write(join(root, "agency.json"), '{"version":2}\n')
-		const handlers = load({
-			ok: true,
-			result: {
-				workbase: { root: join(root, "other") },
-				target: { kind: "epic", epicId: "example" },
-				validation: { valid: true },
-			},
-		})
-
-		expect(
-			await handlers.get("before_agent_start")?.(
-				{ systemPrompt: "Base" },
-				{ cwd: root },
-			),
-		).toBeUndefined()
-	})
-})

package/src/check-commits.test.ts

@@ -1,58 +0,0 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test"
-import { writeFileSync } from "node:fs"
-import { join } from "node:path"
-import { cleanupTempDir, createTempDir } from "./test-utils"
-
-const checkCommits = join(import.meta.dir, "../scripts/check-commits")
-
-describe("check-commits", () => {
-	let root: string
-
-	function git(...args: string[]): void {
-		const result = Bun.spawnSync(["git", ...args], { cwd: root })
-		if (result.exitCode !== 0) {
-			throw new Error(new TextDecoder().decode(result.stderr))
-		}
-	}
-
-	function commit(subject: string, file: string): void {
-		writeFileSync(join(root, file), `${subject}\n`)
-		git("add", file)
-		git("commit", "-m", subject)
-	}
-
-	beforeEach(async () => {
-		root = await createTempDir()
-		git("init", "--initial-branch=main")
-		git("config", "user.email", "agency@example.com")
-		git("config", "user.name", "Agency Test")
-		commit("chore: initial commit", "initial.txt")
-		git("update-ref", "refs/remotes/origin/main", "HEAD")
-	})
-
-	afterEach(async () => cleanupTempDir(root))
-
-	test("allows merge commits with non-conventional subjects", () => {
-		git("checkout", "-b", "feature")
-		commit("feat: add feature", "feature.txt")
-		git("checkout", "-b", "topic", "main")
-		commit("fix: add topic", "topic.txt")
-		git("checkout", "feature")
-		git("merge", "--no-ff", "topic", "-m", "Combine topic into feature")
-
-		const result = Bun.spawnSync([checkCommits], { cwd: root })
-
-		expect(result.exitCode).toBe(0)
-	})
-
-	test("rejects non-conventional non-merge commits", () => {
-		commit("not a conventional commit", "invalid.txt")
-
-		const result = Bun.spawnSync([checkCommits], { cwd: root })
-
-		expect(result.exitCode).toBe(1)
-		expect(new TextDecoder().decode(result.stderr)).toContain(
-			"not a conventional commit",
-		)
-	})
-})