Protokoll #16270

{
    "event": "PreToolUse",
    "tool_name": "Edit",
    "tool_input": {
        "file_path": "\/var\/www\/dev.campus.systemische-tools.de\/src\/UseCases\/Task\/UpdateTaskStatusUseCase.php",
        "old_string": "    public function execute(int $taskId, string $newStatus, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        $task = $this->taskRepository->find($taskId);\n        if ($task === null) {\n            throw new \\InvalidArgumentException(\"Task {$taskId} not found\");\n        }\n\n        $newStatusEnum = TaskStatus::from($newStatus);\n        $oldStatus = $task->getStatus();\n\n        if (!$oldStatus->canTransitionTo($newStatusEnum)) {\n            throw new \\InvalidArgumentException(\n                \"Cannot transition from {$oldStatus->value} to {$newStatus}\"\n            );\n        }\n\n        $task->setStatus($newStatusEnum);\n        $this->taskRepository->update($task);\n\n        $comment = TaskComment::createStatusChange(\n            $taskId,\n            $updatedBy,\n            $updatedByType,\n            $oldStatus->label(),\n            $newStatusEnum->label()\n        );\n        $this->commentRepository->save($comment);\n\n        return $task;\n    }",
        "new_string": "    public function execute(int $taskId, string $newStatus, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        $task = $this->taskRepository->find($taskId);\n        if ($task === null) {\n            throw new \\InvalidArgumentException(\"Task {$taskId} not found\");\n        }\n\n        $oldStatus = $task->getStatus();\n\n        match ($newStatus) {\n            'in_progress' => $task->start(),\n            'completed' => $task->complete(),\n            'failed' => $task->fail(),\n            'cancelled' => $task->cancel(),\n            'pending' => $task->retry(),\n            default => throw new \\InvalidArgumentException(\"Invalid status: {$newStatus}\"),\n        };\n\n        $this->taskRepository->update($task);\n\n        $comment = TaskComment::createStatusChange(\n            $taskId,\n            $updatedBy,\n            $updatedByType,\n            $oldStatus->label(),\n            $task->getStatus()->label()\n        );\n        $this->commentRepository->save($comment);\n\n        return $task;\n    }"
    }
}

{
    "tool_response": {
        "filePath": "\/var\/www\/dev.campus.systemische-tools.de\/src\/UseCases\/Task\/UpdateTaskStatusUseCase.php",
        "oldString": "    public function execute(int $taskId, string $newStatus, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        $task = $this->taskRepository->find($taskId);\n        if ($task === null) {\n            throw new \\InvalidArgumentException(\"Task {$taskId} not found\");\n        }\n\n        $newStatusEnum = TaskStatus::from($newStatus);\n        $oldStatus = $task->getStatus();\n\n        if (!$oldStatus->canTransitionTo($newStatusEnum)) {\n            throw new \\InvalidArgumentException(\n                \"Cannot transition from {$oldStatus->value} to {$newStatus}\"\n            );\n        }\n\n        $task->setStatus($newStatusEnum);\n        $this->taskRepository->update($task);\n\n        $comment = TaskComment::createStatusChange(\n            $taskId,\n            $updatedBy,\n            $updatedByType,\n            $oldStatus->label(),\n            $newStatusEnum->label()\n        );\n        $this->commentRepository->save($comment);\n\n        return $task;\n    }",
        "newString": "    public function execute(int $taskId, string $newStatus, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        $task = $this->taskRepository->find($taskId);\n        if ($task === null) {\n            throw new \\InvalidArgumentException(\"Task {$taskId} not found\");\n        }\n\n        $oldStatus = $task->getStatus();\n\n        match ($newStatus) {\n            'in_progress' => $task->start(),\n            'completed' => $task->complete(),\n            'failed' => $task->fail(),\n            'cancelled' => $task->cancel(),\n            'pending' => $task->retry(),\n            default => throw new \\InvalidArgumentException(\"Invalid status: {$newStatus}\"),\n        };\n\n        $this->taskRepository->update($task);\n\n        $comment = TaskComment::createStatusChange(\n            $taskId,\n            $updatedBy,\n            $updatedByType,\n            $oldStatus->label(),\n            $task->getStatus()->label()\n        );\n        $this->commentRepository->save($comment);\n\n        return $task;\n    }",
        "originalFile": "<?php\n\ndeclare(strict_types=1);\n\nnamespace UseCases\\Task;\n\n\/\/ @responsibility: Aktualisiert Task-Status mit Workflow-Validierung\n\nuse Domain\\Entity\\Task;\nuse Domain\\Entity\\TaskComment;\nuse Domain\\Repository\\TaskRepositoryInterface;\nuse Domain\\ValueObject\\TaskStatus;\nuse Infrastructure\\Persistence\\TaskCommentRepository;\n\nclass UpdateTaskStatusUseCase\n{\n    public function __construct(\n        private TaskRepositoryInterface $taskRepository,\n        private TaskCommentRepository $commentRepository\n    ) {\n    }\n\n    public function execute(int $taskId, string $newStatus, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        $task = $this->taskRepository->find($taskId);\n        if ($task === null) {\n            throw new \\InvalidArgumentException(\"Task {$taskId} not found\");\n        }\n\n        $newStatusEnum = TaskStatus::from($newStatus);\n        $oldStatus = $task->getStatus();\n\n        if (!$oldStatus->canTransitionTo($newStatusEnum)) {\n            throw new \\InvalidArgumentException(\n                \"Cannot transition from {$oldStatus->value} to {$newStatus}\"\n            );\n        }\n\n        $task->setStatus($newStatusEnum);\n        $this->taskRepository->update($task);\n\n        $comment = TaskComment::createStatusChange(\n            $taskId,\n            $updatedBy,\n            $updatedByType,\n            $oldStatus->label(),\n            $newStatusEnum->label()\n        );\n        $this->commentRepository->save($comment);\n\n        return $task;\n    }\n\n    public function startTask(int $taskId, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        return $this->execute($taskId, 'in_progress', $updatedBy, $updatedByType);\n    }\n\n    public function completeTask(int $taskId, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        return $this->execute($taskId, 'completed', $updatedBy, $updatedByType);\n    }\n\n    public function failTask(int $taskId, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        return $this->execute($taskId, 'failed', $updatedBy, $updatedByType);\n    }\n\n    public function cancelTask(int $taskId, string $updatedBy, string $updatedByType = 'human'): Task\n    {\n        return $this->execute($taskId, 'cancelled', $updatedBy, $updatedByType);\n    }\n}\n",
        "structuredPatch": [
            {
                "oldStart": 27,
                "oldLines": 16,
                "newStart": 27,
                "newLines": 17,
                "lines": [
                    "             throw new \\InvalidArgumentException(\"Task {$taskId} not found\");",
                    "         }",
                    " ",
                    "-        $newStatusEnum = TaskStatus::from($newStatus);",
                    "         $oldStatus = $task->getStatus();",
                    " ",
                    "-        if (!$oldStatus->canTransitionTo($newStatusEnum)) {",
                    "-            throw new \\InvalidArgumentException(",
                    "-                \"Cannot transition from {$oldStatus->value} to {$newStatus}\"",
                    "-            );",
                    "-        }",
                    "+        match ($newStatus) {",
                    "+            'in_progress' => $task->start(),",
                    "+            'completed' => $task->complete(),",
                    "+            'failed' => $task->fail(),",
                    "+            'cancelled' => $task->cancel(),",
                    "+            'pending' => $task->retry(),",
                    "+            default => throw new \\InvalidArgumentException(\"Invalid status: {$newStatus}\"),",
                    "+        };",
                    " ",
                    "-        $task->setStatus($newStatusEnum);",
                    "         $this->taskRepository->update($task);",
                    " ",
                    "         $comment = TaskComment::createStatusChange("
                ]
            },
            {
                "oldStart": 44,
                "oldLines": 7,
                "newStart": 45,
                "newLines": 7,
                "lines": [
                    "             $updatedBy,",
                    "             $updatedByType,",
                    "             $oldStatus->label(),",
                    "-            $newStatusEnum->label()",
                    "+            $task->getStatus()->label()",
                    "         );",
                    "         $this->commentRepository->save($comment);",
                    " "
                ]
            }
        ],
        "userModified": false,
        "replaceAll": false
    }
}