lentil/rhizome-node
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update comment about LastWriteWins tie-breaking algorithm

Browse Source 
Addresses PR feedback about the outdated comment in concurrent-writes.ts. The comment now accurately reflects that the resolution uses the LastWriteWins resolver's tie-breaking algorithm rather than delta processing order.
This commit is contained in:
Lentil Hoffman 2025-06-15 12:13:55 -05:00
parent 91514289ae
commit 08fb5778ba
Signed by: lentil
GPG Key ID: 0F5B99F3F4D0C087
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
__tests__/concurrent-writes.ts
View File

@ -58,7 +58,7 @@ describe('Concurrent Write Scenarios', () => {
const result = resolver.resolve(); const result = resolver.resolve();
expect(result).toBeDefined(); expect(result).toBeDefined();
// Should resolve deterministically (likely based on delta processing order) // Should resolve deterministically using the LastWriteWins resolver's tie-breaking algorithm
expect(typeof result!['entity1'].properties.score).toBe('number'); expect(typeof result!['entity1'].properties.score).toBe('number');
expect([100, 200]).toContain(result!['entity1'].properties.score); expect([100, 200]).toContain(result!['entity1'].properties.score);
}); });