@opentask/taskin-task-manager 2.0.0 → 3.0.0

package/CHANGELOG.md

@@ -1,5 +1,253 @@
 # @opentask/taskin-task-manager
 
+## 3.0.0
+
+### Major Changes
+
+- b4b259e: Faz a marca `TaskId` valer algo e passa a validar id na borda.
+  
+  `TaskIdSchema` exigia `.uuid()`, mas id de task é a parte numérica do nome do
+  arquivo — `task-020-foo.md` produz `020`. Nenhuma task real passava no schema:
+  `TaskSchema.safeParse({ id: '020' })` falhava com "Invalid uuid", e o único
+  caminho até um `TaskId` era `satisfies string as TaskId`, um cast. Uma marca que
+  só se alcança por cast não carrega informação: custa cast em toda fronteira e
+  não previne nada.
+  
+  ## Ids
+  
+  - `TaskIdSchema` passa a exigir `/^\d+$/`, a forma que o provider produz.
+  - `parseTaskId` e `parseGroupId` são o único caminho suportado até as marcas —
+    eles validam e lançam. Os casts saíram do provider.
+  - `groupId` no `TaskSchema` passa a ser `GroupId`, marca que já existia e não era
+    usada no domínio.
+  
+  Ao trocar a regex, **11 testes do `types-ts` quebraram de uma vez** — todos por
+  fixture de UUID. As fixtures agora usam ids de verdade e o UUID entra como caso
+  rejeitado.
+  
+  ## Contrato
+  
+  `findTask`, `startTask`, `pauseTask`, `finishTask` e `reviewTask` passam a
+  receber `TaskId` em vez de `string`, em `ITaskProvider`, `ITaskManager`, na
+  classe `TaskManager` e no `FileSystemTaskProvider`. Na CLI, `normalizeTaskId` é a
+  fronteira: aceita o que a pessoa digita (`20`, `020`, `task-020`), normaliza e
+  valida. Substitui o `replace(/^task-/, '').padStart(3, '0')` que estava copiado
+  em quatro comandos.
+  
+  Os membros de `ITaskProvider` e `ITaskManager` passaram a ser propriedades de
+  função em vez de métodos. TypeScript trata método como bivariante mesmo sob
+  `strictFunctionTypes`, e isso deixava `ITaskProvider<TaskFile>` ser atribuído a
+  `ITaskProvider<Task>` — compila e depois quebra em `updateTask`, que abre com
+  `fs.readFile(task.filePath)`.
+  
+  ## Fronteiras que validam
+  
+  - **WebSocket**: `handleUpdateRequest` fazia `message.payload as TTask` e
+    entregava JSON não confiável ao provider. Agora relê a task armazenada e aplica
+    só o bloco de priorização, via `applyTaskUpdate` (módulo novo, com teste). Os
+    campos são substituídos em bloco de propósito: `JSON.stringify` descarta
+    `undefined`, então "desagrupar" chega como chave ausente, não como `null`.
+  - **WebSocket e MCP**: id vindo do cliente passa por `safeParse` e devolve erro
+    claro, em vez de deixar `ZodError` vazar pelo catch genérico.
+  - `getAllTasks` ignora `task-*.md` sem id numérico. Antes viravam task fantasma
+    de id `'unknown'` — e duas delas colidiam no mesmo id.
+  
+  ## Breaking changes
+  
+  **`@opentask/taskin-types`**
+  
+  - `TaskIdSchema` rejeita UUID e aceita `/^\d+$/`. Quem persistiu id em outro
+    formato não passa mais em `TaskSchema`.
+  - `TaskSchema.groupId` é `GroupId`, não `string`.
+  
+  **`@opentask/taskin-task-manager`**
+  
+  - Os parâmetros de id são `TaskId`. Um `string` solto não compila mais — use
+    `parseTaskId`.
+  - `CreateTaskResult.taskId` foi removido: duplicava `task.id` com tipo mais
+    fraco. O provider de arquivos segue devolvendo `filePath` em
+    `CreateTaskFileResult`.
+  - `ITaskProvider` e `ITaskManager` declaram propriedades de função. Classes que
+    os implementam continuam compilando; o que deixa de compilar é atribuir um
+    provider específico onde se espera o genérico — que era exatamente o bug.
+  
+  **`@opentask/taskin-design-vue`**
+  
+  - `TaskId` e `GroupId` são reexportados de `@opentask/taskin-types` em vez de
+    redeclarados. Duas marcas para o mesmo conceito obrigavam um cast em toda
+    fronteira, e foi um desses casts que engoliu campos calado.
+  
+  ## API nova
+  
+  `@opentask/taskin-types` exporta `parseTaskId`, `parseGroupId`,
+  `TaskPrioritizationUpdateSchema` (a fatia mutável de uma task, derivada de
+  `TaskSchema` para não poder divergir) e `NOTIFICATION_PROVIDERS` /
+  `NotificationProviderNameSchema`.
+- 30b3e4a: Torna o provider e o manager genéricos sobre a forma da task, e promove `paused`
+  a status de primeira classe.
+  
+  ## Provider agnóstico
+  
+  `TaskFile` era declarado no pacote agnóstico e o `ITaskProvider` inteiro era
+  tipado nele, obrigando qualquer provider não-arquivo (GitHub, Redmine) a inventar
+  `content` e `filePath`. Agora `ITaskProvider` e `ITaskManager` são genéricos sobre
+  `TTask extends Task`, com default `Task`, e `TaskFile` mora no
+  `file-system-task-provider`.
+  
+  Os call sites não mudam — `new TaskManager(provider)` infere a forma sozinho.
+  
+  Junto disso:
+  
+  - o strip em runtime (`toTask`) saiu do manager; campos específicos do provider
+    agora sobrevivem às transições de status
+  - corrigido broadcast do WebSocket que podia emitir `payload: undefined` após
+    start/finish, deixando os clientes dessincronizados sem erro nenhum
+  - `pinia` e dashboard passam a falar `Task`; o provider de arquivos projeta
+    `content` em `description` na fronteira
+  
+  ## Status `paused`
+  
+  O comando `pause` gravava `pending`, apagando a diferença entre "nunca começou" e
+  "começou e parou". Agora `paused` existe no domínio, com `ITaskManager.pauseTask`
+  e retomada via `startTask`.
+  
+  As quatro listas de status que eram mantidas à mão (schema, linter da CLI,
+  `task-validator`, metrics adapter) agora derivam de `TASK_STATUSES`. A divergência
+  entre elas já era bug: `in-review`, que o próprio `reviewTask` grava, era
+  rejeitado pelo linter, ignorado pelo metrics adapter e sumia tanto de
+  `taskin list --open` quanto de `--closed`.
+  
+  ## Breaking changes
+  
+  **`@opentask/taskin-task-manager`**
+  
+  - `ITaskProvider` e `ITaskManager` ganharam parâmetro de tipo. O default mantém o
+    uso comum compilando, mas quem *implementa* `ITaskManager` precisa adicionar
+    `pauseTask`.
+  - `TaskFile` saiu daqui. Importe de `@opentask/taskin-file-system-provider`.
+  - `CreateTaskResult` não tem mais `filePath`. O provider de arquivos devolve
+    `CreateTaskFileResult`, que o mantém.
+  
+  **`@opentask/taskin-types`**
+  
+  - `TaskStatus` e `TASK_STATUSES` ganharam `'paused'`. Consumidores exaustivos
+    (`Record<TaskStatus, T>`, `switch` sem `default`) precisam tratar o caso novo.
+  
+  **`@opentask/taskin-design-vue`**
+  
+  - `TaskStatus` ganhou `'in-review'` e `'canceled'` para alinhar com o domínio,
+    com o mesmo efeito sobre consumidores exaustivos.
+  
+  **`@opentask/taskin-task-provider-pinia`**
+  
+  - O store guarda `Task` em vez de `TaskFile`: `tasks`, `findTask`, `getAllTasks` e
+    `updateTask` não expõem mais `content` nem `filePath`. Use `description`.
+  
+  **`@opentask/taskin-task-server-ws`**
+  
+  - `TaskWebSocketServer` e `TaskServerConfig` ganharam parâmetro de tipo (com
+    default). `MockTaskProvider` recebe `Task[]` em vez de `TaskFile[]`.
+  
+  **`@opentask/taskin-task-server-mcp`**
+  
+  - `MockMCPTaskManager.getAllTasks()` e `getTask()` devolvem `Task`, sem `filePath`
+    nem `content`.
+
+### Minor Changes
+
+- 2240253: Faz o `provider.type` do `.taskin.json` valer, e move a porta `IUserRegistry`
+  para o pacote agnostico.
+  
+  `provider.type` era lido **apenas** pelo `init`. Todos os comandos instanciavam
+  `new FileSystemTaskProvider(...)` e `new UserRegistry(...)` direto, em treze
+  lugares, entao escolher outro provider na configuracao nao tinha efeito nenhum —
+  foi o que deixou a task-002 (Redmine) parada mesmo com a spec pronta.
+  
+  ## Factory
+  
+  `resolveTaskProvider()` (em `packages/cli/src/lib/provider-factory/`) e agora o
+  unico lugar que nomeia um provider concreto, e devolve o par provider + user
+  registry — os dois eram construidos juntos em cada comando, e drifitaram: `list`
+  apontava o registry para a raiz do projeto em vez de `.taskin/`, entao todo
+  assignee caia silenciosamente em `createTemporaryUser` e o registro real nunca
+  era lido.
+  
+  O mapa `PROVIDER_BUILDERS` e injetavel, o que da o seam para testar que
+  `provider.type` e respeitado sem precisar de um provider real.
+  
+  `init` e a unica excecao, documentada no codigo: ele roda antes de existir
+  `.taskin.json` e e ele quem escreve o `provider.type`, entao usar a factory ali
+  seria circular.
+  
+  ## Porta `IUserRegistry`
+  
+  A interface era declarada dentro do `file-system-task-provider`, entao qualquer
+  registry de outro provider (GitHub, Redmine) teria que importar do provider de
+  arquivos para implementa-la — mesmo defeito de fronteira que a task-031 corrigiu
+  para o `TaskFile`, na direcao inversa. Agora mora no `task-manager`, ao lado do
+  `ITaskProvider`, e o contract test viaja com ela: qualquer implementacao roda
+  `runUserRegistryContractTests` via o subpath novo
+  `@opentask/taskin-task-manager/testing` (subpath separado para que `vitest` nao
+  entre no grafo de import de runtime).
+  
+  A implementacao file-backed (`UserRegistry`, `users-file-location`,
+  `.taskin/.taskin-users.json`) fica onde esta: e backing store de uma
+  implementacao, nao configuracao de projeto.
+  
+  ## Criacao de task sai do comando
+  
+  `new.ts` tinha uma copia inteira da criacao de task — numeracao `max(ids)+1`,
+  slug do titulo, template markdown — que e semantica de sistema de arquivos. Num
+  provider remoto o id vem do proprio store (o numero da issue). Agora o comando
+  chama `provider.createTask()` e so consome o resultado; o `generateTaskMarkdown`
+  duplicado saiu, ficando o do provider, que e i18n-aware.
+  
+  ## Credencial via ambiente
+  
+  A config do provider passa por expansao de `${VAR}` (reusando o resolver que a
+  config de notificacoes ja usava), porque todo provider remoto precisa de
+  credencial e o `.taskin.json` e versionado.
+  
+  ## Breaking changes
+  
+  **`taskin`**
+  
+  - `createTaskin()` e `getTaskin()` agora devolvem `Promise<Taskin>`. Qual
+    provider usar e pergunta de runtime, e o user registry precisa ser carregado
+    antes de qualquer leitura de task.
+  - `provider.type` diferente de `fs` agora **falha com mensagem explicita** em vez
+    de silenciosamente usar o provider de arquivos. Quem tinha type errado na
+    configuracao passa a ver o erro.
+  - `Taskin` recebe `ITaskProvider`/`ITaskManager` em vez de
+    `FileSystemTaskProvider`/`TaskManager` concretos.
+  
+  **`@opentask/taskin-file-system-provider`**
+  
+  - `IUserRegistry` nao e mais exportada daqui. Importe de
+    `@opentask/taskin-task-manager`.
+  - `runUserRegistryContractTests` mudou de lugar: importe de
+    `@opentask/taskin-task-manager/testing`.
+  - `FileSystemTaskProvider` e `FileSystemMetricsAdapter` recebem `IUserRegistry`
+    em vez da classe `UserRegistry`.
+
+### Patch Changes
+
+- Updated dependencies [b4b259e]
+- Updated dependencies [30b3e4a]
+- Updated dependencies [2f6d046]
+  - @opentask/taskin-types@2.0.0
+
+## 2.0.1
+
+### Patch Changes
+
+- Remove unnecessary install scripts that caused pnpm build script warnings
+
+  Removed `install` scripts from all packages that only printed echo messages. These scripts were unnecessary since packages are already pre-built and included in the published bundle. This eliminates the "Ignored build scripts" warning when installing taskin in external projects.
+
+- Updated dependencies
+  - @opentask/taskin-types@1.1.1
+
 ## 1.1.0
 
 ### Minor Changes

package/dist/index.d.ts

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

package/dist/index.js

@@ -1,4 +1,5 @@
-export * from './metrics.types';
-export * from './task-manager';
-export * from './task-manager.types';
+export * from './metrics.types.js';
+export * from './task-manager.js';
+export * from './task-manager.types.js';
+export * from './user-registry.types.js';
 //# 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,iBAAiB,CAAC;AAChC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,sBAAsB,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,cAAc,iBAAiB,CAAC;AAChC,cAAc,gBAAgB,CAAC;AAC/B,cAAc,sBAAsB,CAAC;AACrC,cAAc,uBAAuB,CAAC"}

package/dist/metrics.types.d.ts

@@ -1,9 +1,4 @@
-import type {
-  StatsQuery,
-  TaskStats,
-  TeamStats,
-  UserStats,
-} from '@opentask/taskin-types';
+import type { StatsQuery, TaskStats, TeamStats, UserStats } from '@opentask/taskin-types';
 /**
  * Manages metrics and statistics for users, teams, and tasks.
  * Aggregates data from Git history and task files to provide productivity insights.
@@ -22,71 +17,67 @@ import type {
  * @public
  */
 export interface IMetricsManager {
-  /**
-   * Get productivity metrics for a specific user.
-   *
-   * Includes code metrics (commits, lines of code), temporal patterns
-   * (day of week, time of day), and engagement statistics (completion rate, streaks).
-   *
-   * @param userId - User identifier (username or registry ID)
-   * @param query - Optional query parameters to filter results
-   * @param query.period - Time period to analyze ('day' | 'week' | 'month' | 'year')
-   * @returns Promise resolving to user statistics
-   *
-   * @example
-   * ```ts
-   * const stats = await metrics.getUserMetrics('alice', { period: 'month' });
-   * console.log(`Tasks completed: ${stats.contributionMetrics.tasksCompleted}`);
-   * console.log(`Completion rate: ${(stats.engagementMetrics.completionRate * 100).toFixed(1)}%`);
-   * ```
-   */
-  getUserMetrics(userId: string, query?: StatsQuery): Promise<UserStats>;
-  /**
-   * Get aggregated metrics for an entire team.
-   *
-   * Provides overview of team productivity with breakdown by individual contributors.
-   * Useful for team retrospectives, capacity planning, and identifying top contributors.
-   *
-   * @param teamId - Team identifier
-   * @param query - Optional query parameters to filter results
-   * @returns Promise resolving to team statistics with per-contributor breakdown
-   *
-   * @example
-   * ```ts
-   * const stats = await metrics.getTeamMetrics('frontend', { period: 'week' });
-   * console.log(`Total commits: ${stats.totalCommits}`);
-   * console.log(`Contributors: ${stats.totalContributors}`);
-   * stats.contributors.forEach(c => {
-   *   console.log(`${c.username}: ${c.commits} commits, ${c.tasksCompleted} tasks`);
-   * });
-   * ```
-   */
-  getTeamMetrics(teamId: string, query?: StatsQuery): Promise<TeamStats>;
-  /**
-   * Get detailed metrics for a specific task.
-   *
-   * Includes timeline analysis, contributor activity, code impact metrics,
-   * and work patterns. Useful for task retrospectives and understanding
-   * effort distribution.
-   *
-   * @param taskId - Task identifier (task number like '015' or full ID like 'task-015')
-   * @param query - Optional query parameters
-   * @returns Promise resolving to task-specific statistics
-   *
-   * @example
-   * ```ts
-   * const stats = await metrics.getTaskMetrics('task-015', {});
-   * console.log(`Duration: ${stats.duration} days`);
-   * console.log(`Contributors: ${stats.contributors.map(c => c.name).join(', ')}`);
-   * if (stats.refactoringMetrics) {
-   *   console.log(`Simplification ratio: ${stats.refactoringMetrics.simplificationRatio.toFixed(2)}x`);
-   * }
-   * ```
-   */
-  getTaskMetrics(taskId: string, query?: StatsQuery): Promise<TaskStats>;
+    /**
+     * Get productivity metrics for a specific user.
+     *
+     * Includes code metrics (commits, lines of code), temporal patterns
+     * (day of week, time of day), and engagement statistics (completion rate, streaks).
+     *
+     * @param userId - User identifier (username or registry ID)
+     * @param query - Optional query parameters to filter results
+     * @param query.period - Time period to analyze ('day' | 'week' | 'month' | 'year')
+     * @returns Promise resolving to user statistics
+     *
+     * @example
+     * ```ts
+     * const stats = await metrics.getUserMetrics('alice', { period: 'month' });
+     * console.log(`Tasks completed: ${stats.contributionMetrics.tasksCompleted}`);
+     * console.log(`Completion rate: ${(stats.engagementMetrics.completionRate * 100).toFixed(1)}%`);
+     * ```
+     */
+    getUserMetrics(userId: string, query?: StatsQuery): Promise<UserStats>;
+    /**
+     * Get aggregated metrics for an entire team.
+     *
+     * Provides overview of team productivity with breakdown by individual contributors.
+     * Useful for team retrospectives, capacity planning, and identifying top contributors.
+     *
+     * @param teamId - Team identifier
+     * @param query - Optional query parameters to filter results
+     * @returns Promise resolving to team statistics with per-contributor breakdown
+     *
+     * @example
+     * ```ts
+     * const stats = await metrics.getTeamMetrics('frontend', { period: 'week' });
+     * console.log(`Total commits: ${stats.totalCommits}`);
+     * console.log(`Contributors: ${stats.totalContributors}`);
+     * stats.contributors.forEach(c => {
+     *   console.log(`${c.username}: ${c.commits} commits, ${c.tasksCompleted} tasks`);
+     * });
+     * ```
+     */
+    getTeamMetrics(teamId: string, query?: StatsQuery): Promise<TeamStats>;
+    /**
+     * Get detailed metrics for a specific task.
+     *
+     * Includes timeline analysis, contributor activity, code impact metrics,
+     * and work patterns. Useful for task retrospectives and understanding
+     * effort distribution.
+     *
+     * @param taskId - Task identifier (task number like '015' or full ID like 'task-015')
+     * @param query - Optional query parameters
+     * @returns Promise resolving to task-specific statistics
+     *
+     * @example
+     * ```ts
+     * const stats = await metrics.getTaskMetrics('task-015', {});
+     * console.log(`Duration: ${stats.duration} days`);
+     * console.log(`Contributors: ${stats.contributors.map(c => c.name).join(', ')}`);
+     * if (stats.refactoringMetrics) {
+     *   console.log(`Simplification ratio: ${stats.refactoringMetrics.simplificationRatio.toFixed(2)}x`);
+     * }
+     * ```
+     */
+    getTaskMetrics(taskId: string, query?: StatsQuery): Promise<TaskStats>;
 }
-export type {
-  TaskStats as TaskMetrics,
-  TeamStats as TeamMetrics,
-  UserStats as UserMetrics,
-};
+export type { TaskStats as TaskMetrics, TeamStats as TeamMetrics, UserStats as UserMetrics };

package/dist/provider-stats.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"provider-stats.test.js","sourceRoot":"","sources":["../src/provider-stats.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG9C,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,OAAO,GAAoB;YAC/B,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,CAAQ;YAC7D,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,CAAQ;YAC7D,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,CAAQ;SAC9D,CAAC;QAEF,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAEvD,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;QAClE,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;QAClE,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;IACvE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"provider-stats.test.js","sourceRoot":"","sources":["../src/provider-stats.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG9C,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,OAAO,GAAoB;YAC/B,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CACvC,CAAC,EAAE,MAAM,EAAE,CAAsE;YACnF,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CACvC,CAAC,EAAE,MAAM,EAAE,CAAsE;YACnF,cAAc,EAAE,KAAK,EAAE,MAAc,EAAE,EAAE,CACvC,CAAC,EAAE,MAAM,EAAE,CAAsE;SACpF,CAAC;QAEF,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,CAAC,OAAO,OAAO,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAEvD,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;QAClE,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;QAClE,MAAM,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;IACvE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/task-manager.d.ts

@@ -1,10 +1,26 @@
-import type { CreateTaskOptions, CreateTaskResult, ITaskManager, ITaskProvider, LintResult, TaskFile } from './task-manager.types';
-export declare class TaskManager implements ITaskManager {
+import type { Task, TaskId } from '@opentask/taskin-types';
+import type { CreateTaskOptions, CreateTaskResult, ITaskManager, ITaskProvider, LintResult } from './task-manager.types.js';
+/**
+ * Orchestrates task state transitions on top of any {@link ITaskProvider}.
+ *
+ * `TTask` is inferred from the provider passed to the constructor, so callers
+ * get their provider's task shape back without this class ever naming it.
+ */
+export declare class TaskManager<TTask extends Task = Task> implements ITaskManager<TTask> {
     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>;
+    constructor(taskProvider: ITaskProvider<TTask>);
+    /**
+     * Returns a copy of `task` with a new status.
+     *
+     * TypeScript cannot prove that spreading a generic yields that same generic,
+     * so the assertion is required. It is sound here: `status` is a known key of
+     * `Task`, and every other field is carried over untouched.
+     */
+    private withStatus;
+    startTask(taskId: TaskId): Promise<TTask>;
+    pauseTask(taskId: TaskId): Promise<TTask>;
+    finishTask(taskId: TaskId): Promise<TTask>;
+    reviewTask(taskId: TaskId): Promise<TTask>;
+    createTask(options: CreateTaskOptions): Promise<CreateTaskResult<TTask>>;
     lint(fix?: boolean): Promise<LintResult>;
 }

package/dist/task-manager.js

@@ -1,8 +1,24 @@
+/**
+ * Orchestrates task state transitions on top of any {@link ITaskProvider}.
+ *
+ * `TTask` is inferred from the provider passed to the constructor, so callers
+ * get their provider's task shape back without this class ever naming it.
+ */
 export class TaskManager {
     taskProvider;
     constructor(taskProvider) {
         this.taskProvider = taskProvider;
     }
+    /**
+     * Returns a copy of `task` with a new status.
+     *
+     * TypeScript cannot prove that spreading a generic yields that same generic,
+     * so the assertion is required. It is sound here: `status` is a known key of
+     * `Task`, and every other field is carried over untouched.
+     */
+    withStatus(task, status) {
+        return { ...task, status };
+    }
     async startTask(taskId) {
         const task = await this.taskProvider.findTask(taskId);
         if (!task) {
@@ -14,7 +30,19 @@ export class TaskManager {
         if (task.status === 'done') {
             throw new Error(`Task '${taskId}' is already done.`);
         }
-        const updatedTask = { ...task, status: 'in-progress' };
+        const updatedTask = this.withStatus(task, 'in-progress');
+        await this.taskProvider.updateTask(updatedTask);
+        return updatedTask;
+    }
+    async pauseTask(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 paused. Current status: ${task.status}`);
+        }
+        const updatedTask = this.withStatus(task, 'paused');
         await this.taskProvider.updateTask(updatedTask);
         return updatedTask;
     }
@@ -23,7 +51,7 @@ export class TaskManager {
         if (!task) {
             throw new Error(`Task with ID '${taskId}' not found.`);
         }
-        const updatedTask = { ...task, status: 'done' };
+        const updatedTask = this.withStatus(task, 'done');
         await this.taskProvider.updateTask(updatedTask);
         return updatedTask;
     }
@@ -35,7 +63,7 @@ export class TaskManager {
         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' };
+        const updatedTask = this.withStatus(task, 'in-review');
         await this.taskProvider.updateTask(updatedTask);
         return updatedTask;
     }

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,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"}
+{"version":3,"file":"task-manager.js","sourceRoot":"","sources":["../src/task-manager.ts"],"names":[],"mappings":"AASA;;;;;GAKG;AACH,MAAM,OAAO,WAAW;IACF;IAApB,YAAoB,YAAkC;QAAlC,iBAAY,GAAZ,YAAY,CAAsB;IAAG,CAAC;IAE1D;;;;;;OAMG;IACK,UAAU,CAAC,IAAW,EAAE,MAAkB;QAChD,OAAO,EAAE,GAAG,IAAI,EAAE,MAAM,EAAW,CAAC;IACtC,CAAC;IAED,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,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC;QACzD,MAAM,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAEhD,OAAO,WAAW,CAAC;IACrB,CAAC;IAED,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,mEAAmE,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;QACnH,CAAC;QAED,MAAM,WAAW,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;QACpD,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,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAClD,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,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,WAAW,CAAC,CAAC;QACvD,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.d.ts

@@ -1,5 +1,4 @@
-import type { ITaskProvider, TaskFile } from './task-manager.types';
-export declare const createMockTask: (
-  overrides?: Partial<TaskFile>,
-) => TaskFile;
+import { type Task } from '@opentask/taskin-types';
+import type { ITaskProvider } from './task-manager.types.js';
+export declare const createMockTask: (overrides?: Partial<Task>) => Task;
 export declare const createMockTaskProvider: () => ITaskProvider;

package/dist/task-manager.mock.js

@@ -1,10 +1,9 @@
+import { parseTaskId } from '@opentask/taskin-types';
 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',
+    id: parseTaskId('001'),
     status: 'pending',
     title: 'Implement feature',
     type: 'feat',

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;IACb,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;CACpB,CAAC,CAAC"}
+{"version":3,"file":"task-manager.mock.js","sourceRoot":"","sources":["../src/task-manager.mock.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAa,MAAM,wBAAwB,CAAC;AAChE,OAAO,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG5B,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,SAAyB,EAAQ,EAAE,CAAC,CAAC;IAClE,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;IACnC,WAAW,EAAE,gBAAgB;IAC7B,EAAE,EAAE,WAAW,CAAC,KAAK,CAAC;IACtB,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"}