Hearthpwn

Starting with the basic set, I think a few of the cards are too complicated. The "If you have no empty Mana Crystals" mechanic is sort of strange and took me a while to understand, and I don't really see why it's there as none of the other cards seem to have effects or anything to synergize with it (cards like Refill your mana crystals, or the first card you play costs less). I'd say that the best solution here is to maybe rework those two cards into something really simple, honestly I think a 1 mana 1/3 with Spell Damage +1 and no condition is a fine card - it's simple, it's powerful, it's versatile, it fits the class, without being gamebreakingly overpowered. Not sure what you'd want to do with enchantress. While having one 4-line card might be okay, I think having both Selfish Guardian and Armored Igniter is a bit too much. I don't really see the flavor connection of selfish guardian to this class, and Armored Igniter is really just a Faerie Dragon that's immune to AoE - an effect that I don't really think is worth the extra lines of card text. I think you should consider changes to both of them, but Selfish Guardian needs more work, as Armored Igniter is alright. 

Second, I have to point out that your basic set has too many minions, although I understand why, since I'm assuming that your Spellcraft mechanic is worded like "Your next Basic spell functions differently," so you can only really have basic spells that interact with that. What I would do is to change the wording of the mechanic to "Your next Incantation functions differently" and then adding the "Incantation" tribe to all the spells that are applicable (with whatever word you want in place of Incantation). That way you can maybe trim your minions down to 2 or 3 and introduce some new spells into the basic set without worrying about Spellcraft confusion.

As for the spellcraft base spells themselves, I really only have some minor gripes. I think Shining Light should just be called "Light" for consistency with the other one-word names, and the wording of the effect is extremely confusing for new players as well as probably too powerful with the AoE spellcrafts. I would just make it "Deal 2 damage. If this kills a minion, draw a card." I think shadow could also use a nudge to -4/-4 for the aesthetics, as well as the fact that I think an extra 1 attack doesn't make it much stronger and it could really use the buff compared to Heat. Other than that I really like the basic spells. 

Overall the classic set is really nice, and as I said before spellcraft is a really cool mechanic. I'd say the only problem is that aside from spellcast your themes are rather weak. Spell Damage is a nice theme and you should do something with that, but splitting damage among friendly characters and having no unspent mana seem kinda meh. Maybe if you had a better way of stating the "no empty mana" effect, and you had more cards to synergize it (like mana refreshing/refilling) it could work as a mechanic, but I really don't like splitting among friendly characters. Another concern is that Hammer feels like it's strictly better than knife, and maybe too good in and of itself (none of the basic spells seem like they actually do anything nuanced when they activate twice as opposed to double damage, except in edge cases) so I would consider a rework. Maybe one that returns a copy of the spell you cast to your hand so that you can use it later? Not sure.

One thing - Ventriloquism is basically treason to hearthstone design and it breaks a LOT of design principles by allowing you to attack your opponent's deck and hand. I think it's impossible to balance and you should just stay away from that type of thing.

The legendary is kinda meh, but that's alright since legendaries are by far the hardest thing to design. I'd say it's good enough, gives the class a finisher, but I think it already has a finisher with heat + hammer/knife and stuff like that so I'm not sure how necessary it is.

In the GvG set, having two cards which effectively do the same thing is rather lazy design. I'd try to push the 1-cost spell theme or "whenever you cast a spell" synergy as that is the biggest intersection between this class and GvG flavor. I don't really understand why puppetry would ever be able to target a hero, though, but I think that having these tokens with a ton of different effects is a huge burden of knowledge for a non-legendary card.

The legendary seems kinda cool but I don't get how it fits into your class's themes. It also seems like it could use some tuning, as what's the point of attacking with puppets if they have the same attack as her?

Overall, good job. I think this class is like a diamond in the rough. Spellcraft is a really neat mechanic, although it's one that's very hard to implement. I'd say your main goal right now is to help your class fit better in to hearthstone - more simple cards, more consistent themes, more spells in the basic set. Your class is like a less extreme version of the Pokemon Trainer that was competing with my Bard, which ultimately failed because it functioned so radically differently from any other class that it didn't feel like it fit into hearthstone.

Also - I think you could have gotten away with using the same art for each augmented card, and you probably didn't *have* to make all of those tokens, but damn, you did a good job.