Skip to content

Make speaker profile merges atomic on current main#1571

Merged
r3dbars merged 4 commits into
mainfrom
codex/speaker-merge-current-main-v3
Jul 18, 2026
Merged

Make speaker profile merges atomic on current main#1571
r3dbars merged 4 commits into
mainfrom
codex/speaker-merge-current-main-v3

Conversation

@r3dbars

@r3dbars r3dbars commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

Fresh current-main replacement for #1546/#1566/#1567 after #1564 advanced main.

  • keeps current speaker finalization and transcript-owner behavior
  • makes profile merge and transcript rewrite rollback-safe
  • propagates persistence failures from the naming mutation batch
  • preserves the existing speaker side-effect coordinator and tests

Proof

  • source-list validation and git diff --check
  • 116 focused speaker tests on the reviewed predecessor
  • Claude advisory review found no P1 blocker
  • fresh current-main hosted proof is required before merge

Manual boundary

Settings speaker merge remains manual QA and is not claimed.

Sole serialized merger: Codex final reviewer/merger; Claude advisory risk review; Local unavailable; Windows skipped.

@r3dbars
r3dbars merged commit e1f7440 into main Jul 18, 2026
6 checks passed
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.

1 participant