@task-mcp/shared 1.0.14 → 1.0.16

package/src/utils/hierarchy.test.ts

@@ -22,7 +22,7 @@ function createTask(
     title: `Task ${id}`,
     status: "pending",
     priority: "medium",
-    projectId: "test-project",
+    workspace: "test-workspace",
     createdAt: new Date().toISOString(),
     updatedAt: new Date().toISOString(),
     parentId,

package/src/utils/id.test.ts

@@ -2,16 +2,12 @@ import { describe, test, expect } from "bun:test";
 import {
   generateId,
   generateTaskId,
-  generateProjectId,
   generateInboxId,
   isValidTaskId,
-  isValidProjectId,
   isValidInboxId,
   validateTaskId,
-  validateProjectId,
   validateInboxId,
   assertValidTaskId,
-  assertValidProjectId,
   assertValidInboxId,
   InvalidIdError,
 } from "./id.js";
@@ -42,13 +38,6 @@ describe("ID Generation", () => {
     });
   });
 
-  describe("generateProjectId", () => {
-    test("generates project ID with correct prefix", () => {
-      const id = generateProjectId();
-      expect(id).toMatch(/^proj_[a-z0-9]+$/);
-    });
-  });
-
   describe("generateInboxId", () => {
     test("generates inbox ID with correct prefix", () => {
       const id = generateInboxId();
@@ -79,19 +68,6 @@ describe("Simple ID Validation (boolean)", () => {
     });
   });
 
-  describe("isValidProjectId", () => {
-    test("returns true for valid project IDs", () => {
-      expect(isValidProjectId("proj_abc123")).toBe(true);
-      expect(isValidProjectId("proj_a")).toBe(true);
-    });
-
-    test("returns false for invalid project IDs", () => {
-      expect(isValidProjectId("")).toBe(false);
-      expect(isValidProjectId("project_abc123")).toBe(false);
-      expect(isValidProjectId("task_abc123")).toBe(false);
-    });
-  });
-
   describe("isValidInboxId", () => {
     test("returns true for valid inbox IDs", () => {
       expect(isValidInboxId("inbox_abc123")).toBe(true);
@@ -154,18 +130,6 @@ describe("Detailed ID Validation", () => {
     });
   });
 
-  describe("validateProjectId", () => {
-    test("returns valid: true for valid project IDs", () => {
-      expect(validateProjectId("proj_abc123")).toEqual({ valid: true });
-    });
-
-    test("returns error for missing prefix", () => {
-      const result = validateProjectId("project_abc");
-      expect(result.valid).toBe(false);
-      expect(result.reason).toContain("proj_");
-    });
-  });
-
   describe("validateInboxId", () => {
     test("returns valid: true for valid inbox IDs", () => {
       expect(validateInboxId("inbox_abc123")).toEqual({ valid: true });
@@ -206,16 +170,6 @@ describe("Assert Functions", () => {
     });
   });
 
-  describe("assertValidProjectId", () => {
-    test("does not throw for valid ID", () => {
-      expect(() => assertValidProjectId("proj_abc123")).not.toThrow();
-    });
-
-    test("throws InvalidIdError for invalid ID", () => {
-      expect(() => assertValidProjectId("project_abc")).toThrow(InvalidIdError);
-    });
-  });
-
   describe("assertValidInboxId", () => {
     test("does not throw for valid ID", () => {
       expect(() => assertValidInboxId("inbox_abc123")).not.toThrow();
@@ -245,8 +199,8 @@ describe("InvalidIdError", () => {
   });
 
   test("exposes idType, invalidValue, and reason", () => {
-    const error = new InvalidIdError("project", "wrong", "missing prefix");
-    expect(error.idType).toBe("project");
+    const error = new InvalidIdError("task", "wrong", "missing prefix");
+    expect(error.idType).toBe("task");
     expect(error.invalidValue).toBe("wrong");
     expect(error.reason).toBe("missing prefix");
   });

package/src/utils/id.ts

@@ -16,13 +16,6 @@ export function generateTaskId(): string {
   return generateId("task");
 }
 
-/**
- * Generate a project ID
- */
-export function generateProjectId(): string {
-  return generateId("proj");
-}
-
 /**
  * Generate a view ID
  */
@@ -30,14 +23,6 @@ export function generateViewId(): string {
   return generateId("view");
 }
 
-/**
- * Validate a project ID format
- * @returns true if valid format (proj_[alphanumeric])
- */
-export function isValidProjectId(id: string): boolean {
-  return /^proj_[a-z0-9]+$/.test(id);
-}
-
 /**
  * Validate a task ID format
  * @returns true if valid format (task_[alphanumeric])
@@ -66,7 +51,7 @@ export function isValidInboxId(id: string): boolean {
  */
 export class InvalidIdError extends Error {
   constructor(
-    public readonly idType: "task" | "project" | "inbox" | "view",
+    public readonly idType: "task" | "inbox" | "view",
     public readonly invalidValue: unknown,
     public readonly reason: string
   ) {
@@ -105,28 +90,6 @@ export function validateTaskId(id: unknown): IdValidationResult {
   return { valid: true };
 }
 
-/**
- * Validate project ID with detailed error information
- */
-export function validateProjectId(id: unknown): IdValidationResult {
-  if (id === null || id === undefined) {
-    return { valid: false, reason: "Project ID is required" };
-  }
-  if (typeof id !== "string") {
-    return { valid: false, reason: `Project ID must be a string (received ${typeof id})` };
-  }
-  if (id.length === 0) {
-    return { valid: false, reason: "Project ID cannot be empty" };
-  }
-  if (!id.startsWith("proj_")) {
-    return { valid: false, reason: "Project ID must start with 'proj_' prefix" };
-  }
-  if (!/^proj_[a-z0-9]+$/.test(id)) {
-    return { valid: false, reason: "Project ID contains invalid characters" };
-  }
-  return { valid: true };
-}
-
 /**
  * Validate inbox ID with detailed error information
  */
@@ -159,16 +122,6 @@ export function assertValidTaskId(id: unknown): asserts id is string {
   }
 }
 
-/**
- * Assert valid project ID, throw if invalid
- */
-export function assertValidProjectId(id: unknown): asserts id is string {
-  const result = validateProjectId(id);
-  if (!result.valid) {
-    throw new InvalidIdError("project", id, result.reason!);
-  }
-}
-
 /**
  * Assert valid inbox ID, throw if invalid
  */

package/src/utils/index.ts

@@ -1,9 +1,7 @@
 export {
   generateId,
   generateTaskId,
-  generateProjectId,
   generateViewId,
-  isValidProjectId,
   isValidTaskId,
   generateInboxId,
   isValidInboxId,
@@ -11,10 +9,8 @@ export {
   InvalidIdError,
   type IdValidationResult,
   validateTaskId,
-  validateProjectId,
   validateInboxId,
   assertValidTaskId,
-  assertValidProjectId,
   assertValidInboxId,
 } from "./id.js";
 export { PriorityQueue } from "./priority-queue.js";
@@ -51,8 +47,6 @@ export {
   projectTask,
   projectTasks,
   projectTasksPaginated,
-  projectProject,
-  projectProjects,
   projectInboxItem,
   projectInboxItems,
   formatResponse,
@@ -69,13 +63,16 @@ export {
   renderStatusWidget,
   renderActionsWidget,
   renderInboxWidget,
-  renderProjectsTable,
+  renderWorkspacesTable,
+  renderProjectsTable, // Legacy alias
   renderTasksTable,
   renderDashboard,
-  renderProjectDashboard,
+  renderWorkspaceDashboard,
+  renderProjectDashboard, // Legacy alias
   renderGlobalDashboard,
   type DashboardStats,
   type DependencyMetrics,
+  type WorkspaceInfo,
   type DashboardData,
   type RenderDashboardOptions,
 } from "./dashboard-renderer.js";
@@ -123,3 +120,15 @@ export {
   // Banner
   banner,
 } from "./terminal-ui.js";
+
+// Workspace detection
+export {
+  normalizeWorkspace,
+  getGitRepoRoot,
+  getGitRepoRootSync,
+  getWorkspaceFromGit,
+  getWorkspaceFromGitSync,
+  getWorkspaceFromPath,
+  detectWorkspace,
+  detectWorkspaceSync,
+} from "./workspace.js";

package/src/utils/projection.test.ts

@@ -3,8 +3,6 @@ import {
   projectTask,
   projectTasks,
   projectTasksPaginated,
-  projectProject,
-  projectProjects,
   projectInboxItem,
   projectInboxItems,
   formatResponse,
@@ -13,14 +11,13 @@ import {
   summarizeList,
 } from "./projection.js";
 import type { Task } from "../schemas/task.js";
-import type { Project } from "../schemas/project.js";
 import type { InboxItem } from "../schemas/inbox.js";
 
 // Test fixtures
 const createTask = (overrides: Partial<Task> = {}): Task => {
   const base: Task = {
     id: "task-1",
-    projectId: "project-1",
+    workspace: "test-workspace",
     title: "Test Task",
     status: "pending",
     priority: "medium",
@@ -37,19 +34,6 @@ const createTask = (overrides: Partial<Task> = {}): Task => {
   return { ...base, ...overrides };
 };
 
-const createProject = (overrides: Partial<Project> = {}): Project => ({
-  id: "proj-1",
-  name: "Test Project",
-  status: "active",
-  createdAt: "2025-01-01T00:00:00.000Z",
-  updatedAt: "2025-01-01T00:00:00.000Z",
-  completionPercentage: 50,
-  description: "A test project",
-  totalTasks: 10,
-  completedTasks: 5,
-  ...overrides,
-});
-
 const createInboxItem = (overrides: Partial<InboxItem> = {}): InboxItem => ({
   id: "inbox-1",
   content: "Quick idea for later",
@@ -196,70 +180,6 @@ describe("projectTasksPaginated", () => {
   });
 });
 
-describe("projectProject", () => {
-  test("concise format returns essential fields", () => {
-    const project = createProject();
-    const result = projectProject(project, "concise");
-
-    expect(result).toEqual({
-      id: "proj-1",
-      name: "Test Project",
-      status: "active",
-      completionPercentage: 50,
-    });
-  });
-
-  test("concise format omits undefined completionPercentage", () => {
-    // Create project without completionPercentage by deleting it
-    const project = createProject();
-    delete (project as Record<string, unknown>)["completionPercentage"];
-    const result = projectProject(project, "concise");
-
-    expect(result).not.toHaveProperty("completionPercentage");
-  });
-
-  test("standard format includes more fields", () => {
-    const project = createProject();
-    const result = projectProject(project, "standard");
-
-    expect(result).toHaveProperty("id");
-    expect(result).toHaveProperty("name");
-    expect(result).toHaveProperty("status");
-    expect(result).toHaveProperty("completionPercentage");
-    expect(result).toHaveProperty("description");
-    expect(result).toHaveProperty("totalTasks");
-    expect(result).toHaveProperty("completedTasks");
-  });
-
-  test("detailed format returns full project", () => {
-    const project = createProject();
-    const result = projectProject(project, "detailed");
-
-    expect(result).toBe(project);
-  });
-});
-
-describe("projectProjects", () => {
-  test("projects all projects without limit", () => {
-    const projects = [createProject({ id: "p1" }), createProject({ id: "p2" })];
-    const result = projectProjects(projects, "concise");
-
-    expect(result).toHaveLength(2);
-  });
-
-  test("limits projects when limit provided", () => {
-    const projects = [
-      createProject({ id: "p1" }),
-      createProject({ id: "p2" }),
-      createProject({ id: "p3" }),
-    ];
-    const result = projectProjects(projects, "concise", 1);
-
-    expect(result).toHaveLength(1);
-    expect(result[0]?.id).toBe("p1");
-  });
-});
-
 describe("projectInboxItem", () => {
   test("concise format returns 3 essential fields", () => {
     const item = createInboxItem();

package/src/utils/projection.ts

@@ -6,14 +6,11 @@
  */
 
 import type { Task } from "../schemas/task.js";
-import type { Project } from "../schemas/project.js";
 import type { InboxItem } from "../schemas/inbox.js";
 import type {
   ResponseFormat,
   TaskSummary,
   TaskPreview,
-  ProjectSummary,
-  ProjectPreview,
   InboxSummary,
   InboxPreview,
   PaginatedResponse,
@@ -84,65 +81,6 @@ export function projectTasksPaginated(
   };
 }
 
-/**
- * Project a single project to the specified format
- */
-export function projectProject(
-  project: Project,
-  format: ResponseFormat
-): ProjectSummary | ProjectPreview | Project {
-  switch (format) {
-    case "concise": {
-      const summary: ProjectSummary = {
-        id: project.id,
-        name: project.name,
-        status: project.status,
-      };
-      if (project.completionPercentage !== undefined) {
-        summary.completionPercentage = project.completionPercentage;
-      }
-      return summary;
-    }
-
-    case "standard": {
-      const preview: ProjectPreview = {
-        id: project.id,
-        name: project.name,
-        status: project.status,
-      };
-      if (project.completionPercentage !== undefined) {
-        preview.completionPercentage = project.completionPercentage;
-      }
-      if (project.description !== undefined) {
-        preview.description = project.description;
-      }
-      if (project.totalTasks !== undefined) {
-        preview.totalTasks = project.totalTasks;
-      }
-      if (project.completedTasks !== undefined) {
-        preview.completedTasks = project.completedTasks;
-      }
-      return preview;
-    }
-
-    case "detailed":
-    default:
-      return project;
-  }
-}
-
-/**
- * Project multiple projects with optional limit
- */
-export function projectProjects(
-  projects: Project[],
-  format: ResponseFormat,
-  limit?: number
-): (ProjectSummary | ProjectPreview | Project)[] {
-  const sliced = limit ? projects.slice(0, limit) : projects;
-  return sliced.map((project) => projectProject(project, format));
-}
-
 /**
  * Project a single inbox item to the specified format
  */

package/src/utils/workspace.ts

@@ -0,0 +1,182 @@
+/**
+ * Workspace detection utilities
+ *
+ * Automatically detects workspace name from:
+ * 1. Git repository root directory name
+ * 2. TASKS_DIR path basename (fallback)
+ * 3. Current working directory basename (final fallback)
+ */
+
+import { exec } from "node:child_process";
+import { promisify } from "node:util";
+import { basename } from "node:path";
+
+const execAsync = promisify(exec);
+
+/**
+ * Normalize a string to a valid workspace name.
+ *
+ * Rules:
+ * - Lowercase
+ * - Replace spaces with hyphens
+ * - Keep alphanumeric, hyphens, and underscores
+ * - Trim leading/trailing hyphens
+ *
+ * @param name - Raw name to normalize
+ * @returns Normalized workspace name
+ *
+ * @example
+ * ```typescript
+ * normalizeWorkspace('My Project'); // 'my-project'
+ * normalizeWorkspace('Task-MCP'); // 'task-mcp'
+ * ```
+ */
+export function normalizeWorkspace(name: string): string {
+  return name
+    .toLowerCase()
+    .replace(/\s+/g, "-")
+    .replace(/[^a-z0-9_-]/g, "")
+    .replace(/^-+|-+$/g, "")
+    || "default";
+}
+
+/**
+ * Get the git repository root directory path.
+ *
+ * @param cwd - Working directory (defaults to process.cwd())
+ * @returns Absolute path to git root, or null if not a git repo
+ */
+export async function getGitRepoRoot(cwd?: string): Promise<string | null> {
+  try {
+    const { stdout } = await execAsync("git rev-parse --show-toplevel", {
+      cwd: cwd ?? process.cwd(),
+    });
+    return stdout.trim() || null;
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Synchronously get the git repository root directory path.
+ * Uses execSync - prefer async version when possible.
+ *
+ * @param cwd - Working directory (defaults to process.cwd())
+ * @returns Absolute path to git root, or null if not a git repo
+ */
+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"],
+    });
+    return (result as string).trim() || null;
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Detect workspace name from git repository.
+ *
+ * @param cwd - Working directory (defaults to process.cwd())
+ * @returns Workspace name derived from git repo, or null
+ *
+ * @example
+ * ```typescript
+ * // In /home/user/projects/task-mcp (git repo)
+ * await getWorkspaceFromGit(); // 'task-mcp'
+ * ```
+ */
+export async function getWorkspaceFromGit(cwd?: string): Promise<string | null> {
+  const repoRoot = await getGitRepoRoot(cwd);
+  if (!repoRoot) return null;
+  return normalizeWorkspace(basename(repoRoot));
+}
+
+/**
+ * Synchronous version of getWorkspaceFromGit.
+ */
+export function getWorkspaceFromGitSync(cwd?: string): string | null {
+  const repoRoot = getGitRepoRootSync(cwd);
+  if (!repoRoot) return null;
+  return normalizeWorkspace(basename(repoRoot));
+}
+
+/**
+ * Get workspace name from a file path.
+ * Uses the last directory name in the path.
+ *
+ * @param path - File system path
+ * @returns Normalized workspace name
+ *
+ * @example
+ * ```typescript
+ * getWorkspaceFromPath('/home/user/.tasks'); // 'user'
+ * getWorkspaceFromPath('/projects/my-app/.tasks'); // 'my-app'
+ * ```
+ */
+export function getWorkspaceFromPath(path: string): string {
+  // Remove trailing .tasks if present to get the parent directory
+  const cleanPath = path.replace(/[\/\\]?\.tasks[\/\\]?$/, "");
+  const name = basename(cleanPath);
+  return normalizeWorkspace(name);
+}
+
+/**
+ * Detect workspace name using multiple strategies.
+ *
+ * Strategy order:
+ * 1. Git repository root basename (most reliable)
+ * 2. TASKS_DIR parent directory basename
+ * 3. Current working directory basename
+ *
+ * @param tasksDir - Optional tasks directory path
+ * @returns Normalized workspace name
+ *
+ * @example
+ * ```typescript
+ * // In a git repo called 'my-project'
+ * await detectWorkspace(); // 'my-project'
+ *
+ * // With explicit TASKS_DIR
+ * await detectWorkspace('/projects/app/.tasks'); // 'app'
+ * ```
+ */
+export async function detectWorkspace(tasksDir?: string): Promise<string> {
+  // Strategy 1: Git repo detection
+  const gitWorkspace = await getWorkspaceFromGit();
+  if (gitWorkspace) {
+    return gitWorkspace;
+  }
+
+  // Strategy 2: TASKS_DIR path
+  if (tasksDir) {
+    return getWorkspaceFromPath(tasksDir);
+  }
+
+  // Strategy 3: CWD basename
+  return normalizeWorkspace(basename(process.cwd()));
+}
+
+/**
+ * Synchronous version of detectWorkspace.
+ * Prefer async version when possible.
+ */
+export function detectWorkspaceSync(tasksDir?: string): string {
+  // Strategy 1: Git repo detection
+  const gitWorkspace = getWorkspaceFromGitSync();
+  if (gitWorkspace) {
+    return gitWorkspace;
+  }
+
+  // Strategy 2: TASKS_DIR path
+  if (tasksDir) {
+    return getWorkspaceFromPath(tasksDir);
+  }
+
+  // Strategy 3: CWD basename
+  return normalizeWorkspace(basename(process.cwd()));
+}

package/src/schemas/project.ts

@@ -1,68 +0,0 @@
-import { z } from "zod";
-import { Priority } from "./task.js";
-
-// Project status
-export const ProjectStatus = z.enum([
-  "active",
-  "on_hold",
-  "completed",
-  "archived",
-]);
-export type ProjectStatus = z.infer<typeof ProjectStatus>;
-
-// Context definition
-export const Context = z.object({
-  name: z.string(),
-  color: z.string().optional(), // hex color
-  description: z.string().optional(),
-});
-export type Context = z.infer<typeof Context>;
-
-// Project schema
-export const Project = z.object({
-  id: z.string(),
-  name: z.string(),
-  description: z.string().optional(),
-  status: ProjectStatus,
-
-  // Project-level settings
-  defaultPriority: Priority.optional(),
-  contexts: z.array(Context).optional(),
-
-  // Metadata
-  createdAt: z.string(),
-  updatedAt: z.string(),
-  targetDate: z.string().optional(),
-  sortOrder: z.number().optional(), // User-defined display order (auto-assigned if not specified)
-
-  // Computed stats
-  completionPercentage: z.number().optional(),
-  criticalPathLength: z.number().optional(), // Total minutes on critical path
-  blockedTaskCount: z.number().optional(),
-  totalTasks: z.number().optional(),
-  completedTasks: z.number().optional(),
-});
-export type Project = z.infer<typeof Project>;
-
-// Project creation input
-export const ProjectCreateInput = z.object({
-  name: z.string(),
-  description: z.string().optional(),
-  defaultPriority: Priority.optional(),
-  contexts: z.array(Context).optional(),
-  targetDate: z.string().optional(),
-  sortOrder: z.number().optional(), // Auto-assigned if not specified
-});
-export type ProjectCreateInput = z.infer<typeof ProjectCreateInput>;
-
-// Project update input
-export const ProjectUpdateInput = z.object({
-  name: z.string().optional(),
-  description: z.string().optional(),
-  status: ProjectStatus.optional(),
-  defaultPriority: Priority.optional(),
-  contexts: z.array(Context).optional(),
-  targetDate: z.string().optional(),
-  sortOrder: z.number().optional(),
-});
-export type ProjectUpdateInput = z.infer<typeof ProjectUpdateInput>;