This removes a circular dependency caused by historical reasons
> <sipa> GetTransaction predates node/transaction.cpp, and even the generic index framework itself
And also documentation follow-ups to D12208
This is a backport of [[https://github.com/bitcoin/bitcoin/pull/22528 | core#22528]]