@markjaquith/agency 2.71.0 → 2.71.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@markjaquith/agency",
-  "version": "2.71.0",
+  "version": "2.71.2",
   "description": "Manage agentic work across repositories with durable workbases",
   "keywords": [
     "agents",
@@ -65,6 +65,7 @@
   "scripts": {
     "postinstall": "bun scripts/install-pi-extension.ts install",
     "preuninstall": "bun scripts/install-pi-extension.ts uninstall",
+    "benchmark:workbase": "bun scripts/benchmark-workbase.ts",
     "benchmark:sync": "bun scripts/benchmark-sync.ts",
     "test": "find src \\( -name '*.test.ts' -o -name '*.test.tsx' \\) -print0 | xargs -0 -n 1 -P 4 bun test",
     "test:opencode": "AGENCY_TEST_OPENCODE=1 bun test src/cli.test.ts --test-name-pattern 'provides effective whole-workbase OpenCode access'",

package/src/commands/work.test.ts

@@ -394,6 +394,15 @@ describe("work command", () => {
 			target: "execution-unit:task/example",
 			override: true,
 		})
+		expect(forced.materializeOptions[0]?.validationAlreadyPerformed).toBe(false)
+	})
+
+	test("reuses successful readiness validation during materialization", async () => {
+		const harness = createHarness()
+
+		await harness.run({ taskId: "example", opencode: true })
+
+		expect(harness.materializeOptions[0]?.validationAlreadyPerformed).toBe(true)
 	})
 
 	test("reopens forced terminal tasks through open before launching as working", async () => {

package/src/commands/work.ts

@@ -237,6 +237,7 @@ export const work = (
 			if (!target) return
 		}
 		yield* readiness.guardWorkTarget(targetNodeId(target), root, options.force)
+		const validationAlreadyPerformed = !options.force
 
 		const continuing =
 			target.kind !== "epic" &&
@@ -257,7 +258,10 @@ export const work = (
 			const phaseId = target.kind === "phase" ? target.phaseId : undefined
 			progress.start("Preparing workspace...")
 			const workspace = yield* worktrees
-				.materialize(taskId, phaseId, root, options)
+				.materialize(taskId, phaseId, root, {
+					...options,
+					validationAlreadyPerformed,
+				})
 				.pipe(
 					Effect.tap(() =>
 						Effect.sync(() => progress.succeed("Workspace ready")),

package/src/services/FileSystemService.ts

@@ -87,14 +87,7 @@ export class FileSystemService extends Effect.Service<FileSystemService>()(
 
 			readFile: (path: string) =>
 				Effect.tryPromise({
-					try: async () => {
-						const file = Bun.file(path)
-						const exists = await file.exists()
-						if (!exists) {
-							throw new Error(`File not found: ${path}`)
-						}
-						return await file.text()
-					},
+					try: () => Bun.file(path).text(),
 					catch: () => new FileNotFoundError({ path }),
 				}),
 

package/src/services/GraphService.ts

@@ -144,13 +144,15 @@ export class GraphService extends Effect.Service<GraphService>()(
 					const phases = new Map<string, Document<PhaseData>>()
 
 					const directories = (path: string) =>
-						Effect.gen(function* () {
-							if (!(yield* fs.isDirectory(path))) return []
-							return (yield* fs.readDirectory(path))
-								.filter((entry) => entry.isDirectory)
-								.map((entry) => entry.name)
-								.sort()
-						})
+						fs.readDirectory(path).pipe(
+							Effect.map((entries) =>
+								entries
+									.filter((entry) => entry.isDirectory)
+									.map((entry) => entry.name)
+									.sort(),
+							),
+							Effect.catchAll(() => Effect.succeed([])),
+						)
 
 					const readDocument = <S extends Schema.Schema.AnyNoContext>(
 						id: string,
@@ -185,9 +187,11 @@ export class GraphService extends Effect.Service<GraphService>()(
 						epicIds.map((id) =>
 							Effect.gen(function* () {
 								const path = join(root, "epics", id, "EPIC.md")
-								return (yield* fs.exists(path))
-									? yield* readDocument(id, path, EpicFrontmatter)
-									: null
+								return yield* readDocument(id, path, EpicFrontmatter).pipe(
+									Effect.catchTag("FileNotFoundError", () =>
+										Effect.succeed(null),
+									),
+								)
 							}),
 						),
 						{ concurrency: "unbounded" },
@@ -201,9 +205,15 @@ export class GraphService extends Effect.Service<GraphService>()(
 						taskIds.map((id) =>
 							Effect.gen(function* () {
 								const path = join(root, "tasks", id, "TASK.md")
-								const task = (yield* fs.exists(path))
-									? yield* readDocument(id, path, TaskFrontmatter)
-									: null
+								const task = yield* readDocument(
+									id,
+									path,
+									TaskFrontmatter,
+								).pipe(
+									Effect.catchTag("FileNotFoundError", () =>
+										Effect.succeed(null),
+									),
+								)
 								const phaseIds = yield* directories(
 									join(root, "tasks", id, "phases"),
 								)
@@ -218,13 +228,15 @@ export class GraphService extends Effect.Service<GraphService>()(
 												phaseId,
 												"PHASE.md",
 											)
-											return (yield* fs.exists(phasePath))
-												? yield* readDocument(
-														phaseId,
-														phasePath,
-														PhaseFrontmatter,
-													)
-												: null
+											return yield* readDocument(
+												phaseId,
+												phasePath,
+												PhaseFrontmatter,
+											).pipe(
+												Effect.catchTag("FileNotFoundError", () =>
+													Effect.succeed(null),
+												),
+											)
 										}),
 									),
 									{ concurrency: "unbounded" },

package/src/services/WorkbaseService.ts

@@ -271,11 +271,11 @@ export class WorkbaseService extends Effect.Service<WorkbaseService>()(
 
 					while (true) {
 						const configPath = join(current, "agency.json")
-						if (yield* fs.exists(configPath)) {
-							const content = yield* fs.readFile(configPath)
+						const content = yield* Effect.option(fs.readFile(configPath))
+						if (content._tag === "Some") {
 							let input: unknown
 							try {
-								input = JSON.parse(content)
+								input = JSON.parse(content.value)
 							} catch (cause) {
 								return yield* new WorkbaseConfigError({
 									path: configPath,
@@ -668,16 +668,15 @@ export class WorkbaseService extends Effect.Service<WorkbaseService>()(
 					}
 
 					const readDirectories = (path: string) =>
-						Effect.gen(function* () {
-							if (!(yield* fs.isDirectory(path))) {
-								return []
-							}
-							const entries = yield* fs.readDirectory(path)
-							return entries
-								.filter((entry) => entry.isDirectory)
-								.map((entry) => entry.name)
-								.sort()
-						})
+						fs.readDirectory(path).pipe(
+							Effect.map((entries) =>
+								entries
+									.filter((entry) => entry.isDirectory)
+									.map((entry) => entry.name)
+									.sort(),
+							),
+							Effect.catchAll(() => Effect.succeed([])),
+						)
 
 					const aliases = new Set(yield* service.repositoryAliases(root))
 
@@ -686,14 +685,13 @@ export class WorkbaseService extends Effect.Service<WorkbaseService>()(
 						schema: S,
 					) =>
 						Effect.gen(function* () {
-							if (!(yield* fs.exists(path))) {
+							const content = yield* Effect.option(fs.readFile(path))
+							if (content._tag === "None") {
 								issue(path, "Required document is missing")
 								return null
 							}
-
-							const content = yield* fs.readFile(path)
 							const parsed = yield* Effect.either(
-								parseFrontmatter(content, path),
+								parseFrontmatter(content.value, path),
 							)
 							if (Either.isLeft(parsed)) {
 								issue(path, parsed.left.message)