I'm impressed by your capacity to keep finding new uses for Trade tokens!
Slightly off-topic: what is the current wording of General? I have been thinking that the following wording is a bit simpler, although it works in a different way:
General
Types: Action
Cost: $5
You may play an Action card from your hand twice; when you would remove that card from play, you may put it on top of your deck instead.
I can't think of any one-shot that would break this, and it's shorter.
EDIT: Yeah, technically you can choose to put that card on top of your deck instead of putting that card on top of your deck. But you'll probably get slapped it you do that too many times.
General was that for a while. The problem was that it was crazy confusing with Gambler, Mining Village, etc. You'd play General, using it to play Mining Village, and then trash the Mining Village for $2, which you could put on your deck. Then you'd play the Mining Village a second time, whereupon it would draw
itself. Then of course you could play it a third time, possibly trashing it for another $2. The tracking was just a nightmare. Hence the updated version that prevented the card from leaving play. Now of course it has neither.
I could try going back to the previous General, which was this:
General
Types: Action
Cost: $5
Take a Trade token. You may play an Action card from your hand. You may pay a Trade token to play it again. You may pay another Trade token to play it a third time.
That version also got cut for being too confusing, but I don't know if I gave it enough of a chance.
EDIT: Oh, right, you asked about the current wording of General. Right now it's just:
General
Types: Action
Cost: $5
You may play an Action card from your hand twice. When you discard it from play, you may put it on top of your deck.