I personally prefer to enter addresses when needed, rather than risking getting an xpub compromised, due to the related privacy and security issues.

Also, I have to agree with comments in that discussion which point out that address reuse can still happen if you start entering xpubs in different Bitcoin applications. Even checking addresses for balances does not exclude collisions, due to the 'TOCTOU' nature of this idea combined with asynchronous Bitcoin transactions; only emphasized by a full mempool with long confirmation times.

It's also correct that mediation and support in general will only get worse if people use all sorts of different wallets. By keeping the trade from beginning to end inside Bisq, the team should have a much easier time assisting in case of problems.

That reads really nice, indeed! Much better than before; everything in one file, nice and concise and with one less dependency. I will try it out myself and also use it to debug my own script.
Amazing to see that it was also updated on WalletScrutiny now, that's a pleasure to see.