refactor: Rename 'todo' to 'task' across CLI commands and related components

Author: Coderrob

.github/workflows/validate-docs.yml

@@ -41,5 +41,5 @@ jobs:
       - name: Run validator unit tests
         run: npm test
 
-      - name: List TODO (smoke)
-        run: node dist/cli.js todo list || true
+      - name: List Tasks (smoke)
+        run: node dist/cli.js task list || true

README.md

@@ -123,44 +123,44 @@ dddctl <command> [options]
 
 The Document Driven Development Kit CLI provides the following commands and sub-commands:
 
-#### `todo list`
+#### `task list`
 
 List all tasks in the TODO.md file.
 
 Example:
 
 ```bash
-npm run cli -- todo list
+npm run cli -- task list
 ```
 
-#### `todo show`
+#### `task show`
 
 Show details of a specific task by ID.
 
 Example:
 
 ```bash
-npm run cli -- todo show <task-id>
+npm run cli -- task show <task-id>
 ```
 
-#### `todo complete`
+#### `task complete`
 
 Mark a task as complete.
 
 Example:
 
 ```bash
-npm run cli -- todo complete <task-id>
+npm run cli -- task complete <task-id>
 ```
 
-#### `todo add`
+#### `task add`
 
 Add a new task from a file.
 
 Example:
 
 ```bash
-npm run cli -- todo add <file-path>
+npm run cli -- task add <file-path>
 ```
 
 #### `validate tasks`
@@ -208,7 +208,7 @@ npm run cli -- ref audit
   Example:
 
   ```bash
-  npm run cli -- todo --help
+  npm run cli -- task --help
   ```
 
 For a full list of commands and options, run:

scripts/integration-test.sh

@@ -629,9 +629,9 @@ test_cli_help() {
         return 1
     fi
 
-    info "Testing todo help"
-    if ! run_cli_with_output "todo --help"; then
-        error "Todo help command failed"
+    info "Testing task help"
+    if ! run_cli_with_output "task --help"; then
+        error "Task help command failed"
         return 1
     fi
 
@@ -657,17 +657,17 @@ test_task_management() {
 
     # Add tasks
     info "Adding test tasks..."
-    if ! run_cli_command "todo add $TEST_DIR/task1-auth.md"; then
+    if ! run_cli_command "task add $TEST_DIR/task1-auth.md"; then
         error "Failed to add task1-auth.md"
         return 1
     fi
 
-    if ! run_cli_command "todo add $TEST_DIR/task2-migration.md"; then
+    if ! run_cli_command "task add $TEST_DIR/task2-migration.md"; then
         error "Failed to add task2-migration.md"
         return 1
     fi
 
-    if ! run_cli_command "todo add $TEST_DIR/task3-docs.md"; then
+    if ! run_cli_command "task add $TEST_DIR/task3-docs.md"; then
         error "Failed to add task3-docs.md"
         return 1
     fi
@@ -676,13 +676,13 @@ test_task_management() {
 
     # List tasks
     info "Listing all tasks..."
-    if ! run_cli_with_output "todo list"; then
+    if ! run_cli_with_output "task list"; then
         warning "Task listing had issues (may be due to existing TODO.md format)"
     fi
 
     # Show task details
     info "Showing task details..."
-    if ! run_cli_with_output "todo show integration.test.task.001"; then
+    if ! run_cli_with_output "task show integration.test.task.001"; then
         warning "Task show command had issues (may be due to existing TODO.md format)"
     fi
 
@@ -759,7 +759,7 @@ test_supersede() {
     fi
 
     info "Checking task list after supersede..."
-    if ! run_cli_with_output "todo list"; then
+    if ! run_cli_with_output "task list"; then
         warning "Task list after supersede had issues"
     fi
 
@@ -772,17 +772,17 @@ test_task_completion() {
     step "Testing task completion..."
 
     info "Completing task 002..."
-    if ! run_cli_command "todo complete integration.test.task.002 --message 'Integration test completion'" false; then
+    if ! run_cli_command "task complete integration.test.task.002 --message 'Integration test completion'" false; then
         warning "Task completion had issues (may be due to existing TODO.md format)"
     fi
 
     info "Completing remaining tasks..."
-    if ! run_cli_command "todo complete integration.test.task.003 --message 'Test cleanup'" false; then
+    if ! run_cli_command "task complete integration.test.task.003 --message 'Test cleanup'" false; then
         warning "Task completion had issues (may be due to existing TODO.md format)"
     fi
 
     info "Checking final task list..."
-    if ! run_cli_with_output "todo list"; then
+    if ! run_cli_with_output "task list"; then
         warning "Final task list had issues"
     fi
 
@@ -832,13 +832,13 @@ This integration test comprehensively validated the DDD-Kit CLI functionality wi
 
 1. **CLI Help System**
    - Main help command (\`--help\`)
-   - Subcommand help (\`todo --help\`, \`validate --help\`, \`ref --help\`)
+   - Subcommand help (\`task --help\`, \`validate --help\`, \`ref --help\`)
 
 2. **Task Management**
-   - Adding tasks from files (\`todo add <file>\`)
-   - Listing all tasks (\`todo list\`)
-   - Showing task details (\`todo show <id>\`)
-   - Task completion with messages (\`todo complete <id> --message "..."\`)
+   - Adding tasks from files (\`task add <file>\`)
+   - Listing all tasks (\`task list\`)
+   - Showing task details (\`task show <id>\`)
+   - Task completion with messages (\`task complete <id> --message "..."\`)
 
 3. **Validation System**
    - Task schema validation (\`validate tasks\`)

scripts/validate-local.mjs

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-import { TodoManager } from '../dist/core/storage/task.js';
+import { TaskManager } from '../dist/core/storage/task.manager.js';
 import { getLogger } from '../dist/core/system/logger.js';
 import { validateTasks } from '../dist/validators/validator.js';
 
@@ -20,11 +20,11 @@ import { validateTasks } from '../dist/validators/validator.js';
  */
 async function main() {
   try {
-    const todoManager = new TodoManager(getLogger());
-    const tasks = todoManager.listTasks();
+    const taskManager = new TaskManager(getLogger());
+    const tasks = taskManager.listTasks();
     const res = validateTasks(tasks);
 
-    if (res.valid) {
+    if (res.isValid) {
       console.log(`✅ All ${tasks.length} tasks validate successfully.`);
       process.exitCode = 0;
     } else {

src/commands/rendering/next.command.telemetry.ts

@@ -1,5 +1,6 @@
+import { CommandName } from '../../types';
 import { IObservabilityLogger } from '../../types/observability';
-import { IHydrationOptions } from '../../types/tasks';
+import { IHydrationOptions, TaskProviderType } from '../../types/tasks';
 
 export interface OperationContext {
   operationLogger: IObservabilityLogger;
@@ -24,10 +25,10 @@ export class NextCommandTelemetry {
       operationId: 'next_command_execution',
     });
 
-    operationLogger.counter('commands.next.executions', { provider: options.provider ?? 'todo' });
+    operationLogger.counter('commands.next.executions', { provider: options.provider ?? 'task' });
     operationLogger.event('command_execution_started', {
-      command: 'next',
-      provider: options.provider ?? 'todo',
+      command: CommandName.NEXT,
+      provider: options.provider ?? TaskProviderType.TASK,
       hasFilters: Boolean(options.filters?.length),
       filterCount: options.filters?.length ?? 0,
     });
@@ -37,14 +38,14 @@ export class NextCommandTelemetry {
 
   noTaskFound(op: OperationContext, options: IHydrationOptions): void {
     op.operationLogger.warn('No eligible tasks found for next command', {
-      provider: options.provider ?? 'todo',
+      provider: options.provider ?? TaskProviderType.TASK,
       filters: options.filters,
     });
     op.operationLogger.counter('commands.next.no_tasks_found', {
-      provider: options.provider ?? 'todo',
+      provider: options.provider ?? TaskProviderType.TASK,
     });
     op.operationLogger.event('command_execution_completed', {
-      command: 'next',
+      command: CommandName.NEXT,
       success: false,
       reason: 'no_eligible_tasks',
     });
@@ -57,7 +58,7 @@ export class NextCommandTelemetry {
 
     op.operationLogger.span('next_command_execution', op.startTime, endTime, {
       taskId,
-      provider: provider ?? 'todo',
+      provider: provider ?? TaskProviderType.TASK,
       success: true,
     });
 
@@ -66,9 +67,11 @@ export class NextCommandTelemetry {
       duration,
     });
 
-    op.operationLogger.counter('commands.next.success', { provider: provider ?? 'todo' });
+    op.operationLogger.counter('commands.next.success', {
+      provider: provider ?? TaskProviderType.TASK,
+    });
     op.operationLogger.event('command_execution_completed', {
-      command: 'next',
+      command: CommandName.NEXT,
       success: true,
       taskId,
       duration,
@@ -91,18 +94,18 @@ export class NextCommandTelemetry {
     });
 
     op.operationLogger.counter('commands.next.errors', {
-      provider: provider ?? 'todo',
+      provider: provider ?? TaskProviderType.TASK,
       error_type: errorType,
     });
 
     op.operationLogger.span('next_command_execution', op.startTime, endTime, {
-      provider: provider ?? 'todo',
+      provider: provider ?? TaskProviderType.TASK,
       success: false,
       error: message,
     });
 
     op.operationLogger.event('command_execution_completed', {
-      command: 'next',
+      command: CommandName.NEXT,
       success: false,
       error: message,
       duration,

src/commands/rendering/next.command.ts

@@ -50,7 +50,7 @@ export class NextCommand extends BaseCommand {
     const op: OperationContext = this.telemetry.recordStart(this.observabilityLogger, options);
 
     try {
-      const provider = this.createProvider(options.provider ?? 'todo');
+      const provider = this.createProvider(options.provider ?? TaskProviderType.TASK);
       const task = await this.findNextTask(provider, options);
 
       if (!task) {
@@ -81,7 +81,7 @@ export class NextCommand extends BaseCommand {
         taskProviderType = providerType;
         break;
       default:
-        taskProviderType = TaskProviderType.TODO;
+        taskProviderType = TaskProviderType.TASK;
     }
     return TaskProviderFactory.create(taskProviderType, this.logger);
   }
@@ -134,7 +134,11 @@ export class NextCommand extends BaseCommand {
     program
       .command(CommandName.NEXT)
       .description('Hydrate the next eligible task')
-      .option('--provider <provider>', 'Task provider: todo, issues, projects', 'todo')
+      .option(
+        '--provider <provider>',
+        'Task provider: task, issues, projects',
+        TaskProviderType.TASK,
+      )
       .option('--filters <filters...>', 'Filters for task selection')
       .option('--branch-prefix <prefix>', 'Branch prefix', 'feature/')
       .option('--pin <sha>', 'Pin to specific ddd-kit commit/tag')

src/commands/shared/command.factory.ts

@@ -31,12 +31,12 @@ export class CommandFactory {
     const ref = program.command('ref').description('Reference management');
     RefAuditCommand.configure(ref, logger);
 
-    // Todo commands
-    const todo = program.command('todo').description('Task management commands');
-    AddTaskCommand.configure(todo, logger);
-    CompleteTaskCommand.configure(todo, logger);
-    ListTasksCommand.configure(todo, logger);
-    ShowTaskCommand.configure(todo, logger);
+    // Task commands
+    const task = program.command('task').description('Task management commands');
+    AddTaskCommand.configure(task, logger);
+    CompleteTaskCommand.configure(task, logger);
+    ListTasksCommand.configure(task, logger);
+    ShowTaskCommand.configure(task, logger);
 
     // Validate commands
     const validate = program.command('validate').description('Validation commands');

src/core/processing/hydrate.ts

@@ -91,7 +91,7 @@ export class TaskHydrationService implements ITaskHydrationUseCase {
 
   /**
    * Creates a task provider based on the specified type.
-   * @param providerType - optional provider type to create, defaults to TODO
+   * @param providerType - optional provider type to create, defaults to TASK
    * @returns the created provider instance
    */
   private createProvider(providerType?: string) {
@@ -101,7 +101,7 @@ export class TaskHydrationService implements ITaskHydrationUseCase {
         return TaskProviderFactory.create(providerType, this.logger);
 
       default:
-        return TaskProviderFactory.create(TaskProviderType.TODO, this.logger);
+        return TaskProviderFactory.create(TaskProviderType.TASK, this.logger);
     }
   }
 

src/core/storage/task-provider.factory.ts

@@ -1,9 +1,8 @@
-import { ITaskRepository, TaskProviderType } from '../../types';
-import { ILogger } from '../../types/observability';
+import { TaskProviderType, ILogger, ITaskRepository } from '../../types';
 
-import { TaskProvider } from './task.provider';
 import { IssuesProvider } from './issues.provider';
 import { ProjectsProvider } from './projects.provider';
+import { TaskProvider } from './task.provider';
 
 /**
  * Factory for creating task providers following Factory pattern and OCP.
@@ -12,7 +11,7 @@ import { ProjectsProvider } from './projects.provider';
 export class TaskProviderFactory {
   static create(providerType: TaskProviderType, logger: ILogger): ITaskRepository {
     switch (providerType) {
-      case TaskProviderType.TODO:
+      case TaskProviderType.TASK:
         return new TaskProvider(logger);
 
       case TaskProviderType.ISSUES:
@@ -27,6 +26,6 @@ export class TaskProviderFactory {
   }
 
   static getAvailableProviders(): TaskProviderType[] {
-    return [TaskProviderType.TODO, TaskProviderType.ISSUES, TaskProviderType.PROJECTS];
+    return [TaskProviderType.TASK, TaskProviderType.ISSUES, TaskProviderType.PROJECTS];
   }
 }

src/services/task-render.service.ts

@@ -16,7 +16,7 @@ export class TaskRenderService implements ITaskRenderUseCase {
   }
 
   async execute(taskId: string, options: IRenderOptions): Promise<void> {
-    const provider = TaskProviderFactory.create(TaskProviderType.TODO, this.logger);
+    const provider = TaskProviderFactory.create(TaskProviderType.TASK, this.logger);
     const task = await provider.findById(taskId);
     if (!task) throw new Error(`Task ${taskId} not found`);