Can we make her an unemployed human instead of a shaman?

Shamans don't make much sense as industrialists.

Sure

That's a very risqué name.

Should we add a to the cost of the ability? Most other black cards that generate mana like this generally require some mana to be put into it. (eg. Bog Initiate, Blood Celebrant, etc...)

­Vesper Ghoul and Deathrite Shaman are both more recent examples that say you don't have to pay mana to get mana in black. Paying life seems fine, though since you're getting two mana out of it, you may want a mana investment.

Also, the untap ability works well for getting lands into your graveyard, but makes an otherwise simple common much more complex.

You're right we could probably get away with dropping the untap clause.

Should this maybe just blow up lands always? Like, sac a land, tap, add 2? Or is that too much for common?

I would add mana to the activation if you made it sacrificing lands, as you could essentially generate three mana per land, which is a bit much for a common. That's fine as a one-shot effect like Strike it Rich, but repeating it every turn would be nuts.

Oh, also, I approve of making this just sac lands for . That's a much cleaner card.

I'd rename it to "Iron City Industrialist", referencing "Brass City Aristocrat".

I think it would be a cool black card just with pay 1 life get 2 mana. The second ability makes the card it's own combo. I think you shoul make a second card to work with it insstead rather then giving it all the power.

+1 for "Iron City Industrialist"

+1 to rename.

I think this might be a little too complicated for a common. Removing the untap part would go a long way towards making this a better common, even if it would make the card worse.