fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error#533

Open

smarcet wants to merge 28 commits into

mainfrom

hotfix/doctrine-tx-manager

Commits on Jul 10, 2026

fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error
smarcet
committed
fix(transactions): harden root/nested split against phantom writes and masked errors
smarcet
committed
fix(tx): remove DBAL savepoints, propagate native isRollbackOnly guard
smarcet
committed
test: nested-transaction rollback coverage for SummitOrderService, SummitService, SpeakerService, PresentationService, SummitPromoCodeService
smarcet
committed
test: HTTP-level exception-branch coverage for order/attendee ticket creation
smarcet
committed
docs(adr): nested transaction rollback safety decision record
smarcet
committed
test: nested-transaction rollback coverage for remaining ADR-003 gaps
smarcet
committed
docs(adr): update nested-tx rollback ADR with remaining test coverage
smarcet
committed
docs(adr): remove docs/plans references from ADR-003
smarcet
committed
chore: add nested-tx rollback test classes to push.yml CI matrix
smarcet
committed
docs(adr): clarify origin/main's actual pre-branch behavior in ADR-003
smarcet
committed
fix(transactions): never retry ambiguous commits, refuse closed-EM re-entry
smarcet
committed
fix(transactions): discard broken manager/connection when rollback fails
smarcet
committed
refactor(transactions): deduplicate failure-cleanup paths in DoctrineTransactionService
smarcet
committed
fix(transactions): surface ambiguous commit failures as AmbiguousCommitException
smarcet
committed

Commits on Jul 11, 2026