Restore batch rollback while deleting a move source before ancestry validation

Review the independently proposed patch below, including its unchanged callers. A failed patch exposes earlier writes, and moving a value inside its own subtree fails after removal. Validate ancestry before mutation and return the original document when any operation fails.

Review focus: Repository Implementation

src/controllers/changes.mjs

BeforeProposed

+1−1

1919 if (error instanceof ContractError) return {

2020 ok: false,

2121 error: error.message,

22 document: working

22 document: original

2323 };

2424 throw error;

2525 }

Your review

0 / 10,000 characters · A verdict needs supporting evidence.

30 more characters to give a verdict