As the coinsSent signal is currently only connected to fetchPaymentACK I thought it would be nice to only emit it for payment-requests. You are right that nothing bad is happening currently in fetchPaymentACK for non payment-requests, but who knows ^^.