@markjaquith/agency 1.11.1 → 2.1.0

package/src/utils/colors.ts

@@ -1,178 +0,0 @@
-/**
- * Color highlighting utilities for CLI output
- *
- * This module provides centralized color management for highlighting
- * meaningful values in CLI feedback. All highlight types use the same
- * base color initially, but are separated into buckets to allow for
- * future customization.
- *
- * Colors can be disabled via:
- * - NO_COLOR environment variable (standard)
- * - Setting colorsEnabled to false
- */
-
-// ANSI color codes
-const RESET = "\x1b[0m"
-const CYAN_BRIGHT = "\x1b[96m"
-const GREEN = "\x1b[32m"
-const YELLOW = "\x1b[33m"
-
-/**
- * Central color configuration
- * All buckets use the same color initially (bright cyan)
- * but can be easily changed independently in the future
- */
-const COLORS = {
-	branch: CYAN_BRIGHT,
-	template: CYAN_BRIGHT,
-	file: CYAN_BRIGHT,
-	setting: CYAN_BRIGHT,
-	value: CYAN_BRIGHT,
-	commit: CYAN_BRIGHT,
-	pattern: CYAN_BRIGHT,
-	remote: YELLOW,
-} as const
-
-/**
- * Global flag to enable/disable colors
- * Defaults to enabled unless NO_COLOR environment variable is set
- */
-let colorsEnabled = !process.env.NO_COLOR
-
-/**
- * Enable or disable color output
- * @param enabled - Whether to enable colors
- */
-export function setColorsEnabled(enabled: boolean): void {
-	colorsEnabled = enabled
-}
-
-/**
- * Check if colors are currently enabled
- */
-function areColorsEnabled(): boolean {
-	return colorsEnabled
-}
-
-/**
- * Internal helper to apply color formatting
- * Returns plain text if colors are disabled
- */
-function colorize(text: string, color: string): string {
-	if (!colorsEnabled) {
-		return text
-	}
-	return `${color}${text}${RESET}`
-}
-
-/**
- * Highlight a branch name
- * @example highlight.branch("main") -> "\x1b[96mmain\x1b[0m"
- */
-function branch(name: string): string {
-	return colorize(name, COLORS.branch)
-}
-
-/**
- * Highlight a template name
- * @example highlight.template("my-template") -> "\x1b[96mmy-template\x1b[0m"
- */
-function template(name: string): string {
-	return colorize(name, COLORS.template)
-}
-
-/**
- * Highlight a file name or path
- * @example highlight.file("AGENTS.md") -> "\x1b[96mAGENTS.md\x1b[0m"
- */
-function file(name: string): string {
-	return colorize(name, COLORS.file)
-}
-
-/**
- * Highlight a setting name
- * @example highlight.setting("agency.template") -> "\x1b[96magency.template\x1b[0m"
- */
-function setting(name: string): string {
-	return colorize(name, COLORS.setting)
-}
-
-/**
- * Highlight a numeric value or count
- * @example highlight.value("3") -> "\x1b[96m3\x1b[0m"
- */
-function value(val: string | number): string {
-	return colorize(String(val), COLORS.value)
-}
-
-/**
- * Highlight a commit hash
- * @example highlight.commit("abc123") -> "\x1b[96mabc123\x1b[0m"
- */
-function commit(hash: string): string {
-	return colorize(hash, COLORS.commit)
-}
-
-/**
- * Highlight a pattern or placeholder
- * @example highlight.pattern("{task}") -> "\x1b[96m{task}\x1b[0m"
- */
-function pattern(text: string): string {
-	return colorize(text, COLORS.pattern)
-}
-
-/**
- * Highlight a git remote name
- * @example highlight.remote("origin") -> "\x1b[33morigin\x1b[0m"
- */
-function remote(name: string): string {
-	return colorize(name, COLORS.remote)
-}
-
-/**
- * Prepends a green checkmark to a message for success output
- * Uses the same checkmark symbol and color as ora for consistency
- * @param message - The message to prepend the checkmark to
- * @example log(done(`Merged ${highlight.branch("main")}`))
- * @example log(done("Operation completed"))
- */
-export function done(message: string): string {
-	const checkmark = colorsEnabled ? `\x1b[32m✔\x1b[0m` : "✔"
-	return `${checkmark} ${message}`
-}
-
-/**
- * Prepends an info icon to a message for informational output
- * @param message - The message to prepend the info icon to
- * @example log(info("Skipping task description"))
- * @example log(info(`File already exists`))
- */
-export function info(message: string): string {
-	return `ⓘ ${message}`
-}
-
-/**
- * Returns "s" if count is not 1, empty string otherwise
- * Useful for pluralizing words in messages
- * @param count - The count to check
- * @example `Committed ${count} file${plural(count)}`  -> "Committed 1 file" or "Committed 3 files"
- */
-export function plural(count: number): string {
-	return count === 1 ? "" : "s"
-}
-
-/**
- * Default export as namespace for convenient usage
- * @example import highlight from "./colors"
- * @example console.log(`Switched to ${highlight.branch("main")}`)
- */
-export default {
-	branch,
-	template,
-	file,
-	setting,
-	value,
-	commit,
-	pattern,
-	remote,
-}

package/src/utils/git-path.ts

@@ -1,6 +0,0 @@
-import { isAbsolute, resolve } from "path"
-
-export const resolveGitInternalPath = (
-	gitRoot: string,
-	gitPath: string,
-): string => (isAbsolute(gitPath) ? gitPath : resolve(gitRoot, gitPath))

package/src/utils/glob.test.ts

@@ -1,161 +0,0 @@
-import { test, expect, describe, beforeAll, afterAll } from "bun:test"
-import { mkdtemp, rm, mkdir, writeFile } from "node:fs/promises"
-import { tmpdir } from "node:os"
-import { join } from "node:path"
-import {
-	isGlobPattern,
-	matchesGlob,
-	expandGlobs,
-	dirToGlobPattern,
-	getTopLevelDir,
-} from "./glob"
-
-describe("isGlobPattern", () => {
-	test("returns true for patterns with asterisk", () => {
-		expect(isGlobPattern("*.ts")).toBe(true)
-		expect(isGlobPattern("**/*.ts")).toBe(true)
-		expect(isGlobPattern("plans/*")).toBe(true)
-		expect(isGlobPattern("plans/**")).toBe(true)
-	})
-
-	test("returns true for patterns with question mark", () => {
-		expect(isGlobPattern("file?.ts")).toBe(true)
-	})
-
-	test("returns true for patterns with character classes", () => {
-		expect(isGlobPattern("[abc].ts")).toBe(true)
-		expect(isGlobPattern("file[0-9].ts")).toBe(true)
-	})
-
-	test("returns false for plain file paths", () => {
-		expect(isGlobPattern("foo.ts")).toBe(false)
-		expect(isGlobPattern("plans/README.md")).toBe(false)
-		expect(isGlobPattern("src/utils/glob.ts")).toBe(false)
-	})
-})
-
-describe("matchesGlob", () => {
-	test("matches files with simple wildcard", () => {
-		expect(matchesGlob("*.ts", "file.ts")).toBe(true)
-		expect(matchesGlob("*.ts", "file.js")).toBe(false)
-	})
-
-	test("matches files in directories with **", () => {
-		expect(matchesGlob("plans/**", "plans/foo.md")).toBe(true)
-		expect(matchesGlob("plans/**", "plans/sub/bar.md")).toBe(true)
-		expect(matchesGlob("plans/**", "other/foo.md")).toBe(false)
-	})
-
-	test("matches nested directory patterns", () => {
-		expect(matchesGlob("src/**/*.ts", "src/utils/glob.ts")).toBe(true)
-		expect(matchesGlob("src/**/*.ts", "src/glob.ts")).toBe(true)
-		expect(matchesGlob("src/**/*.ts", "lib/glob.ts")).toBe(false)
-	})
-})
-
-describe("dirToGlobPattern", () => {
-	test("converts directory to glob pattern", () => {
-		expect(dirToGlobPattern("plans")).toBe("plans/**")
-		expect(dirToGlobPattern("src/components")).toBe("src/components/**")
-	})
-
-	test("removes trailing slash before adding glob", () => {
-		expect(dirToGlobPattern("plans/")).toBe("plans/**")
-		expect(dirToGlobPattern("plans//")).toBe("plans/**")
-	})
-})
-
-describe("getTopLevelDir", () => {
-	test("returns top-level directory from path", () => {
-		expect(getTopLevelDir("plans/foo.md")).toBe("plans")
-		expect(getTopLevelDir("plans/sub/bar.md")).toBe("plans")
-		expect(getTopLevelDir("src/utils/glob.ts")).toBe("src")
-	})
-
-	test("returns null for root-level files", () => {
-		expect(getTopLevelDir("README.md")).toBe(null)
-		expect(getTopLevelDir("file.ts")).toBe(null)
-	})
-})
-
-describe("expandGlobs", () => {
-	let tempDir: string
-
-	beforeAll(async () => {
-		// Create temp directory with test files
-		tempDir = await mkdtemp(join(tmpdir(), "glob-test-"))
-
-		// Create directory structure:
-		// tempDir/
-		//   plans/
-		//     foo.md
-		//     sub/
-		//       bar.md
-		//   other/
-		//     baz.md
-		//   root.txt
-		await mkdir(join(tempDir, "plans"))
-		await mkdir(join(tempDir, "plans", "sub"))
-		await mkdir(join(tempDir, "other"))
-		await mkdir(join(tempDir, ".agents", "foo"), { recursive: true })
-
-		await writeFile(join(tempDir, "plans", "foo.md"), "foo")
-		await writeFile(join(tempDir, "plans", "sub", "bar.md"), "bar")
-		await writeFile(join(tempDir, "other", "baz.md"), "baz")
-		await writeFile(join(tempDir, ".agents", "foo", "bar.whatever"), "dot")
-		await writeFile(join(tempDir, "root.txt"), "root")
-	})
-
-	afterAll(async () => {
-		await rm(tempDir, { recursive: true })
-	})
-
-	test("expands glob pattern to matching files", async () => {
-		const files = await expandGlobs(["plans/**"], tempDir)
-		expect(files.sort()).toEqual(["plans/foo.md", "plans/sub/bar.md"].sort())
-	})
-
-	test("preserves non-glob paths as-is", async () => {
-		const files = await expandGlobs(["root.txt"], tempDir)
-		expect(files).toEqual(["root.txt"])
-	})
-
-	test("combines glob and non-glob patterns", async () => {
-		const files = await expandGlobs(["plans/**", "root.txt"], tempDir)
-		expect(files.sort()).toEqual(
-			["plans/foo.md", "plans/sub/bar.md", "root.txt"].sort(),
-		)
-	})
-
-	test("deduplicates results", async () => {
-		const files = await expandGlobs(
-			["plans/**", "plans/foo.md", "plans/**"],
-			tempDir,
-		)
-		// Should not have duplicates
-		const uniqueFiles = [...new Set(files)]
-		expect(files.length).toBe(uniqueFiles.length)
-	})
-
-	test("returns non-glob paths even if they don't exist", async () => {
-		const files = await expandGlobs(["nonexistent.txt"], tempDir)
-		expect(files).toEqual(["nonexistent.txt"])
-	})
-
-	test("returns empty array for non-matching glob", async () => {
-		const files = await expandGlobs(["nonexistent/**"], tempDir)
-		expect(files).toEqual([])
-	})
-
-	test("handles multiple glob patterns", async () => {
-		const files = await expandGlobs(["plans/**", "other/**"], tempDir)
-		expect(files.sort()).toEqual(
-			["plans/foo.md", "plans/sub/bar.md", "other/baz.md"].sort(),
-		)
-	})
-
-	test("expands glob patterns inside dot-directories", async () => {
-		const files = await expandGlobs([".agents/**"], tempDir)
-		expect(files).toEqual([".agents/foo/bar.whatever"])
-	})
-})

package/src/utils/glob.ts

@@ -1,78 +0,0 @@
-/**
- * Utilities for glob pattern matching and expansion.
- * Uses Bun's built-in Glob API.
- */
-
-/**
- * Check if a string contains glob pattern characters.
- * This detects wildcards like *, **, ?, and character classes like [abc].
- */
-export function isGlobPattern(str: string): boolean {
-	return /[*?[\]]/.test(str)
-}
-
-/**
- * Check if a path matches a glob pattern.
- * Uses Bun.Glob for pattern matching.
- */
-export function matchesGlob(pattern: string, path: string): boolean {
-	const glob = new Bun.Glob(pattern)
-	return glob.match(path)
-}
-
-/**
- * Expand glob patterns to actual file paths.
- * Non-glob paths are returned as-is.
- *
- * @param patterns - Array of file paths or glob patterns
- * @param cwd - Working directory to resolve patterns against
- * @returns Array of expanded file paths (deduplicated)
- */
-export async function expandGlobs(
-	patterns: string[],
-	cwd: string,
-): Promise<string[]> {
-	const files = new Set<string>()
-
-	for (const pattern of patterns) {
-		if (isGlobPattern(pattern)) {
-			// Expand glob pattern
-			const glob = new Bun.Glob(pattern)
-			for await (const file of glob.scan({ cwd, dot: true })) {
-				files.add(file)
-			}
-		} else {
-			// Non-glob path, add as-is
-			files.add(pattern)
-		}
-	}
-
-	return Array.from(files)
-}
-
-/**
- * Convert a directory path to a glob pattern.
- * Appends /** to match all files recursively in the directory.
- *
- * @param dirPath - Directory path (e.g., "plans" or "plans/")
- * @returns Glob pattern (e.g., "plans/**")
- */
-export function dirToGlobPattern(dirPath: string): string {
-	// Remove trailing slash if present
-	const normalized = dirPath.replace(/\/+$/, "")
-	return `${normalized}/**`
-}
-
-/**
- * Extract the top-level directory from a file path.
- *
- * @param filePath - File path (e.g., "plans/foo/bar.md")
- * @returns Top-level directory name (e.g., "plans") or null if no directory
- */
-export function getTopLevelDir(filePath: string): string | null {
-	const parts = filePath.split("/")
-	if (parts.length > 1) {
-		return parts[0] || null
-	}
-	return null
-}

package/src/utils/paths.ts

@@ -1,51 +0,0 @@
-/**
- * Utilities for resolving agency configuration paths.
- */
-
-import { homedir } from "node:os"
-import { join } from "node:path"
-
-/**
- * Get the agency configuration directory.
- * Defaults to ~/.config/agency, can be overridden via AGENCY_CONFIG_DIR env var.
- * @param override - Optional override for the config directory (used in testing)
- */
-export function getAgencyConfigDir(override?: string): string {
-	return (
-		override ||
-		process.env.AGENCY_CONFIG_DIR ||
-		join(homedir(), ".config", "agency")
-	)
-}
-
-/**
- * Get the path to the agency config file (agency.json).
- * Can be overridden via AGENCY_CONFIG_PATH env var.
- * @param configDir - Optional config directory override
- */
-export function getAgencyConfigPath(configDir?: string): string {
-	return (
-		process.env.AGENCY_CONFIG_PATH ||
-		join(getAgencyConfigDir(configDir), "agency.json")
-	)
-}
-
-/**
- * Get the templates directory path.
- * @param configDir - Optional config directory override
- */
-export function getTemplatesDir(configDir?: string): string {
-	return join(getAgencyConfigDir(configDir), "templates")
-}
-
-/**
- * Get the path to a specific template directory.
- * @param templateName - Name of the template
- * @param configDir - Optional config directory override
- */
-export function getTemplateDir(
-	templateName: string,
-	configDir?: string,
-): string {
-	return join(getTemplatesDir(configDir), templateName)
-}