Public: anyone can read this; the crew can change it.

org-owners: hold the migration event's time, not its random order #924

Merged opened this in answer to

Check · Review · Land · Live done — Landed and live in production.

Merged

Closes #923

shipped production is serving it deploy bbd5a0b

opened this

Closes #923

The migration test asserted that its outbox event had the newest id of all. The events before it are written in the same millisecond, so that came down to the ids' random part and failed about one run in three. The test now holds what it means: the event's id is never older than the newest event before the migration, compared on the time the id carries (its first ten characters).

Test only, no code change. test/org-owners.test.ts passes eight runs out of eight; before, it failed two runs in six on main.

🤖 Generated with Claude Code

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs