@task-mcp/shared 1.0.20 → 1.0.22

package/src/utils/natural-language.ts

@@ -30,6 +30,170 @@ function validateInputLength(input: string): void {
   }
 }
 
+/**
+ * Extracted metadata from natural language input
+ */
+export interface ExtractedMetadata {
+  /** Extracted priority (!high, !medium, !low, !critical) */
+  priority?: Priority;
+  /** Extracted due date (today, tomorrow, next week, etc.) */
+  dueDate?: string;
+  /** Extracted tags (#tag) */
+  tags?: string[];
+  /** Extracted estimated time in minutes (~2h, ~30m) */
+  estimateMinutes?: number;
+  /** Remaining text after metadata extraction */
+  remaining: string;
+}
+
+/**
+ * Options for metadata extraction
+ */
+export interface ExtractMetadataOptions {
+  /** Extract priority markers (!high, !low, etc.) */
+  extractPriority?: boolean;
+  /** Extract date expressions (today, tomorrow, by Friday, etc.) */
+  extractDate?: boolean;
+  /** Extract hashtags (#tag) */
+  extractTags?: boolean;
+  /** Extract time estimates (~2h, ~30m) */
+  extractEstimate?: boolean;
+}
+
+const DEFAULT_EXTRACT_OPTIONS: ExtractMetadataOptions = {
+  extractPriority: true,
+  extractDate: true,
+  extractTags: true,
+  extractEstimate: true,
+};
+
+/**
+ * Extract metadata from natural language input
+ *
+ * Supports:
+ * - Priority: !high, !medium, !low, !critical, !높음, !보통, !낮음, !긴급
+ * - Date: today, tomorrow, next week, 내일, 오늘, by Friday, 금요일까지
+ * - Tags: #tag, #개발, #backend
+ * - Time estimate: ~2h, ~30m, ~1h30m (with ~ prefix)
+ *
+ * @param input - Raw natural language input
+ * @param options - Options to control which metadata types to extract
+ * @returns Extracted metadata and remaining text
+ *
+ * @example
+ * ```typescript
+ * const result = extractMetadata("Review PR tomorrow #dev !high ~2h");
+ * // {
+ * //   priority: "high",
+ * //   dueDate: "2025-01-01",
+ * //   tags: ["dev"],
+ * //   estimateMinutes: 120,
+ * //   remaining: "Review PR"
+ * // }
+ * ```
+ */
+export function extractMetadata(
+  input: string,
+  options: ExtractMetadataOptions = DEFAULT_EXTRACT_OPTIONS
+): ExtractedMetadata {
+  validateInputLength(input);
+  let remaining = input.trim();
+  const result: ExtractedMetadata = { remaining: "" };
+
+  // Extract priority (!high, !critical, !medium, !low, !높음, !보통, !낮음)
+  if (options.extractPriority !== false) {
+    const priorityMatch = remaining.match(/!([\p{L}\p{N}_]+)/gu);
+    if (priorityMatch) {
+      for (const match of priorityMatch) {
+        const priority = parsePriority(match.slice(1));
+        if (priority) {
+          result.priority = priority;
+          remaining = remaining.replace(match, "").trim();
+          break;
+        }
+      }
+    }
+  }
+
+  // Extract tags (#dev, #backend, #개발)
+  if (options.extractTags !== false) {
+    const tagMatches = remaining.match(/#([\p{L}\p{N}_]+)/gu);
+    if (tagMatches) {
+      result.tags = tagMatches.map((m) => m.slice(1));
+      for (const match of tagMatches) {
+        remaining = remaining.replace(match, "").trim();
+      }
+    }
+  }
+
+  // Extract due date patterns
+  if (options.extractDate !== false) {
+    // "by Friday", "by tomorrow", "until next week"
+    const byMatch = remaining.match(/\b(by|until|before)\s+(\w+(\s+\w+)?)/i);
+    if (byMatch) {
+      const dateStr = byMatch[2]!;
+      const date = parseRelativeDate(dateStr);
+      if (date) {
+        result.dueDate = formatDate(date);
+        remaining = remaining.replace(byMatch[0], "").trim();
+      }
+    }
+
+    // Korean date patterns: "내일까지", "금요일까지"
+    if (!result.dueDate) {
+      const koreanDueMatch = remaining.match(/(\S+)까지/);
+      if (koreanDueMatch) {
+        const dateStr = koreanDueMatch[1]!;
+        const date = parseRelativeDate(dateStr);
+        if (date) {
+          result.dueDate = formatDate(date);
+          remaining = remaining.replace(koreanDueMatch[0], "").trim();
+        }
+      }
+    }
+
+    // "tomorrow", "today" at the end
+    if (!result.dueDate) {
+      const dateWords = ["tomorrow", "today", "내일", "오늘", "모레"];
+      for (const word of dateWords) {
+        if (remaining.toLowerCase().endsWith(word)) {
+          const date = parseRelativeDate(word);
+          if (date) {
+            result.dueDate = formatDate(date);
+            remaining = remaining.slice(0, -word.length).trim();
+            break;
+          }
+        }
+      }
+    }
+  }
+
+  // Extract time estimate (~2h, ~30m, ~1h30m)
+  if (options.extractEstimate !== false) {
+    const timeMatch = remaining.match(/~(\d+h\d+m|\d+h|\d+m)\b/);
+    if (timeMatch) {
+      let minutes = 0;
+      const hoursMatch = timeMatch[1]!.match(/(\d+)h/);
+      const minsMatch = timeMatch[1]!.match(/(\d+)m/);
+      if (hoursMatch) {
+        minutes += parseInt(hoursMatch[1]!, 10) * 60;
+      }
+      if (minsMatch) {
+        minutes += parseInt(minsMatch[1]!, 10);
+      }
+      if (minutes > 0) {
+        result.estimateMinutes = minutes;
+        remaining = remaining.replace(timeMatch[0], "").trim();
+      }
+    }
+  }
+
+  // Clean up extra spaces
+  result.remaining = remaining.replace(/\s+/g, " ").trim();
+
+  return result;
+}
+
 /**
  * Target type for parsed input
  */
@@ -94,21 +258,21 @@ export function parseInput(input: string): ParsedInput {
  * - "성능 개선 아이디어 #performance"
  */
 export function parseInboxInput(input: string): InboxCreateInput {
-  validateInputLength(input);
-  let remaining = input.trim();
+  // Inbox only extracts tags
+  const metadata = extractMetadata(input, {
+    extractPriority: false,
+    extractDate: false,
+    extractTags: true,
+    extractEstimate: false,
+  });
+
   const result: InboxCreateInput = { content: "" };
 
-  // Extract tags (#dev, #backend, #개발)
-  const tagMatches = remaining.match(/#([\p{L}\p{N}_]+)/gu);
-  if (tagMatches) {
-    result.tags = tagMatches.map((m) => m.slice(1));
-    for (const match of tagMatches) {
-      remaining = remaining.replace(match, "").trim();
-    }
+  if (metadata.tags) {
+    result.tags = metadata.tags;
   }
 
-  // Clean up and set content
-  result.content = remaining.replace(/\s+/g, " ").trim();
+  result.content = metadata.remaining;
 
   if (!result.content) {
     throw new InputValidationError(
@@ -123,7 +287,7 @@ export function parseInboxInput(input: string): InboxCreateInput {
  * Parse natural language task input
  *
  * Examples:
- * - "Review PR tomorrow #dev !high"
+ * - "Review PR tomorrow #dev !high ~2h"
  * - "내일까지 보고서 작성 #업무 !높음 @집중"
  * - "Fix bug by Friday #backend !critical"
  * - "Write tests every Monday #testing"
@@ -133,20 +297,7 @@ export function parseTaskInput(input: string): TaskCreateInput {
   let remaining = input.trim();
   const result: TaskCreateInput = { title: "" };
 
-  // Extract priority (!high, !critical, !medium, !low, !높음, !보통, !낮음)
-  const priorityMatch = remaining.match(/!([\p{L}\p{N}_]+)/gu);
-  if (priorityMatch) {
-    for (const match of priorityMatch) {
-      const priority = parsePriority(match.slice(1));
-      if (priority) {
-        result.priority = priority;
-        remaining = remaining.replace(match, "").trim();
-        break;
-      }
-    }
-  }
-
-  // Extract contexts (@focus, @review, @집중)
+  // Extract contexts first (@focus, @review, @집중) - task-specific
   const contextMatches = remaining.match(/@([\p{L}\p{N}_]+)/gu);
   if (contextMatches) {
     result.contexts = contextMatches.map((m) => m.slice(1));
@@ -155,57 +306,20 @@ export function parseTaskInput(input: string): TaskCreateInput {
     }
   }
 
-  // Extract tags (#dev, #backend, #개발)
-  const tagMatches = remaining.match(/#([\p{L}\p{N}_]+)/gu);
-  if (tagMatches) {
-    result.tags = tagMatches.map((m) => m.slice(1));
-    for (const match of tagMatches) {
-      remaining = remaining.replace(match, "").trim();
-    }
-  }
-
-  // Extract due date patterns
-  // "by Friday", "by tomorrow", "until next week"
-  const byMatch = remaining.match(/\b(by|until|before)\s+(\w+(\s+\w+)?)/i);
-  if (byMatch) {
-    const dateStr = byMatch[2]!;
-    const date = parseRelativeDate(dateStr);
-    if (date) {
-      result.dueDate = formatDate(date);
-      remaining = remaining.replace(byMatch[0], "").trim();
-    }
-  }
-
-  // Korean date patterns: "내일까지", "금요일까지"
-  const koreanDueMatch = remaining.match(/(\S+)까지/);
-  if (koreanDueMatch && !result.dueDate) {
-    const dateStr = koreanDueMatch[1]!;
-    const date = parseRelativeDate(dateStr);
-    if (date) {
-      result.dueDate = formatDate(date);
-      remaining = remaining.replace(koreanDueMatch[0], "").trim();
-    }
-  }
-
-  // "tomorrow", "today" at the end
-  const dateWords = ["tomorrow", "today", "내일", "오늘", "모레"];
-  for (const word of dateWords) {
-    if (remaining.toLowerCase().endsWith(word) && !result.dueDate) {
-      const date = parseRelativeDate(word);
-      if (date) {
-        result.dueDate = formatDate(date);
-        remaining = remaining.slice(0, -word.length).trim();
-      }
-    }
+  // Extract sortOrder (^1, ^10) - task-specific
+  const sortMatch = remaining.match(/\^(\d+)/);
+  if (sortMatch) {
+    result.sortOrder = parseInt(sortMatch[1]!, 10);
+    remaining = remaining.replace(sortMatch[0], "").trim();
   }
 
-  // Extract time estimate (30m, 2h, 1h30m)
-  // Pattern matches: "2h30m", "2h", "30m" (but not empty string)
-  const timeMatch = remaining.match(/\b(\d+h\d+m|\d+h|\d+m)\b/);
-  if (timeMatch) {
+  // Extract time estimate without ~ prefix for backward compatibility (30m, 2h, 1h30m)
+  // This handles the legacy format before extractMetadata handles ~prefix format
+  const legacyTimeMatch = remaining.match(/\b(\d+h\d+m|\d+h|\d+m)\b/);
+  if (legacyTimeMatch && !remaining.includes("~" + legacyTimeMatch[1])) {
     let minutes = 0;
-    const hoursMatch = timeMatch[0].match(/(\d+)h/);
-    const minsMatch = timeMatch[0].match(/(\d+)m/);
+    const hoursMatch = legacyTimeMatch[0].match(/(\d+)h/);
+    const minsMatch = legacyTimeMatch[0].match(/(\d+)m/);
     if (hoursMatch) {
       minutes += parseInt(hoursMatch[1]!, 10) * 60;
     }
@@ -214,20 +328,33 @@ export function parseTaskInput(input: string): TaskCreateInput {
     }
     if (minutes > 0) {
       result.estimate = { expected: minutes, confidence: "medium" };
-      remaining = remaining.replace(timeMatch[0], "").trim();
+      remaining = remaining.replace(legacyTimeMatch[0], "").trim();
     }
   }
 
-  // Extract sortOrder (^1, ^10, ^순서1)
-  // Used for manual ordering within task lists
-  const sortMatch = remaining.match(/\^(\d+)/);
-  if (sortMatch) {
-    result.sortOrder = parseInt(sortMatch[1]!, 10);
-    remaining = remaining.replace(sortMatch[0], "").trim();
+  // Use extractMetadata for common patterns
+  const metadata = extractMetadata(remaining, {
+    extractPriority: true,
+    extractDate: true,
+    extractTags: true,
+    extractEstimate: true,
+  });
+
+  // Apply extracted metadata
+  if (metadata.priority) {
+    result.priority = metadata.priority;
+  }
+  if (metadata.dueDate) {
+    result.dueDate = metadata.dueDate;
+  }
+  if (metadata.tags) {
+    result.tags = metadata.tags;
+  }
+  if (metadata.estimateMinutes && !result.estimate) {
+    result.estimate = { expected: metadata.estimateMinutes, confidence: "medium" };
   }
 
-  // Clean up extra spaces
-  result.title = remaining.replace(/\s+/g, " ").trim();
+  result.title = metadata.remaining;
 
   if (!result.title) {
     throw new InputValidationError(

package/src/utils/projection.ts

@@ -27,7 +27,7 @@ function assertNever(value: never): never {
 /**
  * Project a single task to the specified format
  */
-export function projectTask(task: Task, format: ResponseFormat): TaskSummary | TaskPreview | Task {
+export function formatTask(task: Task, format: ResponseFormat): TaskSummary | TaskPreview | Task {
   switch (format) {
     case "concise":
       return {
@@ -59,19 +59,19 @@ export function projectTask(task: Task, format: ResponseFormat): TaskSummary | T
 /**
  * Project multiple tasks with optional limit
  */
-export function projectTasks(
+export function formatTasks(
   tasks: Task[],
   format: ResponseFormat,
   limit?: number
 ): (TaskSummary | TaskPreview | Task)[] {
   const sliced = limit ? tasks.slice(0, limit) : tasks;
-  return sliced.map((task) => projectTask(task, format));
+  return sliced.map((task) => formatTask(task, format));
 }
 
 /**
  * Project tasks with pagination
  */
-export function projectTasksPaginated(
+export function formatTasksPaginated(
   tasks: Task[],
   format: ResponseFormat,
   limit: number = 20,
@@ -79,7 +79,7 @@ export function projectTasksPaginated(
 ): PaginatedResponse<TaskSummary | TaskPreview | Task> {
   const effectiveLimit = Math.min(limit, 100);
   const sliced = tasks.slice(offset, offset + effectiveLimit);
-  const projected = sliced.map((task) => projectTask(task, format));
+  const projected = sliced.map((task) => formatTask(task, format));
 
   return {
     items: projected,
@@ -93,7 +93,7 @@ export function projectTasksPaginated(
 /**
  * Project a single inbox item to the specified format
  */
-export function projectInboxItem(
+export function formatInboxItem(
   item: InboxItem,
   format: ResponseFormat
 ): InboxSummary | InboxPreview | InboxItem {
@@ -124,13 +124,13 @@ export function projectInboxItem(
 /**
  * Project multiple inbox items with optional limit
  */
-export function projectInboxItems(
+export function formatInboxItems(
   items: InboxItem[],
   format: ResponseFormat,
   limit?: number
 ): (InboxSummary | InboxPreview | InboxItem)[] {
   const sliced = limit ? items.slice(0, limit) : items;
-  return sliced.map((item) => projectInboxItem(item, format));
+  return sliced.map((item) => formatInboxItem(item, format));
 }
 
 /**

package/src/utils/workspace.ts

@@ -7,12 +7,18 @@
  * 3. Current working directory basename (final fallback)
  */
 
-import { exec } from "node:child_process";
+import { exec, execSync } from "node:child_process";
 import { promisify } from "node:util";
 import { basename } from "node:path";
 
 const execAsync = promisify(exec);
 
+/**
+ * Maximum length for workspace names.
+ * Prevents excessive path lengths and potential DoS via long names.
+ */
+const MAX_WORKSPACE_LENGTH = 64;
+
 /**
  * Normalize a string to a valid workspace name.
  *
@@ -21,6 +27,7 @@ const execAsync = promisify(exec);
  * - Replace spaces with hyphens
  * - Keep alphanumeric, hyphens, and underscores
  * - Trim leading/trailing hyphens
+ * - Limit to MAX_WORKSPACE_LENGTH characters
  *
  * @param name - Raw name to normalize
  * @returns Normalized workspace name
@@ -29,15 +36,19 @@ const execAsync = promisify(exec);
  * ```typescript
  * normalizeWorkspace('My Project'); // 'my-project'
  * normalizeWorkspace('Task-MCP'); // 'task-mcp'
+ * normalizeWorkspace('a'.repeat(100)); // 64 chars max
  * ```
  */
 export function normalizeWorkspace(name: string): string {
-  return name
+  const normalized = name
     .toLowerCase()
     .replace(/\s+/g, "-")
     .replace(/[^a-z0-9_-]/g, "")
     .replace(/^-+|-+$/g, "")
-    || "default";
+    .slice(0, MAX_WORKSPACE_LENGTH) // Enforce length limit
+    .replace(/-+$/, ""); // Clean trailing hyphens after truncation
+
+  return normalized || "default";
 }
 
 /**
@@ -66,11 +77,12 @@ export async function getGitRepoRoot(cwd?: string): Promise<string | null> {
  */
 export function getGitRepoRootSync(cwd?: string): string | null {
   try {
-    const { execSync } = require("node:child_process");
     const result = execSync("git rev-parse --show-toplevel", {
       cwd: cwd ?? process.cwd(),
       encoding: "utf-8",
       stdio: ["pipe", "pipe", "pipe"],
+      timeout: 5000, // 5 second timeout
+      maxBuffer: 1024, // Limit output buffer
     });
     return (result as string).trim() || null;
   } catch {

package/dist/algorithms/critical-path.d.ts

@@ -1,46 +0,0 @@
-import type { Task } from "../schemas/task.js";
-/**
- * Task with computed CPM (Critical Path Method) values
- */
-export interface CPMTask extends Task {
-    earliestStart: number;
-    earliestFinish: number;
-    latestStart: number;
-    latestFinish: number;
-    slack: number;
-    isCritical: boolean;
-    dependentCount: number;
-}
-/**
- * Result of critical path analysis
- */
-export interface CPMResult {
-    tasks: CPMTask[];
-    criticalPath: CPMTask[];
-    projectDuration: number;
-    bottlenecks: CPMTask[];
-}
-/**
- * Perform Critical Path Method analysis
- *
- * CPM calculates:
- * - Earliest Start (ES): Earliest a task can start
- * - Earliest Finish (EF): ES + duration
- * - Latest Finish (LF): Latest a task can finish without delaying project
- * - Latest Start (LS): LF - duration
- * - Slack: LS - ES (or LF - EF)
- * - Critical Path: Tasks with slack = 0
- */
-export declare function criticalPathAnalysis(tasks: Task[]): CPMResult;
-/**
- * Find tasks that can be executed in parallel (no dependencies between them)
- */
-export declare function findParallelTasks(tasks: Task[]): Task[][];
-/**
- * Suggest the next best task to work on
- */
-export declare function suggestNextTask(tasks: Task[], options?: {
-    contexts?: string[];
-    maxMinutes?: number;
-}): Task | null;
-//# sourceMappingURL=critical-path.d.ts.map

package/dist/algorithms/critical-path.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"critical-path.d.ts","sourceRoot":"","sources":["../../src/algorithms/critical-path.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,oBAAoB,CAAC;AAO/C;;GAEG;AACH,MAAM,WAAW,OAAQ,SAAQ,IAAI;IAEnC,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,OAAO,CAAC;IACpB,cAAc,EAAE,MAAM,CAAC;CACxB;AAED;;GAEG;AACH,MAAM,WAAW,SAAS;IACxB,KAAK,EAAE,OAAO,EAAE,CAAC;IACjB,YAAY,EAAE,OAAO,EAAE,CAAC;IACxB,eAAe,EAAE,MAAM,CAAC;IACxB,WAAW,EAAE,OAAO,EAAE,CAAC;CACxB;AA8ND;;;;;;;;;;GAUG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,IAAI,EAAE,GAAG,SAAS,CAmD7D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,EAAE,CAiDzD;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,IAAI,EAAE,EACb,OAAO,GAAE;IACP,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;CAChB,GACL,IAAI,GAAG,IAAI,CAyDb"}