Removing the check for nDataOut > 1 would allow multiple OP_RETURN outputs in a single transaction, which goes against the standard behavior of the Bitcoin protocol. This could introduce several issues:
Non-standard transactions: Multiple OP_RETURN outputs in a transaction are considered non-standard. Allowing them could lead to network inconsistencies, as some nodes might not be able to properly process or relay these transactions.
Network congestion: More OP_RETURN outputs would increase transaction size, reducing the number of transactions that can be included in each block, which could lead to increased network congestion.
Compatibility problems: Non-standard transactions could cause compatibility issues with other nodes, wallets, and services that expect standard behavior.
Maintaining this check ensures that Bitcoin transactions remain within the protocol’s standards, preventing these potential issues and maintaining network stability.
Originally posted by @mrberlinorg in #32359 (review)
Update:
The core issue isn’t the size itself, but the fact that it allows an unlimited number of OP_RETURN outputs per transaction. The PR author force-resolved my comment, which is shocking — that a single individual, just a contributor, can unilaterally modify such a fundamental Bitcoin rule. Now is the time to push for true decentralization in Bitcoin development itself.