My big concern is the small change in behavior related to when the query execution code throws an exception. The sync code resolves the exception from the promise and re-throws it. lacinia-pedestal will need a change to do something similar, as might your code.
Might be interesting to see what it does to performance. Of there is a lacinia-pedestal with the changes I could put it through a subscription performance test?