Fix rollback exception propagating

lib/active_graph/transaction.rb

@@ -12,6 +12,10 @@ def apply_callbacks
       after_commit_registry.each(&:call)
     end
 
+    def clear_callbacks
+      after_commit_registry.clear
+    end
+
     private
 
     def after_commit_registry

lib/active_graph/transactions.rb

@@ -35,11 +35,13 @@ def lock_node(node)
       private
 
       def send_transaction(method, **config, &block)
-        return yield tx if tx
-        return run_transaction_work(explicit_session, method, **config, &block) if explicit_session&.open?
-        driver.session do |session|
-          run_transaction_work(session, method, **config, &block)
-        end
+        return run_transaction_work(explicit_session, method, **config, &block) if !tx && explicit_session&.open?
+        return driver.session { |session| run_transaction_work(session, method, **config, &block) } unless tx
+
+        yield tx
+      rescue ActiveGraph::Rollback
+        tx.clear_callbacks
+        false
       end
 
       def run_transaction_work(session, method, **config, &block)

spec/active_graph/transactions_spec.rb

@@ -122,4 +122,22 @@
       end
     end
   end
+
+  describe 'rollback in nested transaction' do
+    it 'allows reading model errors after failed update inside an outer transaction' do
+      stub_node_class('ValidatedStudent') do
+        property :name
+        validates :name, presence: true
+      end
+
+      student = ValidatedStudent.create!(name: 'Alice')
+
+      errors = ActiveGraph::Base.transaction do
+        student.update(name: nil)
+        student.errors.full_messages
+      end
+
+      expect(errors).to include("Name can't be blank")
+    end
+  end
 end