Skip to content

Commit 764ce00

Browse files
github-managerclaude
andcommitted
fix: NIP-01 compliance - add deterministic event ordering by event_id
For events with the same created_at timestamp, add secondary sort by event_id in ascending order to ensure deterministic ordering as specified by NIP-01. Fixes #477 - non-deterministic event ordering in findByFilters Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
1 parent 7f56d4a commit 764ce00

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/repositories/event-repository.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,9 @@ export class EventRepository implements IEventRepository {
122122
}
123123

124124
if (typeof currentFilter.limit === 'number') {
125-
builder.limit(currentFilter.limit).orderBy('event_created_at', 'DESC')
125+
builder.limit(currentFilter.limit).orderBy('event_created_at', 'DESC').orderBy('event_id', 'asc')
126126
} else {
127-
builder.limit(500).orderBy('event_created_at', 'asc')
127+
builder.limit(500).orderBy('event_created_at', 'asc').orderBy('event_id', 'asc')
128128
}
129129

130130
const andWhereRaw = invoker(1, 'andWhereRaw')

0 commit comments

Comments
 (0)