get-tbd 0.1.26 → 0.1.27

package/dist/bin.mjs

@@ -6710,6 +6710,18 @@ const IssueKind = enumType([
 	"chore"
 ]);
 /**
+* Maximum issue title length before detail belongs in the description body.
+*/
+const ISSUE_TITLE_MAX_LENGTH = 500;
+/**
+* Maximum issue body section length to keep issue files practical to review and sync.
+*/
+const ISSUE_BODY_MAX_LENGTH = 5e4;
+/**
+* Issue title text as persisted in issue files.
+*/
+const IssueTitle = stringType().min(1).max(ISSUE_TITLE_MAX_LENGTH);
+/**
 * Priority: 0 (highest/critical) to 4 (lowest).
 */
 const Priority = numberType().int().min(0).max(4);
@@ -6743,12 +6755,12 @@ const Dependency = objectType({
 */
 const IssueSchema = BaseEntity.extend({
 	type: literalType("is"),
-	title: stringType().min(1).max(500),
+	title: IssueTitle,
 	kind: IssueKind.default("task"),
 	status: IssueStatus.default("open"),
 	priority: Priority.default(2),
-	description: stringType().max(5e4).nullable().optional(),
-	notes: stringType().max(5e4).nullable().optional(),
+	description: stringType().max(ISSUE_BODY_MAX_LENGTH).nullable().optional(),
+	notes: stringType().max(ISSUE_BODY_MAX_LENGTH).nullable().optional(),
 	spec_path: stringType().nullable().optional(),
 	assignee: stringType().nullable().optional(),
 	labels: arrayType(stringType()).default([]),
@@ -14045,7 +14057,7 @@ function serializeIssue(issue) {
 * Package version, derived from git at build time.
 * Format: X.Y.Z for releases, X.Y.Z-dev.N.hash for dev builds.
 */
-const VERSION$1 = "0.1.26";
+const VERSION$1 = "0.1.27";
 
 //#endregion
 //#region src/cli/lib/version.ts
@@ -99726,6 +99738,29 @@ function formatDebugId(internalId, mapping, prefix = "tbd") {
 	return `${formatDisplayId(internalId, mapping, prefix)} (${internalId})`;
 }
 
+//#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
 /**
@@ -99737,6 +99772,10 @@ function formatDebugId(internalId, mapping, prefix = "tbd") {
 * 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) {
@@ -99754,7 +99793,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.
@@ -99762,7 +99802,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 {
@@ -99771,10 +99812,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 {
@@ -99782,25 +99822,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/utils/lockfile.ts
@@ -100516,6 +100569,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
 /**
@@ -100527,6 +100596,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;
@@ -100543,7 +100616,7 @@ var CreateHandler = class extends BaseCommand {
 			throw new ValidationError(getPathErrorMessage(error));
 		}
 		if (this.checkDryRun("Would create issue", {
-			title,
+			title: validatedTitle,
 			kind,
 			priority,
 			spec: specPath,
@@ -100575,7 +100648,7 @@ var CreateHandler = class extends BaseCommand {
 				type: "is",
 				id,
 				version: 1,
-				title,
+				title: validatedTitle,
 				kind,
 				status: "open",
 				priority,
@@ -100607,9 +100680,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) {
@@ -101425,7 +101498,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;
@@ -101456,10 +101532,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}`);
@@ -104668,6 +104744,7 @@ var DoctorHandler = class extends BaseCommand {
 	cwd = "";
 	config = null;
 	issues = [];
+	invalidIssueFiles = [];
 	async run(options) {
 		const tbdRoot = await requireInit();
 		this.cwd = tbdRoot;
@@ -104676,7 +104753,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();
@@ -104689,14 +104770,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;
@@ -105039,8 +105124,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) {
@@ -105090,7 +105179,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"
 		};