get-tbd 0.1.26 → 0.1.27

package/dist/cli.mjs

@@ -1,9 +1,10 @@
-import { _ as IssueKind, n as AtticEntrySchema, t as ATTIC_ENTRY_FIELD_ORDER, y as IssueStatus } from "./schemas-BQYmDnkv.mjs";
-import { a as insertAfterFrontmatter, c as noopLogger, i as serializeIssue, n as parseIssue, o as parseMarkdown, r as parseMarkdownWithFrontmatter, s as stripFrontmatter, t as VERSION$1 } from "./src-DQcOQnFp.mjs";
+import { S as IssueTitle, b as IssueSchema, g as ISSUE_TITLE_MAX_LENGTH, n as AtticEntrySchema, t as ATTIC_ENTRY_FIELD_ORDER, x as IssueStatus, y as IssueKind } from "./schemas-C8mOQykE.mjs";
+import { a as insertAfterFrontmatter, c as noopLogger, i as serializeIssue, n as parseIssue, o as parseMarkdown, r as parseMarkdownWithFrontmatter, s as stripFrontmatter, t as VERSION$1 } from "./src-BIE27KDA.mjs";
 import { a as parseYamlWithConflictDetection, d as PAGINATION_LINE_THRESHOLD, f as PARENT_CONTEXT_MAX_LINES, l as comparisonChain, n as detectDuplicateYamlKeys, o as sortKeys, s as stringifyYaml, u as ordering } from "./yaml-utils-BPy991by.mjs";
-import { A as isValidWorkspaceName, C as TBD_SHORTCUTS_STANDARD, D as WORKTREE_DIR, E as WORKSPACES_DIR, M as resolveDataSyncDir, O as WORKTREE_DIR_NAME, S as TBD_GUIDELINES_DIR, T as TBD_TEMPLATES_DIR, _ as DEFAULT_SHORTCUT_PATHS, a as isInitialized, b as TBD_DIR, c as readConfigWithMigration, d as writeConfig, g as DEFAULT_GUIDELINES_PATHS, h as DATA_SYNC_DIR_NAME, i as initConfig, j as resolveAtticDir, k as getWorkspaceDir, l as readLocalState, m as DATA_SYNC_DIR, n as findTbdRoot, o as markWelcomeSeen, p as CHARS_PER_TOKEN, r as hasSeenWelcome, s as readConfig, u as updateLocalState, v as DEFAULT_TEMPLATE_PATHS, w as TBD_SHORTCUTS_SYSTEM, x as TBD_DOCS_DIR, y as SYNC_BRANCH } from "./config-b20Kf5pW.mjs";
-import { _ as formatDisplayId, a as hasShortId, b as normalizeIssueId, c as parseIdMappingFromYaml, d as resolveToInternalId, f as saveIdMapping, g as formatDebugId, h as extractUlidFromInternalId, i as generateUniqueShortId, l as reconcileMappings, m as extractShortId, o as loadIdMapping, p as extractPrefix, s as mergeIdMappings, t as addIdMapping, u as resolveIdMappingConflicts, v as generateInternalId, x as validateIssueId, y as makeInternalId } from "./id-mapping-BtBwq5nG.mjs";
+import { A as isValidWorkspaceName, C as TBD_SHORTCUTS_STANDARD, D as WORKTREE_DIR, E as WORKSPACES_DIR, M as resolveDataSyncDir, O as WORKTREE_DIR_NAME, S as TBD_GUIDELINES_DIR, T as TBD_TEMPLATES_DIR, _ as DEFAULT_SHORTCUT_PATHS, a as isInitialized, b as TBD_DIR, c as readConfigWithMigration, d as writeConfig, g as DEFAULT_GUIDELINES_PATHS, h as DATA_SYNC_DIR_NAME, i as initConfig, j as resolveAtticDir, k as getWorkspaceDir, l as readLocalState, m as DATA_SYNC_DIR, n as findTbdRoot, o as markWelcomeSeen, p as CHARS_PER_TOKEN, r as hasSeenWelcome, s as readConfig, u as updateLocalState, v as DEFAULT_TEMPLATE_PATHS, w as TBD_SHORTCUTS_SYSTEM, x as TBD_DOCS_DIR, y as SYNC_BRANCH } from "./config-B38rbI9u.mjs";
+import { _ as formatDisplayId, a as hasShortId, b as normalizeIssueId, c as parseIdMappingFromYaml, d as resolveToInternalId, f as saveIdMapping, g as formatDebugId, h as extractUlidFromInternalId, i as generateUniqueShortId, l as reconcileMappings, m as extractShortId, o as loadIdMapping, p as extractPrefix, s as mergeIdMappings, t as addIdMapping, u as resolveIdMappingConflicts, v as generateInternalId, x as validateIssueId, y as makeInternalId } from "./id-mapping-CqrrLgeX.mjs";
 import { createRequire } from "node:module";
+import { ZodError } from "zod";
 import matter from "gray-matter";
 import { parse } from "yaml";
 import { Command } from "commander";
@@ -1754,7 +1755,7 @@ async function migrateDataToWorktree(baseDir, removeSource = false) {
 		for (const file of issueFiles) await cp(join(wrongIssuesPath, file), join(correctIssuesPath, file));
 		for (const file of mappingFiles) if (file === "ids.yml") {
 			const { readFile } = await import("node:fs/promises");
-			const { loadIdMapping, mergeIdMappings, saveIdMapping, resolveIdMappingConflicts } = await import("./id-mapping-BA_xn516.mjs");
+			const { loadIdMapping, mergeIdMappings, saveIdMapping, resolveIdMappingConflicts } = await import("./id-mapping-Ctfl_nc1.mjs");
 			const sourceMapping = resolveIdMappingConflicts(await readFile(join(wrongMappingsPath, file), "utf-8"));
 			let targetMapping;
 			try {
@@ -1883,6 +1884,29 @@ const initCommand = new Command("init").description("Initialize tbd in a git rep
 	await new InitHandler(command).run(options);
 });
 
+//#endregion
+//#region src/utils/zod-error-utils.ts
+/**
+* Helpers for rendering Zod errors without relying on object inspection.
+*/
+/**
+* Format a ZodError as concise path-qualified messages for CLI output.
+*/
+function formatZodError(error) {
+	const messages = error.issues.map((issue) => {
+		return `${issue.path.length > 0 ? issue.path.join(".") : "<root>"}: ${issue.message}`;
+	});
+	return messages.length > 0 ? messages.join("; ") : error.message;
+}
+/**
+* Format unknown thrown values as safe strings for warnings and diagnostics.
+*/
+function formatUnknownError(error) {
+	if (error instanceof ZodError) return formatZodError(error);
+	if (error instanceof Error) return error.message;
+	return String(error);
+}
+
 //#endregion
 //#region src/file/storage.ts
 /**
@@ -1894,6 +1918,10 @@ const initCommand = new Command("init").description("Initialize tbd in a git rep
 * See: tbd-design.md §3.2 Storage Layer
 */
 /**
+* Maximum issue files read concurrently to avoid exhausting file descriptors in large repos.
+*/
+const ISSUE_READ_BATCH_SIZE = 200;
+/**
 * Get the path to an issue file.
 */
 function getIssuePath(baseDir, id) {
@@ -1911,7 +1939,8 @@ async function readIssue(baseDir, id) {
 * Uses atomic write to prevent corruption.
 */
 async function writeIssue(baseDir, issue) {
-	await writeFile(getIssuePath(baseDir, issue.id), serializeIssue(issue));
+	const validIssue = IssueSchema.parse(issue);
+	await writeFile(getIssuePath(baseDir, validIssue.id), serializeIssue(validIssue));
 }
 /**
 * List all issues in the worktree.
@@ -1919,7 +1948,8 @@ async function writeIssue(baseDir, issue) {
 *
 * Uses parallel file reading for better performance with many issues.
 */
-async function listIssues(baseDir) {
+async function listIssues(baseDir, options = {}) {
+	const warnOnInvalid = options.warnOnInvalid ?? true;
 	const issuesDir = join(baseDir, "issues");
 	let files;
 	try {
@@ -1928,10 +1958,9 @@ async function listIssues(baseDir) {
 		return [];
 	}
 	const mdFiles = files.filter((f) => f.endsWith(".md"));
-	const BATCH_SIZE = 200;
 	const issues = [];
-	for (let i = 0; i < mdFiles.length; i += BATCH_SIZE) {
-		const batch = mdFiles.slice(i, i + BATCH_SIZE);
+	for (let i = 0; i < mdFiles.length; i += ISSUE_READ_BATCH_SIZE) {
+		const batch = mdFiles.slice(i, i + ISSUE_READ_BATCH_SIZE);
 		const fileContents = await Promise.all(batch.map(async (file) => {
 			const filePath = join(issuesDir, file);
 			try {
@@ -1939,25 +1968,38 @@ async function listIssues(baseDir) {
 					file,
 					content: await readFile(filePath, "utf-8")
 				};
-			} catch {
+			} catch (error) {
 				return {
 					file,
-					content: null
+					error: formatUnknownError(error)
 				};
 			}
 		}));
-		for (const { file, content } of fileContents) {
-			if (content === null) continue;
+		for (const result of fileContents) {
+			if ("error" in result) {
+				reportInvalidIssueFile({
+					file: result.file,
+					reason: `failed to read file: ${result.error}`
+				}, warnOnInvalid, options.onInvalidIssue);
+				continue;
+			}
 			try {
-				const issue = parseIssue(content);
+				const issue = parseIssue(result.content);
 				issues.push(issue);
 			} catch (error) {
-				console.warn(`Skipping invalid issue file: ${file}`, error);
+				reportInvalidIssueFile({
+					file: result.file,
+					reason: formatUnknownError(error)
+				}, warnOnInvalid, options.onInvalidIssue);
 			}
 		}
 	}
 	return issues;
 }
+function reportInvalidIssueFile(invalidIssue, warnOnInvalid, onInvalidIssue) {
+	onInvalidIssue?.(invalidIssue);
+	if (warnOnInvalid) console.warn(`Skipping invalid issue file: ${invalidIssue.file}: ${invalidIssue.reason}`);
+}
 
 //#endregion
 //#region src/lib/priority.ts
@@ -2150,6 +2192,22 @@ async function resolveAndValidatePath(inputPath, projectRoot, cwd) {
 	return resolved;
 }
 
+//#endregion
+//#region src/cli/lib/issue-input-validation.ts
+/**
+* CLI validation helpers for user-provided issue fields.
+*/
+/**
+* Validate a CLI-provided issue title with actionable user-facing errors.
+*/
+function validateIssueTitle(title, options) {
+	if (options.rejectBlank ? title.trim().length === 0 : title.length === 0) throw new ValidationError(options.emptyMessage);
+	if (title.length > ISSUE_TITLE_MAX_LENGTH) throw new ValidationError(`Title is too long (${title.length} chars, max ${ISSUE_TITLE_MAX_LENGTH}). Move detail into the description body.`);
+	const result = IssueTitle.safeParse(title);
+	if (!result.success) throw new ValidationError(`Invalid title: ${formatZodError(result.error)}`);
+	return title;
+}
+
 //#endregion
 //#region src/cli/commands/create.ts
 /**
@@ -2161,6 +2219,10 @@ var CreateHandler = class extends BaseCommand {
 	async run(title, options) {
 		const tbdRoot = await requireInit();
 		if (!title && !options.fromFile) throw new ValidationError("Title is required. Use: tbd create \"Issue title\"");
+		const validatedTitle = title === void 0 ? void 0 : validateIssueTitle(title, {
+			emptyMessage: "Title is required. Use: tbd create \"Issue title\"",
+			rejectBlank: true
+		});
 		const kind = this.parseKind(options.type ?? "task");
 		const priority = this.validatePriority(options.priority ?? "2");
 		let description = options.description;
@@ -2177,7 +2239,7 @@ var CreateHandler = class extends BaseCommand {
 			throw new ValidationError(getPathErrorMessage(error));
 		}
 		if (this.checkDryRun("Would create issue", {
-			title,
+			title: validatedTitle,
 			kind,
 			priority,
 			spec: specPath,
@@ -2209,7 +2271,7 @@ var CreateHandler = class extends BaseCommand {
 				type: "is",
 				id,
 				version: 1,
-				title,
+				title: validatedTitle,
 				kind,
 				status: "open",
 				priority,
@@ -2241,9 +2303,9 @@ var CreateHandler = class extends BaseCommand {
 		this.output.data({
 			id: displayId,
 			internalId: id,
-			title
+			title: validatedTitle
 		}, () => {
-			this.output.success(`Created ${displayId}: ${title}`);
+			this.output.success(`Created ${displayId}: ${validatedTitle}`);
 		});
 	}
 	parseKind(value) {
@@ -3059,7 +3121,10 @@ var UpdateHandler = class extends BaseCommand {
 			}
 			try {
 				const { frontmatter, description, notes } = parseMarkdownWithFrontmatter(content);
-				if (typeof frontmatter.title === "string") updates.title = frontmatter.title;
+				if (typeof frontmatter.title === "string") updates.title = validateIssueTitle(frontmatter.title, {
+					emptyMessage: "Title cannot be empty",
+					rejectBlank: true
+				});
 				if (typeof frontmatter.status === "string") {
 					const result = IssueStatus.safeParse(frontmatter.status);
 					if (result.success) updates.status = result.data;
@@ -3090,10 +3155,10 @@ var UpdateHandler = class extends BaseCommand {
 			}
 			return updates;
 		}
-		if (options.title !== void 0) {
-			if (!options.title.trim()) throw new ValidationError("Title cannot be empty");
-			updates.title = options.title;
-		}
+		if (options.title !== void 0) updates.title = validateIssueTitle(options.title, {
+			emptyMessage: "Title cannot be empty",
+			rejectBlank: true
+		});
 		if (options.status) {
 			const result = IssueStatus.safeParse(options.status);
 			if (!result.success) throw new ValidationError(`Invalid status: ${options.status}`);
@@ -6302,6 +6367,7 @@ var DoctorHandler = class extends BaseCommand {
 	cwd = "";
 	config = null;
 	issues = [];
+	invalidIssueFiles = [];
 	async run(options) {
 		const tbdRoot = await requireInit();
 		this.cwd = tbdRoot;
@@ -6310,7 +6376,11 @@ var DoctorHandler = class extends BaseCommand {
 			this.config = await readConfig(this.cwd);
 		} catch {}
 		try {
-			this.issues = await listIssues(this.dataSyncDir);
+			this.invalidIssueFiles = [];
+			this.issues = await listIssues(this.dataSyncDir, {
+				warnOnInvalid: false,
+				onInvalidIssue: (invalidIssue) => this.invalidIssueFiles.push(invalidIssue)
+			});
 		} catch {}
 		const statusInfo = await this.gatherStatusInfo();
 		const statsInfo = await this.gatherStatsInfo();
@@ -6323,14 +6393,18 @@ var DoctorHandler = class extends BaseCommand {
 		healthChecks.push(await this.checkIdMappingConflicts(options.fix));
 		healthChecks.push(await this.checkIdMappingDuplicates(options.fix));
 		healthChecks.push(await this.checkTempFiles(options.fix));
-		healthChecks.push(this.checkIssueValidity(this.issues));
+		healthChecks.push(this.checkIssueValidity(this.issues, this.invalidIssueFiles));
 		healthChecks.push(await this.checkWorktree(options.fix));
 		const dataLocationResult = await this.checkDataLocation(options.fix);
 		healthChecks.push(dataLocationResult);
 		if (dataLocationResult.status === "ok" && dataLocationResult.message?.includes("migrated")) {
 			this.dataSyncDir = await resolveDataSyncDir(this.cwd);
 			try {
-				this.issues = await listIssues(this.dataSyncDir);
+				this.invalidIssueFiles = [];
+				this.issues = await listIssues(this.dataSyncDir, {
+					warnOnInvalid: false,
+					onInvalidIssue: (invalidIssue) => this.invalidIssueFiles.push(invalidIssue)
+				});
 			} catch {}
 		}
 		const parsedMaxHistory = options.maxHistory ? parseInt(options.maxHistory, 10) : 50;
@@ -6561,7 +6635,7 @@ var DoctorHandler = class extends BaseCommand {
 			status: "ok"
 		};
 		if (fix && !this.checkDryRun("Resolve merge conflicts in ids.yml")) try {
-			const { resolveIdMappingConflicts, saveIdMapping } = await import("./id-mapping-BA_xn516.mjs");
+			const { resolveIdMappingConflicts, saveIdMapping } = await import("./id-mapping-Ctfl_nc1.mjs");
 			const resolved = resolveIdMappingConflicts(content);
 			await saveIdMapping(this.dataSyncDir, resolved);
 			return {
@@ -6610,7 +6684,7 @@ var DoctorHandler = class extends BaseCommand {
 			status: "ok"
 		};
 		if (fix && !this.checkDryRun("Fix duplicate ID mapping keys")) try {
-			const { loadIdMapping, saveIdMapping } = await import("./id-mapping-BA_xn516.mjs");
+			const { loadIdMapping, saveIdMapping } = await import("./id-mapping-Ctfl_nc1.mjs");
 			const mapping = await loadIdMapping(this.dataSyncDir);
 			await saveIdMapping(this.dataSyncDir, mapping);
 			return {
@@ -6673,8 +6747,12 @@ var DoctorHandler = class extends BaseCommand {
 			suggestion: "Run: tbd doctor --fix"
 		};
 	}
-	checkIssueValidity(issues) {
+	checkIssueValidity(issues, invalidIssueFiles) {
 		const invalid = [];
+		for (const invalidIssueFile of invalidIssueFiles) invalid.push({
+			id: invalidIssueFile.file,
+			reason: invalidIssueFile.reason
+		});
 		for (const issue of issues) {
 			const issueId = issue.id ?? "unknown";
 			if (!issue.id) {
@@ -6724,7 +6802,8 @@ var DoctorHandler = class extends BaseCommand {
 		return {
 			name: "Issue validity",
 			status: "error",
-			message: `${invalid.length} invalid issue(s)`,
+			message: `${invalid.length} invalid issue file(s)`,
+			path: join(CONFIG_DIR, "issues"),
 			details: invalid.map((i) => `${i.id}: ${i.reason}`),
 			suggestion: "Manually fix or delete invalid issue files"
 		};
@@ -6744,7 +6823,7 @@ var DoctorHandler = class extends BaseCommand {
 			name: "ID mapping coverage",
 			status: "ok"
 		};
-		const { loadIdMapping, saveIdMapping, reconcileMappings } = await import("./id-mapping-BA_xn516.mjs");
+		const { loadIdMapping, saveIdMapping, reconcileMappings } = await import("./id-mapping-Ctfl_nc1.mjs");
 		const mapping = await loadIdMapping(this.dataSyncDir);
 		const missingIds = [];
 		for (const issue of this.issues) {
@@ -6756,10 +6835,10 @@ var DoctorHandler = class extends BaseCommand {
 			status: "ok"
 		};
 		if (fix && !this.checkDryRun("Create missing ID mappings")) {
-			const { parseIdMappingFromYaml, mergeIdMappings } = await import("./id-mapping-BA_xn516.mjs");
+			const { parseIdMappingFromYaml, mergeIdMappings } = await import("./id-mapping-Ctfl_nc1.mjs");
 			let historicalMapping;
 			try {
-				const syncBranch = (await import("./config-BZte2m3w.mjs").then((m) => m.readConfig(this.cwd))).sync.branch;
+				const syncBranch = (await import("./config-C0ITTrtc.mjs").then((m) => m.readConfig(this.cwd))).sync.branch;
 				const logArgs = ["log", "--format=%H"];
 				if (maxHistory > 0) logArgs.push(`-${maxHistory}`);
 				logArgs.push(syncBranch, "--", `${DATA_SYNC_DIR}/mappings/ids.yml`);