found the cause of the loop in inkmoth, it has to do with the way manaproducers set thier "needsTapping" depending on the way the ability is called, it can set it 3 ways, {t}<--sets only the manaproducers "tap" and completely ignores the "doTap" which is the way actiavted mana producers call it.

projects/mtg/src/AIPlayer.cpp

@@ -124,7 +124,7 @@ bool AIPlayer::tapLandsForMana(ManaCost * cost, MTGCardInstance * target)
                         break;
                     }
                 }
-                if (doTap)
+                if (doTap || amp->tap)
                 {
                     AIAction * action = NEW AIAction(amp, card);
                     clickstream.push(action);