@markjaquith/agency 0.6.1 → 0.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@markjaquith/agency",
-  "version": "0.6.1",
+  "version": "0.7.0",
   "description": "Manages personal agents files",
   "license": "MIT",
   "author": "Mark Jaquith",
@@ -14,7 +14,7 @@
     "agency": "cli.ts"
   },
   "scripts": {
-    "test": "bun test",
+    "test": "find src -name '*.test.ts' -print0 | xargs -0 -n 1 -P 4 bun test",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
     "knip": "knip --production",

package/src/commands/push.ts

@@ -278,23 +278,41 @@ const openGitHubPR = (
 		const { verbose = false } = options
 
 		// Run gh pr create --web with --head to specify the emit branch
-		const ghResult = yield* spawnProcess(
+		// Set environment variables to ensure non-interactive mode in CI
+		// Add a 3 second timeout to prevent hanging in CI environments
+		const ghEffect = spawnProcess(
 			["gh", "pr", "create", "--web", "--head", branchName],
 			{
 				cwd: gitRoot,
 				stdout: verbose ? "inherit" : "pipe",
 				stderr: "pipe",
+				env: {
+					GH_PROMPT_DISABLED: "1",
+					NO_COLOR: "1",
+				},
 			},
 		).pipe(
-			Effect.catchAll((error) =>
-				Effect.succeed({
-					exitCode: error.exitCode,
+			Effect.timeout("3 seconds"),
+			Effect.catchAll((error) => {
+				// Handle timeout or process errors
+				if (error._tag === "TimeoutException") {
+					return Effect.succeed({
+						exitCode: -1,
+						stdout: "",
+						stderr: "gh command timed out after 3 seconds",
+					})
+				}
+				// Handle ProcessError
+				return Effect.succeed({
+					exitCode: (error as any).exitCode ?? -1,
 					stdout: "",
-					stderr: error.stderr,
-				}),
-			),
+					stderr: (error as any).stderr ?? String(error),
+				})
+			}),
 		)
 
+		const ghResult = yield* ghEffect
+
 		if (ghResult.exitCode !== 0) {
 			return yield* Effect.fail(
 				new Error(`gh CLI command failed: ${ghResult.stderr.trim()}`),

package/src/commands/task-main.test.ts

@@ -759,6 +759,49 @@ describe("task command", () => {
 
 			expect(await fileExists(join(tempDir, "AGENTS.md"))).toBe(true)
 		})
+
+		test("fails when on agency source branch without branch name in silent mode", async () => {
+			await initGitRepo(tempDir)
+			process.chdir(tempDir)
+
+			await initAgency(tempDir, "test")
+
+			// Create a task branch with agency files
+			await runTestEffect(task({ silent: true, emit: "original-feature" }))
+
+			// Verify we're on the source branch with agency.json
+			const currentBranch = await getCurrentBranch(tempDir)
+			expect(currentBranch).toBe("agency/original-feature")
+			expect(await fileExists(join(tempDir, "agency.json"))).toBe(true)
+
+			// Try to run agency task again without providing a branch name
+			// This should fail because we're on an agency source branch
+			await expect(runTestEffect(task({ silent: true }))).rejects.toThrow(
+				"agency source branch",
+			)
+		})
+
+		test("prompts for branch name when on agency source branch", async () => {
+			await initGitRepo(tempDir)
+			process.chdir(tempDir)
+
+			await initAgency(tempDir, "test")
+
+			// Create a task branch with agency files
+			await runTestEffect(task({ silent: true, emit: "original-feature" }))
+
+			// Verify we're on the source branch with agency.json
+			const currentBranch = await getCurrentBranch(tempDir)
+			expect(currentBranch).toBe("agency/original-feature")
+			expect(await fileExists(join(tempDir, "agency.json"))).toBe(true)
+
+			// With a branch name provided, it should work
+			await runTestEffect(task({ silent: true, emit: "new-feature" }))
+
+			// Verify we're now on the new branch
+			const newBranch = await getCurrentBranch(tempDir)
+			expect(newBranch).toBe("agency/new-feature")
+		})
 	})
 
 	describe("template-based source files", () => {

package/src/commands/task.ts

@@ -212,11 +212,21 @@ const taskContinue = (options: TaskOptions) =>
 			targetPath,
 			baseBranchToBranchFrom,
 		)
+		// Calculate the emit branch name for display
+		const cleanBranchForDisplay =
+			extractCleanBranch(sourceBranchName, config.sourceBranchPattern) ||
+			sourceBranchName
+		const emitBranchForDisplay = makeEmitBranchName(
+			cleanBranchForDisplay,
+			config.emitBranch,
+		)
+
 		log(
-			done(
-				`Created and switched to branch ${highlight.branch(sourceBranchName)} based on ${highlight.branch(baseBranchToBranchFrom)}`,
+			info(
+				`(${highlight.branch(baseBranchToBranchFrom)}) ${highlight.branch(sourceBranchName)} → ${highlight.branch(emitBranchForDisplay)}`,
 			),
 		)
+		log(done(`Created and switched to ${highlight.branch(sourceBranchName)}`))
 
 		// Calculate the new emit branch name
 		const newEmitBranchName = makeEmitBranchName(branchName, config.emitBranch)
@@ -367,6 +377,11 @@ export const task = (options: TaskOptions = {}) =>
 		const isFeature = yield* git.isFeatureBranch(currentBranch, targetPath)
 		verboseLog(`Is feature branch: ${isFeature}`)
 
+		// Check if we're on an agency source branch (has agency.json with backpacked files)
+		const agencyJsonPath = resolve(targetPath, "agency.json")
+		const hasAgencyJson = yield* fs.exists(agencyJsonPath)
+		verboseLog(`Has agency.json: ${hasAgencyJson}`)
+
 		// Determine base branch to branch from
 		let baseBranchToBranchFrom: string | undefined
 
@@ -458,8 +473,8 @@ export const task = (options: TaskOptions = {}) =>
 		// Determine branch name logic
 		let branchName = options.emit || options.branch
 
-		// If on main branch or using --from without a branch name, prompt for it (unless in silent mode)
-		if ((!isFeature || options.from) && !branchName) {
+		// If on main branch, using --from, or on an agency source branch without a branch name, prompt for it (unless in silent mode)
+		if ((!isFeature || options.from || hasAgencyJson) && !branchName) {
 			if (silent) {
 				if (options.from) {
 					return yield* Effect.fail(
@@ -469,6 +484,15 @@ export const task = (options: TaskOptions = {}) =>
 						),
 					)
 				}
+				if (hasAgencyJson) {
+					return yield* Effect.fail(
+						new Error(
+							`You're currently on ${highlight.branch(currentBranch)}, which is an agency source branch.\n` +
+								`Branch name is required when re-importing backpacked files.\n` +
+								`Use: 'agency task <branch-name>' or 'agency task --continue <branch-name>'`,
+						),
+					)
+				}
 				return yield* Effect.fail(
 					new Error(
 						`You're currently on ${highlight.branch(currentBranch)}, which appears to be your main branch.\n` +
@@ -823,11 +847,27 @@ const createFeatureBranchEffect = (
 		}
 
 		yield* git.createBranch(branchName, targetPath, baseBranch)
+
+		// Load config for emit pattern calculation
+		const configService = yield* ConfigService
+		const config = yield* configService.loadConfig()
+
+		// Calculate the emit branch name for display
+		const cleanBranchForDisplay =
+			extractCleanBranch(branchName, config.sourceBranchPattern) || branchName
+		const emitBranchForDisplay = makeEmitBranchName(
+			cleanBranchForDisplay,
+			config.emitBranch,
+		)
+
 		log(
-			done(
-				`Created and switched to branch ${highlight.branch(branchName)}${baseBranch ? ` based on ${highlight.branch(baseBranch)}` : ""}`,
+			info(
+				baseBranch
+					? `(${highlight.branch(baseBranch)}) ${highlight.branch(branchName)} → ${highlight.branch(emitBranchForDisplay)}`
+					: `${highlight.branch(branchName)} → ${highlight.branch(emitBranchForDisplay)}`,
 			),
 		)
+		log(done(`Created and switched to ${highlight.branch(branchName)}`))
 	})
 
 // Helper: Discover template files

package/templates/opencode.json

@@ -1,4 +1,4 @@
 {
-	"$schema": "https://opencode.ai/config.json",
-	"instructions": ["AGENCY.md", "TASK.md"]
+  "$schema": "https://opencode.ai/config.json",
+  "instructions": ["AGENCY.md", "TASK.md"]
 }