diff --git a/CHANGELOG.md b/CHANGELOG.md index 89d67e2c3..6f7178aaf 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,13 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). ### Added +- **Data Integrity Fix: ValidationAttachmentRepository Argument Swap** (#7492): + Fixed a critical data integrity issue in `ValidationAttachmentRepository.attach` + where arguments were being silently swapped during method invocation. The incorrect + argument order caused data to be stored with reversed parameters, leading to subtle + but serious data corruption without any error being raised. Arguments are now passed + in the correct sequence, ensuring data is stored with proper parameter values. + - **Git Worktree Sandbox Apply** (#4454): The `plan apply` command now merges LLM-generated changes via `git merge` from an isolated worktree branch instead of flat `shutil.copy2`. Displays spec-aligned Apply Summary diff --git a/features/steps/tdd_issue_7492_validation_attachment_argument_swap_steps.py b/features/steps/tdd_issue_7492_validation_attachment_argument_swap_steps.py index 8bbfc8e6b..73ef411ad 100644 --- a/features/steps/tdd_issue_7492_validation_attachment_argument_swap_steps.py +++ b/features/steps/tdd_issue_7492_validation_attachment_argument_swap_steps.py @@ -7,7 +7,6 @@ swap validation_name and resource_id when resource_id contains a slash. from __future__ import annotations import json -from typing import Any from behave import given, then, when from behave.runner import Context @@ -18,7 +17,6 @@ from cleveragents.infrastructure.database.models import Base from cleveragents.infrastructure.database.repositories import ( ValidationAttachmentRepository, ) -from cleveragents.core.exceptions import BusinessRuleViolation # ---------------------------------------------------------------------------