feat: enhance task fixing functionality with TaskFixResult interface and refactor applyBasicFixes method

Author: Coderrob

src/core/fixers/task.fixer.ts

@@ -1,4 +1,4 @@
-import { FixRecord, IFixerOptions, ITask } from '../../types/tasks';
+import { FixRecord, IFixerOptions, ITask, TaskFixResult } from '../../types/tasks';
 
 import { fixPriority } from './priority.fixer';
 import { fixStatus } from './status.fixer';
@@ -22,15 +22,15 @@ export class TaskFixer {
 
   /**
    * Applies basic automatic fixes to common validation issues in a task object.
-   * @param asObj - The original task object (not modified).
+   * @param task - The original task object (not modified).
    * @returns An object containing the fixed task and an array of FixRecord objects describing the fixes applied.
    */
-  applyBasicFixes(asObj: ITask): { fixedTask: ITask; fixes: FixRecord[] } {
+  applyBasicFixes(task: ITask): TaskFixResult {
     const fixes: FixRecord[] = [];
-    const id = String(asObj.id);
+    const id = String(task.id);
 
     // Chain the immutable fixes
-    let fixedTask = fixPriority(asObj, fixes, id);
+    let fixedTask = fixPriority(task, fixes, id);
     fixedTask = fixStatus(fixedTask, fixes, id);
     fixedTask = fixDateField({
       nowIso: this.nowIso,

src/core/storage/task.manager.ts

@@ -168,7 +168,10 @@ export class TaskManager implements ITaskStore, IChangelogStore {
   ): void {
     const afterUnreleasedIndex = content.indexOf('\n', unreleasedIndex);
     if (afterUnreleasedIndex === -1) {
-      this.logger.warn('Could not append entry to CHANGELOG.md: no newline found after "Unreleased" section.', { entry });
+      this.logger.warn(
+        'Could not append entry to CHANGELOG.md: no newline found after "Unreleased" section.',
+        { entry },
+      );
       return;
     }
 

src/types/tasks/ITaskFixer.ts

@@ -1,5 +1,5 @@
-import type { FixRecord } from './FixRecord';
 import { ITask } from './ITask';
+import { TaskFixResult } from './TaskFixResult';
 
 /**
  * Interface for automatically fixing common task validation issues.
@@ -42,5 +42,5 @@ export interface ITaskFixer {
    * // result.fixes contains descriptions of applied fixes
    * ```
    */
-  applyBasicFixes(task: ITask): { fixedTask: ITask; fixes: FixRecord[] };
+  applyBasicFixes(task: ITask): TaskFixResult;
 }

src/types/tasks/TaskFixResult.ts

@@ -0,0 +1,19 @@
+import type { FixRecord } from './FixRecord';
+import { ITask } from './ITask';
+
+/**
+ * Result of applying fixes to a task object.
+ *
+ * Contains both the corrected task and a record of what fixes were applied.
+ */
+export interface TaskFixResult {
+  /**
+   * The task object with fixes applied.
+   */
+  fixedTask: ITask;
+
+  /**
+   * Array of records describing what fixes were applied to the task.
+   */
+  fixes: FixRecord[];
+}

src/types/tasks/index.ts

@@ -13,6 +13,7 @@ export * from './TaskStatus';
 export * from './TaskProviderType';
 export * from './IFixerOptions';
 export * from './FixRecord';
+export * from './TaskFixResult';
 
 // Todo command types (task-specific commands)
 export * from './AddTaskArgs';