If I understand this correctly you name e.g. Copper in a 3P game and then 2 Coppers from the Supply land in the trash. In this case you don't need the revealing of hands, it just slows down games.
I fear that this could lead to too much pile control, even more than Lurker, cost reducers and Salt. On the other hand it is not like the opponents don't see it coming that you go for Brothel so if they try to win conventionally via Provinces they can still react to you trashing Provinces and gaining VPs via Brothel. I'd get rid of the Debt though, it just incentives you, as Asper has pointed out, even more to "go all in", i.e. to only win via VPs. Arguably this problem is even more pronounced with Brothel 2.0 as you now have pile control.
I'd test something simple, like the first version but without Debt. There is the issue though of the infinite loop that Asper mentioned, a deck than can generate 4VPs every turn and potentially more if it is supported by draw. So you could simply make Brothel non-terminal (and provide a Coin to compensate and make it closer to Asper's Hotell which is again closer to Monument) to make this harder. You'd just need a Village to then play your 5 Brothels but it nonetheless makes it trickier.
While I also think that it isn't strictly wrong to do non-terminal Cultist versions (+1 Card +1 Action |You may play a Weak Village from your hand. would e.g. be a perfectly fine $2 Village) I also think that a card that does something problematic like gaining VP tokens should be careful about such things.