@md2do/core 0.2.3 → 0.4.0

package/dist/index.mjs

@@ -10,7 +10,7 @@ var ASSIGNEE = /@([\w-]+)/;
 var PRIORITY_URGENT = /!!!/;
 var PRIORITY_HIGH = /!!/;
 var PRIORITY_NORMAL = /(?<!!)!(?!!)/;
-var DUE_DATE_ABSOLUTE = /\[due:\s*(\d{4}-\d{2}-\d{2})\s*\]/i;
+var DUE_DATE_ABSOLUTE = /\[due:\s*(\d{4}-\d{2}-\d{2})(?:\s+(\d{1,2}:\d{2}))?\s*\]/i;
 var DUE_DATE_RELATIVE = /\[due:\s*(tomorrow|today|next\s+week|next\s+month)\]/i;
 var DUE_DATE_SHORT = /\[due:\s*(\d{1,2}\/\d{1,2}(?:\/\d{2,4})?)\]/i;
 var TAG = /#([\w-]+)/g;
@@ -43,17 +43,38 @@ import {
   addDays,
   addWeeks,
   addMonths,
-  startOfWeek
+  startOfWeek,
+  setHours,
+  setMinutes
 } from "date-fns";
-function parseAbsoluteDate(dateStr) {
+function parseTime(timeStr) {
+  const match = timeStr.match(/^(\d{1,2}):(\d{2})$/);
+  if (!match || !match[1] || !match[2]) return null;
+  const hours = parseInt(match[1], 10);
+  const minutes = parseInt(match[2], 10);
+  if (hours < 0 || hours > 23 || minutes < 0 || minutes > 59) {
+    return null;
+  }
+  return { hours, minutes };
+}
+function parseAbsoluteDate(dateStr, timeStr) {
   const referenceDate = /* @__PURE__ */ new Date();
   let date = parse(dateStr, "yyyy-MM-dd", referenceDate);
-  if (isValid(date)) return date;
-  date = parse(dateStr, "M/d/yy", referenceDate);
-  if (isValid(date)) return date;
-  date = parse(dateStr, "M/d/yyyy", referenceDate);
-  if (isValid(date)) return date;
-  return null;
+  if (!isValid(date)) {
+    date = parse(dateStr, "M/d/yy", referenceDate);
+  }
+  if (!isValid(date)) {
+    date = parse(dateStr, "M/d/yyyy", referenceDate);
+  }
+  if (!isValid(date)) return null;
+  if (timeStr) {
+    const time = parseTime(timeStr);
+    if (time) {
+      date = setHours(date, time.hours);
+      date = setMinutes(date, time.minutes);
+    }
+  }
+  return date;
 }
 function resolveRelativeDate(relative, baseDate) {
   const normalized = relative.toLowerCase().trim();
@@ -145,12 +166,29 @@ function extractCompletedDate(text) {
 function extractDueDate(text, context) {
   const absoluteMatch = text.match(PATTERNS.DUE_DATE_ABSOLUTE);
   if (absoluteMatch?.[1]) {
-    const date = parseAbsoluteDate(absoluteMatch[1]);
+    const dateStr = absoluteMatch[1];
+    let timeStr = absoluteMatch[2];
+    if (!timeStr && context.defaultDueTime) {
+      if (context.defaultDueTime === "start" && context.workdayStartTime) {
+        timeStr = context.workdayStartTime;
+      } else if (context.defaultDueTime === "end" && context.workdayEndTime) {
+        timeStr = context.workdayEndTime;
+      }
+    }
+    const date = parseAbsoluteDate(dateStr, timeStr);
     return { date: date ?? void 0 };
   }
   const shortMatch = text.match(PATTERNS.DUE_DATE_SHORT);
   if (shortMatch?.[1]) {
-    const date = parseAbsoluteDate(shortMatch[1]);
+    let timeStr;
+    if (context.defaultDueTime) {
+      if (context.defaultDueTime === "start" && context.workdayStartTime) {
+        timeStr = context.workdayStartTime;
+      } else if (context.defaultDueTime === "end" && context.workdayEndTime) {
+        timeStr = context.workdayEndTime;
+      }
+    }
+    const date = parseAbsoluteDate(shortMatch[1], timeStr);
     return { date: date ?? void 0 };
   }
   const relativeMatch = text.match(PATTERNS.DUE_DATE_RELATIVE);
@@ -159,11 +197,15 @@ function extractDueDate(text, context) {
       return {
         date: void 0,
         warning: {
+          severity: "warning",
+          source: "md2do",
+          ruleId: "relative-date-no-context",
           file: "",
           // Will be filled in by caller
           line: 0,
           // Will be filled in by caller
           text: text.trim(),
+          message: "Relative due date without context date from heading. Add a heading with a date above this task.",
           reason: "Relative due date without context date from heading. Add a heading with a date above this task."
         }
       };
@@ -180,36 +222,52 @@ function parseTask(line, lineNumber, file, context) {
   const warnings = [];
   if (/^\s*[*+]\s+\[[ xX]\]/.test(line)) {
     warnings.push({
+      severity: "warning",
+      source: "md2do",
+      ruleId: "unsupported-bullet",
       file,
       line: lineNumber,
       text: line.trim(),
+      message: "Unsupported bullet marker (* or +). Use dash (-) for task lists.",
       reason: "Unsupported bullet marker (* or +). Use dash (-) for task lists."
     });
     return { task: null, warnings };
   }
   if (/^\s*-\s+\[[xX]\s+\]/.test(line) || /^\s*-\s+\[\s+[xX]\]/.test(line)) {
     warnings.push({
+      severity: "warning",
+      source: "md2do",
+      ruleId: "malformed-checkbox",
       file,
       line: lineNumber,
       text: line.trim(),
+      message: "Malformed checkbox with extra spaces. Use [x] or [ ] without extra spaces.",
       reason: "Malformed checkbox with extra spaces. Use [x] or [ ] without extra spaces."
     });
     return { task: null, warnings };
   }
   if (/^\s*-\s+\[[ xX]\][^\s]/.test(line)) {
     warnings.push({
+      severity: "warning",
+      source: "md2do",
+      ruleId: "missing-space-after",
       file,
       line: lineNumber,
       text: line.trim(),
+      message: 'Missing space after checkbox. Use "- [x] Task" format.',
       reason: 'Missing space after checkbox. Use "- [x] Task" format.'
     });
     return { task: null, warnings };
   }
   if (/^\s*-\[[ xX]\]/.test(line)) {
     warnings.push({
+      severity: "warning",
+      source: "md2do",
+      ruleId: "missing-space-before",
       file,
       line: lineNumber,
       text: line.trim(),
+      message: 'Missing space before checkbox. Use "- [x] Task" format.',
       reason: 'Missing space before checkbox. Use "- [x] Task" format.'
     });
     return { task: null, warnings };
@@ -235,17 +293,25 @@ function parseTask(line, lineNumber, file, context) {
   }
   if (!completed && !dueDateResult.date && !context.currentDate) {
     warnings.push({
+      severity: "info",
+      source: "md2do",
+      ruleId: "missing-due-date",
       file,
       line: lineNumber,
       text: fullText.trim(),
+      message: "Task has no due date. Add [due: YYYY-MM-DD] or place under a heading with a date.",
       reason: "Task has no due date. Add [due: YYYY-MM-DD] or place under a heading with a date."
     });
   }
   if (completed && !completedDate) {
     warnings.push({
+      severity: "info",
+      source: "md2do",
+      ruleId: "missing-completed-date",
       file,
       line: lineNumber,
       text: fullText.trim(),
+      message: "Completed task missing completion date. Add [completed: YYYY-MM-DD].",
       reason: "Completed task missing completion date. Add [completed: YYYY-MM-DD]."
     });
   }
@@ -289,9 +355,10 @@ var MarkdownScanner = class {
    *
    * @param filePath - Relative file path (for context extraction)
    * @param content - File content as string
+   * @param options - Optional scanner options including workday config
    * @returns Object containing tasks and warnings
    */
-  scanFile(filePath, content) {
+  scanFile(filePath, content, options) {
     const tasks = [];
     const warnings = [];
     const todoistIds = /* @__PURE__ */ new Map();
@@ -300,6 +367,15 @@ var MarkdownScanner = class {
     if (project !== void 0) context.project = project;
     const person = extractPersonFromFilename(filePath);
     if (person !== void 0) context.person = person;
+    if (options?.workdayStartTime) {
+      context.workdayStartTime = options.workdayStartTime;
+    }
+    if (options?.workdayEndTime) {
+      context.workdayEndTime = options.workdayEndTime;
+    }
+    if (options?.defaultDueTime) {
+      context.defaultDueTime = options.defaultDueTime;
+    }
     const lines = content.split("\n");
     for (let i = 0; i < lines.length; i++) {
       const line = lines[i];
@@ -318,9 +394,13 @@ var MarkdownScanner = class {
           const existing = todoistIds.get(result.task.todoistId);
           if (existing) {
             warnings.push({
+              severity: "error",
+              source: "md2do",
+              ruleId: "duplicate-todoist-id",
               file: filePath,
               line: lineNumber,
               text: result.task.text,
+              message: `Duplicate Todoist ID [todoist:${result.task.todoistId}]. Also found at ${existing.file}:${existing.line}.`,
               reason: `Duplicate Todoist ID [todoist:${result.task.todoistId}]. Also found at ${existing.file}:${existing.line}.`
             });
           } else {
@@ -359,9 +439,13 @@ var MarkdownScanner = class {
           const existing = todoistIds.get(task.todoistId);
           if (existing && existing.file !== task.file) {
             allWarnings.push({
+              severity: "error",
+              source: "md2do",
+              ruleId: "duplicate-todoist-id",
               file: task.file,
               line: task.line,
               text: task.text,
+              message: `Duplicate Todoist ID [todoist:${task.todoistId}] across files. Also found at ${existing.file}:${existing.line}.`,
               reason: `Duplicate Todoist ID [todoist:${task.todoistId}] across files. Also found at ${existing.file}:${existing.line}.`
             });
           } else if (!existing) {
@@ -810,6 +894,32 @@ function combineComparators(comparators) {
 function reverse(comparator) {
   return (a, b) => -comparator(a, b);
 }
+
+// src/warnings/filter.ts
+function filterWarnings(warnings, config = {}) {
+  if (config.enabled === false) {
+    return [];
+  }
+  if (!config.rules) {
+    return warnings;
+  }
+  return warnings.filter((warning) => {
+    const level = config.rules?.[warning.ruleId];
+    if (level === void 0) {
+      return true;
+    }
+    return level !== "off";
+  });
+}
+function groupWarningsBySeverity(warnings) {
+  return warnings.reduce(
+    (acc, warning) => {
+      acc[warning.severity].push(warning);
+      return acc;
+    },
+    { error: [], warning: [], info: [] }
+  );
+}
 export {
   ASSIGNEE,
   COMPLETED_DATE,
@@ -838,10 +948,13 @@ export {
   extractProjectFromPath,
   extractTags,
   extractTodoistId,
+  filterWarnings,
   filters_exports as filters,
   generateTaskId,
+  groupWarningsBySeverity,
   parseAbsoluteDate,
   parseTask,
+  parseTime,
   resolveRelativeDate,
   sorting_exports as sorting,
   updateTask,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@md2do/core",
-  "version": "0.2.3",
+  "version": "0.4.0",
   "description": "Core parsing, filtering, scanning, and file writing for md2do",
   "keywords": [
     "markdown",

package/src/index.ts

@@ -19,6 +19,9 @@ export * as filters from './filters/index.js';
 // Sorting (export as namespace to avoid conflicts)
 export * as sorting from './sorting/index.js';
 
+// Warnings
+export * from './warnings/filter.js';
+
 // Utilities
 export * from './utils/dates.js';
 export * from './utils/id.js';

package/src/parser/index.ts

@@ -94,27 +94,50 @@ export function extractCompletedDate(text: string): Date | undefined {
  * Extract due date from task text with context awareness
  *
  * Handles both absolute dates ([due: 2026-01-25]) and relative dates
- * ([due: tomorrow]) which require context.
+ * ([due: tomorrow]) which require context. Supports optional time ([due: 2026-01-25 17:00]).
  *
  * @param text - Task text
- * @param context - Parsing context (for relative dates)
+ * @param context - Parsing context (for relative dates and workday config)
  * @returns Object with parsed date and optional warning
  */
 export function extractDueDate(
   text: string,
   context: ParsingContext,
 ): { date: Date | undefined; warning?: Warning } {
-  // Try absolute date first
+  // Try absolute date first (with optional time)
   const absoluteMatch = text.match(PATTERNS.DUE_DATE_ABSOLUTE);
   if (absoluteMatch?.[1]) {
-    const date = parseAbsoluteDate(absoluteMatch[1]);
+    const dateStr = absoluteMatch[1];
+    let timeStr = absoluteMatch[2]; // May be undefined
+
+    // If no time specified, apply default from workday config
+    if (!timeStr && context.defaultDueTime) {
+      if (context.defaultDueTime === 'start' && context.workdayStartTime) {
+        timeStr = context.workdayStartTime;
+      } else if (context.defaultDueTime === 'end' && context.workdayEndTime) {
+        timeStr = context.workdayEndTime;
+      }
+    }
+
+    const date = parseAbsoluteDate(dateStr, timeStr);
     return { date: date ?? undefined };
   }
 
   // Try short format (M/D or M/D/YY)
   const shortMatch = text.match(PATTERNS.DUE_DATE_SHORT);
   if (shortMatch?.[1]) {
-    const date = parseAbsoluteDate(shortMatch[1]);
+    let timeStr: string | undefined;
+
+    // Apply default time from workday config
+    if (context.defaultDueTime) {
+      if (context.defaultDueTime === 'start' && context.workdayStartTime) {
+        timeStr = context.workdayStartTime;
+      } else if (context.defaultDueTime === 'end' && context.workdayEndTime) {
+        timeStr = context.workdayEndTime;
+      }
+    }
+
+    const date = parseAbsoluteDate(shortMatch[1], timeStr);
     return { date: date ?? undefined };
   }
 
@@ -126,9 +149,14 @@ export function extractDueDate(
       return {
         date: undefined,
         warning: {
+          severity: 'warning',
+          source: 'md2do',
+          ruleId: 'relative-date-no-context',
           file: '', // Will be filled in by caller
           line: 0, // Will be filled in by caller
           text: text.trim(),
+          message:
+            'Relative due date without context date from heading. Add a heading with a date above this task.',
           reason:
             'Relative due date without context date from heading. Add a heading with a date above this task.',
         },
@@ -212,9 +240,14 @@ export function parseTask(
   // Check for asterisk/plus bullet markers (not supported)
   if (/^\s*[*+]\s+\[[ xX]\]/.test(line)) {
     warnings.push({
+      severity: 'warning',
+      source: 'md2do',
+      ruleId: 'unsupported-bullet',
       file,
       line: lineNumber,
       text: line.trim(),
+      message:
+        'Unsupported bullet marker (* or +). Use dash (-) for task lists.',
       reason:
         'Unsupported bullet marker (* or +). Use dash (-) for task lists.',
     });
@@ -224,9 +257,14 @@ export function parseTask(
   // Check for extra spaces inside checkbox: [x ] or [ x]
   if (/^\s*-\s+\[[xX]\s+\]/.test(line) || /^\s*-\s+\[\s+[xX]\]/.test(line)) {
     warnings.push({
+      severity: 'warning',
+      source: 'md2do',
+      ruleId: 'malformed-checkbox',
       file,
       line: lineNumber,
       text: line.trim(),
+      message:
+        'Malformed checkbox with extra spaces. Use [x] or [ ] without extra spaces.',
       reason:
         'Malformed checkbox with extra spaces. Use [x] or [ ] without extra spaces.',
     });
@@ -236,9 +274,13 @@ export function parseTask(
   // Check for missing space after checkbox: [x]Task
   if (/^\s*-\s+\[[ xX]\][^\s]/.test(line)) {
     warnings.push({
+      severity: 'warning',
+      source: 'md2do',
+      ruleId: 'missing-space-after',
       file,
       line: lineNumber,
       text: line.trim(),
+      message: 'Missing space after checkbox. Use "- [x] Task" format.',
       reason: 'Missing space after checkbox. Use "- [x] Task" format.',
     });
     return { task: null, warnings };
@@ -247,9 +289,13 @@ export function parseTask(
   // Check for missing space before checkbox: -[x]
   if (/^\s*-\[[ xX]\]/.test(line)) {
     warnings.push({
+      severity: 'warning',
+      source: 'md2do',
+      ruleId: 'missing-space-before',
       file,
       line: lineNumber,
       text: line.trim(),
+      message: 'Missing space before checkbox. Use "- [x] Task" format.',
       reason: 'Missing space before checkbox. Use "- [x] Task" format.',
     });
     return { task: null, warnings };
@@ -288,9 +334,14 @@ export function parseTask(
   // Incomplete tasks without any date (no explicit due date and no context date)
   if (!completed && !dueDateResult.date && !context.currentDate) {
     warnings.push({
+      severity: 'info',
+      source: 'md2do',
+      ruleId: 'missing-due-date',
       file,
       line: lineNumber,
       text: fullText.trim(),
+      message:
+        'Task has no due date. Add [due: YYYY-MM-DD] or place under a heading with a date.',
       reason:
         'Task has no due date. Add [due: YYYY-MM-DD] or place under a heading with a date.',
     });
@@ -299,9 +350,14 @@ export function parseTask(
   // Completed tasks should have completion dates
   if (completed && !completedDate) {
     warnings.push({
+      severity: 'info',
+      source: 'md2do',
+      ruleId: 'missing-completed-date',
       file,
       line: lineNumber,
       text: fullText.trim(),
+      message:
+        'Completed task missing completion date. Add [completed: YYYY-MM-DD].',
       reason:
         'Completed task missing completion date. Add [completed: YYYY-MM-DD].',
     });

package/src/parser/patterns.ts

@@ -63,17 +63,20 @@ export const PRIORITY_HIGH = /!!/;
 export const PRIORITY_NORMAL = /(?<!!)!(?!!)/;
 
 /**
- * Matches absolute due date in ISO format [due: YYYY-MM-DD]
+ * Matches absolute due date in ISO format [due: YYYY-MM-DD] with optional time [due: YYYY-MM-DD HH:MM]
  *
  * Examples:
- *   "[due: 2026-01-25]" → "2026-01-25"
- *   "[due:2026-01-25]" → "2026-01-25" (spaces optional)
- *   "[due:  2026-01-25  ]" → "2026-01-25" (whitespace allowed)
+ *   "[due: 2026-01-25]" → "2026-01-25", undefined
+ *   "[due: 2026-01-25 17:00]" → "2026-01-25", "17:00"
+ *   "[due: 2026-01-25 9:00]" → "2026-01-25", "9:00"
+ *   "[due:2026-01-25]" → "2026-01-25", undefined (spaces optional)
  *
  * Groups:
  *   [1] - Date string in YYYY-MM-DD format
+ *   [2] - Optional time string in H:MM or HH:MM format (24-hour)
  */
-export const DUE_DATE_ABSOLUTE = /\[due:\s*(\d{4}-\d{2}-\d{2})\s*\]/i;
+export const DUE_DATE_ABSOLUTE =
+  /\[due:\s*(\d{4}-\d{2}-\d{2})(?:\s+(\d{1,2}:\d{2}))?\s*\]/i;
 
 /**
  * Matches relative due date keywords

package/src/scanner/index.ts

@@ -62,11 +62,17 @@ export class MarkdownScanner {
    *
    * @param filePath - Relative file path (for context extraction)
    * @param content - File content as string
+   * @param options - Optional scanner options including workday config
    * @returns Object containing tasks and warnings
    */
   scanFile(
     filePath: string,
     content: string,
+    options?: {
+      workdayStartTime?: string;
+      workdayEndTime?: string;
+      defaultDueTime?: 'start' | 'end';
+    },
   ): {
     tasks: Task[];
     warnings: Warning[];
@@ -86,6 +92,17 @@ export class MarkdownScanner {
     const person = extractPersonFromFilename(filePath);
     if (person !== undefined) context.person = person;
 
+    // Add workday config to context if provided
+    if (options?.workdayStartTime) {
+      context.workdayStartTime = options.workdayStartTime;
+    }
+    if (options?.workdayEndTime) {
+      context.workdayEndTime = options.workdayEndTime;
+    }
+    if (options?.defaultDueTime) {
+      context.defaultDueTime = options.defaultDueTime;
+    }
+
     // Process file line by line
     const lines = content.split('\n');
 
@@ -114,9 +131,13 @@ export class MarkdownScanner {
           const existing = todoistIds.get(result.task.todoistId);
           if (existing) {
             warnings.push({
+              severity: 'error',
+              source: 'md2do',
+              ruleId: 'duplicate-todoist-id',
               file: filePath,
               line: lineNumber,
               text: result.task.text,
+              message: `Duplicate Todoist ID [todoist:${result.task.todoistId}]. Also found at ${existing.file}:${existing.line}.`,
               reason: `Duplicate Todoist ID [todoist:${result.task.todoistId}]. Also found at ${existing.file}:${existing.line}.`,
             });
           } else {
@@ -168,9 +189,13 @@ export class MarkdownScanner {
             // Only add warning if duplicate is in a different file
             // (same-file duplicates are already caught by scanFile)
             allWarnings.push({
+              severity: 'error',
+              source: 'md2do',
+              ruleId: 'duplicate-todoist-id',
               file: task.file,
               line: task.line,
               text: task.text,
+              message: `Duplicate Todoist ID [todoist:${task.todoistId}] across files. Also found at ${existing.file}:${existing.line}.`,
               reason: `Duplicate Todoist ID [todoist:${task.todoistId}] across files. Also found at ${existing.file}:${existing.line}.`,
             });
           } else if (!existing) {

package/src/types/index.ts

@@ -36,6 +36,10 @@ export interface ParsingContext {
   person?: string;
   currentDate?: Date;
   currentHeading?: string;
+  // Workday configuration for time handling
+  workdayStartTime?: string; // e.g., "08:00"
+  workdayEndTime?: string; // e.g., "17:00"
+  defaultDueTime?: 'start' | 'end'; // Which time to use when no time specified
 }
 
 export interface TaskFilterCriteria {
@@ -65,9 +69,38 @@ export interface ScanResult {
   };
 }
 
+export type WarningSeverity = 'info' | 'warning' | 'error';
+
+export type WarningCode =
+  | 'unsupported-bullet' // * or + instead of -
+  | 'malformed-checkbox' // [x ] or [ x]
+  | 'missing-space-after' // -[x]Task
+  | 'missing-space-before' // -[x] Task
+  | 'relative-date-no-context' // [due: tomorrow] without heading date
+  | 'missing-due-date' // Incomplete task with no due date
+  | 'missing-completed-date' // [x] without [completed: date]
+  | 'duplicate-todoist-id' // Same Todoist ID in multiple tasks
+  | 'file-read-error'; // Failed to read file
+
 export interface Warning {
+  // Position
   file: string;
   line: number;
-  text: string;
-  reason: string;
+  column?: number;
+
+  // Classification
+  severity: WarningSeverity;
+  source: 'md2do';
+  ruleId: WarningCode;
+
+  // Content
+  message: string; // User-facing message
+  text?: string; // The actual text that triggered it
+
+  // Documentation (optional - for future use)
+  url?: string;
+
+  // Legacy field for backward compatibility (deprecated)
+  /** @deprecated Use message instead */
+  reason?: string;
 }