@gitgov/core 1.1.0 → 1.3.0

package/README.md

@@ -10,14 +10,20 @@
 
 This example shows how to create a new task using the `BacklogAdapter`. The SDK uses dependency injection - each adapter requires its dependencies to be explicitly provided.
 
-*Note: This example assumes it is run inside an initialized GitGovernance project.*
+_Note: This example assumes it is run inside an initialized GitGovernance project._
 
 ```typescript
 import { Adapters, Store, EventBus } from "@gitgov/core";
-import type { TaskRecord, CycleRecord, ActorRecord, AgentRecord } from "@gitgov/core";
+import type {
+  TaskRecord,
+  CycleRecord,
+  ActorRecord,
+  AgentRecord,
+} from "@gitgov/core";
 
 // Extract classes from namespaces
-const { IdentityAdapter, BacklogAdapter, WorkflowMethodologyAdapter } = Adapters;
+const { IdentityAdapter, BacklogAdapter, WorkflowMethodologyAdapter } =
+  Adapters;
 const { RecordStore } = Store;
 const { EventBus: EventBusClass } = EventBus;
 
@@ -97,11 +103,11 @@ console.log({
 ### Adapter Ecosystem (9/9 Adapters)
 
 - **ProjectAdapter**: Project initialization engine with 3-adapter orchestration (18 tests)
-- **BacklogAdapter**: Task and cycle lifecycle management with workflow validation (43 tests)
+- **BacklogAdapter**: Task and cycle lifecycle management with workflow validation (71 tests)
 - **MetricsAdapter**: Pure calculation engine for system analytics (32 tests)
 - **ChangelogAdapter**: System historian for change documentation (31 tests)
 - **ExecutionAdapter**: Audit log for work execution (13 tests)
-- **FeedbackAdapter**: Structured communication and blocking management (15 tests)
+- **FeedbackAdapter**: Structured communication and blocking management (21 tests)
 - **IdentityAdapter**: Cryptographic identity and agent management (25 tests)
 - **WorkflowMethodologyAdapter**: Configurable workflow validation engine (51 tests)
 - **IndexerAdapter**: Local cache optimization for performance (5 tests)
@@ -229,7 +235,7 @@ const generator = new DiagramGenerator();
 // Generate Mermaid diagrams from GitGovernance records
 const result = await generator.generateFromRecords(cycles, tasks, {
   cycleId: "1704067200-cycle-identity-adapter",
-  layout: 'TD',
+  layout: "TD",
   showWarnings: true,
 });
 
@@ -240,7 +246,7 @@ console.log(result.mermaidCode);
 ## 🧪 Testing & Development
 
 ```bash
-# Run all tests (704 tests total)
+# Run all tests (737 tests total)
 npm test
 npm run test:coverage    # Run tests with coverage report
 
@@ -266,13 +272,13 @@ npm test -- --watch
 
 ### Test Coverage
 
-- **704 tests total** with EARS methodology
+- **737 tests total** with EARS methodology
 - **ProjectAdapter**: 18 tests (project initialization + template processing + error recovery)
-- **BacklogAdapter**: 43 tests (workflow lifecycle + event handlers + E2E simulation)
+- **BacklogAdapter**: 71 tests (workflow lifecycle + event handlers + E2E simulation + deduplication)
 - **MetricsAdapter**: 32 tests (Tier 1+2 calculations + performance validation)
 - **ChangelogAdapter**: 31 tests (multi-entity changelog + conditional validation)
 - **EventBusModule**: 32 tests (20 unit + 12 integration tests with cross-adapter scenarios)
-- **FeedbackAdapter**: 15 tests (EARS coverage with dual event emission)
+- **FeedbackAdapter**: 21 tests (EARS coverage with dual event emission + duplicate assignment prevention)
 - **ExecutionAdapter**: 13 tests (EARS coverage with performance validation)
 - **WorkflowMethodologyAdapter**: 51 tests (29 unit + 22 integration tests)
 - **Identity Domain**: 66 tests (Adapter + ActorRecord/AgentRecord factories & validators)

package/dist/src/index.d.ts

@@ -1766,6 +1766,7 @@ interface IBacklogAdapter {
     pauseTask(taskId: string, actorId: string, reason?: string): Promise<TaskRecord>;
     resumeTask(taskId: string, actorId: string, force?: boolean): Promise<TaskRecord>;
     discardTask(taskId: string, actorId: string, reason?: string): Promise<TaskRecord>;
+    deleteTask(taskId: string, actorId: string): Promise<void>;
     createCycle(payload: Partial<CycleRecord>, actorId: string): Promise<CycleRecord>;
     getCycle(cycleId: string): Promise<CycleRecord | null>;
     getAllCycles(): Promise<CycleRecord[]>;
@@ -1865,6 +1866,11 @@ declare class BacklogAdapter implements IBacklogAdapter {
      * Supports both cancellation (ready/active) and rejection (review) operations
      */
     discardTask(taskId: string, actorId: string, reason?: string): Promise<TaskRecord>;
+    /**
+     * Deletes a draft task completely (no discarded state)
+     * Only works for tasks in 'draft' status that never entered formal workflow
+     */
+    deleteTask(taskId: string, actorId: string): Promise<void>;
     /**
      * Updates a task with new payload
      */

package/dist/src/index.js

@@ -2878,6 +2878,15 @@ var FeedbackAdapter = class {
     if (payloadWithEntityId.entityType && !["task", "execution", "changelog", "feedback"].includes(payloadWithEntityId.entityType)) {
       throw new Error("InvalidEntityTypeError: entityType must be task, execution, changelog, or feedback");
     }
+    if (payload.type === "assignment" && payload.assignee) {
+      const existingFeedbacks = await this.getFeedbackByEntity(payloadWithEntityId.entityId);
+      const duplicateAssignment = existingFeedbacks.find(
+        (feedback) => feedback.type === "assignment" && feedback.assignee === payload.assignee && feedback.status === "open"
+      );
+      if (duplicateAssignment) {
+        throw new Error(`DuplicateAssignmentError: Task ${payloadWithEntityId.entityId} is already assigned to ${payload.assignee} (feedback: ${duplicateAssignment.id})`);
+      }
+    }
     const enrichedPayload = {
       ...payload,
       status: "open"
@@ -4383,6 +4392,9 @@ var BacklogAdapter = class {
     const task = taskRecord.payload;
     const actor = await this.getActor(actorId);
     if (!["ready", "active", "review"].includes(task.status)) {
+      if (task.status === "draft") {
+        throw new Error(`ProtocolViolationError: Cannot cancel task in 'draft' state. Use 'gitgov task delete ${taskId}' to remove draft tasks.`);
+      }
       throw new Error(`ProtocolViolationError: Task is in '${task.status}' state. Cannot cancel from this state. Only 'ready', 'active', and 'review' tasks can be cancelled.`);
     }
     const context = {
@@ -4421,6 +4433,39 @@ ${task.status === "review" ? "[REJECTED]" : "[CANCELLED]"} ${reason} (${(/* @__P
     });
     return updatedPayload;
   }
+  /**
+   * Deletes a draft task completely (no discarded state)
+   * Only works for tasks in 'draft' status that never entered formal workflow
+   */
+  async deleteTask(taskId, actorId) {
+    const taskRecord = await this.taskStore.read(taskId);
+    if (!taskRecord) {
+      throw new Error(`RecordNotFoundError: Task not found: ${taskId}`);
+    }
+    const task = taskRecord.payload;
+    if (task.status !== "draft") {
+      if (task.status === "review") {
+        throw new Error(`ProtocolViolationError: Cannot delete task in 'review' state. Use 'gitgov task reject ${taskId}' to discard tasks under review.`);
+      } else if (task.status === "ready" || task.status === "active") {
+        throw new Error(`ProtocolViolationError: Cannot delete task in '${task.status}' state. Use 'gitgov task cancel ${taskId}' to discard tasks from ready/active states.`);
+      }
+      throw new Error(`ProtocolViolationError: Cannot delete task in '${task.status}' state. Only draft tasks can be deleted.`);
+    }
+    await this.getActor(actorId);
+    await this.taskStore.delete(taskId);
+    this.eventBus.publish({
+      type: "task.status.changed",
+      timestamp: Date.now(),
+      source: "backlog_adapter",
+      payload: {
+        taskId,
+        oldStatus: "draft",
+        newStatus: "deleted",
+        actorId,
+        reason: "Draft task deleted"
+      }
+    });
+  }
   /**
    * Updates a task with new payload
    */
@@ -4450,8 +4495,9 @@ ${task.status === "review" ? "[REJECTED]" : "[CANCELLED]"} ${reason} (${(/* @__P
         assignedTaskIds.push(record.payload.entityId);
       }
     }
+    const uniqueTaskIds = [...new Set(assignedTaskIds)];
     const assignedTasks = [];
-    for (const taskId of assignedTaskIds) {
+    for (const taskId of uniqueTaskIds) {
       const task = await this.getTask(taskId);
       if (task) {
         assignedTasks.push(task);