BugIndicatingErrorRecovery: Transaction already finished!

[vs-code-engineering]

Error: BugIndicatingErrorRecovery: Transaction already finished!
at dSt in ./file:/Users/cloudtest/vss/_work/1/s/src/vs/base/common/observableInternal/base.ts:448:14
at handleBugIndicatingErrorRecovery in ./file:/Users/cloudtest/vss/_work/1/s/src/vs/base/common/observableInternal/base.ts:410:4
at yW.set in ./file:/Users/cloudtest/vss/_work/1/s/src/vs/base/common/observableInternal/base.ts:515:8
at $tt.accept in vs/workbench/contrib/chat/browser/chatEditing/chatEditingModifiedFileEntry.ts:213:18

Go to Errors Site

[hediet]

I think we should try to avoid async transactions (unless there is really need for it), and instead start a new transaction after every await.

At least, there should be IAsyncTransaction, so that it is not possible to pass a sync transaction into a method which uses the transaction after an await.