@opentask/taskin-task-manager 1.0.8 → 2.0.0

package/.prettierignore

@@ -0,0 +1 @@
+dist/

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
 
-> @opentask/taskin-task-manager@1.0.8 build /Users/sidarta/repositorios/taskin/packages/task-manager
+> @opentask/taskin-task-manager@2.0.0 build /Users/sidarta/repositorios/taskin/packages/task-manager
 > tsc
 

package/.turbo/turbo-format.log

@@ -1,31 +1,18 @@
 
 
-> @opentask/taskin-task-manager@1.0.5 format /Users/sidarta/repositorios/taskin/packages/task-manager
+> @opentask/taskin-task-manager@2.0.0 format /Users/sidarta/repositorios/taskin/packages/task-manager
 > prettier --write .
 
-.turbo/turbo-build.log.turbo/turbo-format.log.turbo/turbo-test.logdist/index.d.tsdist/index.d.ts 59ms (unchanged)
-dist/index.jsdist/index.js 12ms (unchanged)
-dist/index.js.mapdist/metrics.types.d.tsdist/metrics.types.d.ts 19ms (unchanged)
-dist/metrics.types.jsdist/metrics.types.js 3ms (unchanged)
-dist/metrics.types.js.mapdist/provider-stats.test.d.tsdist/provider-stats.test.d.ts 2ms (unchanged)
-dist/provider-stats.test.jsdist/provider-stats.test.js 13ms (unchanged)
-dist/provider-stats.test.js.mapdist/task-manager.d.tsdist/task-manager.d.ts 8ms (unchanged)
-dist/task-manager.jsdist/task-manager.js 19ms (unchanged)
-dist/task-manager.js.mapdist/task-manager.mock.d.tsdist/task-manager.mock.d.ts 8ms (unchanged)
-dist/task-manager.mock.jsdist/task-manager.mock.js 6ms (unchanged)
-dist/task-manager.mock.js.mapdist/task-manager.test.d.tsdist/task-manager.test.d.ts 5ms (unchanged)
-dist/task-manager.test.jsdist/task-manager.test.js 25ms (unchanged)
-dist/task-manager.test.js.mapdist/task-manager.types.d.tsdist/task-manager.types.d.ts 19ms (unchanged)
-dist/task-manager.types.jsdist/task-manager.types.js 2ms (unchanged)
-dist/task-manager.types.js.mapeslint.config.jseslint.config.js 9ms (unchanged)
-package.jsonpackage.json 7ms (unchanged)
-README.mdREADME.md 26ms (unchanged)
-src/index.tssrc/index.ts 4ms (unchanged)
-src/metrics.types.tssrc/metrics.types.ts 14ms (unchanged)
-src/provider-stats.test.tssrc/provider-stats.test.ts 8ms (unchanged)
-src/task-manager.mock.tssrc/task-manager.mock.ts 13ms (unchanged)
-src/task-manager.test.tssrc/task-manager.test.ts 48ms (unchanged)
-src/task-manager.tssrc/task-manager.ts 27ms (unchanged)
-src/task-manager.types.tssrc/task-manager.types.ts 9ms (unchanged)
-tsconfig.jsontsconfig.json 2ms (unchanged)
+.prettierignore.turbo/turbo-build.log.turbo/turbo-format.log.turbo/turbo-install.log.turbo/turbo-lint.log.turbo/turbo-lint$colon$fix.log.turbo/turbo-test.log.turbo/turbo-typecheck.logCHANGELOG.mdCHANGELOG.md 81ms (unchanged)
+eslint.config.jseslint.config.js 290ms (unchanged)
+package.jsonpackage.json 6ms (unchanged)
+README.mdREADME.md 133ms (unchanged)
+src/index.tssrc/index.ts 27ms (unchanged)
+src/metrics.types.tssrc/metrics.types.ts 13ms (unchanged)
+src/provider-stats.test.tssrc/provider-stats.test.ts 129ms (unchanged)
+src/task-manager.mock.tssrc/task-manager.mock.ts 15ms (unchanged)
+src/task-manager.test.tssrc/task-manager.test.ts 102ms (unchanged)
+src/task-manager.tssrc/task-manager.ts 13ms (unchanged)
+src/task-manager.types.tssrc/task-manager.types.ts 28ms (unchanged)
+tsconfig.jsontsconfig.json 4ms (unchanged)
 tsconfig.tsbuildinfovitest.config.tsvitest.config.ts 4ms (unchanged)

package/.turbo/turbo-install.log

@@ -0,0 +1,6 @@
+
+
+> @opentask/taskin-task-manager@2.0.0 install /Users/sidarta/repositorios/taskin/packages/task-manager
+> echo 'Installing taskin-task-manager dependencies...'
+
+Installing taskin-task-manager dependencies...

package/.turbo/turbo-lint$colon$fix.log

@@ -0,0 +1,12 @@
+
+> @opentask/taskin-task-manager@1.0.8 lint:fix /Users/sidarta/repositorios/taskin/packages/task-manager
+> eslint . --fix
+
+
+/Users/sidarta/repositorios/taskin/packages/task-manager/src/provider-stats.test.ts
+  7:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+  8:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+  9:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+
+✖ 3 problems (0 errors, 3 warnings)
+

package/.turbo/turbo-lint.log

@@ -0,0 +1,13 @@
+
+
+> @opentask/taskin-task-manager@2.0.0 lint /Users/sidarta/repositorios/taskin/packages/task-manager
+> eslint .
+
+
+/Users/sidarta/repositorios/taskin/packages/task-manager/src/provider-stats.test.ts
+  7:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+  8:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+  9:65  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
+
+✖ 3 problems (0 errors, 3 warnings)
+

package/.turbo/turbo-test.log

@@ -1,15 +1,15 @@
 
-> @opentask/taskin-task-manager@1.0.5 test /Users/sidarta/repositorios/taskin/packages/task-manager
+> @opentask/taskin-task-manager@2.0.0 test /Users/sidarta/repositorios/taskin/packages/task-manager
 > vitest run
 
 
- RUN  v4.0.16 /Users/sidarta/repositorios/taskin/packages/task-manager
+ RUN  v4.0.18 /Users/sidarta/repositorios/taskin/packages/task-manager
 
- ✓ src/provider-stats.test.ts (1 test) 1ms
- ✓ src/task-manager.test.ts (9 tests) 60ms
+ ✓ src/provider-stats.test.ts (1 test) 3ms
+ ✓ src/task-manager.test.ts (14 tests) 7ms
 
  Test Files  2 passed (2)
-      Tests  10 passed (10)
-   Start at  09:48:51
-   Duration  496ms (transform 59ms, setup 0ms, import 108ms, tests 62ms, environment 0ms)
+      Tests  15 passed (15)
+   Start at  13:46:24
+   Duration  429ms (transform 362ms, setup 0ms, import 414ms, tests 11ms, environment 0ms)
 

package/.turbo/turbo-typecheck.log

@@ -0,0 +1,4 @@
+
+> @opentask/taskin-task-manager@2.0.0 typecheck /Users/sidarta/repositorios/taskin/packages/task-manager
+> tsc --noEmit
+

package/CHANGELOG.md

@@ -1,5 +1,56 @@
 # @opentask/taskin-task-manager
 
+## 1.1.0
+
+### Minor Changes
+
+- feat(cli): implement review command with unified hook system
+
+  Add new `taskin review` command to prepare tasks for code review with automated quality checks.
+
+  **New Features:**
+  - Add `review` command to transition tasks from in-progress to in-review status
+  - Implement unified hook system supporting pre/during/post phases for all commands
+  - Add config-manager for .taskin.json hook configuration
+  - Add reviewTask() method to TaskManager
+  - Add comprehensive TypeScript schemas and types for hooks and configuration
+
+  **UX Improvements:**
+  - Standardize --open flag semantics across list and dashboard commands
+  - Add --browser/-b flag for launching browser in dashboard
+  - Remove redundant --filter-open and --filter-closed flags
+
+  **Build Fixes:**
+  - Add composite:true to CLI tsconfig for project references
+  - Add @modelcontextprotocol/sdk as direct dependency
+  - Fix VS Code TypeScript Language Server cache issues
+
+  **BREAKING CHANGE:**
+
+  The `--open` flag in `taskin dashboard` now filters open tasks (consistent with list command) instead of opening a browser. Use the new `--browser` or `-b` flag to open the browser automatically.
+
+  **New Files:**
+  - packages/cli/src/commands/review.ts (321 lines)
+  - packages/cli/src/lib/hook-runner.ts (119 lines)
+  - packages/cli/src/lib/hook-runner.test.ts (242 lines)
+  - packages/cli/src/lib/config-manager.ts (46 lines)
+  - packages/types-ts/src/taskin.schemas.ts (+137 lines)
+  - packages/types-ts/src/taskin.types.ts (+108 lines)
+
+  Closes #014
+
+### Patch Changes
+
+- Updated dependencies
+  - @opentask/taskin-types@1.1.0
+
+## 1.0.9
+
+### Patch Changes
+
+- Updated dependencies
+  - @opentask/taskin-types@1.0.6
+
 ## 1.0.8
 
 ### Patch Changes

package/dist/index.d.ts

@@ -1,3 +1,4 @@
-export type { Task, TaskStatus, TaskType, User } from '@opentask/taskin-types';
+export * from './metrics.types';
 export * from './task-manager';
 export * from './task-manager.types';
+export type { Task, TaskStatus, TaskType, User } from '@opentask/taskin-types';

package/dist/index.js

@@ -1,3 +1,4 @@
+export * from './metrics.types';
 export * from './task-manager';
 export * from './task-manager.types';
-//# sourceMappingURL=index.js.map
+//# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,gBAAgB,CAAC;AAC/B,cAAc,sBAAsB,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,iBAAiB,CAAC;AAChC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,sBAAsB,CAAC"}

package/dist/provider-stats.test.js

@@ -1,17 +1,17 @@
 import { describe, expect, it } from 'vitest';
 describe('IMetricsManager contract', () => {
-  it('accepts an object implementing IMetricsManager', async () => {
-    const metrics = {
-      getUserMetrics: async (userId) => ({ userId }),
-      getTeamMetrics: async (teamId) => ({ teamId }),
-      getTaskMetrics: async (taskId) => ({ taskId }),
-    };
-    expect(typeof metrics.getUserMetrics).toBe('function');
-    expect(typeof metrics.getTeamMetrics).toBe('function');
-    expect(typeof metrics.getTaskMetrics).toBe('function');
-    await expect(metrics.getUserMetrics('u1')).resolves.toBeDefined();
-    await expect(metrics.getTeamMetrics('t1')).resolves.toBeDefined();
-    await expect(metrics.getTaskMetrics('task1')).resolves.toBeDefined();
-  });
+    it('accepts an object implementing IMetricsManager', async () => {
+        const metrics = {
+            getUserMetrics: async (userId) => ({ userId }),
+            getTeamMetrics: async (teamId) => ({ teamId }),
+            getTaskMetrics: async (taskId) => ({ taskId }),
+        };
+        expect(typeof metrics.getUserMetrics).toBe('function');
+        expect(typeof metrics.getTeamMetrics).toBe('function');
+        expect(typeof metrics.getTaskMetrics).toBe('function');
+        await expect(metrics.getUserMetrics('u1')).resolves.toBeDefined();
+        await expect(metrics.getTeamMetrics('t1')).resolves.toBeDefined();
+        await expect(metrics.getTaskMetrics('task1')).resolves.toBeDefined();
+    });
 });
-//# sourceMappingURL=provider-stats.test.js.map
+//# sourceMappingURL=provider-stats.test.js.map

package/dist/task-manager.d.ts

@@ -1,16 +1,10 @@
-import type {
-  CreateTaskOptions,
-  CreateTaskResult,
-  ITaskManager,
-  ITaskProvider,
-  LintResult,
-  TaskFile,
-} from './task-manager.types';
+import type { CreateTaskOptions, CreateTaskResult, ITaskManager, ITaskProvider, LintResult, TaskFile } from './task-manager.types';
 export declare class TaskManager implements ITaskManager {
-  private taskProvider;
-  constructor(taskProvider: ITaskProvider);
-  startTask(taskId: string): Promise<TaskFile>;
-  finishTask(taskId: string): Promise<TaskFile>;
-  createTask(options: CreateTaskOptions): Promise<CreateTaskResult>;
-  lint(fix?: boolean): Promise<LintResult>;
+    private taskProvider;
+    constructor(taskProvider: ITaskProvider);
+    startTask(taskId: string): Promise<TaskFile>;
+    finishTask(taskId: string): Promise<TaskFile>;
+    reviewTask(taskId: string): Promise<TaskFile>;
+    createTask(options: CreateTaskOptions): Promise<CreateTaskResult>;
+    lint(fix?: boolean): Promise<LintResult>;
 }

package/dist/task-manager.js

@@ -1,37 +1,49 @@
 export class TaskManager {
-  taskProvider;
-  constructor(taskProvider) {
-    this.taskProvider = taskProvider;
-  }
-  async startTask(taskId) {
-    const task = await this.taskProvider.findTask(taskId);
-    if (!task) {
-      throw new Error(`Task with ID '${taskId}' not found.`);
+    taskProvider;
+    constructor(taskProvider) {
+        this.taskProvider = taskProvider;
     }
-    if (task.status === 'in-progress') {
-      throw new Error(`Task '${taskId}' is already in progress.`);
+    async startTask(taskId) {
+        const task = await this.taskProvider.findTask(taskId);
+        if (!task) {
+            throw new Error(`Task with ID '${taskId}' not found.`);
+        }
+        if (task.status === 'in-progress') {
+            throw new Error(`Task '${taskId}' is already in progress.`);
+        }
+        if (task.status === 'done') {
+            throw new Error(`Task '${taskId}' is already done.`);
+        }
+        const updatedTask = { ...task, status: 'in-progress' };
+        await this.taskProvider.updateTask(updatedTask);
+        return updatedTask;
     }
-    if (task.status === 'done') {
-      throw new Error(`Task '${taskId}' is already done.`);
+    async finishTask(taskId) {
+        const task = await this.taskProvider.findTask(taskId);
+        if (!task) {
+            throw new Error(`Task with ID '${taskId}' not found.`);
+        }
+        const updatedTask = { ...task, status: 'done' };
+        await this.taskProvider.updateTask(updatedTask);
+        return updatedTask;
     }
-    const updatedTask = { ...task, status: 'in-progress' };
-    await this.taskProvider.updateTask(updatedTask);
-    return updatedTask;
-  }
-  async finishTask(taskId) {
-    const task = await this.taskProvider.findTask(taskId);
-    if (!task) {
-      throw new Error(`Task with ID '${taskId}' not found.`);
+    async reviewTask(taskId) {
+        const task = await this.taskProvider.findTask(taskId);
+        if (!task) {
+            throw new Error(`Task with ID '${taskId}' not found.`);
+        }
+        if (task.status !== 'in-progress') {
+            throw new Error(`Task '${taskId}' must be in 'in-progress' status to be reviewed. Current status: ${task.status}`);
+        }
+        const updatedTask = { ...task, status: 'in-review' };
+        await this.taskProvider.updateTask(updatedTask);
+        return updatedTask;
+    }
+    async createTask(options) {
+        return this.taskProvider.createTask(options);
+    }
+    async lint(fix) {
+        return this.taskProvider.lint(fix);
     }
-    const updatedTask = { ...task, status: 'done' };
-    await this.taskProvider.updateTask(updatedTask);
-    return updatedTask;
-  }
-  async createTask(options) {
-    return this.taskProvider.createTask(options);
-  }
-  async lint(fix) {
-    return this.taskProvider.lint(fix);
-  }
 }
-//# sourceMappingURL=task-manager.js.map
+//# sourceMappingURL=task-manager.js.map

package/dist/task-manager.js.map

@@ -1 +1 @@
-{"version":3,"file":"task-manager.js","sourceRoot":"","sources":["../src/task-manager.ts"],"names":[],"mappings":"AASA,MAAM,OAAO,WAAW;IACF;IAApB,YAAoB,YAA2B;QAA3B,iBAAY,GAAZ,YAAY,CAAe;IAAG,CAAC;IAEnD,KAAK,CAAC,SAAS,CAAC,MAAc;QAC5B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAEtD,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,cAAc,CAAC,CAAC;QACzD,CAAC;QAED,IAAI,IAAI,CAAC,MAAM,KAAK,aAAa,EAAE,CAAC;YAClC,MAAM,IAAI,KAAK,CAAC,SAAS,MAAM,2BAA2B,CAAC,CAAC;QAC9D,CAAC;QAED,IAAI,IAAI,CAAC,MAAM,KAAK,MAAM,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CAAC,SAAS,MAAM,oBAAoB,CAAC,CAAC;QACvD,CAAC;QAED,MAAM,WAAW,GAAa,EAAE,GAAG,IAAI,EAAE,MAAM,EAAE,aAAa,EAAE,CAAC;QACjE,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,MAAc;QAC7B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAEtD,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,cAAc,CAAC,CAAC;QACzD,CAAC;QAED,MAAM,WAAW,GAAa,EAAE,GAAG,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;QAC1D,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,OAA0B;QACzC,OAAO,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;IAC/C,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,GAAa;QACtB,OAAO,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACrC,CAAC;CACF"}
+{"version":3,"file":"task-manager.js","sourceRoot":"","sources":["../src/task-manager.ts"],"names":[],"mappings":"AASA,MAAM,OAAO,WAAW;IACF;IAApB,YAAoB,YAA2B;QAA3B,iBAAY,GAAZ,YAAY,CAAe;IAAG,CAAC;IAEnD,KAAK,CAAC,SAAS,CAAC,MAAc;QAC5B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAEtD,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,cAAc,CAAC,CAAC;QACzD,CAAC;QAED,IAAI,IAAI,CAAC,MAAM,KAAK,aAAa,EAAE,CAAC;YAClC,MAAM,IAAI,KAAK,CAAC,SAAS,MAAM,2BAA2B,CAAC,CAAC;QAC9D,CAAC;QAED,IAAI,IAAI,CAAC,MAAM,KAAK,MAAM,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CAAC,SAAS,MAAM,oBAAoB,CAAC,CAAC;QACvD,CAAC;QAED,MAAM,WAAW,GAAa,EAAE,GAAG,IAAI,EAAE,MAAM,EAAE,aAAa,EAAE,CAAC;QACjE,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,MAAc;QAC7B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAEtD,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,cAAc,CAAC,CAAC;QACzD,CAAC;QAED,MAAM,WAAW,GAAa,EAAE,GAAG,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;QAC1D,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,MAAc;QAC7B,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,YAAY,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAEtD,IAAI,CAAC,IAAI,EAAE,CAAC;YACV,MAAM,IAAI,KAAK,CAAC,iBAAiB,MAAM,cAAc,CAAC,CAAC;QACzD,CAAC;QAED,IAAI,IAAI,CAAC,MAAM,KAAK,aAAa,EAAE,CAAC;YAClC,MAAM,IAAI,KAAK,CACb,SAAS,MAAM,qEAAqE,IAAI,CAAC,MAAM,EAAE,CAClG,CAAC;QACJ,CAAC;QAED,MAAM,WAAW,GAAa,EAAE,GAAG,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC;QAC/D,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,KAAK,CAAC,UAAU,CAAC,OAA0B;QACzC,OAAO,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;IAC/C,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,GAAa;QACtB,OAAO,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACrC,CAAC;CACF"}

package/dist/task-manager.mock.js

@@ -1,21 +1,22 @@
 import { vi } from 'vitest';
 export const createMockTask = (overrides) => ({
-  content: '# Task 001 - Implement feature',
-  createdAt: new Date().toISOString(),
-  description: 'A test feature',
-  filePath: '/tasks/task-001.md',
-  id: 'task-001',
-  status: 'pending',
-  title: 'Implement feature',
-  type: 'feat',
-  userId: 'user-123',
-  ...overrides,
+    content: '# Task 001 - Implement feature',
+    createdAt: new Date().toISOString(),
+    description: 'A test feature',
+    filePath: '/tasks/task-001.md',
+    id: 'task-001',
+    status: 'pending',
+    title: 'Implement feature',
+    type: 'feat',
+    userId: 'user-123',
+    ...overrides,
 });
 export const createMockTaskProvider = () => ({
-  findTask: vi.fn(),
-  getAllTasks: vi.fn(),
-  updateTask: vi.fn(),
-  createTask: vi.fn(),
-  lint: vi.fn(),
+    findTask: vi.fn(),
+    getAllTasks: vi.fn(),
+    updateTask: vi.fn(),
+    createTask: vi.fn(),
+    lint: vi.fn(),
+    initialize: vi.fn(),
 });
-//# sourceMappingURL=task-manager.mock.js.map
+//# sourceMappingURL=task-manager.mock.js.map

package/dist/task-manager.mock.js.map

@@ -1 +1 @@
-{"version":3,"file":"task-manager.mock.js","sourceRoot":"","sources":["../src/task-manager.mock.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG5B,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,SAA6B,EAAY,EAAE,CAAC,CAAC;IAC1E,OAAO,EAAE,gCAAgC;IACzC,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;IACnC,WAAW,EAAE,gBAAgB;IAC7B,QAAQ,EAAE,oBAAoB;IAC9B,EAAE,EAAE,UAAqC;IACzC,MAAM,EAAE,SAAS;IACjB,KAAK,EAAE,mBAAmB;IAC1B,IAAI,EAAE,MAAM;IACZ,MAAM,EAAE,UAAU;IAClB,GAAG,SAAS;CACb,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,sBAAsB,GAAG,GAAkB,EAAE,CAAC,CAAC;IAC1D,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;IACjB,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;IACpB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;IACnB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;IACnB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;CACd,CAAC,CAAC"}
+{"version":3,"file":"task-manager.mock.js","sourceRoot":"","sources":["../src/task-manager.mock.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG5B,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,SAA6B,EAAY,EAAE,CAAC,CAAC;IAC1E,OAAO,EAAE,gCAAgC;IACzC,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;IACnC,WAAW,EAAE,gBAAgB;IAC7B,QAAQ,EAAE,oBAAoB;IAC9B,EAAE,EAAE,UAAqC;IACzC,MAAM,EAAE,SAAS;IACjB,KAAK,EAAE,mBAAmB;IAC1B,IAAI,EAAE,MAAM;IACZ,MAAM,EAAE,UAAU;IAClB,GAAG,SAAS;CACb,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,sBAAsB,GAAG,GAAkB,EAAE,CAAC,CAAC;IAC1D,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;IACjB,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;IACpB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;IACnB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;IACnB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;IACb,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;CACpB,CAAC,CAAC"}

package/dist/task-manager.test.js

@@ -2,103 +2,124 @@ import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { TaskManager } from './task-manager';
 import { createMockTask, createMockTaskProvider } from './task-manager.mock';
 describe('TaskManager', () => {
-  let taskManager;
-  let mockTaskProvider;
-  let mockTask;
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockTaskProvider = createMockTaskProvider();
-    mockTask = createMockTask();
-    taskManager = new TaskManager(mockTaskProvider);
-  });
-  describe('startTask', () => {
-    it('should start a pending task', async () => {
-      mockTaskProvider.findTask.mockResolvedValue(mockTask);
-      const updatedTask = await taskManager.startTask('task-001');
-      expect(mockTaskProvider.findTask).toHaveBeenCalledWith('task-001');
-      expect(mockTaskProvider.updateTask).toHaveBeenCalledWith(
-        expect.objectContaining({ status: 'in-progress' }),
-      );
-      expect(updatedTask.status).toBe('in-progress');
+    let taskManager;
+    let mockTaskProvider;
+    let mockTask;
+    beforeEach(() => {
+        vi.clearAllMocks();
+        mockTaskProvider = createMockTaskProvider();
+        mockTask = createMockTask();
+        taskManager = new TaskManager(mockTaskProvider);
     });
-    it('should throw an error if task is not found', async () => {
-      mockTaskProvider.findTask.mockResolvedValue(undefined);
-      await expect(taskManager.startTask('not-found')).rejects.toThrow(
-        "Task with ID 'not-found' not found.",
-      );
+    describe('startTask', () => {
+        it('should start a pending task', async () => {
+            mockTaskProvider.findTask.mockResolvedValue(mockTask);
+            const updatedTask = await taskManager.startTask('task-001');
+            expect(mockTaskProvider.findTask).toHaveBeenCalledWith('task-001');
+            expect(mockTaskProvider.updateTask).toHaveBeenCalledWith(expect.objectContaining({ status: 'in-progress' }));
+            expect(updatedTask.status).toBe('in-progress');
+        });
+        it('should throw an error if task is not found', async () => {
+            mockTaskProvider.findTask.mockResolvedValue(undefined);
+            await expect(taskManager.startTask('not-found')).rejects.toThrow("Task with ID 'not-found' not found.");
+        });
+        it('should throw an error if task is already in progress', async () => {
+            const inProgressTask = { ...mockTask, status: 'in-progress' };
+            mockTaskProvider.findTask.mockResolvedValue(inProgressTask);
+            await expect(taskManager.startTask('task-001')).rejects.toThrow("Task 'task-001' is already in progress.");
+        });
     });
-    it('should throw an error if task is already in progress', async () => {
-      const inProgressTask = { ...mockTask, status: 'in-progress' };
-      mockTaskProvider.findTask.mockResolvedValue(inProgressTask);
-      await expect(taskManager.startTask('task-001')).rejects.toThrow(
-        "Task 'task-001' is already in progress.",
-      );
+    describe('finishTask', () => {
+        it('should finish an in-progress task', async () => {
+            const inProgressTask = { ...mockTask, status: 'in-progress' };
+            mockTaskProvider.findTask.mockResolvedValue(inProgressTask);
+            const updatedTask = await taskManager.finishTask('task-001');
+            expect(mockTaskProvider.updateTask).toHaveBeenCalledWith(expect.objectContaining({ status: 'done' }));
+            expect(updatedTask.status).toBe('done');
+        });
     });
-  });
-  describe('finishTask', () => {
-    it('should finish an in-progress task', async () => {
-      const inProgressTask = { ...mockTask, status: 'in-progress' };
-      mockTaskProvider.findTask.mockResolvedValue(inProgressTask);
-      const updatedTask = await taskManager.finishTask('task-001');
-      expect(mockTaskProvider.updateTask).toHaveBeenCalledWith(
-        expect.objectContaining({ status: 'done' }),
-      );
-      expect(updatedTask.status).toBe('done');
+    describe('reviewTask', () => {
+        it('should change status to in-review for an in-progress task', async () => {
+            const inProgressTask = { ...mockTask, status: 'in-progress' };
+            mockTaskProvider.findTask.mockResolvedValue(inProgressTask);
+            const updatedTask = await taskManager.reviewTask('task-001');
+            expect(mockTaskProvider.findTask).toHaveBeenCalledWith('task-001');
+            expect(mockTaskProvider.updateTask).toHaveBeenCalledWith(expect.objectContaining({ status: 'in-review' }));
+            expect(updatedTask.status).toBe('in-review');
+        });
+        it('should throw an error if task is not found', async () => {
+            mockTaskProvider.findTask.mockResolvedValue(undefined);
+            await expect(taskManager.reviewTask('not-found')).rejects.toThrow("Task with ID 'not-found' not found.");
+        });
+        it('should throw an error if task is not in-progress', async () => {
+            const pendingTask = { ...mockTask, status: 'pending' };
+            mockTaskProvider.findTask.mockResolvedValue(pendingTask);
+            await expect(taskManager.reviewTask('task-001')).rejects.toThrow("Task 'task-001' must be in 'in-progress' status to be reviewed");
+        });
+        it('should throw an error if task is already done', async () => {
+            const doneTask = { ...mockTask, status: 'done' };
+            mockTaskProvider.findTask.mockResolvedValue(doneTask);
+            await expect(taskManager.reviewTask('task-001')).rejects.toThrow("Task 'task-001' must be in 'in-progress' status to be reviewed");
+        });
+        it('should throw an error if task is already in-review', async () => {
+            const reviewTask = { ...mockTask, status: 'in-review' };
+            mockTaskProvider.findTask.mockResolvedValue(reviewTask);
+            await expect(taskManager.reviewTask('task-001')).rejects.toThrow("Task 'task-001' must be in 'in-progress' status to be reviewed");
+        });
     });
-  });
-  describe('createTask', () => {
-    it('should delegate task creation to provider', async () => {
-      const createOptions = { title: 'New Task', type: 'feat' };
-      const createResult = {
-        task: mockTask,
-        taskId: '001',
-        filePath: '/tasks/task-001.md',
-      };
-      mockTaskProvider.createTask.mockResolvedValue(createResult);
-      const result = await taskManager.createTask(createOptions);
-      expect(mockTaskProvider.createTask).toHaveBeenCalledWith(createOptions);
-      expect(result).toEqual(createResult);
+    describe('createTask', () => {
+        it('should delegate task creation to provider', async () => {
+            const createOptions = { title: 'New Task', type: 'feat' };
+            const createResult = {
+                task: mockTask,
+                taskId: '001',
+                filePath: '/tasks/task-001.md',
+            };
+            mockTaskProvider.createTask.mockResolvedValue(createResult);
+            const result = await taskManager.createTask(createOptions);
+            expect(mockTaskProvider.createTask).toHaveBeenCalledWith(createOptions);
+            expect(result).toEqual(createResult);
+        });
+        it('should pass through description and assignee to provider', async () => {
+            const createOptions = {
+                title: 'New Task',
+                type: 'fix',
+                description: 'Fix something',
+                assignee: 'john-doe',
+            };
+            const createResult = {
+                task: mockTask,
+                taskId: '002',
+                filePath: '/tasks/task-002.md',
+            };
+            mockTaskProvider.createTask.mockResolvedValue(createResult);
+            const result = await taskManager.createTask(createOptions);
+            expect(mockTaskProvider.createTask).toHaveBeenCalledWith(createOptions);
+            expect(result.taskId).toBe('002');
+        });
     });
-    it('should pass through description and assignee to provider', async () => {
-      const createOptions = {
-        title: 'New Task',
-        type: 'fix',
-        description: 'Fix something',
-        assignee: 'john-doe',
-      };
-      const createResult = {
-        task: mockTask,
-        taskId: '002',
-        filePath: '/tasks/task-002.md',
-      };
-      mockTaskProvider.createTask.mockResolvedValue(createResult);
-      const result = await taskManager.createTask(createOptions);
-      expect(mockTaskProvider.createTask).toHaveBeenCalledWith(createOptions);
-      expect(result.taskId).toBe('002');
+    describe('lint', () => {
+        it('should delegate lint without fix to provider', async () => {
+            const lintResult = { errors: 0, warnings: 2, details: [] };
+            mockTaskProvider.lint.mockResolvedValue(lintResult);
+            const result = await taskManager.lint();
+            expect(mockTaskProvider.lint).toHaveBeenCalledWith(undefined);
+            expect(result).toEqual(lintResult);
+        });
+        it('should delegate lint with fix=true to provider', async () => {
+            const lintResult = { errors: 0, warnings: 0, details: [] };
+            mockTaskProvider.lint.mockResolvedValue(lintResult);
+            const result = await taskManager.lint(true);
+            expect(mockTaskProvider.lint).toHaveBeenCalledWith(true);
+            expect(result).toEqual(lintResult);
+        });
+        it('should delegate lint with fix=false to provider', async () => {
+            const lintResult = { errors: 1, warnings: 2, details: [] };
+            mockTaskProvider.lint.mockResolvedValue(lintResult);
+            const result = await taskManager.lint(false);
+            expect(mockTaskProvider.lint).toHaveBeenCalledWith(false);
+            expect(result).toEqual(lintResult);
+        });
     });
-  });
-  describe('lint', () => {
-    it('should delegate lint without fix to provider', async () => {
-      const lintResult = { errors: 0, warnings: 2, details: [] };
-      mockTaskProvider.lint.mockResolvedValue(lintResult);
-      const result = await taskManager.lint();
-      expect(mockTaskProvider.lint).toHaveBeenCalledWith(undefined);
-      expect(result).toEqual(lintResult);
-    });
-    it('should delegate lint with fix=true to provider', async () => {
-      const lintResult = { errors: 0, warnings: 0, details: [] };
-      mockTaskProvider.lint.mockResolvedValue(lintResult);
-      const result = await taskManager.lint(true);
-      expect(mockTaskProvider.lint).toHaveBeenCalledWith(true);
-      expect(result).toEqual(lintResult);
-    });
-    it('should delegate lint with fix=false to provider', async () => {
-      const lintResult = { errors: 1, warnings: 2, details: [] };
-      mockTaskProvider.lint.mockResolvedValue(lintResult);
-      const result = await taskManager.lint(false);
-      expect(mockTaskProvider.lint).toHaveBeenCalledWith(false);
-      expect(result).toEqual(lintResult);
-    });
-  });
 });
-//# sourceMappingURL=task-manager.test.js.map
+//# sourceMappingURL=task-manager.test.js.map