refactor: make task fields optional and update parsing logic to handle undefined values

Author: aitorllj93

src/parse.test.ts

@@ -174,10 +174,6 @@ export function parse<TScenario extends string = string>(
 		}
 	}
 
-	if (!type) {
-		throw new Error("Invalid task format: missing task type");
-	}
-
 	// Extract additional focuses that appear after metadata but before dates
 	// They can appear anywhere in the remaining text
 	// Convert remaining to array once and reuse it
@@ -250,12 +246,14 @@ export function parse<TScenario extends string = string>(
 	}
 
 	// Extract priority
-	let priority: Priority = "normal";
+	let priority: Priority | undefined;
 	if (remaining.includes("🔺")) priority = "maximum";
 	else if (remaining.includes("⏫")) priority = "high";
 	else if (remaining.includes("🔼")) priority = "medium";
 	else if (remaining.includes("🔽")) priority = "low";
 	else if (remaining.includes("⏬")) priority = "minimum";
+	// Only set 'normal' as default when validation is enabled and other fields exist
+	// Otherwise leave it undefined to match the markdown standard
 
 	// Extract recurrence
 	let recurrence: string | undefined;
@@ -322,17 +320,14 @@ export function parse<TScenario extends string = string>(
 	const task: Partial<Task> & {
 		state: TaskState;
 		content: string;
-		type: TaskType;
-		tags: string[];
-		priority: Priority;
 	} = {
 		state,
 		content,
-		type,
-		tags,
-		priority,
+		tags, // Always include tags (as array, even if empty)
 	};
 
+	if (type) task.type = type;
+	if (priority) task.priority = priority;
 	if (id) task.id = id;
 	if (focuses.length > 0) task.focuses = focuses;
 	if (scenarios.length > 0) task.scenarios = scenarios;

src/parse.ts

@@ -9,7 +9,6 @@ describe("validateTask", () => {
 			content: "Valid task",
 			type: "quick",
 			tags: ["Tasks/Quick"],
-			priority: "normal",
 		};
 		expect(validateTask(task)).toBe(true);
 	});
@@ -21,7 +20,6 @@ describe("validateTask", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			duration: "invalid",
-			priority: "normal",
 		};
 		expect(validateTask(task)).toBe(false);
 	});
@@ -33,7 +31,6 @@ describe("validateTask", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			dueAt: "15-01-2025",
-			priority: "normal",
 		};
 		expect(validateTask(task)).toBe(false);
 	});
@@ -45,7 +42,6 @@ describe("validateTask", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			time: { start: "9:00", end: "10:30" },
-			priority: "normal",
 		};
 		expect(validateTask(task)).toBe(false);
 	});
@@ -56,7 +52,6 @@ describe("validateTask", () => {
 			content: "",
 			type: "quick",
 			tags: ["Tasks/Quick"],
-			priority: "normal",
 		};
 		expect(validateTask(task)).toBe(false);
 	});
@@ -98,7 +93,6 @@ describe("isValidTask", () => {
 			content: "Valid task",
 			type: "quick",
 			tags: ["Tasks/Quick"],
-			priority: "normal",
 		};
 
 		if (isValidTask(task)) {
@@ -117,7 +111,6 @@ describe("isValidTask", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			duration: "invalid",
-			priority: "normal",
 		};
 		expect(isValidTask(task)).toBe(false);
 	});
@@ -138,7 +131,6 @@ describe("validateTaskWithErrors", () => {
 			content: "Valid task",
 			type: "quick",
 			tags: ["Tasks/Quick"],
-			priority: "normal",
 		};
 		const result = validateTaskWithErrors(task);
 		expect(result.success).toBe(true);
@@ -154,7 +146,6 @@ describe("validateTaskWithErrors", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			duration: "invalid",
-			priority: "normal",
 		};
 		const result = validateTaskWithErrors(task);
 		expect(result.success).toBe(false);
@@ -173,7 +164,6 @@ describe("validateTaskWithErrors", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			dueAt: "15-01-2025",
-			priority: "normal",
 		};
 		const result = validateTaskWithErrors(task);
 		expect(result.success).toBe(false);
@@ -192,7 +182,6 @@ describe("validateTaskWithErrors", () => {
 			type: "quick",
 			tags: ["Tasks/Quick"],
 			time: { start: "9:00", end: "10:30" },
-			priority: "normal",
 		};
 		const result = validateTaskWithErrors(task);
 		expect(result.success).toBe(false);
@@ -207,7 +196,6 @@ describe("validateTaskWithErrors", () => {
 			content: "",
 			type: "quick",
 			tags: ["Tasks/Quick"],
-			priority: "normal",
 		};
 		const result = validateTaskWithErrors(task);
 		expect(result.success).toBe(false);

src/schema.test.ts

@@ -80,8 +80,8 @@ export const HooksSchema = z.object({
 export const TaskSchema = z.object({
 	state: TaskStateSchema,
 	content: z.string().min(1),
-	type: TaskTypeSchema,
-	tags: z.array(z.string()),
+	type: TaskTypeSchema.optional(),
+	tags: z.array(z.string()).optional(),
 	id: z.string().optional(),
 	focuses: z.array(FocusSchema).optional(),
 	scenarios: z.array(z.string()).optional(),
@@ -96,7 +96,7 @@ export const TaskSchema = z.object({
 	completedAt: DateFieldSchema,
 	cancelledAt: DateFieldSchema,
 	time: TimeSchema.optional(),
-	priority: PrioritySchema.default("normal"),
+	priority: PrioritySchema.optional(),
 	recurrence: z.string().optional(),
 	dependencies: z.array(z.string()).optional(),
 	hooks: HooksSchema.optional(),