I think the comment above the function DecodeInsert() in src/backend/replication/logical/decode.c should be
+ * Inserts can contain the new tuple.
, rather than
- * Deletes can contain the new tuple.
Nice catch. +1.
I kind of wonder if it would be clearer to state that "XLOG_HEAP_INSERT can contain the new tuple", in order to differentiate it from XLOG_HEAP2_MULTI_INSERT.