Skip to content

feat(curation): merge event auditability - reason field + note merge event#969

Draft
ohdearquant wants to merge 7 commits into
mainfrom
fix-809-merge-events
Draft

feat(curation): merge event auditability - reason field + note merge event#969
ohdearquant wants to merge 7 commits into
mainfrom
fix-809-merge-events

Conversation

@ohdearquant

Copy link
Copy Markdown
Owner

Fixes #809. Additive payload changes only. Local gates deferred to CI due to build-lane contention; codex review follows.

Leo and others added 7 commits July 13, 2026 12:32
…event

EntityMerged carried no reason and merge_note emitted no event at all,
leaving curation actions unauditable. Adds an optional reason threaded
into the EntityMerged payload (absent when not supplied) and a merge
event emitted from merge_note with the same payload discipline.

Fixes #809

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ky tx-age sweep

run_writer_task registers a writer_task_tx handle in the process-wide
tx_registry singleton; the checkpoint tx_age_sweep_* tests read that
singleton under #[serial(tx_registry)]. The writer_task.rs tests that
spawn a writer task were missing from the group, leaking a longer-lived
writer_task_tx into the sweep's oldest() read and intermittently failing
the assertion on main CI. Join them to the serial group. No prod change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

curation: EntityMerged has no reason field; merge_note emits NO event at all

1 participant